Skip to content

Commit f3825f8

Browse files
authored
Change the AR cleanup policy error to a warning for CI/CD workloads (#8513)
* change error to warn * update changelog
1 parent 6acae63 commit f3825f8

File tree

2 files changed

+3
-1
lines changed

2 files changed

+3
-1
lines changed

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
- Changed artifact registry cleanup policy error to warn for CI/CD workloads #8513

src/deploy/functions/release/index.ts

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -196,7 +196,8 @@ async function setupArtifactCleanupPolicies(
196196
`${locationsWithErrors.join(", ")}.` +
197197
"This could result in a small monthly bill as container images accumulate over time.",
198198
);
199-
throw new FirebaseError(
199+
utils.logLabeledWarning(
200+
"functions",
200201
`Functions successfully deployed but could not set up cleanup policy in ` +
201202
`${locationsWithErrors.length > 1 ? "regions" : "region"} ${locationsWithErrors.join(", ")}. ` +
202203
`Pass the --force option to automatically set up a cleanup policy or ` +

0 commit comments

Comments
 (0)