update statefulset to inlcude verbose PVC reference

[ritzk]

Description

The issue is that the verbose reference is added once ArgoCD applies the chart to cluster. This results in ArgoCD thinking there is a diff, when there is none. This can be solved by adding a patch which adds the verbose reference.

Issues Resolved

n/a

Check List

• Commits are signed per the DCO using --signoff

For any changes to files within Helm chart directories:

• Helm chart version bumped
• Helm chart CHANGELOG.md updated to reflect change

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.

Similar to bitnami/charts#17140

[DandyDeveloper]

@ritzk Please do DCO Checks and check the linting output.

[ritzk]

@ritzk Please do DCO Checks and check the linting output.

@DandyDeveloper lint failed due to n/w issues on the action runner.

→ ct lint
Linting charts...

------------------------------------------------------------------------------------------------------------------------
 Charts to be processed:
------------------------------------------------------------------------------------------------------------------------
 opensearch => (version: "2.31.1", path: "charts/opensearch")
------------------------------------------------------------------------------------------------------------------------

Linting chart "opensearch => (version: \"2.31.1\", path: \"charts/opensearch\")"
Checking chart "opensearch => (version: \"2.31.1\", path: \"charts/opensearch\")" for a version bump...
Old chart version: 2.31.0
New chart version: 2.31.1
Chart version ok.
Validating charts/opensearch/Chart.yaml...
Validation success! 👍
Validating maintainers...

Linting chart with values file "charts/opensearch/ci/ci-ingress-class-name-values.yaml"...

==> Linting charts/opensearch
[INFO] Chart.yaml: icon is recommended

1 chart(s) linted, 0 chart(s) failed

Linting chart with values file "charts/opensearch/ci/ci-rbac-enabled-values.yaml"...

==> Linting charts/opensearch
[INFO] Chart.yaml: icon is recommended

1 chart(s) linted, 0 chart(s) failed

Linting chart with values file "charts/opensearch/ci/ci-values.yaml"...

==> Linting charts/opensearch
[INFO] Chart.yaml: icon is recommended

1 chart(s) linted, 0 chart(s) failed

------------------------------------------------------------------------------------------------------------------------
 ✔︎ opensearch => (version: "2.31.1", path: "charts/opensearch")
------------------------------------------------------------------------------------------------------------------------
All charts linted successfully

[DandyDeveloper]

@ritzk One thing you will wanna test, can you install the original, then upgrade with your changes. Can you confirm if this "breaks" because the STS isn't immutable? This will cause some real problems because people would effectively need to remove the STS, then apply these updates which could bring multiple nodes down on an upgrade if forced.

[ritzk]

@ritzk One thing you will wanna test, can you install the original, then upgrade with your changes. Can you confirm if this "breaks" because the STS isn't immutable? This will cause some real problems because people would effectively need to remove the STS, then apply these updates which could bring multiple nodes down on an upgrade if forced.

This is what is written to kubernetes, atleast on 1.24.x & 1.29.x

→ helm install -n opensearch --create-namespace opensearch  opensearch/opensearch
→ kubectl -n opensearch  get statefulsets.apps  opensearch-cluster-master -o jsonpath-as-json='{.spec.volumeClaimTemplates[0]}'|jq .
[
  {
    "apiVersion": "v1",
    "kind": "PersistentVolumeClaim",
    "metadata": {
      "creationTimestamp": null,
      "name": "opensearch-cluster-master"
    },
    "spec": {
      "accessModes": [
        "ReadWriteOnce"
      ],
      "resources": {
        "requests": {
          "storage": "8Gi"
        }
      },
      "volumeMode": "Filesystem"
    },
    "status": {
      "phase": "Pending"
    }
  }
]