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

Check cluster UUID annotation in E2E tests #2404

Merged
merged 1 commit into from
Jan 10, 2020

Conversation

sebgl
Copy link
Contributor

@sebgl sebgl commented Jan 10, 2020

Ensure the cluster is eventually annotated with its UUID.
The nice side effect is that we also make sure the cluster bootstrap is
over before applying a mutation.

Fixes #2397.

Ensure the cluster is eventually annotated with its UUID.
The nice side effect is that we also make sure the cluster bootstrap is
over before applying a mutation.
@sebgl sebgl added >test Related to unit/integration/e2e tests v1.1.0 labels Jan 10, 2020
Copy link
Contributor

@thbkrkr thbkrkr left a comment

Choose a reason for hiding this comment

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

LGTM

@sebgl sebgl merged commit 0bc4ab2 into elastic:master Jan 10, 2020
mjmbischoff pushed a commit to mjmbischoff/cloud-on-k8s that referenced this pull request Jan 13, 2020
Ensure the cluster is eventually annotated with its UUID.
The nice side effect is that we also make sure the cluster bootstrap is
over before applying a mutation.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
>test Related to unit/integration/e2e tests v1.1.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Cluster UUID changed when upgrading single master 6.8.x to new nodeSet 7.3.x
2 participants