Skip to content

[Test] Fix user profile YAML tests in a mixed cluster #88208

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

Merged
merged 1 commit into from
Jun 30, 2022

Conversation

ywangd
Copy link
Member

@ywangd ywangd commented Jun 30, 2022

User profile is behind a feature flag in 8.3. When 8.3 is still a
snapshot build, the feature flag is enabled automatically. But now 8.3
is released, the feature flag is by default off. This PR adjust the node
selector to only select node version 8.4 or higher.

Resolves: #88202

User profile is behind a feature flag in 8.3. When 8.3 is still a
snapshot build, the feature flag is enabled automatically. But now 8.3
is released, the feature flag is by default off. This PR adjust the node
selector to only select node version 8.4 or higher.

Resolves: elastic#88202
@ywangd ywangd added >test Issues or PRs that are addressing/adding tests :Security/Security Security issues without another label v8.4.0 labels Jun 30, 2022
@ywangd ywangd requested a review from n1v0lg June 30, 2022 11:42
@elasticmachine elasticmachine added the Team:Security Meta label for security team label Jun 30, 2022
@elasticmachine
Copy link
Collaborator

Pinging @elastic/es-security (Team:Security)

Copy link
Contributor

@n1v0lg n1v0lg left a comment

Choose a reason for hiding this comment

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

Good catch!

@ywangd ywangd added the auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) label Jun 30, 2022
@elasticsearchmachine elasticsearchmachine merged commit 4cdfe7e into elastic:master Jun 30, 2022
@ywangd ywangd deleted the es-88202-test branch June 30, 2022 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto-merge-without-approval Automatically merge pull request when CI checks pass (NB doesn't wait for reviews!) :Security/Security Security issues without another label Team:Security Meta label for security team >test Issues or PRs that are addressing/adding tests v8.4.0
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[CI] UpgradeClusterClientYamlTestSuiteIT test {p0=mixed_cluster/140_user_profile/Test User Profile feature will work in a mixed cluster} failing
4 participants