Skip to content

Commit

Permalink
ID-1276 Fix bard url value injection to cromwell-runner-app (#4709)
Browse files Browse the repository at this point in the history
  • Loading branch information
Ghost-in-a-Jar authored Jul 10, 2024
1 parent 5759fb0 commit a9a9c0f
Show file tree
Hide file tree
Showing 2 changed files with 2 additions and 2 deletions.
Original file line number Diff line number Diff line change
Expand Up @@ -134,7 +134,7 @@ class CromwellRunnerAppInstall[F[_]](config: CromwellRunnerAppConfig,
raw"sam.acrPullActionIdentityResourceId=${params.billingProfileId.value}",

// Bard configs
raw"bard.baseUri=${config.bardBaseUri}",
raw"bard.bardUrl=${config.bardBaseUri}",
raw"bard.enabled=${config.bardEnabled}"
)
} yield Values(values.mkString(","))
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -67,7 +67,7 @@ class CromwellRunnerAppInstallSpec extends BaseAppInstallSpec {
s"ecm.baseUri=https://externalcreds.dsde-dev.broadinstitute.org," +
s"sam.baseUri=https://sam.test.org:443," +
s"sam.acrPullActionIdentityResourceId=spend-profile," +
"bard.baseUri=https://terra-bard-dev.appspot.com," +
"bard.bardUrl=https://terra-bard-dev.appspot.com," +
"bard.enabled=false"
}

Expand Down

0 comments on commit a9a9c0f

Please sign in to comment.