Skip to content

Commit

Permalink
Update CI to inherit secrets between workflows (#335)
Browse files Browse the repository at this point in the history
By default called workflows don't have access to secrets, this
causes issues when needing access to our dockerhib secrets for push.

This adds `secrets: inherit` to all local workflow calls

Signed-off-by: Alex Bozarth <ajbozart@us.ibm.com>
  • Loading branch information
ajbozarth authored Dec 18, 2024
1 parent dfb9248 commit 9758c50
Show file tree
Hide file tree
Showing 14 changed files with 32 additions and 6 deletions.
11 changes: 5 additions & 6 deletions .github/workflow-templates/template.yml
Original file line number Diff line number Diff line change
Expand Up @@ -92,9 +92,8 @@ jobs:
push:
if: ${{ github.repository == 'open-quantum-safe/oqs-demos' && github.ref == 'refs/heads/main' && github.event_name != 'pull_request' && inputs.build_main != 'true' }}
needs: build
runs-on: ubuntu-latest
steps:
- uses: ./.github/workflows/manifest.yml
with:
image_name: <demo_name>
release_tag: ${{ inputs.release_tag || 'latest' }}
uses: ./.github/workflows/push-manifest.yml
secrets: inherit
with:
image_name: <demo_name>
release_tag: ${{ inputs.release_tag || 'latest' }}
12 changes: 12 additions & 0 deletions .github/workflows/build.yml
Original file line number Diff line number Diff line change
Expand Up @@ -29,72 +29,84 @@ on:
jobs:
curl:
uses: ./.github/workflows/curl.yml
secrets: inherit
with:
build_main: ${{ !contains(inputs.build_main == 'true', 'false') }}
release_tag: ${{ inputs.release_tag }}

h2load:
uses: ./.github/workflows/h2load.yml
secrets: inherit
with:
build_main: ${{ !contains(inputs.build_main == 'true', 'false') }}
release_tag: ${{ inputs.release_tag }}

haproxy:
uses: ./.github/workflows/haproxy.yml
secrets: inherit
with:
build_main: ${{ !contains(inputs.build_main == 'true', 'false') }}
release_tag: ${{ inputs.release_tag }}

httpd:
uses: ./.github/workflows/httpd.yml
secrets: inherit
with:
build_main: ${{ !contains(inputs.build_main == 'true', 'false') }}
release_tag: ${{ inputs.release_tag }}

locust:
uses: ./.github/workflows/locust.yml
secrets: inherit
with:
build_main: ${{ !contains(inputs.build_main == 'true', 'false') }}
release_tag: ${{ inputs.release_tag }}

mosquitto:
uses: ./.github/workflows/mosquitto.yml
secrets: inherit
with:
build_main: ${{ !contains(inputs.build_main == 'true', 'false') }}
release_tag: ${{ inputs.release_tag }}

nginx:
uses: ./.github/workflows/nginx.yml
secrets: inherit
with:
build_main: ${{ !contains(inputs.build_main == 'true', 'false') }}
release_tag: ${{ inputs.release_tag }}

ngtcp2:
uses: ./.github/workflows/ngtcp2.yml
secrets: inherit
with:
build_main: ${{ !contains(inputs.build_main == 'true', 'false') }}
release_tag: ${{ inputs.release_tag }}

openssh:
uses: ./.github/workflows/openssh.yml
secrets: inherit
with:
build_main: ${{ !contains(inputs.build_main == 'true', 'false') }}
release_tag: ${{ inputs.release_tag }}

openssl3:
uses: ./.github/workflows/openssl3.yml
secrets: inherit
with:
build_main: ${{ !contains(inputs.build_main == 'true', 'false') }}
release_tag: ${{ inputs.release_tag }}

openvpn:
uses: ./.github/workflows/openvpn.yml
secrets: inherit
with:
build_main: ${{ !contains(inputs.build_main == 'true', 'false') }}
release_tag: ${{ inputs.release_tag }}

wireshark:
uses: ./.github/workflows/wireshark.yml
secrets: inherit
with:
build_main: ${{ !contains(inputs.build_main == 'true', 'false') }}
release_tag: ${{ inputs.release_tag }}
3 changes: 3 additions & 0 deletions .github/workflows/curl.yml
Original file line number Diff line number Diff line change
Expand Up @@ -142,6 +142,7 @@ jobs:
if: ${{ github.repository == 'open-quantum-safe/oqs-demos' && github.ref == 'refs/heads/main' && github.event_name != 'pull_request' && inputs.build_main != 'true' }}
needs: build
uses: ./.github/workflows/push-manifest.yml
secrets: inherit
with:
image_name: curl
release_tag: optimized
Expand All @@ -150,6 +151,7 @@ jobs:
if: ${{ github.repository == 'open-quantum-safe/oqs-demos' && github.ref == 'refs/heads/main' && github.event_name != 'pull_request' && inputs.build_main != 'true' }}
needs: build
uses: ./.github/workflows/push-manifest.yml
secrets: inherit
with:
image_name: curl-dev
release_tag: latest
Expand All @@ -158,6 +160,7 @@ jobs:
if: ${{ github.repository == 'open-quantum-safe/oqs-demos' && github.ref == 'refs/heads/main' && github.event_name != 'pull_request' && inputs.build_main != 'true' }}
needs: build
uses: ./.github/workflows/push-manifest.yml
secrets: inherit
with:
image_name: curl
release_tag: ${{ inputs.release_tag || 'latest' }}
1 change: 1 addition & 0 deletions .github/workflows/h2load.yml
Original file line number Diff line number Diff line change
Expand Up @@ -101,6 +101,7 @@ jobs:
if: ${{ github.repository == 'open-quantum-safe/oqs-demos' && github.ref == 'refs/heads/main' && github.event_name != 'pull_request' && inputs.build_main != 'true' }}
needs: build
uses: ./.github/workflows/push-manifest.yml
secrets: inherit
with:
image_name: h2load
release_tag: ${{ inputs.release_tag || 'latest' }}
1 change: 1 addition & 0 deletions .github/workflows/haproxy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -104,6 +104,7 @@ jobs:
if: ${{ github.repository == 'open-quantum-safe/oqs-demos' && github.ref == 'refs/heads/main' && github.event_name != 'pull_request' && inputs.build_main != 'true' }}
needs: build
uses: ./.github/workflows/push-manifest.yml
secrets: inherit
with:
image_name: haproxy
release_tag: ${{ inputs.release_tag || 'latest' }}
1 change: 1 addition & 0 deletions .github/workflows/httpd.yml
Original file line number Diff line number Diff line change
Expand Up @@ -104,6 +104,7 @@ jobs:
if: ${{ github.repository == 'open-quantum-safe/oqs-demos' && github.ref == 'refs/heads/main' && github.event_name != 'pull_request' && inputs.build_main != 'true' }}
needs: build
uses: ./.github/workflows/push-manifest.yml
secrets: inherit
with:
image_name: httpd
release_tag: ${{ inputs.release_tag || 'latest' }}
1 change: 1 addition & 0 deletions .github/workflows/locust.yml
Original file line number Diff line number Diff line change
Expand Up @@ -128,6 +128,7 @@ jobs:
if: ${{ github.repository == 'open-quantum-safe/oqs-demos' && github.ref == 'refs/heads/main' && github.event_name != 'pull_request' && inputs.build_main != 'true' }}
needs: build
uses: ./.github/workflows/push-manifest.yml
secrets: inherit
with:
image_name: locust
release_tag: ${{ inputs.release_tag || 'latest' }}
1 change: 1 addition & 0 deletions .github/workflows/mosquitto.yml
Original file line number Diff line number Diff line change
Expand Up @@ -95,6 +95,7 @@ jobs:
if: ${{ github.repository == 'open-quantum-safe/oqs-demos' && github.ref == 'refs/heads/main' && github.event_name != 'pull_request' && inputs.build_main != 'true' }}
needs: build
uses: ./.github/workflows/push-manifest.yml
secrets: inherit
with:
image_name: mosquitto
release_tag: ${{ inputs.release_tag || 'latest' }}
1 change: 1 addition & 0 deletions .github/workflows/nginx.yml
Original file line number Diff line number Diff line change
Expand Up @@ -105,6 +105,7 @@ jobs:
if: ${{ github.repository == 'open-quantum-safe/oqs-demos' && github.ref == 'refs/heads/main' && github.event_name != 'pull_request' && inputs.build_main != 'true' }}
needs: build
uses: ./.github/workflows/push-manifest.yml
secrets: inherit
with:
image_name: nginx
release_tag: ${{ inputs.release_tag || 'latest' }}
2 changes: 2 additions & 0 deletions .github/workflows/ngtcp2.yml
Original file line number Diff line number Diff line change
Expand Up @@ -121,6 +121,7 @@ jobs:
if: ${{ github.repository == 'open-quantum-safe/oqs-demos' && github.ref == 'refs/heads/main' && github.event_name != 'pull_request' && inputs.build_main != 'true' }}
needs: build
uses: ./.github/workflows/push-manifest.yml
secrets: inherit
with:
image_name: ngtcp2-server
release_tag: ${{ inputs.release_tag || 'latest' }}
Expand All @@ -129,6 +130,7 @@ jobs:
if: ${{ github.repository == 'open-quantum-safe/oqs-demos' && github.ref == 'refs/heads/main' && github.event_name != 'pull_request' && inputs.build_main != 'true' }}
needs: build
uses: ./.github/workflows/push-manifest.yml
secrets: inherit
with:
image_name: ngtcp2-client
release_tag: ${{ inputs.release_tag || 'latest' }}
1 change: 1 addition & 0 deletions .github/workflows/openssh.yml
Original file line number Diff line number Diff line change
Expand Up @@ -91,6 +91,7 @@ jobs:
if: ${{ github.repository == 'open-quantum-safe/oqs-demos' && github.ref == 'refs/heads/main' && github.event_name != 'pull_request' && inputs.build_main != 'true' }}
needs: build
uses: ./.github/workflows/push-manifest.yml
secrets: inherit
with:
image_name: openssh
release_tag: ${{ inputs.release_tag || 'latest' }}
1 change: 1 addition & 0 deletions .github/workflows/openssl3.yml
Original file line number Diff line number Diff line change
Expand Up @@ -92,6 +92,7 @@ jobs:
if: ${{ github.repository == 'open-quantum-safe/oqs-demos' && github.ref == 'refs/heads/main' && github.event_name != 'pull_request' && inputs.build_main != 'true' }}
needs: build
uses: ./.github/workflows/push-manifest.yml
secrets: inherit
with:
image_name: openssl3
release_tag: ${{ inputs.release_tag || 'latest' }}
1 change: 1 addition & 0 deletions .github/workflows/openvpn.yml
Original file line number Diff line number Diff line change
Expand Up @@ -92,6 +92,7 @@ jobs:
if: ${{ github.repository == 'open-quantum-safe/oqs-demos' && github.ref == 'refs/heads/main' && github.event_name != 'pull_request' && inputs.build_main != 'true' }}
needs: build
uses: ./.github/workflows/push-manifest.yml
secrets: inherit
with:
image_name: openvpn
release_tag: ${{ inputs.release_tag || 'latest' }}
1 change: 1 addition & 0 deletions .github/workflows/wireshark.yml
Original file line number Diff line number Diff line change
Expand Up @@ -85,6 +85,7 @@ jobs:
if: ${{ github.repository == 'open-quantum-safe/oqs-demos' && github.ref == 'refs/heads/main' && github.event_name != 'pull_request' && inputs.build_main != 'true' }}
needs: build
uses: ./.github/workflows/push-manifest.yml
secrets: inherit
with:
image_name: wireshark
release_tag: ${{ inputs.release_tag || 'latest' }}

0 comments on commit 9758c50

Please sign in to comment.