charm-k8s-mm-pd-bot

~ballot/charm-k8s-mm-pd-bot/+git/charm-k8s-mm-pd-bot:rework_ssl

  1. Git
  2. lp:~ballot/charm-k8s-mm-pd-bot/+git/charm-k8s-mm-pd-bot
  3. rework_ssl
Last commit made on 2020-08-13
Get this branch:
git clone -b rework_ssl https://git.launchpad.net/~ballot/charm-k8s-mm-pd-bot/+git/charm-k8s-mm-pd-bot
Only Benjamin Allot can upload to this branch. If you are Benjamin Allot please log in for upload directions.

Branch merges

Branch information

Name:
rework_ssl
Repository:
lp:~ballot/charm-k8s-mm-pd-bot/+git/charm-k8s-mm-pd-bot

Recent commits

48b8a7b... by Benjamin Allot

Rework SSL. When tls_secret_name is set, terminate SSL on the ingress

Previous logic was relying on a setting of the bot to configure the TLS
for the ingress. That was wrong because the bot would expect also to
terminate the SSL handshake itself.

Instead, let's infer the scheme to 'https' when the juju setting
tls_secret_name is set.

a5a7e68... by Benjamin Allot

First version of a working mm-pd-bot k8s charm with the operator framework.

Reviewed-on: https://code.launchpad.net/~ballot/charm-k8s-mm-pd-bot/+git/charm-k8s-mm-pd-bot/+merge/388497
Reviewed-by: Tom Haddon <email address hidden>
Reviewed-by: John Lenton <email address hidden>

d5156d3... by Benjamin Allot

Review comments addressed

95ab60d... by Benjamin Allot

Automatically get the list of juju settings declared in config.yaml

This is useful in the serie of tests, where cleaning all the juju
settings between each test is needed.
This way we don't need to sync anything but the config.yaml.

0dc2ff2... by Benjamin Allot

Remove reference to Canonical

004a918... by Benjamin Allot

Upgrade to the latest ops framework version for some functionalities

da6b3d4... by Benjamin Allot

A working charm with some basic manual test

* currently able to spawn a mm-pd-bot and have an ingess (tested locally
  only) without TLS
* Some check in the configuration provided will set the blocked status
  if not provided (see the README.md)

TODO:
* add a healthcheck to the pod (will probably required modification of
the bot)
* test ingress with TLS

90946ac... by Benjamin Allot

Add basic files like README, LICENSE and .*ignore files

Reviewed-on: https://code.launchpad.net/~ballot/charm-k8s-mm-pd-bot/+git/charm-k8s-mm-pd-bot/+merge/388482
Reviewed-by: Tom Haddon <email address hidden>

184b17e... by Benjamin Allot

Add .jujuignore to avoid unecessary files during charmcraft build

95b6c3a... by Benjamin Allot

Add license related parts