Currently neutron client is written to auto generate
methods on fly. This is very difficult to debug and
make code very difficult to read.
As neutron client will be moved to lib, it will be better to
have its interfaces more readable and consistent with other
service clients.
This commit re-factor it for 'create_*' methods.
Change-Id: I77cec3cdb1bdf404fab6083c1417368bc8fb6cf5
[GET /resources] methods should be "list_<resource name>s"
or "show_<resource name>", so this patch applies the rule
to compute clients which names are "[i-k]*".
Partially implements blueprint consistent-service-method-names
Change-Id: Ibf88309d7b013e805ced76b4ec88c22240e3ec50
Ic2e5bd127c9f3a0f2807a25f29d8693baf3d65f2 of oslo.log has changed
the description of use_syslog, but current tempest.conf.sample does
not contain this change. As the result, the pep8 test continuously
fails on the gate now.
This patch fixes this description.
In addition, this patch adds fake body message to NotFound in
javelin test for avoiding the gate problem.
These gate problems happened at the same time, so this patch contains
mutiple purposes but that is necessary for fixing at the same time.
Closes-Bug: #1468141
Closes-Bug: #1468149
Change-Id: Iada215718651dc153b977008b376a956247ebc5d
Through consistent-service-method-names dev, list_<resources>_detail
methods are merged into list_<resources> method. But I forgot to do
it on I3fdfa1101b966015798a61aa6ba5acfdf4649831 .
This patch does it for consistent methods.
Partially implements blueprint consistent-service-method-names
Change-Id: Icb3404f18c24ff3a0ec1f34fde4f65811dce22a0
[GET /resources] methods should be "list_<resource name>s"
or "show_<resource name>", so this patch applies the rule
to compute clients which names are "q*".
In addition, this patch removes unnecessary str() because
all callers specify string values and we need to pass py3
tests when migrating them into tempest-lib.
Partially implements blueprint consistent-service-method-names
Change-Id: Ifbb0a0bea4bfd1e94a054e1eafcc75e443ead1bb
Now Nova team is allowing additional properties with microversions
only, that means Nova v2 and v2.1 should not contain additional
properties forever. In addition, we should not allow vender-specific
properties for OpenStack interoperability.
This patch disables additionalProperties for blocking such properties
for v2 and v2.1 API.
Change-Id: Ica929c402a3d042ae751302384e68853eb28b405
Our remote ssh code makes all kinds of assumptions about the shell and
path of the guest. Recently some code was added to the command to generate
errors more promptly but it does not work if the shell is /bin/sh. This
commit allows the user to configure tempest with a desired prologue to handle
this case. It does nothing to solve the more general problem and, like the
other image-related options, cannot be customized individually for various
images that tempest may use.
Debug logging of remote commands was also added.
Change-Id: I669dd4386ffb539dad88a9487bef6c172b5d65fa
Closes-Bug: #1465682
This change adds a common create_test_server function including the new ssh
configuration parameters.
Partially implements: blueprint ssh-auth-strategy
Change-Id: Idb8a9b1fd9d933ab1fe7c99f7cd4dea41b22038a
cls.is_dvr_router attribute is invalid in the case which result of the
is_extension_enabled() called in resource_setup() is false.
This patch fixes it.
Closes-bug: #1465623
Change-Id: If3e9f80991b2675b4700c7cdc80b5689b444d40b
data_utils.rand_name() appends randam charactors with a hypen like:
def rand_name(name=''):
randbits = str(random.randint(1, 0x7fffffff))
if name:
return name + '-' + randbits
So it is not necessary to specify a hypen in caller side.
This patch adds a hacking rule for blocking "-" at the end of argument
of rand_name() calls.
Change-Id: I9b9f25dbe5a3ef5ac5900113bcc46e0d911becc9
introduced in Id049b6cb8ab3092c50f35894cf5a6bda7ff04617 and
disabled by I763212bad849980bea7fdb96384a5fba7889cce7
improvements to sql backend were made in Juno (see bug)
this patch re-enables the tests and makes a modification to the
expected list of meters (some of original meters are deprecated)
Change-Id: Ia87c120c50d8edf55da3d4b9d748c0680e25a97b
Related-Bug: #1351627
There is a bug https://bugs.launchpad.net/tempest/+bug/1452915
that describes failing test_get_object_after_expiry_time test
because in real clouds hardcoded 3 seconds waiting for the
container is not enough. Replaced hardcoded 3 seconds with the
checking and waiting in a loop for object to be deleted.
Change-Id: If90d9ef047344e2e094aa1b143c93c9fa4b56fbd
Closes-Bug: #1452915
limits client should be just for getting a resource "limits",
it is not necessary to specify key 'absolute' at all as the library
method.
In addition, this patch removes get_specific_absolute_limit because
the method was just wrapper. That should be implemented on test side.
Change-Id: I5c0ffc136939d0aeb8574c180353715f2a0e2665
Improving test coverage of Javelin by covering more resources:
* routers: create/destroy
* secgroups: create/destroy
* subnets: destroy
Change-Id: Iccc827c6a50ea79de26cc0e8e00d83fe2498c858
[GET /resources] methods should be "list_<resource name>s"
or "show_<resource name>", so this patch applies the rule
to compute clients which name is "networks".
In addition, this patch moves "name" filter of list_networks
to caller side because this filter is used at one place.
Partially implements blueprint consistent-service-method-names
Change-Id: Ic2f7b8a2724035f0b2cd9def13cd7f7fc1e9924d
Each resource should be implemented in each separated module.
quota_classes and quotas are different resources like
os-quota-class-sets
os-quota-sets
So this patch separates quota_classes client from quotas client.
Partially implements blueprint consistent-service-method-names
Change-Id: Icd98294c7a161f5c8c1d7202df9f3e6d9daee1d9
list_networks() of compute network client is implemented like:
if name:
networks = [n for n in body['networks'] if n['label'] == name]
So if the network of the specified name don't exist, the return value
is an empty dict. lib_exc.NotFound never happens.
This patch removes lib_exc.NotFound handling.
Change-Id: I8a48d31b6e9b932729d2156cf08669a1970d0b5b
[GET /resources] methods should be "list_<resource name>s"
or "show_<resource name>", so this patch applies the rule
to compute clients which name is "limits".
Partially implements blueprint consistent-service-method-names
Change-Id: Ibe30d01057936cea68f38984064dff107cdcbe5c
[GET /resources] methods should be "list_<resource name>s"
or "show_<resource name>", so this patch applies the rule
to compute clients which names are "h*".
Partially implements blueprint consistent-service-method-names
Change-Id: Id2346a6c65cd931b8876d6597fbb82361d1f5839
- lbaas tests are now in neutron-lbaas
I2790239b8cd361bc4ac7905b4118f34648b97840
Ie4679b7131f29f33a18db4b5070add50563a0af9
I77a9038b81f01f9d85f20d4e2ad5cbe182a86da3
- fw/vpn tests are now in neutron api tests
Ie90671fbfe2f633e851da82728e152482133fd87
Change-Id: I3a8660838c1af2baefe21bbc57b006c9889ac35a