Merge ~pjdc/charm-k8s-mattermost/+git/charm-k8s-mattermost:body-size into charm-k8s-mattermost:master

Proposed by Paul Collins
Status: Merged
Approved by: Junien F
Approved revision: 06bc2e95b4756a6e38df0b56bab1565d6ced09df
Merged at revision: b74d03312da2b934b147e7328725e40d0345d904
Proposed branch: ~pjdc/charm-k8s-mattermost/+git/charm-k8s-mattermost:body-size
Merge into: charm-k8s-mattermost:master
Diff against target: 20 lines (+3/-1)
1 file modified
src/charm.py (+3/-1)
Reviewer Review Type Date Requested Status
Junien F Approve
Canonical IS Reviewers Pending
Review via email: mp+386052@code.launchpad.net

Commit message

update ingress to handle uploads up to max_file_size

To post a comment you must log in.
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

This merge proposal is being monitored by mergebot. Change the status to Approved to merge.

Revision history for this message
Junien F (axino) wrote :

+1

review: Approve
Revision history for this message
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Change successfully merged at revision b74d03312da2b934b147e7328725e40d0345d904

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1diff --git a/src/charm.py b/src/charm.py
2index 1ebd6c9..31d1fd2 100755
3--- a/src/charm.py
4+++ b/src/charm.py
5@@ -263,12 +263,14 @@ class MattermostK8sCharm(CharmBase):
6 return pod_spec
7
8 parsed = urlparse(site_url)
9- annotations = {}
10
11 if not parsed.scheme.startswith('http'):
12 return pod_spec
13
14 pod_spec = copy.deepcopy(pod_spec)
15+ annotations = {
16+ 'nginx.ingress.kubernetes.io/proxy-body-size': '{}m'.format(config['max_file_size'])
17+ }
18 ingress = {
19 "name": self.app.name,
20 "spec": {

Subscribers

People subscribed via source and target branches