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

RAS-1306: Migrate Party from GCR to GAR #439

Merged
merged 6 commits into from
Nov 4, 2024
Merged

Conversation

SteveScorfield
Copy link
Contributor

@SteveScorfield SteveScorfield commented Oct 29, 2024

What and Why?

We are required to migrate across our Docker images from GCR to GAR. This PR migrated party over to GAR.

How to test?

Build out your env
deploy frontstage using this PR and migrate-frontstage-to-GAR config branch
update the image location in the helm deployment to point to the GAR - currently pointing at the management one to prevent issues with other PRs: europe-west2-docker.pkg.dev/management project id/images

Jira

https://jira.ons.gov.uk/browse/RAS-1306

@SteveScorfield
Copy link
Contributor Author

/deploy scorfs

@ras-rm-pr-bot
Copy link
Collaborator

Deploying to dev cluster with following parameters:

  • namespace: scorfs

  • tag: pr-439

  • configBranch: main

  • paramKey: ``

  • paramValue: ``

Copy link
Contributor

@mfcrocker mfcrocker left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

One small improvement

.github/workflows/main.yml Outdated Show resolved Hide resolved
Copy link
Contributor

@matthew-robinson-ons matthew-robinson-ons left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Deployed and updated yaml. all working and code changes look good.

@SteveScorfield SteveScorfield merged commit a230598 into main Nov 4, 2024
1 check passed
@SteveScorfield SteveScorfield deleted the migrate-party-to-gar branch November 4, 2024 10:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants