Remove unused config

broadinstitute · Nov 1, 2024 · 71dd01d · 71dd01d
1 parent 32685d1
commit 71dd01d
Show file tree

Hide file tree

Showing 4 changed files with 0 additions and 14 deletions.
diff --git a/...src/main/scala/cromwell/backend/google/batch/models/GcpBatchConfigurationAttributes.scala b/...src/main/scala/cromwell/backend/google/batch/models/GcpBatchConfigurationAttributes.scala
@@ -40,7 +40,6 @@ case class GcpBatchConfigurationAttributes(
   project: String,
   computeServiceAccount: String,
   auths: GcpBatchAuths,
-  restrictMetadataAccess: Boolean,
   dockerhubToken: String,
   enableFuse: Boolean,
   executionBucket: String,
@@ -327,7 +326,6 @@ object GcpBatchConfigurationAttributes extends GcpBatchReferenceFilesMappingOper
           project = project,
           computeServiceAccount = computeServiceAccount,
           auths = GcpBatchAuths(batchAuth, gcsAuth),
-          restrictMetadataAccess = restrictMetadata,
           dockerhubToken = dockerhubToken,
           enableFuse = enableFuse,
           executionBucket = bucket,

diff --git a/...scala/cromwell/backend/google/batch/actors/GcpBatchBackendLifecycleActorFactorySpec.scala b/...scala/cromwell/backend/google/batch/actors/GcpBatchBackendLifecycleActorFactorySpec.scala
@@ -19,7 +19,6 @@ class GcpBatchBackendLifecycleActorFactorySpec extends AnyFlatSpecLike with Matc
       project = "project",
       computeServiceAccount = "computeServiceAccount",
       auths = null,
-      restrictMetadataAccess = true,
       dockerhubToken = "test",
       enableFuse = true,
       executionBucket = "executionBucket",

diff --git a/...cala/cromwell/backend/google/batch/callcaching/BatchBackendCacheHitCopyingActorSpec.scala b/...cala/cromwell/backend/google/batch/callcaching/BatchBackendCacheHitCopyingActorSpec.scala
@@ -419,7 +419,6 @@ class BatchBackendCacheHitCopyingActorSpec
       project = null,
       computeServiceAccount = null,
       auths = null,
-      restrictMetadataAccess = false,
       dockerhubToken = null,
       enableFuse = false,
       executionBucket = null,

diff --git a/...test/scala/cromwell/backend/google/batch/models/GcpBatchConfigurationAttributesSpec.scala b/...test/scala/cromwell/backend/google/batch/models/GcpBatchConfigurationAttributesSpec.scala
@@ -34,7 +34,6 @@ class GcpBatchConfigurationAttributesSpec
     gcpBatchAttributes.executionBucket should be("gs://myBucket")
     gcpBatchAttributes.maxPollingInterval should be(600)
     gcpBatchAttributes.computeServiceAccount should be("default")
-    gcpBatchAttributes.restrictMetadataAccess should be(false)
     gcpBatchAttributes.referenceFileToDiskImageMappingOpt.isEmpty should be(true)
     gcpBatchAttributes.logsPolicy should be(GcpBatchLogsPolicy.CloudLogging)
   }
@@ -112,15 +111,6 @@ class GcpBatchConfigurationAttributesSpec
     gcpBatchAttributes.computeServiceAccount should be("testing")
   }
 
-  it should "parse restrict-metadata-access" in {
-
-    val backendConfig = ConfigFactory.parseString(configString(batch = "restrict-metadata-access = true"))
-
-    val gcpBatchAttributes = GcpBatchConfigurationAttributes(googleConfig, backendConfig, "batch")
-    gcpBatchAttributes.restrictMetadataAccess should be(true)
-
-  }
-
   it should "parse localization-attempts" in {
 
     val backendConfig = ConfigFactory.parseString(configString(batch = "localization-attempts = 31380"))