Skip FIPS JVMs in testReloadCredentialsFromKeystore (#116814) (#117100)

ywangd · DaveCTurner · web-flow · commit 18fe9d55430c · 2024-11-20T09:28:09.000+11:00
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 <david.turner@elastic.co>
diff --git a/modules/repository-s3/src/javaRestTest/java/org/elasticsearch/repositories/s3/RepositoryS3RestIT.java b/modules/repository-s3/src/javaRestTest/java/org/elasticsearch/repositories/s3/RepositoryS3RestIT.java
@@ -52,6 +52,8 @@ protected String getTestRestCluster() {
     }
 
     public void testReloadCredentialsFromKeystore() throws IOException {
+        assumeFalse("doesn't work in a FIPS JVM, but that's ok", inFipsJvm());
+
         // Register repository (?verify=false because we don't have access to the blob store yet)
         final var repositoryName = randomIdentifier();
         registerRepository(

Original file line number	Diff line number	Diff line change
`@@ -52,6 +52,8 @@ protected String getTestRestCluster() {`
`52`	`52`	`}`
`53`	`53`
`54`	`54`	`public void testReloadCredentialsFromKeystore() throws IOException {`
	`55`	`+ assumeFalse("doesn't work in a FIPS JVM, but that's ok", inFipsJvm());`
	`56`	`+`
`55`	`57`	`// Register repository (?verify=false because we don't have access to the blob store yet)`
`56`	`58`	`final var repositoryName = randomIdentifier();`
`57`	`59`	`registerRepository(`