Graylog Charm

Merge ~tiago.pasqualini/charm-graylog:fix_1794630 into ~graylog-charmers/charm-graylog:master

  1. Git
  2. lp:~tiago.pasqualini/charm-graylog
  3. fix_1794630
  4. Merge into master
Proposed by Tiago Pasqualini da Silva
Status: Merged
Approved by: Tom Haddon
Approved revision: a13693800f358639dc220781ae3c6abad7b84255
Merged at revision: a48a96bf742c6af701d8542ded754f5c4fac7f1c
Proposed branch: ~tiago.pasqualini/charm-graylog:fix_1794630
Merge into: ~graylog-charmers/charm-graylog:master
Diff against target: 12 lines (+1/-0)
1 file modified
reactive/graylog.py (+1/-0)
Reviewer Review Type Date Requested Status
Stuart Bishop (community) Approve
Canonical IS Reviewers Pending
Review via email: mp+367145@code.staging.launchpad.net

Commit message

Fix configure_mongodb_connection called on stop

Description of the change

This commit fixes LP 1794630 by adding an extra condition to run configure_mongodb_connection. This avoids it being called when the application is being removed.

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
Stuart Bishop (stub) wrote :

Yup. The mongodb interface is undocumented, but a quick look at the source indicates that this is the correct flag to use.

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

Change successfully merged at revision a48a96bf742c6af701d8542ded754f5c4fac7f1c

There was an error fetching revisions from git servers. Please try again in a few minutes. If the problem persists, contact Launchpad support.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches