property

snazy · snazy · commit 4102876a251f · 2025-05-12T09:18:32.000+02:00
diff --git a/plugins/spark/v3.5/regtests/docker-compose.yml b/plugins/spark/v3.5/regtests/docker-compose.yml
@@ -30,7 +30,7 @@ services:
       quarkus.otel.sdk.disabled: "true"
       polaris.features.defaults."ALLOW_INSECURE_STORAGE_TYPES": "true"
       polaris.features.defaults."SUPPORTED_CATALOG_STORAGE_TYPES": "[\"FILE\",\"S3\",\"GCS\",\"AZURE\"]"
-      polaris.readiness.ignore-security-issues: "true"
+      polaris.readiness.ignore-severe-issues: "true"
     healthcheck:
       test: ["CMD", "curl", "http://localhost:8182/q/health"]
       interval: 10s
diff --git a/quarkus/defaults/src/main/resources/application-it.properties b/quarkus/defaults/src/main/resources/application-it.properties
@@ -42,7 +42,7 @@ polaris.features.defaults."INITIALIZE_DEFAULT_CATALOG_FILEIO_FOR_it"=true
 polaris.features.defaults."SKIP_CREDENTIAL_SUBSCOPING_INDIRECTION"=true
 polaris.features.defaults."SUPPORTED_CATALOG_STORAGE_TYPES"=["FILE","S3","GCS","AZURE"]
 polaris.features.defaults."ENABLE_CATALOG_FEDERATION"=true
-polaris.readiness.ignore-security-issues=true
+polaris.readiness.ignore-severe-issues=true
 
 polaris.realm-context.realms=POLARIS,OTHER
 
diff --git a/quarkus/service/src/main/java/org/apache/polaris/service/quarkus/config/ProductionReadinessChecks.java b/quarkus/service/src/main/java/org/apache/polaris/service/quarkus/config/ProductionReadinessChecks.java
@@ -101,13 +101,13 @@ public void warnOnFailedChecks(
               "Refer to https://polaris.apache.org/in-dev/unreleased/configuring-polaris-for-production for more information.");
 
       if (hasSevere) {
-        if (!config.ignoreSecurityIssues()) {
+        if (!config.ignoreSevereIssues()) {
           throw new IllegalStateException(
               "Severe production readiness issues detected, startup aborted!");
         }
         LOGGER.error(
             "{} severe production readiness issues detected, but user explicitly requested startup by setting "
-                + "polaris.readiness.ignore-security-issues=true and accepts the risk of denial-of-service, "
+                + "polaris.readiness.ignore-severe-issues=true and accepts the risk of denial-of-service, "
                 + "data-loss, corruption and others !",
             severe);
       }
diff --git a/quarkus/service/src/main/java/org/apache/polaris/service/quarkus/config/QuarkusReadinessConfiguration.java b/quarkus/service/src/main/java/org/apache/polaris/service/quarkus/config/QuarkusReadinessConfiguration.java
@@ -32,5 +32,5 @@ public interface QuarkusReadinessConfiguration {
    * risks.
    */
   @WithDefault("false")
-  boolean ignoreSecurityIssues();
+  boolean ignoreSevereIssues();
 }
diff --git a/quarkus/service/src/test/java/org/apache/polaris/service/quarkus/admin/PolarisAuthzTestBase.java b/quarkus/service/src/test/java/org/apache/polaris/service/quarkus/admin/PolarisAuthzTestBase.java
@@ -121,7 +121,7 @@ public Map<String, String> getConfigOverrides() {
           "true",
           "polaris.features.defaults.\"SUPPORTED_CATALOG_STORAGE_TYPES\"",
           "[\"FILE\",\"S3\"]",
-          "polaris.readiness.ignore-security-issues",
+          "polaris.readiness.ignore-severe-issues",
           "true");
     }
   }
diff --git a/quarkus/service/src/test/java/org/apache/polaris/service/quarkus/catalog/GenericTableCatalogTest.java b/quarkus/service/src/test/java/org/apache/polaris/service/quarkus/catalog/GenericTableCatalogTest.java
@@ -114,7 +114,7 @@ public Map<String, String> getConfigOverrides() {
           "true",
           "polaris.features.defaults.\"SUPPORTED_CATALOG_STORAGE_TYPES\"",
           "[\"FILE\"]",
-          "polaris.readiness.ignore-security-issues",
+          "polaris.readiness.ignore-severe-issues",
           "true");
     }
   }
diff --git a/quarkus/service/src/test/java/org/apache/polaris/service/quarkus/catalog/IcebergCatalogTest.java b/quarkus/service/src/test/java/org/apache/polaris/service/quarkus/catalog/IcebergCatalogTest.java
@@ -181,7 +181,7 @@ public Map<String, String> getConfigOverrides() {
           "true",
           "polaris.event-listener.type",
           "test",
-          "polaris.readiness.ignore-security-issues",
+          "polaris.readiness.ignore-severe-issues",
           "true");
     }
   }
diff --git a/quarkus/service/src/test/java/org/apache/polaris/service/quarkus/catalog/IcebergCatalogViewTest.java b/quarkus/service/src/test/java/org/apache/polaris/service/quarkus/catalog/IcebergCatalogViewTest.java
@@ -114,7 +114,7 @@ public Map<String, String> getConfigOverrides() {
           "[\"FILE\",\"S3\"]",
           "polaris.event-listener.type",
           "test",
-          "polaris.readiness.ignore-security-issues",
+          "polaris.readiness.ignore-severe-issues",
           "true");
     }
   }
diff --git a/quarkus/service/src/test/java/org/apache/polaris/service/quarkus/catalog/PolicyCatalogTest.java b/quarkus/service/src/test/java/org/apache/polaris/service/quarkus/catalog/PolicyCatalogTest.java
@@ -129,7 +129,7 @@ public Map<String, String> getConfigOverrides() {
           "[\"FILE\"]",
           "polaris.features.defaults.\"ALLOW_INSECURE_STORAGE_TYPES\"",
           "true",
-          "polaris.readiness.ignore-security-issues",
+          "polaris.readiness.ignore-severe-issues",
           "true");
     }
   }
diff --git a/quarkus/service/src/test/java/org/apache/polaris/service/quarkus/it/QuarkusApplicationIntegrationTest.java b/quarkus/service/src/test/java/org/apache/polaris/service/quarkus/it/QuarkusApplicationIntegrationTest.java
@@ -64,7 +64,7 @@ public Map<String, String> getConfigOverrides() {
           "true",
           "polaris.features.defaults.\"SUPPORTED_CATALOG_STORAGE_TYPES\"",
           "[\"FILE\",\"S3\"]",
-          "polaris.readiness.ignore-security-issues",
+          "polaris.readiness.ignore-severe-issues",
           "true");
     }
   }
diff --git a/quarkus/service/src/test/java/org/apache/polaris/service/quarkus/it/QuarkusPolicyServiceIntegrationTest.java b/quarkus/service/src/test/java/org/apache/polaris/service/quarkus/it/QuarkusPolicyServiceIntegrationTest.java
@@ -38,7 +38,7 @@ public Map<String, String> getConfigOverrides() {
           "true",
           "polaris.features.defaults.\"SUPPORTED_CATALOG_STORAGE_TYPES\"",
           "[\"FILE\",\"S3\"]",
-          "polaris.readiness.ignore-security-issues",
+          "polaris.readiness.ignore-severe-issues",
           "true");
     }
   }
diff --git a/quarkus/service/src/test/java/org/apache/polaris/service/quarkus/it/QuarkusRestCatalogIntegrationTest.java b/quarkus/service/src/test/java/org/apache/polaris/service/quarkus/it/QuarkusRestCatalogIntegrationTest.java
@@ -41,7 +41,7 @@ public Map<String, String> getConfigOverrides() {
           "true",
           "polaris.features.defaults.\"SUPPORTED_CATALOG_STORAGE_TYPES\"",
           "[\"FILE\",\"S3\"]",
-          "polaris.readiness.ignore-security-issues",
+          "polaris.readiness.ignore-severe-issues",
           "true");
     }
   }
diff --git a/quarkus/service/src/test/java/org/apache/polaris/service/quarkus/it/QuarkusRestCatalogViewFileIntegrationTest.java b/quarkus/service/src/test/java/org/apache/polaris/service/quarkus/it/QuarkusRestCatalogViewFileIntegrationTest.java
@@ -43,7 +43,7 @@ public Map<String, String> getConfigOverrides() {
           "true",
           "polaris.features.defaults.\"SUPPORTED_CATALOG_STORAGE_TYPES\"",
           "[\"FILE\"]",
-          "polaris.readiness.ignore-security-issues",
+          "polaris.readiness.ignore-severe-issues",
           "true");
     }
   }
diff --git a/regtests/docker-compose.yml b/regtests/docker-compose.yml
@@ -36,7 +36,7 @@ services:
       quarkus.otel.sdk.disabled: "true"
       polaris.features.defaults."ALLOW_INSECURE_STORAGE_TYPES": "true"
       polaris.features.defaults."SUPPORTED_CATALOG_STORAGE_TYPES": "[\"FILE\",\"S3\",\"GCS\",\"AZURE\"]"
-      polaris.readiness.ignore-security-issues: "true"
+      polaris.readiness.ignore-severe-issues: "true"
     volumes:
       - ./credentials:/tmp/credentials/
     healthcheck:

Original file line number	Diff line number	Diff line change
`@@ -32,5 +32,5 @@ public interface QuarkusReadinessConfiguration {`
`32`	`32`	`* risks.`
`33`	`33`	`*/`
`34`	`34`	`@WithDefault("false")`
`35`		`- boolean ignoreSecurityIssues();`
	`35`	`+ boolean ignoreSevereIssues();`
`36`	`36`	`}`
Original file line number	Diff line number	Diff line change
`@@ -121,7 +121,7 @@ public Map<String, String> getConfigOverrides() {`
`121`	`121`	`"true",`
`122`	`122`	`"polaris.features.defaults.\"SUPPORTED_CATALOG_STORAGE_TYPES\"",`
`123`	`123`	`"[\"FILE\",\"S3\"]",`
`124`		`- "polaris.readiness.ignore-security-issues",`
	`124`	`+ "polaris.readiness.ignore-severe-issues",`
`125`	`125`	`"true");`
`126`	`126`	`}`
`127`	`127`	`}`
Original file line number	Diff line number	Diff line change
`@@ -114,7 +114,7 @@ public Map<String, String> getConfigOverrides() {`
`114`	`114`	`"true",`
`115`	`115`	`"polaris.features.defaults.\"SUPPORTED_CATALOG_STORAGE_TYPES\"",`
`116`	`116`	`"[\"FILE\"]",`
`117`		`- "polaris.readiness.ignore-security-issues",`
	`117`	`+ "polaris.readiness.ignore-severe-issues",`
`118`	`118`	`"true");`
`119`	`119`	`}`
`120`	`120`	`}`
Original file line number	Diff line number	Diff line change
`@@ -181,7 +181,7 @@ public Map<String, String> getConfigOverrides() {`
`181`	`181`	`"true",`
`182`	`182`	`"polaris.event-listener.type",`
`183`	`183`	`"test",`
`184`		`- "polaris.readiness.ignore-security-issues",`
	`184`	`+ "polaris.readiness.ignore-severe-issues",`
`185`	`185`	`"true");`
`186`	`186`	`}`
`187`	`187`	`}`
Original file line number	Diff line number	Diff line change
`@@ -129,7 +129,7 @@ public Map<String, String> getConfigOverrides() {`
`129`	`129`	`"[\"FILE\"]",`
`130`	`130`	`"polaris.features.defaults.\"ALLOW_INSECURE_STORAGE_TYPES\"",`
`131`	`131`	`"true",`
`132`		`- "polaris.readiness.ignore-security-issues",`
	`132`	`+ "polaris.readiness.ignore-severe-issues",`
`133`	`133`	`"true");`
`134`	`134`	`}`
`135`	`135`	`}`
Original file line number	Diff line number	Diff line change
`@@ -64,7 +64,7 @@ public Map<String, String> getConfigOverrides() {`
`64`	`64`	`"true",`
`65`	`65`	`"polaris.features.defaults.\"SUPPORTED_CATALOG_STORAGE_TYPES\"",`
`66`	`66`	`"[\"FILE\",\"S3\"]",`
`67`		`- "polaris.readiness.ignore-security-issues",`
	`67`	`+ "polaris.readiness.ignore-severe-issues",`
`68`	`68`	`"true");`
`69`	`69`	`}`
`70`	`70`	`}`
Original file line number	Diff line number	Diff line change
`@@ -38,7 +38,7 @@ public Map<String, String> getConfigOverrides() {`
`38`	`38`	`"true",`
`39`	`39`	`"polaris.features.defaults.\"SUPPORTED_CATALOG_STORAGE_TYPES\"",`
`40`	`40`	`"[\"FILE\",\"S3\"]",`
`41`		`- "polaris.readiness.ignore-security-issues",`
	`41`	`+ "polaris.readiness.ignore-severe-issues",`
`42`	`42`	`"true");`
`43`	`43`	`}`
`44`	`44`	`}`
Original file line number	Diff line number	Diff line change
`@@ -41,7 +41,7 @@ public Map<String, String> getConfigOverrides() {`
`41`	`41`	`"true",`
`42`	`42`	`"polaris.features.defaults.\"SUPPORTED_CATALOG_STORAGE_TYPES\"",`
`43`	`43`	`"[\"FILE\",\"S3\"]",`
`44`		`- "polaris.readiness.ignore-security-issues",`
	`44`	`+ "polaris.readiness.ignore-severe-issues",`
`45`	`45`	`"true");`
`46`	`46`	`}`
`47`	`47`	`}`
Original file line number	Diff line number	Diff line change
`@@ -43,7 +43,7 @@ public Map<String, String> getConfigOverrides() {`
`43`	`43`	`"true",`
`44`	`44`	`"polaris.features.defaults.\"SUPPORTED_CATALOG_STORAGE_TYPES\"",`
`45`	`45`	`"[\"FILE\"]",`
`46`		`- "polaris.readiness.ignore-security-issues",`
	`46`	`+ "polaris.readiness.ignore-severe-issues",`
`47`	`47`	`"true");`
`48`	`48`	`}`
`49`	`49`	`}`