set.value: required -> optional

[BBBmau]

Description

Fixes #1570

Acceptance tests

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

󰀵 mau  …/terraform-provider-helm   set-value-optional $!?   v1.23.4   21:01  
 TESTARGS="-run TestAccResourceRelease_SetNull" make testacc
==> Checking that code complies with gofmt requirements...
TF_ACC=1 go test "./helm" -v -run TestAccResourceRelease_SetNull -timeout 10m
2025/01/24 21:01:54 Created repository package for "broken-chart"
2025/01/24 21:01:54 Created repository package for "crds-chart"
2025/01/24 21:01:54 Created repository package for "dependency-bar"
2025/01/24 21:01:54 Created repository package for "dependency-foo"
2025/01/24 21:01:54 Created repository package for "failed-deploy"
2025/01/24 21:01:54 Created repository package for "kube-version"
2025/01/24 21:01:54 Created repository package for "test-chart"
2025/01/24 21:01:54 Created repository package for "test-chart-v2"
2025/01/24 21:01:54 Created repository package for "umbrella-chart"
2025/01/24 21:01:54 Built chart repository index
=== RUN   TestAccResourceRelease_SetNull
--- PASS: TestAccResourceRelease_SetNull (7.41s)
PASS
ok      github.com/hashicorp/terraform-provider-helm/helm       8.831s

We aren't able to have the value be null due to the following error occurring:

Release Note

Release note for CHANGELOG:

change `set.value` && `set_list.value` to optional instead of required

References

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

[JaylonmcShan03]

Thanks for catching this!