Skip to content

Added min_vault_version details to HVD TF provider#1448

Merged
sivajayaraman merged 3 commits intomainfrom
sivajayaraman/min-version-details-in-hvd-provider
Mar 3, 2026
Merged

Added min_vault_version details to HVD TF provider#1448
sivajayaraman merged 3 commits intomainfrom
sivajayaraman/min-version-details-in-hvd-provider

Conversation

@sivajayaraman
Copy link
Contributor

@sivajayaraman sivajayaraman commented Feb 28, 2026

🛠️ Description

Updates the min_vault_version attribute description for the hcp_vault_cluster resource and data source to include a sample version value and a link to the HCP changelog , making it easier for users to discover valid versions. Users previously had no guidance on valid values for min_vault_version or where to find them. This change reduces friction by providing a concrete example and pointing to the changelog as the canonical source for available versions.

🏗️ Acceptance tests

  • Are there any feature flags that are required to use this functionality?
  • Have you added an acceptance test for the functionality being added?
  • Have you run the acceptance tests on this branch?

Output from acceptance testing:

Screenshot 2026-03-02 at 5 36 43 PM
$ make testacc TESTARGS='-run=TestAccXXX'

...

PCI review checklist

  • If applicable, I've documented a plan to revert these changes if they require more than reverting the pull request.

  • If applicable, I've worked with GRC to document the impact of any changes to security controls.

    Examples of changes to controls include access controls, encryption, logging, etc.

  • If applicable, I've worked with GRC to ensure compliance due to a significant change to the in-scope PCI environment.

    Examples include changes to operating systems, ports, protocols, services, cryptography-related components, PII processing code, etc.

@gautam-hashicorp
Copy link

TestAcc Vault was failing. Re-ran it again just in case.

@sivajayaraman
Copy link
Contributor Author

TestAcc Vault was failing. Re-ran it again just in case.

@gautam-hashicorp . I checked this, We have issue with VPC limit for acceptance test. Have to clean up some HVNs to make this succeed.

@gautam-hashicorp
Copy link

Got it. Let me know when that is done too. Thanks Siva!

@sivajayaraman
Copy link
Contributor Author

@gautam-hashicorp . Can you check now? Thank you!

Copy link

@gautam-hashicorp gautam-hashicorp left a comment

Choose a reason for hiding this comment

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

LGTM

@gautam-hashicorp
Copy link

Thanks for the changes, SIva!

Copy link

@monika-kumari1 monika-kumari1 left a comment

Choose a reason for hiding this comment

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

LGTM!!

@sivajayaraman sivajayaraman merged commit bfdbbbf into main Mar 3, 2026
24 of 28 checks passed
@sivajayaraman sivajayaraman deleted the sivajayaraman/min-version-details-in-hvd-provider branch March 3, 2026 05:32
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