summaryrefslogtreecommitdiff
path: root/openstackclient
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | Merge "add keystone v3 region object"Jenkins2014-11-193-0/+626
|\ \ \ \
| * | | | add keystone v3 region objectwanghong2014-11-183-0/+626
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Co-Authored-By: Steve Martinelli <stevemar@ca.ibm.com> Change-Id: Ia6f607630dbf507681733c3ab3b9b7c55de30f49 Closes-Bug: #1387932
* | | | | Merge "Add --or-show support for v3 identity resources"Jenkins2014-11-185-27/+106
|\ \ \ \ \ | |_|/ / / |/| | | |
| * | | | Add --or-show support for v3 identity resourcesSteve Martinelli2014-11-185-27/+106
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Add --or-show for the following: * v3 roles * v3 projects * v3 domains * v3 users * v3 groups Closes-Bug: #1390389 Change-Id: Id4ef043e5fda6be49a515eb3fe138c813c393ec9
* | | | | Merge "Add environment variable in the os-auth-type help"Jenkins2014-11-181-1/+2
|\ \ \ \ \ | |_|_|_|/ |/| | | |
| * | | | Add environment variable in the os-auth-type helpMarek Denis2014-11-171-1/+2
| | |_|/ | |/| | | | | | | | | | | | | | | | | | | | | | Help for option --os-auth-type doesn't specify what environment variable configures it. This patch fixes that. Change-Id: Id2e29e477d5ca56339bd777fb73b5af13788615b
* | | | Merge "Add arg to 'server image create' tests"Jenkins2014-11-182-5/+10
|\ \ \ \
| * | | | Add arg to 'server image create' testsDean Troyer2014-11-142-5/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The 'protected' column was not being checked. Also add it to image.fakes.IMAGE. Change-Id: Ie431e9871a7da78b5a3924bfbc51d5575d994d86
* | | | | Merge "Add additional support for --or-show"Jenkins2014-11-184-7/+194
|\ \ \ \ \ | |_|/ / / |/| | / / | | |/ / | |/| |
| * | | Add additional support for --or-showSteve Martinelli2014-11-144-7/+194
| |/ / | | | | | | | | | | | | | | | | | | | | | Add --or-show for the following: * v2 roles * v2 projects Change-Id: Ibbae19cda668575b9527fbd259f1298c48b8265b
* | | Merge "Remove links from oauth consumers"Jenkins2014-11-141-6/+4
|\ \ \
| * | | Remove links from oauth consumersSteve Martinelli2014-11-131-6/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This should be the last of the v3 identity objects that return a links section upon create or show. Change-Id: I45a3b43c303bfed73950095bec8860cbea7a559c
* | | | Merge "Remove links from federation related commands in identity v3"Jenkins2014-11-143-12/+11
|\ \ \ \ | |/ / / | | / / | |/ / |/| |
| * | Remove links from federation related commands in identity v3Steve Martinelli2014-11-133-12/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We should remove the 'links' portion from the returned object for the following commands: * create/show federation protocol * create/show mapping * create/show identity provider Change-Id: I55654cce1f89de8e532f9acd8092257be33efd85
* | | Merge "Beef up object-store tests"Jenkins2014-11-135-58/+228
|\ \ \ | |/ / |/| |
| * | Beef up object-store testsDean Troyer2014-10-225-58/+228
| | | | | | | | | | | | | | | | | | | | | | | | * Add object top-to-bottom tests * Move some fakes around * Clean up existing object tests Change-Id: If8406da611c11bbd2b1bf5153e45b720b0eea442
* | | Merge "Use fixtures from keystoneclient for static data"Jenkins2014-11-121-134/+11
|\ \ \
| * | | Use fixtures from keystoneclient for static dataJamie Lennox2014-10-311-134/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | We should use the fixture generation code from keystoneclient rather than keep our own copies of the token and discovery structure. Change-Id: I53c1d2935d1d65c39b8abea89427af2fc3edd181
* | | | Swap remaining assertEqual argumentsDean Troyer2014-11-071-22/+22
| | | | | | | | | | | | | | | | Change-Id: I1abdebb298b93074657a7ba65a7186d814969780
* | | | Add --or-show option to user createDean Troyer2014-11-073-7/+103
| |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The --or-show option is added to create commands for the common case of needing to ensure an object exists and getting its properties if it does or creating a new one if it does not exist. Note that if the object exists, any additional options that would set values in a newly created object are ignored if the object exists. FakeResource needs the __name__ attribute to fall through utils.find_resource. Prove the concept on v2 user create then propogate once we're happy with it... Change-Id: I6268566514840c284e6a1d44b409a81d6699ef99
* | | Unscoped federated user-specific commandsMatthieu Huin2014-10-304-0/+225
|/ / | | | | | | | | | | | | | | | | | | | | | | A federated user can authenticate with the v3unscopedsaml plugin and list the domains and projects she is allowed to scope to. This patch introduces the new commands 'federation domain list' and 'federation project list'. Note that for these commands -and plugin- to be available, the lxml library must be installed. Change-Id: I2707b624befcfb0a01b40a094e12fd68a3ee7773 Co-Authored-By: Florent Flament <florent.flament-ext@cloudwatt.com>
* | Fix server create for boot-from-volumeDean Troyer2014-10-242-21/+52
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * server create required --image even when booting the server from a volume. Change options to require either --image or --volume to specify the server boot disk. Using --volume currently uses device 'vda' for the block mapping and ignores any other block mappings given in --block-device-mapping. * server create and server show are both affected by bug 1378842 where an excepion was thrown when no image ID was present in the returned server object, which is the case for a server booted from a volume. * Fix the remaining assertEqual() order problems in test_server.py Closes-Bug: 1378842 Closes-Bug: 1383338 Change-Id: I5daebf4e50a765d4920088dfead95b6295af6a4d
* | Adjust some logging levelsDean Troyer2014-10-242-15/+24
| | | | | | | | | | | | | | * Promote select messages to INFO so lower logging levels can be useful * Help more modules not say so much all the time Change-Id: I814023c1489595998ae74efe40ef439b3522ee74
* | Merge "Close files on image create"Jenkins2014-10-242-28/+42
|\ \
| * | Close files on image createDean Troyer2014-10-142-28/+42
| | | | | | | | | | | | | | | | | | | | | The file opened for --file was never closed. Close it if it is a file object. Change-Id: I7bd120a2413de42339771d01e8fd1894d38c3011
* | | Merge "Close files on server create, add tests"Jenkins2014-10-244-4/+194
|\ \ \
| * | | Close files on server create, add testsDean Troyer2014-10-144-4/+194
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | The files opened for the --files and --user-data options were never closed, potentially leaking memory in a long-running client. Close them if they are file objects. Add a couple of basic tests for server create. Change-Id: I1658b0caa2d6af17308149cb52196ee28266ddf2
* | | Merge "Change --os-auth-plugin to --os-auth-type"Jenkins2014-10-244-68/+66
|\ \ \
| * | | Change --os-auth-plugin to --os-auth-typeDean Troyer2014-10-234-68/+66
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | User's don't know what a plugin is. * Internally, os_auth_type and/or auth_type represents what the user supplied. * auth_plugin_name is the name of the selected plugin * auth_plugin is the actual plugin object Plugin selection process: * if --os-auth-type is supplied: * if it matches against an available plugin, done * (if it can map to an availble plugin type, done; TODO in a followup) * if --os-auth-type is not supplied: * if --os-url and --os-token are supplied, select 'token_endpoint' * if --os-username supplied, select identity_api_version + 'password' * if --os-token supplied, select identity_api_version + 'token' Change-Id: Ice4535214e311ebf924087cf77f6d84d76f5f3ee
* | | | Merge "Update use of open() in object API"Jenkins2014-10-242-5/+24
|\ \ \ \
| * | | | Update use of open() in object APIDean Troyer2014-10-132-5/+24
| | |/ / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Switch to use io.open() for py3 compatibility and simpler testing. * Open files in 'rb' mode to avoid translation on Windows Previously tests simply relied on files that were present in the repository to run tests using open(). Change the filenames to ensure that no longer happens. requests_mock doesn't have a way to match against the request body for PUT/POST; an attempt to add a new Matcher to do that worked but it needs to subclass the currently private adapter._Matcher class or duplicate most of its functionality. Change-Id: I8c30b41db20af8ecafe67e760e872fc08adec905
* | | | Merge "Clean up shell authentication"Jenkins2014-10-223-126/+74
|\ \ \ \ | | |/ / | |/| |
| * | | Clean up shell authenticationDean Troyer2014-10-213-126/+74
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Remove the auth option checks as the auth plugins will validate their own options * Move the initialization of client_manager to the end of initialize_app() so it is always called. Note that no attempts to actually authenticate occur until the first use of one of the client attributes in client_manager. This leaves initialize_clientmanager() (formerly uathenticate_user()) empty so remove it. * Remove interact() as the client_manager has already been created And there is nothing left. * prepare_to_run_command() is reduced to trigger an authentication attempt for the best_effort auth commands, currently the only one is 'complete'. * Add prompt_for_password() to ask the user to enter a password when necessary. Passed to ClientManager in a new kward pw_func. Bug: 1355838 Change-Id: I9fdec9144c4c84f65aed1cf91ce41fe1895089b2
* | | Merge "Fix token issue after auth changeup"Jenkins2014-10-213-9/+9
|\ \ \ | |/ / |/| |
| * | Fix token issue after auth changeupDean Troyer2014-10-203-9/+9
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | IssueToken.take_action() was missed in updating the structure of the ClientManager. Also, TOKEN_WITH_TENANT_ID in v3 is just wrong... Closes-Bug: #1383396 Change-Id: If2dd82a26af1d743ee9df73e0c1aebce497bf22e
* | | only generate one clientmanager instance in interactive modewanghong2014-10-201-5/+11
|/ / | | | | | | | | | | | | | | Currently, we repeated to generate clientmanager instance when run command in interactive mode. This should be avoided. Change-Id: I0536a690bc173be38af08a2e4443115532041efd Closes-Bug: #1383083
* | Remove ClientManager._service_catalogDean Troyer2014-10-187-40/+29
| | | | | | | | | | | | | | | | | | | | Anything that needs a service catalog can get it directly from auth_ref.service_catalog, no need to carry the extra attribute. ClientManager.get_endpoint_for_service_type() reamins the proper method to get an endpoint for clients that still need one directly. Change-Id: I809091c9c71d08f29606d7fd8b500898ff2cb8ae
* | Remove now-unnecessary client creation hacksDean Troyer2014-10-186-41/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Clients that can use ksc Session don't need the old junk to fake auth anymore: * compute * volume Clients that still need to be fed credentials can pick directly from the auth object in clientmanager. The _token attribute is removed, the token can be retrieved from the auth object: openstackclient/tests/common/test_clientmanager.py This change will break any plugin that relies on getting a token from instance._token. They should be updated to use the above, or preferable, to use keystoneclient.session.Session to create its HTTP interface object. Change-Id: I877a29de97a42f85f12a14c274fc003e6fba5135
* | Merge "Add plugin to support token-endpoint auth"Jenkins2014-10-174-58/+121
|\ \
| * | Add plugin to support token-endpoint authDean Troyer2014-10-124-58/+121
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The ksc auth plugins do not have support for the original token-endpoint (aka token flow) auth where the user supplies a token (possibly the Keystone admin_token) and an API endpoint. This is used for bootstrapping Keystone but also has other uses when a scoped user token is provided. The api.auth:TokenEndpoint class is required to provide the same interface methods so all of the special-case code branches to support token-endpoint can be removed. Some additional cleanups related to ClientManager and creating the Compute client also were done to streamline using sessions. Change-Id: I1a6059afa845a591eff92567ca346c09010a93af
* | | Merge "use jsonutils in oslo.serialization instead of keystoneclient"Jenkins2014-10-171-1/+1
|\ \ \
| * | | use jsonutils in oslo.serialization instead of keystoneclientwanghong2014-10-171-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | keystoneclient/openstack/common/jsonutils.py is removed in this patch https://review.openstack.org/#/c/128454/ Now, we should use jsonutils in oslo.serialization package. Change-Id: I7c8e8e6d5dffa85244368fd578616c9b19f4fd21
* | | | Merge "Move plugin stuff to clientmanager"Jenkins2014-10-172-18/+30
|\ \ \ \ | |/ / / |/| | |
| * | | Move plugin stuff to clientmanagerDean Troyer2014-10-132-18/+30
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The OSC plugins work by adding an object as an attribute to a ClientManager instance. The initialization and management of thos plugins belongs in clientmanager.py. At this point the only part not moved is the API version dict initialization bcause the timing and connection to the CommandManager initialization. It gets refactored anyway when API discovery becomes operational. Change-Id: If9cb9a0c45a3a577082a5cdbb793769211f20ebb
* | | Merge "Add 'command list' command"Jenkins2014-10-144-0/+80
|\ \ \
| * | | Add 'command list' commandDean Troyer2014-10-084-0/+80
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * Add method to CommandManager to retrieve command names by group * Add ListCommands To list command groups loaded by cliff Change-Id: I37fe2471aa2fafa8aa223159452d52b1981021d6
* | | | Merge "Fix operation on clouds with availability-zones"Jenkins2014-10-144-5/+6
|\ \ \ \
| * | | | Fix operation on clouds with availability-zonesMonty Taylor2014-10-114-5/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In a cloud with AZs, you can get multiple entries back from the service catalog - one for each AZ and then one that is AZ agnostic that's tied to the region. If the region_name is plumbed all the way through, this works as intended. Change-Id: I3b365ea306e8111fc80830672ae8080a5d1dc8e0
* | | | | Merge "Remove 'links' section from several v3 Identity objects"Jenkins2014-10-1312-44/+32
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | Remove 'links' section from several v3 Identity objectsSteve Martinelli2014-10-1112-44/+32
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The links field in the returned objects from the v3 Identity API aren't really useful, so let's remove them. Managed to remove most of them from the core API. I'll likely remove the extension/contribution (oauth/federation) related ones in another patch. Also in this patch the code for setting services and projects was changed. Though not incorrect, it was not needed to copy the entire returned object, we should just need to pass in the fields we want to update. Change-Id: I164ca9ad8b28fa10b291e9115ef40753e387c547