Skip to content

Commit e681516

Browse files
authored
Fix release workflow (#3764)
- Fixes a recent regression in permissions on the publish-artifact job (introduced by #3706). - Makes image publishing rely on the same jobs as artifact publishing so that we don't publish images but fail to publish the release if there is a failure in the windows jobs. Signed-off-by: Andrew Harding <[email protected]>
1 parent e04226a commit e681516

File tree

1 file changed

+3
-2
lines changed

1 file changed

+3
-2
lines changed

.github/workflows/release_build.yaml

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -542,7 +542,7 @@ jobs:
542542
lint-windows, unit-test-windows, artifact-windows, integration-windows]
543543

544544
permissions:
545-
contents: read
545+
contents: write
546546

547547
steps:
548548
- name: Checkout
@@ -566,7 +566,8 @@ jobs:
566566

567567
publish-images:
568568
runs-on: ubuntu-20.04
569-
needs: [lint, unit-test, unit-test-race-detector, artifacts, integration]
569+
needs: [lint, unit-test, unit-test-race-detector, artifacts, integration,
570+
lint-windows, unit-test-windows, artifact-windows, integration-windows]
570571

571572
permissions:
572573
contents: read

0 commit comments

Comments
 (0)