We can support this simply by allowing ldap-user and ldap-password
configuration options to be optional.
Closes-Bug: #1762587
Co-authored-by: Vern Hart <vern.hart@canonical.com>
Change-Id: I2668d90a58aac9d103240dc67061612358a67150
The mock third party library was needed for mock support in py2
runtimes. Since we now only support py36 and later, we can use the
standard lib unittest.mock module instead.
Note that https://github.com/openstack/charms.openstack is used during tests
and he need `mock`, unfortunatelly it doesn't declare `mock` in its
requirements so it retrieve mock from other charm project (cross dependency).
So we depend on charms.openstack first and when
Ib1ed5b598a52375e29e247db9ab4786df5b6d142 will be merged then CI
will pass without errors.
Depends-On: Ib1ed5b598a52375e29e247db9ab4786df5b6d142
Change-Id: I107cabe4608950da02ef2b89e811791ad912cf7e
Add new options to keystone-ldap charm to explicitly specify
necessary ldap configuration values. The default values for
the new configuration parameters introduced in this patch are
empty so the default upstream keystone ldap config options are
used when undeclared.
If the same LDAP config options are specified in
ldap-config-flags and the respective charm config
option, then the value from the charm config option
will be used.
Func-Test-Pr: https://github.com/openstack-charmers/zaza-openstack-tests/pull/458
Change-Id: Ib269e74c7d313d9c6d09da29661dd279995dffe4
Closes-Bug: #1832765
When the primary keystone is deployed with rocky and python3 the charm
fails to install the correct python3 packages and use the correct
clients.
Note: A related bug #1798184, will cause the tests to fail. A complete
Rocky python3 solution is dependent on #1798184 being resolved.
Change-Id: I42d8a5bfff3200d18e7bad0bd29edf12aa6a05c7
Closes-Bug: #1806111
Implement the series-upgrade feature allowing to move between Ubuntu
series.
Depends-On: If42efab93dca85137d8758cae4f2e8baab75ae72
Change-Id: I0d2f2eec25c8753a4d6d0f7dffdbb54a6b35ebe1
* Both ldaps and ldap + StartTLS require a CA certificate configuration
option;
* use_tls option should only be used for StartTLS and will result in an
error if used with a URL that starts with ldaps;
* if a certificate is specified then LDAP backend server's certificate
validation is considered mandatory ("demand" option).
Depends-On: Ied4b6ed64354e3de3c78e6ac809666ee9ae29d1a
Change-Id: I659683ffec91560ebbd77969840c27e3d7048689
Closes-Bug: #1728155
* replace singletons with provide_charm_instance (in the code and tests)
* select an openstack release based on keystone package due to the lack
of openstack-origin from the principal layer (this is a subordinate, see
https://git.io/vNTyx)
* use reactive triggers to drop config.complete (new configuration
completeness) and config.rendered (non-stale config is rendered) on
config.changed
* do not check config completeness on every event - only when config has
actually changed
* remove the domain configuration file when relation with keystone is
removed (service restart should be performed on the keystone charm side)
* replace path_hash with file_hash (path_hash returns a new dict)
* modify unit tests to reflect the changes
Change-Id: Ied4b6ed64354e3de3c78e6ac809666ee9ae29d1a
Closes-Bug: #1741661
Currently any config changes post-deployment are
ignored once the domain name is set. This patch
ensures that any changes are registered and applied.
Change-Id: Id37db3c74d3712bc96c3f8ed6a04c907e4d18bc5
Closes-Bug: #1712972
Unit test for layered charms fail due to the new snap support.
snap_install_requested must be mocked out.
Change-Id: Id2e10f436300813a6c223ef4148cd82bd6f46ba5
Closes-Bug: #1722587
This change adds a guard state around the configure_domain_name()
function so that it only gets called ONCE when the relation connects.
This is to prevent restarting keystone everytime the update-status
hook runs on this charm.
Change-Id: I85d14ddb97e78be966f0cc8dbbcea312599d7327
- Turn on Zesty-Ocata Amulet test definitions.
- Standardize test-requirements.txt
- Sync charm helpers for various fixes
Change-Id: Id3cbb8944e559ddb57dd249274572d264183f56f
Samuel Walladge