Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

add option for deploying awx with a local pv #31

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

yonahd
Copy link
Contributor

@yonahd yonahd commented Oct 19, 2021

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 yonahd changed the title add option for deploying with a local pv add option for deploying awx with a local pv Oct 19, 2021
@yonahd
Copy link
Contributor Author

yonahd commented Oct 19, 2021

@jbielick What do you think?

@jbielick
Copy link
Member

What does this do?

@yonahd
Copy link
Contributor Author

yonahd commented Oct 19, 2021

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

@jbielick
Copy link
Member

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

@yonahd
Copy link
Contributor Author

yonahd commented Oct 19, 2021

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
Copy link
Contributor Author

yonahd commented Oct 20, 2021

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants