external-snapshotter icon indicating copy to clipboard operation
external-snapshotter copied to clipboard

Add secrets in CreateVolumeGroupSnapshots

Open xing-yang opened this issue 2 years ago • 8 comments
trafficstars

xing-yang avatar May 08 '23 14:05 xing-yang

Secrets are passed here

https://github.com/kubernetes-csi/external-snapshotter/blob/a942b5b9b81b1bc3079c0c009ef252eaba6600de/pkg/group_snapshotter/group_snapshotter.go#L50-L64

Is there an other location where you expect secrets to be passed?

nixpanic avatar May 25 '23 15:05 nixpanic

Currently nothing is passed through snapshotterCredentials. There's a TODO here: https://github.com/kubernetes-csi/external-snapshotter/blob/master/pkg/sidecar-controller/groupsnapshot_helper.go#L368

See this PR as an example: https://github.com/kubernetes-csi/external-snapshotter/pull/165/

xing-yang avatar May 25 '23 18:05 xing-yang

/assign

carlory avatar Jun 08 '23 09:06 carlory

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 22 '24 03:01 k8s-triage-robot

@carlory can you please update on this one, without this support we cannot implement volumegroup in cephcsi driver.

Madhu-1 avatar Feb 08 '24 09:02 Madhu-1

Thanks for your reminder, I didn't start to solve this issue yet. Due to Chinese New Year, I may not have enough bandwidth to solve this issue in the next few days. If you want this changes to be made as soon as possible, feel free to submit a pull request if you are also interested in doing it.

carlory avatar Feb 08 '24 10:02 carlory

/remove-lifecycle stale

carlory avatar Feb 08 '24 10:02 carlory

@carlory great thank you, will take care of this one.

/assign @Madhu-1

Madhu-1 avatar Feb 08 '24 11:02 Madhu-1