Charm Helpers

charm-helpers:master

  1. Git
  2. lp:charm-helpers
  3. master
Last commit made on 2024-04-12
Get this branch:
git clone -b master https://git.launchpad.net/charm-helpers

Branch merges

Branch information

Name:
master
Repository:
lp:charm-helpers

Recent commits

807f705... by James Page

Merge pull request #881 from dosaboy/bug/1920770

Support checking service ports with ssl connection

0071e27... by Edward Hope-Morley

Support checking service ports with ssl connection

By default netcat is used to check if a service is
listening on a port. This is generally ok except
for services expecting SSL connections which need
to be properly closed and netcat can't do that. So
here we add support for optionally using the python
ssl library to create an ssl connection to the port
and close it properly once finished.

Related-Bug: #1920770

d3a8682... by Alex Kavanagh <email address hidden>

Merge pull request #893 from MylesJP/audit-middleware

Add support for keystone audit middleware

12afd33... by Myles Penner

Add support for keystone audit middleware

Created a context for enabling keystone audit middleware as per https://bugs.launchpad.net/charm-helpers/+bug/1856555 along with two unit tests.

58ab965... by Alex Kavanagh <email address hidden>

Merge pull request #882 from dosaboy/bug/2058505

Skip service_resume if already enabled

a334500... by Edward Hope-Morley

Skip service_enable if already enabled

Repeatedly calling service_enable for a service that
is already enabled can lead to unintended consequences.
Some charms frequently call servie_resume which will
call service('enable') and this adds a check to only
do so of the service is not enabled.

Related-Bug: #2058505

511bc03... by Alex Kavanagh <email address hidden>

Merge pull request #880 from wolsen/fix-platform-check

Fix platform check for Debian 12

274cb37... by Billy Olsen

Fix platform check for Debian 12

The osplatform.get_platform() check fails on Debian Bookworm
because the output determined by the /etc/os-release file has
"Debian GNU/Linux" and the get_platform() check only looks for
lowercase debian.

Signed-off-by: Billy Olsen <email address hidden>

b78107d... by Alex Kavanagh <email address hidden>

Merge pull request #636 from lmlg/fix-popen-communicate

Fix call to Popen.communicate

e44a15e... by Luciano Lo Giudice

Remove unused imports