Merge ~hloeung/ubuntu-mirror-charm:master into ubuntu-mirror-charm:master

Proposed by Haw Loeung
Status: Rejected
Rejected by: Haw Loeung
Proposed branch: ~hloeung/ubuntu-mirror-charm:master
Merge into: ubuntu-mirror-charm:master
Diff against target: 25 lines (+7/-0)
2 files modified
hooks/hooks.py (+6/-0)
hooks/update-status (+1/-0)
Reviewer Review Type Date Requested Status
Alexandre Gomes Approve
Canonical IS Reviewers Pending
Review via email: mp+389761@code.launchpad.net

Commit message

Reduce noise about missing update-status hook

To post a comment you must log in.
Revision history for this message
Alexandre Gomes (alejdg) wrote :

+1

review: Approve
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.

Unmerged commits

4ad76e4... by Haw Loeung

Reduce noise about missing update-status hook

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
diff --git a/hooks/hooks.py b/hooks/hooks.py
index 0b80434..0f6ce2b 100755
--- a/hooks/hooks.py
+++ b/hooks/hooks.py
@@ -992,6 +992,12 @@ def upgrade_charm():
992 install()992 install()
993993
994994
995@hooks.hook("update-status")
996def update_status():
997 # Do nothing to reduce noise about missing hook.
998 return
999
1000
995@hooks.hook("nrpe-external-master-relation-changed")1001@hooks.hook("nrpe-external-master-relation-changed")
996def nrpe_external_master_relation_changed():1002def nrpe_external_master_relation_changed():
997 conf = Config()1003 conf = Config()
diff --git a/hooks/update-status b/hooks/update-status
998new file mode 1200001004new file mode 120000
index 0000000..9416ca6
--- /dev/null
+++ b/hooks/update-status
@@ -0,0 +1 @@
1hooks.py
0\ No newline at end of file2\ No newline at end of file

Subscribers

People subscribed via source and target branches