Move rl_obj_name before rbac_objs

Author: simu

lib/resource-locker.libjsonnet

@@ -95,6 +95,27 @@ local clusterRoleName(name) =
 local replaceColon(str) =
   std.strReplace(str, ':', '-');
 
+local rl_obj_name(objdata) =
+  // Some objects like ClusterRoleBinding can contain colons.
+  local name = replaceColon(objdata.name);
+  if objdata.namespace != null then
+    '%s-%s' % [ objdata.namespace, name ]
+  else
+    // Reduce potential for name collisions when generating names for
+    // cluster-scoped ResourceLocker configurations.
+    local prefix =
+      if objdata.apigroup != '' then
+        '%s-%s' % [
+          std.asciiLower(objdata.kind),
+          std.strReplace(objdata.apigroup, '.', '-'),
+        ]
+      else
+        std.asciiLower(objdata.kind);
+    '%s-%s' % [
+      prefix,
+      name,
+    ];
+
 local rbac_objs(objdata, verbs=[ 'create', 'get', 'update', 'patch' ]) =
   local dest_ns = objdata.namespace;
   // Some objects like ClusterRoleBinding can contain colons.
@@ -162,27 +183,6 @@ local obj_data(obj) =
     namespace: if std.objectHas(obj.metadata, 'namespace') then obj.metadata.namespace,
   };
 
-local rl_obj_name(objdata) =
-  // Some objects like ClusterRoleBinding can contain colons.
-  local name = replaceColon(objdata.name);
-  if objdata.namespace != null then
-    '%s-%s' % [ objdata.namespace, name ]
-  else
-    // Reduce potential for name collisions when generating names for
-    // cluster-scoped ResourceLocker configurations.
-    local prefix =
-      if objdata.apigroup != '' then
-        '%s-%s' % [
-          std.asciiLower(objdata.kind),
-          std.strReplace(objdata.apigroup, '.', '-'),
-        ]
-      else
-        std.asciiLower(objdata.kind);
-    '%s-%s' % [
-      prefix,
-      name,
-    ];
-
 /**
  * \brief Create a managed resource (similar to Espejo, but for single NS)
  *