[GitHub] [couchdb-helm] yekibud opened a new pull request, #102: Add persistent volume annotations

[GitBox <gi...@apache.org>]

yekibud opened a new pull request, #102:
URL: https://github.com/apache/couchdb-helm/pull/102

   <!--
   Thank you for contributing to couchdb-helm. Before you submit this PR we'd like to
   make sure you are aware of the chart technical requirements and best practices:
   
   * https://github.com/helm/charts/blob/master/CONTRIBUTING.md#technical-requirements
   * https://github.com/helm/helm/tree/master/docs/chart_best_practices
   
   For a quick overview across what we will look at reviewing your PR, please read
   our review guidelines:
   
   * https://github.com/helm/charts/blob/master/REVIEW_GUIDELINES.md
   
   Following our best practices right from the start will accelerate the review process and
   help get your PR merged quicker.
   
   When updates to your PR are requested, please add new commits and do not squash the
   history. This will make it easier to identify new changes. The PR will be squashed
   anyways when it is merged. Thanks.
   
   Please make sure you test your changes before you push them.
   -->
   
   #### What this PR does / why we need it:
   
   The ability to add annotations to k8s resources is generally useful.   The ability to add annotations to the PVCs that support couchdb is especially useful because it enables the use of volume tagging utilities such as [k8s-pvc-tagger](https://github.com/mtougeron/k8s-pvc-tagger) which can be used to support automated backup snapshots of volumes.
   
   #### Which issue this PR fixes
   
   No related issue tracker.
   
   #### Special notes for your reviewer:
   
   * The previous charts `Chart.yaml` showed version 3.6.2 has the latest chart version, but no corresponding tarball was found.  I bumped the version to 3.6.3, but let me know if it should remain 3.6.2.
   * `make test` indicated some sort of error but I'm not sure what it means or if it is expected
   ![image](https://user-images.githubusercontent.com/1264077/202265273-9706e8a0-1ccc-40ea-94c6-e902fcf318bb.png)
   
   #### Checklist
   [Place an '[x]' (no spaces) in all applicable fields. Please remove unrelated fields.
   - [x] Chart Version bumped
   - [ ] e2e tests pass
   - [x] Variables are documented in the README.md
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@couchdb.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org

[GitHub] [couchdb-helm] willholley merged pull request #102: Add persistent volume annotations

[GitBox <gi...@apache.org>]

willholley merged PR #102:
URL: https://github.com/apache/couchdb-helm/pull/102


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: notifications-unsubscribe@couchdb.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org