summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Fix --shared/block-migration options in server migrate commandnewton-eolstable/newtonCedric Brandily2017-07-051-8/+8
| | | | | | | | | | | | | Currently, --shared-migration and --block-migration options effects are reversed: --block-migration requests a migration with share, --shared-migration a block-migration. This change corrects OSC implementation and clarifies arguments passed to novaclient (the root cause of the bug). Change-Id: Ib682cff0c44d3b1304670f8606907b1762d8b1e7 Closes-Bug: #1518059 (cherry picked from commit 694a24c3093f5b2595b63ccf988da7972e532084)
* Merge "Add server test for image and flavor lookups" into stable/newtonJenkins2017-03-061-15/+49
|\
| * Add server test for image and flavor lookupsDean Troyer2017-02-061-15/+49
| | | | | | | | | | | | | | | | | | | | | | | | Review Ia66e44e530799ce6531922dcf6a84e38528c8725 changes OSC's server commands to use the image client rather than compute clirnt (yay!) but we never really tested any of this in functional tests. This review adds a simple functional test (in the new style) to watch the client change; it passes locally for me without the client change, due to timing we went ahead and merged that first. Change-Id: I5529f412578c50090e70d17aa0129217bf803fed (cherry picked from commit 9ada3b529d60c174b7daa8db2a2fe5ebe619cced)
* | Merge "Use image client for images instead of compute" into stable/newtonJenkins2017-03-032-21/+22
|\ \ | |/
| * Use image client for images instead of computeArtom Lifshitz2017-02-062-21/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | With the deprecation of the Nova proxy APIs in microversion 2.36 [1], any operation that uses a microversion higher than 2.36 and works with images will fail because the /images endpoint will return 404. This patch updates openstackclient to query images using the image client in places where previously the compute client was used. [1] http://docs.openstack.org/developer/nova/api_microversion_history.html#id33 Change-Id: Ia66e44e530799ce6531922dcf6a84e38528c8725 Closes-bug: 1630161 (cherry picked from commit 98f803e0f0e622e7770cfab52e99168f6a68f427)
* | Fix OSC networking commands help errorsRui Chen2017-02-063-2/+38
|/ | | | | | | | | | | | | | | OSC networking commands need to authenticate to get service catalog, then decide to show nova-network or neutron command help message. Fake token and fake auth_type in prepare_to_run_command() casue os-cloud-config use AdminToken auth plugin, but pass all the auth information (include: username, password and so on) to it, that casue the class initialization error. Pop the fake token and url, then try to load auth plugin again to fix the issue. Change-Id: I8b140f0b0a60681fc2a35a013bb0c84ff8cb9589 Closes-Bug: #1650026 (cherry picked from commit 4d9da2c40ae02086258cfde852b297754d8085fa)
* Merge "Add relnotes for backports for 3.2.1 release" into stable/newton3.2.1Jenkins2016-12-093-0/+15
|\
| * Add relnotes for backports for 3.2.1 releaseDean Troyer2016-12-093-0/+15
| | | | | | | | Change-Id: Iba8d8c6d2d9ce4a665e527f23b2277ccc8b94dfa
* | Defer auth prompting until it is actually neededDean Troyer2016-12-093-3/+81
|/ | | | | | | | | | | | | | | | Auth option prompting happens waaaay to early in the default os-client-config flow, we need to defer it until adter the commands have been parsed. This is why ClientManager.setup_auth() exists, as it is not called until the first attempt to connect to a server occurs. Commands that do not require authentication never hit this. Also, required options were not being enforced. By doing this we handle when no authentication info is present, we fail on missing auth-url rather than attempt to prompt for a password (default auth is password). Closes-Bug: 1619274 Change-Id: Ia4eae350e6904c9eb2c8507d9b3429fe52418726 (cherry picked from commit 14dbfe44741b65c9e0514a34669f52de8629b1c0)
* Merge "Use project_domain_id only in password auth" into stable/newtonJenkins2016-12-091-0/+1
|\
| * Use project_domain_id only in password authBoris Bobrov2016-12-091-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The method being changed constructs domain-related parameters that will further be passed to the auth plugin. If project domain is not passed, the method sets it to the default domain. token_endpoint does not expect any information about domain, because it uses only a token and URL. Passing it to auth plugin causes an exception. Construct domain-related parameters only for specific plugins, such as password or totp. Change-Id: I13db3bbe31a0ed843e9f4528d37c768546e2bee9 Closes-Bug: 1642301 (cherry picked from commit e51a8d63747932f2ee4ffab02dfb0cd43e4a103d)
* | Merge "Use stable/newton branch in tox_install.sh and tox.ini" into ↵Jenkins2016-12-092-2/+2
|\ \ | | | | | | | | | stable/newton
| * | Use stable/newton branch in tox_install.sh and tox.iniMichal Pryc2016-11-242-2/+2
| |/ | | | | | | | | | | | | | | | | | | This change modifies tox_install.sh wrapper to pick dependencies for the correct branch for stable/newton release. Adjust upper-constraints.txt in tox.ini to pick dependencies from stable/newton branch. Change-Id: Ic63c363bf12e2f31b18271ef8bf26cbe5f5ca212
* | Mask passwords in debug logs for auth_config_hookMatt Riedemann2016-10-061-1/+5
|/ | | | | | | | | | | | | | | | | The auth config hook can have credentials in it so we have to mask the config before logging it. To avoid doing the work of masking the password if we aren't going to log it, there is a conditional put around the actual debug statement. Conflicts: openstackclient/common/client_config.py NOTE(mriedem): The conflict was due to imports. Change-Id: I8e626672ec94fc837610216bccb4354dbdedca17 Closes-Bug: #1630822 (cherry picked from commit cd1a412408f068aeef97c1ee368400307fce7733)
* Updated from global requirementsOpenStack Proposal Bot2016-09-202-2/+2
| | | | Change-Id: Id761ca7747f305057c39005cadf79522995d0071
* Update .gitreview for stable/newtonDoug Hellmann2016-09-021-0/+1
| | | | Change-Id: I19f6894d60e0ece6320b39d7f31ad6fac3c33dad
* Updated from global requirements3.2.0OpenStack Proposal Bot2016-08-301-1/+1
| | | | Change-Id: I2eedd06457425f48353e416b4d5145ce1967d57c
* Provide fallback prompt function for current osc-libDean Troyer2016-08-292-0/+8
| | | | | | | | | | Leaving the pw_func uninitialize in osc-lib turned out to be a bad idea as the test to prompt in setup_auth() doesn't check for a callback of None. Also, release note Change-Id: I8f875fa8a942d02a040238359ee22c603a4e5956
* Merge "Fix auth prompt brokenness"Jenkins2016-08-294-3/+102
|\
| * Fix auth prompt brokennessDean Troyer2016-08-294-3/+102
| | | | | | | | | | | | | | | | | | | | | | | | | | | | We start by fixing this in the already-present OSC_Config class so OSC can move forward. This change needs to get ported down into os-client-config in the near future, maybe even soon enough to make the client library freeze this week. * Add the pw-func argument to the OSC_Config (or OpenStackConfig) __init__() * When looping through the auth options from the KSA plugin look for any that have a prompt defined and do not have a value already, so ask for one. Closes-bug: #1617384 Change-Id: Ic86d56b8a6844516292fb74513712b486fec4442
* | Merge "Updated from global requirements"Jenkins2016-08-291-1/+1
|\ \
| * | Updated from global requirementsOpenStack Proposal Bot2016-08-291-1/+1
| | | | | | | | | | | | Change-Id: I90afe332e959ba8bbfb1f04cc84454d30a4cf4a8
* | | Merge "[docs] fix incorrect rst markups"Jenkins2016-08-291-22/+50
|\ \ \
| * | | [docs] fix incorrect rst markupsKATO Tomoyuki2016-08-241-22/+50
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The incorrect md-style markups break Sphinx builds, which do not properly generate web page. http://docs.openstack.org/developer/python-openstackclient/plugin-commands.html Change-Id: I36ae2cf922836cff42653283c0a683359bd91344
* | | | Merge "Clean imports in code"Jenkins2016-08-291-2/+2
|\ \ \ \
| * | | | Clean imports in codeCao Xuan Hoang2016-08-251-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In some part in the code we import objects. In the Openstack style guidelines they recommend to import only modules. http://docs.openstack.org/developer/hacking/#imports Change-Id: I2eb35dc53f0fdb61c31022bb70293d1df8aaf482
* | | | | Merge "Fix six typos"Jenkins2016-08-294-6/+6
|\ \ \ \ \ | |_|_|/ / |/| | | |
| * | | | Fix six typosAtsushi SAKAI2016-08-254-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | disassoiate => disassociate nmaes => names mutiually => mutually aviod => avoid CLustering => Clustering availble => available Change-Id: I84545bf46acfccc9dde3e85020700edb5a8375a6
* | | | | Cleanup after installAndreas Jaeger2016-08-281-1/+1
| |_|_|/ |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Tox tests are run on developer machines and on long lived slaves. We should not leave lots of directories like /tmp/python-openstackclient-tox_install-sdnltRu lying around. Instead delete the temporary directory after our run. Remove also an obsolete comment. Change-Id: I939eae82dba3287fd4e4086128ebf4609a0e0770
* | | | Merge "Restore default auth-type for token/endpoint"3.1.0Jenkins2016-08-251-2/+6
|\ \ \ \
| * | | | Restore default auth-type for token/endpointDean Troyer2016-08-241-2/+6
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | The split to osc-lib shell lost the detection of --os-token and --os-url to set --os-auth-type token_endpoint Closes-bug: 1615988 Change-Id: I248f776a3a7b276195c162818f41ba20760ee545
* | | | Updated from global requirementsOpenStack Proposal Bot2016-08-251-2/+2
| |/ / |/| | | | | | | | Change-Id: I87cd3d1aaf2d7e8706b4e89ef679c03de69cf611
* | | Updated from global requirementsOpenStack Proposal Bot2016-08-251-1/+1
|/ / | | | | | | Change-Id: I14eec90a7bfe8301b8698dd4337f3748c3732b78
* | Updated from global requirementsOpenStack Proposal Bot2016-08-241-1/+1
|/ | | | Change-Id: I53797c068d919027c7cfadcb245ab06eb9affc9e
* Merge "Fix post and periodic jobs"Jenkins2016-08-231-33/+33
|\
| * Fix post and periodic jobsAndreas Jaeger2016-08-221-33/+33
| | | | | | | | | | | | | | | | | | | | | | | | | | The usage of zuul-cloner in tox_install breaks post and periodic jobs. Update the script, it does not need to handle unconstrainted installs anymore (see tox.ini). There's no need to set ZUUL_BRANCH explicitely - it's set via the environment and branch is passed in as well. Note that this script is needed for the edit-constraints call, add a comment. Change-Id: I0077c986a17d6bb92791474e03d1e77776e9382f Closes-Bug: #1615430
* | Work around a version clash issue with os-client-config3.0.1Dean Troyer2016-08-221-2/+2
|/ | | | | | | Need to add the fixed_arguments arg to _validate_auth() so os-client-config 1.19.1 and 1.20.0 can call our version properly. Change-Id: I328e47ba2f8115e6b18bf1482fd4aa35056907a4
* Merge "Updated from global requirements"3.0.0Jenkins2016-08-212-6/+6
|\
| * Updated from global requirementsOpenStack Proposal Bot2016-08-212-6/+6
| | | | | | | | Change-Id: Ia8340e97b90f7497244f57c7ab7e46d7a461b896
* | Merge "Integ test cleanup"Jenkins2016-08-211-42/+91
|\ \ | |/ |/|
| * Integ test cleanupDean Troyer2016-08-181-42/+91
| | | | | | | | Change-Id: Ie58a7bec569421097e92a7ddf3cb164fc3f07413
* | Merge "Fix errors in volume set/unset image properties unit tests"Jenkins2016-08-191-3/+3
|\ \
| * | Fix errors in volume set/unset image properties unit testsHuanxuan Ao2016-08-111-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These errors may caused by copy-paste, we should specify a fake return value for get() method but not create() method. Create() mothod will not called in set/unset command. After specifing the return value of get(), we can use the fake volume in the assert. Change-Id: I1e0df4c28ece373168788c396e9082d565e36cc7
* | | Merge "Add support for domain specific roles"Jenkins2016-08-189-18/+505
|\ \ \ | |_|/ |/| |
| * | Add support for domain specific rolesHenry Nash2016-08-109-18/+505
| | | | | | | | | | | | | | | | | | | | | A role entity can now be specified as domain specific. Closes-bug: #1606105 Change-Id: I564cf3da1d61f5bfcf85be591480d2f5c8d694a0
* | | Merge "Add shell integration test"Jenkins2016-08-181-0/+561
|\ \ \
| * | | Add shell integration testDean Troyer2016-08-181-0/+561
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | These run next to unit tests, but unlike unit tests, they test the dependent libraries also. This is to detect incompatible breakage in those dependencies. The tests provide CLI-level input and verify the API calls being made via requests using requests_mock so the entire stack is tested. It is possible we want to run these separate from the unit tests. They do not belong in the functional tests as they do not require a functional cloud for testing. Depends-on: I426548376fc7d3cdb36501310dafd8c44d22ae30 Change-Id: I356956fcc4ff35191a6ad6a085b75cf370434b09
* | | | Merge "Fix up last-minute imports to use osc-lib"Jenkins2016-08-182-4/+6
|\ \ \ \
| * | | | Fix up last-minute imports to use osc-libDean Troyer2016-08-182-4/+6
| | | | | | | | | | | | | | | | | | | | Change-Id: I1ed2983cf574ebd565eeac4f8199fbc3a2e29c8e
* | | | | Merge "update requirements and test requirements"Jenkins2016-08-182-4/+3
|\ \ \ \ \
View Lorry Controller Status