Skip to content

add option to generate certificates from a third-party CA #6

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

Open
wants to merge 5 commits into
base: main
Choose a base branch
from

Conversation

kevin11-allen-nttd
Copy link

@kevin11-allen-nttd kevin11-allen-nttd commented May 5, 2025

Breaking changes

Removes the option to deploy a private DNS zone with the key vault. The private DNS zone is better deployed with a vnet, in case multiple key vaults are being created on the same network. The vnet module currently allows for this.

Users can now specify the IDs of existing private DNS zones to link the private endpoint to, or leave it blank in case Azure Policy is configured to link the private endpoints to a centralized DNS zone automatically

Updated the major version of the naming module, which will cause issues for any existing deployments that abbreviate the region name but are not being deployed to a US region

Features

Added option to generate certificates, either self-signed or with an integrated third-party CA like DigiCert

Fixes

Reverts changes to the private_vault example which caused it to deploy a public vault

Fixes secrets and certificates not able to be created when using RBAC instead of access policies

  • requires that a role assignment be defined in var.role_assignments for the current user

@kevin11-allen-nttd kevin11-allen-nttd requested a review from a team as a code owner May 5, 2025 17:47
@kevin11-allen-nttd kevin11-allen-nttd marked this pull request as draft May 5, 2025 17:59
@kevin11-allen-nttd kevin11-allen-nttd changed the title initial refactor add option to generate certificates from a third-party CA May 5, 2025
@kevin11-allen-nttd kevin11-allen-nttd marked this pull request as ready for review May 7, 2025 18:29
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