Merge ~moon127/charm-k8s-discourse/+git/charm-k8s-discourse:lib-resource-fixes into charm-k8s-discourse:master

Proposed by Gareth Woolridge
Status: Merged
Approved by: Jay Kuri
Approved revision: b7c9bdd0bb41d312068c8429c714f63f472c172b
Merged at revision: 93261e51462fa279142898cb4270c61a42458428
Proposed branch: ~moon127/charm-k8s-discourse/+git/charm-k8s-discourse:lib-resource-fixes
Merge into: charm-k8s-discourse:master
Diff against target: 25 lines (+2/-1)
2 files modified
config.yaml (+1/-1)
src/charm.py (+1/-0)
Reviewer Review Type Date Requested Status
Canonical IS Reviewers Pending
Discourse Charm Maintainers Pending
Review via email: mp+383148@code.launchpad.net

Commit message

Fix config.yaml to match smtp_username as used in charm.py

Set ImagePullPolicy: Never, for microk8s testing, otherwise we get ErrImagePull, as we don't have a registry

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
🤖 Canonical IS Merge Bot (canonical-is-mergebot) wrote :

Unable to determine commit message from repository - please click "Set commit message" and enter the commit message manually.

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

Change successfully merged at revision 93261e51462fa279142898cb4270c61a42458428

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/config.yaml b/config.yaml
index 4d1c4bd..6d656db 100644
--- a/config.yaml
+++ b/config.yaml
@@ -47,7 +47,7 @@ options:
47 type: int47 type: int
48 description: "Port to use when connecting to SMTP server"48 description: "Port to use when connecting to SMTP server"
49 default: 58749 default: 587
50 smtp_user_name:50 smtp_username:
51 type: string51 type: string
52 description: "Username to use when sending mail via SMTP"52 description: "Username to use when sending mail via SMTP"
53 default: ""53 default: ""
diff --git a/src/charm.py b/src/charm.py
index 117b304..86f8ab6 100755
--- a/src/charm.py
+++ b/src/charm.py
@@ -53,6 +53,7 @@ class DiscourseCharm(CharmBase):
53 'containers': [{53 'containers': [{
54 'name': self.framework.model.app.name,54 'name': self.framework.model.app.name,
55 'imageDetails': discourse_image_details,55 'imageDetails': discourse_image_details,
56 'imagePullPolicy': 'Never',
56 'ports': [{57 'ports': [{
57 'containerPort': int(config['service_port']),58 'containerPort': int(config['service_port']),
58 'protocol': 'TCP',59 'protocol': 'TCP',

Subscribers

People subscribed via source and target branches