Skip to content
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

e2e: add ability to wait for different conditions in kubeclient.WaitFor #888

Open
wants to merge 14 commits into
base: main
Choose a base branch
from

Conversation

miampf
Copy link
Contributor

@miampf miampf commented Sep 25, 2024

This adds the ability to wait for different conditions using the kubeclient.WaitFor function. Supported conditions are:

  • Ready: This has the same functionality as the old kubeclient.WaitFor.
  • Running: Waits until all containers of all pods of the corresponding resource are running or until the context timeouts.
  • InitContainersRunning: Same as Running, but only check the initial containers of all pods of the corresponding resource.
  • Added, Bookmark, Deleted, Modified: Those are synonymous for waiting on the corresponding watch.EventType found in k8s.io/apimachinery/pkg/watch.

@miampf miampf added the no changelog PRs not listed in the release notes label Sep 25, 2024
@miampf miampf force-pushed the miampf/kubeclient/improve-waiting branch from 2a42396 to 843da8f Compare September 25, 2024 13:25
@miampf miampf marked this pull request as ready for review September 25, 2024 13:54
@miampf miampf requested review from Freax13 and katexochen and removed request for katexochen September 25, 2024 13:54
e2e/internal/kubeclient/deploy.go Outdated Show resolved Hide resolved
e2e/internal/kubeclient/deploy.go Outdated Show resolved Hide resolved
e2e/internal/kubeclient/deploy.go Outdated Show resolved Hide resolved
@miampf miampf force-pushed the miampf/kubeclient/improve-waiting branch from 1114b89 to c53c6c7 Compare September 27, 2024 11:22
@miampf miampf force-pushed the miampf/kubeclient/improve-waiting branch from c53c6c7 to 4b7a266 Compare September 27, 2024 13:19
@miampf miampf requested a review from Freax13 September 27, 2024 13:41
e2e/internal/kubeclient/deploy.go Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog PRs not listed in the release notes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants