Skip to content

Commit e83876e

Browse files
authored
Initial workflows static analysis fixes (#39097)
* Initial zizmor findings * add rule about action hashes * add comment * fix another workflow * redo build release candidate workflow after change * another fix * add more initial findings * fix more zizmor initial issues after merge conflict resolution * chore: empty commit
1 parent c323a7c commit e83876e

335 files changed

Lines changed: 1047 additions & 303 deletions

File tree

Some content is hidden

Large Commits have some content hidden by default. Use the searchbox below for content that may be hidden.

.github/workflows/IO_Iceberg_Integration_Tests.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,8 @@ jobs:
6464
job_phrase: ["Run IcebergIO Integration Test"]
6565
steps:
6666
- uses: actions/checkout@v7
67+
with:
68+
persist-credentials: false
6769
- name: Setup repository
6870
uses: ./.github/actions/setup-action
6971
with:
@@ -75,4 +77,4 @@ jobs:
7577
- name: Run IcebergIO Integration Test
7678
uses: ./.github/actions/gradle-command-self-hosted-action
7779
with:
78-
gradle-command: :sdks:java:io:iceberg:integrationTest --info
80+
gradle-command: :sdks:java:io:iceberg:integrationTest --info

.github/workflows/IO_Iceberg_Integration_Tests_Dataflow.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,8 @@ jobs:
6464
job_phrase: ["Run IcebergIO Integration Tests on Dataflow"]
6565
steps:
6666
- uses: actions/checkout@v7
67+
with:
68+
persist-credentials: false
6769
- name: Setup repository
6870
uses: ./.github/actions/setup-action
6971
with:
@@ -75,4 +77,4 @@ jobs:
7577
- name: Run IcebergIO Integration Tests on Dataflow
7678
uses: ./.github/actions/gradle-command-self-hosted-action
7779
with:
78-
gradle-command: :sdks:java:io:iceberg:dataflowIntegrationTest --info
80+
gradle-command: :sdks:java:io:iceberg:dataflowIntegrationTest --info

.github/workflows/IO_Iceberg_Managed_Integration_Tests_Dataflow.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,8 @@ jobs:
6464
job_phrase: ["Run IcebergIO Managed Integration Tests on Dataflow"]
6565
steps:
6666
- uses: actions/checkout@v7
67+
with:
68+
persist-credentials: false
6769
- name: Setup repository
6870
uses: ./.github/actions/setup-action
6971
with:
@@ -78,4 +80,4 @@ jobs:
7880
gradle-command: :sdks:java:io:iceberg:dataflowIntegrationTest
7981
arguments: |
8082
--info \
81-
-PenableManagedTransforms
83+
-PenableManagedTransforms

.github/workflows/IO_Iceberg_Performance_Tests.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -64,6 +64,8 @@ jobs:
6464
job_phrase: ["Run IcebergIO Performance Test"]
6565
steps:
6666
- uses: actions/checkout@v7
67+
with:
68+
persist-credentials: false
6769
- name: Setup repository
6870
uses: ./.github/actions/setup-action
6971
with:
@@ -75,4 +77,4 @@ jobs:
7577
- name: Run IcebergIO Performance Test
7678
uses: ./.github/actions/gradle-command-self-hosted-action
7779
with:
78-
gradle-command: :sdks:java:io:iceberg:loadTest
80+
gradle-command: :sdks:java:io:iceberg:loadTest

.github/workflows/IO_Iceberg_Unit_Tests.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -83,6 +83,8 @@ jobs:
8383
runs-on: [self-hosted, ubuntu-24.04, main]
8484
steps:
8585
- uses: actions/checkout@v7
86+
with:
87+
persist-credentials: false
8688
- name: Setup repository
8789
uses: ./.github/actions/setup-action
8890
with:

.github/workflows/assign_milestone.yml

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -34,6 +34,7 @@ jobs:
3434
- uses: actions/checkout@v7
3535
with:
3636
fetch-depth: 2
37+
persist-credentials: false
3738

3839
- uses: actions/github-script@v9
3940
with:

.github/workflows/beam_CancelStaleDataflowJobs.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,8 @@ jobs:
6363
github.event.comment.body == 'Run Cancel Stale Dataflow Jobs'
6464
steps:
6565
- uses: actions/checkout@v7
66+
with:
67+
persist-credentials: false
6668
- name: Setup repository
6769
uses: ./.github/actions/setup-action
6870
with:

.github/workflows/beam_CleanUpDataprocResources.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -56,6 +56,8 @@ jobs:
5656
name: "beam_CleanUpDataprocResources"
5757
steps:
5858
- uses: actions/checkout@v7
59+
with:
60+
persist-credentials: false
5961
- name: Delete leaked resources for all the jobs that generates flink clusters
6062
run: |
61-
cd ${{ github.workspace }}/.test-infra/dataproc; ./cleanup.sh -xe
63+
cd ${{ github.workspace }}/.test-infra/dataproc; ./cleanup.sh -xe

.github/workflows/beam_CleanUpGCPResources.yml

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,8 @@ jobs:
6363
github.event.comment.body == 'Run Clean GCP Resources'
6464
steps:
6565
- uses: actions/checkout@v7
66+
with:
67+
persist-credentials: false
6668
- name: Setup repository
6769
uses: ./.github/actions/setup-action
6870
with:

.github/workflows/beam_CleanUpPrebuiltSDKImages.yml

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,8 @@ jobs:
6363
github.event.comment.body == 'Run Clean Prebuilt Images'
6464
steps:
6565
- uses: actions/checkout@v7
66+
with:
67+
persist-credentials: false
6668
- name: Setup repository
6769
uses: ./.github/actions/setup-action
6870
with:
@@ -80,4 +82,4 @@ jobs:
8082
- name: run remove stale sdk container images
8183
uses: ./.github/actions/gradle-command-self-hosted-action
8284
with:
83-
gradle-command: :beam-test-tools:removeStaleSDKContainerImages
85+
gradle-command: :beam-test-tools:removeStaleSDKContainerImages

0 commit comments

Comments
 (0)