summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* OpenDev Migration Patchpike-eolstable/pikeOpenDev Sysadmins2019-04-191-1/+1
| | | | | | | | | | | | | | | This commit was bulk generated and pushed by the OpenDev sysadmins as a part of the Git hosting and code review systems migration detailed in these mailing list posts: http://lists.openstack.org/pipermail/openstack-discuss/2019-March/003603.html http://lists.openstack.org/pipermail/openstack-discuss/2019-April/004920.html Attempts have been made to correct repository namespaces and hostnames based on simple pattern matching, but it's possible some were updated incorrectly or missed entirely. Please reach out to us via the contact information listed at https://opendev.org/ with any questions you may have.
* import zuul job settings from project-configpike-em1.28.1Vieri2018-08-211-0/+8
| | | | | | | | | | | | | | | | | | | | This is a mechanically generated patch to complete step 1 of moving the zuul job settings out of project-config and into each project repository. Because there will be a separate patch on each branch, the branch specifiers for branch-specific jobs have been removed. Because this patch is generated by a script, there may be some cosmetic changes to the layout of the YAML file(s) as the contents are normalized. See the python3-first goal document for details: https://governance.openstack.org/tc/goals/stein/python3-first.html Change-Id: If2b0ccc4c5544bbd2d88c4bcbfa0a7960c0e1a43 Story: #2002586 Task: #24321
* Do not apply format expansions to passwordsJohn Dennis2018-06-142-2/+64
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | get_one_cloud() and get_one_cloud_osc() iterate over config values and try to expand any variables in those values by calling value.format(), however some config values (e.g. password) should never have format() applied to them, not only might that change the password but it will also cause the format() function to raise an exception if it can not parse the format string. Examples would be single brace (e.g. 'foo{') which raises an ValueError because it's looking for a matching end brace or a brace pair with a key value that cannot be found (e.g. 'foo{bar}') which raises a KeyError. It is not reasonsable to try to escape any braces because: 1) Escaping all braces breaks valid use of the format string syntax. 2) Trying to determine exactly which braces should be escaped and which should be preserved is a daunting task and likely would not be robust. 3) Some strings might look like valid format syntax but still should be escaped (e.g. "foo{bar}", if this appeared in a password we wouldn't escape it and there would be a key error on the 'bar' key. 4) In general passwords should never be modified, you never want to apply formatting to them. The right approach is to maintain a list of config values which are excluded from having formatting applied to them. At the moment that list just includes 'password' but perhaps down the road other exceptions might crop up. This patch follows this approach, the list of excluded values can easily be updated if others are discovered. Change-Id: I187bdec582d4c2cc6c7fda47a1538194137c616b Closes-Bug: 1635696 Signed-off-by: John Dennis <jdennis@redhat.com> (cherry picked from commit c7243f1747b679c6bc4df8aead8133ec25898182)
* Fix sphinx docs job on stable/pikeJulie Pichon2018-06-142-1/+7
| | | | | | | | | | | | | | | | | | | It looks like the CLIENT_NAME env variable is not passed onto the script when running the tox job. Also include the fix from I35e9ede7103031ea3ea45e35d743e63059fb8d0b: build-openstack-sphinx-docs jobs fail on stable branches because the new pip version 10 changed a Warning into an Error: When calling "pip install" without any package name, the command now fails. tox_install.sh is being called during docs job without any package passed to pip. Note that this only affects stable/pike and older, so there is no patch in master to cherry-pick this from. Newer branches use a different way to set up the tox environment. Change-Id: I8b413799fd3d1c84a6a27222641267b4abad5651
* Merge "Update UPPER_CONSTRAINTS_FILE for stable/pike" into stable/pikeJenkins2017-08-231-1/+1
|\
| * Update UPPER_CONSTRAINTS_FILE for stable/pikeOpenStack Release Bot2017-07-281-1/+1
| | | | | | | | Change-Id: Idb8c3110e939c3135f712c2fb95c2f67426f6c40
* | Updated from global requirementsOpenStack Proposal Bot2017-08-113-21/+28
|/ | | | Change-Id: Ic0983d6acb014697d6c376e01bfa1038a450f1eb
* Update .gitreview for stable/pikeOpenStack Release Bot2017-07-281-0/+1
| | | | Change-Id: Ie2c9535d8152946fd7b2ba12240fb4b9eca28b8b
* use openstackdocstheme html context1.28.0Doug Hellmann2017-06-291-1/+8
| | | | | | | | | | Set some of the new config values and enable openstackdocstheme as an extension so it will inject values into the page context as it writes each documentation page. This ensures the pages link to the right bug tracker, etc. Change-Id: Id9cc61e81aa43f4b69883d338090716005477d0a Signed-off-by: Doug Hellmann <doug@doughellmann.com>
* switch from oslosphinx to openstackdocsthemeDoug Hellmann2017-06-223-4/+8
| | | | | Change-Id: Ie45909df0b5a118d0200a1ee71277f4dbfe41d08 Signed-off-by: Doug Hellmann <doug@doughellmann.com>
* turn on warning-is-error in documentation buildDoug Hellmann2017-06-222-0/+2
| | | | | Change-Id: I18cdecec84f8dd5f11741ac1ffc35630f7eb64b8 Signed-off-by: Doug Hellmann <doug@doughellmann.com>
* rearrange existing documentation to follow the new standard layoutDoug Hellmann2017-06-2211-488/+520
| | | | | | Depends-On: Ia750cb049c0f53a234ea70ce1f2bbbb7a2aa9454 Change-Id: Ib68812eda4ab0ce09c6d438f9f236111d61ef38b Signed-off-by: Doug Hellmann <doug@doughellmann.com>
* Merge "Revert "Revert "Use interface not endpoint_type for keystoneclient"""Jenkins2017-05-252-11/+14
|\
| * Revert "Revert "Use interface not endpoint_type for keystoneclient""Monty Taylor2017-05-132-11/+14
| | | | | | | | | | | | | | | | | | | | Unrevert the endpoint_type/interface patch. But this time around, put in a check for API version 2.0 and only apply the interface arg if it's for v3. This reverts commit 4493871824839782846c3825c754895876f8d08e. Change-Id: Ib347ec686d4d01788ee943c4c4f809aad06d9ccf
* | Merge "Don't pop from os.environ"Jenkins2017-05-251-11/+13
|\ \
| * | Don't pop from os.environMatthew Booth2017-05-251-11/+13
| |/ | | | | | | | | | | It's rude to other users and subsequent callers. Change-Id: I7789f381c99311bfd1c1e0a9869cbacbc96b17d6
* | Merge "Keep a singleton to support multiple get_config calls"Jenkins2017-05-251-5/+8
|\ \
| * | Keep a singleton to support multiple get_config callsMonty Taylor2017-05-161-5/+8
| | | | | | | | | | | | | | | | | | | | | | | | | | | We are destructive to os.environ in the OpenStackConfig constructor- so it really should only ever be called once. Make sure get_config does this. Change-Id: I279bdf68408a807ec18fba634df3769c9b8fc4dc Closes-Bug: #1691294
* | | Merge "Add ironicclient to constructors list"Jenkins2017-05-251-0/+1
|\ \ \
| * | | Add ironicclient to constructors listMonty Taylor2017-05-121-0/+1
| | | | | | | | | | | | | | | | Change-Id: I29db3c830759a80d8ea9f0d93a213b4bae4c8b59
* | | | Merge "Add helper method to fetch service catalog"Jenkins2017-05-251-0/+4
|\ \ \ \ | |/ / / | | | / | |_|/ |/| |
| * | Add helper method to fetch service catalogMonty Taylor2017-05-081-0/+4
| |/ | | | | | | | | | | Grabbing the catalog is weird. OCC should help. Change-Id: I6e7176568311c1f0e644a8e8876f56c3e153d6e8
* | Revert "Use interface not endpoint_type for keystoneclient"Jordan Pittier2017-05-132-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 38e5eba621e48d74c05315da2b89e6c801f4c43f. This patch introduced a bug when using Keystone v2. With this patch, the following works: python -c "import os_client_config; print(os_client_config.make_client('identity', auth_url='http://localhost/identity_admin', username='admin', project_name='admin', password='testtest', identity_api_version='3').roles.list())" But changing identity_api_version from 3 to 2.0 raises an exception. Without this patch, both 3 and 2.0 works. Change-Id: I8d2ad71ff51a08af1166d36805b740ea272939ed
* | Fix interactions with keystoneauth from newtonMonty Taylor2017-04-283-6/+39
|/ | | | | | | | | keystoneauth in newton did not have app_name or app_version as Session parameters. Although it isn't a super common combination, user agent strings aren't a reason to break something. Add a simple workaround. Change-Id: Ib5774389fefdbc190a4b78dd6784c8006afbb270
* Merge "Make _fix_argv() somewhat compatible with Argparse action='append'"1.27.0Jenkins2017-04-252-1/+15
|\
| * Make _fix_argv() somewhat compatible with Argparse action='append'Jordan Pittier2017-04-232-1/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Python Argparse supports the 'append' action [1] which is super handy to allow a user to repeat several times the same argument, each time with different values. This doesn't work with occ that tries to "fix argv" but raises this error: os_client_config.exceptions.OpenStackConfigException: The following options were given: '--foo,--foo' which contain duplicates except that one has _ and one has -. There is no sane way for us to know what you're doing. Remove the duplicate option and try again This patch tweak the _fix_argv() function so that it doesn't explode if the duplicate option has no '_' not '-' in its name. Change-Id: I4f06b6aff8d3ab1df45637399bc3a9b4b61764a9 Related-bug: #1685630
* | Merge "Add designateclient to constructors list"Jenkins2017-04-251-0/+1
|\ \ | |/ |/|
| * Add designateclient to constructors listMonty Taylor2017-03-211-0/+1
| | | | | | | | | | | | | | We've had this mapping over in shade for a while. No sense in keeping the fun all to ourselves. Change-Id: Icb2b98b621cfa8cff86c534bfba8f5de2c818e95
* | Merge "Add ability to pass in user_agent"Jenkins2017-04-185-7/+48
|\ \
| * | Add ability to pass in user_agentMonty Taylor2017-04-185-7/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | keystoneauth supports adding a user_agent info to the Session and Adapter via app_name. Allow users to add app_name/app_name and versions as desired. Also, add os-client-config into additional_user_agent. As an example, once this is landed and plumbed through shade, nodepool will set app_name='nodepool' and we'll have: User-Agent: nodepool/0.4.0 os-client-config/1.26.1 shade/1.19.1 keystoneauth1/2.18.0 python-requests/2.13.0 CPython/2.7.12 Change-Id: I1eb4dbd2587dcbe297b5c060c3c34b68ef51ef5e
* | | Docs: add a note about rackspace API keysJim Rollenhagen2017-03-291-0/+8
|/ / | | | | | | | | | | | | Some users are forced to use these instead of passwords (whether because they use 2-factor auth or by policy). Document it so they know how. Change-Id: I558c2e8d3e8b0fad0a96a361232f14443e82a35f
* | Remove out of date commentMonty Taylor2017-03-281-1/+0
| | | | | | | | Change-Id: I8a26f5952456a96429ff1413b90aef3091a8b5bf
* | Stop special-casing idenity catalog lookupsMonty Taylor2017-03-282-38/+17
| | | | | | | | | | | | | | | | | | We have a special case to work around a thing we're pretty sure keystoneclient used to do but apparently doesn't do anymore. Remove the workaround. Co-Authored-By: Jamie Lennox <jamielennox@gmail.com> Change-Id: I873ad91816150b593d4aef13dcd1520e8c91b22a
* | Merge "Add support for bailing on invalid service versions"Jenkins2017-03-223-17/+93
|\ \
| * | Add support for bailing on invalid service versionsMonty Taylor2017-03-093-17/+93
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | At least for cinder for now, allow a consumer of get_legacy_client to express the minimum version they find acceptable. This will use cinder_client logic to figure out the version from the url. As a follow on, expand this to all of the clients and make it support microversions for the clients that support microversions. (Right now it's just going to be major versions, so min_version=1 will throw an exception if the cinder service returns a v1 endpoint. Also, because we override the volume/volumev2/volumev3 service type stuff, we need to do extra special logic in get_session_endpoint to try all three in the case where do not have a configured api_version. Change-Id: I7b6b3588fec9a6be892cf20d344667f0b9a62f0a
* | | OVH supports qcow2Clark Boylan2017-03-132-2/+1
| | | | | | | | | | | | | | | | | | | | | OVH supports qcow2 images too. Update the docs and vendor json file to reflect this. You can continue to use raw images just fine as well. Change-Id: Ic7dc4c70c681947a0475bbabf5621672825dfb3c
* | | Use interface not endpoint_type for keystoneclientMonty Taylor2017-03-092-3/+3
| |/ |/| | | | | | | | | keystoneclient wants the interface argument. Change-Id: I5898d8621259256f962fc006df38049d0cb059f8
* | Merge "modify test-requirement according to requirements project"Jenkins2017-03-091-3/+3
|\ \
| * | modify test-requirement according to requirements projectxhzhf2017-03-091-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | the OpenStack requirements project has modify version requirement of docutils/oslosphinx. The link is below https://review.openstack.org/#/c/418772/ So modify test-requirement like other project Closes-Bug: #1666149 Change-Id: I145ba596926cac0efab75fb4a1548eea338a2d5a
* | | Merge "Pass ironic microversion through from api_version"Jenkins2017-03-072-0/+10
|\ \ \ | | |/ | |/|
| * | Pass ironic microversion through from api_versionMonty Taylor2017-02-282-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | If someone sets baremetal_api_version to 1.29 right now, we don't really do anything with that information. Pass it through to the constructor for ironicclient in get_legacy_client(). Change-Id: I470fbb8852eac7d5cb35aef549ac591d63f3636f
* | | Merge "Add ability to skip yaml loading"Jenkins2017-03-074-5/+51
|\ \ \ | |/ /
| * | Add ability to skip yaml loadingMonty Taylor2017-02-274-5/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Added a flag, 'load_yaml_config' that defaults to True. If set to false, no clouds.yaml files will be loaded. This is beneficial if os-client-config wants to be used inside of a service where end-user clouds.yaml files would make things more confusing. Change-Id: Idbc82bb931e9edf1bbcc575237c0e202e219c218
* | | Merge "Remove the keystoneclient auth fallback"Jenkins2017-03-044-77/+13
|\ \ \
| * | | Remove the keystoneclient auth fallbackMonty Taylor2017-02-154-77/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | OSC doesn't use this codepath anyway (to my knowledge) and it masks errors in exceptionally strange ways. Change-Id: I15ec5aacb037813a98ac9ea8e9504a5d1cc90837
* | | | Merge "Add support for overriding mistral service type"Jenkins2017-03-043-1/+11
|\ \ \ \ | |/ / /
| * | | Add support for overriding mistral service typeMonty Taylor2017-02-153-1/+11
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | The mistral team copied the heinous pervsion that the cinder team propagated upon the world and appended a version to their service_type. That's ok - there is nice copy-pastable code here we can use to prevent users from feeling the pain. Change-Id: Icf280f932014e4d9abeab3e944aece125988562e
* | | [Fix gate]Update test requirementricolin2017-03-021-1/+1
|/ / | | | | | | | | | | | | | | | | Since pbr already landed and the old version of hacking seems not work very well with pbr>=2, we should update it to match global requirement. Partial-Bug: #1668848 Change-Id: I09ae994782889aae05250a8e5bf9f5b630b2d502
* | Add helper scripts to print version discovery infoMonty Taylor2017-02-152-0/+145
| | | | | | | | | | | | | | These are simple scripts I made to investigate things. Each show the version discovery info for all of the clouds in a clouds.yaml. Change-Id: I742a59c737c53c05851015b9734c7aa85a5466ca
* | Add support for indicating required floating IPsMonty Taylor2017-02-087-3/+25
|/ | | | | | | | | | | | | | | | | | | Some clouds require that users add a floating IP to a server if the user wants that server to be able to talk to things that are not on the cloud. Some clouds do not require this and instead give servers a directly attached IP. The only way a user can know is to boot a server, then ask neutron for the port associated with that server, then find the network the port came from and then try to infer whether or not that network has the ability to route packets northbound. Of course, networks don't actually communicate that quality directly, (router:external doesn't mean a network routes externally, it means the network can have a router attached to it to provide floating ips) so it's still hit and miss. Where we can, save the user the stress and strain of not knowing how their cloud wants them to get an externally routable IP. Change-Id: I1baf804ce28bc1997b2347c4648c5cc56c750ead
View Lorry Controller Status