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

Last set of tweaks for RMI Azure deployment #213

Merged
merged 2 commits into from
Nov 16, 2024
Merged

Conversation

bcspragu
Copy link
Collaborator

@bcspragu bcspragu commented Oct 3, 2024

  • Creates/updates relevant configs (mostly frontend)
  • Moves our GH Actions deploy workflow to point at RMI's Azure

- Creates/updates relevant configs (mostly frontend)
- Moves our GH Actions deploy workflow to point at RMI's Azure
@bcspragu bcspragu requested a review from gbdubs October 3, 2024 03:26
imageToDeploy: rmisa.azurecr.io/pacta:dev
containerAppName: pactasrv-test
containerAppEnvironment: pacta-test
resourceGroup: RMI-SP-PACTA-WEU-PAT-DEV
Copy link
Contributor

Choose a reason for hiding this comment

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

*Test?

Copy link
Contributor

Choose a reason for hiding this comment

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

Same next line?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This is correct, it's just an unfortunate mismatch of names. I did the same for OPGEE, the rational is that we already had an rmi-{pacta,opgee}-dev for our SA environment, so the RMI deployments are named test to disambiguate them. They are also test environments in a sense, but yeah the resource group names are -DEV and -PROD

@bcspragu bcspragu merged commit 4dc23b1 into main Nov 16, 2024
2 checks passed
@bcspragu bcspragu deleted the brandon/rmi-deploy branch November 16, 2024 19:19
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.

2 participants