lp:~erik-lonroth/charm-haproxy/charmcraft_build

Branch merges

Propose for merging

No branches dependent on this one.

Rejected for merging into lp:charm-haproxy

Tom Haddon: Needs Information on 2022-02-16

Canonical IS Reviewers: Pending requested 2022-02-04

Diff: 47 lines (+31/-1)

2 files modified

README.md (+1/-1)
charmcraft.yaml (+30/-0)

api

Related source package recipes

No recipes using this branch. (?)

Create packaging recipe

Related snap packages

Related bugs

Related blueprints

152. By Erik Lönroth on 2022-02-04

Builds with charmcraft v.1.4+

151. By Erik Lönroth on 2022-02-04

Fixed typo

150. By Stephan Pampel on 2021-12-02

Extend check_haproxy.sh:
  * more detailed output
  * WARNING if a service is down
  * CRITICAL if a proxy (=all services of a frontend/backend) is down

Reviewed-on: https://code.launchpad.net/~stephanpampel/charm-haproxy/charm-haproxy/+merge/411165
Reviewed-by: Junien Fridrick <email address hidden>

149. By Simon Fels on 2021-11-05

Convert port to integer to allow comparison

The update_service_ports method compares ports from both supplied lists. One list is coming from get_service_ports which keeps ports as integer and the other one from config_changed which keeps them as utf8 strings. Comparing ports will now fail inside update_service_ports as we compare integers vs strings.

With Juju 2.9 this leads to ports randomly being opened as semantics on
open-port/close-port have slightly changed.

We saw ports being never opened after initial deployment in a 2.9 model with the services config option set. Only after remove-relation/relate we were able to get the ports correctly exposed.

Fixing the port comparison resolves this and makes things work on both Juju 2.8 and 2.9

Reviewed-on: https://code.launchpad.net/~morphis/charm-haproxy/fix-port-comparison/+merge/411405
Reviewed-by: Tom Haddon <email address hidden>

148. By Vern Hart on 2021-10-19

Simplify install hook (works on bionic and focal).

Fixes: https://bugs.launchpad.net/charm-haproxy/+bug/1946798

Reviewed-on: https://code.launchpad.net/~vern/charm-haproxy/fix-install-hook/+merge/410140
Reviewed-by: Tom Haddon <email address hidden>
Reviewed-by: Paul Collins <email address hidden>

147. By Junien F on 2021-10-13

we want to install the python2 package for focal and above, not the other way around

Reviewed-on: https://code.launchpad.net/~axino/charm-haproxy/trunk/+merge/410129
Reviewed-by: Joel Sing <email address hidden>

146. By Haw Loeung on 2021-10-13

Fix distro/release detection - LP:1946798

Reviewed-on: https://code.launchpad.net/~hloeung/charm-haproxy/fix-distro-release-detection/+merge/410096
Reviewed-by: Paul Collins <email address hidden>

145. By Tom Haddon on 2021-10-11

Update README to document monitoring via Telegraf, use 'relate' rather than 'add-relation' and remove personal pronoun

Reviewed-on: https://code.launchpad.net/~mthaddon/charm-haproxy/doc-updates/+merge/409521
Reviewed-by: Loïc Gomez <email address hidden>

144. By Thomas Cuthbert on 2021-10-11

fix: general fixes for using correct python version.

Reviewed-on: https://code.launchpad.net/~tcuthbert/charm-haproxy/haproxy-focal/+merge/409962
Reviewed-by: Joel Sing <email address hidden>

143. By Haw Loeung on 2020-08-28

Bump DH params used, it's year 2020

Reviewed-on: https://code.launchpad.net/~hloeung/charm-haproxy/bump-global-dh-param/+merge/388543
Reviewed-by: Tom Haddon <email address hidden>