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

DTSPO-8837 - Azurerm v3 provider backend app gw #204

Merged
merged 4 commits into from
Jun 20, 2022

Conversation

endakelly
Copy link
Contributor

@endakelly endakelly commented Jun 15, 2022

JIRA link (if applicable)

https://tools.hmcts.net/jira/browse/DTSPO-8837

Change description

Use azurerm v3 provider for backend app gw
Once azure-platform-terraform repo has also been updated, the branches on the modules will be merged to master and the references to them in this repo will be reverted to master.

Does this PR introduce a breaking change? (check one with "x")

[ ] Yes
[x] No

@endakelly endakelly marked this pull request as ready for review June 16, 2022 10:40
@endakelly endakelly requested a review from a team as a code owner June 16, 2022 10:40
@@ -11,25 +11,33 @@ gateways:
# Toffee application
- product: toffee
component: recipe-backend
priority: 10
Copy link
Contributor

Choose a reason for hiding this comment

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

what's this?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This has to be set in the latest azurerm api for some reason. In previous versions this is set automatically but now it has to be done manually. Haven't yet figured out how to generate the figure dynamically so it doesn't need to be coded in the vars
hashicorp/terraform-provider-azurerm#17079

Copy link
Contributor

Choose a reason for hiding this comment

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

is there some way to get an index and then add 10?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Matt sent me a code snippet that I'm going to look to adapt for this

@endakelly endakelly requested a review from timja June 20, 2022 08:17
@endakelly endakelly merged commit 511b5e9 into master Jun 20, 2022
@endakelly endakelly deleted the DTSPO-8837/azurerm-v3-backendappgw branch June 20, 2022 08:27
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