Skip to content

Commit 6fdd287

Browse files
authored
Merge pull request kubernetes-csi#1094 from leonardoce/vsc-namespace-marker
Use a marker for the Namespace VSC additional printer column
2 parents 0bec91d + 5c6b2fb commit 6fdd287

File tree

5 files changed

+5
-4
lines changed

5 files changed

+5
-4
lines changed

client/apis/volumesnapshot/v1/types.go

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -264,6 +264,7 @@ type VolumeSnapshotClassList struct {
264264
// +kubebuilder:printcolumn:name="Driver",type=string,JSONPath=`.spec.driver`,description="Name of the CSI driver used to create the physical snapshot on the underlying storage system."
265265
// +kubebuilder:printcolumn:name="VolumeSnapshotClass",type=string,JSONPath=`.spec.volumeSnapshotClassName`,description="Name of the VolumeSnapshotClass to which this snapshot belongs."
266266
// +kubebuilder:printcolumn:name="VolumeSnapshot",type=string,JSONPath=`.spec.volumeSnapshotRef.name`,description="Name of the VolumeSnapshot object to which this VolumeSnapshotContent object is bound."
267+
// +kubebuilder:printcolumn:name="VolumeSnapshotNamespace",type=string,JSONPath=`.spec.volumeSnapshotRef.namespace`,description="Namespace of the VolumeSnapshot object to which this VolumeSnapshotContent object is bound."
267268
// +kubebuilder:printcolumn:name="Age",type=date,JSONPath=`.metadata.creationTimestamp`
268269
type VolumeSnapshotContent struct {
269270
metav1.TypeMeta `json:",inline"`

client/config/crd/snapshot.storage.k8s.io_volumesnapshotcontents.yaml

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,8 @@ spec:
4747
jsonPath: .spec.volumeSnapshotRef.name
4848
name: VolumeSnapshot
4949
type: string
50-
- description: Namespace of the VolumeSnapshot object to which this VolumeSnapshotContent object is bound.
50+
- description: Namespace of the VolumeSnapshot object to which this VolumeSnapshotContent
51+
object is bound.
5152
jsonPath: .spec.volumeSnapshotRef.namespace
5253
name: VolumeSnapshotNamespace
5354
type: string

client/hack/README.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -105,8 +105,6 @@ Update the restoreSize property to use type string only:
105105
106106
```
107107
108-
* Add the VolumeSnapshot namespace to the `additionalPrinterColumns` section in `client/config/crd/snapshot.storage.k8s.io_volumesnapshotcontents.yaml`. Refer https://github.com/kubernetes-csi/external-snapshotter/pull/535 for more details.
109-
110108
## Test suite
111109
112110
The `test-suite` directory contains several test cases that are useful to

pkg/utils/patch.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,8 +3,8 @@ package utils
33
import (
44
"context"
55
"encoding/json"
6-
crdv1alpha1 "github.com/kubernetes-csi/external-snapshotter/client/v8/apis/volumegroupsnapshot/v1alpha1"
76

7+
crdv1alpha1 "github.com/kubernetes-csi/external-snapshotter/client/v8/apis/volumegroupsnapshot/v1alpha1"
88
crdv1 "github.com/kubernetes-csi/external-snapshotter/client/v8/apis/volumesnapshot/v1"
99
clientset "github.com/kubernetes-csi/external-snapshotter/client/v8/clientset/versioned"
1010
metav1 "k8s.io/apimachinery/pkg/apis/meta/v1"

vendor/github.com/kubernetes-csi/external-snapshotter/client/v8/apis/volumesnapshot/v1/types.go

Lines changed: 1 addition & 0 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

0 commit comments

Comments
 (0)