From: Zack Cerza <zack@cerza.org>
Date: Thu, 8 Jan 2026 20:27:24 +0000 (-0700)
Subject: maas: Correct image names
X-Git-Tag: 1.2.3~8^2
X-Git-Url: http://git-server-git.apps.pok.os.sepia.ceph.com/?a=commitdiff_plain;h=refs%2Fpull%2F2105%2Fhead;p=teuthology.git

maas: Correct image names

Signed-off-by: Zack Cerza <zack@cerza.org>
---

diff --git a/teuthology/provision/maas.py b/teuthology/provision/maas.py
index ac13f7c04..b477d272f 100644
--- a/teuthology/provision/maas.py
+++ b/teuthology/provision/maas.py
@@ -193,6 +193,16 @@ class MAAS(object):
             )
         return resp[0]
 
+    def get_image_name(self) -> str:
+        match self.os_type:
+            case 'ubuntu':
+                os_version = OS._version_to_codename(self.os_type, self.os_version)
+                return f"{self.os_type}/{os_version}"
+            case 'centos':
+                os_version = self.os_version.replace('.', '-')
+                return f"{self.os_type}/{self.os_type}{os_version}"
+        return f"{self.os_type}/{self.os_version}"
+
     def get_image_data(self) -> Dict[str, Any]:
         """Locate the image we want to use
 
@@ -202,7 +212,7 @@ class MAAS(object):
         if len(resp) == 0:
             raise RuntimeError("MaaS has no images available")
 
-        name = self._normalize_image_name()
+        name = self.get_image_name()
         for image in resp:
             if image["name"] == name and self.cpu_arch in image["architecture"]:
                 return image
@@ -362,22 +372,6 @@ class MAAS(object):
         self.log.info(f"Releasing machine '{self.shortname}'")
         self.release_machine()
 
-    def _normalize_image_name(self) -> str:
-        match self.os_type:
-            case "ubuntu":
-                os_version = OS._version_to_codename(self.os_type, self.os_version)
-                return f"{self.os_type}/{os_version}"
-
-            case "centos":
-                os_version = self.os_version.replace('.', '-')
-                return f"{self.os_type}/{self.os_type}{os_version}"
-
-            case "rocky":
-                _os_version = self.os_version.split('.')[0]
-                return f"rocky{_os_version}"
-
-        return f"{self.os_type}/{self.os_version}"
-
     def _get_user_data(self) -> Optional[io.BytesIO]:
         """Get user data for cloud-init