summaryrefslogtreecommitdiff
path: root/openstackclient
Commit message (Collapse)AuthorAgeFilesLines
...
| * | Don't create empty quota set requestsJamie Lennox2015-05-051-2/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The way that getattr is called with the None default you will always create a compute_kwargs dictionary with key: None values. This means that we will always send these empty requests to the servers. Change so that only actually changed values end up in the quota set requests and get sent. Change-Id: I33bc3f4e1a8013ec672e995648d27513064baf26 Closes-Bug: #1451640
* | | Merge "Add docs for service provider CRUD"Jenkins2015-05-061-16/+18
|\ \ \
| * | | Add docs for service provider CRUDSteve Martinelli2015-04-151-16/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This patch adds service providers to command-objects, and makes a few changes to the help text, to align it more with the already established identity provider resource. Change-Id: Ibf3d2bc04bf5588d1fc9c37b8ca28c007496c021
* | | | Merge "Federation Service Providers CRUD operations"Jenkins2015-05-063-0/+644
|\ \ \ \ | |/ / / | | / / | |/ / |/| |
| * | Federation Service Providers CRUD operationsMarek Denis2015-04-153-0/+644
| | | | | | | | | | | | | | | | | | | | | | | | | | | Adds CRUD support for service providers as it's now available through keystoneclient Closes-Bug: 1435962 Depends-On: If802e8a47e45ae00112de3739334b4b5482d0500 Change-Id: Ic55101e50209070aa49ca2adc91c89ba754c8c68
* | | Merge "Add image show tests"Jenkins2015-05-014-8/+166
|\ \ \
| * | | Add image show testsDean Troyer2015-05-014-8/+166
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | Image v2 uses warlock objects rather than the usua Resource objects so we need to test for those. This adds a subset of the Image v2 schema that should be enough to test for proper warlock image handling. Depends-On: Ic95db2f63d9f5f37e29f0d7e048397da311fbf8c Change-Id: Ib89cce87f110a554f40e726718e31d39b500a6ae
* | | Minor fix to openstack image show commandAmey Bhide2015-05-011-1/+1
|/ / | | | | | | | | | | | | | | | | | | image show using V2 api was failing. openstack --os-image-api-version 2 image show <image_id> ERROR: openstack _info Closes-Bug: #1450829 Change-Id: Ic95db2f63d9f5f37e29f0d7e048397da311fbf8c
* | Fix tiny typo in comment messageLin Yang2015-04-302-2/+2
| | | | | | | | | | | | | | compatability => compatibility Change-Id: I3181fb2b83df1e2cb60a9eedf319f2ad0a487dba Signed-off-by: Lin Yang <lin.a.yang@intel.com>
* | Merge "Add parent field to project creation"Jenkins2015-04-303-2/+123
|\ \
| * | Add parent field to project creationRodrigo Duarte2015-03-233-2/+123
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adding the possibility to create projects hierarchies by adding the parent field in the create project call. Co-Authored-By: Victor Silva <victor@lsd.ufcg.edu.br> Implements: bp hierarchical-multitenancy Change-Id: I4eac4f5bc067634cc38c305dacc59ab1da63c153
* | | Minor logging/debug cleanupsDean Troyer2015-04-281-6/+2
| | | | | | | | | | | | | | | | | | This removed the rarely useful cliff command list from the debug output. Change-Id: I48f22086733acf90e79a6ddac8712734ee2d0b60
* | | Raise exception if no session is createdJuan Antonio Osorio Robles2015-04-271-6/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | If the clientmanager is unable to create a keystone session (due to insufficient auth parameters or something else) then the exception caused by this shouldn't be ignored, as was the case. On the other hand, we don't want this behaviour in the case of the 'complete' command, so this is now properly detected. Change-Id: If4f453d23cc87900cda752e9ffbcf41ded59e26f Closes-Bug: #1444640
* | | Merge "Fix security group create description bug"Jenkins2015-04-233-1/+206
|\ \ \
| * | | Fix security group create description bugDean Troyer2015-04-223-1/+206
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | --description is optional in our CLI but the server requires it to be non-empty. Set a default value of the given name. Closes-Bug: #1434172 Change-Id: I81507a77ad8d815000ff411784ae71e229c77f78
* | | | Merge "Security group rule create fails"Jenkins2015-04-231-0/+1
|\ \ \ \ | |/ / / |/| | |
| * | | Security group rule create failsRamaraja Ramachandran2015-04-221-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | By default the --dst-port value is set to None when no --dst-port argument is provided. By making the default value (0, 0), this allows novaclient to proceed without any error. Change-Id: Ibb58f5df5ed1890a8f499dd2467b12b0e79d547b Closes-Bug: #1443963
* | | | Merge "Refactor utility to find identity resources"Jenkins2015-04-221-56/+41
|\ \ \ \
| * | | | Refactor utility to find identity resourcesSteve Martinelli2015-04-201-56/+41
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Based on the comments made in this patch: https://review.openstack.org/#/c/174908/2/ We should simplify and refactor the way we handle finding identity resources. Change-Id: I77db2e3564faa90a917082a6c6cb87269e93aebe
* | | | | Adds support for container selection for backupDag Stenstad2015-04-211-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Changed argument from parsed_args.volume to parsed_args.container as per what is expected in python-cinderclient. If not defined, defaults to "backup_swift_container" in cinder.conf. Stops there being a container for every volume you take backups off. Change-Id: I4a34894222f71b0f207d007c32427040589406aa Closes-Bug: 1446751
* | | | | Merge "Handle the pagination for image list"1.1.0Jenkins2015-04-204-15/+52
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Handle the pagination for image listTerryHowe2015-04-204-15/+52
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Handle the paginatiion for image list. We were sorting the data here, so nothing lost for the generator. Change-Id: I2d7d4b3d5c9f650953f309c971ac53b64f6f7f77
* | | | | Merge "Update the docs for new nic options"Jenkins2015-04-191-1/+1
|\ \ \ \ \
| * | | | | Update the docs for new nic optionsSteve Martinelli2015-04-191-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I4c8b93ede80f993eab8badfbba6bf1c530844d54
* | | | | | Merge "remove unnecessary conditionals"Jenkins2015-04-194-18/+10
|\ \ \ \ \ \
| * | | | | | remove unnecessary conditionalsSteve Martinelli2015-04-194-18/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In several places we had else branches where a reasonable default would do the job. This makes the code a mean cleaer and easier to read. Change-Id: I231e09aab85fd32b8300bc33c48d0899b728b96e
* | | | | | | Merge "Role operations should not require list object permission"Jenkins2015-04-193-49/+108
|\ \ \ \ \ \ \ | |/ / / / / / |/| | | | | |
| * | | | | | Role operations should not require list object permissionNathan Kinder2015-04-173-49/+108
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When using Keystone's policy.v3cloudsample.json policy file, a project admin is supposed to be able to manage role assignments. Unfortunately, a project admin isn't allowed to perform these operations using python-openstackclient, as we attempt to perform list operations for any of the object types specified (users, groups, projects). This is done in an attempt to lookup the id of the object by name, but we perform this list operation even when the user specifies everything by id. This causes 403 errors. This patch still attempts to look up the object id by name, but we catch the 403 and assume that the user specified an id if the list operation is not allowed. This is similar to what we do with the --domain option for other commands. Closes-bug: #1445528 Change-Id: Id95a8520e935c1092d5a22ecd8ea01f572334ac8
* | | | | | Merge "Add --os-cloud support"Jenkins2015-04-195-317/+375
|\ \ \ \ \ \
| * | | | | | Add --os-cloud supportDean Troyer2015-04-185-317/+375
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This adds a new option --os-cloud that allows the configuration values for multiple clouds to be stored in a local file and selected with a single option. Internal option names have had 'os_' removed to be comptible with the options returned from OpenStackConfig().get_one_cloud(). The config file is ~/.config/openstack/clouds.yaml: Sample ------ clouds: devstack: auth: auth_url: http://192.168.122.10:35357/ project_name: demo username: demo password: 0penstack region_name: RegionOne devstack: auth: auth_url: http://192.168.122.10:35357/ project_name: demo username: demo password: 0penstack region_name: RegionOne Co-Authored-By: Monty Taylor <mordred@inaugust.com> Change-Id: I4939acf8067e44ffe06a2e26fc28f1adf8985b7d Depends-On: I45e2550af58aee616ca168d20a557077beeab007
* | | | | | Merge "Add support for showing limits of a specific project"Jenkins2015-04-191-1/+28
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Add support for showing limits of a specific projectSteve Martinelli2015-04-031-1/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Looks like the option to show limits of a specific project was missing. This resulted in always using the authenticated project. Change-Id: I512a05df20860ffb52af34f3b64c9eb81dae8c61 Closes-Bug: 1438379
* | | | | | Merge "Defer client imports"Jenkins2015-04-195-39/+73
|\ \ \ \ \ \
| * | | | | | Defer client importsDean Troyer2015-04-155-39/+73
| | |_|/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | So we really weren't deferring the loading of client libs dadgummit, do that for real where possible. This shaves a couple of tenths off the static import times. Also defer as much import-time procesing as possible. This is a little ugly in api.auth but this also eliminates import of the auth plugins until they are needed. Change-Id: Ia11d4b9cf98231d37449103fc29101dc17afb009
* | | | | | Merge "Suppress warnings user can't fix"Jenkins2015-04-181-0/+5
|\ \ \ \ \ \
| * | | | | | Suppress warnings user can't fixDean Troyer2015-04-021-0/+5
| | |/ / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Requests/urllib3 started issuing warnings about certificates and SSL that our users are unable to do anything about. This is a very blunt way to suppress these warnings unless --verbose or --debug is supplied on the command line. Being more precise in the suppression requires importing the warning classes from urllib3 and dealing with the platforms where it has been unvendored from requests. Maybe in the future if there are concerns that this mutes too much otherwise. Change-Id: I50bb10a16222de12c5b95bfe042b92e43ea8ee7c
* | | | | | Merge "Print warning on authentication error"Jenkins2015-04-181-1/+2
|\ \ \ \ \ \
| * | | | | | Print warning on authentication errorTerryHowe2015-04-171-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | At least print a warning on authentication error. I have no idea why an exception is being ignored here because if there is no session, nothing is going to happen. This at least will print some useful warning: (.venv)terry@f350:~/python-openstackclient$ os flavor list --os-cloud pro WARNING: openstackclient.shell Possible error authenticating: __init__() got an unexpected keyword argument 'asdf' ERROR: openstack Authentication requires 'auth_url', which should be specified in 'HTTPClient' This error was caused by having a bogus value 'asdf' in the cloud.yaml for the cloud pro. Change-Id: Ie08432e0464cfa86b3b3f67ca29d3b7d23d2f46f
* | | | | | | Merge "Fix session timing"Jenkins2015-04-175-19/+71
|\ \ \ \ \ \ \
| * | | | | | | Fix session timingDean Troyer2015-04-075-19/+71
| | |_|_|_|/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Subclass keystoneclient.session.Session to add the timing hooks to record the elapsed time returned by requests.Response objects, including the redirection history. Redirects are included individually and not rolled into the total time for the original request. This works for all clients that use OSC's session. Closes-Bug: #1402577 Change-Id: I9360c90c151579b89a37edb8c11c17feb15b3cb9
* | | | | | | Fix skipped image create attribute location attrMarek Aufart2015-04-171-1/+1
| |/ / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Image create action accepts attribute called location, in method body is list of allowed attributes, which contain localtion, what is typo and this attribute was not passed to glance. Fixed. Change-Id: I357b06b63b8aa97f7a5f587ef3fcee7a4b360ed1 Closes-Bug: #1445460
* | | | | | Merge "Better help for --nic in create server"Jenkins2015-04-171-2/+9
|\ \ \ \ \ \ | |_|_|_|/ / |/| | | | |
| * | | | | Better help for --nic in create serverDavanum Srinivas2015-04-151-2/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Use the help string from python-novaclient for a better user experience. Closes-Bug: #1444685 Change-Id: If7b8e3f68a0c6ad82b9959f162670b5568d5d12d
* | | | | | Merge "Add support to specify volume quotas per volume type"Jenkins2015-04-161-2/+10
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Add support to specify volume quotas per volume typeSteve Martinelli2015-04-151-2/+10
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add a --volume-type option to quota set, this will allow users to set quotas for volume attributes on a per volume-type basis. for example: openstack quota set admin --volume-type myvol --volumes 12 Change-Id: I3ce9cf82a65d4f012b339f0e0dedb752cb132c33 Closes-Bug: 1438377
* | | | | Merge "Use cliff deferred help instead of homemade one"Jenkins2015-04-141-32/+3
|\ \ \ \ \
| * | | | | Use cliff deferred help instead of homemade oneCedric Brandily2015-03-171-32/+3
| | |/ / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This change removes openstackclient homemade hack to defer help printing in initialize_app and uses cliff (new) option to defer help printing. Change-Id: Ie3e94ec96254745bfef8c5ff5abc405facfe1bea Related-Bug: #1316622
* | | | | Merge "Add warning message if unknown version supplied"Jenkins2015-04-142-0/+8
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | Add warning message if unknown version suppliedTerryHowe2015-04-132-0/+8
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | Print a warning message if an unknown api version is supplied. An attempt will be made to run the command anyway. Change-Id: Idec8e88fe9621f10ec4b7eecd90708fb3730f56f
* | | | Use glanceclient's inbuilt images findJamie Lennox2015-04-021-55/+1
|/ / / | | | | | | | | | | | | | | | | | | | | | Glanceclient image listing was special cased as it wasn't implemented in glanceclient directly. This is no longer the case and we should use glanceclient's functions. Change-Id: If8d1246f1bd97c07f9f10f5457aa32124efa0be3
View Lorry Controller Status