Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable gcp wif in review env #424

Merged
merged 1 commit into from
Nov 26, 2024
Merged

Enable gcp wif in review env #424

merged 1 commit into from
Nov 26, 2024

Conversation

RMcVelia
Copy link
Contributor

@RMcVelia RMcVelia commented Nov 25, 2024

Context

Enable GCP WIF for the review environment

Changes proposed in this pull request

Add required config

Guidance to review

see https://github.com/DFE-Digital/check-childrens-barred-list/actions/runs/12014379903
make review terraform-plan APP_NAME=pr-nnnn
make test terraform-plan (no change)

Link to Trello card

https://trello.com/c/zBJkGucG/2142-ccbl-migrate-to-gcp-wif

Checklist

  • Attach to Trello card
  • Rebased main
  • Cleaned commit history
  • Tested by running locally

@RMcVelia RMcVelia added the deploy Deploys a review app label Nov 25, 2024
@github-actions github-actions bot temporarily deployed to review-424 November 25, 2024 16:03 Destroyed
@github-actions github-actions bot temporarily deployed to review-424 November 25, 2024 16:26 Destroyed
@RMcVelia
Copy link
Contributor Author

CCBL: migrate to GCP WIF

@github-actions github-actions bot temporarily deployed to review-424 November 26, 2024 09:13 Destroyed
@github-actions github-actions bot temporarily deployed to review-424 November 26, 2024 09:50 Destroyed
@vipin-dfe
Copy link

lgtm 👍

@RMcVelia RMcVelia merged commit b835a42 into main Nov 26, 2024
12 checks passed
@RMcVelia RMcVelia deleted the 2142-ccbl-migrate-to-gcp-wif branch November 26, 2024 10:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy Deploys a review app devops
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants