Merge pull request #11 from bswartz/image-name

Change data-source-validator to volume-data-source-validator

Author: k8s-ci-robot

Makefile

@@ -12,9 +12,9 @@
 # See the License for the specific language governing permissions and
 # limitations under the License.
 
-.PHONY: all data-source-validator clean test
+.PHONY: all volume-data-source-validator clean test
 
-CMDS=data-source-validator
+CMDS=volume-data-source-validator
 all: build
 
 include release-tools/build.make

cmd/data-source-validator/Dockerfile

@@ -0,0 +1,7 @@
+FROM gcr.io/distroless/base:latest
+LABEL maintainers="Kubernetes Authors"
+LABEL description="Volume Data Source Validator Controller"
+ARG binary=./bin/volume-data-source-validator
+
+COPY ${binary} volume-data-source-validator
+ENTRYPOINT ["/volume-data-source-validator"]

cmd/volume-data-source-validator/Dockerfile

@@ -1,21 +1,22 @@
-# RBAC file for the data-source-validator controller
+# RBAC file for the volume-data-source-validator controller
 #
-# The data-source-validator controller does asynchronous validation of PVC data sources
+# The volume-data-source-validator controller does asynchronous validation of PVC data
+# sources.
 # It should be installed as part of the base Kubernetes distribution in an appropriate
 # namespace for components implementing base system functionality. For installing with
 # Vanilla Kubernetes, kube-system makes sense for the namespace.
 
 apiVersion: v1
 kind: ServiceAccount
 metadata:
-  name: data-source-validator
+  name: volume-data-source-validator
   namespace: kube-system
 
 ---
 kind: ClusterRole
 apiVersion: rbac.authorization.k8s.io/v1
 metadata:
-  name: data-source-validator
+  name: volume-data-source-validator
 rules:
   - apiGroups: [populator.storage.k8s.io]
     resources: [volumepopulators]
@@ -31,12 +32,12 @@ rules:
 kind: ClusterRoleBinding
 apiVersion: rbac.authorization.k8s.io/v1
 metadata:
-  name: data-source-validator
+  name: volume-data-source-validator
 subjects:
   - kind: ServiceAccount
-    name: data-source-validator
+    name: volume-data-source-validator
     namespace: kube-system
 roleRef:
   kind: ClusterRole
-  name: data-source-validator
+  name: volume-data-source-validator
   apiGroup: rbac.authorization.k8s.io

cmd/data-source-validator/main.go renamed to cmd/volume-data-source-validator/main.go

@@ -1,6 +1,7 @@
-# This YAML file shows how to deploy the data-source-validator controller
+# This YAML file shows how to deploy the volume-data-source-validator controller
 
-# The data-source-validator controller does asynchronous validation of PVC data sources
+# The volume-data-source-validator controller does asynchronous validation of PVC data
+# sources.
 # It should be installed as part of the base Kubernetes distribution in an appropriate
 # namespace for components implementing base system functionality. For installing with
 # Vanilla Kubernetes, kube-system makes sense for the namespace.
@@ -9,23 +10,23 @@
 kind: StatefulSet
 apiVersion: apps/v1
 metadata:
-  name: data-source-validator
+  name: volume-data-source-validator
   namespace: kube-system
 spec:
-  serviceName: data-source-validator
+  serviceName: volume-data-source-validator
   replicas: 1
   selector:
     matchLabels:
-      app: data-source-validator
+      app: volume-data-source-validator
   template:
     metadata:
       labels:
-        app: data-source-validator
+        app: volume-data-source-validator
     spec:
-      serviceAccount: data-source-validator
+      serviceAccount: volume-data-source-validator
       containers:
-        - name: data-source-validator
-          image: data-source-validator:latest
+        - name: volume-data-source-validator
+          image: volume-data-source-validator:latest
           args:
             - "--v=5"
             - "--leader-election=false"

deploy/kubernetes/rbac-data-source-validator.yaml

@@ -40,6 +40,7 @@ import (
 	"k8s.io/klog/v2"
 
 	volumesnapshotv1 "github.com/kubernetes-csi/external-snapshotter/client/v4/apis/volumesnapshot/v1"
+
 	popv1alpha1 "github.com/kubernetes-csi/volume-data-source-validator/client/apis/volumepopulator/v1alpha1"
 )
 
@@ -75,7 +76,7 @@ func NewDataSourceValidator(
 	broadcaster.StartLogging(klog.Infof)
 	broadcaster.StartRecordingToSink(&corev1.EventSinkImpl{Interface: client.CoreV1().Events(v1.NamespaceAll)})
 	var eventRecorder record.EventRecorder
-	eventRecorder = broadcaster.NewRecorder(scheme.Scheme, v1.EventSource{Component: fmt.Sprintf("data-source-validator")})
+	eventRecorder = broadcaster.NewRecorder(scheme.Scheme, v1.EventSource{Component: fmt.Sprintf("volume-data-source-validator")})
 
 	ctrl := &populatorController{
 		dynClient:     dynClient,
@@ -105,8 +106,8 @@ func NewDataSourceValidator(
 func (ctrl *populatorController) Run(workers int, stopCh <-chan struct{}) {
 	defer ctrl.queue.ShutDown()
 
-	klog.Infof("Starting data-source-validator controller")
-	defer klog.Infof("Shutting down data-source-validator controller")
+	klog.Infof("Starting volume-data-source-validator controller")
+	defer klog.Infof("Shutting down volume-data-source-validator controller")
 
 	if !cache.WaitForCacheSync(stopCh, ctrl.popListerSynced, ctrl.pvcListerSynced) {
 		klog.Errorf("Cannot sync caches")