Skip to content

Conversation

pstibrany
Copy link
Contributor

Add optional uid parameter to grafana_folder data source:

  • Make title optional and add uid parameter for more flexible folder lookup
  • Support finding folders by title only, uid only, or both title and uid
  • Add comprehensive error handling with specific error messages for different lookup scenarios
  • Add extensive test coverage for all lookup combinations and error cases

* Make title optional and add uid parameter for more flexible folder lookup.
* Support finding folders by title only, uid only, or both title and uid
* Add comprehensive error handling with specific error messages for different lookup scenarios
* Add extensive test coverage for all lookup combinations and error cases
@pstibrany pstibrany requested a review from a team as a code owner August 12, 2025 10:44
Copy link
Contributor

In order to lower resource usage and have a faster runtime, PRs will not run Cloud tests automatically.
To do so, a Grafana Labs employee must trigger the cloud acceptance tests workflow manually.

@pstibrany
Copy link
Contributor Author

cloud acceptance tests

@pstibrany
Copy link
Contributor Author

I've tried running cloud acceptance tests here: https://github.com/grafana/terraform-provider-grafana/actions/runs/16907204533

Copy link

@JohnnyQQQQ JohnnyQQQQ left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@pstibrany pstibrany merged commit 3f91662 into main Aug 13, 2025
31 of 32 checks passed
@pstibrany pstibrany deleted the add-uid-to-folder-datasource branch August 13, 2025 09:12
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