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
Three more lifecycle checks to demonstrate various validation techniques for containers lifecycle checks #116426
Three more lifecycle checks to demonstrate various validation techniques for containers lifecycle checks #116426
Conversation
/sig node |
/triage accepted |
97234ba
to
bb1acbd
Compare
bb1acbd
to
72e366f
Compare
/assign @andrewsykim can you please approve for the new wait method? I think we will use it more with other lifecycle tests |
3bc23c0
to
8adb519
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Passes the previous failed test with persistent file.
/lgtm
LGTM label has been added. Git tree hash: 874d393f246a665df4b642e6ba58aef443a09ced
|
8adb519
to
1385537
Compare
} | ||
|
||
// WaitForPodContainerRestartCount waits for the given Pod container to achieve at least a given restartCount | ||
func WaitForPodContainerRestartCount(ctx context.Context, c clientset.Interface, namespace, podName string, containerIndex int, desiredRestartCount int32, timeout time.Duration) error { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tzneal @andrewsykim moved method here temporarily so I can self-approve this PR and rebase the sidecar container branch
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@tzneal can you please re-lgtm?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
/lgtm
LGTM label has been added. Git tree hash: 2b03382982b0723379093d0ccc6014dde0a8064e
|
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: SergeyKanzhelev, tzneal 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 |
/test pull-kubernetes-unit |
@SergeyKanzhelev: The following test failed, say
Full PR test history. Your PR dashboard. Please help us cut down on flakes by linking to an open issue when you hit one in your PR. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. I understand the commands that are listed here. |
/retest-required |
What type of PR is this?
Added two more tests to the lifecycle tests to simplify testing of Sidecar
/kind cleanup
/area test
What this PR does / why we need it:
This increases test coverage and simplifies testing of upcoming Sidecar KEP.
Does this PR introduce a user-facing change?