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

Adjust waiting for KUDO init in e2e tests #1592

Merged
merged 1 commit into from
Jul 14, 2020
Merged

Conversation

ANeumann82
Copy link
Member

Removed 00-assert that checks if KUDO is running and use --wait in install

Signed-off-by: Andreas Neumann aneumann@mesosphere.com

…stall

Signed-off-by: Andreas Neumann <aneumann@mesosphere.com>
@@ -3,7 +3,7 @@ kind: TestSuite
testDirs:
- ./test/e2e
commands:
- command: ./bin/kubectl-kudo init --unsafe-self-signed-webhook-ca --kudo-image kudobuilder/controller:%version% --kudo-image-pull-policy Never
- command: ./bin/kubectl-kudo init --wait --unsafe-self-signed-webhook-ca --kudo-image kudobuilder/controller:%version% --kudo-image-pull-policy Never
Copy link
Member Author

Choose a reason for hiding this comment

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

This is the only actual change.

All other changes are:

  • Remove the 00-assert.yaml from all e2e-tests (which ensured that KUDO manager was running)
  • Move all other test files one number up ( which git didn't recognise correctly, therefore the many diffs :( )

Copy link
Contributor

@zen-dog zen-dog left a comment

Choose a reason for hiding this comment

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

🚢

Copy link
Contributor

@alenkacz alenkacz left a comment

Choose a reason for hiding this comment

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

WOOHOO, nice work 👏

@ANeumann82 ANeumann82 merged commit 738a730 into main Jul 14, 2020
@ANeumann82 ANeumann82 deleted the an/adjust-e2e-kudo-init branch July 14, 2020 09:49
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.

None yet

3 participants