Skip to content

Conversation

@Hsn723
Copy link

@Hsn723 Hsn723 commented Nov 19, 2025

PR Checklist

[Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.]

  • Chart Version bumped
  • CHANGELOG.md updated
  • Variables and other changes are documented in the README.md
  • Title of the PR starts with chart name (e.g. [artifactory])

What this PR does / why we need it:

Currently, .artifactory.persistence.annotations is defined in values.yaml, but its value is unused. Placing annotations on volume claim templates is useful and sometimes necessary. For instance, when the backing storage uses a CSI driver that supports volume expansion where the parameters are controlled via annotations. This PR addresses this by consuming the already-defined .artifactory.persistence.annotations value in the artifactory statefulset's .spec.volumeClaimTemplate where appropriate.

@github-actions
Copy link

github-actions bot commented Nov 19, 2025

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@Hsn723
Copy link
Author

Hsn723 commented Nov 19, 2025

I have read the CLA Document and I hereby sign the CLA

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.

1 participant