Skip to content

Commit

Permalink
NRL-817 Do not store permissions in artifact
Browse files Browse the repository at this point in the history
  • Loading branch information
axelkrastek1-nhs committed Jul 1, 2024
1 parent 58eca28 commit 99747ba
Show file tree
Hide file tree
Showing 2 changed files with 39 additions and 2 deletions.
24 changes: 23 additions & 1 deletion .github/workflows/persistent-environment.yml
Original file line number Diff line number Diff line change
Expand Up @@ -74,7 +74,15 @@ jobs:
uses: actions/upload-artifact@v4
with:
name: build-artifacts
path: dist/*.zip
path: |
dist/*.zip
!dist/nrlf-permissions.zip
- name: Save NRLF Permissions cache
uses: actions/cache/save@v4
with:
key: ${{ github.run_id }}-nrlf-permissions
path: dist/nrlf-permissions.zip

terraform-plan:
name: Terraform Plan - ${{ inputs.environment }}
Expand Down Expand Up @@ -117,6 +125,13 @@ jobs:
name: build-artifacts
path: dist

- name: Restore NRLF permissions cache
uses: actions/cache/restore@v4
with:
key: ${{ github.run_id }}-nrlf-permissions
path: dist/nrlf-permissions.zip
fail-on-cache-miss: true

- name: Terraform Init
run: |
terraform -chdir=terraform/infrastructure init
Expand Down Expand Up @@ -165,6 +180,13 @@ jobs:
name: build-artifacts
path: dist

- name: Restore NRLF permissions cache
uses: actions/cache/restore@v4
with:
key: ${{ github.run_id }}-nrlf-permissions
path: dist/nrlf-permissions.zip
fail-on-cache-miss: true

- name: Configure Management Credentials
uses: aws-actions/configure-aws-credentials@v4
with:
Expand Down
17 changes: 16 additions & 1 deletion .github/workflows/pr-env-deploy.yml
Original file line number Diff line number Diff line change
Expand Up @@ -93,7 +93,15 @@ jobs:
uses: actions/upload-artifact@v4
with:
name: build-artifacts
path: dist/*.zip
path: |
dist/*.zip
!dist/nrlf-permissions.zip
- name: Save NRLF Permissions cache
uses: actions/cache/save@v4
with:
key: ${{ github.run_id }}-nrlf-permissions
path: dist/nrlf-permissions.zip

- name: Add Failure Pull Request Comment
uses: actions/github-script@v7
Expand Down Expand Up @@ -143,6 +151,13 @@ jobs:
name: build-artifacts
path: dist

- name: Restore NRLF permissions cache
uses: actions/cache/restore@v4
with:
key: ${{ github.run_id }}-nrlf-permissions
path: dist/nrlf-permissions.zip
fail-on-cache-miss: true

- name: Retrieve Server Certificates
run: make truststore-pull-server ENV=dev

Expand Down

0 comments on commit 99747ba

Please sign in to comment.