| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
|\ \
| |/
|/| |
|
| |
| |
| |
| |
| |
| |
| | |
Zuul no longer requires the project-name for in-repo configuration.
Omitting it makes forking or renaming projects easier.
Change-Id: I0593533e6b1c3dedd9a8c6180af2d2b30942f4ce
|
| |
| |
| |
| | |
Change-Id: Ic9f04683292438e56d8b61ab8528ed2d4f3f9d34
|
|\ \ |
|
| | |
| | |
| | |
| | | |
Change-Id: I0e66c5ef109dc237455eedc59ae1c3af0bbf8b47
|
|/ /
| |
| |
| | |
Change-Id: I9625bc6ae94f37b8a07206ae931ed1ec60f75198
|
|\ \ |
|
| | |
| | |
| | |
| | | |
Change-Id: I6f0d1ff2c139d77511937326d0d050135b97e0e2
|
|\ \ \
| |/ / |
|
| |/
| |
| |
| |
| |
| | |
Get ourselves up to date with the latest in PTI hotness.
Change-Id: Ied98273dcfa0b9622d603a8e9a8e2146fb1e081f
|
|\ \
| |/
|/| |
|
| |
| |
| |
| | |
Change-Id: I599156ee4e2ff52e4db5669fce6cacd22447c3b0
|
|\ \
| |/
|/| |
|
| |
| |
| |
| | |
Change-Id: I760104d4a94f67da4677e1ecf8f47150605d4d33
|
|\ \ |
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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>
|
| |
| |
| |
| |
| |
| |
| |
| | |
The service type overrides in get_service_type() fail if the
API version keys are not present in the config dict, which happens
when CloudConfig is created without reading defaults.
Change-Id: I8d035cfd1afc1cad01ceac7cd643568e94897e27
|
|/
|
|
|
|
|
| |
They dont' work, and we need this patch to land to land the patch to
make them work.
Change-Id: I14584aebdfbecc9b7fea071048f07cc9b804f597
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Release notes are version independent, so remove version/release
values. We've found that projects now require the service package
to be installed in order to build release notes, and this is entirely
due to the current convention of pulling in the version information.
Release notes should not need installation in order to build, so this
unnecessary version setting needs to be removed.
This is needed for new release notes publishing, see
I56909152975f731a9d2c21b2825b972195e48ee8 and the discussion starting
at
http://lists.openstack.org/pipermail/openstack-dev/2017-November/124480.html
.
Change-Id: I8eca3dae9db90d7c592eb5bcebb2273a81d1a571
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
shade provides a project-template to run its functional tests against
tips of key libraries, such as os-client-config and keystoneauth.
Add these to os-client-config so we can ensure we don't break anything.
Change-Id: I9619fc09ca7c0f6fd3021d3b5194131ad172ad16
Depends-On: I544fdf515f8df38d73cee64a6de799ab384a326e
|
|/
|
|
|
| |
Depends-on: Ia8c08be81605da885b9eee799fc58129305dfc41
Change-Id: Id331adbc5862b651f2754b3cb53f2bccee1be46c
|
|
|
|
|
|
| |
The word is 'router'.
Change-Id: Icf27d930f877d57f2b09a4e4e201c89f1d64b2a8
|
|
|
|
|
|
|
|
|
| |
In some more complex clouds there can not only be more than one valid network
on a server that NAT can attach to, there can also be more than one valid
network from which to get a NAT address. Allow flagging a network so that it
can be found.
Change-Id: I3d8dd6d734a1013d2d4a43e11c3538c3a345820b
|
|
|
|
|
|
|
|
|
|
|
|
| |
Breaking shade with changes to os-client-config is bad. Add shade's
unittest jobs that are configured to work against os-client-config
master.
Also move the legacy osc dsvm job here so we can remove it from
project-config.
Change-Id: Ie3e9a4deca1d74b94e810e87e130706fe15fe2c9
Depends-On: I5b411be5c5aa43535fa89a51d6099aadd7a8ea60
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Using make_rest_client on clouds that put unversioned endpoints in the
catalog results in incorrectly set up adapters.
Add the plumbing to get_session_client to pass version args to
keystoneauth. Then use that from make_rest_client.
Change-Id: I69ad746f672ef0b12680e9db3c7b0c691c9f87e4
|
|\ \ |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
If there is only one cloud and that cloud is envvars, things work as
expected. If there is only one cloud in clouds.yaml and no envvars
cloud, we throw an error, even though it should be obvious which cloud
was intended.
Change-Id: Ia49d0fb2cc7dca36476d0e5ae3fe2b2aa1209e59
|
|\ \ \
| |_|/
|/| | |
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
In ansible/ansible#28746 it was reported that there are times
when a p_opt is getting here that does not have a prompt
attribute. Protecting against that is fairly easy to do.
Change-Id: Ia02528f4a107893e480135bc214aa156b8684507
Closes-Bug: #1717906
|
|\ \ \
| | |/
| |/| |
|
| |/
| |
| |
| |
| |
| |
| | |
This isn't a required piece of the config, so it might be unset. Use get
instead of [].
Change-Id: I1bbbcb4ac63a4f6d4399c0fa8881c21264a03e4b
|
|/
|
|
| |
Change-Id: I1bbda934cc65d508f1cece8c5adc714e9f464707
|
|\ |
|
| |
| |
| |
| | |
Change-Id: I344cd6ce38d8db8fe24e1611c9c61e1ffa1b586d
|
| |
| |
| |
| | |
Change-Id: I6ecb4e80d2944bf592a2cbd41695643bc49f832d
|
|\ \ |
|
| | |
| | |
| | |
| | | |
Change-Id: I7b1217cd50a3e3c89edcb5bda1d22cded90e9b4b
|
|\ \ \
| |/ /
|/| | |
|
| | |
| | |
| | |
| | | |
Change-Id: Ia8c656e2c6b97c877f5028fef8a94a2c41909bc5
|
| | |
| | |
| | |
| | | |
Change-Id: I805c30c0a522c03721a97118594030b9c8dfcd51
|
| |/
|/|
| |
| | |
Change-Id: Iad6eba535f48d0f09e5507db32399623f63a4f88
|
|\ \
| |/
|/| |
|
| |
| |
| |
| |
| |
| |
| | |
This patch is proposed according to the Direction 10 of doc
migration(https://etherpad.openstack.org/p/doc-migration-tracking).
Change-Id: Ida458338d353cbd6cc0162263db25f533b0bd9fd
|
|/
|
|
|
|
|
|
|
|
|
|
|
| |
The right way to update these globals is to use a lock
and ensure that nobody else is updating them at the same
time. Also update a temporary dictionary before setting
the global one so that nobody sees partial updates to the
global one.
This should help fix the thread-safety of shade (and other
tooling built ontop of this library).
Change-Id: Ie0e0369d98ba6a01edcbf447378a786eec3f13f9
|
|
|
|
|
|
|
| |
OSIC has been decommissioned, remove the now useless vendor data.
Change-Id: I57c6043018e96c0069c7db777b9f585cb7d535e7
Related-Change: I2d1b0710e875bd1ebc305fb5b184b68bf18f2ef7
|
|
|
|
| |
Change-Id: Ie69ff1e5a5d3cf3a762d6915c5d596de4f919931
|
|
|
|
| |
Change-Id: I611163aecdc4810e6fd1d7e47e60171d72db0ea3
|
|
|
|
| |
Change-Id: I4298bb7c2d66632b716b0dbeae64c9dca2b3434d
|