Merge ~mthaddon/charm-k8s-ingress/+git/charm-k8s-ingress:prompt-for-trust into charm-k8s-ingress:master

Proposed by Tom Haddon
Status: Merged
Approved by: Jon Seager
Approved revision: b8eda252da812216b2aaf7e7eba72d29053e2f71
Merged at revision: 9e732fac59dfe1fdbc3329bfe3bd1530bebe9181
Proposed branch: ~mthaddon/charm-k8s-ingress/+git/charm-k8s-ingress:prompt-for-trust
Merge into: charm-k8s-ingress:master
Diff against target: 26 lines (+2/-2)
2 files modified
src/charm.py (+1/-1)
tests/unit/test_charm.py (+1/-1)
Reviewer Review Type Date Requested Status
ingress-charmers Pending
Review via email: mp+403050@code.launchpad.net

Commit message

Fix trivial typo in blockedstatus

Description of the change

Fix trivial typo in blockedstatus

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 :

Change successfully merged at revision 9e732fac59dfe1fdbc3329bfe3bd1530bebe9181

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 8c3560e..4d0d235 100755
3--- a/src/charm.py
4+++ b/src/charm.py
5@@ -386,7 +386,7 @@ class NginxIngressCharm(CharmBase):
6 "will request `juju trust` to be run"
7 )
8 self.unit.status = BlockedStatus(
9- "Insufficuent permissions, try: `juju trust {} --scope=cluster`".format(
10+ "Insufficient permissions, try: `juju trust {} --scope=cluster`".format(
11 self.app.name
12 )
13 )
14diff --git a/tests/unit/test_charm.py b/tests/unit/test_charm.py
15index ffd1aaa..47994c8 100644
16--- a/tests/unit/test_charm.py
17+++ b/tests/unit/test_charm.py
18@@ -69,7 +69,7 @@ class TestCharm(unittest.TestCase):
19 self.assertEqual(
20 self.harness.charm.unit.status,
21 BlockedStatus(
22- "Insufficuent permissions, try: "
23+ "Insufficient permissions, try: "
24 "`juju trust nginx-ingress-integrator --scope=cluster`"
25 ),
26 )

Subscribers

People subscribed via source and target branches

to all changes: