Correct list of parameters

CreateInstancePubPrivNetTests was moved to the openstack_staging_tests
suite but with the wrong list of parameters. This fix corrects that.

Change-Id: Id32cc19f2e6c4dfa5e0c07b83fd4911de208658c
Signed-off-by: Cristina Pauna <cristina.pauna@enea.com>

Move CreateInstancePubPrivNetTests test to staging

CreateInstancePubPrivNetTests is failing on Fuel and it seems that the
test is not behaving as expected on all pods. SNAPS-191 was created to
refactor the code so until that is fixed we should not run this test
in functest.

JIRA: SNAPS-184

Change-Id: I2286b9fab72e705092a521cd3611a63a8e12f855
Signed-off-by: Cristina Pauna <cristina.pauna@enea.com>

Increse ssh timeout in Ansible

The SNAPS smoke tests pass on Apex  when the timeout
is a bit higher than 10 (default)

JIRA: SNAPS-186

Change-Id: I2f42f0fc9a78bfad4ae3940813e87ba7572ad2f5
Signed-off-by: Jose Lausuch <jalausuch@suse.com>
(cherry picked from commit 440acd77fbda4cf944cc915bef4183935527f651)

Fix to a UserSettings pydoc explaination of an attribute.

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

Merge "Grant the new user to os_creds.project_name"

Grant the new user to os_creds.project_name

For the test test_create_admin_user, the new user is
granted to the project "admin", so the role retrieval
is None with os_creds.project_name, which is not "admin".

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

Get the correct endpoints URL based on OS_REGION_NAME env property

It also ignores *.pyc.

Change-Id: I94a5e297108b839fd6d2811faa34eed3023c0e3a
Signed-off-by: Laurent Strullu <laurent.strullu@orange.com>

Specify segmentation_id when creating network

When the network_type is vlan, it is necessary to specify
physical_network and segmentation_id.

JIRA: FUNCTEST-863

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

Fix validate_ssh_client

After the "try..finally.." statement is executed, the ssh_client is
closed by the "finally" statement. So the out.channel.in_buffer is
not readable and the testcases failed.

Change-Id: I3fbf620cb9ccee62c515b83fed9fd01238ad3262
Signed-off-by: Alex Yang <yangyang1@zte.com.cn>

Merge "Scripts to allow Jenkins to run the SNAPS-OO tests."

Merge "SNAPS Stack creators can now return SNAPS network creators."

Scripts to allow Jenkins to run the SNAPS-OO tests.

JIRA: SNAPS-80

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

Assign a role to the new user

Fix the patch: https://gerrit.opnfv.org/gerrit/#/c/37079
2 tests are influenced:
    test_create_project_sec_grp_one_user
    test_create_project_sec_grp_two_users

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

SNAPS Stack creators can now return SNAPS network creators.

As Heat Stacks are responsible for spawning objects in OpenStack,
the class OpenStackHeatStack which is responsible for applying and
managing the state of a stack now can retrieve OpenStackNetwork
objects for the networks created in the stack for clients who would
like to query the networks and subnets or update them outside of
Heat.

JIRA: SNAPS-171

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

Added feature to update the quotas on a project/tenant.

JIRA: SNAPS-170

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

Merge "Reformat auth_url based on the ID API version."

Added test ability for 2 VMs on different networks to ping each other.

JIRA: SNAPS-48

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

Merge "Refactored neutron_utils#get_router_by_name() to get_router()"

Configure requirements for AAVMF(UEFI) with Cirros image on AArch64

Cirros image on AArch64 with AAVMF(UEFI) requires at least 256MB RAM.
The default 128MB was changed to 256MB.

Change-Id: I7414de519dcb53337f83690d1470ea672cec739b
Signed-off-by: Delia Popescu <delia.popescu@enea.com>
JIRA: ARMBAND-279

Refactored neutron_utils#get_router_by_name() to get_router()

Added a router_settings (RouterSettings) object parameter
in addition to router_name for more robust query options.

JIRA: SNAPS-163

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

Refactored neutron_utils#get_security_group()

Added a sec_grp_settings (SecurityGroupSettings) object parameter
in addition to sec_grp_name for more robust query options.

JIRA: SNAPS-164

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

Merge "Refactored neutron_utils#get_subnet_by_name() to get_subnet()"

Merge "Refactor server retrieval API calls."

Merge "Refactored port retrieval to include PortSettigs."

Merge "Delete 'default' security group associated to new projects."

Add domain name when creating projects and users

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

Refactored neutron_utils#get_subnet_by_name() to get_subnet()

Renamed utility function and added a subnet_settings parameter to
hold a SubnetSettings object to help make the query more robust
by being able to leverage values on the settings object.

JIRA: SNAPS-165

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

Refactor server retrieval API calls.

Refactored nova_utils#get_servers_by_name() to get_server() and
include a VmInstSettings object in addition to the server_name
parameter as well as only returning one VmInst object.

JIRA: SNAPS-168

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

Refactored port retrieval to include PortSettigs.

Refactored neutron_utils#get_port_by_name to get_port while
adding in a port_settings object for more robust query options.

JIRA: SNAPS-162

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

Delete 'default' security group associated to new projects.

OpenStack can create a security group named 'default' each time a
new project/tenant has been created but is not cleaned up when the
project has been deleted. This patch adds this functionality into
OpenStackProject#clean() method.

Also added similar routine to KeystoneUtilsTests#tearDown() method.

JIRA: SNAPS-169

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

Merge "Refactor network retrieval API calls."

Merge "Add domain name when creating projects and users"

Add domain name when creating projects and users

(spisarski)
Also clarified the difference between domain ID and name
Added helper methods and domain object for Domains

JIRA: FUNCTEST-858

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

Refactor network retrieval API calls.

Refactored neutron_utils#get_network() to also accept a
NetworkSettings object for more robust queries in addition to
the old network_name parameter. Also refactored neutron_utils#
get_network_by_id to add in the ID to list_networks((**{'id': id)
and returning the first item contains the expected ID value.

JIRA: SNAPS-161

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

Merge "Changed heat_utils#get_stack_by_name to get_stack."

Fixed logging out the OSCreds.

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

Merge "Changed keystone_utils#get_project to accept settings"

Merge "Added image_settings parameter to get_image()."

Fixed failing router test on Aricent pod.

Removed project ID from network lookup for the router_external
network as the test test_create_router_admin_user_to_new_project()
as the OpenStackRouter object was including its own project ID to
the API call causing the network lookup to fail.
Note: this is the only pod where this bug has reared it's ugly head.

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

Changed keystone_utils#get_project to accept settings

Added project_settings parameter to allow for more robust
queries based on the known settings as required.

JIRA: SNAPS-159

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

Reformat auth_url based on the ID API version.

No longer raising an OSCredsError when the auth_url does
not end with a 'v' + some number. Additionally, the auth_url
will be massaged to remove any 'v' + num from the end of
the URL and generate its own version value based on the ID
API version configured.

JIRA: SNAPS-144

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

Changed heat_utils#get_stack_by_name to get_stack.

Added stack_settings parameter to allow for more robust
queries based on the known settings as required.

JIRA: SNAPS-158

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

Added image_settings parameter to get_image().

With the ImageSettings object, was able to add logic to
filter on the image format field in addition to the name.

JIRA: SNAPS-157

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

Attempt to fix domain support.

The prior code treated *_domain_name & *_domain_id the same
where the ID was taken first and the name was used as the
fallback which was incorrect. This patch treats the name and
ID fields as separate members.

JIRA: SNAPS-154

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

Merge "Remove TODOs from create_router.py"

Remove TODOs from create_router.py

Determined not to support the optional router attributes:
'enable_snat' & 'external_fixed_ips' at this time.

JIRA: SNAPS-150

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

Removed floating IP list from OpenStackVmInstance.

There was a list and dict both holding the same floating IP
objects which has been problematic especially when trying
to initialize the object with a VM instance that already
exists.

JIRA: SNAPS-149

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

Ensure library and tests close all necessary resources.

The SNAPS-OO library and tests had left open files, ssh, and scp
connections. These have all now been wrapped with try/finally
blocks.

JIRA: SNAPS-152

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

Merge "Changes to enable overriding the OSCreds for tests."

Merge "Expand OpenStackSecurityGroup class tests."

Merge "Expanded project creation tests and validation."

Merge "Added configuration for keypair size."

Changes to enable overriding the OSCreds for tests.

JIRA: FUNCTEST-847

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

Updated docs for the OSCreds class

JIRA: SNAPS-122

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

Expand OpenStackSecurityGroup class tests.

Improved validation and added another test case.

JIRA: SNAPS-148

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

Expanded project creation tests and validation.

JIRA: SNAPS-147

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

Comment housekeeping...

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

Removed TODO from comment about the physical network value.

JIRA: SNAPS-123

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

Removed TODO comment from lanucher app.

Determined that the potential issue was not one.

JIRA: SNAPS-123

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

Removed TODO comment from sample playbook.

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

Added configuration for keypair size.

Removed hardcoded key byte value of 1024 and added the abiltiy
for the user to define their key size.

JIRA: SNAPS-20

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

Added region support.

Added region_name attribute to OSCreds
Added region_name to neutron, nova, glance, heat, and keystone
client retrieval
Fixed false positive heat connection test.

JIRA: SNAPS-50 & SNAPS-146

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

Merge "Added validation router is attached to the external network."

Merge "Added keypair config for key file deletion."

Merge "Updated docs."

Added validation router is attached to the external network.

Verifying that the external network's ID is the same as the 'network_id'
value in router.external_gateway_info.

JIRA: SNAPS-145

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

Added keypair config for key file deletion.

Added KeypairSetting attribute named 'delete_on_clean' which is used
to override the keypair creator's logic for deleting its associated
public/private key files.

JIRA: SNAPS-105

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

Updated docs.

JIRA: SNAPS-122

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

Fixed problems when setting the OpenStack project ID.

There were several calls to retrieve the project ID that required
named parameters which was causing the utility function to always
return None. Additionally, when creating the dict() required by the
Neutron create API 'body' parameter key 'project_id' is not supported
and still uses 'tenant_id'.

As these problems should have been found earlier, this patch also
extends the tests for networks, subnets, ports, and security groups
to use 'tenant_id' rather than 'project_id'.

JIRA: SNAPS-143

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

Merge "Default OSCreds cacert attribute to False."

Merge "Fixed potential problems with image creation exceptions."

Merge "Remove temporary directory created for tests."

Merge "Created new exceptions for security group settings"

Merge "Created new classes RouterSettingsError & RouterCreationError."

Merge "Created new class ProjectSettingsError."

Fixed potential problems with image creation exceptions.

With Glance client v2, it appears that rolling back on image
creation error was sending in an OpenStack image object into
the delete_image() function where it should have been expecting
a SNAPS-OO Image domain object.
Also added log messages to help expose other potential issues
if this patch does not address the root problem at hand.

JIRA: SNAPS-139

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

Merge "Refactor glance_utils.py v1 image creation."

Added and improved flavor creation tests.

JIRA: SNAPS-142

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

Refactor glance_utils.py v1 image creation.

Replace 4 blocks using named parameters with on call to
create() with kwargs.

JIRA: SNAPS-141

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

Remove temporary directory created for tests.

JIRA: SNAPS-140

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

Created new classes RouterSettingsError & RouterCreationError.

Raising RouterSettingsError in RouterSettings and RouterCreationError in
OpenStackRouter instead of Exception.

JIRA: SNAPS-136

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

Default OSCreds cacert attribute to False.

The default of True causes more problems than False when
dealing with unsecure and secure (HTTPS) API calls.
This was issue was found while testing against the new OPNFV
CI test pod running Pike.

JIRA: SNAPS-80

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

Created new exceptions for security group settings

Raising SecurityGroupSettingsError in SecurityGroupSettings
and SecurityGroupRuleSettingsError in SecurityGroupRuleSettings
instead of Exception.

JIRA: SNAPS-137

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

Created new class ProjectSettingsError.

Raising ProjectSettingsError in ProjectSettings instead of Exception.

JIRA: SNAPS-135

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

Merge "Created new exception classes for networks."

Merge "Created new class KeypairSettingsError."

Merge "Changes required for running CI tests (Pike pod)."

Created new exception classes for networks.

Raising NetworkSettingsError in NetworkSettings, SubnetSettingsError in
SubnetSettings and PortSettingsError in PortSettings instead of Exception.

JIRA: SNAPS-134

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

Changes required for running CI tests (Pike pod).

Added support for HTTPS proxies
HTTPS OSCreds settings for both RC file and os_env.yaml
OSCreds 'cacert' T/F and cert file path support
OSCreds API version support cleanup including the addition
of heat
Added more OSCreds test validations
Disabling of InsecureRequestWarning
PEP8 line width refactoring
heat_utils_test.py fix when stack status is error
test suite for CI
removed default flavor metadata of mem_page_size: Any to None

JIRA: SNAPS-80

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

Created new class KeypairSettingsError.

Raising KeypairSettingsError in KeypairSettings instead of Exception.

JIRA: SNAPS-133

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

Merge "Created custom exceptions for VM instance creation."

Merge "Created new class FlavorSettingsError."

Fixed logger name.

Logger was named incorrectly causing confusion.

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

Created custom exceptions for VM instance creation.

Created VmInstanceSettingsError for errors creating VmInstanceSettings
objects
Created FloatingIpSettingsError for errors creating FloatingIpSettings
objects
Created VmInstanceCreationError for errors creating VM instances

JIRA: SNAPS-132

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

Created new class FlavorSettingsError.

Raising FlavorSettingsError in FlavorSettings instead of Exception.

JIRA: SNAPS-131

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

Merge "Added proper assertion config NIC test."

Merge "Created new class AnsibleException."

Merge "Created new class NovaException."

Merge "Created new class NeutronException."

Created new class AnsibleException.

Raising AnsibleException in ansible_utils.py instead of Exception.

JIRA: SNAPS-130

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