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
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:
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
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 :
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:
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
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:
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:
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:
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
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 :
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
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 :
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
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.
15 matches
Mail list logo