Skip to content

Commit e1998c6

Browse files
Migrate some secrets to vars (#35)
* switch branch * update linter * move DV_AWS_REGION to vars * migrate devops and qms to vars * update super linter * update seuper linter * get DV_AWS_ECR_ROLE * skip image scan * fix lint * migrate remaining * remove logs * revert branches
1 parent a2b316f commit e1998c6

13 files changed

+33
-33
lines changed

.github/actions/lint-github-actions/action.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,7 +9,7 @@ runs:
99
run: |
1010
sudo apt-get install -y shellcheck
1111
echo "::add-matcher::$GITHUB_ACTION_PATH/actionlint-matcher.json"
12-
bash <(curl https://raw.githubusercontent.com/rhysd/actionlint/v1.6.22/scripts/download-actionlint.bash)
12+
bash <(curl https://raw.githubusercontent.com/rhysd/actionlint/v1.6.26/scripts/download-actionlint.bash)
1313
./actionlint -color
1414
shell: bash
1515

.github/workflows/build-product.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -129,8 +129,8 @@ jobs:
129129
- name: retag images
130130
uses: ./tmp/github-workflows/.github/actions/retag-image
131131
with:
132-
aws_role: ${{ secrets.DV_AWS_ECR_ROLE }}
133-
aws_region: ${{ secrets.DV_AWS_REGION }}
132+
aws_role: ${{ vars.DV_AWS_ECR_ROLE }}
133+
aws_region: ${{ vars.DV_AWS_REGION }}
134134
ecr_repositories: ${{ needs.prepare-build.outputs.ecr_repos }}
135135
current_image_tag: ${{ needs.prepare-build.outputs.image_tag1 }}
136136
additional_image_tag: ${{ needs.prepare-build.outputs.image_tag2 }}

.github/workflows/build-push-docker.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -53,9 +53,9 @@ jobs:
5353
path: ./tmp/github-workflows
5454
- uses: ./tmp/github-workflows/.github/actions/build-push-image
5555
with:
56-
aws_role: ${{ secrets.DV_AWS_ECR_ROLE }}
57-
aws_region: ${{ secrets.DV_AWS_REGION }}
58-
ecr_registry: ${{ secrets.DV_AWS_ECR_REGISTRY }}
56+
aws_role: ${{ vars.DV_AWS_ECR_ROLE }}
57+
aws_region: ${{ vars.DV_AWS_REGION }}
58+
ecr_registry: ${{ vars.DV_AWS_ECR_REGISTRY }}
5959
ecr_repository: ${{ inputs.ecr_repository || secrets.DV_ECR_REPOSITORY }}
6060
docker_file: ${{ inputs.docker_file }}
6161
current_directory: ${{ inputs.current_directory }}

.github/workflows/build-push-helm-chart.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ jobs:
4747
path: ./tmp/github-workflows
4848
- uses: ./tmp/github-workflows/.github/actions/build-push-helm-chart
4949
with:
50-
chart_repository_url: ${{ inputs.chart_repository_url || secrets.DV_CHARTMUSEUM_URL }}
50+
chart_repository_url: ${{ inputs.chart_repository_url || vars.DV_CHARTMUSEUM_URL }}
5151
chart_repository_username: ${{ secrets.DV_CHARTMUSEUM_USER }}
5252
chart_repository_password: ${{ secrets.DV_CHARTMUSEUM_PASSWORD }}
5353
current_directory: ${{ inputs.current_directory }}

.github/workflows/build-single-product-part.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -223,9 +223,9 @@ jobs:
223223
# checkout this workflow repository to get actions
224224
- uses: ./tmp/github-workflows/.github/actions/build-push-image
225225
with:
226-
aws_role: ${{ secrets.DV_AWS_ECR_ROLE }}
227-
aws_region: ${{ secrets.DV_AWS_REGION }}
228-
ecr_registry: ${{ secrets.DV_AWS_ECR_REGISTRY }}
226+
aws_role: ${{ vars.DV_AWS_ECR_ROLE }}
227+
aws_region: ${{ vars.DV_AWS_REGION }}
228+
ecr_registry: ${{ vars.DV_AWS_ECR_REGISTRY }}
229229
ecr_repository: ${{ steps.get-parameters.outputs.ecr_repo }}
230230
docker_file: ./tmp/${{ inputs.component }}/${{ steps.get-parameters.outputs.app }}/docker/Dockerfile
231231
current_directory: ./tmp/${{ inputs.component }}/${{ steps.get-parameters.outputs.app }}
@@ -250,9 +250,9 @@ jobs:
250250
id: get-ecr-scan-result
251251
uses: ./tmp/github-workflows/.github/actions/get-ecr-scan-result
252252
with:
253-
aws_role: ${{ secrets.DV_AWS_ECR_ROLE }}
254-
aws_region: ${{ secrets.DV_AWS_REGION }}
255-
ecr_registry: ${{ secrets.DV_AWS_ECR_REGISTRY }}
253+
aws_role: ${{ vars.DV_AWS_ECR_ROLE }}
254+
aws_region: ${{ vars.DV_AWS_REGION }}
255+
ecr_registry: ${{ vars.DV_AWS_ECR_REGISTRY }}
256256
ecr_repository: ${{ steps.get-parameters.outputs.ecr_repo }}
257257
image_tag: ${{ inputs.image_tag1 }}
258258
- name: check scan results

.github/workflows/build-workspace-product-part.yml

Lines changed: 6 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -294,9 +294,9 @@ jobs:
294294
# checkout this workflow repository to get actions
295295
- uses: ./tmp/github-workflows/.github/actions/build-push-image
296296
with:
297-
aws_role: ${{ secrets.DV_AWS_ECR_ROLE }}
298-
aws_region: ${{ secrets.DV_AWS_REGION }}
299-
ecr_registry: ${{ secrets.DV_AWS_ECR_REGISTRY }}
297+
aws_role: ${{ vars.DV_AWS_ECR_ROLE }}
298+
aws_region: ${{ vars.DV_AWS_REGION }}
299+
ecr_registry: ${{ vars.DV_AWS_ECR_REGISTRY }}
300300
ecr_repository: ${{ steps.get-parameters.outputs.ecr_repo }}
301301
docker_file: ./tmp/${{ inputs.component }}/docker/Dockerfile
302302
current_directory: ./tmp/${{ inputs.component }}
@@ -321,9 +321,9 @@ jobs:
321321
id: get-ecr-scan-result
322322
uses: ./tmp/github-workflows/.github/actions/get-ecr-scan-result
323323
with:
324-
aws_role: ${{ secrets.DV_AWS_ECR_ROLE }}
325-
aws_region: ${{ secrets.DV_AWS_REGION }}
326-
ecr_registry: ${{ secrets.DV_AWS_ECR_REGISTRY }}
324+
aws_role: ${{ vars.DV_AWS_ECR_ROLE }}
325+
aws_region: ${{ vars.DV_AWS_REGION }}
326+
ecr_registry: ${{ vars.DV_AWS_ECR_REGISTRY }}
327327
ecr_repository: ${{ steps.get-parameters.outputs.ecr_repo }}
328328
image_tag: ${{ inputs.image_tag1 }}
329329
- name: check scan results

.github/workflows/deploy-product.yml

Lines changed: 5 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -66,8 +66,8 @@ jobs:
6666
path: ./tmp/github-workflows
6767
- uses: ./tmp/github-workflows/.github/actions/check-actor
6868
with:
69-
dv_devops: ${{ secrets.DV_DEVOPS }}
70-
dv_qms: ${{ secrets.DV_QMS }}
69+
dv_devops: ${{ vars.DV_DEVOPS }}
70+
dv_qms: ${{ vars.DV_QMS }}
7171
actor: ${{ github.actor }}
7272
qms_are_allowed: "true"
7373
- uses: ./tmp/github-workflows/.github/actions/get-product-parameters
@@ -90,9 +90,9 @@ jobs:
9090
if: ${{ inputs.add_revision_as_tag == true }}
9191
uses: aws-actions/[email protected]
9292
with:
93-
role-to-assume: ${{ secrets.DV_AWS_ECR_ROLE }}
94-
aws_role: ${{ secrets.DV_AWS_ECR_ROLE }}
95-
aws-region: ${{ secrets.DV_AWS_REGION }}
93+
role-to-assume: ${{ vars.DV_AWS_ECR_ROLE }}
94+
aws_role: ${{ vars.DV_AWS_ECR_ROLE }}
95+
aws-region: ${{ vars.DV_AWS_REGION }}
9696
- name: Login to Amazon ECR
9797
if: ${{ inputs.add_revision_as_tag == true }}
9898
id: login-ecr

.github/workflows/lint.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -23,7 +23,7 @@ jobs:
2323
# Run Linter against code base #
2424
################################
2525
- name: Lint Code Base
26-
uses: github/super-linter/slim@v4
26+
uses: super-linter/super-linter/slim@v5
2727
env:
2828
VALIDATE_ALL_CODEBASE: false
2929
VALIDATE_BASH: true

.github/workflows/publish-node-python.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -57,7 +57,7 @@ jobs:
5757
path: ./tmp/github-workflows
5858
- uses: ./tmp/github-workflows/.github/actions/check-actor
5959
with:
60-
dv_devops: ${{ secrets.DV_DEVOPS }}
60+
dv_devops: ${{ vars.DV_DEVOPS }}
6161
actor: ${{ github.actor }}
6262
qms_are_allowed: "false"
6363
- uses: ./tmp/github-workflows/.github/actions/build-node
@@ -87,7 +87,7 @@ jobs:
8787
path: ./tmp/github-workflows
8888
- uses: ./tmp/github-workflows/.github/actions/check-actor
8989
with:
90-
dv_devops: ${{ secrets.DV_DEVOPS }}
90+
dv_devops: ${{ vars.DV_DEVOPS }}
9191
actor: ${{ github.actor }}
9292
qms_are_allowed: "false"
9393
- uses: ./tmp/github-workflows/.github/actions/build-python

.github/workflows/publish-node.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -48,7 +48,7 @@ jobs:
4848
path: ./tmp/github-workflows
4949
- uses: ./tmp/github-workflows/.github/actions/check-actor
5050
with:
51-
dv_devops: ${{ secrets.DV_DEVOPS }}
51+
dv_devops: ${{ vars.DV_DEVOPS }}
5252
actor: ${{ github.actor }}
5353
qms_are_allowed: "false"
5454
- uses: ./tmp/github-workflows/.github/actions/build-node

0 commit comments

Comments
 (0)