summaryrefslogtreecommitdiff
path: root/openstackclient/identity
Commit message (Collapse)AuthorAgeFilesLines
* take_action() method from command.Command shouldn't returnMohan Muppidi2016-02-2916-43/+18
| | | | | | | | | | | | command.Command and command.Showone are base classes implemented in cliff framework. Showone extends Command to allow take_action() to return data to be formatted using a user-selectable formatter. Most of the classes which are extended from Command in openstackclient/identity/v3/ in some cases return data or return nothing where it is not necessary, this commit fixes most of them. Change-Id: I84c72ea4d6680f8bdbef5449316dd9a8af8c8286 Closes-Bug: 1550892
* Fixed a bunch of spacingBrandon Palm2016-02-233-2/+4
| | | | | | | Nothing too complicated here. I fixed a bunch of spacing issues that I saw in OSC. Change-Id: I935ab48e7c5bac5f88ecdb3a05f73fb44fc9f41d
* Support unscoped token requestguang-yee2016-02-192-1/+8
| | | | | | | | Make scope check optional for the "token issue" command as unscoped token is a valid Keystone V2/V3 API. Change-Id: Ie1cded4dbfdafd3a78c0ebdf89e3f66762509930 Closes-Bug: #1543214
* Merge "Return names in list role assignments"Jenkins2016-02-171-6/+34
|\
| * Return names in list role assignmentsTom Cocozzello2016-01-271-6/+34
| | | | | | | | | | | | | | | | | | | | | | | | Utilize the new include names functionality added to list role assignments (GET /role_assignments?include_names=True). Which will return the names of the entities instead of their IDs. Change-Id: I6dc03baf61ef9354a8a259a9f17ff47ce1665ce7 Depends-On: I4aa77c08660a0cbd021502155938a46121ca76ef Closes-Bug: #1479569 Implements: blueprint list-assignment-with-names
* | Add "token revoke" for identity v3lin-hua-cheng2016-02-091-0/+19
| | | | | | | | | | Change-Id: Ie631600d02942fe6ce035f31af46abe44e543631 Closes-bug: #1543226
* | log take_action parameters in a single placeAkihiro Motoki2016-02-0229-607/+112
|/ | | | | | | | | | | | Previously each command logs take_action parameters explicitly by using @utils.log_method decorator or log.debug(). Some commands have no logging. This commit calls a logger in the base class and drops all logging definition from individual commands. Closes-Bug: #1532294 Change-Id: I43cd0290a4353c68c075bade9571c940733da1be
* Support non-interactive user password updateGuang Yee2016-01-111-2/+29
| | | | | | | | | | | | | | Currently user password update require interactive prompting of user's original password. This is problematic because we can't support non-interactive applications and therefore hinders automation. This patch make it possible by optionally accepting an '--original-password' argument. If specified, we would use it instead of prompting. DocImpact Change-Id: I2d994e8c2be949f7ae616ac1d1594fb94e1a27cd Closes-Bug: 1531360
* Merge "Implementation for project unset cmd for python-openstackclient."Jenkins2016-01-041-0/+56
|\
| * Implementation for project unset cmd for python-openstackclient.Jude Job2016-01-041-0/+56
| | | | | | | | | | | | | | | | | | This patch introduces a unit test class TestProjectUnset for testing unset cmd. Co-Authored-By: Steve Martinelli <stevemar@ca.ibm.com> Change-Id: Ib4a414d2313e3d37e48d1cb3639f064231aec508 Closes-Bug: #1486597
* | Enabling domain lookup for project set v3 commandJude Job2015-12-311-6/+2
|/ | | | | | | | | | | | | | Currently the domain option for `project set` attempts to set a new domain that owns the project. This is actually an action that is denied by keystone server. Instead, the domain option should be used as a lookup, to find projects that exist in the non-default domain. Co-Authored-By: Steve Martinelli <stevemar@ca.ibm.com> Closes-Bug: #1524456 Change-Id: I30a3812184fe262667e09baa106d2275c2cbb981
* Merge "Fix "sevice show" cannot catch NoUniqueMatch Exception"Jenkins2015-12-101-0/+5
|\
| * Fix "sevice show" cannot catch NoUniqueMatch ExceptionMin Min Ren2015-12-091-0/+5
| | | | | | | | | | | | | | Fix a bug for "service show" subcommand cannot cache NoUniqueMatch Exception Change-Id: I393c5417de0fef424618b08119ddbc8fea27e114 Closes-Bug: #1524305
* | Switch to ksa SessionDean Troyer2015-12-028-18/+18
| | | | | | | | | | | | | | | | * Change session imports to keystoneauth1 * Change keystoneclient.exception imports to keystoneauth1 * Change exceptions raised from internal API from keystoneclient to openstack.common Change-Id: I046d89f561d6fe04baae53726f9749d2e7fe2056
* | Merge "better format remote IDs for identity providers"Jenkins2015-11-151-3/+7
|\ \
| * | better format remote IDs for identity providersSteve Martinelli2015-11-121-3/+7
| | | | | | | | | | | | | | | | | | | | | | | | | | | remote-ids are a list, and we should format these values as such, rather than python representations of lists/arrays. Closes-Bug: 1478995 Change-Id: Ia6ced0fab2435b8cb486822c676c0dee32613abe
* | | Add capability to update description of an IdPlin-hua-cheng2015-11-121-2/+11
|/ / | | | | | | | | Change-Id: I854067642bbfde6fdf84b22b9cc1de8afc7767c0 Closes-Bug: #1515815
* | Merge "Mark arguments for 'credential' commands as required"Jenkins2015-11-041-16/+15
|\ \
| * | Mark arguments for 'credential' commands as requiredSean Perry2015-09-301-16/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | According to the [1], 'user_id', 'type', and 'blob' are all required arguments for 'credential set' but the code treats them as optional. Set the 'required' flag and remove logic supporting missing arguments. [1]: https://github.com/openstack/keystone-specs/blob/master/api/v3/identity-api-v3.rst#credentials-v3credentials "spec" Change-Id: I597c9616ad744385fc6dd92379feb03daec54458 Closes-Bug: #1418837
* | | Merge "remove url from v3 regions"Jenkins2015-10-291-19/+3
|\ \ \
| * | | remove url from v3 regionsSteve Martinelli2015-10-211-19/+3
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | the parameter "url" was removed from Keystone, it was only added for one release as part of an experimental support for adding service providers. BackwardsIncompatibleImpact Closes-Bug: 1506841 Change-Id: I7a62fbf1d9bfa8e6dd8d619e98c32b9860348d2e
* | | Merge "Change Identity API default version to 3"Jenkins2015-10-211-1/+1
|\ \ \ | |/ / |/| |
| * | Change Identity API default version to 3Steve Martinelli2015-10-091-1/+1
| |/ | | | | | | | | | | | | | | | | | | Devstack now issues all v3 OSC commands for our CI, and since v3 has everything v2 has, we should be able to switch to v3. Furthermore, most OSC users were initially using OSC since they were looking to exploit v3 keystone capabilities. Change-Id: If7de86946f6dd0f148aa5f526155cccf90adfcc3
* | Evaluate --inherited in role listRudolf Vriend2015-10-061-0/+5
|/ | | | | | | | the --inherited option was not being passed into keystoneclient Closes-Bug: #1502822 Change-Id: I48170dc67b23cc9b0665b1e0f38118eea952f131
* Merge "Remove backticks from help in role commands"Jenkins2015-09-191-1/+1
|\
| * Remove backticks from help in role commandsHidekazu Nakamura2015-09-191-1/+1
| | | | | | | | | | | | | | | | the docs and code had inconsistencies with how it references other arguments, lets just remove the backticks from around them. Change-Id: I43d17b07364e45387c6b9d86c2aca26eeea8ed93
* | Merge "Support listing users by group name"Jenkins2015-09-191-6/+4
|\ \ | |/ |/|
| * Support listing users by group nameSteve Martinelli2015-09-071-6/+4
| | | | | | | | | | | | | | | | | | | | | | Listing users within a group is not presently domain scoped. We do not use the domain info at all when the group option is present. A new --group-domain option is not needed since we cannot list users by --project and --group, they are mutually exclusive (as per the identity API). Closes-Bug: 1492916 Change-Id: I50f995ee4a03c2bdb21f2b5722546ab8fe786eb6
* | Use a common decorator to log 'take_action' activationJoshua Harlow2015-09-0126-113/+114
|/ | | | | | | | | Instead of duplicating the same log statement throughout the code, the same logic can be provided by a shared decorator that abstracts away the logging capability and unifies it behind a common function instead. Change-Id: Icc63bced7347c8bbf0299a4c5821425a10892a79
* Use correct domain to find project1.6.0Jamie Lennox2015-08-111-1/+1
| | | | | | | | | When adding a role to a group and project OSC is mistakenly using the group_domain to find the project which will fail if the group_domain != project_domain. Change-Id: I4c1bec9b3b183c755be121b91f40e026d707192b Closes-Bug: #1483520
* Merge "Inherited info/option when listing role assignment"Jenkins2015-08-081-4/+10
|\
| * Inherited info/option when listing role assignmentSamuel de Medeiros Queiroz2015-08-061-4/+10
| | | | | | | | | | | | | | | | | | | | Adds inherited information when listing role assignments. In addition, it makes possible to list only inherited ones by adding --inherited option. Change-Id: Idf889603d584716da95e2c7b4880142fbd8291c4 Closes-Bug: 1370546
* | Fixes inherited role assignments CRUD callsSamuel de Medeiros Queiroz2015-08-061-1/+1
|/ | | | | | | | | | The paremeter to Keystone Client was passed as 'inherited', when it should be 'os_inherit_extension_inherited'. Closes-Bug: #1482254 Change-Id: I1cb46add532223ef0b9620763b1047cc80e19ec0
* Add domain scoping in 'role assignment list'lin-hua-cheng2015-07-311-0/+6
| | | | | | | | Add ability to set which domain the user, group or project belong to. Change-Id: Idf6c7f23ab96261a73682226fb10ce5f0133924e Closes-Bug: #1472909
* Merge "Do not set default versions in parsed args"Jenkins2015-07-241-5/+3
|\
| * Do not set default versions in parsed argsTerryHowe2015-07-181-5/+3
| | | | | | | | | | | | | | | | | | | | | | Setting default versions in parsed args makes it so OCC cannot tell if the argument was parsed, an environment variable was set or it is just defaulted. In order to set api versions from OCC, it will have to be defaulted after processing OCC. Closes-Bug: #1453229 Change-Id: I4d065919397b783f3bdd4022c986c0234a7a16e6
* | Fix --os-auth-plugin in auth_with_unscoped_samlHugh Saunders2015-07-221-1/+1
| | | | | | | | | | | | | | | | The error message refers to --os-auth-plugin which is not a valid option. This patch changes that to --os-auth-type. Change-Id: I02ec0b7855131180bb8c674051930ebb51cb7303 Closes-Bug: #1477083
* | Merge "Make trustee/trustor/project searchable by ID"Jenkins2015-07-211-23/+8
|\ \
| * | Make trustee/trustor/project searchable by IDSteve Martinelli2015-07-201-23/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the previous implementation, we were always including the domain argument, which caused a lookup by name for trustee/trustor and project. By excluding it when not necessary, we do a search by ID in find_resources. Change-Id: Id756aeab522b5dccb2dc6b31d137a28514b0fdf6 Closes-Bug: 1473298
* | | Merge "Removes trailing blank in trust show"Jenkins2015-07-191-2/+2
|\ \ \ | |_|/ |/| |
| * | Removes trailing blank in trust showGilles Dubreuil2015-07-191-2/+2
| | | | | | | | | | | | | | | | | | | | | Previously a blank character at the end of the roles value would remain Change-Id: I0961a5f9fb4b270a6055ee69898eadee315e416a Closes-Bug: 1474707
* | | Merge "Rename endpoint type to interface"Jenkins2015-07-171-3/+2
|\ \ \
| * | | Rename endpoint type to interfaceTerryHowe2015-07-151-3/+2
| | | | | | | | | | | | | | | | | | | | Change-Id: I4e21d09bc747e8210f4f79a1d6c4c7ccf2f25d1c Closes-Bug: #1454392
* | | | Merge "only return endpoints that have url"Jenkins2015-07-171-2/+4
|\ \ \ \
| * | | | only return endpoints that have urlGuojian Shao2015-07-161-2/+4
| | |/ / | |/| | | | | | | | | | | | | | Change-Id: I97a502252c0c377fce573e92b83c0122812f6f80 Closes-Bug: #1474656
* | | | add --project-domain option for user v3Guojian Shao2015-07-161-6/+8
| |_|/ |/| | | | | | | | | | | | | | | | | | | | | | | user v3 create/set only support --project option, we need --project-domain to prevent collisions between project names exist. Change-Id: I2d62e5b9bb6df4c5c5a9542514faf2e4365bb18b Closes-Bug: #1475357
* | | Fix the way we call find_resource when only using IDSteve Martinelli2015-07-161-6/+18
|/ / | | | | | | | | Change-Id: I6fb08edd5499767863e0e67f363bcd9fff3aea60 Closes-Bug: 1475127
* | openstack catalog list always returns publicURL for internalURL and adminURLMartin Schuppert2015-07-081-1/+1
|/ | | | | | | | With this change 'openstack catalog list' returns the correspoding URL for publicURL, internalURL and adminURL in _format_endpoints . Change-Id: I5d946c9d70a2d3c22a7cc77067fec8e2e9aa4940 Closes-Bug: 1472629
* Merge "Add --os-endpoint-type cli optional argument"Jenkins2015-07-031-1/+6
|\
| * Add --os-endpoint-type cli optional argumentRoxana Gherle2015-07-021-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | User should be able to specify the endpoint type through a CLI optional argument/ENV variable setting. We will name this new optional argument: --os-endpoint-type (Env: OS_ENDPOINT_TYPE) and based on the value given, the service API will use that specific endpoint type. Possible values: public, admin, internal. DocImpact Closes-Bug: #1454392 Change-Id: Ife3d4e46b44c0ddcd712b1130e27e362545a9a29