Skip to content

Commit

Permalink
Keep working on environment secrets
Browse files Browse the repository at this point in the history
  • Loading branch information
DerekRoberts committed Mar 1, 2024
1 parent 17ea3e1 commit 8879daf
Show file tree
Hide file tree
Showing 2 changed files with 6 additions and 5 deletions.
9 changes: 5 additions & 4 deletions .github/workflows/.deployer.yml
Original file line number Diff line number Diff line change
Expand Up @@ -4,6 +4,10 @@ on:
workflow_call:
inputs:
### Required
oc_namespace:
description: OpenShift namespace
required: true
type: string
release:
description: Deployment release; usually repo-name-[PR|test|prod]
required: true
Expand Down Expand Up @@ -56,9 +60,6 @@ on:
value: ${{ jobs.deployer.outputs.triggered }}

secrets:
oc_namespace:
description: OpenShift namespace
required: true
oc_token:
description: OpenShift token
required: true
Expand Down Expand Up @@ -111,7 +112,7 @@ jobs:
- run: |
# OC Login
oc login --token=${{ secrets.oc_token }} --server=${{ inputs.oc_server }}
oc project ${{ secrets.oc_namespace }} # Safeguard!
oc project ${{ inputs.oc_namespace }} # Safeguard!
# Stop pre-existing deployments
- run: |
Expand Down
2 changes: 1 addition & 1 deletion .github/workflows/merge.yml
Original file line number Diff line number Diff line change
Expand Up @@ -34,10 +34,10 @@ jobs:
needs: [vars]
uses: ./.github/workflows/.deployer.yml
secrets:
oc_namespace: ${{ vars.oc_namespace }}
oc_token: ${{ secrets.oc_token }}
with:
environment: test
oc_namespace: ${{ vars.oc_namespace }}
params: --set global.autoscaling=false
release: test
tag: ${{ needs.vars.outputs.pr }}
Expand Down

0 comments on commit 8879daf

Please sign in to comment.