summaryrefslogtreecommitdiffstats
AgeCommit message (Collapse)AuthorFilesLines
2018-02-20openstack: nova_utils_tests: Wait for volume detachmentMarkos Chandras1-6/+14
The 'test_add_remove_volum' is supposed to wait for volume attachment and detachments. However, the code for the latter was missing. Moreover, we remove some code that checks for attached volumes since that should never happen when detaching the volume. Change-Id: I9ed54a467062d0ddc4efeddf03af7ebe6173e789 Signed-off-by: Markos Chandras <mchandras@suse.de>
2018-02-20openstack: nova_utils_tests: Increase timeout for API callsMarkos Chandras1-2/+2
Patch I2245adb84d0e6d4e3350d17d0c2e44baf5202d51 increased the timeout for the API calls to 2 minutes which was then somewhat reverted in Iea3aeab59c378917fbd175d673113e8d30e2e4b9. However, the problem still persists on slow systems and 30 seconds may not be enough to confirm a volume attachment to the VM. Change-Id: I860b2a797917e13772e95c5d756919e9792b63a6 Signed-off-by: Markos Chandras <mchandras@suse.de>
2018-02-16Ensure project IDs are handled correctly for Network/Subnetsspisarski20-137/+335
JIRA: SNAPS-265 Change-Id: If58f5817e52e21be377d2b06a5f7c3a413454350 Signed-off-by: spisarski <s.pisarski@cablelabs.com>
2018-02-15Added functionality for OpenStackSecurityGroup#initialize() not to acceptspisarski10-33/+132
security groups with the same name from a different project JIRA: SNAPS-264 Change-Id: I7c905e5588d5e503b47ea0e9f5997be89e841aec Signed-off-by: spisarski <s.pisarski@cablelabs.com>
2018-02-15Merge "Compare strings instead of objects in SecurityGroupConfig"Steven Pisarski1-1/+1
2018-02-15Merge "Set region_name in get_os_creds()"Steven Pisarski1-0/+1
2018-02-14Compare strings instead of objects in SecurityGroupConfigCédric Ollivier1-1/+1
It could have worked thanks to interned strings. Change-Id: If56cc44fe35ca06bc9033af620ddd3c0586877fe Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
2018-02-14Merge "Fixed comment"Steven Pisarski1-1/+1
2018-02-14Fixed commentspisarski1-1/+1
Change-Id: Ib226ef99eed2fda2c7112faf0f0e9339afed90fe Signed-off-by: spisarski <s.pisarski@cablelabs.com>
2018-02-14Set region_name in get_os_creds()Cédric Ollivier1-0/+1
Change-Id: Idb6912319e24edb39e18114bd3c1ef5bca750752 Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
2018-02-08Update the DNS default value in docstringsCédric Ollivier1-1/+1
It completes the previous patch [1] about that topic. [1] https://gerrit.opnfv.org/gerrit/#/c/51845/ Change-Id: Ib2ba07c0d5608bd452ed209544d4dbfc17124299 Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
2018-02-07Stop setting 8.8.8.8 as default DNS serverCédric Ollivier3-15/+8
That could easily raise side effect as it overrides default configs set by Cloud providers. Change-Id: I659d16f8f58bba3a07b8ae2ee2270aa69c200ed5 Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
2018-02-06Merge "Fixed timeout logic when attaching/detaching volumes."Steven Pisarski4-49/+149
2018-02-06Fixed timeout logic when attaching/detaching volumes.spisarski4-49/+149
The timeout logic in nova_utils.attach_volume() and detach_volume() was not correct which may have been the root cause behind the issue FUNCTEST-927. Timeout in both attach and detach is no longer optional. Also added a test to attach and detach without timeout as that path was never tested. Updated associated test documentation as well. JIRA: SNAPS-263 JIRA: FUNCTEST-927 Change-Id: Iea3aeab59c378917fbd175d673113e8d30e2e4b9 Signed-off-by: spisarski <s.pisarski@cablelabs.com>
2018-02-05Merge "Fixed OSCreds retrieval."Steven Pisarski1-0/+3
2018-02-05Added 3 new subnet values for ansible substitutionsspisarski1-0/+6
Change-Id: Idd39e4f6e3f22fcbafe3aae2512ac36cf9e2a417 Signed-off-by: spisarski <s.pisarski@cablelabs.com>
2018-02-05Fixed OSCreds retrieval.spisarski1-0/+3
The optional parameter 'project_name' was never getting a default value when not used causing an OSCredsError to be raised. Change-Id: I2a0986c62c5d9520f60ebd06db1f3a749e62361c Signed-off-by: spisarski <s.pisarski@cablelabs.com>
2018-02-01Merge "openstack: nova_utils_tests: Use API timeout argument instead of sleep()"Steven Pisarski1-6/+2
2018-01-31openstack: nova_utils_tests: Use API timeout argument instead of sleep()Markos Chandras1-6/+2
On slow environments, time.sleep(10) is not enough for attach/detach API calls to propage properly leading to failures like the following one (reduced api_check test) 2018-01-31 18:53:19,297 - functest.ci.run_tests - INFO - Running test case 'api_check'... 2018-01-31 18:54:27,434 - functest.ci.run_tests - INFO - Test result: +-------------------+------------------+------------------+----------------+ | TEST CASE | PROJECT | DURATION | RESULT | +-------------------+------------------+------------------+----------------+ | api_check | functest | 01:07 | FAIL | +-------------------+------------------+------------------+----------------+ We can workaround this by using the nova_utils.{de,at}tach_volume} 'timeout' argument to wait up to 2 minutes for the attach/detach call to complete. This doesn't affect normal exception whilst it helps get the test going on slow systems. WIth this change, the api_check (reduced test) passes as follows: 2018-01-31 18:48:35,469 - functest.ci.run_tests - INFO - Running test case 'api_check'... 2018-01-31 18:50:56,705 - functest.ci.run_tests - INFO - Test result: +-------------------+------------------+------------------+----------------+ | TEST CASE | PROJECT | DURATION | RESULT | +-------------------+------------------+------------------+----------------+ | api_check | functest | 02:20 | PASS | +-------------------+------------------+------------------+----------------+ Change-Id: I2245adb84d0e6d4e3350d17d0c2e44baf5202d51 Signed-off-by: Markos Chandras <mchandras@suse.de>
2018-01-31Remove a wrong @propertyCédric Ollivier1-1/+0
Else OSCreds.__str__() simply doesn't work. Change-Id: I0a0147862ad3015dc9cf284c61429f1534a47d61 Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
2018-01-31Merge "Changed invalid gateway IP to ensure fails with all OS configurations."Steven Pisarski1-1/+1
2018-01-31Changed invalid gateway IP to ensure fails with all OSspisarski1-1/+1
configurations. Change-Id: I35374ea5fa79974c43c2a474e43ff275b26dfc02 Signed-off-by: spisarski <s.pisarski@cablelabs.com>
2018-01-31Merge "Changed the way floating IPs are getting assigned to VMs as the ↵Steven Pisarski6-79/+29
previous means was not compatible with python-novaclient==10.0.0"
2018-01-30Changed network and keypair naming to be dynamicspisarski3-3/+8
Change-Id: I227433e92fc7f42924fa9404ff3c394c79500a1b Signed-off-by: spisarski <s.pisarski@cablelabs.com>
2018-01-29Changed the way floating IPs are getting assigned to VMs as the previousspisarski6-79/+29
means was not compatible with python-novaclient==10.0.0 JIRA: SNAPS-258 Change-Id: I56f7a8341c81e0ae5596f67d4d15ef6c26e0e680 Signed-off-by: spisarski <s.pisarski@cablelabs.com>
2018-01-29Avoid checking Keystone v3 domains when using API v2.0Georg Kunz2-5/+9
Introducing a version check to avoid checking keystone domains, which are an auth API v3 feature, when only auth API v2.0 is being used. Change-Id: I220c6637e5f65124e5dbe7d5246490c3986b7a66 Signed-off-by: Georg Kunz <georg.kunz@ericsson.com>
2018-01-23Nova 10 breaks the libspisarski1-1/+1
Change-Id: Iadb5e44836258e5d9a89090c4b0302d2689a40c5 Signed-off-by: spisarski <s.pisarski@cablelabs.com>
2018-01-23Merge "Added password support for SSH and Ansible Additional protections ↵Steven Pisarski9-53/+127
when initializing network resources Enhanced playbook runner variable support"
2018-01-23Added password support for SSH and Ansiblespisarski9-53/+127
Additional protections when initializing network resources Enhanced playbook runner variable support Change-Id: Id897c4ece1de706afbc52d0a034ca3bfb68fdce3 Signed-off-by: spisarski <s.pisarski@cablelabs.com>
2018-01-19Merge "Fix Malformed Table in Integration Tests doc"Steven Pisarski1-1/+1
2018-01-18Change exception message.spisarski1-1/+1
Change-Id: I120faaa8695db5f83e39b521aaa6c1ab68e7fcc1 Signed-off-by: spisarski <s.pisarski@cablelabs.com>
2018-01-16Added post processing task after application of an Ansible playbook.spisarski1-1/+12
The only task currently supported is reboot. Change-Id: I9089c593af30db071508a47667381238fe91e44e Signed-off-by: spisarski <s.pisarski@cablelabs.com>
2018-01-11Fixed vm instance instantiation from Heat when using nested resourcesspisarski10-28/+418
* Simplified the configuration of file resources * Added integration test JIRA: SNAPS-255 Change-Id: I1065d9352865d7a9f946a5d9947e32e7340f20bc Signed-off-by: spisarski <s.pisarski@cablelabs.com>
2018-01-10Added port security flag support to ports.spisarski4-17/+46
Change-Id: Ib66607b7656093c5f0cd0e2cb4c0aa9c3b9c661a Signed-off-by: spisarski <s.pisarski@cablelabs.com>
2018-01-09Merge "Added support for Heat resource files."Steven Pisarski2-0/+9
2018-01-03Merge "Support network configuration overridden"Steven Pisarski2-8/+16
2018-01-03Added new configuration option for OpenStackVmInstance for cloud-initspisarski3-11/+17
Added cloud_init_timeout to VM instance configuration to override the default timeout value hardcoded @ 120 seconds. This patch not only expands the default value to 300, but also allows the client to override this value as cloud-init can take quite some time depending on what is being requested. Change-Id: Ia4d0a54ce853a93748b16945f5ddd4d5887feb7d Signed-off-by: spisarski <s.pisarski@cablelabs.com>
2018-01-03Merge "Enhancements to the SNAPS orchestrator/launcher"Steven Pisarski19-1081/+912
2018-01-03Support network configuration overriddenLinda Wang2-8/+16
When the network_type is vlan, the params physical_network and segmentation_id should be accepted as arguments when creating network. JIRA: SNAPS-252 Change-Id: I3a7bec8a592ede65b97eff0911d09a8154349385 Signed-off-by: Linda Wang <wangwulin@huawei.com>
2018-01-02Configure physical_network when creating networkLinda Wang3-12/+38
When the network_type is vlan, configuring physical_network and segmentation_id should be supported by snaps. Here only testcase test_check_vm_ip_dhcp is changed, which is consumed by functest. JIRA: SNAPS-252 Change-Id: I12ae12f62f73f2ae66fe7ee12a49abf0394669cd Signed-off-by: Linda Wang <wangwulin@huawei.com>
2017-12-29Added support for Heat resource files.spisarski2-0/+9
Change-Id: I936c0ef5570b5b417e4649504a817f0ca216812f Signed-off-by: spisarski <s.pisarski@cablelabs.com>
2017-12-28Enhancements to the SNAPS orchestrator/launcherspisarski19-1081/+912
Added support for more Ansible substitution values extracted from OpenStack instances (i.e. subnet broadcast address, router external IP for gateway configuration, CIDR IP, netmask, etc.) Removed most try/except blocks to ensure processing does not continue when a problem should arise Added the ability to know when cloud-init has completed Removed nic configuration from OpenStackVmInstance as userdata/ cloud-init/cloud-config should be performing that functionality Fixed some logging messages Misc launcher bug fixes Change-Id: I73e7607ee158cce3d16f9c1c1fc7c32ef5899a1d Signed-off-by: spisarski <s.pisarski@cablelabs.com>
2017-12-21Added substitution variable support.spisarski1-1/+11
Change-Id: I57e10be315c288d75d61b014940cd02c0c0134fa Signed-off-by: spisarski <s.pisarski@cablelabs.com>
2017-12-19Return the floating ip objectManuel Buil1-0/+2
Sometimes it might be useful to have the floating ip object which was assigned Change-Id: I883ec2af448cd38dad2e0d24843c92e7ab495c2f Signed-off-by: Manuel Buil <mbuil@suse.com>
2017-12-18Refactored VmInst domain class for Ports.spisarski12-92/+103
Removed network dict and replaced with a list of Port domain objects. The old network member was directly copied into the VmInst object from neutron meaning that Neutron implementation details were leaking through. This refactor not only makes VmInst objects easier to use, it also ensures that future Neutron API changes will not break the library. JIRA: SNAPS-251 Change-Id: I3e5e811e433bf7670ecd80405186ac2ac0739def Signed-off-by: spisarski <s.pisarski@cablelabs.com>
2017-12-14Added utility function to retrieve the names of all hosts with a hypervisor.spisarski2-0/+25
Change-Id: I25b91c6eee78ebf35785c4884dded7c9ab2cc412 Signed-off-by: spisarski <s.pisarski@cablelabs.com>
2017-12-11Set the ram of flavor to 1024Linda Wang1-2/+6
The setting of hw:mem_page_size=large indicates larger page sizes for guest RAM, eg., either 2MB or 1GB on x86. When tested on compass, 1GB is required Change-Id: I22d2fcab190e465dd1c36bc4264c75c724c6f64d Signed-off-by: Linda Wang <wangwulin@huawei.com>
2017-12-04Added the ability to reboot OpenStack server instancesspisarski4-0/+104
JIRA: SNAPS-243 Change-Id: I7232ee14cac81d39d86e7391a4e006b8276ed536 Signed-off-by: spisarski <s.pisarski@cablelabs.com>
2017-12-04Merge "Added ability to add a floating IP to an existing instance"Steven Pisarski5-48/+180
2017-12-01Move CreateInstanceIPv6NetworkTests from staging to integration.spisarski1-3/+5
Change-Id: Icf381661c56ecf19311ebc95210686bc20eb124e Signed-off-by: spisarski <s.pisarski@cablelabs.com>