We have duplicated bulk ops code on tempest.lib and network_client
module. So this patch removes these methods from the module and
makes tests use tempest.lib ones for the code cleanup.
Change-Id: I685be88bbd47837d99b49938cfaa66ff2e199cb3
Now service_client is unnecessary because that is the same as
rest_client. This patch makes network clients use the rest_client,
and we will remove service_client for the cleanup in the future.
This patch removes these clients from a certain test because this
test is just for service_client, not rest_client as the test name.
We have added this test for verifying all arguments are passed into
rest_client via service_client. However, this patch some clients use
rest_client directly, so this test becomes meaningless for these
clients. That is a reason why the migrated clients(into tempest.lib
before) are not contained in this test.
Partially implements blueprint consistent-service-method-names
Change-Id: I1922d0937162827f5c720bf3d331f9c7695506d2
Now service_client is unnecessary because that is the same as
rest_client. This patch makes identity clients use the rest_client,
and we will remove service_client for the cleanup in the future.
This patch removes these clients from a certain test because this
test is just for service_client, not rest_client as the test name.
We have added this test for verifying all arguments are passed into
rest_client via service_client. However, this patch some clients use
rest_client directly, so this test becomes meaningless for these
clients. That is a reason why the migrated clients(into tempest.lib
before) are not contained in this test.
Partially implements blueprint consistent-service-method-names
Change-Id: I5de906bcc0e32063ff17ee75f8dcf91b99165d03
Now service_client is unnecessary because that is the same as
rest_client. This patch makes image clients use the rest_client,
and we will remove service_client for the cleanup in the future.
This patch removes these clients from a certain test because this
test is just for service_client, not rest_client as the test name.
We have added this test for verifying all arguments are passed into
rest_client via service_client. However, this patch some clients use
rest_client directly, so this test becomes meaningless for these
clients. That is a reason why the migrated clients(into tempest.lib
before) are not contained in this test.
NOTE: This patch changes __init__() of images_client to use
**kwargs because service_client removed them if each value
is None, but rest_client doesn't do it. The purpose of the
__init__() is just for calling self.skip_path(), so this
change is more readable also.
Partially implements blueprint consistent-service-method-names
Change-Id: I6655b31239d4f3d0c4601b875a4a26778fc9dd2d
Now service_client is unnecessary because that is the same as
rest_client. This patch makes volume clients use the rest_client,
and we will remove service_client for the cleanup in the future.
This patch removes these clients from a certain test because this
test is just for service_client, not rest_client as the test name.
We have added this test for verifying all arguments are passed into
rest_client via service_client. However, this patch some clients use
rest_client directly, so this test becomes meaningless for these
clients. That is a reason why the migrated clients(into tempest.lib
before) are not contained in this test.
Partially implements blueprint consistent-service-method-names
Change-Id: I1f7d8bfdb23500342984508333321422f8e2366b
Now service_client is unnecessary because that is the same as
rest_client. This patch makes database and telemetry clients
use the rest_client, and we will remove service_client for the
cleanup in the future.
This patch removes these clients from a certain test because
this test is just for service_client, not rest_client as the
test name. We have added this test for verifying all arguments
are passed into rest_client via service_client. However, this
patch some clients use rest_client directly, so this test becomes
meaningless for these clients. That is a reason why the migrated
clients(into tempest.lib before) are not contained in this test.
NOTE: This patch changes __init__() of versions_client to use
**kwargs because service_client removed them if each value
is None, but rest_client doesn't do it. The purpose of the
__init__() is just for calling self.skip_path(), so this
change is more readable also.
Partially implements blueprint consistent-service-method-names
Change-Id: Ie99bbbe7655138c4c3ecae69ae0b361198a9651d
Similar to what Cinder did here [1], this patch mocks time.sleep
to make tests run faster. Some code actually measure the wall clock
to wait for a specific duration before raising a TimeoutError, so
we also need to mock time.time.
This removes ~5sec in unit test and also removes some busy waiting.
[1] https://review.openstack.org/#/c/285658/
Change-Id: I69ba35eff591a5df28049273f3aba15c31f52d00
When EndpointNotFound exception happens, the detail contains service
value only, but the exception depends on the other values also.
So this patch adds these value to EndpointNotFound exception for
more easy debugging.
Change-Id: I05095e6ce56bc62485e964241047fd3b3864f181
Now service_client is unnecessary because that is the same as
rest_client. This patch makes object_storage and orchestration
clients use the rest_client, and we will remove service_client
for the cleanup in the future.
This patch removes these clients from a certain test because
this test is just for service_client, not rest_client as the
test name. We have added this test for verifying all arguments
are passed into rest_client via service_client. However, this
patch some clients use rest_client directly, so this test becomes
meaningless for these clients. That is a reason why the migrated
clients(into tempest.lib before) are not contained in this test.
Partially implements blueprint consistent-service-method-names
Change-Id: I763107de6a9f7b411a5892d2c53dcf99b7afa3ef
This is a pattern we've had in Nova for a while that allows individual
test classes to be marked as slower. This timeout adjustment is needed
for test_volume_boot_pattern because going up and down with 3 guests,
and cleaning them all up, plus volumes and snapshots, takes
time. Especially on slower personal equipment. These fail for people a
lot, which makes them think their devstack is broken. And it means
that it leaves crud around when it times out, which ruins follow on
runs.
Change-Id: Ibf21bdf44339d00f0291d0ff14eb6bcc69798ccc
If multi-node is enabled, tests traffic and secgroup across multiple
compute nodes. Uses different_host as scheduler_hints to distribute
servers
Fixes call to secgroup-rule creation to use proper tenant.
Change-Id: I6c2d04656beb7fd04e365cae01293561a8c0446f
These tests fail a lot, more than nearly any other tests we have. When
reading the REST logs it's a bit hard to figure out what we were
conceptually doing because we repeat many of the same actions over and
over again.
This adds some LOG.info to provide a road map when looking at logs.
Change-Id: Ic9e3bf9b5720d6f250c9b003357ba4bc5fdebbe3
This patch deletes skip decorator used in test_get_service_by_host_name
for making only one-way dependencies. This patch fixes this issue.
Related-Bug: #1530144
Change-Id: Id72863c4a4fa77aff42a7e69e7e8be8080166ae5
Depends-On: I21775106693176ca128dbfd9db0d43cfc58de00a
Now service_client is unnecessary because that is the same as
rest_client. This patch makes keypairs_client use the rest_client,
and we will remove service_client for the cleanup in the future.
Partially implements blueprint consistent-service-method-names
Change-Id: I250c7a4de9585dd299cddd515e141635095d1a5a
Now service_client is unnecessary because that is the same as
rest_client. This patch makes both data_processing_client and
baremetal_client use the rest_client, and we will remove
service_client for the cleanup in the future.
This patch removes these clients from a certain test because
this test is just for service_client, not rest_client as the
test name. We have added this test for verifying all arguments
are passed into rest_client via service_client. However, this
patch some clients use rest_client directly, so this test becomes
meaningless for these clients. That is a reason why the migrated
clients(into tempest.lib before) are not contained in this test.
Partially implements blueprint consistent-service-method-names
Change-Id: Icf85bcde4c0d1ee644c950323f4c6abe3b86a24e
The upcoming tempest release will include the stable library interface
this commit adds a release note for that and all the required supported
openstack releases boilerplate.
Change-Id: I87105c647f6cc621bbdd77ff1c44457dd1573276
The current code that cleans-up network resouces from the dynamic
credentials provider does not account for failures during the
cleanup itself (apart from 404).
When the cleanup is triggered by a failure during the setup, the
original failure is hidden which makes debugging impossible.
Catching all exceptions on cleanup, logging a warning and
re-rasing the original exception should fix that.
Fixes bug 1551264
Change-Id: I49dbb314a09fd9b49c9604bc8b5b8ed03ff218d4
This commit adds documentation for the library interface, formerly
known as tempest-lib, and the release versioning.
Partially implements bp tempest-lib-reintegration
Change-Id: I623222a17fe3563bbe767fa803e8f771a21687a1
When calling get_token() there are many assumptions on the
domain when user_domain_id or user_domain_name and
project_domain_id or project_domain_name are not specified.
This is okay when running tempest against devstack, since
the user is in the default domain but in productions the
test user is rarely in the default domain. Instead we should
also pass user_domain_id or user_domain_name and
project_domain_id or project_domain_name to
get_token to create a token.
If there is a case where user_domain_id is None when passed
to get_token() the previous functionality still takes into effect.
Assuming the user is in the 'Default' domain.
Change-Id: I78c472f2284a8641c0c7cf0b3e0994984a04c5b0
Closes-Bug: #1548987