close
The Wayback Machine - https://web.archive.org/web/20200905010125/https://github.com/sourcegraph/deploy-sourcegraph/pull/815
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

overlays: collect PVCs into a separate base #815

Merged
merged 2 commits into from Aug 13, 2020

Conversation

@uwedeportivo
Copy link
Contributor

uwedeportivo commented Aug 7, 2020

collects PVC resources into a separate base. this allows kustomizations to pick and choose which PVCs they want or avoid creating them altogether. we have customers that need this because they explicitly create PVs and map them to deployments/statefulsets with named claim refs.

(kustomize does not allow deletes)

@pecigonzalo
Copy link
Contributor

pecigonzalo commented Aug 10, 2020

Wouldn't this be a problem anyway because we specify the PVC in the volume definition of the Deployment? We might need to do something like kubernetes-sigs/kustomize#2037

@uwedeportivo
Copy link
Contributor Author

uwedeportivo commented Aug 10, 2020

it assumes the overlay the customer uses also replaces/edits the mention in the deployment

@uwedeportivo uwedeportivo merged commit 8c61cd3 into master Aug 13, 2020
2 checks passed
2 checks passed
lsif-go
Details
buildkite/deploy-sourcegraph Build #3576 passed (14 minutes, 15 seconds)
Details
@uwedeportivo uwedeportivo deleted the separate_pvcs_for_kustomize branch Aug 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

2 participants
You can’t perform that action at this time.