snaps.git
5 years agoSwitch to OS::Neutron::FloatingIPAssociation 01/59801/2
Cédric Ollivier [Fri, 13 Jul 2018 15:49:35 +0000 (17:49 +0200)]
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>
5 years agoAdded additional check for the returned project object is not None. 09/59709/1
spisarski [Wed, 11 Jul 2018 18:09:51 +0000 (14:09 -0400)]
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>
5 years agoAdds Stack Update 97/57297/16
mbeierl [Thu, 10 May 2018 20:16:25 +0000 (16:16 -0400)]
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>
5 years agoAdds Threads to Heat Calls 99/58599/5
mbeierl [Fri, 15 Jun 2018 20:01:39 +0000 (16:01 -0400)]
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>
5 years agoMerge "Fixes additional legacy problems found when reviewing patch 58483"
Steven Pisarski [Fri, 15 Jun 2018 14:51:28 +0000 (14:51 +0000)]
Merge "Fixes additional legacy problems found when reviewing patch 58483"

5 years agoRemoving flavor metadata from test heat template. 61/58561/1
spisarski [Thu, 14 Jun 2018 21:42:11 +0000 (15:42 -0600)]
Removing flavor metadata from test heat template.

Tests can break with this setting.

Change-Id: Idab15bc5d83f8d88a08c46c71fb179c1e3d0282b
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
5 years agoFixes additional legacy problems found when reviewing patch 58483 03/58503/2
spisarski [Wed, 13 Jun 2018 15:28:49 +0000 (09:28 -0600)]
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>
5 years agoCheck dev_os_env_file exists before loading it 83/58483/1
Cédric Ollivier [Wed, 13 Jun 2018 12:19:24 +0000 (14:19 +0200)]
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>
5 years agoAdded MTU network config and updated flavor_metadata test support 39/58439/2
spisarski [Mon, 11 Jun 2018 19:24:26 +0000 (13:24 -0600)]
Added MTU network config and updated flavor_metadata test support

JIRA: SNAPS-315

Change-Id: Ie4cbab24ea0af41e93c475e04c9440422617466f
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
5 years agoMerge "Added support for OSCreds output to SNAPS-OO orchestrator app."
Steven Pisarski [Fri, 8 Jun 2018 17:24:21 +0000 (17:24 +0000)]
Merge "Added support for OSCreds output to SNAPS-OO orchestrator app."

5 years agoAdded support for OSCreds output to SNAPS-OO orchestrator app. 11/58211/1
spisarski [Mon, 4 Jun 2018 15:29:39 +0000 (09:29 -0600)]
Added support for OSCreds output to SNAPS-OO orchestrator app.

Change-Id: Ic690ebff47d137fdd172f5dd5f263cd7f4a6ae04
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
5 years agoFix to a router's internal subnet resoulution. 07/58207/1
spisarski [Mon, 4 Jun 2018 15:23:53 +0000 (09:23 -0600)]
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>
5 years agoFixed issue when attempting to add an internal subnet to a router. 41/58041/1
spisarski [Tue, 29 May 2018 16:53:55 +0000 (10:53 -0600)]
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>
5 years agoMiscellaneous minor stability fixes 53/57853/1
spisarski [Thu, 24 May 2018 19:36:56 +0000 (13:36 -0600)]
Miscellaneous minor stability fixes

Change-Id: Iad8a7b6358f881d1c2935cc619cfa96f1196a675
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
5 years agoMerge "New functionality for snaps-oo application."
Steven Pisarski [Thu, 24 May 2018 19:34:57 +0000 (19:34 +0000)]
Merge "New functionality for snaps-oo application."

5 years agoNew functionality for snaps-oo application. 51/57851/1
spisarski [Thu, 24 May 2018 18:35:33 +0000 (12:35 -0600)]
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>
5 years agoAdded ability for integration tests to define additional 37/57837/1
spisarski [Thu, 24 May 2018 17:08:06 +0000 (11:08 -0600)]
Added ability for integration tests to define additional
existing users to a project.

Change-Id: I91be434dcd7b580f3bfd69e3a97b9a9a13ee4676
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
5 years agoMerge "Remove conditional on external_fixed_ips to allow for case that it is not...
Steven Pisarski [Thu, 24 May 2018 16:31:38 +0000 (16:31 +0000)]
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"

5 years agoMerge "Restricting the magnum-pythonclient version to 2.1-2.2"
Steven Pisarski [Thu, 24 May 2018 16:05:04 +0000 (16:05 +0000)]
Merge "Restricting the magnum-pythonclient version to 2.1-2.2"

5 years agoRemove conditional on external_fixed_ips to allow for case that 85/57785/1
Stuart Mackie [Wed, 23 May 2018 20:11:12 +0000 (16:11 -0400)]
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>
5 years agoExtra changes on Flavor Config 75/57475/4
Panagiotis Karalis [Wed, 16 May 2018 14:33:40 +0000 (17:33 +0300)]
Extra changes on Flavor Config

JIRA: SNAPS-254

Change-Id: I4f18c0d11fe2e8456c0ebce0681f22e3e1daf6bf
Signed-off-by: Panagiotis Karalis <pkaralis@intracom-telecom.com>
6 years agoMerge "Removed project query restriction when looking up network for port creation."
Steven Pisarski [Wed, 16 May 2018 17:52:59 +0000 (17:52 +0000)]
Merge "Removed project query restriction when looking up network for port creation."

6 years agoRestricting the magnum-pythonclient version to 2.1-2.2 81/57481/2
spisarski [Wed, 16 May 2018 15:00:27 +0000 (09:00 -0600)]
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>
6 years agoMerge "Patch to make flavor configuration for tests more flexible."
Steven Pisarski [Wed, 16 May 2018 15:46:18 +0000 (15:46 +0000)]
Merge "Patch to make flavor configuration for tests more flexible."

6 years agoRemoved project query restriction when looking up network for port creation. 85/57485/1
spisarski [Wed, 16 May 2018 15:06:57 +0000 (09:06 -0600)]
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>
6 years agoConform ProjectConfig args with description 57/57457/1
Cédric Ollivier [Wed, 16 May 2018 09:09:41 +0000 (11:09 +0200)]
Conform ProjectConfig args with description

Change-Id: Ib865fdc6d48c377c5a28092015b807ef79a15ce3
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
6 years agoPatch to make flavor configuration for tests more flexible. 37/57237/3
spisarski [Wed, 9 May 2018 16:48:09 +0000 (10:48 -0600)]
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>
6 years agoFix NoneType in create_router.py 89/56889/1
Cédric Ollivier [Sun, 29 Apr 2018 16:38:48 +0000 (18:38 +0200)]
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>
6 years agoMerge "Fix NoneType error when creating router"
Steven Pisarski [Mon, 23 Apr 2018 18:26:23 +0000 (18:26 +0000)]
Merge "Fix NoneType error when creating router"

6 years agoAddressed issue with test when replicated networks are configured on the pod. 77/56277/1
spisarski [Mon, 23 Apr 2018 14:06:26 +0000 (08:06 -0600)]
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>
6 years agoFix NoneType error when creating router 71/56171/1
Linda Wang [Sat, 21 Apr 2018 02:31:09 +0000 (02:31 +0000)]
Fix NoneType error when creating router

Change-Id: Icef5f8a769b5c33da68b7da33ac8ba87132830e2
Signed-off-by: Linda Wang <wangwulin@huawei.com>
6 years agoFixed bug with regards to subnet lookups. 17/55917/6
spisarski [Tue, 17 Apr 2018 14:24:18 +0000 (08:24 -0600)]
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>
6 years agoAdding assertion message to test validation. 85/56085/1
spisarski [Thu, 19 Apr 2018 21:11:23 +0000 (15:11 -0600)]
Adding assertion message to test validation.

Change-Id: I609d09ed8b4f75baeb1ad5454c9ddde9e996a3fe
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
6 years agoAdd flavor metadata for volume related tests 07/55907/2
Michael Polenchuk [Tue, 17 Apr 2018 13:15:36 +0000 (17:15 +0400)]
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>
6 years agoAdd listing security groups functionality 97/55797/6
Manuel Buil [Mon, 16 Apr 2018 08:24:34 +0000 (10:24 +0200)]
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>
6 years agoInitial release-notes. 17/55617/2
spisarski [Thu, 12 Apr 2018 19:20:52 +0000 (13:20 -0600)]
Initial release-notes.

JIRA: SNAPS-288

Change-Id: I61e96eeb2128fc4e4abc88595504cad5f70f4378
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
6 years agoMerge "Fixed bug when use_keystone parameter is False."
Steven Pisarski [Thu, 12 Apr 2018 17:00:01 +0000 (17:00 +0000)]
Merge "Fixed bug when use_keystone parameter is False."

6 years agoMerge "Fixing security group in CreateInstanceTwoNetTests"
Steven Pisarski [Thu, 12 Apr 2018 16:59:54 +0000 (16:59 +0000)]
Merge "Fixing security group in CreateInstanceTwoNetTests"

6 years agoFixed bug when use_keystone parameter is False. 05/55605/1
spisarski [Thu, 12 Apr 2018 16:32:44 +0000 (10:32 -0600)]
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>
6 years agoUpdated from global requirements 03/55503/3
Cédric Ollivier [Thu, 12 Apr 2018 07:19:53 +0000 (09:19 +0200)]
Updated from global requirements

Change-Id: Ib6c90b7a0fc5f23a9882c3ca4cc4d3031cebe5b9
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
6 years agoFixing security group in CreateInstanceTwoNetTests 79/55479/1
Georg Kunz [Wed, 11 Apr 2018 21:19:00 +0000 (23:19 +0200)]
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>
6 years agoEnable get_project_by_id when only tenant_id is provided 59/54859/2
Linda Wang [Wed, 4 Apr 2018 03:19:52 +0000 (03:19 +0000)]
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>
6 years agoRemove test_create_volume_bad_zone 07/54307/1
Linda Wang [Thu, 22 Mar 2018 11:51:49 +0000 (11:51 +0000)]
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>
6 years agoAdded ability to set several quota settings upon project creation. 33/54133/1
spisarski [Mon, 19 Mar 2018 17:54:34 +0000 (11:54 -0600)]
Added ability to set several quota settings upon project creation.

JIRA: SNAPS-275

Change-Id: Ibab81e835dddcb666fb3c829857b1c4676b57ed1
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
6 years agoAdded test to ensure a port to the external network can be added to 85/53685/2
spisarski [Tue, 13 Mar 2018 14:58:38 +0000 (08:58 -0600)]
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>
6 years agoAdded tests to ensure multiple IP addresses can be assigned to a VM port. 27/53427/3
spisarski [Fri, 9 Mar 2018 17:15:15 +0000 (10:15 -0700)]
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>
6 years agoAdded test to ensure VM instances spawned by an admin user can 27/53327/2
spisarski [Thu, 8 Mar 2018 17:39:37 +0000 (10:39 -0700)]
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>
6 years agoReverse engineer OpenStackHeatStack 69/53269/4
spisarski [Wed, 7 Mar 2018 23:58:42 +0000 (16:58 -0700)]
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>
6 years agoMerge "Fixed a few Py 2.7 -> 3.x issues."
Steven Pisarski [Tue, 13 Mar 2018 13:25:56 +0000 (13:25 +0000)]
Merge "Fixed a few Py 2.7 -> 3.x issues."

6 years agoFixed a few Py 2.7 -> 3.x issues. 03/53603/1
spisarski [Mon, 12 Mar 2018 22:29:46 +0000 (16:29 -0600)]
Fixed a few Py 2.7 -> 3.x issues.

JIRA: SNAPS-286

Change-Id: Ib4976529168655df0ed906276b503b70a8459f21
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
6 years agoAdded support for running tests in parallel. 85/53585/1
spisarski [Mon, 12 Mar 2018 16:22:01 +0000 (10:22 -0600)]
Added support for running tests in parallel.

JIRA: SNAPS-215

Change-Id: I94923a9f184b0d370159b499919e8fc20a2fef90
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
6 years agoClosing keystone sessions after done with them. 35/53435/4
spisarski [Fri, 9 Mar 2018 21:42:34 +0000 (14:42 -0700)]
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>
6 years agoAdded members to VmInst that will contain the availability_zone 63/53263/4
spisarski [Wed, 7 Mar 2018 21:52:44 +0000 (14:52 -0700)]
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>
6 years agoEnsure the project for volumes are handled properly. 55/53255/4
spisarski [Wed, 7 Mar 2018 20:01:37 +0000 (13:01 -0700)]
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>
6 years agoUpdated stack tests to only use an admin user when necessary. 87/53187/2
spisarski [Tue, 6 Mar 2018 21:52:31 +0000 (14:52 -0700)]
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>
6 years agoEnsure project for routers are handled properly. 75/52675/6
spisarski [Mon, 26 Feb 2018 16:51:45 +0000 (09:51 -0700)]
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>
6 years agoFixed subnet gateway support to allow for disabling the gateway. 77/52777/3
spisarski [Tue, 27 Feb 2018 20:27:47 +0000 (13:27 -0700)]
Fixed subnet gateway support to allow for disabling the gateway.

JIRA: SNAPS-276

Change-Id: Ia676b91ee881097ca7502fb4b9f5c64961de2214
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
6 years agoChanged pattern on how objects lookup themselves by name and project. 73/52473/1
spisarski [Thu, 22 Feb 2018 18:42:52 +0000 (11:42 -0700)]
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>
6 years agoopenstack: nova_utils_tests: Wait for volume detachment 71/52371/1
Markos Chandras [Tue, 20 Feb 2018 17:19:13 +0000 (17:19 +0000)]
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>
6 years agoopenstack: nova_utils_tests: Increase timeout for API calls 59/52359/1
Markos Chandras [Tue, 20 Feb 2018 15:13:29 +0000 (15:13 +0000)]
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>
6 years agoEnsure project IDs are handled correctly for Network/Subnets 45/52245/1
spisarski [Fri, 16 Feb 2018 15:35:07 +0000 (08:35 -0700)]
Ensure project IDs are handled correctly for Network/Subnets

JIRA: SNAPS-265

Change-Id: If58f5817e52e21be377d2b06a5f7c3a413454350
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
6 years agoAdded functionality for OpenStackSecurityGroup#initialize() not to accept 57/52157/3
spisarski [Wed, 14 Feb 2018 16:38:54 +0000 (09:38 -0700)]
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>
6 years agoMerge "Compare strings instead of objects in SecurityGroupConfig"
Steven Pisarski [Thu, 15 Feb 2018 14:24:14 +0000 (14:24 +0000)]
Merge "Compare strings instead of objects in SecurityGroupConfig"

6 years agoMerge "Set region_name in get_os_creds()"
Steven Pisarski [Thu, 15 Feb 2018 14:21:36 +0000 (14:21 +0000)]
Merge "Set region_name in get_os_creds()"

6 years agoCompare strings instead of objects in SecurityGroupConfig 73/52173/1
Cédric Ollivier [Wed, 14 Feb 2018 20:59:04 +0000 (21:59 +0100)]
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>
6 years agoMerge "Fixed comment"
Steven Pisarski [Wed, 14 Feb 2018 16:04:53 +0000 (16:04 +0000)]
Merge "Fixed comment"

6 years agoFixed comment 53/52153/1
spisarski [Wed, 14 Feb 2018 16:04:16 +0000 (09:04 -0700)]
Fixed comment

Change-Id: Ib226ef99eed2fda2c7112faf0f0e9339afed90fe
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
6 years agoSet region_name in get_os_creds() 43/52143/1
Cédric Ollivier [Wed, 14 Feb 2018 09:06:31 +0000 (10:06 +0100)]
Set region_name in get_os_creds()

Change-Id: Idb6912319e24edb39e18114bd3c1ef5bca750752
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
6 years agoUpdate the DNS default value in docstrings 33/51933/1
Cédric Ollivier [Thu, 8 Feb 2018 21:13:47 +0000 (22:13 +0100)]
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>
6 years agoStop setting 8.8.8.8 as default DNS server 45/51845/2
Cédric Ollivier [Wed, 7 Feb 2018 08:50:42 +0000 (09:50 +0100)]
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>
6 years agoMerge "Fixed timeout logic when attaching/detaching volumes."
Steven Pisarski [Tue, 6 Feb 2018 22:27:01 +0000 (22:27 +0000)]
Merge "Fixed timeout logic when attaching/detaching volumes."

6 years agoFixed timeout logic when attaching/detaching volumes. 95/51695/12
spisarski [Mon, 5 Feb 2018 18:52:55 +0000 (11:52 -0700)]
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>
6 years agoMerge "Fixed OSCreds retrieval."
Steven Pisarski [Mon, 5 Feb 2018 20:12:24 +0000 (20:12 +0000)]
Merge "Fixed OSCreds retrieval."

6 years agoAdded 3 new subnet values for ansible substitutions 99/51699/1
spisarski [Mon, 5 Feb 2018 19:10:43 +0000 (12:10 -0700)]
Added 3 new subnet values for ansible substitutions

Change-Id: Idd39e4f6e3f22fcbafe3aae2512ac36cf9e2a417
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
6 years agoFixed OSCreds retrieval. 97/51697/1
spisarski [Mon, 5 Feb 2018 19:07:40 +0000 (12:07 -0700)]
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>
6 years agoMerge "openstack: nova_utils_tests: Use API timeout argument instead of sleep()"
Steven Pisarski [Thu, 1 Feb 2018 14:33:22 +0000 (14:33 +0000)]
Merge "openstack: nova_utils_tests: Use API timeout argument instead of sleep()"

6 years agoopenstack: nova_utils_tests: Use API timeout argument instead of sleep() 15/51415/1
Markos Chandras [Wed, 31 Jan 2018 19:03:43 +0000 (19:03 +0000)]
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>
6 years agoRemove a wrong @property 11/51411/1
Cédric Ollivier [Wed, 31 Jan 2018 18:57:53 +0000 (19:57 +0100)]
Remove a wrong @property

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

Change-Id: I0a0147862ad3015dc9cf284c61429f1534a47d61
Signed-off-by: Cédric Ollivier <cedric.ollivier@orange.com>
6 years agoMerge "Changed invalid gateway IP to ensure fails with all OS configurations."
Steven Pisarski [Wed, 31 Jan 2018 17:02:22 +0000 (17:02 +0000)]
Merge "Changed invalid gateway IP to ensure fails with all OS configurations."

6 years agoChanged invalid gateway IP to ensure fails with all OS 65/51365/2
spisarski [Tue, 30 Jan 2018 19:30:56 +0000 (12:30 -0700)]
Changed invalid gateway IP to ensure fails with all OS
configurations.

Change-Id: I35374ea5fa79974c43c2a474e43ff275b26dfc02
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
6 years agoMerge "Changed the way floating IPs are getting assigned to VMs as the previous means...
Steven Pisarski [Wed, 31 Jan 2018 17:00:27 +0000 (17:00 +0000)]
Merge "Changed the way floating IPs are getting assigned to VMs as the previous means was not compatible with python-novaclient==10.0.0"

6 years agoChanged network and keypair naming to be dynamic 69/51369/1
spisarski [Tue, 30 Jan 2018 19:56:20 +0000 (12:56 -0700)]
Changed network and keypair naming to be dynamic

Change-Id: I227433e92fc7f42924fa9404ff3c394c79500a1b
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
6 years agoChanged the way floating IPs are getting assigned to VMs as the previous 27/51327/1
spisarski [Mon, 29 Jan 2018 21:05:18 +0000 (14:05 -0700)]
Changed the way floating IPs are getting assigned to VMs as the previous
means was not compatible with python-novaclient==10.0.0

JIRA: SNAPS-258

Change-Id: I56f7a8341c81e0ae5596f67d4d15ef6c26e0e680
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
6 years agoAvoid checking Keystone v3 domains when using API v2.0 17/51317/3
Georg Kunz [Mon, 29 Jan 2018 15:24:48 +0000 (16:24 +0100)]
Avoid checking Keystone v3 domains when using API v2.0

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>
6 years agoNova 10 breaks the lib 01/51001/1
spisarski [Tue, 23 Jan 2018 16:26:17 +0000 (09:26 -0700)]
Nova 10 breaks the lib

Change-Id: Iadb5e44836258e5d9a89090c4b0302d2689a40c5
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
6 years agoMerge "Added password support for SSH and Ansible Additional protections when initial...
Steven Pisarski [Tue, 23 Jan 2018 16:18:51 +0000 (16:18 +0000)]
Merge "Added password support for SSH and Ansible Additional protections when initializing network resources Enhanced playbook runner variable support"

6 years agoAdded password support for SSH and Ansible 71/50971/2
spisarski [Tue, 23 Jan 2018 02:27:31 +0000 (19:27 -0700)]
Added password support for SSH and Ansible
Additional protections when initializing network resources
Enhanced playbook runner variable support

Change-Id: Id897c4ece1de706afbc52d0a034ca3bfb68fdce3
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
6 years agoMerge "Fix Malformed Table in Integration Tests doc"
Steven Pisarski [Fri, 19 Jan 2018 14:38:30 +0000 (14:38 +0000)]
Merge "Fix Malformed Table in Integration Tests doc"

6 years agoChange exception message. 77/50777/1
spisarski [Thu, 18 Jan 2018 15:09:59 +0000 (08:09 -0700)]
Change exception message.

Change-Id: I120faaa8695db5f83e39b521aaa6c1ab68e7fcc1
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
6 years agoAdded post processing task after application of an Ansible playbook. 01/50701/1
spisarski [Tue, 16 Jan 2018 14:46:29 +0000 (07:46 -0700)]
Added post processing task after application of an Ansible playbook.

The only task currently supported is reboot.

Change-Id: I9089c593af30db071508a47667381238fe91e44e
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
6 years agoFixed vm instance instantiation from Heat when using nested resources 43/50443/3
spisarski [Thu, 11 Jan 2018 17:24:32 +0000 (10:24 -0700)]
Fixed vm instance instantiation from Heat when using nested resources

* Simplified the configuration of file resources
* Added integration test

JIRA: SNAPS-255

Change-Id: I1065d9352865d7a9f946a5d9947e32e7340f20bc
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
6 years agoAdded port security flag support to ports. 79/50379/1
spisarski [Wed, 10 Jan 2018 19:13:56 +0000 (12:13 -0700)]
Added port security flag support to ports.

Change-Id: Ib66607b7656093c5f0cd0e2cb4c0aa9c3b9c661a
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
6 years agoMerge "Added support for Heat resource files."
Steven Pisarski [Tue, 9 Jan 2018 18:30:12 +0000 (18:30 +0000)]
Merge "Added support for Heat resource files."

6 years agoMerge "Support network configuration overridden"
Steven Pisarski [Wed, 3 Jan 2018 17:05:52 +0000 (17:05 +0000)]
Merge "Support network configuration overridden"

6 years agoAdded new configuration option for OpenStackVmInstance for cloud-init 51/49951/1
spisarski [Wed, 3 Jan 2018 17:00:43 +0000 (10:00 -0700)]
Added new configuration option for OpenStackVmInstance for cloud-init

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>
6 years agoMerge "Enhancements to the SNAPS orchestrator/launcher"
Steven Pisarski [Wed, 3 Jan 2018 15:02:33 +0000 (15:02 +0000)]
Merge "Enhancements to the SNAPS orchestrator/launcher"

6 years agoSupport network configuration overridden 99/49899/1
Linda Wang [Wed, 3 Jan 2018 03:13:02 +0000 (03:13 +0000)]
Support network configuration overridden

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>
6 years agoConfigure physical_network when creating network 43/49643/6
Linda Wang [Tue, 26 Dec 2017 02:49:05 +0000 (02:49 +0000)]
Configure physical_network when creating network

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>
6 years agoAdded support for Heat resource files. 61/49761/1
spisarski [Fri, 29 Dec 2017 16:21:05 +0000 (09:21 -0700)]
Added support for Heat resource files.

Change-Id: I936c0ef5570b5b417e4649504a817f0ca216812f
Signed-off-by: spisarski <s.pisarski@cablelabs.com>
6 years agoEnhancements to the SNAPS orchestrator/launcher 45/49745/1
spisarski [Thu, 28 Dec 2017 23:09:48 +0000 (16:09 -0700)]
Enhancements to the SNAPS orchestrator/launcher

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>