Skip to content
This repository has been archived by the owner on Jan 19, 2024. It is now read-only.

feat: Replace polling for jobs with watch #294

Closed
wants to merge 2 commits into from

Conversation

Raffy23
Copy link
Member

@Raffy23 Raffy23 commented Jun 23, 2022

This PR

  • replaces the polling loop in AwaitK8sJobDone with a watch that is able to listen to changes to the job
  • introduces a watch for events in AwaitK8sJobDone to catch errors while creating a Pod (unsound spec, image pull errors)

Related

@github-actions
Copy link
Contributor

github-actions bot commented Jun 23, 2022

Unit Test Results

101 tests   90 ✔️  0s ⏱️
  13 suites  10 💤
    1 files      1

For more details on these failures, see this check.

Results for commit 5aaf2c3.

♻️ This comment has been updated with latest results.

@Raffy23 Raffy23 force-pushed the feat/implement-watch-for-jobs branch from 6221047 to 5aaf2c3 Compare June 30, 2022 14:49
@github-actions
Copy link
Contributor

The following Docker Images have been built:

  • keptncontrib/job-executor-service:0.2.3-dev-PR-294,keptncontrib/job-executor-service:0.2.3-dev-PR-294.202206301450
  • keptncontrib/job-executor-service-initcontainer:0.2.3-dev-PR-294,keptncontrib/job-executor-service-initcontainer:0.2.3-dev-PR-294.202206301450

@Raffy23 Raffy23 closed this Mar 19, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant