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

Snapshot & VolumeContentSource Support #47

Draft
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

maennchen
Copy link
Collaborator

Closes #12 & #13

TODO

  • Update CI Secret to provide secret for CreateSnapshot & DeleteSnapshot
  • Add instructions for installation
  • Add Deploy Setup (External Snapshotter)
  • Test thoroughly

@maennchen maennchen added the enhancement New feature or request label May 17, 2020
@maennchen maennchen requested a review from ofek May 17, 2020 15:49
@maennchen maennchen self-assigned this May 17, 2020
@ofek
Copy link
Owner

ofek commented May 20, 2020

I just updated the CI secret (added rights to the provisioner service account)

@maennchen maennchen linked an issue May 26, 2020 that may be closed by this pull request
@maennchen maennchen changed the title Snapshot Support Snapshot & VolumeContentSource Support May 27, 2020
@maennchen
Copy link
Collaborator Author

@ofek Snapshots were promoted to Beta in K8S v1.17. With it, the way to deploy also changed.

If you agree, I would create a new kustomize overlay that adds the relevant stuff for snapshots, but only for >= v1.17. I would not support older versions of K8S.

When snapshots are completely stable and most people have updated their clusters, I would move the feature from the overlay into the main definition.

@ofek
Copy link
Owner

ofek commented May 27, 2020

That sounds fine!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add Support for VolumeContentSource Add Support for Snapshots
2 participants