Skip to content

Commit 3e17e84

Browse files
committed
ci: migrate to hardened runners, disable publish during freeze
Switch all 7 workflow jobs from `ubuntu-latest` to the `databricks-protected-runner-group` hardened runner group per go/hardened-gha step 3. Disable the release publish job during the release freeze per go/hardened-gha step 7. The build job remains active for validation. A clear comment marks when and how to re-enable. Fix `.npmrc` from `package-lock=false` to `package-lock=true` so local dev keeps the lockfile in sync with `npm ci` in CI. Co-authored-by: Isaac
1 parent 95fee42 commit 3e17e84

File tree

4 files changed

+41
-25
lines changed

4 files changed

+41
-25
lines changed

.github/workflows/dco-check.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -8,7 +8,9 @@ permissions:
88

99
jobs:
1010
check:
11-
runs-on: ubuntu-latest
11+
runs-on:
12+
group: databricks-protected-runner-group
13+
labels: linux-ubuntu-latest
1214
steps:
1315
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
1416
with:

.github/workflows/main.yml

Lines changed: 12 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,9 @@ permissions:
1313

1414
jobs:
1515
lint:
16-
runs-on: ubuntu-latest
16+
runs-on:
17+
group: databricks-protected-runner-group
18+
labels: linux-ubuntu-latest
1719
steps:
1820
- uses: actions/checkout@34e114876b0b11c390a56381ad16ebd13914f8d5 # v4
1921
- name: Cache node modules
@@ -34,7 +36,9 @@ jobs:
3436
npm run lint
3537
3638
unit-test:
37-
runs-on: ubuntu-latest
39+
runs-on:
40+
group: databricks-protected-runner-group
41+
labels: linux-ubuntu-latest
3842
strategy:
3943
matrix:
4044
# only LTS versions starting from the lowest we support
@@ -75,7 +79,9 @@ jobs:
7579
retention-days: 1
7680

7781
e2e-test:
78-
runs-on: ubuntu-latest
82+
runs-on:
83+
group: databricks-protected-runner-group
84+
labels: linux-ubuntu-latest
7985
environment: azure-prod
8086
env:
8187
E2E_HOST: ${{ secrets.DATABRICKS_HOST }}
@@ -113,7 +119,9 @@ jobs:
113119

114120
coverage:
115121
needs: [unit-test, e2e-test]
116-
runs-on: ubuntu-latest
122+
runs-on:
123+
group: databricks-protected-runner-group
124+
labels: linux-ubuntu-latest
117125
env:
118126
cache-name: cache-node-modules
119127

.github/workflows/release.yml

Lines changed: 25 additions & 19 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,9 @@ on:
66

77
jobs:
88
build:
9-
runs-on: ubuntu-latest
9+
runs-on:
10+
group: databricks-protected-runner-group
11+
labels: linux-ubuntu-latest
1012
permissions:
1113
contents: read
1214
steps:
@@ -22,21 +24,25 @@ jobs:
2224
path: "*.tgz"
2325
retention-days: 1
2426

25-
publish:
26-
needs: [build]
27-
runs-on: ubuntu-latest
28-
environment: npm-publish
29-
permissions:
30-
contents: read
31-
id-token: write
32-
steps:
33-
- uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4
34-
with:
35-
node-version: 22
36-
registry-url: https://registry.npmjs.org/
37-
- uses: actions/download-artifact@d3f86a106a0bac45b974a628896c90dbdf5c8093 # v4
38-
with:
39-
name: package-tarball
40-
- run: npm publish --provenance --access public *.tgz
41-
env:
42-
NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}
27+
# NOTE: Publish job disabled during release freeze per go/hardened-gha step 7.
28+
# Re-enable after completing NPM Release SOP and #unblock-releases-public approval.
29+
# publish:
30+
# needs: [build]
31+
# runs-on:
32+
# group: databricks-protected-runner-group
33+
# labels: linux-ubuntu-latest
34+
# environment: npm-publish
35+
# permissions:
36+
# contents: read
37+
# id-token: write
38+
# steps:
39+
# - uses: actions/setup-node@49933ea5288caeca8642d1e84afbd3f7d6820020 # v4
40+
# with:
41+
# node-version: 22
42+
# registry-url: https://registry.npmjs.org/
43+
# - uses: actions/download-artifact@d3f86a106a0bac45b974a628896c90dbdf5c8093 # v4
44+
# with:
45+
# name: package-tarball
46+
# - run: npm publish --provenance --access public *.tgz
47+
# env:
48+
# NODE_AUTH_TOKEN: ${{ secrets.NPM_TOKEN }}

.npmrc

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1 +1 @@
1-
package-lock=false
1+
package-lock=true

0 commit comments

Comments
 (0)