Commit Graph

104 Commits

Author SHA1 Message Date
John L. Villalovos ac7988fe14 Remove ironic_tempest_plugin/ directory
People can read the release notes to find out that the
ironic_tempest_plugin/ has been removed and moved to the
openstack/ironic-tempest-plugin project

Change-Id: Iabd7a7a19bb5c4bd07f9776bb1cdeadb1ac2e7c7
2018-01-10 09:42:53 -08:00
John L. Villalovos 7e1287f243 Remove ironic_tempest_plugin/ directory
We now use the project openstack/ironic-tempest-plugin to store our
tempest plugin. All content from the ironic_tempest_plugin/ directory
has been ported to that project.

We no longer want to have the plugin content stored here so we delete
it.

Remove check in tools/flake8wrap.sh that prevented changes to the
ironic_tempest_plugin/ directory.

Change-Id: I700bd7b71472fa91f6bc02aebc055584df08e0ef
2018-01-05 15:12:13 -08:00
Vasyl Saienko f4fc9a044c Ensure ping actually succed
Check return value from ping_ip_address() and make sure it is True

Change-Id: I45d03ac02731935395dd69e10a835cd2950d0f5f
2017-10-31 14:36:31 +02:00
Vladyslav Drok e0514cb85b Add I202 to flake ignore list
Pep8 job currently fails complaining about I202 in a bunch of
modules. This change fixes the genuine errors in the module
import order, and adds I202 to the skip list so that we don't
have to add noqa comments everywhere we can not satisfy the
check. After we split out the tempest plugin, we'll remove
I202 from the ignore list and add noqa comments in the places
where it will still be needed.

Change-Id: Ia170a41d35dea8c9eda2b36c907dbc518169a824
2017-10-25 14:32:52 +03:00
Jenkins 03fd6328b6 Merge "Update tests to do not use deprecated test.services()" 2017-10-13 03:47:21 +00:00
Vasyl Saienko 0ce051db23 Update tests to do not use deprecated test.services()
Function 'tempest.test.services()' has moved to
'tempest.common.utils.services()' in version 'Pike'.
This commit update tests accordingly.

Change-Id: I1924ccabb1eef8913c1e20775df21ccbef1e6095
2017-10-05 17:01:12 +03:00
Luong Anh Tuan 5ef15a4dc3 Update after recent removal of cred manager aliases
In tempest, alias 'admin_manager' has been moved to 'os_admin'
'alt_manager' to 'os_alt' and 'manager' to 'os_primary' in
version Pike, and it will be removed in version Queens[1].

[1]I5f7164f7a7ec5d4380ca22885000caa0183a0bf7

Change-Id: Ifec0e661031647555dbc03ad1000c50c590afa8c
Closes-bug: 1697588
2017-09-28 13:30:28 +07:00
Vasyl Saienko bab6dd6de2 Fix API VIF tests when using flat network
Instead of creating new network, pick one that is specified in
config when using shared network.

Change-Id: I927fe55de77956427916dba682f6927c4a8d9882
2017-09-22 21:18:04 +03:00
Vasyl Saienko 5cb1720b6e Update vif_attach from NeutronVIFPortIDMixin
This patch updates vif_attach from NeutronVIFPortIDMixin to rely
on presence of Neutron. Drop incorrect logic that skips mac_address
update when show_port() to neutron failed as NeutronVIFPortIDMixin
is used only in flat and neutron network interfaces which require
neutron presence.
Update api tests to use real neutron port.

Change-Id: Iffaf3569aa296b4466729cf7f62c92995b6b3723
2017-09-08 10:10:14 +00:00
Jenkins 09627c8436 Merge "Add functional API tests for volume connector and volume target" 2017-08-19 11:31:37 +00:00
Nguyen Hung Phuong 74a126ab0e Add functional API tests for volume connector and volume target
Extend baremetal json client with volume connector and volume target.
Add basic positive tests for volume connector and volume target:

    test_create_volume_connector_error
    test_delete_volume_connector
    test_delete_volume_connector_error
    test_show_volume_connector
    test_list_volume_connectors
    test_list_with_limit
    test_update_volume_connector_replace
    test_update_volume_connector_replace_error
    test_update_volume_connector_remove_item
    test_update_volume_connector_remove_collection
    test_update_volume_connector_add

    test_create_volume_target_error
    test_delete_volume_target
    test_delete_volume_target_error
    test_show_volume_target
    test_list_volume_targets
    test_list_with_limit
    test_update_volume_target_replace
    test_update_volume_target_replace_error
    test_update_volume_target_remove_item
    test_update_volume_target_remove_collection
    test_update_volume_target_add

Change-Id: I642a02cff2afe7f89b6800d69e14d05e04e6a59c
Partial-bug: #1559691
2017-08-15 18:42:10 -04:00
Jenkins c02fa7193f Merge "Add tempest plugin API tests for driver" 2017-08-11 12:41:26 +00:00
Dmitry Tantsur a212e58d09 [tempest] also catch BadRequest in negative tests with physical_network in old API
Before this field was introduced we would raise BadRequest (HTTP 400),
after - NotAcceptable (HTTP 406).

Check for any of the exceptions to allow branchless testing.

Change-Id: I3fafd4ef7f046a9195fc15f47f7fc967fb959550
2017-08-04 12:35:37 +02:00
Dmitry Tantsur aaa429d887 [Tempest] fix negative tests on old API versions
Currently, these tests rely on an arbitrary chosen version, without checking
if the installation supports it. This results in failures on older installations.

Change these tests to not request any version, thus using the minimum available one.

Change-Id: I624fbad36d25328b6eb6c38a9a5fb5837e6751d4
2017-08-01 14:30:50 +02:00
Jenkins 4ff2120f3b Merge "Add negative attribute to negative port tests" 2017-08-01 04:49:06 +00:00
Dao Cong Tien b05a6a64b1 Tempest scenario test for boot-from-volume
Added a tempest test to create a cinder volume, then create a
server with appropriate config options for block device mapping.
That will result in the baremetal node to boot from the cinder
volume.

Change-Id: Ife0db2b8a1e639ac1e3d617b981eb81f793a71f1
Partial-bug: #1559691
2017-07-28 08:19:08 +07:00
Mark Goddard 63a852eb8c Add negative attribute to negative port tests
Change I8b30764d797f2f8b45c2ae46ce559e74e0281a49 added tempest tests for
the port.physical_network field. In that review, it was noted that the
negative tests should have a decorator applied to mark them as negative.
This change applies that decorator.

Change-Id: I69446703894b89596183c50482a4286eaac4ed42
Related-Bug: #1666009
2017-07-26 19:36:04 +01:00
Jenkins a5205bdd3e Merge "Add tempest tests for physical networks" 2017-07-25 11:44:01 +00:00
Mark Goddard 975040b3e1 Add tempest tests for physical networks
This change adds tests to the ironic tempest plugin to cover the
API changes made for the physical network awareness feature in
I7023a1d6618608c867c31396fa677d3016ca493e.

Change-Id: I8b30764d797f2f8b45c2ae46ce559e74e0281a49
Partial-Bug: #1666009
2017-07-21 14:31:49 +01:00
Jenkins 6fba11dfdb Merge "Replace the usage of 'manager' with 'os_primary'" 2017-07-21 10:30:18 +00:00
wangkf f1966536f9 Fix broken links in tempest plugin README
The link is broken due to doc-migration, this patch fixes following
broken links:

- the link to tempest plugin
- the link to running tempest part in ironic dev-quickstart

Change-Id: I7b4287ad74795712fb81d3177d950450c0dadf65
2017-07-10 09:22:12 +08:00
Vu Cong Tuan c10bfc9152 Replace the usage of 'manager' with 'os_primary'
In tempest, alias 'manager' has been moved to 'os_primary'
in version Pike, and it will be removed in version Queens [1].

The other usages which also need to be replaced, are listed below:
* admin_manager > os_admin [2]
* os_adm > os_admin [3]
* os > os_primary [4]

[1] https://review.openstack.org/#/c/468036/
[2] https://review.openstack.org/#/c/467852/
[3] https://review.openstack.org/#/c/467605/
[4] https://review.openstack.org/#/c/466991/

Change-Id: I295bc37b903ada341320b73de6ebb4bf31aa955b
2017-06-23 20:25:32 +07:00
Jenkins 552a058f0c Merge "Replace test.attr with decorators.attr" 2017-05-29 21:10:01 +00:00
xgwang5843 b4d8bfa067 Replace test.attr with decorators.attr
Function 'tempest.test.attr()' has moved to 'tempest.lib.decorators
.attr()' in Pike and will be removed in a future version[1].
This patch replaces the 'tempest.test.attr()' with the 'tempest.lib
.decorators.attr().'
[1] Iaafbb112b6eee458089cc49918359a8a8d0485e2

Change-Id: Ib0384f67b3fdd8812eaa9a71ba2f3ead17b04577
2017-05-26 17:33:47 +08:00
xgwang5843 a03b5bba8b Replace test.attr with decorators.attr
Function 'tempest.test.attr()' has moved to 'tempest.lib.decorators
.attr()' in Pike and will be removed in a future version[1].
This patch replaces the 'tempest.test.attr()' with the 'tempest.lib
.decorators.attr().'

[1] Iaafbb112b6eee458089cc49918359a8a8d0485e2

Change-Id: If39f271124be63888073d3179090e508b4b1bc1f
2017-05-26 09:30:16 +08:00
Luong Anh Tuan 8118a18da2 Replace test.attr with decorators.attr
Function 'tempest.test.attr()' has moved to 'tempest.lib.decorators
.attr()' in Pike and will be removed in a future version[1].
This patch replaces the 'tempest.test.attr()' with the 'tempest.lib
.decorators.attr().'

[1] Iaafbb112b6eee458089cc49918359a8a8d0485e2

Change-Id: I4796a1e10311cae2910e0d3ee5d0cb8368fbed1e
2017-05-19 15:59:05 +07:00
Kyrylo Romanenko 061dfb0d97 Add functional api tests for node resource class
Add following tests:
  Create new node with specified longest name of resource class.
  Update existing node with specified resource class.
  Show resource class field of specified node.
  List nodes of specified resource class only.
  Get detailed nodes list of specified resource class only.
  Try to create a node with too long resource class name.
  Try to update a node with too long resource class name.
  Try to create a node with resource class using older api version.
  Try to update a node with resource class using older api version.
  Try to list nodes with resource class using older api version.

Closes-Bug: #1668605
Change-Id: If412a2aeab2ab457f04067cf296cac80b4378482
2017-05-03 17:31:24 +03:00
Jenkins a8abc54c1b Merge "Add assertion of name to test_list_portgroups test" 2017-04-04 14:12:59 +00:00
Jenkins fb76658318 Merge "Skip PortNotFound when unbinding port" 2017-03-30 13:29:23 +00:00
Kyrylo Romanenko 0594b85ea4 Add assertion of name to test_list_portgroups test
Change-Id: Ic745433cd425c874b14ab99a4e326cf41a1d15cf
2017-03-30 15:38:55 +03:00
Jenkins 70fd2e36b7 Merge "Skip VIF tests for standalone ironic" 2017-03-30 11:32:15 +00:00
Vasyl Saienko 940c87d6c4 Skip PortNotFound when unbinding port
There might be cases when user deleted port before doing vif_detach.
With this patch info message will be shown in the logs for such cases.

Change-Id: Ic3c5073130a4839f27692db1862100bdd7dfca1e
2017-03-30 12:17:54 +03:00
Jenkins 8aec5fb6a5 Merge "Fix docstrings for creating methods in baremetal api tests" 2017-03-24 15:13:46 +00:00
Vasyl Saienko beb186721f Define minimum required API ver for portgroups
This patch defines min_microversion for API portgroup tests to
skip them when this API version is not supported/available.
For example on stable/mitaka branches.

Change-Id: I8895e1e6a819b5b3821fdfc10e1cff8626c9eb29
2017-03-24 10:41:47 +00:00
Jenkins 604a6fe3e9 Merge "Add standalone tests using direct HTTP links" 2017-03-23 17:55:06 +00:00
Kyrylo Romanenko cc5fc354ce Fix docstrings for creating methods in baremetal api tests
Change-Id: Iec39fbf4ceac9e7b697ddacc3eb9a4f1defb699e
2017-03-23 11:20:09 +00:00
Kyrylo Romanenko 7359c360e5 Extend tests and checks for node VIFs
Add more checks to the test_vifs test:
  - check that attached VIF appears in port internal_info;
  - check that VIF was detached and does not appear in
    node's VIF list anymore.
Add test for VIF attachement to node with portgroup.

Partial-Bug: #1666534
Change-Id: I755df18e03d01fe93f85df114b979ea7ea70ca53
2017-03-23 10:10:58 +00:00
Jenkins e6850f780f Merge "Add functional API tests for portgroups" 2017-03-22 22:57:09 +00:00
Kyrylo Romanenko af1759da01 Add functional API tests for portgroups
Extend baremetal json client with portgroups.
Add basic positive tests for portgroups baremetal api resources:
  test_create_portgroup_with_address,
  test_create_portgroup_no_address,
  test_delete_portgroup,
  test_show_portgroup,
  test_list_portgroups.

Closes-Bug: #1666858
Change-Id: Ifb17a43fa61d0ffb22b7af25174206cf798ca0ee
2017-03-22 12:37:05 +00:00
Pavlo Shchelokovskyy c5eaafdf9f Fix local copy of scenario manager
add back some methods that were pruned. They are used in tests running
on stable branches.

Change-Id: I2e0c979f870777d3f643ab8b87c5b1f0b9601c34
Closes-Bug: #1674597
2017-03-21 08:00:17 +00:00
Vasyl Saienko f7d0fbd830 Add standalone tests using direct HTTP links
Adds two tests that uses direct HTTP link instead of glance:
  * pxe_ipmitool + wholedisk on HTTP server
  * agent_ipmitool + wholedisk on HTTP server

Change-Id: I8f1a5b0ecb3d53ffdefd7018c1b9700210572ffc
2017-03-21 09:46:17 +02:00
Solio Sarabia 352aacfd45 Prune local copy of tempest.scenario.manager.py
Remove unused code from manager.py to better identify ironic
dependencies on this tempest interface. This is a follow-up
patch to https://review.openstack.org/#/c/439252/.

Change-Id: I3ca7cf3af42e0e55431d5affa36e567f147aa850
2017-03-16 23:45:16 -05:00
Solio Sarabia 881e5ad9db Include a copy of tempest.scenario.manager module
Add tempest/scenario/manager.py from openstack/tempest [1], and update
imports in ironic. As announced in [2], the interface in manager.py is
not stable, and should not be consumed outside tempest. With this
temporary change, QA will have time to improve the tempest.scenario
interface for other projects to consume, without breaking gate jobs.

An alternative to copying the local file was proposed in [3], which
suggested creating a new interface class, living along with the old
interface. Through a tempest config option projects would choose which
scenario interface class to use. QA reasoned that this would increase
the number of jobs, and slow their work. Thus the option was rejected.

[1] http://git.openstack.org/cgit/openstack/tempest/tree/tempest/scenario/manager.py?id=82a278e88c9e9f9ba49f81c1f8dba0bca7943daf
[2] http://lists.openstack.org/pipermail/openstack-dev/2017-February/112938.html
[3] http://lists.openstack.org/pipermail/openstack-dev/2017-March/113496.html

Closes-Bug: #1668807
Change-Id: Iaa06df6ab75fb8f79cbcd3f6b21e68622e7799c1
2017-03-16 10:17:14 -05:00
Jenkins 34bfba5209 Merge "Add ironic standlaone test with ipmi dynamic driver" 2017-03-16 11:12:47 +00:00
Jenkins 23a61f2d7b Merge "Add Ironic standalone tests" 2017-03-16 00:06:25 +00:00
Vasyl Saienko 3648f060ed Add ironic standlaone test with ipmi dynamic driver
This patch adds the following ironic standalone tests:
  * ipmi + wholedisk_image + bios
  * ipmi + partitioned_image + bios

Change-Id: I99b7dd23eab7e7c0a6c621c800e98c7c786e1244
Depends-On: I348a27a13d3025fe7e8ddfa31992b895a3809730
2017-03-15 16:49:08 +00:00
Vasyl Saienko e48375538c Add Ironic standalone tests
This patch adds the following standalone tests:

  * agent_ipmitool + wholedisk image + bios
  * agent_ipmitool + partitioned image + bios
  * pxe_ipmitool + wholedisk image + bios
  * pxe_ipmitool + partitioned image + bios

Partial-Bug: #1660606

Change-Id: Ic04b0f134e20d9937a610a14d7c4128f45738eeb
2017-03-15 16:54:45 +02:00
Vladyslav Drok ac075b7f35 Fix the exception message in tempest plugin
Follow up to commit 733ff3fff7

Change-Id: I3498268c6dd7626c1e99ded9fdfb287b4197c999
2017-03-14 06:41:23 +00:00
Vladyslav Drok 632f082b2c Refactor waiters in our tempest plugin
Deduplicates some of the code between common.waiters module and
BaremetalScenarioTest functions.

Change-Id: Ia87646cccdefba22caf121f4e347e8f2edf736e0
2017-03-13 19:30:58 +00:00
SofiiaAndriichenko 598f783825 This adds a tempest test for creating a chassis with a specific UUID.
Change-Id: I51fbcd0613c16618d5e4a52ae1d798321a21bced
2017-03-13 13:33:23 +00:00