Skip to content

Commit

Permalink
Fix integration test.
Browse files Browse the repository at this point in the history
Concurrency will aggressively default to 80 per #5196.
  • Loading branch information
taeold committed Nov 2, 2022
1 parent 7abccc6 commit 6ab1628
Showing 1 changed file with 4 additions and 1 deletion.
5 changes: 4 additions & 1 deletion scripts/functions-deploy-tests/tests.ts
Original file line number Diff line number Diff line change
Expand Up @@ -306,7 +306,10 @@ describe("firebase deploy", function (this) {
if (e.platform === "gcfv2") {
expect(e).to.include({
cpu: 2,
concurrency: 42,
// EXCEPTION: concurrency
// Firebase will aggressively set concurrency to 80 when the CPU setting allows for it
// AND when the concurrency is NOT set on the source code.
concurrency: 80,
});
}
// BUGBUG: As implemented, Cloud Tasks update doesn't preserve existing setting. Instead, it overwrites the
Expand Down

0 comments on commit 6ab1628

Please sign in to comment.