make bundle tag names configurable (#532)

doozerlib/distgit.py

@@ -1867,6 +1867,7 @@ def _update_csv(self, version, release):
         registry = csv_config['registry'].rstrip("/")
         image_map = csv_config.get('image-map', {})
 
+        name_in_bundle = self.metadata.config.get('name_in_bundle', None)
         for ref in image_refs:
             try:
                 name = build_image_ref_name(ref['name'])
@@ -1876,10 +1877,15 @@ def _update_csv(self, version, release):
                 raise DoozerFatalError('Error loading image-references data for {}'.format(self.metadata.distgit_key))
 
             try:
-                if name == self.metadata.image_name_short:  # ref is current image
+                # ref is current image
+                if name == self.metadata.image_name_short or name_in_bundle and ref['name'] == name_in_bundle:
                     image_tag = '{}:{}-{}'.format(name, version, release)
                 else:
-                    distgit = self.runtime.image_distgit_by_name(name)
+                    # try to find by ref['name'], which is enabled by name_in_bundle field
+                    # indexed in map at initialization
+                    distgit = self.runtime.image_distgit_by_name(ref['name'])
+                    if not distgit:
+                        distgit = self.runtime.image_distgit_by_name(name)
                     # if upstream is referring to an image we don't actually build, give up.
                     if not distgit:
                         raise DoozerFatalError('Unable to find {} in image-references data for {}'.format(name, self.metadata.distgit_key))

doozerlib/runtime.py

@@ -593,6 +593,15 @@ def filter_disabled(n, d):
                 self.image_name_map[name] = img.key
                 self.image_name_map[short_name] = img.key
 
+                name_in_bundle = img.data.get('name_in_bundle', None)
+                if name_in_bundle and name_in_bundle!=name and name_in_bundle!=name:
+                    if name_in_bundle in self.image_name_map:
+                        self.logger.warning(
+                            f"name_in_bundle field in conflict with another image name/shortname, for image={name} "
+                            f"name_in_bundle={name_in_bundle}.")
+                    else:
+                        self.image_name_map[name_in_bundle] = img.key
+
             image_data = self.gitdata.load_data(path='images', keys=image_keys,
                                                 exclude=image_ex,
                                                 replace_vars=replace_vars,
@@ -847,7 +856,7 @@ def add_child_branch(child, branch, level=1):
                     self.image_order.append(i)
 
     def image_distgit_by_name(self, name):
-        """Returns image meta by full name, short name, or distgit"""
+        """Returns image meta by full name, short name, name_in_bundle or distgit"""
         return self.image_name_map.get(name, None)
 
     def rpm_metas(self) -> List[RPMMetadata]: