lp:~billy-olsen/charm-helpers/stable-fix-internal-int

Branch merges

Propose for merging

No branches dependent on this one.

Merged into lp:~openstack-charmers/charm-helpers/stable at revision 785

Frode Nordahl: Approve on 2017-10-10

Diff: 64 lines (+11/-8)

2 files modified

charmhelpers/contrib/openstack/ha/utils.py (+7/-4)
tests/contrib/openstack/ha/test_ha_utils.py (+4/-4)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

785. By Billy Olsen on 2017-10-04

Fix mismatch between 'internal' and 'int' in DNSHA

DNSHA keys off the string in configuration variables os-*-hostname.
os-internal-hostname is the name of the config variable. However,
resolve_address uses a long standing constant ADDRESS_MAP that refers
to this as 'int'.

This change handles this and hands resolve_address what it expects.
Also, use a clearer variable name for endpoint_type.

* Fix to unit test for 'int' vs 'internal'

784. By Alex Kavanagh on 2017-08-17

[joeborg, r=tinwood] Adding test coverage for FilePermissionAudit

(note tinwood had to patch the test slightly to get it to merge due
to recent changes in charm-helpers).

783. By Alex Kavanagh on 2017-08-15

[stub, r=tinwood] A feature of the PostgreSQL charm had stopped working, as
charm-helpers was attempting to do more validation of GPG key formats and the
PG charm happens to add comments to its keys so they don't get mixed up.

While fixing this, noticed that insecure usage still seems to be promoted.
Clearly flag this cases in the docstring and add WARNING messages to logs when
people open themselves up to attack (the key retrieval protocol is unencrypted
for historical reasons and the same man-in-the-middle attack that poisons an
archive can also make people trust keys retrieved this way).

782. By David Ames on 2017-08-14

[tinwood, r=thedac] Update the core.host.write_file(...) function to do less work

The primary motivation for this change is to do less work, touch the
filesystem less, and try to clean the logs up a bit. This change is
to the rendering of configuration files, which generates noise in the
logs (at INFO) level, and often results in writing the exact same
content, thus updating the atime of the file even when the contents
haven't changed.

This change detects if the content will change, and if not, doesn't
write the file. It does detect if the uid,gid is changing and takes
the appropriate action. The logs are demoted to the lowest level of
DEBUG/TRACE, and are only generated IFF the file is actually written
or the uid/gid is changed.

781. By Billy Olsen on 2017-08-11

[thedac,r=billy-olsen] Allow IPv4/IPv6 Dual Stack

780. By David Ames on 2017-08-08

[jamespage, r=thedac,tinwood] Refactoring ApacheSSLContext for network-space support

779. By David Ames on 2017-08-07

[coreycb, r=thedac] Add 2.15.0 to swift list of releases for pike

778. By Liam Young on 2017-08-07

[gnuoy, r=james-page] Change name of OpenStack extensions

The neutron-api charm already manages the extensions setting within a context
inside the charm. In the neutron-api charm it is called 'extension_drivers'
rather than 'extensions'. In a bid to keep things consistent this change
updates the recently added patch to switch from 'extensions' to
'extension_drivers'

777. By James Page on 2017-08-07

[gnuoy,r=james-page] Add support for qos driver in Neutron.

776. By James Page on 2017-08-03

[james-page,r=tinwood] Fix compat between os-*-hostname and SSL