Merge "Add missing licences in requirements.txt"

Add missing licences in requirements.txt

Change-Id: Ic03c3aea6984458a6fc987bb278f6f25459bc2db
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Fix an invalid filter attribute when listing floating ips

The request doesn't conform with API [1] and raises exceptions vs
Neutron Hunter:

neutronclient.common.exceptions.BadRequest: [u'ip'] is invalid attribute for filtering
Neutron server returns request_ids: ['req-857b6294-eea5-46cf-8c95-0e57279ed5e6']

[1] https://developer.openstack.org/api-ref/network/v2/index.html?expanded=list-floating-ips-detail

Change-Id: I52bac8806152695834f8ece4f0213c3c63134c79
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Set MTU=1442 in ansible_utils_tests.py

The former MTU (1450) is fine for VXLAN but reducing it to 1442
allows GENEVE tunneled networks.

Else snaps_smoke fails when verifying OVN [1].

[1] https://artifacts.opnfv.org/logs/functest/lf-pod1/gambia/2018-11-05_11-43-15/functest.log

Change-Id: Icf38c30837858f59ed25f443245fee0007e9354d
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Merge "Local Documentation Builds"

Deprecating launch app and ansible support.

Change-Id: Iba28a784a4e78e1009d2038a30ccea8a1bfaa25f
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Updates to the ansible API implementation to allow for
playbooks to be applied directly against the localhost
as well as removing the return code in favor of reliance
of unchecked Errors/Exceptions.

Change-Id: Ib963aa9b4f5a75010d8f7b25db7d66a2e1950cce
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Local Documentation Builds

    This adds configuration for performing local documentation builds
    with the following simple command:

    tox -e docs

added index.rst so that docs build succsesfully

Change-Id: Iab747c90f7f4b7866270c5850b2792e194b5378a
Signed-off-by: Aric Gardner <agardner@linuxfoundation.org>

Opened up API to allow for playbook application to localhost
by making the user, password, and private key attributes optional.

Change-Id: I7e1e2b5848730ae5febde7d7243e741c87e5c7ea
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Merge "Auto Generated INFO.yaml file"

Updated ansible API calls to Ansible library version >=2.4

JIRA: SNAPS-287

Change-Id: Id422bb6e85eee249a2cf7969130b472e110a7662
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Pass flavor metadata into heat template

Change-Id: I660c7421c9c43d4603cf05489ca39f781467e3a1
Signed-off-by: Michael Polenchuk <mpolenchuk@mirantis.com>

Removed the use-keystone option for running tests.

As these tests have evolved, the -k option is no longer necessary and
actually cause some tests to fail now and I have only been testing with
that option always being set.

Change-Id: Ib866cc4f3df1c790619c64508ac9733a2b6d4102
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Support both admin role name

Depending of the installer the admin role name can be admin
or Admin.

This patch was tested:
+-------------------+------------------+------------------+----------------+
|     TEST CASE     |     PROJECT      |     DURATION     |     RESULT     |
+-------------------+------------------+------------------+----------------+
|     api_check     |     functest     |      09:60       |      PASS      |
+-------------------+------------------+------------------+----------------+

Change-Id: I99815777794cd6524213abca1f04ec84dae1e760
Signed-off-by: Valentin Boucher <valentin.boucher@kontron.com>

Merge "Added a check for cloud-init completion before final ssh client check."

Added a check for cloud-init completion before final ssh client check.

Change-Id: I040da15623a7da03d1e3805cfa113b1c40916d11
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Small fix to network/project when creating ports and
added new tests for routers that create ports to external
and shared networks created by the 'admin' user.

Change-Id: I0365e9843e0837d720ca8dd907c45afa2c512047
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Skip searching network in project if external or shared

It meets the OpenStack rules:
  - any router can be connected to an external network.
  - any VM can be connected to a shared network

It completes https://gerrit.opnfv.org/gerrit/59865 which partially
skip the lookup.

Then CreateRouterSuccessTests passes when the external network is not
owned by the user running snaps_smoke.

JIRA: SNAPS-320

Change-Id: I2c60363002c6b667f90ff3f5f984eace4fa86c99
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Merge "Cleaned up extra network created in the test test_set_one_port_two_ip_two_subnets()"

Cleaned up extra network created in the test
test_set_one_port_two_ip_two_subnets()

JIRA: SNAPS-323

Change-Id: I7610d7b3d2262b0f75c9620d2ec6907b62458158
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Added additional check when creating a port on a network for
the external flag is not True to query the network for the
given project.

JIRA: SNAPS-320

Change-Id: Ie755633520fe30664fc89e9daaea6aa2d14c2e52
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Switch to OS::Neutron::FloatingIPAssociation

OS::Nova::FloatingIPAssociation is hugely deprecated and doesn't work
with latest OpenStack Queens deliverables.

Change-Id: Icc43ba8a3466efbcc03e190b7e56f84eb69b7e51
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Added additional check for the returned project object is not None.

There were 3 other instances of this check in neutron_utils.py but
this one was missed and could potentially cause issues.

Change-Id: I599e63e14a9b08ba34f50e060c7dfa5fbdd631f7
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Adds Stack Update

Adds function to allow stack update to
occur.

Includes higher level object calls for stack
update.

Added blocking parameter to create() and update()
Rebase screwed up some changes.
Added new test to test_suite_builder.py and updated
IntegrationTests.rst accordingly

Change-Id: I4558befb3ea8ea7982faff79d1ebb54fbb3d44a7
Signed-off-by: mbeierl <mark.beierl@dell.com>

Adds Threads to Heat Calls

Adds thread pool and coverts looping calls to OpenStack to be
submitted to it, allowing them to be run in parallel.

Places a configurable maximum size (default 5) of concurrent calls
on the thread pool.

JIRA: SNAPS-316

Change-Id: Id209c5fe9b7ee7d755d3210a124b38fc3234f0d7
Signed-off-by: mbeierl <mark.beierl@dell.com>

Merge "Fixes additional legacy problems found when reviewing patch 58483"

Removing flavor metadata from test heat template.

Tests can break with this setting.

Change-Id: Idab15bc5d83f8d88a08c46c71fb179c1e3d0282b
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Fixes additional legacy problems found when reviewing patch 58483

Second revision removes an unnecessary check on self.ext_net_name.

Change-Id: I0611d0130824fb0c2fa147d3fa503b2826a61ed5
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Check dev_os_env_file exists before loading it

The testcases should work even if no specific metadata are set.

Change-Id: I3fa6aea3588f37946a847042903f313e33a728f8
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Added MTU network config and updated flavor_metadata test support

JIRA: SNAPS-315

Change-Id: Ie4cbab24ea0af41e93c475e04c9440422617466f
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Merge "Added support for OSCreds output to SNAPS-OO orchestrator app."

Added support for OSCreds output to SNAPS-OO orchestrator app.

Change-Id: Ic690ebff47d137fdd172f5dd5f263cd7f4a6ae04
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Fix to a router's internal subnet resoulution.

Ensures that any subnet added to the router is available to that project.

Change-Id: Iddfad14a093551bc0f4a05c52b20f02db2474d78
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Fixed issue when attempting to add an internal subnet to a router.

Internal subnets that are owned by the same user who is creating a
router now can be added properly. Attempted to fix an issue with the
change in https://gerrit.opnfv.org/gerrit/#/c/57853/ that was allowing
for subnets with the same name to be assigned but ended up breaking
this functionality completely.

JIRA: SNAPS-312

Change-Id: I1687f66db47520e93e401d3e9fb5f0c4f45d460f
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Miscellaneous minor stability fixes

Change-Id: Iad8a7b6358f881d1c2935cc619cfa96f1196a675
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Merge "New functionality for snaps-oo application."

New functionality for snaps-oo application.

1. Utility for saving a dict to a YAML file
2. Added new hook that will create a file with the VMs created
   with the associated OpenStack credentials

Change-Id: I124db00e4375603e9473cceab3e2846582d6cb22
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Added ability for integration tests to define additional
existing users to a project.

Change-Id: I91be434dcd7b580f3bfd69e3a97b9a9a13ee4676
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Merge "Remove conditional on external_fixed_ips to allow for case that it is not set but network_id is set in external_gateway_info"

Merge "Restricting the magnum-pythonclient version to 2.1-2.2"

Remove conditional on external_fixed_ips to allow for case that
it is not set but network_id is set in external_gateway_info

Change-Id: Ib0fdfbf49730ca68d7c1d47c18c2ac17be637380
Signed-off-by: Stuart Mackie <wsmackie@juniper.net>

Extra changes on Flavor Config

JIRA: SNAPS-254

Change-Id: I4f18c0d11fe2e8456c0ebce0681f22e3e1daf6bf
Signed-off-by: Panagiotis Karalis <pkaralis@intracom-telecom.com>

Merge "Removed project query restriction when looking up network for port creation."

Restricting the magnum-pythonclient version to 2.1-2.2

Using version >2.2 results in a runtime exception.

Change-Id: I1c5d5c650004981f13a350d04c3ab359fc98799b
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Merge "Patch to make flavor configuration for tests more flexible."

Removed project query restriction when looking up network for port creation.

When creating a router to an external network, the port configuration object
was performing a lookup using the port's user and project ID; however, as
external networks created in other projects and are visible amongst all
projects can also be added to a port. This patch removes the project restriction
from the network query. I do not believe this fix should ultimately break other
functionality as networks that cannot be looked up by a certain user would
still return None raising a valid exception.

JIRA: APEX-598

Change-Id: I6cd32373fd4f42d8ad9b63a89c2131f95b113ce8
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Conform ProjectConfig args with description

Change-Id: Ib865fdc6d48c377c5a28092015b807ef79a15ce3
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Patch to make flavor configuration for tests more flexible.

Also fixed potential issues with downloading images from https locations

JIRA: SNAPS-254

Change-Id: Ib81569df499bbf0652c2025fc7de37af92525f7f
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Fix NoneType in create_router.py

It lets Neutron handle str or unicode instead of returning None.
It reverts the first proposal [1] which raises side effects [2].

[1] https://gerrit.opnfv.org/gerrit/#/c/56171/
[2] https://build.opnfv.org/ci/view/functest/job/functest-compass-baremetal-daily-master/126/console

Change-Id: I1b45c7d6118321dd67343f31253f154abae73655
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Merge "Fix NoneType error when creating router"

Addressed issue with test when replicated networks are configured on the pod.

According to the defect, "neutron OVS agent uses a replicated network for DVR."

JIRA: SNAPS-305

Change-Id: I3ceb94582d74a297f5ff41520ffe4e9f5e638266
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Fix NoneType error when creating router

Change-Id: Icef5f8a769b5c33da68b7da33ac8ba87132830e2
Signed-off-by: Linda Wang <wangwulin@huawei.com>

Fixed bug with regards to subnet lookups.
Neutron returns all subnets regardless of visibility which cause problems
within routers if there is another subnet with the same name attached to
a different network.

JIRA: SNAPS-304

In addition, this patch contains two other minor fixes.
  launch_utils.py - raise an exception when the creator is not properly instantiated
  network.py - allow fixed IPs to be none.

Change-Id: Ib343074d925be4592a713727a03d5b531890eada
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Adding assertion message to test validation.

Change-Id: I609d09ed8b4f75baeb1ad5454c9ddde9e996a3fe
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Add flavor metadata for volume related tests

These tests are very sensitive to the state of instance which might be
active in terms of nova, but in fact still trying to get an ip address in
case of dpdk scenarios.

Change-Id: I739044c2edd360535ce1d3531549263163cfea19
Signed-off-by: Michael Polenchuk <mpolenchuk@mirantis.com>

Add listing security groups functionality

JIRA: SNAPS-302

Be able to list the available security groups

Change-Id: I82e2daeb388f4eb3bc8cbc5fd02006b30b959c2f
Signed-off-by: Manuel Buil <mbuil@suse.com>

Initial release-notes.

JIRA: SNAPS-288

Change-Id: I61e96eeb2128fc4e4abc88595504cad5f70f4378
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Merge "Fixed bug when use_keystone parameter is False."

Merge "Fixing security group in CreateInstanceTwoNetTests"

Fixed bug when use_keystone parameter is False.

When false, the self.keystone member is None and now it is being
used in many more places with some of the other changes during
the Fraser release

JIRA: SNAPS-299

Change-Id: I752afb127287036a16f3c9b00e6feae9cab80143
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Updated from global requirements

Change-Id: Ib6c90b7a0fc5f23a9882c3ca4cc4d3031cebe5b9
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Fixing security group in CreateInstanceTwoNetTests

The test CreateInstanceTwoNetTests does define and create a security
group and a corresponding rule but never associates them with the
created instances. Moreover, some commercial systems require explicit
egress rules in addition to ingress rules in order to allow traffic to
pass. Anm egress rule has been added.

JIRA: SNAPS-300

Change-Id: Ie0da75781f8b4e97af11e4e7588c7a53acad59ac
Signed-off-by: Georg Kunz <georg.kunz@ericsson.com>

Enable get_project_by_id when only tenant_id is provided

This is to improve:
https://gerrit.opnfv.org/gerrit/#/c/52675/

Change-Id: I7228e5eadcc121fe94a869c6cbbeac412d972868
Signed-off-by: Linda Wang <wangwulin@huawei.com>

Remove test_create_volume_bad_zone

JIRA: FUEL-356

Fuel/MCP has set allow_availability_zone_fallback option to True.
If the requested Cinder availability zone is unavailable, fall back
to the value of default_availability_zone, then storage_availability_zone,
instead of failing.

Change-Id: I04f45f8435dbdf1113ac6b2c50bf913c1f29cd20
Signed-off-by: Linda Wang <wangwulin@huawei.com>

Added ability to set several quota settings upon project creation.

JIRA: SNAPS-275

Change-Id: Ibab81e835dddcb666fb3c829857b1c4676b57ed1
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Added test to ensure a port to the external network can be added to
a router by an 'admin' user.

JIRA: SNAPS-283

Change-Id: I78a48862de5bd48dca7dc38d197c0df315003638
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Added tests to ensure multiple IP addresses can be assigned to a VM port.

JIRA: SNAPS-284

Change-Id: I43528434523a137a78d1f1f8d22496341f78492b
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Added test to ensure VM instances spawned by an admin user can
have an external network used on a port.

JIRA: SNAPS-282

Change-Id: I3ee2fef65d1751880e576dc8f6a797260d1028ee
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Reverse engineer OpenStackHeatStack

Added support for obtaining an initialized
OpenStackHeatStack object from an existing Stack object

JIRA: SNAPS-278

Change-Id: I68cb0c85179868e4e3899b5516316dda3321e9f6
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Merge "Fixed a few Py 2.7 -> 3.x issues."

Fixed a few Py 2.7 -> 3.x issues.

JIRA: SNAPS-286

Change-Id: Ib4976529168655df0ed906276b503b70a8459f21
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Added support for running tests in parallel.

JIRA: SNAPS-215

Change-Id: I94923a9f184b0d370159b499919e8fc20a2fef90
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Closing keystone sessions after done with them.

By not closing all of the keystone sessions being created when
running all of the tests, this may be the root cause to the IOError
occasionally being observed:

IOError: [Errno 24] Too many open files

JIRA: SNAPS-285

Change-Id: I7fc7ab0c6cdd02f1ae32bb3ae4f121cb465d5693
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Added members to VmInst that will contain the availability_zone
and compute_host names while deprecating the method get_vm_info().

JIRA: SNAPS-277

Change-Id: Idc8578b3f2cf2be8ef90f52dd025dbea729b222b
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Ensure the project for volumes are handled properly.

Although this patch has added the project_name member to
VolumeConfig, it does not work properly due to a cinder bug
https://bugs.launchpad.net/cinder/+bug/1641982. However, the
code will now allow for volumes with the same name to be
created on the same pod just as long as they do not exist
within the same project/tenant.

JIRA: SNAPS-270

Change-Id: Ia6f98e9b66793a69c0205d2d8e38bb3d03f3333e
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Updated stack tests to only use an admin user when necessary.

Changed super test class OSIntegrationTestCase to allow subclasses
to set the user roles when required. For all heat stack test,
the user role will always be 'heat_stack_owner'. 'admin' will be
used for tests requiring this role (i.e. heat template creates
flavors)

JIRA: SNAPS-268

Change-Id: I0ec01cfea6297547a6a158eec8e5c69af6d6c7d7
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Ensure project for routers are handled properly.

Routers should be able to be created by admin users to other
projects. Routers without admin users should only be able to
create ones to their default project.

JIRA: SNAPS-266

Change-Id: I5ecfbcc2ed94bbab211751da9521f3ec1d4ff66d
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Fixed subnet gateway support to allow for disabling the gateway.

JIRA: SNAPS-276

Change-Id: Ia676b91ee881097ca7502fb4b9f5c64961de2214
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Changed pattern on how objects lookup themselves by name and project.

The pattern being replaced has unwittingly added the requirement that
all creator credentials must be of type 'admin' as when looking up
the associated project ID required a call to keystone.projects.list().

As the SNAPS integration tests were always creating users with an 'admin'
role, this issue was not caught. As part of this patch, integration test
users will no longer be admin.

JIRA: SNAPS-274

Change-Id: I02957f69e31a9d4dfa63362d371f061687e59fbf
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

openstack: nova_utils_tests: Wait for volume detachment

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>

openstack: nova_utils_tests: Increase timeout for API calls

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>

Ensure project IDs are handled correctly for Network/Subnets

JIRA: SNAPS-265

Change-Id: If58f5817e52e21be377d2b06a5f7c3a413454350
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Added functionality for OpenStackSecurityGroup#initialize() not to accept
security groups with the same name from a different project

JIRA: SNAPS-264

Change-Id: I7c905e5588d5e503b47ea0e9f5997be89e841aec
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Merge "Compare strings instead of objects in SecurityGroupConfig"

Merge "Set region_name in get_os_creds()"

Compare strings instead of objects in SecurityGroupConfig

It could have worked thanks to interned strings.

Change-Id: If56cc44fe35ca06bc9033af620ddd3c0586877fe
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Merge "Fixed comment"

Fixed comment

Change-Id: Ib226ef99eed2fda2c7112faf0f0e9339afed90fe
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Set region_name in get_os_creds()

Change-Id: Idb6912319e24edb39e18114bd3c1ef5bca750752
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>

Update the DNS default value in docstrings

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>

Stop setting 8.8.8.8 as default DNS server

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>

Merge "Fixed timeout logic when attaching/detaching volumes."

Fixed timeout logic when attaching/detaching volumes.

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>

Merge "Fixed OSCreds retrieval."

Auto Generated INFO.yaml file

INFO.yaml meant to be machine readable.
This file was auto generated from an ldap lookup
and your projects INFO file.

Meeting times are not currently filled out.
please ammend this patchset to complete

Going forward comitter promotions should be done
against this file.

PTLs:
Must be merged for you to gain delete access on
testresults.opnfv.org

If project_lead: is missing (my script could not parse)
please amend with same from commiters section eg:

project_lead: &opnfv_$project_ptl
    name: 'Lead Name'
    email: 'lead email'
    id: 'lead lfid'
    company: 'lead company'

Change-Id: I576d23ec10ef47973e0211765b88d397b9c2bf9a
Signed-off-by: agardner <agardner@linuxfoundation.org>

Added 3 new subnet values for ansible substitutions

Change-Id: Idd39e4f6e3f22fcbafe3aae2512ac36cf9e2a417
Signed-off-by: spisarski <s.pisarski@cablelabs.com>

Fixed OSCreds retrieval.

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>

Merge "openstack: nova_utils_tests: Use API timeout argument instead of sleep()"

openstack: nova_utils_tests: Use API timeout argument instead of sleep()

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>

Remove a wrong @property

Else OSCreds.__str__() simply doesn't work.

Change-Id: I0a0147862ad3015dc9cf284c61429f1534a47d61
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>