Merge pull request #31 from anfredette/app-crd-2

Fixups for pr #6
bpfman · Jun 21, 2024 · 1353f8e · 1353f8e
2 parents 4c3fa0b + c71264c
commit 1353f8e
Show file tree

Hide file tree

Showing 2 changed files with 8 additions and 1 deletion.
diff --git a/controllers/bpfman-agent/application-program.go b/controllers/bpfman-agent/application-program.go
@@ -280,5 +280,11 @@ func (r *BpfApplicationReconciler) SetupWithManager(mgr ctrl.Manager) error {
 			&handler.EnqueueRequestForObject{},
 			builder.WithPredicates(predicate.And(predicate.LabelChangedPredicate{}, nodePredicate(r.NodeName))),
 		).
+		// Watch for changes in Pod resources in case we are using a container selector.
+		Watches(
+			&v1.Pod{},
+			&handler.EnqueueRequestForObject{},
+			builder.WithPredicates(podOnNodePredicate(r.NodeName)),
+		).
 		Complete(r)
 }
diff --git a/controllers/bpfman-agent/common.go b/controllers/bpfman-agent/common.go
@@ -896,5 +896,6 @@ func getClientset() (*kubernetes.Clientset, error) {
 // sanitize a string to work as a bpfProgram name
 func sanitize(name string) string {
 	name = strings.TrimPrefix(name, "/")
-	return strings.Replace(strings.Replace(name, "/", "-", -1), "_", "-", -1)
+	name = strings.Replace(strings.Replace(name, "/", "-", -1), "_", "-", -1)
+	return strings.ToLower(name)
 }