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

start e2e before all clusters are ready #127

Merged
merged 1 commit into from
Apr 10, 2023

Conversation

venkataanil
Copy link
Contributor

Currently watcher will wait for all clusters to move to ready state and once it happens then notifies all threads to run e2e testing. If the user wants the created threads to start e2e before all clusters are moved to ready state, he can create a file named "e2e" dynamically in test directory. Watcher will check for this file and notify all threads to start running e2e tests.

With this new change, e2e testing will start in either

  1. all clusters moved to ready state
  2. or user created a file name "e2e" in test directory i.e /tmp//e2e

Fixes

#126

Currently watcher will wait for all clusters to move to ready state
and once it happens then notifies all threads to run e2e testing.
If the user wants the created threads to start e2e before all
clusters are moved to ready state, he can create a file named "e2e"
dynamically in test directory. Watcher will check for this file and
notify all threads to start running e2e tests.

With this new change, e2e testing will start in either
1. all clusters moved to ready state
2. or user created a file name "e2e" in test directory i.e
   /tmp/<uuid>/e2e
@dry923 dry923 requested review from morenod, dry923 and mukrishn March 23, 2023 15:07
@smalleni
Copy link

smalleni commented Apr 7, 2023

What are the next steps for this patch?
@morenod Please review and merge as appropriate

Copy link
Collaborator

@morenod morenod left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@morenod morenod merged commit 56b394a into cloud-bulldozer:main Apr 10, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants