52 Commits

Author SHA1 Message Date
Jenkins
27700f9b36 Merge "Allow name argument to flavor-access-add" 2013-08-23 01:33:06 +00:00
Yuuichi Fujioka
476e50f4d7 Allow name argument to flavor-access-add
Administrator cannot use the flavor name in arguments of
"nova flavor-access-add" and "nova flavor-access-remove"
in following conditions.

1. The flavor is non-public.
2. The falvor doesn't belong administrator's tenant.

This patch fixes this problem.

Change-Id: Ic5e3222d0ff62667e003a61e6a94e85833da5d11
Fixes: bug #1205298
2013-08-15 03:06:51 +09:00
Vincent Hou
852e2e5f31 Check whether the security group id is integer
Fixed Bug 1206847.

Change-Id: I2194d6538a5c5475b876ee026b5d625be2df076f
2013-08-01 09:59:29 -04:00
Dirk Mueller
af7ca70e3e Fix and enable gating on H402
End one-line docstrings with punctuation.
Change them to command style where necessary.

Change-Id: I8ff689c3a2f20d489286f80112c6dc95c97f2f31
2013-07-17 15:44:05 +02:00
Jenkins
a45c49bafc Merge "python3: Fix unicode compatibility python2/python3" 2013-07-10 04:09:58 +00:00
Chuck Short
a25d4fe59a python3: Compatibility for iteritems differences
In python3 dict.iteritems(), dict.iterkeys(), and
dict.itervalues() are no longer supported. So use
six.iteritems() where it is appropriate.

Change-Id: I8b07dc2a89d790ec275d45f859e1644e9b00c837
Signed-off-by: Chuck Short <chuck.short@canonical.com>
2013-06-24 11:51:04 -05:00
Chuck Short
8c4e145b92 python3: Fix unicode compatibility python2/python3
Python3 enforces the distinction between byte strings and text strings
more rigorously than python2. So use six.text_type/six.u()
where appropriate

Change-Id: I890e19cb857e10f0292aabdaebaa8e7a7bd8db23
Signed-off-by: Chuck Short <chuck.short@canonical.com>
2013-06-24 11:47:15 -05:00
Kaushik Chandrashekar
ee411a6a2e Cells Support
Adding support for Cell show and Cell capacities calls

Implements: blueprint cell-capacity
Change-Id: I83243cf224a4487d720d55d8942d28c52924b734
2013-06-11 12:09:04 -05:00
Emanuele Rocca
ecbf7705d1 Cleanup unused local variables
This patch cleans up a few unused local variables, found with pyflakes.

Change-Id: Id65acc5d47380c7d6bfbbfe075dc23e1de30813c
2013-05-22 18:24:08 +02:00
Shane Wang
bc0ad1cb9d Reuse oslo for is_uuid_like() implementation
In the original code, isalnum() in is_uuid_like() doesn't allow "-",
while for UUID, format like aaaaaaaa-aaaa-aaaa-aaaa-aaaaaaaaaaaa should be
allowed. This patch is to reuse uuidutils's API in oslo.

Change-Id: I339974c75a32d27f8e4443a1b97bb6e410933aa4
Signed-off-by: Shane Wang <shane.wang@intel.com>
2013-05-22 11:47:23 +08:00
Shane Wang
3c97f768e5 Cleanup some flavor commands
Some cleanups include:
 - Add flavor sub-commands into README.rst
 - Check flavor ID when creating a flavor
 - Remove check_uuid_like() because it isn't used
 - Remove parameter cs in some _print_XXX functions because cs is
   not used

Change-Id: If47ce557d33db05f53e382f0670f436e05a340b7
Signed-off-by: Shane Wang <shane.wang@intel.com>
2013-05-15 15:55:27 +08:00
Shane Wang
075e9ca76e Fix the default parameter in print_list
Because sort_index = 0 by default, "if sort_index is not None" doesn't
make any sense.
Here sort_index should be None by default, which means "donot want to
sort".

Change-Id: I823ab75eb7a92bdc426dd96a3d1e56f187118729
Signed-off-by: Shane Wang <shane.wang@intel.com>
2013-05-14 13:20:56 +08:00
Alvaro Lopez Garcia
abd75f24b1 Improve authentication plugins management.
The current auth plugin system lacks some functionality to be used with
other methods that might require additional configuration options or
that do not require a user to pass some options that are now compulsory
(for example, X.509 authentication needs to get a certificate file, and
does not need either a username or a password). This commit extends
the current system to handle these extra features, while remaining
compatible with older plugins.

DocImpact: We should documment how to implement additional
authentication plugins, such as BasicAuth, X509, etc.
Implements: blueprint authentication-plugins

Change-Id: I7b0ef4981efba8160dea94bf852dba7e2e4068f5
2013-04-01 14:19:01 +00:00
Jenkins
5c9fc02468 Merge "Check if tenant flag is uuid_like for all quota operations" 2013-03-11 16:43:23 +00:00
Jenkins
2f2f4fcf2b Merge "Add wrap option to nova credentials for humans" 2013-03-11 16:43:19 +00:00
Joe Gordon
21dca44202 Add wrap option to nova credentials for humans
When using Keystone PKI, a token ID can be over 3200 chars long.
Add optional --wrap option to make 'keystone token-get' human readable
By default wrap=64 (to fit in 80 char terminal).  And can be turned off by
setting wrap=0

Fix bug 1131001

Change-Id: I50be7ebb4323ab1bf07af557403f5136b49080a4
2013-03-07 17:30:01 -08:00
Joe Gordon
1882b99734 Check if tenant flag is uuid_like for all quota operations
Fix bug 1145706

Change-Id: I9089ea4f968797b248f80bf84027a602e59ccd00
2013-03-07 14:55:55 -08:00
Flaper Fesp
47e6bc25ae Decodes input and encodes output
Currently novaclient doesn't handle properly incoming and outgoing
encode / decode process. As a solution for this, this patch implements a
decoding process for all data incoming from the user side and decodes
everything going out of the client, i.e: http requests, prints, etc.

This patch introduces a new module (strutils.py) taken from
oslo-incubator in order to use 2 of the functions present in it:

About safe_(decode|encode):

    Both functions try to encode / decode the incoming text using the
    stdin encoding, fallback to python's default encoding if that
    returns None or to UTF-8 as the last option.

    In both functions only basestring objects are accepted and they both
    raise TypeError if an object of another type is passed.

About the general novaclient changes:

    In order to better support non-ASCII characters, it is a good
    practice to use unicode interanlly and encode everything that has to
    go out. This patch aims to do that and introduces this behaviour in
    the client.

Testing:

    A good test (besides using tox) is to use nova client with and
    without setting any locale (export LANG=).

Fixes bug: 1061156

Change-Id: I20b75e42b0c3dac89f1048faa1127253a64f86c7
2013-03-04 19:01:27 +01:00
Thomas Schreiber
68e6af73ba A minimum of Python3 fixes so that installation works without errors/warnings.
Fixes bug: 1130937
Change-Id: I740652fcd5804fc1c120fc409afdf4693c8e5781
2013-02-20 23:17:49 +01:00
Kieran Spear
e800196860 Support showing extra fields in server list
Adds a --fields argument that sets the fields to display.
ID is always displayed.

Fixes bug #1076473

Change-Id: If3462e6a490ea16da4834a7f40f96b111c9e8227
2013-02-11 11:19:42 +11:00
Jenkins
c70144aced Merge "Check tenant_id's format in "nova quota-update"" 2013-01-29 20:04:35 +00:00
Jenkins
dbc186aa88 Merge "make print_dict split strings with newlines into multiple rows" 2013-01-29 20:03:36 +00:00
Haiwei Xu
168636e744 Check tenant_id's format in "nova quota-update"
Fix bug 1088835
"nova quota-update" command is executed without checking the format of
the tenant_id argument. The tenant_id should be in the format of UUID.
The tenant_id of quotas should be in accord with the form of keystone's
tenant_id. So this patch checks the format of the tenant_id when "nova
quota-update" command is executed.

Change-Id: I47c4f2ff9adbab5da4697270dcf024ac88e24529
2013-01-24 08:04:28 +00:00
Melanie Witt
f9aa5ec834 make print_dict split strings with newlines into multiple rows
When printing a dict e.g. "nova show" one of the properties
is "fault". When the user is admin, "details" of the fault
is shown, which includes a backtrace. This causes the
printing of the table to be unreadable as the backtrace
overflows the table row. This patch adds a separate row
for each substring after splitting on newlines.

Change-Id: I4c1bc8725a2bb6970be2c884c5e044d9eade8302
2013-01-21 23:37:31 +00:00
zhiyanliu
3e190c5e49 Ensure list output function can support non-sorting printing
Ensure list printing function in utils can handle non-sorting
calls. Allow result list can be formatted and outputted without any
sorting field, keep natural order to the list printing. Adding necessary
checking to avoid prettytable (0.6.1 for me) exception: "Invalid field
name: None!".

Fix bug: #1099732

Change-Id: Ied869d987e608fff8b8b5f5a65d21e02f0cebeaa
Signed-off-by: zhiyanliu <lzy.dev@gmail.com>
2013-01-17 11:51:46 +08:00
Rohan Rhishikesh Kanade
9937419888 Fix find for alphanumeic flavor id/name
Fixes LP bug #1089299

Change-Id: Ia9cc756b85096532acfc9ecacd1330de8a765fba
2012-12-20 03:14:39 -08:00
Nikola Dipanov
7ddc2fdfcd Fixes utils.findresource checking for integer
Fixes the novaclient.utils.find_resource function to properly handle
non integer values and not break with an AttributeError.

Adds a test case to test a few more non valid inputs.

Fixes bug #1065367

Change-Id: Iaa2025f4eb580234f754596c3572e2f87872170e
2012-11-02 18:21:04 +01:00
Russell Cloran
7bf93a52f8 Validate that boolean parameters are boolean
Ensure that values which are supposed to be boolean look like correct
user input, instead of assuming that any non true-looking input is
False.

Also, update the flavor manager to raise a CommandError if is_public is
not a boolean value.

Fixes bug 1059414

Change-Id: I3275e4bba103b14081becf91f723f1be060391e5
2012-10-30 11:38:58 -07:00
unicell
f160a20fd9 Implement project specific flavors API, client bindings
blueprint project-specific-flavors

This change implements client bindings for the project specific flavor API in following change
https://review.openstack.org/#/c/11270/

Change-Id: Id8d559985f9369f53926e63ee5f5ce23a051e25b
2012-08-27 22:38:36 +08:00
Vishvananda Ishaya
576a64fbb5 Allow resources to use any field as 'name'
The 'name' field for some resources is called something different,
for example 'display_name' for volumes. There was a hack in our
find method to search for display_name as well.

This change adds a new class variable to a Resource to tell it
which attribute to use for searching by name. Volumes and snapshots
were switched to use 'display_name' and hypervisors were switched
to use 'hypervisor_hostname'.

Tests fixed and added.

Fixes bug 1034536

Change-Id: I1b4fb969d42c59d1ab8e3e275a563bbe158e9264
2012-08-08 12:01:40 -07:00
Josh Kearney
d4c9b12f39 Indicate unused variables and other misc. house cleaning.
Change-Id: I4529d8b6b27dddb1b79ee2167a054b471eaf0dbc
2012-06-27 11:55:36 -05:00
Sandy Walsh
a2a62a5f71 option to bypass managment endpoint and timings support
--timings = show timings for each call made to nova (including auth)
--bypass_url = api node endpoint to use instead of one from service catalog

For example:
nova --timings --bypass_url=http://10.24.31.37:8774/v1.1/nova-staging boot --image f304d266-0a49-4877-b34c-63aea8360297 --flavor 3 delete_me_2

Change-Id: Ib2a258b7e969ad56ce4ee2bd64c61310278cb856
2012-06-18 13:03:58 -03:00
Brian Waldon
7745b35b69 Lock prettytable dep at v0.6
Specifically set the prettytable dependency so we can reliably
align data in table cells.

Change-Id: I1787c4100248b2a75405df690931110c76803413
2012-06-11 09:57:03 -07:00
Ghe Rivero
d743f92dc2 Updated to new prettytable api. Fixes bug 995818
In 0.6, printt() (wrapper to get_string) has been removed.
Updated to use get_string(), backward available since 0.2 release.

Change-Id: Ica292757ec3de5004f27afbc5c8ee11d839421df
2012-05-07 18:45:57 +02:00
Vishvananda Ishaya
d43f172071 Don't force volume id to int and allow search by name
* required for change 6511 to merge

Change-Id: Ia8d28ca7cce4c00aa0b3f1fe1da6719ec99d6fe4
2012-05-02 15:06:55 +00:00
Rick Harris
0028eb4ac2 Request ID when name is ambiguous.
Fixes bug 931605

Change-Id: If0778915d964995dbb6647d56ed21075aec08baa
2012-04-09 21:13:57 +00:00
Rick Harris
b22ec22336 Add human-friendly ID support.
Allows a user to interact with certain models (image, flavors, and
servers currently) using a human-friendly identifier which is a
slugified form of the model name.

Example:

    nova boot --image debian-6-squeeze --flavor 256mb-instance myinst

Change-Id: I43dbedac3493d010c1ec9ba8b8bb1007ff7ac499
2012-03-08 04:53:47 +00:00
Vishvananda Ishaya
03f54c57e1 Makes novaclient use the volumes endpoint
* Depends on https://review.openstack.org/#change,4479
 * Adds support to change service type including tests
 * Adds decorator for methods that need to use another service type
 * Changes volume and snapshots to use the volume endpoint
 * These extensions will move into the volume client once it exists
 * Fixes bug 940017

Change-Id: I683e4ca6c67e278d8aa8a9acec3dc0f1872f43f2
2012-02-26 06:11:12 +00:00
Dave Walker (Daviey)
1ee77c7fe3 Allow UUID_CACHE_DIR overriding via env variable.
This is declared as NOVACLIENT_UUID_CACHE_DIR.
Resolves bug 932468 , defaulting to previous behaviour.
Added myself to AUTHORS

Change-Id: I154500517d7c882a4a090588a95f4b3bfee70595
2012-02-15 21:58:27 +00:00
Paul Voccio
cb9a7a37a8 adding credentials and endpoints output for debugging
Change-Id: I74e12172d8c05df257c4b908e3278712a66f795a
2012-02-06 11:14:16 -06:00
Jenkins
4c61a89457 Merge "Abstract Client building into novaclient.client" 2012-01-04 16:45:47 +00:00
Brian Waldon
e0174b51bd Abstract Client building into novaclient.client
This prevents clients of the pythonic api from having to know the
internal module structure.

Change-Id: Idd5c522ac3ff6c2d7915f96ed327323ec83d54fc
2011-12-29 15:40:49 -05:00
Julien Danjou
3758c260a0 utils.find_resource fixes + fix for volumes
Nova volumes do not have a name property but a displayName one, therefore we
modify find_resource() so it looks for this. Then, we modify the test suite
for utils.py so it uses the novaclient.base.ManagerWithFind class for its
FakeManager rather than implementing its own find() method. This way we are
sure that the utils.py functions work correctly with ManagerWithFind and its
way to search in objects list.

FakeResource class is now used, whereas it was not before.

Change-Id: I4c4838250e6987e05022e1607e8d099eb8fb7fff
Signed-off-by: Julien Danjou <julien@danjou.info>
2011-12-22 12:03:59 +01:00
Rick Harris
88bdfdd12f Extensions can now modify resources.
Change-Id: I0d9c1050428d0ccf2e2b18053d75e0465463c08b
2011-12-21 19:25:39 +00:00
Ziad Sawalha
49284dc5cd Add 'discover' command for Keystone discovery and version listing
Added @unauthenticated decorator to mark subcommands that do not
need authentication. And checks to skip authentication for these
commands.
Added novaclient.keystone to setup.py

Change-Id: Id2fd60af305c30a950bdbae8f897192bfae4d797
2011-12-20 12:33:50 -06:00
Rick Harris
bb879dd10b Adding extension framework.
Change-Id: If882f7a822ef6b1e58666b3af6f7166ab0a230fe
2011-12-15 19:44:33 +00:00
Jason Straw
690f3d3866 PEP8 cleanups of utils, and the v1_?/shell.py files 2011-11-03 22:44:22 -05:00
Jason Straw
66d8bc49b7 Updated the novaclient shell to display the parent server id that the image came from. 2011-10-30 17:20:47 -05:00
Brian Waldon
79a532b910 cleaning up find_resource method to support str/int ids, uuids, and integer-like names 2011-08-22 15:13:26 -04:00
termie
536be15a87 whitespace cleanups 2011-08-08 13:25:29 -07:00