-
Notifications
You must be signed in to change notification settings - Fork 25k
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
Skip FIPS JVMs in testReloadCredentialsFromKeystore
#116814
Skip FIPS JVMs in testReloadCredentialsFromKeystore
#116814
Conversation
This test doesn't need to run in FIPS mode, and apparently it fails sometimes, so with this commit we skip it. Closes elastic#116811
Pinging @elastic/es-security (Team:Security) |
Pinging @elastic/es-distributed-coordination (Team:Distributed Coordination) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
We need to remove the mute from muted-tests.yml as well.
I guess it is possible to make the test work with FIPS. But the value is questionable since this test does not even have security enabled due to its usage of the INGEST_TEST
distribution type.
💔 Backport failed
You can use sqren/backport to manually backport by running |
This test doesn't need to run in FIPS mode, and apparently it fails sometimes, so with this commit we skip it. Closes elastic#116811 Backport of elastic#116814 to 8.x
💚 All backports created successfully
Questions ?Please refer to the Backport tool documentation |
This test doesn't need to run in FIPS mode, and apparently it fails sometimes, so with this commit we skip it. Closes elastic#116811 (cherry picked from commit ea90fbc) # Conflicts: # muted-tests.yml
) This test doesn't need to run in FIPS mode, and apparently it fails sometimes, so with this commit we skip it. Closes #116811 (cherry picked from commit ea90fbc) # Conflicts: # muted-tests.yml Co-authored-by: David Turner <[email protected]>
This test doesn't need to run in FIPS mode, and apparently it fails sometimes, so with this commit we skip it. Closes elastic#116811
This test doesn't need to run in FIPS mode, and apparently it fails sometimes, so with this commit we skip it. Closes elastic#116811
This test doesn't need to run in FIPS mode, and apparently it fails
sometimes, so with this commit we skip it.
Closes #116811