diff --git a/pytest_fixtures/component/provision_pxe.py b/pytest_fixtures/component/provision_pxe.py index f422319c9a0..601750c3a33 100644 --- a/pytest_fixtures/component/provision_pxe.py +++ b/pytest_fixtures/component/provision_pxe.py @@ -238,13 +238,10 @@ def provision_multiple_hosts(module_ssh_key_file, pxe_loader, request): cd_iso = ( "" # TODO: Make this an optional fixture parameter (update vm_firmware when adding this) ) - # Keeping the default value to 2 - count = request.param if request.param is not None else 2 - with Broker( workflow="deploy-configure-pxe-provisioning-host-rhv", host_class=ContentHost, - _count=count, + _count=getattr(request, 'param', 2), target_vlan_id=vlan_id, target_vm_firmware=pxe_loader.vm_firmware, target_vm_cd_iso=cd_iso, diff --git a/tests/foreman/api/test_discoveredhost.py b/tests/foreman/api/test_discoveredhost.py index 1bf31452aeb..a82197a8d63 100644 --- a/tests/foreman/api/test_discoveredhost.py +++ b/tests/foreman/api/test_discoveredhost.py @@ -400,9 +400,7 @@ def test_positive_reboot_pxe_host( @pytest.mark.on_premises_provisioning @pytest.mark.parametrize('module_provisioning_sat', ['discovery'], indirect=True) - @pytest.mark.parametrize('pxe_loader', ['bios'], indirect=True) @pytest.mark.rhel_ver_match('9') - @pytest.mark.parametrize('provision_multiple_hosts', [2]) @pytest.mark.tier3 def test_positive_reboot_all_pxe_hosts( self, @@ -418,13 +416,15 @@ def test_positive_reboot_all_pxe_hosts( :parametrized: yes - :Setup: Provisioning should be configured and hosts should be discovered via PXE boot. + :setup: Provisioning should be configured and hosts should be discovered via PXE boot. :steps: PUT /api/v2/discovered_hosts/reboot_all :expectedresults: All discovered hosst should be rebooted successfully :CaseImportance: Medium + + :BZ: 2264195 """ sat = module_discovery_sat.sat for host in provision_multiple_hosts: @@ -440,8 +440,9 @@ def test_positive_reboot_all_pxe_hosts( discovered_host.location = provisioning_hostgroup.location[0] discovered_host.organization = provisioning_hostgroup.organization[0] discovered_host.build = True + # Until BZ 2264195 is resolved, reboot_all is expected to fail result = sat.api.DiscoveredHost().reboot_all() - assert 'Discovered hosts are rebooting now' in result['message'] + assert 'Discovered hosts are rebooting now' in result['success_msg'] class TestFakeDiscoveryTests: