Skip to content

fix: logic to interrupt a k8sjob logs as soon as it fails (#8847)#8871

Merged
renzodavid9 merged 1 commit into
GoogleContainerTools:release/v2.5from
renzodavid9:cherry-pick-k8s-ca-issue
Jun 9, 2023
Merged

fix: logic to interrupt a k8sjob logs as soon as it fails (#8847)#8871
renzodavid9 merged 1 commit into
GoogleContainerTools:release/v2.5from
renzodavid9:cherry-pick-k8s-ca-issue

Conversation

@renzodavid9

Copy link
Copy Markdown
Contributor

This is a cherry pick to fix #8839 inv2.5:

@renzodavid9 renzodavid9 changed the base branch from main to release/v2.5 June 8, 2023 20:18
@renzodavid9 renzodavid9 force-pushed the cherry-pick-k8s-ca-issue branch from 96be367 to 82b507d Compare June 8, 2023 20:21
@codecov

codecov Bot commented Jun 8, 2023

Copy link
Copy Markdown

Codecov Report

❗ No coverage uploaded for pull request base (release/v2.5@bb0ed01). Click here to learn what that means.
The diff coverage is n/a.

❗ Current head 76d6532 differs from pull request most recent head 4020378. Consider uploading reports for the commit 4020378 to get more accurate results

@@               Coverage Diff               @@
##             release/v2.5    #8871   +/-   ##
===============================================
  Coverage                ?   63.93%           
===============================================
  Files                   ?      620           
  Lines                   ?    31460           
  Branches                ?        0           
===============================================
  Hits                    ?    20115           
  Misses                  ?     9842           
  Partials                ?     1503           

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

…ainerTools#8847)

* fix: logic to interrupt a k8sjob logs as soon as it fails and change in delete order, now the job is deleted before its pods

* test: integration tests to check when a log should not appear in the output

(cherry picked from commit c7c72a1)
@renzodavid9 renzodavid9 force-pushed the cherry-pick-k8s-ca-issue branch from 76d6532 to 4020378 Compare June 9, 2023 16:57
@renzodavid9 renzodavid9 marked this pull request as ready for review June 9, 2023 17:44
@renzodavid9 renzodavid9 requested review from ericzzzzzzz and plumpy June 9, 2023 17:44
@renzodavid9 renzodavid9 merged commit 1c990f2 into GoogleContainerTools:release/v2.5 Jun 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

2 participants