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 location_from_id provider-defined function #2007

Conversation

modular-magician
Copy link
Collaborator

This PR adds support for a new provider-defined function:

  • location_from_id

There are unit and acceptance tests for both of these, and documentation

Note about testing

Currently, testing this PR requires downloading an alpha release of TF 1.8.0 and running the acceptance test locally. The unit tests should work fine because they don't use a TF binary like acc tests do, and the dependencies on this branch include provider function support.

I'm working on adding automated testing for the feature branch and provider-defined function PR here: hashicorp/terraform-provider-google#17451


Release Note Template for Downstream PRs (will be copied)

provider: added provider-defined function `location_from_id` for retrieving the location from a resource's self link or id

Derived from GoogleCloudPlatform/magic-modules#10061

* Add `location_from_id` function, tests, docs

* Fix whitespace in acc test HCL config

[upstream:880aad8f718fff47db62a04c3d39b570069f9bfe]

Signed-off-by: Modular Magician <[email protected]>
@modular-magician modular-magician requested a review from a team as a code owner March 1, 2024 13:03
@modular-magician modular-magician requested review from melinath and removed request for a team March 1, 2024 13:03
@modular-magician modular-magician merged commit 30dcd40 into GoogleCloudPlatform:FEATURE-BRANCH-provider-functions Mar 1, 2024
1 check passed
Copy link

🤖 I detect that the PR title and the commit message differ and there's only one commit. To use the PR title for the commit history, you can use Github's automerge feature with squashing, or use automerge label. Good luck human!

-- conventional-commit-lint bot
https://conventionalcommits.org/

@modular-magician modular-magician deleted the downstream-pr-880aad8f718fff47db62a04c3d39b570069f9bfe branch November 18, 2024 06:41
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.

1 participant