summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Add a doc that dictates backwards incompatible changesSteve Martinelli2015-04-232-0/+48
| | | | | | | | start tracking any changes in a standard format, so users may have a heads up about any impacts. Change-Id: Ibc06926a53592e927d11440362cd3598e0d4b2bf Closes-Bug: 1406470
* 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
* | | Merge "Adds support for container selection for backup"Jenkins2015-04-211-1/+1
|\ \ \
| * | | 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
* | | | Create 1.1.0 release notesDean Troyer2015-04-211-0/+17
|/ / / | | | | | | | | | Change-Id: Ib5dd052ed938a844d39a61fcbea0510a2a427ab7
* | | 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
* | | | Imported Translations from TransifexOpenStack Proposal Bot2015-04-203-41/+87
| |/ / |/| | | | | | | | | | | | | | | | | For more information about this automatic import see: https://wiki.openstack.org/wiki/Translations/Infrastructure Change-Id: I0e7cdac01f3dc8b9368aa891e93125a5e838ae3d
* | | Merge "Begin documenting --os-cloud"Jenkins2015-04-194-4/+236
|\ \ \
| * | | Begin documenting --os-cloudDean Troyer2015-04-184-4/+236
| | | | | | | | | | | | | | | | Change-Id: Id2e98ac5601840f6d380cabcd578f1a6d6d9b245
* | | | Merge "Update the docs for new nic options"Jenkins2015-04-192-4/+9
|\ \ \ \
| * | | | Update the docs for new nic optionsSteve Martinelli2015-04-192-4/+9
| | | | | | | | | | | | | | | | | | | | 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-199-321/+411
|\ \ \ \ \ | | |_|/ / | |/| | |
| * | | | Add --os-cloud supportDean Troyer2015-04-189-321/+411
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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-192-1/+38
|\ \ \ \ | |/ / / |/| | |
| * | | Add support for showing limits of a specific projectSteve Martinelli2015-04-032-1/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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 "Imported Translations from Transifex"Jenkins2015-04-183-305/+334
|\ \ \ \
| * | | | Imported Translations from TransifexOpenStack Proposal Bot2015-04-183-305/+334
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For more information about this automatic import see: https://wiki.openstack.org/wiki/Translations/Infrastructure Change-Id: I489da64d5c1b14506dfa6619df71afa763f4b3b6
* | | | | 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
* | | | | | Re-organize functional testsSteve Martinelli2015-04-1710-79/+95
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The tests should be further divded by project version, similar to the structure of the unit tests. Change-Id: Ied3a4204983cdd253c5602a60968c066038d88f2
* | | | | | Merge "Fix skipped image create attribute location attr"Jenkins2015-04-171-1/+1
|\ \ \ \ \ \ | |_|/ / / / |/| | | | |
| * | | | | 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 "Uncap library requirements for liberty"Jenkins2015-04-172-8/+8
|\ \ \ \ \ \ | |/ / / / / |/| | | | |
| * | | | | Uncap library requirements for libertyDoug Hellmann2015-04-162-8/+8
| | |_|_|/ | |/| | | | | | | | | | | | | | | | | | Change-Id: Ia2b0c00c5b1da19f2f6995aae3c2e0226a3fa2dc Depends-On: Ib948b756b8e6ca47a4c9c44c48031e54b7386a06
* | | | | 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-162-2/+15
|\ \ \ \ \ | |/ / / / |/| | | |
| * | | | Add support to specify volume quotas per volume typeSteve Martinelli2015-04-152-2/+15
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | 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
* | | Merge "Updated from global requirements"Jenkins2015-04-012-10/+10
|\ \ \