summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Merge "Don't build releasenotes in normal docs build" into stable/newtonnewton-eolstable/newtonJenkins2016-09-121-1/+2
|\
| * Don't build releasenotes in normal docs buildMonty Taylor2016-09-021-1/+2
| | | | | | | | | | | | | | | | The release notes system exists in parallel and publishes to http://docs.openstack.org/releasenotes/os-client-config/. When we build them in the normal doc build, it causes problems for distro packagers. Change-Id: I6b084a1ad6836beac991d03c5f134203512150ac
* | Update .gitreview for stable/newtonDoug Hellmann2016-09-021-0/+1
|/ | | | Change-Id: I2dada3ca39683ca5ff0940b9f13503492c9fdb11
* Go ahead and handle YAML list in region_name1.21.1Monty Taylor2016-08-291-1/+4
| | | | | | | | The right thing is to use "regions" as a yaml list. However, it's even more right to be friendly to people - we emit a warning about region_name as a list anyway. Change-Id: Ia0b27ef8d1d52c655c2736a97bd5e59a4a2fe9d8
* Add prompting for KSA options1.21.0Dean Troyer2016-08-292-1/+59
| | | | | | | | | | | | | | | | Teach OpenStackConfig to prompt the user for KSA plugin options that have no value but have a prompt string defined. * Add pw_func argument to __init__() to be used as the callback for prompting the user. The default is None which skips the prompt step. * Add option_prompt() method to perform the checks for prompting, call the callback and save the returned value. This is public to handle cases where simply passing in a callback is insufficient for the prompt mechanism. Related-Bug: #1617384 Change-Id: I5faa86e94d6f71282ac270e2acfbd3016638c780
* Clean up vendor support listMonty Taylor2016-08-251-55/+29
| | | | | | | | | | | | The IBM Cloud isn't really a thing at that address yet (jumped the gun) OSIC probably shouldn't have been added either (turns out there are like 8 OSIC clouds currently, and they're test clouds. Also, update the location on the RegionOne clouds, and rename "Human Name" to "Location" - which is the useful information in that column anyway. Change-Id: I04451836330aacc3e2b91cfbe7d7d9bba7a47346
* Merge "Precedence final solution"1.20.1Jenkins2016-08-222-56/+200
|\
| * Precedence final solutionDean Troyer2016-08-202-56/+200
| | | | | | | | | | | | | | | | | | | | * Revert most of 'fixed_argparse change' from 1.19.1 * Create a new _validate_auth_correctly() method that contains the logic from 1.19.0 * Create a new get_one_cloud_osc() method for use by OSC to get the correct argument precedence without disrupting anyone else Change-Id: Iae86cc4e267f23dbe8d010688a288db5514f329d
* | Add support for configuring split-stack networks1.20.0Monty Taylor2016-08-185-4/+65
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Some clouds, like OSIC and v1 of DreamCompute, have a split stack network. This means that a single Neutron Network has both an IPv4 and an IPv6 subnet, but that the IPv4 subnet is a private/RFC-1918 and the IPv6 subnet is a Global network. As any inferrance information is attached to the Network, it's impossible to properly categorize IP addresses that are on the Server in such a scenario. Add support for ipv4 and ipv6 versions of the current routes_externally config value, with each of them defaulting to the value of the un-specialized routes_externally. Change-Id: I1e87a1423d20eac31175f44f5f7b38dfcf3a11cb
* | Pop domain-id from the config if we infer valuesMonty Taylor2016-08-112-0/+3
|/ | | | | | | | | | | | | | | | | | | | | | | | If the user specifies a project_{name,id}, then we currently infer that a domain_{name,id} is meant to be shorthand for user_domain_{name,id} and project_domain_{name,id}. However, in other contexts, domain_id is a perfectly valid value to pass to the auth plugins - such as when doing domain-scoped activities. The problem that was uncovered by the correction of argument precedence is that we didn't pop the domain-id out of the root config dict, so if a user set OS_DOMAIN_ID in the environment, we were happily setting the user and project versions, but then leaving domain_id set as well. This then meant that when we do the pass to pull valid arguments from the root dict to the auth dict, we also pulled in domain_id - which led to the error: AuthorizationFailure: Authentication cannot be scoped to multiple targets. Pick one of: project, domain, trust or unscoped Popping the value from the root dict causes the things to work as documented. Change-Id: I6d208e5ec4115d2e72d30b2fedc90a81ea754d5a
* Update Internap informationMathieu Gagné2016-08-052-3/+5
| | | | | | | * Add sin01 and sjc01 regions * Add support for Glance v2 Change-Id: Iaf4ad7c807a28c24040b928f65f4aadc1a234d6e
* Merge "Add test for precedence rules"1.19.1Jenkins2016-08-051-0/+53
|\
| * Add test for precedence rulesMonty Taylor2016-08-051-0/+53
| | | | | | | | | | | | This should cover both the OSC and the ansible incoming use cases. Change-Id: I3fdc83837692d31c5579d91892a387a5d1023785
* | Merge "Pass the argparse data into to validate_auth"Jenkins2016-08-051-13/+47
|\ \ | |/
| * Pass the argparse data into to validate_authMonty Taylor2016-08-041-13/+47
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We have two contradictory precedence needs that are impossible to satisfy because we're losing knowledge of the source of data as we build the ultimate dict of data. If we carry the argparse data along in a separate bucket for longer, we can check to see if it's there so that it can win, but so that in situations where kwargs is complex and contains both a top-level and an auth dict we don't assume that the values that are not in the auth dict came from argparse. By doing this, we can establish that precedence for auth args is: - argparse - auth dict - top level kwargs Change-Id: I9eca5937077f5873f7896b6745951fb8d8c4747c
* | Merge "Revert "Fix precedence for pass-in options""Jenkins2016-08-051-5/+5
|\ \ | |/
| * Revert "Fix precedence for pass-in options"Monty Taylor2016-08-041-5/+5
| | | | | | | | | | | | This reverts commit 05b3c933b34e9cec9eb859a15392862918b3eb5f. Change-Id: I07dd701ca911cd12701519d2e6d624c69baa0848
* | Merge "Add release notes for 1.19.0 release"1.19.0Jenkins2016-08-034-0/+23
|\ \
| * | Add release notes for 1.19.0 releaseDean Troyer2016-08-034-0/+23
| | | | | | | | | | | | Change-Id: I92ffcf611d31f7a4f11e5228022ea64864823389
* | | Merge "Add the new DreamCompute cloud"Jenkins2016-08-033-0/+30
|\ \ \ | |/ / |/| |
| * | Add the new DreamCompute cloudMonty Taylor2016-08-033-0/+30
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | It does things the happy way - direct routing of IPv4 and IPv6. However, it's a respin, so we're naming it dreamcompute rather than dreamhost so that users don't get broken by the shift. Remove dreamhost from the docs - people looking at documentation should be using the new region. Change-Id: I92eb38635c4389d2e9326fab038137a673497fa8
* | | Merge "Fix precedence for pass-in options"Jenkins2016-08-031-5/+5
|\ \ \ | | |/ | |/|
| * | Fix precedence for pass-in optionsDean Troyer2016-08-021-5/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The passed-in argparse Namespace is flat and does not have an 'auth' dict, all of the auth options are in the top level. If the loaded dict from clouds.yaml as an 'auth' dict, that used to totally override any passed-in options. This is backwards. Make the passed-in auth options always win over clouds.yaml as this is the only way to change/override those values from the command line. Change-Id: Ic2752a396d45deeda19a16389437679829e0844d
* | | Merge "Refactor fix magic in get_one_cloud()"Jenkins2016-08-031-34/+37
|\ \ \
| * | | Refactor fix magic in get_one_cloud()Dean Troyer2016-06-181-34/+37
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Extract the magic argument fixes from get_ne_cloud() so they can be extended or modified. Needed by OSC in order to maintain compatibility due to the much earlier loading of auth plugins than before. Have I mentioned that really fouled things up for OSC? Change-Id: I22cd890f9cbd605dcd615f82b3e65c58f52ff114
* | | Merge "Add support for listing a cloud as shut down"Jenkins2016-08-032-1/+6
|\ \ \ | | |/ | |/|
| * | Add support for listing a cloud as shut downMonty Taylor2016-07-162-1/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | We've had one vendor cloud go away in the past, and there is one existing deprecated cloud currently. Add support for providing the user with an informative error message in the case where they attempt to use such a cloud. Change-Id: I894e0c0a4786e60fce1238bb2883828e89d44b01
* | | Merge "Add support for deprecating cloud profiles"Jenkins2016-08-033-0/+19
|\ \ \ | |/ /
| * | Add support for deprecating cloud profilesMonty Taylor2016-07-093-0/+19
| |/ | | | | | | | | | | | | | | | | | | | | | | | | We've had HP shut down their public cloud, and DreamHost has recent spun up a new cloud with a different operational profile and auth_url that is essentially a replacement cloud. That means people using the old information need to do things, so we need to be able to communicate that to them. Add support for adding a deprecated status to a vendor profile, as well as a verbose message explaining what the user may do to remediate. Change-Id: I19b67d7cd71fba2d9da0e3a6adb2d229ead65396
* | Update citycloud to list new regionsMonty Taylor2016-08-022-1/+7
|/ | | | | | Frankfurt, Buffalo and Los Angeles - oh my! Change-Id: I17d6f46de2a9af82f221b971a359d53eb471f8fa
* Merge "Reword the entries in the README a bit"1.18.0Jenkins2016-06-021-5/+6
|\
| * Reword the entries in the README a bitMonty Taylor2016-06-021-5/+6
| | | | | | | | | | | | | | Wanted to make each section a little better, but also to start to indicate that legacy clients should really not be your first choice. Change-Id: I26e08d037c7b28ced22a2a0126693d7e3e779f58
* | Merge "Add shade constructor helper method"Jenkins2016-06-023-0/+49
|\ \ | |/
| * Add shade constructor helper methodMonty Taylor2016-06-023-0/+49
| | | | | | | | | | | | | | | | | | We have helper factory methods for REST Client, legacy client and OpenStack SDK all with the same interface ... we might as well have one for shade too. It makes documenting and talking about the simple case of all of them easy. Change-Id: I046da85ae4a3e2a6333223921d5ae9ce3673121d
* | Merge "Rename session_client to make_rest_client"Jenkins2016-06-013-20/+26
|\ \ | |/
| * Rename session_client to make_rest_clientMonty Taylor2016-06-013-20/+26
| | | | | | | | | | | | | | | | | | | | While writing some docs, it became clear that session_client was just a horrible horrible name and that I'm a bad person. Rename it so that we can make docs that make humans happy. Also, move the REST client section of the README up a bit. Change-Id: I1a27853e3031489da5916308a76f19bc72185d24
* | Merge "Add helper method for OpenStack SDK constructor"Jenkins2016-06-013-0/+54
|\ \ | |/
| * Add helper method for OpenStack SDK constructorMonty Taylor2016-06-013-0/+54
| | | | | | | | | | | | | | | | openstacksdk already has a helper method for dealing with occ, but if a user is already using the occ helper methods, there is no reason we should not provide them an easy path to using the SDK. Change-Id: I1040efb94385fdac0aa02ac960ba95089b954377
* | Merge "Trivial: Remove 'MANIFEST.in'"Jenkins2016-06-011-6/+0
|\ \
| * | Trivial: Remove 'MANIFEST.in'ChangBo Guo(gcb)2016-05-071-6/+0
| | | | | | | | | | | | | | | | | | | | | Everything in this file is automatically generated by pbr. There appears to be no good reason to keep it around. Change-Id: I73eb120dedbdb6685862d26493fc178e6dee1353
* | | Merge "Trivial: remove openstack/common from flake8 exclude list"Jenkins2016-06-011-1/+1
|\ \ \
| * | | Trivial: remove openstack/common from flake8 exclude listChangBo Guo(gcb)2016-05-071-1/+1
| |/ / | | | | | | | | | | | | | | | | | | | | | openstack/common was used to kepp copied files from oslo-incubator, we don't use oslo-incubator stuff, so remove it. Change-Id: Id426c41e6ae277ed1f829820771d5ffc31a81166
* | | Merge "drop python3.3 support in classifier"Jenkins2016-06-011-1/+0
|\ \ \
| * | | drop python3.3 support in classifierChangBo Guo(gcb)2016-05-071-1/+0
| |/ / | | | | | | | | | | | | | | | | | | We don't run python 3.3 CI jobs anymore, so just drop it from classifier. Change-Id: I871269ae54067aae40f5dc06affbd4354104ee91
* | | Add missing "cloud" argument to _validate_auth_kscAlvaro Lopez Garcia2016-05-231-2/+2
| |/ |/| | | | | | | | | | | The function _validate_auth_ksc was missing a "cloud" parameter so the exception formatting failed as it could not find that variable. Change-Id: Ia1caaa29fcb14d6ce7c16de1f78bbcae6c24adb0
* | Workaround bad required params in troveclientMonty Taylor2016-05-091-0/+7
|/ | | | | | | | | troveclient requires username and password as parameters to the Client object, but if a Session is passed (like we do) that's not needed. A patch has been submitted to troveclient, but until that has been released, simply send None to both parameters. Change-Id: Ie130a4e83cceb7cab69bfbeb559493d195ef35e1
* Merge "Fix formatting in readme file"Jenkins2016-05-021-0/+2
|\
| * Fix formatting in readme fileIlya Shakhat2016-04-221-0/+2
| | | | | | | | Change-Id: Ifa37d38b3c7689f703c7129459b15a367e2aafff
* | Merge "Remove discover from test-requirements.txt"Jenkins2016-05-021-1/+0
|\ \ | |/ |/|
| * Remove discover from test-requirements.txtThomas Bechtold2016-04-111-1/+0
| | | | | | | | | | | | | | discover is only needed for python 2.6 which is no longer supported. Change-Id: I8faeb05def94ac4adb2fe870fe141678dbd412ae