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

TES-358: Added Key Vault NACLs #17

Merged
merged 1 commit into from
Nov 27, 2023
Merged

Conversation

mihailradkov
Copy link
Contributor

@mihailradkov mihailradkov commented Nov 23, 2023

Changes

  • Added key vault service endpoints to the gateway and vmss subnets
  • Configured the key vault with NACLs to restrict the access only to the virtual network and given management CIDRs
  • Fixed the graphdb password configuration secret to reference the correct value
  • Organized the backup module
  • Added the resource name prefix to the custom roles to avoid clashes

@mihailradkov mihailradkov force-pushed the TES-358-key-vault-security branch from f777e4f to d3ee86a Compare November 23, 2023 18:04
@mihailradkov mihailradkov marked this pull request as ready for review November 23, 2023 18:05
@mihailradkov mihailradkov force-pushed the TES-358-key-vault-security branch from d3ee86a to c7a7d23 Compare November 24, 2023 07:17
- Added key vault service endpoints to the gateway and vmss subnets
- Configured the key vault with NACLs to restrict the access only
  to the virtual network and given management CIDRs
- Fixed the graphdb password configuration secret to reference
  the correct value
- Organized the backup module
- Added the resource name prefix to the custom roles to avoid clashes
@mihailradkov mihailradkov force-pushed the TES-358-key-vault-security branch from c7a7d23 to 4451c05 Compare November 27, 2023 08:53
@mihailradkov mihailradkov merged commit 76e5e2b into main Nov 27, 2023
2 checks passed
@mihailradkov mihailradkov deleted the TES-358-key-vault-security branch November 27, 2023 08:55
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