summaryrefslogtreecommitdiff
path: root/tempest/api/compute/servers/test_instance_actions.py
Commit message (Collapse)AuthorAgeFilesLines
* Add reboot_server() helperArtom Lifshitz2021-08-241-3/+1
| | | | | | | | The new helper waits for the server to become ACTIVE again after the reboot. This saves us a few lines of code in various tests, and is also useful for the whitebox-tempest-plugin (see dependant patch). Change-Id: I5c89983de0cbdae91c868fc3fefdc45b9a96fa68
* Adding description for testcases - compute part12zhufl2020-08-131-3/+10
| | | | | | | | | | | | | | | When Tempest is used in customer site, often we are required to provide a testcase list including testcase names and descriptions. Now no this kind of doc is available, so we can add descriptions with the format of doc string for every testcase, so later we can generata such a testcase description list. There are hundreds of testcases missing descriptions, so we can add them gradually, and limit the modified files in one patch for the convenience of reviewing. Change-Id: I5564ff0b71f56445977a79c2c813b2e3e31000c9 partially-implements: blueprint testcase-description
* Create default network for more compute testsEric Fried2020-01-161-0/+1
| | | | | | | | | | | | Chipping away at the referenced bug, this commit uses the create_default_network class-level variable introduced via [1] to trigger setup_credentials to create a default network for a handful more test cases in which the problem was observed. [1] Ia34fe3b41261de02918713c2a948e6c56554bf6f Partial-Bug: #1844568 Change-Id: Ic38c74c245fddf29826e6ca0cc469f919076355e
* Create default network for several compute testsEric Fried2020-01-101-0/+1
| | | | | | | | | | | | Chipping away at the referenced bug, this commit introduces a class-level variable on BaseV2ComputeTest that triggers setup_credentials to create a default network, and sets it for several classes where the problem has been observed. The class variable is so we can whack-a-mole with one line instead of four whenever this problem pops up. Change-Id: Ia34fe3b41261de02918713c2a948e6c56554bf6f Partial-Bug: #1844568
* Switch to decorators.idempotent_id on compute.serversKen'ichi Ohmichi2017-01-271-4/+4
| | | | | | | | So many modules are using test.idempotent_id and this is for switching to decorators.idempotent_id on compute.servers. Change-Id: I6a7f8633b4808469dd92f2ac668674fba0bdb0ca Related-Bug: #1616913
* Merge "Trivial: Remove redundant variable"Jenkins2016-11-031-8/+8
|\
| * Trivial: Remove redundant variableguo yunxian2016-09-281-8/+8
| | | | | | | | | | | | | | | | | | As I5b077f403e22ce1ff6833bc397afd21bf4d3e498, we can simply do self.***['id'] instead of create a new variable. But some other files are also the case. This patch changes these files to make the code consistent. Change-Id: Id317f81362a2c066994cc3385c2401494382de6e
* | Test case with list-instance-actions for deleted serverAnna Babich2016-10-051-3/+27
|/ | | | | | | | | | | | | In accordance with [1], we can use the 'nova instance-action-list' command to list actions on a deleted server. To do this, UUID of a server must be used, and API microversions ‘2.21’ - ‘2.latest’ must be supported [2]. [1] http://docs.openstack.org/cli-reference/nova.html [2] http://developer.openstack.org/api-ref-compute-v2.1.html #os-instance-actions-v2.1 Change-Id: I3b7d99321bb7f931e45b15b5c5d1ec3a7b7b7169
* Make server action methods[m-z] use **kwargsKen'ichi Ohmichi2015-12-151-1/+1
| | | | | | | | | | | As we discussed on http://lists.openstack.org/pipermail/openstack-dev/2015-July/068864.html All http POST/PUT methods need to contain **kwargs as their arguments. This patch makes server action methods[m-z] use **kwargs. Partially implements blueprint consistent-service-method-names Change-Id: I7174723f907fd8962d9d12fe76bec2dc06883938
* Fix T110 violations for servers_clientKen'ichi Ohmichi2015-11-201-1/+1
| | | | | | | servers_client contained T110 violations, and we need to fix them before migrating the client to tempest-lib for consistent interfaces. Change-Id: I10a8c32b53f15063dab087d67123598800b141a0
* Return complete response from servers_clientghanshyam2015-09-111-3/+4
| | | | | | | | | | | | | | | | | | | Currently compute servers_client returns Response by removing top key from Response. For example- return service_client.ResponseBody(resp, body['server']) As service clients are in direction to move to Tempest-lib, all service clients should return Response without any truncation. One good example is Resource pagination links which are lost with current way of return value. Resource pagination links are present in parallel (not inside) to top key of Response. This patch makes compute servers_client to return complete Response body. Change-Id: I6a7b5c6144cafcb8f0780db01bf67999a4229389 Implements: blueprint method-return-value-and-move-service-clients-to-lib
* Add "server" to action methodsKen'ichi Ohmichi2015-08-131-1/+1
| | | | | | | | | | As the qa-spec of consistent-service-method-names, basically all methods’ names should be "<verb>_<resource/object name>". So this patch adds the word "server" to compute action methods. Partially implements blueprint consistent-service-method-names Change-Id: Ib91dc75055b9e2331d243cac686fe19c7516f6ba
* Remove wait_for_server_status from servers_clientKen'ichi Ohmichi2015-07-131-1/+2
| | | | | | | | | | | | | wait_for_server_status of compute client just calls the one of waiters module and the method is a wrapper. So the method seems redundant and it doesn't fit as service clients. This patch removes wait_for_server_status from compute client and replaces it with waiters' one. Partially implements blueprint consistent-service-method-names Change-Id: Ie821ea4ba0924b71caba80132e1c383fb5bb125d
* remove gate tag (part 1)Sean Dague2015-04-271-2/+0
| | | | | | | | This removes all bare gate attribute tags from functions, the gate tag was a never fully implemented idea in the past, and it's existence in the code mostly just confuses people. Change-Id: I462ca7a8cadeb5013e2bce44ed1e69bcf8f9d968
* Split resource_setup for compute testsRohan Kanade2015-03-091-1/+5
| | | | | | | | | Split up the resource_setup method for all the compute tests, as per the latest spec. Partially-implements bp:resource-cleanup Change-Id: I6562534ca5eb5e33c6f8d6cbcac5e7735535916a
* Add UUIDs to all tempest tests and gate checkChris Hoge2015-02-261-0/+2
| | | | | | | | | | | | | Adds uuid4 as a decorator of the form: @test.idempotent_id('12345678-1234-1234-1234-123456789abc') to every test in the Tempest tree. Includes a gate check to ensure the existence and uniqueness of the ids. Modify check tool to ignore Tempest unit tests. Change-Id: I19e3c7dd555a3ea09d585fb9091c357a300e6559 Co-Authored-By: Sergey Slipushenko <sslypushenko@mirantis.com> Implements: bp test-uuid
* Change servers client to return one value and update testsDavid Kranz2015-02-171-6/+4
| | | | | | | | | get_console_output and rescue_server are a little different but I did not change them fundamentally in this commit. Partially implements: blueprint clients-return-one-value Change-Id: I44026cac62084b868eb47aedb43c17d38ac8e43b
* Change basic server client methods to return one value and update testsDavid Kranz2015-02-121-2/+2
| | | | | | | | | This includes create_server, update_server, get_server, delete_server and also some helper methods such as create_test_server Partially implements: blueprint clients-return-one-value Change-Id: I8e1fba7a31a0ed27e683dacd9a8e763759552c11
* delete all compute xml testsSean Dague2014-11-251-4/+0
| | | | | | part of operation drop xml Change-Id: I00f1ea08a5e9dbda6e6fe765558100106026a931
* Migrate computev2 API tests to resource_* fixturesAndrea Frittoli2014-09-181-2/+2
| | | | | | Partially-implements bp:resource-cleanup Change-Id: I59eb2d9789368c812282ce12ae7e571be720badb
* Cleanup _interface class variables in computeMasayuki Igawa2014-03-051-1/+0
| | | | | | | | | We've already removed these variables in Nova v3 tests. Change-Id: Ie9ad2242d20ce90fca74ca7a5390e0971f830caa But we can remove these not only v3 tests but also v2 tests. This commit sets _interface = 'json' to BaseV2ComputeTest and removes redundancy. Change-Id: I10110b92ed9a34eedef210ad8408f78d64ba4ee6
* Remove vim headersMasayuki Igawa2014-01-201-2/+0
| | | | | | | | | | | | We don't need to have the vim headers in each source files. We can set it in our vimrc file instead. Also if set incorrectly, gate (pep8 check) will catch the mistakes. More discussions: http://lists.openstack.org/pipermail/openstack-dev/2013-October/017353.html Change-Id: I43703e2289212389c7841f44691ae7849ed1f505 Closes-Bug: #1229324
* Moves negative tests from test_instance_actionsHoisaleshwara Madan V S2014-01-081-16/+3
| | | | | | | | | | | | | | | This change set moves the negative tests in tempest/api/compute/servers/test_instance_actions to a new file test_instance_actions_negative Also removes usage of modules directly. Note: HACKING.rst The second requirement is that all negative tests must be added to a negative test file. If such a file doesn't exist for the particular resource being tested a new test file should be added Change-Id: I6fd392e3fca84f7daeb3769ff59a8d5dadb900c3
* Merge "Rename to create_test_server in API tests"Jenkins2013-11-121-1/+1
|\
| * Rename to create_test_server in API testsKen'ichi Ohmichi2013-11-101-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | There are two methods of the name create_server(). The one is a client which just sends a request for creating a server instance. The other is BaseComputeTest class' which sends the same request, registers the created server to the list for deleting it after test, and waits for the server status change. They would be confusing due to the same name, and there is unnecessary addCleanup() calls for deleting a server. This patch renames the one of BaseComputeTest to create_test_server for making the difference clear, and removes unnecessary addCleanup() call. Change-Id: If09fc09f77d27a48dc19dd2dbc47ea460b0c7920
* | assertTrue to assertInAttila Fazekas2013-10-251-1/+1
|/ | | | | | | | * replacing the assertTrue where simply replaceable with assertIn * The assertTrue(len(foo)>1) style extended with str(foo) details, where expected len is greater than 1. Change-Id: I7515e879e3e24ce041a4fcae63d9d4a5f257f37d
* add BaseV2ComputeTest as the base class of nova v2 api testsivan-zhu2013-10-181-1/+1
| | | | | | | | | | | | this does refactor in compute.base, add BaseV2ComputeTest as the base class of nova v2 api test. Because there will be nova v3 tests, this change will be convenient to add BaseV3ComputeTest as the base class of nova v3 api tests. In this way, we can split v2 tests with v3 tests easily, and share some common method. Partially implements blueprint nova-v3-api-tests Change-Id: Ie8e0d17a95e8378733dd21ab341dfad1331edf6e
* Removes 'positive' tag from tests.Giulio Fidente2013-05-291-2/+2
| | | | | | | | | | | | | | Also removes brackets where tag is a single value. This should be finally cleanup the tags, see the following: $ grep -r "@attr" tempest | cut -d: -f2 | sort -u @attr(type='gate') @attr(type=['negative', 'gate']) @attr(type='smoke') $ Change-Id: I243800c597996c0dbcf153d66946062df6d298ab Implements: blueprint set-gate-attribute
* Set smoke/gate attributes for tests in "compute"Giampaolo Lauria2013-05-231-4/+4
| | | | | Change-Id: I3f6d496bd88b10f780b168caeb4b62d9138ae8e4 Implements: blueprint set-gate-attribute
* rename tests -> apiSean Dague2013-05-201-0/+69
Now that all the other tests are moved out of the tests directory we can rename tests -> api to reflect that these tests are api testing, and need to use only the internal clients. Clean up references from other parts of OpenStack to the new api namespace. Reorder the imports as required with this naming change. Added README.rst Change-Id: I19203957f917b59e7c8a3838c590937752461a2f
View Lorry Controller Status