Added support for using '~' for SSH key location.

While testing changes, fixed problems found with querying for
floating IPs which also required adding network data to the
VMInst domain object.

JIRA: SNAPS-85

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

Making test user password dynamic for testing against secure pods.

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

Merge "Refactor OSCreds to leverage kwargs instead of named parameters."

Refactor OSCreds to leverage kwargs instead of named parameters.

JIRA: SNAPS-109

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

Merge "Add a warning of keypair creation"

Add a warning of keypair creation

Put a warning if the public key already exist in OpenStack
but the private key file is not found

Change-Id: Ie2dc5b9f6a6abb0831e1826130d5c283cf3f34ec
Signed-off-by: boucherv <valentin.boucher@orange.com>

Sync requirements.txt with OpenStack (novaclient)

It completes "Use neutron to create floating IPs" [1]

[1] https://gerrit.opnfv.org/gerrit/#/c/36389/

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

Fixed tests to delete security groups.

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

Merge "Changed credentials for deploying heat templates."

Changed credentials for deploying heat templates.

The old credentials had been working on other pods but are not working
with our latest deployed by Kolla. This change does not affect any of
the functionality included in the tests. It simply takes the administrative
user credentials and changes the project to the newly created one.

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

Merge "Fixed test to not download images when configured with a disk file path."

FIX mistake in import keypair

Change-Id: I3d0a3111efcb4200c9b867663a45c07a3b47c09d
Signed-off-by: boucherv <valentin.boucher@orange.com>

Fixed test to not download images when configured with a disk file path.

JIRA: FUNCTEST-809

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

Added offline test support to NovaUtilsInstanceTests missed initially.

JIRA: FUNCTEST-809

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

Enable snaps work using public endpoint

When functest is not run on jumpserver but on other external host without
admin network, snaps failed to connect the admin endpoint. So running
snaps tests using public endpoint should be allowed.

JIRA: SNAPS-110

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

Use neutron to create floating IPs.

This patch moves the floating IP creation out of nova and into neutron.
Other changes include the use of domain objects for VM and Floating IP
instances, addition of new nova_utils tests to exercise the create
server functionality, and more PEP8 compliance.

JIRA: SNAPS-92

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

Enable https for Openstack in Snaps

When running in https environment, snaps should provide two options:
1. To support certification verify when https certification file is provided;
2. To disable server certificate verification without cert file.

JIRA: SNAPS-84

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

Fixed launcher app to support new settings kwargs.

Additionally, found issue with importing ansible libraries.
Needed to rename the snaps.provisioning.ansible package to 'ansible_pb'.

JIRA: SNAPS-107 & SNAPS-93

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

Merge "Changes UserSettings constructor to use kwargs."

Merge "Changes StackSettings constructor to use kwargs."

Merge "Changes SecurityGroupSettings & SecurityGroupRuleSettings constructor to use kwargs."

Merge "Changes to RouterSettings constructors to use kwargs."

Merge "Changes to ProjectSettings constructors to use kwargs."

Merge "Changes to NetworkSettings, SubnetSettings, and PortSettings constructors to use kwargs."

Merge "Changes to KeypairSettings constructor to use kwargs."

Merge "Changes to VMInstanceSettings and FloatingIPSettings constructors."

Merge "Changes ImageSettings constructor to use kwargs."

Merge "Changes FlavorSettings constructor to use kwargs."

Changes UserSettings constructor to use kwargs.

And changed line lengths to 79 for pep8.

JIRA: SNAPS-104

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

Changes StackSettings constructor to use kwargs.

And changed line lengths to 79 for pep8.

JIRA: SNAPS-103

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

Changes SecurityGroupSettings & SecurityGroupRuleSettings constructor to use kwargs.

And changed line lengths to 79 for pep8.

JIRA: SNAPS-102

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

Changes to RouterSettings constructors to use kwargs.

And changed line lengths to 79 for pep8

JIRA: SNAPS-101

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

Changes to ProjectSettings constructors to use kwargs.

And changed line lengths to 79 for pep8

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

Changes to NetworkSettings, SubnetSettings, and PortSettings constructors to use kwargs.

And changed line lengths to 79 for pep8

JIRA: SNAPS-99

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

Changes to KeypairSettings constructor to use kwargs.

And changed line lengths to 79 for pep8.

JIRA: SNAPS-98

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

Changes to VMInstanceSettings and FloatingIPSettings constructors.

Removed named parameters and replaced with kwargs.

JIRA: SNAPS-97

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

Changes ImageSettings constructor to use kwargs.

And changed line lengths to 79 for pep8.

JIRA: SNAPS-96

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

Merge "Missed image config for configuring multiple NICs test"

Changes FlavorSettings constructor to use kwargs.

And changed line lengths to 79 for pep8.

JIRA: SNAPS-95

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

Merge "Ignore several files"

Merge "Leverage on pbr (Python Build Reasonableness)"

Ignore several files

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

Leverage on pbr (Python Build Reasonableness)

It also updates requirements according to OpenStack stable/ocata
except for nova client version as snaps needs nova support of
floating IP which has been removed in pike [1] (8.0.0 [2]).

It must be reverted to >=6.0.0,!=7.0.0 as soon as snaps calls the
related neutron commands.

[1] https://github.com/openstack/python-novaclient/blob/master/releasenotes/notes/pike-rm-deprecated-net-272aeb62b329a5bc.yaml
[2] https://docs.openstack.org/releasenotes/python-novaclient/unreleased.html

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

Need to move file resources required by tests.

Functest was not installing the heat resource files required
by the snaps tests being run by Functest as the heat template
file used by the tests as it was under the repository's root
directory which is causing tests to fail.

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

Changed logic to return only active availability zones.

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

Create download directory if not exists before downloading image file.

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

Missed image config for configuring multiple NICs test

JIRA: SNAPS-89

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

Fixed problems found in Functest after implementing offline testing.

JIRA: FUNCTEST-809

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

Removed current working directory logic retrieving test file resources.

Some of the SNAPS tests leverage file resources contained within the
project. These resources were being accessed via relative paths which
required the test clients to ensure that the CWD was the <repo>/snaps
directory. Replaced that logic to leverage the import pkg_resources
making the tests much more flexible and robust.

JIRA: SNAPS-89

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

Merge "Added custom security group with ICMP and SSH rules."

Added custom security group with ICMP and SSH rules.

This was done for all tests exercising floating IPs.
Also added check to all of these tests to ensure the VMs
all have received their private IP address via DHCP.

JIRA: FUNCTEST-834

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

Added support for applying Heat Templates

Second patch expanded support to both files and dict() objects.
Third patch exposes new accessor for status and outputs.

JIRA: SNAPS-86

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

Fixed RST warning.

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

Refactor keystone endpoint tests to the proper test class.
Also added documentation for those three tests.

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

Updated installation documents and fixed problems found during investigation.

The code changes are primarily required when the runtime environment is Windows.

JIRA: SNAPS-75

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

Find packages in setup.py

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

Merge "Fixed error in ansible_utils_tests.py and added ansible helper method to the OpenStackVMInstance class."

Fix typo in setup.py

It forbids gating in functest.

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

Fixed error in ansible_utils_tests.py and added ansible helper
method to the OpenStackVMInstance class.

Test error was due to the image configuration having the incorrect
user applied. The helper method will make it easier for adhoc
ansible playbooks to be applied to running instances.

JIRA: SNAPS-83

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

Add the function of getting endpoint

Functest is trying to leverage SNAPS-OO library to replace some
utility functions, in which def get_endpoint is called by testcases
onos, odl, multisite and cloudify_ims.

JIRA: SNAPS-78

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

bugfix: get_security_group_by_id always return first element

add check on id before return value
fixed kwarg into the neutron.list_security_groups to the correct one 'id'
added unit test to ensure this function always works as designed

JIRA: SNAPS-82

Change-Id: I965e1dd54fd1efa8c2d7f6efb87c78ab24cca0e5
Signed-off-by: Morgan Richomme <morgan.richomme@orange.com>

Added support for offline testing

Expanded the image_metadata used by the tests for overriding default
images for not only supporting 3part images but also to support offline
testing as required by Functest.

JIRA: SNAPS-67

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

Ensure creators' constructors cannot raise exceptions.

While creating unit tests for vPing that is using SNAPS, exceptions can
be raised when retrieving their respective OpenStack clients with
incorrect credentials. Moved client retrieval to the create() method
which one should expect exceptions to be raised.

JIRA: SNAPS-81

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

Merge "Restricted dependency of the novaclient."

Merge "Modified code to support both Python 2.7 and 3.x"

Fixed some redundant dict() key checks that can cause runtime errors.

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

Modified code to support both Python 2.7 and 3.x

* Tested on Python 2.7.10 and 3.4.4
* Updated installation documentation

JIRA: SNAPS-30

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

Restricted dependency of the novaclient.

The 8.0.0 version is not backwards compatible with 7.1

JIRA: SNAPS-73

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

Changed image metadata to support cirros, ubuntu, and centos.

The image metadata that was designed for multi-part form support
did not take into account tests that require ubuntu or centos
tests.

JIRA: SNAPS-76

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

Merge "Refactored multi-part images."

Merge "Matching test_check_vm_ip_dhcp on ip only"

Matching test_check_vm_ip_dhcp on ip only

No nee dto look for the string surrounding the ip that's being looked
for in this test. simplifying the code to only look for the ip

Change-Id: I80e6336c58573e5970eafe2608daf7c61e43b717
Signed-off-by: Dan Radez <dradez@redhat.com>

Merge "Fixed test for security groups when checking for project/tenant ID"

Refactored multi-part images.

The logic used for creating multi-part images for the Danube release
was copied all throughout the tests, which was not sustainable.
This patch encapsulates the kernel and ramdisk images within the
same OpenStackImage object as the disk image.

JIRA: SNAPS-59

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

Fixed issue caused previous patch which did not merge correctly.

JIRA: SNAPS-57

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

Fixed the Glance connection test routines.
test_glance_connect_success gave a false positive and
test_glance_connect_fail gave a false negative.
Both were calling incorrect APIs due to copying from the equivalent
nova_utils.py test.

JIRA: SNAPS-57

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

Fixed test for security groups when checking for project/tenant ID

This problem has only been observed on the WindRiver pod during the
OPNFV Plug/Hackfest

JIRA: SNAPS-72

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

Added support for Glance v2

Updated copyright date on new and edited files to current year.

JIRA: SNAPS-66

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

Merge "Run all test suites when none chosen."

Run all test suites when none chosen.

Changed the test_runner.py to run all four suites of tests
when none have been chosen.

JIRA: SNAPS-10

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

Ensuring each port manipulation test waits for the instance to become active

The problem with OpenStack is that when one attempts to delete an instance
that is still spawning, it can take an inordinate amount of time for OpenStack
to complete the request resulting in artifacts being left.

JIRA: SNAPS-64

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

Refactored SNAPS tests to take in the actual OSCreds object.

The code previously had individual parameters for different
aspects of the credentials. This change not only makes the
method calls easier, but it also helps the logs from becoming
too verbose as each test case was instantiating its own OSCreds
instance.

JIRA: SNAPS-23

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

Merge "Renamed test application and added new command line arguments."

Renamed test application and added new command line arguments.

JIRA: SNAPS-10

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

Merge "Fixed two tests that never had their networks removed on tearDown()"

Fixed two tests that never had their networks removed on tearDown()

JIRA: SNAPS-56

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

Problem found while working on FUNCTEST-577 where instances with
security groups need to be active before application.

JIRA: FUNCTEST-577

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

Merge "Check for kernel_url and ramdisk_url keys in dict"

Call to ansible_utils.apply_playbook needed named paramteters.

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

Fixed cleanup after failed deploy

Added exception handling when performing a cleanup after
a failed deploy.

This is a temporay fix, that will not impact FuncTest.

Router and network clean calls will eventually need to be
updated.

JIRA : SNAPS-55

Change-Id: I24d79802f02357015ada20726e170dd5128754f1
Signed-off-by: randyl <r.levensalor@cablelabs.com>

Correcting minor issues with the documentation

Correcting some formatting and minor content changes.

Change-Id: Idf75e8c6985bae9a2df26c54c559a20d46b6ac1f
Signed-off-by: randyl <r.levensalor@cablelabs.com>

Check for kernel_url and ramdisk_url keys in dict

Need to check if the above keys exist in image_metadata
before checking for their assigned values in case
they are not declared (e.g. obsolete when creating a
one part image)

Change-Id: I22fa259b320a425d6ad030eeb9bf68b4c06b667f
Signed-off-by: Catalina Focsa <catalina.focsa@enea.com>

Merge "Add DEBUG logs to SimpleHealthCheck for VM console output."

Add DEBUG logs to SimpleHealthCheck for VM console output.

The test in the class SimpleHealthCheck uses the console output
to determine whether or not the VM obtains a DHCP lease which
contains a very image-specific check. When a client changes
the image from what the test was designed for, the console
output can change causing the test to fail unless it is simply
timing out. This patch will output the full console log on
failure or on success when the logger is configured as DEBUG.

JIRA: SNAPS-47

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

reduce instance's memory size

when using snaps to test small OpenStack env(devstack or all-in-one),
testcase  will fail due to not enough memory for a vm. As a
basic test using cirros, only allocating ip and test the connectivity is
needed, 128M should be enough.
the error msg is:

  File "/usr/local/lib/python2.7/dist-packages/novaclient/client.py",
  line 80, in request
      raise exceptions.from_response(resp, body, url, method)
ClientException: Unknown Error (HTTP 504)

Change-Id: Ibfb27018ffd25292bcf80f69e8cb205b2c9c334b
Signed-off-by: SerenaFeng <feng.xiaowei@zte.com.cn>

Merge "Adapt integration tests to use custom image config"

Adapt integration tests to use custom image config

This fix uses the possibility to pass custom imformation about the
image via image_metadata variable. In the integration tests it checks
for this info and creates a 3-part image if the kernel_url and
ramdisk_url are set. The fix also allows the default url to be overriden
as well as passing extra-properties. All the metadata info comes from
the app.

The part in functest where the custom config is set for creating a 3-part
image is https://gerrit.opnfv.org/gerrit/29987

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

Add check that SSH validate returns true

The SSH Validate call needs to be checked before passing tests.
test_ssh_client_fip_before_active and test_ssh_client_fip_after_active
run the SSH Verify function, but do not check the return value.

You will see an error in the test log file, but the test will
appear to have passed.

JIRA: SNAPS-45

Added an Assert True test to the SSH Validate return.

Change-Id: Ib86ec38c7b5a64061a1ace4293336e3021c59013
Signed-off-by: randyl <r.levensalor@cablelabs.com>

Adapt simple health check to use 3-part image

This fix adds the possibility to pass custom imformation about the
image via image_metadata variable. In SimpleHealthcheck it checks
for this info and creates a 3-part image if the kernel_url and
ramdisk_url are set. The fix also allows the default url to be
overriden as well as passing extra-properties. All the metadata info
comes from the app.

The part in functest where the custom config is set for creating
a 3-part image is https://gerrit.opnfv.org/gerrit/#/c/29851/

JIRA: ARMBAND-222

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

Merge "Added the ability to give the tests the ability to add in flavor metadata."