Merge pull request #387 from SUSE/bsc1249593

Implement x86_64 architecture version check as a pre-check

Author: schaefi

package/suse-migration-sle16-activation-spec-template

@@ -46,7 +46,7 @@ install -D -m 755 grub.d/99_migration \
 
 %pre
 # Prevent installation for CPUs that are too old
-# SLE16 image panics on CPUs older than x86_64_v1 (i.e. x86_64) (bsc#1249593), so we need to block migration before getting into the image.
+# SLE16 image panics on CPUs older than x86_64_v2 (i.e. x86_64) (bsc#1249593), so we need to block migration before getting into the image.
 # We need the ZYPP_READONLY_HACK here since we are running inside a zypper transaction; without this the invocation of zypper will fail due to locking issues.
 if [ "$(ZYPP_READONLY_HACK=1 zypper system-architecture)" == "x86_64" ]; then
    echo "CPU Architecture too old. Aborting installation."

suse_migration_services/migration_target.py

@@ -0,0 +1,77 @@
+
+# Copyright (c) 2025 SUSE Linux LLC.  All rights reserved.
+#
+# This file is part of suse-migration-services.
+#
+# suse-migration-services is free software: you can redistribute it and/or
+# modify it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, either version 3 of the License, or
+# (at your option) any later version.
+#
+# suse-migration-services is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with suse-migration-services. If not, see <http://www.gnu.org/licenses/>
+#
+import logging
+import yaml
+import os
+import platform
+from glob import glob
+from fnmatch import fnmatch
+
+from suse_migration_services.defaults import Defaults
+
+log = logging.getLogger(Defaults.get_migration_log_name())
+
+
+class MigrationTarget:
+    """Implements the detection of migration target"""
+    @staticmethod
+    def _parse_custom_config():
+        migration_config = '/etc/sle-migration-service.yml'
+        if os.path.isfile(migration_config):
+            try:
+                with open(migration_config, 'r') as config:
+                    return yaml.safe_load(config) or {}
+            except Exception as issue:
+                message = 'Loading {0} failed: {1}: {2}'.format(
+                    migration_config, type(issue).__name__, issue
+                )
+                log.error(message)
+
+        return {}
+
+    @staticmethod
+    def get_migration_target():
+        config_data = MigrationTarget._parse_custom_config()
+        migration_product = config_data.get('migration_product')
+        if migration_product:
+            product = migration_product.split('/')
+            return {
+                'identifier': product[0],
+                'version': product[1],
+                'arch': product[2]
+            }
+        migration_iso = glob('/migration-image/*-*Migration.*.iso')
+        if not migration_iso:
+            return {}
+        migration_iso = migration_iso[0]
+
+        if fnmatch(migration_iso, '*SLES16*-*Migration*.iso'):
+            # return default migration target
+            return {
+                'identifier': 'SLES',
+                'version': '16.0',
+                'arch': platform.machine()
+            }
+        else:
+            # return default migration target
+            return {
+                'identifier': 'SLES',
+                'version': '15.4',
+                'arch': platform.machine()
+            }

suse_migration_services/prechecks/cpu_arch.py

@@ -0,0 +1,50 @@
+# Copyright (c) 2025
+#
+# SUSE Linux LLC.  All rights reserved.
+#
+# This file is part of suse-migration-services.
+#
+# suse-migration-services is free software: you can redistribute it and/or
+# modify it under the terms of the GNU General Public License as published by
+# the Free Software Foundation, either version 3 of the License, or
+# (at your option) any later version.
+#
+# suse-migration-services is distributed in the hope that it will be useful,
+# but WITHOUT ANY WARRANTY; without even the implied warranty of
+# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the
+# GNU General Public License for more details.
+#
+# You should have received a copy of the GNU General Public License
+# along with suse-migration-services. If not, see <http://www.gnu.org/licenses/>
+"""Module for checking for CPU architecture support"""
+import logging
+import os
+
+from suse_migration_services.defaults import Defaults
+from suse_migration_services.command import Command
+from suse_migration_services.migration_target import MigrationTarget
+
+
+def x86_64_version():
+    """Function to check whether current CPU architecture is new enough to support SLE16"""
+    target = MigrationTarget.get_migration_target()
+
+    if target.get('version') != '16.0':
+        # This check is only necessary for migration to SLE16
+        return
+
+    log = logging.getLogger(Defaults.get_migration_log_name())
+    os.environ['ZYPP_READONLY_HACK'] = '1'
+    zypper_call = Command.run(['zypper', 'system-architecture'])
+    arch = zypper_call.output
+
+    if arch.strip().lower() == "x86_64":
+        log.error(
+            'SLE16 requires x86_64_v2 at minimum. The architecture version '
+            'of this system is too old.'
+        )
+
+
+def cpu_arch():
+    """Function for CPU architecture related checks"""
+    x86_64_version()

suse_migration_services/prechecks/pre_checks.py

@@ -30,6 +30,7 @@
 import suse_migration_services.prechecks.kernels as check_multi_kernels
 import suse_migration_services.prechecks.scc as check_scc
 import suse_migration_services.prechecks.wicked2nm as check_wicked2nm
+import suse_migration_services.prechecks.cpu_arch as check_cpu_arch
 
 
 def main():
@@ -92,6 +93,9 @@ def main():
 
     log.info('Checking harmful migration conditions')
 
+    log.info('--> Checking system architecture version...')
+    check_cpu_arch.cpu_arch()
+
     log.info('--> Checking for local private repos...')
     check_repos.remote_repos(
         migration_system=migration_system_mode

suse_migration_services/prechecks/scc.py

@@ -15,15 +15,14 @@
 # You should have received a copy of the GNU General Public License
 # along with suse-migration-services. If not, see <http://www.gnu.org/licenses/>#
 from lxml import etree
-import platform
 import requests
 import yaml
 import logging
 import os
 import glob
 
-from suse_migration_services.command import Command
 from suse_migration_services.defaults import Defaults
+from suse_migration_services.migration_target import MigrationTarget
 
 log = logging.getLogger(Defaults.get_migration_log_name())
 
@@ -43,7 +42,7 @@ def migration(migration_system=False):
         return
 
     installed_products = get_installed_products()
-    installed_products['target_base_product'] = get_migration_target()
+    installed_products['target_base_product'] = MigrationTarget.get_migration_target()
 
     if installed_products.get('installed_products') and \
        installed_products.get('target_base_product'):
@@ -129,51 +128,6 @@ def get_registration_server_url():
             log.error(message)
 
 
-def get_migration_target():
-    migration_config = '/etc/sle-migration-service.yml'
-    config_data = {}
-    if os.path.isfile(migration_config):
-        try:
-            with open(migration_config, 'r') as config:
-                config_data = yaml.safe_load(config) or {}
-        except Exception as issue:
-            message = 'Loading {0} failed: {1}: {2}'.format(
-                migration_config, type(issue).__name__, issue
-            )
-            log.error(message)
-            return {}
-
-    migration_product = config_data.get('migration_product')
-    if migration_product:
-        product = migration_product.split('/')
-        return {
-            'identifier': product[0],
-            'version': product[1],
-            'arch': product[2]
-        }
-    sles15_migration = Command.run(
-        ['rpm', '-q', 'SLES15-Migration'], raise_on_error=False
-    )
-    if sles15_migration.returncode == 0:
-        # return default migration target
-        return {
-            'identifier': 'SLES',
-            'version': '15.3',
-            'arch': platform.machine()
-        }
-    sles16_migration = Command.run(
-        ['rpm', '-q', 'SLES16-Migration'], raise_on_error=False
-    )
-    if sles16_migration.returncode == 0:
-        # return default migration target
-        return {
-            'identifier': 'SLES',
-            'version': '16.0',
-            'arch': platform.machine()
-        }
-    return {}
-
-
 def get_installed_products():
     installed_products = {
         'installed_products': []

test/unit/migration_target_test.py

@@ -0,0 +1,55 @@
+from unittest.mock import (
+    patch
+)
+
+from suse_migration_services.migration_target import MigrationTarget
+
+
+class TestMigrationTarget(object):
+    @patch('platform.machine')
+    @patch('os.path.isfile')
+    @patch('suse_migration_services.migration_target.glob')
+    def test_get_migration_target_empty(
+        self, mock_glob, mock_os_path_isfile,
+        mock_platform_machine
+    ):
+        mock_platform_machine.return_value = 'x86_64'
+        mock_glob.return_value = []
+        mock_os_path_isfile.return_value = False
+        assert MigrationTarget.get_migration_target() == {}
+
+    @patch('platform.machine')
+    @patch('os.path.isfile')
+    @patch('suse_migration_services.migration_target.glob')
+    def test_get_migration_target_sles15(
+        self, mock_glob, mock_os_path_isfile,
+        mock_platform_machine
+    ):
+        mock_platform_machine.return_value = 'x86_64'
+        mock_glob.return_value = [
+            '/migration-image/SLES15-Migration.x86_64-2.1.9-Build6.64.99.iso'
+        ]
+        mock_os_path_isfile.return_value = False
+        assert MigrationTarget.get_migration_target() == {
+            'identifier': 'SLES',
+            'version': '15.4',
+            'arch': 'x86_64'
+        }
+
+    @patch('platform.machine')
+    @patch('os.path.isfile')
+    @patch('suse_migration_services.migration_target.glob')
+    def test_get_migration_target_sles16(
+        self, mock_glob, mock_os_path_isfile,
+        mock_platform_machine
+    ):
+        mock_glob.return_value = [
+            '/migration-image/SLES16-Migration.x86_64-2.1.23-Build2.1.iso'
+        ]
+        mock_platform_machine.return_value = 'x86_64'
+        mock_os_path_isfile.return_value = False  # No /etc/sle-migration-service.yml
+        assert MigrationTarget.get_migration_target() == {
+            'identifier': 'SLES',
+            'version': '16.0',
+            'arch': 'x86_64'
+        }