Skip to content

Conversation

@BUITHICAMVAN
Copy link

@BUITHICAMVAN BUITHICAMVAN commented Apr 15, 2025

Description

Add attribute to set disable_schema_validation to skip schema validation for helm_template and helm_release. Please help me to review and investigate the case.

Acceptance tests

  • Have you added an acceptance test for the functionality being added?

Release Note

Release note for CHANGELOG:

...

References

https://github.com//issues/1609

Community Note

  • Please vote on this issue by adding a 👍 reaction to the original issue to help the community and maintainers prioritize this request
  • If you are interested in working on this issue or have submitted a pull request, please leave a comment

@BUITHICAMVAN BUITHICAMVAN requested a review from a team as a code owner April 15, 2025 09:40
@hashicorp-cla-app
Copy link

CLA assistant check

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement

Learn more about why HashiCorp requires a CLA and what the CLA includes


btcv seems not to be a GitHub user.
You need a GitHub account to be able to sign the CLA.
If you have already a GitHub account, please add the email address used for this commit to your account.

Have you signed the CLA already but the status is still pending? Recheck it.

1 similar comment
@hashicorp-cla-app
Copy link

CLA assistant check

Thank you for your submission! We require that all contributors sign our Contributor License Agreement ("CLA") before we can accept the contribution. Read and sign the agreement

Learn more about why HashiCorp requires a CLA and what the CLA includes


btcv seems not to be a GitHub user.
You need a GitHub account to be able to sign the CLA.
If you have already a GitHub account, please add the email address used for this commit to your account.

Have you signed the CLA already but the status is still pending? Recheck it.

resource.TestCheckResourceAttr("helm_release.imported", "dependency_update", "false"),
resource.TestCheckResourceAttr("helm_release.imported", "replace", "false"),
resource.TestCheckResourceAttr("helm_release.imported", "disable_openapi_validation", "false"),
resource.TestCheckResourceAttr("helm_release.imported", "disable_scheme_validation", "false"),
Copy link

Choose a reason for hiding this comment

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

scheme → schema
(I'm just passing by, not a reviewer, and noticed that typo here)

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.

2 participants