Skip to content
This repository has been archived by the owner on Apr 7, 2022. It is now read-only.

[RFR] fix cloud-init tests #10264

Merged
merged 3 commits into from
Aug 19, 2020
Merged

[RFR] fix cloud-init tests #10264

merged 3 commits into from
Aug 19, 2020

Conversation

jarovo
Copy link
Contributor

@jarovo jarovo commented Jul 22, 2020

Purpose or Intent

  • Fixing cloud-init tests that didn't work as they needed IPv6 connectivity between the test executing machine and the VM on provider. This was solved by tunnelling to the provider host and using any IP reported by provder that is connective from the host.
    Adding test for DHCP configuration.

{{py.test: -sv cfme/tests/cloud_infra_common/test_cloud_init_provisioning.py::test_provision_cloud_init_payload --long-running --use-provider complete }}

@jarovo jarovo changed the title fix cloud-init tests [WIPTEST] fix cloud-init tests Jul 22, 2020
@jarovo jarovo changed the title [WIPTEST] fix cloud-init tests [RFR] fix cloud-init tests Jul 27, 2020
'custom_template': {'name': 'oVirt cloud-init'}
}

THERE_IS_A_WAY_TO_CHECK_STATIC_IP_VM = True
# TODO remove this condition
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yes please do :)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

@jarovo jarovo changed the title [RFR] fix cloud-init tests [WIPTEST] fix cloud-init tests Aug 10, 2020
@jarovo jarovo changed the title [WIPTEST] fix cloud-init tests [RFR] fix cloud-init tests Aug 12, 2020
username='root', password=static_mode_payload['root_password'],
timeout=ssh.CONNECT_TIMEOUT)

# Cleanup this explicitly because we can get to problems with ordering the cleanups of
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for including this explanation of the explicit closes!

@mshriver mshriver merged commit 2b861a3 into ManageIQ:master Aug 19, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants