Skip to content

Release v2.31.0 #minor#730

Merged
optik-aper merged 1 commit intovultr:masterfrom
optik-aper:release-v2.31.0
Apr 21, 2026
Merged

Release v2.31.0 #minor#730
optik-aper merged 1 commit intovultr:masterfrom
optik-aper:release-v2.31.0

Conversation

@optik-aper
Copy link
Copy Markdown
Contributor

Description

Release v2.31.0 #minor

Related Issues

#706
#713
#723
#712
#719
#710
#705
#704
#716
#723
#724
#725
#721
#717
#726
#727
#715
#711
#729
#728

@optik-aper optik-aper self-assigned this Apr 21, 2026
@github-actions
Copy link
Copy Markdown

PR Reviewer Guide 🔍

Here are some key observations to aid the review process:

🎫 Ticket compliance analysis 🔶

717 - Partially compliant

Compliant requirements:

  • Changelog entry references the fix (PR 717) for removing count fields from firewall_group schema

Non-compliant requirements:

  • Actual code changes for removing the count fields are not in this release PR diff (they were in a prior PR)

Requires further human verification:

  • Verify that the underlying code change (PR 717) correctly removed instance_count, rule_count, and max_rule_count from the firewall group schema

713 - Partially compliant

Compliant requirements:

  • Changelog entries reference all organization/IAM data sources and resources (PR 713)

Non-compliant requirements:

  • Actual code changes for organization/IAM are not in this release PR diff (they were in a prior PR)

Requires further human verification:

  • Verify that all organization/IAM resources and data sources were correctly merged from the beta branch

729 - Partially compliant

Compliant requirements:

  • Changelog entry references the fix (PR 729) for ensuring IPv6 is an expanded string for the API

Non-compliant requirements:

  • Actual code changes are not in this release PR diff (they were in a prior PR)

Requires further human verification:

  • Verify that the schema validation for the reverse DNS IP field was added in the underlying PR
⏱️ Estimated effort to review: 1 🔵⚪⚪⚪⚪
🧪 No relevant tests
🔒 No security concerns identified
⚡ No major issues detected

@optik-aper optik-aper merged commit db92bda into vultr:master Apr 21, 2026
4 checks passed
@github-actions
Copy link
Copy Markdown

PR Code Suggestions ✨

Explore these optional code suggestions:

CategorySuggestion                                                                                                                                    Impact
Possible issue
Fix "attachement" typo to "attachment"

The resource names organization_policy_group_attachement and
organization_policy_user_attachement appear to have a typo — "attachement" should be
"attachment". This is inconsistent with other entries in the same changelog that
correctly use "attachment" (e.g., organization_role_group_attachment). If this typo
also exists in the actual Terraform resource names, it would affect user-facing API
and should be fixed before release.

CHANGELOG.md [13-14]

-* resource/organization_policy_group_attachement: Added organization/IAM from beta branch [PR 713](https://github.com/vultr/terraform-provider-vultr/pull/713)
-* resource/organization_policy_user_attachement: Added organization/IAM from beta branch [PR 713](https://github.com/vultr/terraform-provider-vultr/pull/713)
+* resource/organization_policy_group_attachment: Added organization/IAM from beta branch [PR 713](https://github.com/vultr/terraform-provider-vultr/pull/713)
+* resource/organization_policy_user_attachment: Added organization/IAM from beta branch [PR 713](https://github.com/vultr/terraform-provider-vultr/pull/713)
Suggestion importance[1-10]: 7

__

Why: The suggestion correctly identifies a typo in the changelog where attachement should be attachment. Fixing this is important because if the typo also exists in the actual Terraform resource names, it would affect the user-facing API and should be corrected before release.

Medium

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant