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

Add Configurable Inputs and Outputs to Enhance Flexibility #5

Conversation

rswrz
Copy link
Member

@rswrz rswrz commented Nov 15, 2024

This pull request implements the changes requested in issue #4. It introduces the following updates:

  • Inputs

    • Added name input to customize the base name of resources, allowing for easier migration and alignment with best practices.
    • Added dns_servers input to configure the DNS servers for the virtual network as per the azurerm_virtual_network documentation.
    • Added name_suffix input to allow further customization of resource names.
  • Outputs

These changes enhance the module's configurability and align it with common practices, improving its flexibility for various use cases.

Signed-off-by: Roman Schwarz <[email protected]>
Signed-off-by: Roman Schwarz <[email protected]>
- `key_vault_private_endpoint_private_ip_address`
- `network_security_group_id`
- `network_security_group_name`

Signed-off-by: Roman Schwarz <[email protected]>
Ensures that when name_suffix is not set, the resource name does not end with a trailing -.

Signed-off-by: Roman Schwarz <[email protected]>
@rswrz rswrz force-pushed the 4-add-missing-inputs-and-outputs-to-module-for-enhanced-flexibility-and-functionality branch from 68055fb to a244751 Compare November 15, 2024 13:52
@rswrz rswrz added enhancement New feature or request feature and removed enhancement New feature or request labels Nov 15, 2024
@rswrz rswrz merged commit f2cbb14 into main Nov 15, 2024
39 of 41 checks passed
@rswrz rswrz deleted the 4-add-missing-inputs-and-outputs-to-module-for-enhanced-flexibility-and-functionality branch November 15, 2024 14:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Add Missing Inputs and Outputs to Module for Enhanced Flexibility and Functionality
2 participants