enhancements icon indicating copy to clipboard operation
enhancements copied to clipboard

VolumeGroupSnapshot

Open xing-yang opened this issue 2 years ago β€’ 49 comments

Enhancement Description

  • One-line enhancement description (can be used as a release note): Introduce a VolumeGroupSnapshot API to take a snapshot of multiple volumes together.

  • Kubernetes Enhancement Proposal: https://github.com/kubernetes/enhancements/blob/master/keps/sig-storage/3476-volume-group-snapshot/README.md

  • Discussion Link:

  • Primary contact (assignee): @xing-yang

  • Responsible SIGs: sig-storage

  • Enhancement target (which target equals to which milestone):

    • Alpha release target (x.y): 1.27
    • Beta release target (x.y): 1.32
    • Stable release target (x.y):
  • [x] Alpha

    • [x] KEP (k/enhancements) update PR(s): https://github.com/kubernetes/enhancements/pull/1551
    • [ ] Code (k/k) update PR(s):
    • [ ] Docs (k/website) update PR(s): Placeholder blog PR: https://github.com/kubernetes/website/pull/39415
  • [ ] Beta

    • [x] KEP (k/enhancements) update PR(s): https://github.com/kubernetes/enhancements/pull/4734
    • [ ] Code (k/k) update PR(s): https://github.com/kubernetes/test-infra/pull/33417, https://github.com/kubernetes/test-infra/pull/33567, https://github.com/kubernetes/kubernetes/pull/126326
    • [ ] Docs (k/website) update(s): doc PR: https://github.com/kubernetes/website/pull/48419, blog PR: https://github.com/kubernetes/website/pull/48420

Please keep this description up to date. This will help the Enhancement Team to track the evolution of the enhancement efficiently.

xing-yang avatar Aug 25 '22 00:08 xing-yang

/sig storage

xing-yang avatar Aug 25 '22 00:08 xing-yang

/milestone v1.26

xing-yang avatar Sep 07 '22 19:09 xing-yang

Hello @xing-yang πŸ‘‹, 1.26 Enhancements team here.

Just checking in as we approach enhancements freeze on 18:00 PDT on Thursday 6th October 2022.

This enhancement is targeting for stage alpha for 1.26 (correct me, if otherwise).

Here's where this enhancement currently stands:

  • [ ] KEP file using the latest template has been merged into the k/enhancements repo.
  • [ ] KEP status is marked as implementable
  • [ ] KEP has a updated detailed test plan section filled out
  • [ ] KEP has up to date graduation criteria
  • [ ] KEP has a production readiness review that has been completed and merged into k/enhancements.

For this KEP, we would just need to update the following:

  • Update README.md on PR https://github.com/kubernetes/enhancements/pull/1551 to include test plan details as per the latest template
  • Include Design Details section and move Test Plan and Graduation Criteria sections into Design Details.
  • Merge PR https://github.com/kubernetes/enhancements/pull/1551 which will take care of meeting the above requirements.

The status of this enhancement is marked as at risk. Please keep the issue description up-to-date with appropriate stages as well. Thank you!

ruheenaansari34 avatar Sep 20 '22 23:09 ruheenaansari34

Quick reminder - Enhancement freeze is 2 days away. If you are still looking to get this enhancement into v1.26, please plan to make the updates to the kep.yaml, and README and get the PR merged.

ruheenaansari34 avatar Oct 04 '22 15:10 ruheenaansari34

@ruheenaansari34 Thanks for the reminder! I updated the KEP and moved test plan and graduation criteria under design details section.

xing-yang avatar Oct 05 '22 14:10 xing-yang

@xing-yang Thank you. I've re-assessed it and looks like we still need

Specifically, this statement:

[ ] I/we understand the owners of the involved components may require updates to existing tests to make this code solid enough prior to committing the changes necessary to implement this enhancement.

  • "Drawbacks" and "Alternatives" sections (if applicable) are added after "Implementation History".

Once the PR is merged with the above change, the enhancements freeze requirements will be fulfilled.

ruheenaansari34 avatar Oct 05 '22 15:10 ruheenaansari34

@ruheenaansari34 I've updated the KEP PR with the owner's acknowledgment and also moved "Alternatives" after "Implementation History". PTAL. Thanks.

xing-yang avatar Oct 06 '22 00:10 xing-yang

Thank you. Please ensure the PR is merged before enhancements freeze at 18:00 PDT today and that will fulfill requirements.

ruheenaansari34 avatar Oct 06 '22 16:10 ruheenaansari34

Hi @ruheenaansari34, we need more time to finalize the design details so the KEP won't be merged today. Since the implementation is completely out-of-tree, we will continue to work on it. When you remove the tracked/yes label, can you please also add the tracked/out-of-tree label? Thanks!

xing-yang avatar Oct 06 '22 23:10 xing-yang

Hello πŸ‘‹, 1.26 Enhancements Lead here.

Unfortunately, this enhancement did not meet requirements for enhancements freeze.

If you still wish to progress this enhancement in v1.26, please file an exception request. Thanks!

/milestone clear /label tracked/no /remove-label tracked/yes /remove-label lead-opted-in

rhockenbury avatar Oct 07 '22 01:10 rhockenbury

The Kubernetes project currently lacks enough contributors to adequately respond to all issues and PRs.

This bot triages issues and PRs according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue or PR as fresh with /remove-lifecycle stale
  • Mark this issue or PR as rotten with /lifecycle rotten
  • Close this issue or PR with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

k8s-triage-robot avatar Jan 05 '23 02:01 k8s-triage-robot

/remove-lifecycle stale

xing-yang avatar Jan 12 '23 15:01 xing-yang

Hello @xing-yang πŸ‘‹, Enhancements team here.

Just checking in as we approach Enhancements freeze on 18:00 PDT Thursday 9th February 2023.

This enhancement is targeting for stage alpha for 1.27 (correct me, if otherwise)

Here's where this enhancement currently stands:

  • [ ] KEP readme using the latest template has been merged into the k/enhancements repo.
  • [X] KEP status is marked as implementable for latest-milestone: 1.27
  • [X] KEP readme has a updated detailed test plan section filled out
  • [X] KEP readme has up to date graduation criteria
  • [x] KEP has a production readiness review that has been completed and merged into k/enhancements.

For this KEP, we would just need to update the following:

  • Add response for this question in the Scalability questionnaire of the KEP readme

The status of this enhancement is marked as at risk. Please keep the issue description up-to-date with appropriate stages as well. Thank you!

Atharva-Shinde avatar Jan 31 '23 19:01 Atharva-Shinde

The new question is not needed for alpha but would be nice to have. The PRR approval is still in place from last cycle, and I went through commit-by-commit and all the answers on the updated KEP look consistent with the previous discussions. So PRR is approved.

johnbelamaric avatar Feb 07 '23 18:02 johnbelamaric

@Atharva-Shinde I updated the PRR checkbox above

johnbelamaric avatar Feb 07 '23 18:02 johnbelamaric

Hey again @xing-yang Please try to get the KEP PR #1551, merged before tomorrow's Enhancement Freeze :) The status of this enhancement is still marked as at risk

Atharva-Shinde avatar Feb 08 '23 18:02 Atharva-Shinde

@Atharva-Shinde The KEP is merged.

xing-yang avatar Feb 09 '23 17:02 xing-yang

With all the KEP requirements in place and merged into k/enhancements, this enhancement is all good for the upcoming enhancements freeze. πŸš€

The status of this enhancement is marked as tracked. Please keep the issue description up-to-date with appropriate stages as well. Thank you!

Atharva-Shinde avatar Feb 09 '23 17:02 Atharva-Shinde

Hi @xing-yang, I’m reaching out from the 1.27 Release Docs team. This enhancement is marked as β€˜Needs Docs’ for the 1.27 release.

Please follow the steps detailed in the documentation to open a PR against dev-1.27 branch in the k/website repo. This PR can be just a placeholder at this time, and must be created by March 16. For more information, please take a look at Documenting for a release to familiarize yourself with the documentation requirements for the release.

Please feel free to reach out with any questions. Thanks!

LukeMwila avatar Mar 11 '23 04:03 LukeMwila

Hey @xing-yang πŸ‘‹ Enhancements team here, Can you please include the kubernetes/kubernetes code PRs related to this KEP in the Issue description, this will make it easier to track the code related PRs of this KEP.

Atharva-Shinde avatar Mar 12 '23 16:03 Atharva-Shinde

Hi @Atharva-Shinde, all code development are out of tree in this repo: https://github.com/kubernetes-csi/external-snapshotter. So we don't need to follow the k/k code freeze deadline. It already has a "tracked/out-of-tree" label. Thanks!

xing-yang avatar Mar 14 '23 01:03 xing-yang

Hi @LukeMwila, since all the code will be developed out of tree, the docs will also be out of tree here: https://kubernetes-csi.github.io/docs/. So I think we don't need to add docs in k/website. Thanks!

xing-yang avatar Mar 14 '23 01:03 xing-yang

/remove-label tracked/no

marosset avatar Mar 20 '23 17:03 marosset

The Kubernetes project currently lacks enough contributors to adequately respond to all issues.

This bot triages un-triaged issues according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue as fresh with /remove-lifecycle stale
  • Close this issue with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

k8s-triage-robot avatar Jan 20 '24 07:01 k8s-triage-robot

/remove-lifecycle stale

xing-yang avatar Jan 22 '24 03:01 xing-yang

The Kubernetes project currently lacks enough contributors to adequately respond to all issues.

This bot triages un-triaged issues according to the following rules:

  • After 90d of inactivity, lifecycle/stale is applied
  • After 30d of inactivity since lifecycle/stale was applied, lifecycle/rotten is applied
  • After 30d of inactivity since lifecycle/rotten was applied, the issue is closed

You can:

  • Mark this issue as fresh with /remove-lifecycle stale
  • Close this issue with /close
  • Offer to help out with Issue Triage

Please send feedback to sig-contributor-experience at kubernetes/community.

/lifecycle stale

k8s-triage-robot avatar Apr 21 '24 03:04 k8s-triage-robot

@xing-yang what's the status of this feature? is it still alpha in 1.29?

andyzhangx avatar May 09 '24 03:05 andyzhangx

This is still alpha. We may target beta in 1.32 as we still need to add metrics, e2e tests, etc.

xing-yang avatar May 09 '24 16:05 xing-yang

/remove-lifecycle stale

xing-yang avatar May 09 '24 16:05 xing-yang

@xing-yang can we target this for beta in 1.31 as we are already working on metrics and the e2e test? will you be taking care of updating the KEP, docs etc? please let me know if you need any help with it. Thank you.

Madhu-1 avatar May 17 '24 07:05 Madhu-1