[Wordpress-charmers] [Merge] ~sajoupa/charm-k8s-wordpress:nginx-ingress-modsec into charm-k8s-wordpress:master

2022-01-25 Thread mp+414554
The proposal to merge ~sajoupa/charm-k8s-wordpress:nginx-ingress-modsec into charm-k8s-wordpress:master has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~sajoupa/charm-k8s-wordpress/+git/charm-k8s-wordpress/+merge/414554 -- Your team Wordpress

Re: [Wordpress-charmers] [Merge] ~sajoupa/charm-k8s-wordpress:nginx-ingress-modsec into charm-k8s-wordpress:master

2022-01-25 Thread Canonical IS Mergebot
Change successfully merged at revision ab5df3eaee62443a4dcb3e1245a67b0e9860ee01 -- https://code.launchpad.net/~sajoupa/charm-k8s-wordpress/+git/charm-k8s-wordpress/+merge/414554 Your team Wordpress Charmers is subscribed to branch charm-k8s-wordpress:master. -- Mailing list:

[Wordpress-charmers] [Merge] ~sajoupa/charm-k8s-wordpress:nginx-ingress-modsec into charm-k8s-wordpress:master

2022-01-25 Thread Laurent Sesquès
The proposal to merge ~sajoupa/charm-k8s-wordpress:nginx-ingress-modsec into charm-k8s-wordpress:master has been updated. Status: Needs review => Approved For more details, see: https://code.launchpad.net/~sajoupa/charm-k8s-wordpress/+git/charm-k8s-wordpress/+merge/414554 -- Your team

Re: [Wordpress-charmers] [Merge] ~sajoupa/charm-k8s-wordpress:nginx-ingress-modsec into charm-k8s-wordpress:master

2022-01-25 Thread Nick Moffitt
Review: Approve Worth a try. -- https://code.launchpad.net/~sajoupa/charm-k8s-wordpress/+git/charm-k8s-wordpress/+merge/414554 Your team Wordpress Charmers is subscribed to branch charm-k8s-wordpress:master. -- Mailing list: https://launchpad.net/~wordpress-charmers Post to :

[Wordpress-charmers] [Merge] ~sajoupa/charm-k8s-wordpress:nginx-ingress-modsec into charm-k8s-wordpress:master

2022-01-25 Thread Canonical IS Mergebot
The proposal to merge ~sajoupa/charm-k8s-wordpress:nginx-ingress-modsec into charm-k8s-wordpress:master has been updated. Commit message changed to: ingress modsec: add crs exclusions for wordpress For more details, see:

Re: [Wordpress-charmers] [Merge] ~sajoupa/charm-k8s-wordpress:nginx-ingress-modsec into charm-k8s-wordpress:master

2022-01-25 Thread Canonical IS Mergebot
This merge proposal is being monitored by mergebot. Change the status to Approved to merge. -- https://code.launchpad.net/~sajoupa/charm-k8s-wordpress/+git/charm-k8s-wordpress/+merge/414554 Your team Wordpress Charmers is requested to review the proposed merge of

[Wordpress-charmers] [Merge] ~sajoupa/charm-k8s-wordpress:nginx-ingress-modsec into charm-k8s-wordpress:master

2022-01-25 Thread Laurent Sesquès
Laurent Sesquès has proposed merging ~sajoupa/charm-k8s-wordpress:nginx-ingress-modsec into charm-k8s-wordpress:master. Requested reviews: Wordpress Charmers (wordpress-charmers) For more details, see:

Re: [Wordpress-charmers] [Merge] ~sajoupa/charm-k8s-wordpress:nginx-ingress-modsec into charm-k8s-wordpress:master

2022-01-25 Thread Laurent Sesquès
Fixes uRT#37351 -- https://code.launchpad.net/~sajoupa/charm-k8s-wordpress/+git/charm-k8s-wordpress/+merge/414554 Your team Wordpress Charmers is requested to review the proposed merge of ~sajoupa/charm-k8s-wordpress:nginx-ingress-modsec into charm-k8s-wordpress:master. -- Mailing list:

Re: [Wordpress-charmers] [Merge] ~sajoupa/charm-k8s-wordpress:nginx-ingress-modsec into charm-k8s-wordpress:master

2022-01-24 Thread Canonical IS Mergebot
Change successfully merged at revision 69406f17853141b21307c09243e7ccc2da74d0f6 -- https://code.launchpad.net/~sajoupa/charm-k8s-wordpress/+git/charm-k8s-wordpress/+merge/414465 Your team Wordpress Charmers is subscribed to branch charm-k8s-wordpress:master. -- Mailing list:

[Wordpress-charmers] [Merge] ~sajoupa/charm-k8s-wordpress:nginx-ingress-modsec into charm-k8s-wordpress:master

2022-01-24 Thread mp+414465
The proposal to merge ~sajoupa/charm-k8s-wordpress:nginx-ingress-modsec into charm-k8s-wordpress:master has been updated. Status: Approved => Merged For more details, see: https://code.launchpad.net/~sajoupa/charm-k8s-wordpress/+git/charm-k8s-wordpress/+merge/414465 -- Your team Wordpress

Re: [Wordpress-charmers] [Merge] ~sajoupa/charm-k8s-wordpress:nginx-ingress-modsec into charm-k8s-wordpress:master

2022-01-24 Thread Tom Haddon
Review: Approve LGTM, thx -- https://code.launchpad.net/~sajoupa/charm-k8s-wordpress/+git/charm-k8s-wordpress/+merge/414465 Your team Wordpress Charmers is subscribed to branch charm-k8s-wordpress:master. -- Mailing list: https://launchpad.net/~wordpress-charmers Post to :

Re: [Wordpress-charmers] [Merge] ~sajoupa/charm-k8s-wordpress:nginx-ingress-modsec into charm-k8s-wordpress:master

2022-01-21 Thread Tom Haddon
Also alpha sorting of annotation keys in tests pls. Diff comments: > diff --git a/tests/unit/test_charm.py b/tests/unit/test_charm.py > index 7606877..e4af690 100644 > --- a/tests/unit/test_charm.py > +++ b/tests/unit/test_charm.py > @@ -133,6 +133,11 @@ class

Re: [Wordpress-charmers] [Merge] ~sajoupa/charm-k8s-wordpress:nginx-ingress-modsec into charm-k8s-wordpress:master

2022-01-21 Thread Loïc Gomez
Review: Approve LGTM -- https://code.launchpad.net/~sajoupa/charm-k8s-wordpress/+git/charm-k8s-wordpress/+merge/414465 Your team Wordpress Charmers is subscribed to branch charm-k8s-wordpress:master. -- Mailing list: https://launchpad.net/~wordpress-charmers Post to :

Re: [Wordpress-charmers] [Merge] ~sajoupa/charm-k8s-wordpress:nginx-ingress-modsec into charm-k8s-wordpress:master

2022-01-21 Thread Canonical IS Mergebot
This merge proposal is being monitored by mergebot. Change the status to Approved to merge. -- https://code.launchpad.net/~sajoupa/charm-k8s-wordpress/+git/charm-k8s-wordpress/+merge/414465 Your team Wordpress Charmers is requested to review the proposed merge of

[Wordpress-charmers] [Merge] ~sajoupa/charm-k8s-wordpress:nginx-ingress-modsec into charm-k8s-wordpress:master

2022-01-21 Thread Laurent Sesquès
Laurent Sesquès has proposed merging ~sajoupa/charm-k8s-wordpress:nginx-ingress-modsec into charm-k8s-wordpress:master. Commit message: replace modsec configuration in the docker image with the k8s ingress built-in modsec. Make it a juju config option. Also, update the wordpress image version.