testing discovery provisioning failure

Signed-off-by: Shubham Ganar <[email protected]>
SatelliteQE · Apr 12, 2024 · 0673b9e · 0673b9e
1 parent 483aecc
commit 0673b9e
Show file tree

Hide file tree

Showing 3 changed files with 116 additions and 118 deletions.
diff --git a/tests/foreman/api/test_provisioning.py b/tests/foreman/api/test_provisioning.py
@@ -61,9 +61,9 @@ def assert_host_logs(channel, pattern):
 
 @pytest.mark.e2e
 @pytest.mark.upgrade
-@pytest.mark.parametrize('pxe_loader', ['bios', 'uefi'], indirect=True)
+@pytest.mark.parametrize('pxe_loader', ['bios'], indirect=True)
 @pytest.mark.on_premises_provisioning
-@pytest.mark.rhel_ver_match('[^6]')
+@pytest.mark.rhel_ver_match('[9]')
 def test_rhel_pxe_provisioning(
     request,
     module_provisioning_sat,
@@ -130,28 +130,39 @@ def test_rhel_pxe_provisioning(
     )
     host = host.read()
     assert host.build_status_label == 'Installed'
-
     # Change the hostname of the host as we know it already.
     # In the current infra environment we do not support
     # addressing hosts using FQDNs, falling back to IP.
     provisioning_host.hostname = host.ip
     # Host is not blank anymore
     provisioning_host.blank = False
 
+    provisioning_host.key_filename = None
     # Wait for the host to be rebooted and SSH daemon to be started.
     provisioning_host.wait_for_connection()
 
+    disable_ssh_cmd = 'echo -e "\nPermitRootLogin yes" >> /etc/ssh/sshd_config; systemctl restart sshd'
+    # Run a command on the host using REX to verify that Satellite's SSH key is present on the host
+    template_id = (
+        sat.api.JobTemplate().search(query={'search': 'name="Run Command - Script Default"'})[0].id
+    )
+    job = sat.api.JobInvocation().run(
+        data={
+            'job_template_id': template_id,
+            'inputs': {
+                'command': disable_ssh_cmd
+            },
+            'search_query': f"name = {host.name}",
+            'targeting_type': 'static_query',
+        },
+    )
+    assert job['result'] == 'success', 'Job invocation failed'
+
     # Perform version check and check if root password is properly updated
     host_os = host.operatingsystem.read()
     expected_rhel_version = f'{host_os.major}.{host_os.minor}'
 
-    if int(host_os.major) >= 9:
-        assert (
-            provisioning_host.execute(
-                'echo -e "\nPermitRootLogin yes" >> /etc/ssh/sshd_config; systemctl restart sshd'
-            ).status
-            == 0
-        )
+    disable_ssh_cmd = 'echo -e "\nPermitRootLogin yes" >> /etc/ssh/sshd_config; systemctl restart sshd'
     host_ssh_os = sat.execute(
         f'sshpass -p {settings.provisioning.host_root_password} '
         'ssh -o StrictHostKeyChecking=no -o PubkeyAuthentication=no -o PasswordAuthentication=yes '
@@ -165,23 +176,6 @@ def test_rhel_pxe_provisioning(
     # Verify provisioning log exists on host at correct path
     assert provisioning_host.execute('test -s /root/install.post.log').status == 0
     assert provisioning_host.execute('test -s /mnt/sysimage/root/install.post.log').status == 1
-
-    # Run a command on the host using REX to verify that Satellite's SSH key is present on the host
-    template_id = (
-        sat.api.JobTemplate().search(query={'search': 'name="Run Command - Script Default"'})[0].id
-    )
-    job = sat.api.JobInvocation().run(
-        data={
-            'job_template_id': template_id,
-            'inputs': {
-                'command': f'subscription-manager config | grep "hostname = {sat.hostname}"'
-            },
-            'search_query': f"name = {host.name}",
-            'targeting_type': 'static_query',
-        },
-    )
-    assert job['result'] == 'success', 'Job invocation failed'
-
     # check if katello-ca-consumer is not used while host registration
     assert provisioning_host.execute('rpm -qa |grep katello-ca-consumer').status == 1
     assert (
@@ -277,30 +271,11 @@ def test_rhel_ipxe_provisioning(
     # Host is not blank anymore
     provisioning_host.blank = False
 
+    provisioning_host.key_filename = None
     # Wait for the host to be rebooted and SSH daemon to be started.
     provisioning_host.wait_for_connection()
 
-    # Perform version check and check if root password is properly updated
-    host_os = host.operatingsystem.read()
-    expected_rhel_version = f'{host_os.major}.{host_os.minor}'
-
-    if int(host_os.major) >= 9:
-        assert (
-            provisioning_host.execute(
-                'echo -e "\nPermitRootLogin yes" >> /etc/ssh/sshd_config; systemctl restart sshd'
-            ).status
-            == 0
-        )
-    host_ssh_os = sat.execute(
-        f'sshpass -p {settings.provisioning.host_root_password} '
-        'ssh -o StrictHostKeyChecking=no -o PubkeyAuthentication=no -o PasswordAuthentication=yes '
-        f'-o UserKnownHostsFile=/dev/null root@{provisioning_host.hostname} cat /etc/redhat-release'
-    )
-    assert host_ssh_os.status == 0
-    assert (
-        expected_rhel_version in host_ssh_os.stdout
-    ), f'The installed OS version differs from the expected version {expected_rhel_version}'
-
+    disable_ssh_cmd = 'echo -e "\nPermitRootLogin yes" >> /etc/ssh/sshd_config; systemctl restart sshd'
     # Run a command on the host using REX to verify that Satellite's SSH key is present on the host
     template_id = (
         sat.api.JobTemplate().search(query={'search': 'name="Run Command - Script Default"'})[0].id
@@ -309,14 +284,28 @@ def test_rhel_ipxe_provisioning(
         data={
             'job_template_id': template_id,
             'inputs': {
-                'command': f'subscription-manager config | grep "hostname = {sat.hostname}"'
+                'command': disable_ssh_cmd
             },
             'search_query': f"name = {host.name}",
             'targeting_type': 'static_query',
         },
     )
     assert job['result'] == 'success', 'Job invocation failed'
 
+    # Perform version check and check if root password is properly updated
+    host_os = host.operatingsystem.read()
+    expected_rhel_version = f'{host_os.major}.{host_os.minor}'
+
+    disable_ssh_cmd = 'echo -e "\nPermitRootLogin yes" >> /etc/ssh/sshd_config; systemctl restart sshd'
+    host_ssh_os = sat.execute(
+        f'sshpass -p {settings.provisioning.host_root_password} '
+        'ssh -o StrictHostKeyChecking=no -o PubkeyAuthentication=no -o PasswordAuthentication=yes '
+        f'-o UserKnownHostsFile=/dev/null root@{provisioning_host.hostname} cat /etc/redhat-release'
+    )
+    assert host_ssh_os.status == 0
+    assert (
+        expected_rhel_version in host_ssh_os.stdout
+    ), f'The installed OS version differs from the expected version {expected_rhel_version}'
     # assert that the host is subscribed and consumes
     # subsctiption provided by the activation key
     assert provisioning_host.subscribed, 'Host is not subscribed'
@@ -405,30 +394,11 @@ def test_rhel_httpboot_provisioning(
     # Host is not blank anymore
     provisioning_host.blank = False
 
+    provisioning_host.key_filename = None
     # Wait for the host to be rebooted and SSH daemon to be started.
     provisioning_host.wait_for_connection()
 
-    # Perform version check and check if root password is properly updated
-    host_os = host.operatingsystem.read()
-    expected_rhel_version = f'{host_os.major}.{host_os.minor}'
-
-    if int(host_os.major) >= 9:
-        assert (
-            provisioning_host.execute(
-                'echo -e "\nPermitRootLogin yes" >> /etc/ssh/sshd_config; systemctl restart sshd'
-            ).status
-            == 0
-        )
-    host_ssh_os = sat.execute(
-        f'sshpass -p {settings.provisioning.host_root_password} '
-        'ssh -o StrictHostKeyChecking=no -o PubkeyAuthentication=no -o PasswordAuthentication=yes '
-        f'-o UserKnownHostsFile=/dev/null root@{provisioning_host.hostname} cat /etc/redhat-release'
-    )
-    assert host_ssh_os.status == 0
-    assert (
-        expected_rhel_version in host_ssh_os.stdout
-    ), f'The installed OS version differs from the expected version {expected_rhel_version}'
-
+    disable_ssh_cmd = 'echo -e "\nPermitRootLogin yes" >> /etc/ssh/sshd_config; systemctl restart sshd'
     # Run a command on the host using REX to verify that Satellite's SSH key is present on the host
     template_id = (
         sat.api.JobTemplate().search(query={'search': 'name="Run Command - Script Default"'})[0].id
@@ -437,14 +407,28 @@ def test_rhel_httpboot_provisioning(
         data={
             'job_template_id': template_id,
             'inputs': {
-                'command': f'subscription-manager config | grep "hostname = {sat.hostname}"'
+                'command': disable_ssh_cmd
             },
             'search_query': f"name = {host.name}",
             'targeting_type': 'static_query',
         },
     )
     assert job['result'] == 'success', 'Job invocation failed'
 
+    # Perform version check and check if root password is properly updated
+    host_os = host.operatingsystem.read()
+    expected_rhel_version = f'{host_os.major}.{host_os.minor}'
+
+    disable_ssh_cmd = 'echo -e "\nPermitRootLogin yes" >> /etc/ssh/sshd_config; systemctl restart sshd'
+    host_ssh_os = sat.execute(
+        f'sshpass -p {settings.provisioning.host_root_password} '
+        'ssh -o StrictHostKeyChecking=no -o PubkeyAuthentication=no -o PasswordAuthentication=yes '
+        f'-o UserKnownHostsFile=/dev/null root@{provisioning_host.hostname} cat /etc/redhat-release'
+    )
+    assert host_ssh_os.status == 0
+    assert (
+        expected_rhel_version in host_ssh_os.stdout
+    ), f'The installed OS version differs from the expected version {expected_rhel_version}'
     # assert that the host is subscribed and consumes
     # subsctiption provided by the activation key
     assert provisioning_host.subscribed, 'Host is not subscribed'
@@ -534,20 +518,32 @@ def test_rhel_pxe_provisioning_fips_enabled(
     # Host is not blank anymore
     provisioning_host.blank = False
 
+    provisioning_host.key_filename = None
     # Wait for the host to be rebooted and SSH daemon to be started.
     provisioning_host.wait_for_connection()
 
+    disable_ssh_cmd = 'echo -e "\nPermitRootLogin yes" >> /etc/ssh/sshd_config; systemctl restart sshd'
+    # Run a command on the host using REX to verify that Satellite's SSH key is present on the host
+    template_id = (
+        sat.api.JobTemplate().search(query={'search': 'name="Run Command - Script Default"'})[0].id
+    )
+    job = sat.api.JobInvocation().run(
+        data={
+            'job_template_id': template_id,
+            'inputs': {
+                'command': disable_ssh_cmd
+            },
+            'search_query': f"name = {host.name}",
+            'targeting_type': 'static_query',
+        },
+    )
+    assert job['result'] == 'success', 'Job invocation failed'
+
     # Perform version check and check if root password is properly updated
     host_os = host.operatingsystem.read()
     expected_rhel_version = f'{host_os.major}.{host_os.minor}'
 
-    if int(host_os.major) >= 9:
-        assert (
-            provisioning_host.execute(
-                'echo -e "\nPermitRootLogin yes" >> /etc/ssh/sshd_config; systemctl restart sshd'
-            ).status
-            == 0
-        )
+    disable_ssh_cmd = 'echo -e "\nPermitRootLogin yes" >> /etc/ssh/sshd_config; systemctl restart sshd'
     host_ssh_os = sat.execute(
         f'sshpass -p {settings.provisioning.host_root_password} '
         'ssh -o StrictHostKeyChecking=no -o PubkeyAuthentication=no -o PasswordAuthentication=yes '
@@ -566,23 +562,6 @@ def test_rhel_pxe_provisioning_fips_enabled(
     else:
         result = provisioning_host.execute('cat /proc/sys/crypto/fips_enabled')
         assert (0 if is_open('BZ:2240076') else 1) == int(result.stdout)
-
-    # Run a command on the host using REX to verify that Satellite's SSH key is present on the host
-    template_id = (
-        sat.api.JobTemplate().search(query={'search': 'name="Run Command - Script Default"'})[0].id
-    )
-    job = sat.api.JobInvocation().run(
-        data={
-            'job_template_id': template_id,
-            'inputs': {
-                'command': f'subscription-manager config | grep "hostname = {sat.hostname}"'
-            },
-            'search_query': f"name = {host.name}",
-            'targeting_type': 'static_query',
-        },
-    )
-    assert job['result'] == 'success', 'Job invocation failed'
-
     # assert that the host is subscribed and consumes
     # subsctiption provided by the activation key
     assert provisioning_host.subscribed, 'Host is not subscribed'
@@ -670,30 +649,11 @@ def test_capsule_pxe_provisioning(
     # Host is not blank anymore
     provisioning_host.blank = False
 
+    provisioning_host.key_filename = None
     # Wait for the host to be rebooted and SSH daemon to be started.
     provisioning_host.wait_for_connection()
 
-    # Perform version check and check if root password is properly updated
-    host_os = host.operatingsystem.read()
-    expected_rhel_version = f'{host_os.major}.{host_os.minor}'
-
-    if int(host_os.major) >= 9:
-        assert (
-            provisioning_host.execute(
-                'echo -e "\nPermitRootLogin yes" >> /etc/ssh/sshd_config; systemctl restart sshd'
-            ).status
-            == 0
-        )
-    host_ssh_os = sat.execute(
-        f'sshpass -p {settings.provisioning.host_root_password} '
-        'ssh -o StrictHostKeyChecking=no -o PubkeyAuthentication=no -o PasswordAuthentication=yes '
-        f'-o UserKnownHostsFile=/dev/null root@{provisioning_host.hostname} cat /etc/redhat-release'
-    )
-    assert host_ssh_os.status == 0
-    assert (
-        expected_rhel_version in host_ssh_os.stdout
-    ), f'The installed OS version differs from the expected version {expected_rhel_version}'
-
+    disable_ssh_cmd = 'echo -e "\nPermitRootLogin yes" >> /etc/ssh/sshd_config; systemctl restart sshd'
     # Run a command on the host using REX to verify that Satellite's SSH key is present on the host
     template_id = (
         sat.api.JobTemplate().search(query={'search': 'name="Run Command - Script Default"'})[0].id
@@ -702,14 +662,28 @@ def test_capsule_pxe_provisioning(
         data={
             'job_template_id': template_id,
             'inputs': {
-                'command': f'subscription-manager config | grep "hostname = {cap.hostname}"'
+                'command': disable_ssh_cmd
             },
             'search_query': f"name = {host.name}",
             'targeting_type': 'static_query',
         },
     )
     assert job['result'] == 'success', 'Job invocation failed'
 
+    # Perform version check and check if root password is properly updated
+    host_os = host.operatingsystem.read()
+    expected_rhel_version = f'{host_os.major}.{host_os.minor}'
+
+    disable_ssh_cmd = 'echo -e "\nPermitRootLogin yes" >> /etc/ssh/sshd_config; systemctl restart sshd'
+    host_ssh_os = sat.execute(
+        f'sshpass -p {settings.provisioning.host_root_password} '
+        'ssh -o StrictHostKeyChecking=no -o PubkeyAuthentication=no -o PasswordAuthentication=yes '
+        f'-o UserKnownHostsFile=/dev/null root@{provisioning_host.hostname} cat /etc/redhat-release'
+    )
+    assert host_ssh_os.status == 0
+    assert (
+        expected_rhel_version in host_ssh_os.stdout
+    ), f'The installed OS version differs from the expected version {expected_rhel_version}'
     # assert that the host is subscribed and consumes
     # subsctiption provided by the activation key
     assert provisioning_host.subscribed, 'Host is not subscribed'

diff --git a/tests/foreman/api/test_provisioning_puppet.py b/tests/foreman/api/test_provisioning_puppet.py
@@ -180,9 +180,31 @@ def test_host_provisioning_with_external_puppetserver(
     # Host is not blank anymore
     provisioning_host.blank = False
 
+    provisioning_host.key_filename = None
     # Wait for the host to be rebooted and SSH daemon to be started.
     provisioning_host.wait_for_connection()
 
+    disable_ssh_cmd = 'echo -e "\nPermitRootLogin yes" >> /etc/ssh/sshd_config; systemctl restart sshd'
+    # Run a command on the host using REX to verify that Satellite's SSH key is present on the host
+    template_id = (
+        sat.api.JobTemplate().search(query={'search': 'name="Run Command - Script Default"'})[0].id
+    )
+    job = sat.api.JobInvocation().run(
+        data={
+            'job_template_id': template_id,
+            'inputs': {
+                'command': disable_ssh_cmd
+            },
+            'search_query': f"name = {host.name}",
+            'targeting_type': 'static_query',
+        },
+    )
+    assert job['result'] == 'success', 'Job invocation failed'
+
+    # Perform version check and check if root password is properly updated
+    host_os = host.operatingsystem.read()
+    expected_rhel_version = f'{host_os.major}.{host_os.minor}'
+
     # Perform version check
     host_os = host.operatingsystem.read()
     expected_rhel_version = Version(f'{host_os.major}.{host_os.minor}')

diff --git a/tests/foreman/cli/test_discoveredhost.py b/tests/foreman/cli/test_discoveredhost.py
@@ -59,6 +59,7 @@ def test_rhel_pxe_discovery_provisioning(
     discovered_host.location = provisioning_hostgroup.location[0]
     discovered_host.organization = provisioning_hostgroup.organization[0]
     discovered_host.build = True
+    discovered_host.update(['hostgroup', 'location', 'organization', 'build'])
     result = sat.cli.DiscoveredHost.provision(
         {
             'id': discovered_host.id,
@@ -101,7 +102,7 @@ def test_rhel_pxeless_discovery_provisioning(
     :Setup: Provisioning should be configured and a host should be
         discovered via cli
 
-    :expectedresults: Host should be provisioned successfully
+    :expectedresults: Host should be provisioned successfully.
 
     :CaseImportance: Critical
     """
@@ -119,6 +120,7 @@ def test_rhel_pxeless_discovery_provisioning(
     discovered_host.location = provisioning_hostgroup.location[0]
     discovered_host.organization = provisioning_hostgroup.organization[0]
     discovered_host.build = True
+    discovered_host.update(['hostgroup', 'location', 'organization', 'build'])
     result = sat.cli.DiscoveredHost.provision(
         {
             'id': discovered_host.id,