charts icon indicating copy to clipboard operation
charts copied to clipboard

add option for deploying awx with a local pv

Open yonahd opened this issue 3 years ago • 6 comments

What this PR does / why we need it:

Which issue this PR fixes

(optional, in fixes #<issue number>(, fixes #<issue_number>, ...) format, will close that issue when PR gets merged)

  • fixes #

Special notes for your reviewer:

Checklist

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

  • [ ] Developer Certificate of Origin signed
  • [ ] Chart Version bumped (SEMVER 2)
  • [ ] Variables are documented in the README.md
  • [ ] Title of the PR starts with chart name (e.g. [example]: Add service for feature xyz)

yonahd avatar Oct 19 '21 10:10 yonahd

@jbielick What do you think?

yonahd avatar Oct 19 '21 10:10 yonahd

What does this do?

jbielick avatar Oct 19 '21 14:10 jbielick

This allows the user to provision a persistent volume for the pvc created by the postgres chart

yonahd avatar Oct 19 '21 14:10 yonahd

I might be missing something. How does the postgres subchart know to use this PV?

jbielick avatar Oct 19 '21 14:10 jbielick

Pvc bind themselves to an available pv with capacity and the correct type of storage for them. At least to the extent of my knowledge Let me see if I can force the pv to be bound to the specific pvc

yonahd avatar Oct 19 '21 15:10 yonahd

@jbielick used the pvc's selector from the postgres chart to bind to the local pv

yonahd avatar Oct 20 '21 10:10 yonahd