-
Notifications
You must be signed in to change notification settings - Fork 502
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add trap to create a path when BR container exit (#5431) #5443
Add trap to create a path when BR container exit (#5431) #5443
Conversation
/retest |
Codecov Report
Additional details and impacted files@@ Coverage Diff @@
## release-1.5 #5443 +/- ##
===============================================
+ Coverage 61.50% 67.96% +6.46%
===============================================
Files 228 241 +13
Lines 29092 33037 +3945
===============================================
+ Hits 17893 22455 +4562
+ Misses 9446 8795 -651
- Partials 1753 1787 +34
|
/run-pull-e2e-kind-across-kubernetes |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: BornChanger The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
/run-pull-e2e-kind |
1 similar comment
/run-pull-e2e-kind |
/run-pull-e2e-kind-across-kubernetes |
[LGTM Timeline notifier]Timeline:
|
New changes are detected. LGTM label has been removed. |
/retest |
/run-pull-e2e-kind-serial |
/run-pull-e2e-kind-basic |
This is an automated cherry-pick of #5431
What problem does this PR solve?
Closes #5430
What is changed and how does it work?
Create a path when the main container exits, the sidecar can check the path once it shows up it can exit.
Code changes
Tests
Side effects
Related changes
Release Notes
Please refer to Release Notes Language Style Guide before writing the release note.