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

SSL mode value set based in use_azure flag #107

Merged
merged 1 commit into from
Jul 1, 2024

Conversation

VenkataChalla554
Copy link
Contributor

@VenkataChalla554 VenkataChalla554 commented Jul 1, 2024

Context

Removed hard code value of SSL Mode.

Changes proposed in this pull request

SSL Mode Value set based on user_azure.

Guidance to review

Deployed review app using this branch : https://github.com/DFE-Digital/teaching-record-system/tree/1813-standardise-logging-terraform-module-test.

make dv_review terraform-apply CLUSTER=cluster2 IMAGE=ghcr.io/dfe-digital/teaching-record-system:b96c149d3b2db5f4ce4ddb29928cfec6af0e286a APP_NAME=2222

Checklist

  • I have performed a self-review of my code, including formatting and typos
  • I have cleaned the commit history
  • I have added the Devops label
  • I have attached the pull request to the trello card

@VenkataChalla554
Copy link
Contributor Author

aks/postgres/outputs.tf Outdated Show resolved Hide resolved
aks/postgres/outputs.tf Outdated Show resolved Hide resolved
@VenkataChalla554 VenkataChalla554 force-pushed the 1813-standardise-logging-fields branch from 4280d3c to 4c0da0b Compare July 1, 2024 12:52
@VenkataChalla554 VenkataChalla554 force-pushed the 1813-standardise-logging-fields branch from 4c0da0b to b91199b Compare July 1, 2024 13:31
@VenkataChalla554 VenkataChalla554 merged commit b2bc395 into main Jul 1, 2024
3 checks passed
@VenkataChalla554 VenkataChalla554 deleted the 1813-standardise-logging-fields branch July 1, 2024 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants