The set_time_override function in timeutils was written as a
helper function to mock utcnow for unittests before 'mock' was
generally used. Now that we have mock and fixture, we no longer
need to use it.
Change-Id: I809825560b0324498010bd93aa1ceef552554375
Partial-Bug: #1266962
The coverage extension has been removed from Nova V2 API by the
I07d798129ee277a6f7691c25f88c07a5204c0943
This commit remove the code.
Change-Id: I430a8b17be11bb961dc5b89b2d098f373e0a27fa
In the shell command when we use the command boot, at the end
we use code to print some server information.
We should use the already defined function _print_server.
Change-Id: I133d7706458c71974ba464ab686cf3679a562592
Closes-Bug: #1265534
Print hyphen(-) in python client output when attribute value is not
set(NoneType) so that it is distinguishable between "None" string
and NoneType value.
Change-Id: I5ec0f39f13aca45fa95ddb59472c20207c63bf91
Closes-Bug: #1266414
Nova api supported a parameter named 'deleted' which allow Admin to list
deleted servers. This patch make novaclient support this.
Change-Id: Ifc492fd0ba9d885e37c04c165d6397832ae6ebe0
Closes-Bug: #1266631
This patch implements a blueprint to add more consistency into
nova command, especially for the subcommand "floating-ip-*"
Currently when we want to associate or disassociate an ip with
nova we have to use add-floating-ip and remove-floating-ip. This
is not consitent with the actual scheme of the subcommands:
example:
nova image-*
nova flavor-*
nova floating-ip-*
+ In the client v1.1 this patch displays a deprecated message when printing
the help message for add-floating-ip and remove-floating-ip.
+ In the client v3 this patch do nothing because all floating ip commands are being
removed.
$:~/python-novaclient$ nova help | grep floating
add-floating-ip DEPRECATED, use floating-ip-associate instead.
floating-ip-associate
Associate a floating IP address to a server.
floating-ip-bulk-create
Bulk create floating ips by range.
floating-ip-bulk-delete
Bulk delete floating ips by range.
floating-ip-bulk-list
List all floating ips.
floating-ip-create Allocate a floating IP for the current tenant.
floating-ip-delete De-allocate a floating IP.
floating-ip-disassociate
Remove a floating IP address from a server.
floating-ip-list List floating ips for this tenant.
floating-ip-pool-list
List all floating ip pools.
remove-floating-ip DEPRECATED, use floating-ip-disassociate instead.
Implements: blueprint commands-floating-ip
Change-Id: I5337d0f1ce5ec4826da6ecd2b6ae4ae7b97801e0
- Removes vim headers: It's not needed to set tabstop tons of times,
this can be done by setting vimrc.
- I did not update files in common/* and install_venv_common.py because
these files are sync with oslo.
Note:
http://lists.openstack.org/pipermail/openstack-dev/2013-October/017353.html
Closes-Bug: #1265474
Change-Id: Ia09dc2c908187a756bf55eaba74655484304517d
Adds the ability to do a detailed query from the
os-quota-sets V3 API extension. This was an interface
added in V3 to allow for the querying of current quota
usage. Adds a shell command quota-usage which
uses this interface to show current quota usage as well
as the limit.
The absolute limits response was removed from
the limits extension in a9e29b7e9fbe14ab42f24802a6d2a457a6317ba3
and so the corresponding novaclient command is also removed.
The "quota-show-usage" command now shows the same information.
Differences between the V2 and V3 API are described here:
https://wiki.openstack.org/wiki/NovaAPIv2tov3
Partially implements blueprint v3-api
Change-Id: I5db72d42120f4ad7a86fbfce20382988f6bbf5d3
--flags were not tab-completing under OS X. This was due to the whitespace
collapsing regular expression using the '+' token (meaning one-or-more), which
isn't supported in OS X's version of sed.
Using the more portable ' *' instead of '[ ]+' fixes this.
Additional cleanups
* Remove unecessary embedding of tabs in the file (nova bash-complete doesn't
emit tabs, so the regular expressions don't have to handle them)
* Restore logic to exclude -h from tab-completion (was lost in last
tab-completion cleanup)
* Add similar exclude logic for -i
Fixes bug 1266667
Change-Id: I7e1fe8382d9b5295d0bbc1cde2b89550d5a4e21c
class_name parameter is not used in both Nova v1.1/v3 API.
Nova considers the part of url as class_name.
For example, bar is considered as class_name in the following case:
curl -i 'http://localhost:8774/v3/os-quota-class-sets/bar' -X PUT ..
This patch removes this unused paramter from quota_class.
Change-Id: Id4eba5b2a17506bc04ea23ef1097ef9cdb9caf8a
In the case when the value is a dictionary or array or longer than
the line then the diagnostics output will automatically wrap around
to ensure that the output is user friendly.
Part of the blueprint v3-diagnostics
Change-Id: Ia158fd99aeb0e6296fb232e881d0f01a13407dfc
Closes-bug: #1266402
The bash completion code is the sole user of the slugify() function
in utils, which is substantially similar to to_slug() provided in
strutils from oslo.
Change-Id: Ib4eb7e4c0fa0e9bc5c4a0856f6391911d8f8fd3b
Closes-bug: #1266118
Directory doc is excluded from pep8 check in tox.ini, since we
modified file doc/source/config.py, we should enable the check.
Change-Id: I057f1e6cb1c5f9a6a07043056078e4475f66f288
Print the dictionaries and arrays without the unicode tags.
The patch also updated tests that did not return valid data.
Change-Id: Ia787f98a9510b68beb3ceaf00c285ca5c934f5c0
Closes-bug: #1265002
Instead of using novaclient.utils' bool_from_str, use bool_from_string
from the oslo incubating strutils
Notes:
0. utils.bool_from_str was strict, only accepted a small set of
values, while strutils.bool_from_string is not strict by default,
anything not true is false
1. bool_from_string accepts on/off, which bool_from_str did not
Change-Id: I04744844b55697819289def081d3c9117ed0713f
If a user runs 'nova service-list --binary something-weird', where
something-weird is a binary that is not listed the result is a confusing
IndexError. That is because the result is expected to always have at
least one result.
Part of the reason for that is that the fake client does not make it
easy to simulate an empty response in tests. That is why there is no
regression test associated with this bug-fix. That is also why I am
marking thas as Partial-Bug, as this corrects the symptom, but it does
not prevent it from regressing.
Change-Id: I3e6798d63a662c245196b3bf95bb7e1b0ea9c3a4
Partial-Bug: #1260504
'nova aggregate-details' shows Hosts, Availability Zone
and Metadata in the python unicode notation.
+ Adds some tests about aggregates ported from v1 to the
v3 shell
Change-Id: I69d80b2e76833d78248dee782ae5e53f42a4f4a9
Closes-Bug: #1132961
