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

fix: get naming convention module from git #138

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

Conversation

christian-calabrese
Copy link
Contributor

List of changes

Get the naming convention module from git instead of local path

Motivation and context

When we'll download the DevEx terraform modules from the terraform registry, it will break due to the missing "dependency" model azure_naming_convention.
For this reason, it's important to get this dependent module from a remote source (in our case, github or the registry when set up)

image

Type of changes

  • Add new resources
  • Update configuration to existing resources
  • Remove existing resources

Does this introduce a change to production resources with possible user impact?

  • Yes, users may be impacted applying this change
  • No

Other information

@christian-calabrese christian-calabrese requested a review from a team as a code owner October 14, 2024 13:53
Copy link

changeset-bot bot commented Oct 14, 2024

🦋 Changeset detected

Latest commit: f4d01ad

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 10 packages
Name Type
azure_function_app_exposed Minor
azure_app_service_exposed Minor
azure_role_assignments Minor
azure_postgres_server Minor
azure_storage_account Minor
azure_api_management Minor
azure_cosmos_account Minor
azure_function_app Minor
azure_app_service Minor
azure_event_hub Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

@christian-calabrese christian-calabrese requested a review from a team as a code owner October 14, 2024 13:54
@gunzip
Copy link
Contributor

gunzip commented Oct 14, 2024

can't we publish the naming_convention module first and get it from terraform registry?

@christian-calabrese
Copy link
Contributor Author

can't we publish the naming_convention module first and get it from terraform registry?

I think the alternatives to download the naming convention module from other modules are:

  • from the terraform registry without specifying a tag => I think the script that checks hashes would break at any release and it's probably not a good practice
  • from the main of the dx repo => Not very elegant
  • from the terraform registry pinning always the last version => I'm not really a fan of what we used to do in the past with the storage account module

@gunzip
Copy link
Contributor

gunzip commented Oct 15, 2024

why do you need to pin the latest version? from the terraform registry you can point to a major version and get free updates

@gunzip
Copy link
Contributor

gunzip commented Oct 15, 2024

I think the script that checks hashes would break at any release

that's the expected behaviour if you don't update the lock file

@christian-calabrese christian-calabrese force-pushed the get-naming-convention-from-git branch from eb44289 to 7c40a7f Compare December 6, 2024 13:33
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