Skip to content

Remove all deprecated attributes and resources #1529

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

Merged
merged 2 commits into from
May 10, 2024

Conversation

julienduchesne
Copy link
Contributor

@julienduchesne julienduchesne commented May 2, 2024

Preparing the next major release, this PR removes deprecated attributes and resources
Their use was also replaced with their non-deprecated alternative

Note: this is a pretty big PR, but there's nothing really crazy in there. Mostly removing some attributes and where they are plugged in. The one thing of note is a workaround I had to do in the annotations resource: https://github.com/grafana/terraform-provider-grafana/pull/1529/files#diff-2557a290b338e0264c77cd6aead40eb6907ee01698f66757b51e281a72df841aR134-R154

@julienduchesne julienduchesne added this to the v3 milestone May 2, 2024
Copy link

github-actions bot commented May 2, 2024

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.

@julienduchesne julienduchesne force-pushed the julienduchesne/deprecations branch 7 times, most recently from 0950232 to b07dc3e Compare May 2, 2024 17:59
@julienduchesne julienduchesne changed the title Remove all deprecated attributes Remove all deprecated attributes and resources May 2, 2024
@julienduchesne julienduchesne marked this pull request as ready for review May 2, 2024 18:06
@julienduchesne julienduchesne requested a review from a team as a code owner May 2, 2024 18:06
@julienduchesne julienduchesne requested a review from a team May 2, 2024 18:06
@julienduchesne julienduchesne requested review from a team as code owners May 2, 2024 18:06
Copy link
Contributor

@gerboland gerboland left a comment

Choose a reason for hiding this comment

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

Looks good from the ML-team! Thank you

Copy link
Contributor

@joeyorlando joeyorlando left a comment

Choose a reason for hiding this comment

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

LGTM from OnCall side of things. Just one question

Copy link
Contributor

Choose a reason for hiding this comment

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

agree with getting rid of this, just curious if the next release be a major version bump? (as this feels like a breaking change)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Yup. That will be in the 3.0.0

Copy link
Contributor Author

@julienduchesne julienduchesne May 3, 2024

Choose a reason for hiding this comment

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

I'll be merging all of the v3 milestone PRs at the same time. When I'm ready to do the major release

@eleijonmarck
Copy link
Contributor

@julienduchesne is there a estimate as yo when the v3 of terraform will be out?

@julienduchesne
Copy link
Contributor Author

Monday, I think

Preparing the next major release, this PR removes deprecated attributes and resources
Their use was also replaced with their non-deprecated alternative
@julienduchesne julienduchesne force-pushed the julienduchesne/deprecations branch from b07dc3e to 354bfca Compare May 10, 2024 18:45
@julienduchesne julienduchesne merged commit 6ecee50 into main May 10, 2024
27 checks passed
@julienduchesne julienduchesne deleted the julienduchesne/deprecations branch May 10, 2024 19:06
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.

4 participants