Read-only property 'os' has moved to 'os_primary' in version 'Pike'
and will be removed in version 'Queens', so this is to replace
the usage of 'os' with 'os_primary'.
Change-Id: I9f7c13da05a8c4a63529c11aa6213a7269abee6d
In ServersNegativeTestJSON, two testcases need deleted
server, so we can create it in resource_setup to reduce
the total run time.
Change-Id: I5ef7188318cc2907055ce31fd77345df8d3e791f
When use floating as connect_method, tests:
test_attach_detach_volume[id-52e9045a-e90d-4c0d-9087-79d657faffff]
test_list_get_volume_attachments[id-7fa563fe-f0f7-43eb-9e22-a1ece036b513]
will fail due to that they are under the same class, so when these tests
setup, and using floating as connect_method only one floating IP was
created.We first associated it to instance created at test1 and then we
created another instance in test2 and tried to associate this IP again to
the new server, since the IP was already associated to instance created in
test1, error will happen when try to do this.
This patch fix this by delete the servers after first test finished.
Closes-Bug: #1686664
Change-Id: Ie3afb31bd0d3245760f799b2256228159b328619
This patch is to replace assertEqual([], items) with assertEmpty(items),
which the items is a empty list.
Change-Id: Ibdd1e141740497b819e08ac8abeb74740b8a6cf9
To make endpoint_type default values consistent across services,
keystone v3_endpoint_type default value is deprecated in Pike and
will be changed to publicURL in next release
Change-Id: Iacdfb61e718b8ae45eab25e8041829989486a2ed
Closes-Bug: #1687141
This is to add check for flavor_extra_specs disappearing
after unset_flavor_extra_spec, now no any such check in
Tempest.
Change-Id: I8b749bdbf4e1e34cec2b37f25ec734a5c99422d8
When a transferred volume is accepted by another user in different
tenant, the user and tenant of the volume should be changed. This
patch is to add explicit assertions for these two attributes in
volume transfer test.
Change-Id: Iaaee981c5d0c17133cc426479904c584e28f8578
test_snapshot_create_with_volume_in_use can be mostly
covered in test_snapshot_delete_with_volume_in_use, except
"Snapshot a volume which attached to an instance with
force=False", so this is to move that test step to
test_snapshot_delete_with_volume_in_use and remove
test_snapshot_create_with_volume_in_use, so the total
run time can be reduced.
These two testcases are not used in the newest defcore
json files.
Change-Id: Ib04fe2d2127681cd89bfea3cc2d09f4aafbc6f34
Server rescue/unrescue is time consuming,
test_rescued_vm_associate_dissociate_floating_ip [23.838659s]
test_rescued_vm_add_remove_security_group [21.670585s]
This is to:
1. create a rescued server in resource_setup to be shared
between two testcases
2. delete server in RESCUE status directly without unrescue it
in resource_cleanup
3. move setup of floating ip and security groups from resource_setup
to inside the testcases because they are used only once
Change-Id: Ic47dc1fa1bcb6ae7f43a6f37565971c94e5494af
This commit updates the document theme. It is clear, simple, up-to-date
and beautiful. And this commit also updates section headers to fix the
document index.
Change-Id: Ia1e8e16e5f5ddb06e4c4ccb4979b0750b88a4db4
In ServersNegativeTestMultiTenantJSON's setUp, self.client
should be self.servers_client, otherwise we will get:
<AttributeError: "'ServersNegativeTestMultiTenantJSON' object
has no attribute 'client'">, and so we will encounter twice
unnecessary rebuild_server.
Change-Id: Ie5559bd4f1cbbb0b128946c21bd737585f79ffad
Show volume summary is a new api implemented in volume v3. This patch
is to add a simple test case for this new feature.
Including:
[1] Add v3 volumes_client as library
[2] Add show volume summary api to v3 volumes_client
[3] Add unit tests for show volume summary api
[4] Add test case: test_show_volume_summary
[5] Add release note
Change-Id: I9904bc53896e7d58a22aef11f6a3128147c0a373
This test is still failing at a really high rate:
http://status.openstack.org/elastic-recheck/#1687044
And that's just the jobs where elastic-recheck didn't time out
waiting to get logs, so the actual failure rate is probably higher.
Let's skip it for now.
Change-Id: I49cfa97c5a2a8d30514adb59d9ad3ae35a6ade37
Related-Bug: #1687044
In ServersOnMultiNodesTest, server01 can be safely shared among
testcases, so we can save two create_test_server.
Change-Id: I2b9165a26feeb5ce5fa7442872b2212d22d7f575
base_routers just contained some methods for test_routers only.
This patch moves such methods into test_routers directly and removes
base_routers module for simple inheritance.
Change-Id: I855a345ea6331103901c512d790ed802674c8fcc
test_routers_dvr doesn't use BaseRouterTest specific methods at all,
so it is not necessary to inherit from BaseRouterTest. This patch
removes such unnecessary dependency for simple inheritance.
Change-Id: Ic9d741f64128e4532c2206a57954c7d1958bacef
The moved methods are used at test_metering_extensions only, so the
base module is not a good place to keep such methods. This patch moves
them into test_metering_extensions for keeping the base method simple.
Change-Id: I84c36cc3597b53953b4fb9fc7c42932682d44795
In AggregatesAdminTestJSON, the multi lines of creating
aggregate repeated for many times, so this is to use a
helper to create aggregate, so to make the code focus on
main test steps.
Change-Id: I3625fa9073ebe09a43e9ebeb5a117890f9385189
Most tests of RoutersNegativeTest don't require admin credential,
test_router_set_gateway_used_ip_returns_409 only requires it.
So this patch makes RoutersNegativeTest inherit from BaseNetworkTest
which is based on non-admin credential. In addition, this patch makes
a test stop using _add_router_interface_with_subnet_id() on the test
class for simple code and removing the dependency to BaseRouterTest.
NOTE: test_router_set_gateway_used_ip_returns_409 is not used on
defcore side. So this path change doesn't affect on defcore.
Change-Id: I8bf2d523e52353989be8f83b06c0ce6718931e8c
_remove_router_interface_with_port_id() is just a wrapper method
which calls remove_router_interface() with an assertion.
The assertion seems unnecessary because it is called on the cleanup
and we don't do such check on the other places.
This patch just replaces the redunadnt method with direct call.
Change-Id: I82fd96be1f27dc9f9e58b5eeaa562ef3404f3382
