Xav Paice has proposed merging ~xavpaice/nagios-charm:fix-lp1623658 into
nagios-charm:master.
Requested reviews:
Nagios Charm developers (nagios-charmers)
For more details, see:
https://code.launchpad.net/~xavpaice/nagios-charm/+git/nagios-charm/+merge/328677
--
Your team Nagios Charm
https://code.launchpad.net/~xavpaice/nagios-charm/+git/nagios-
charm/+merge/328677
--
You received this bug notification because you are a member of Nagios
Charm developers, which is subscribed to Nagios Charm.
https://bugs.launchpad.net/bugs/1623658
Title:
livestatus socket permission
charm) to monitor the local nagios box. Ideally, it
would be a different nagios server monitoring the nagios server anyway.
** Tags added: canonical-bootstack
** Changed in: nagios-charm
Status: New => In Progress
** Changed in: nagios-charm
Assignee: (unassigned) => Xav
The proposal to merge ~jhebden/nagios-charm:readonly-account into
nagios-charm:master has been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~jhebden/nagios-charm/+git/nagios-charm/+merge/318564
--
Your team Nagios Charm developers is subscribed
cancel that last one, I must have been on another planet.
https://code.launchpad.net/~xavpaice/nagios-charm/+git/nagios-
charm/+merge/329236
--
You received this bug notification because you are a member of Nagios
Charm developers, which is subscribed to Nagios Charm.
Xav Paice has proposed merging ~xavpaice/nagios-charm:fix-lp1666763 into
nagios-charm:master.
Requested reviews:
Nagios Charm developers (nagios-charmers)
For more details, see:
https://code.launchpad.net/~xavpaice/nagios-charm/+git/nagios-charm/+merge/326195
bug fix for https
Public bug reported:
Using the current charm, on Xenial, when I setup a fresh unit and enable
pagerduty, if I do something which causes an alert but the pagerduty
script doesn't work (e.g. because of bug 1718833) then the alert is lost
into the dark void of nowhere. Maybe we're not doing
in a host definition which would do this, but we
don't have that info when defining the host in a container
- hostdependency definitions could work?
** Affects: nagios-charm
Importance: Undecided
Assignee: Xav Paice (xavpaice)
Status: New
** Affects: nrpe-charm
Importance
Review: Approve
Trivial
--
https://code.launchpad.net/~xavpaice/nagios-charm/+git/nagios-charm/+merge/332351
Your team Nagios Charm developers is subscribed to branch nagios-charm:master.
--
Mailing list: https://launchpad.net/~nagios-charmers
Post to : nagios-charmers@lists.launchpad.net
Xav Paice has proposed merging ~xavpaice/nagios-charm:master into
nagios-charm:master.
Requested reviews:
Nagios Charm developers (nagios-charmers)
For more details, see:
https://code.launchpad.net/~xavpaice/nagios-charm/+git/nagios-charm/+merge/332351
--
Your team Nagios Charm developers
I've added a preliminary 'parents' option:
NRPE MP:
https://code.launchpad.net/~xavpaice/nrpe-charm/+git/nrpe-charm/+merge/332126
Nagios MP:
https://code.launchpad.net/~xavpaice/nagios-charm/+git/nagios-charm/+merge/332127
What this does is, for a host that is a container on a machine, and
** Changed in: nagios-charm
Status: Fix Committed => Fix Released
--
You received this bug notification because you are a member of Nagios
Charm developers, which is subscribed to Nagios Charm.
https://bugs.launchpad.net/bugs/1623658
Title:
livestatus socket permission
Status in
** Changed in: nagios-charm
Status: Fix Committed => Fix Released
--
You received this bug notification because you are a member of Nagios
Charm developers, which is subscribed to Nagios Charm.
https://bugs.launchpad.net/bugs/1677580
Title:
nagios unit local checks should be editable
** Changed in: nagios-charm
Status: Confirmed => Fix Released
--
You received this bug notification because you are a member of Nagios
Charm developers, which is subscribed to Nagios Charm.
https://bugs.launchpad.net/bugs/1569044
Title:
Perl script permissions wrong for nagios
** Package changed: nagios (Juju Charms Collection) => nagios-charm
--
You received this bug notification because you are a member of Nagios
Charm developers, which is subscribed to Nagios Charm.
https://bugs.launchpad.net/bugs/1569044
Title:
Perl script permissions wrong for nagios
** Changed in: nagios-charm
Status: Fix Committed => Fix Released
--
You received this bug notification because you are a member of Nagios
Charm developers, which is subscribed to Nagios Charm.
https://bugs.launchpad.net/bugs/1718833
Title:
pagerduty integration needs
Xav Paice has proposed merging ~xavpaice/nagios-charm:lp1623658 into
nagios-charm:master.
Requested reviews:
Nagios Charm developers (nagios-charmers)
For more details, see:
https://code.launchpad.net/~xavpaice/nagios-charm/+git/nagios-charm/+merge/329344
fix for https://bugs.launchpad.net
This also affects the Nagios charm, e.g.:
(added pdb.set_trace() to L210 of core/hookenv.py)
root@juju-machine-0-lxc-6:/var/lib/juju/agents/unit-nagios-0/charm#
./hooks/upgrade-charm
> /var/lib/juju/agents/unit-nagios-0/charm/hooks/charmhelpers/core/hookenv.py(218)principal_unit()
-> for
** Tags added: canonical-bootstack
** Also affects: nagios-charm
Importance: Undecided
Status: New
--
You received this bug notification because you are a member of Nagios
Charm developers, which is subscribed to Nagios Charm.
https://bugs.launchpad.net/bugs/1712977
Title:
FWIW, this problem makes it impossible to upgrade charms to the current
head/master.
--
You received this bug notification because you are a member of Nagios
Charm developers, which is subscribed to Nagios Charm.
https://bugs.launchpad.net/bugs/1712977
Title:
config-changed hook fails:
Xav Paice has proposed merging ~xavpaice/nagios-charm:lp1712977 into
nagios-charm:master.
Requested reviews:
Nagios Charm developers (nagios-charmers)
For more details, see:
https://code.launchpad.net/~xavpaice/nagios-charm/+git/nagios-charm/+merge/329768
--
Your team Nagios Charm developers
rev 786 of charmhelpers fixes the bug, need to update both nrpe and
nagios charms to collect that update.
https://code.launchpad.net/~xavpaice/nrpe-charm/+git/nrpe-charm/+merge/329767
https://code.launchpad.net/~xavpaice/nagios-charm/+git/nagios-charm/+merge/329768
--
You received this bug
Xav Paice has proposed merging ~xavpaice/nagios-charm:add-bindings into
nagios-charm:master.
Requested reviews:
Nagios Charm developers (nagios-charmers)
For more details, see:
https://code.launchpad.net/~xavpaice/nagios-charm/+git/nagios-charm/+merge/348028
--
Your team Nagios Charm
Xav Paice has proposed merging ~xavpaice/nagios-charm:pagerduty into
nagios-charm:master.
Requested reviews:
Nagios Charm developers (nagios-charmers)
For more details, see:
https://code.launchpad.net/~xavpaice/nagios-charm/+git/nagios-charm/+merge/352052
--
Your team Nagios Charm developers
Public bug reported:
If the model is configured with https-proxy set, we should use the
--proxy switch in the cron job. There's currently no setting for this
in the template.
** Affects: nagios-charm
Importance: Undecided
Status: New
** Tags: canonical-bootstack
--
You
The proposal to merge ~xavpaice/nagios-charm:lp1763576 into nagios-charm:master
has been updated.
Status: Needs review => Superseded
For more details, see:
https://code.launchpad.net/~xavpaice/nagios-charm/+git/nagios-charm/+merge/343254
--
Your team Nagios Charm developers is requested to
Released fix in cs:nagios-24
** Changed in: nagios-charm
Status: In Progress => Fix Committed
** Changed in: nagios-charm
Status: Fix Committed => Fix Released
--
You received this bug notification because you are a member of Nagios
Charm developers, which is subscribed to Nagios
** Changed in: nrpe-charm
Status: Confirmed => Fix Released
** Changed in: nagios-charm
Status: Confirmed => Fix Released
--
You received this bug notification because you are a member of Nagios
Charm developers, which is subscribed to Nagios Charm.
I'm adding bionic packages for 2.24 to the PPA now - should avoid needing this.
--
https://code.launchpad.net/~aieri/thruk-agent-charm/+git/thruk-agent-charm/+merge/358387
Your team Nagios Charm developers is subscribed to branch
thruk-agent-charm:master.
--
Mailing list:
Review: Resubmit
Thanks for the detailed review - I've address (most) of the comments. I've not
added all the docstrings requested as it's hard to know where to draw the line
between changes relevant to this branch and changes that just make the overall
code better - I suggest raising bugs
The proposal to merge ~xavpaice/hw-health-charm:add_ipmi into
hw-health-charm:master has been updated.
Status: Needs review => Superseded
For more details, see:
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/363593
--
Your team Nagios Charm developers is
The proposal to merge ~xavpaice/hw-health-charm:fix_tests into
hw-health-charm:master has been updated.
Status: Needs review => Superseded
For more details, see:
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/363758
--
Your team Nagios Charm developers is
Thanks Peter, I've updated as suggested.
--
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/363813
Your team Nagios Charm developers is subscribed to branch
hw-health-charm:master.
--
Mailing list: https://launchpad.net/~nagios-charmers
Post to :
Xav Paice has proposed merging ~xavpaice/hw-health-charm:fix_tests into
hw-health-charm:master.
Requested reviews:
Peter Sabaini (peter-sabaini)
For more details, see:
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/363813
--
Your team Nagios Charm developers
Xav Paice has proposed merging ~xavpaice/hw-health-charm:add_ipmi into
hw-health-charm:master.
Requested reviews:
Nagios Charm developers (nagios-charmers)
For more details, see:
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/363593
--
Your team Nagios Charm
Xav Paice has proposed merging ~xavpaice/hw-health-charm:add_smartctl into
hw-health-charm:master.
Requested reviews:
Nagios Charm developers (nagios-charmers)
For more details, see:
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/363833
--
Your team Nagios
The proposal to merge ~xavpaice/hw-health-charm:add_smartctl into
hw-health-charm:master has been updated.
Status: Needs review => Superseded
For more details, see:
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/363760
--
Your team Nagios Charm developers
Xav Paice has proposed merging ~xavpaice/hw-health-charm:add_smart into
hw-health-charm:master.
Requested reviews:
Nagios Charm developers (nagios-charmers)
For more details, see:
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/363834
--
Your team Nagios
Xav Paice has proposed merging ~xavpaice/hw-health-charm:add_smartctl into
hw-health-charm:master.
Requested reviews:
Nagios Charm developers (nagios-charmers)
For more details, see:
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/363760
--
Your team Nagios
Please review
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/363758
first, which will make this change smaller
--
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/363760
Your team Nagios Charm developers is requested to review
The proposal to merge ~xavpaice/hw-health-charm:add_ipmi into
hw-health-charm:master has been updated.
Status: Needs review => Superseded
For more details, see:
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/363638
--
Your team Nagios Charm developers is
The proposal to merge ~xavpaice/hw-health-charm:add_ipmi into
hw-health-charm:master has been updated.
Status: Needs review => Superseded
For more details, see:
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/363681
--
Your team Nagios Charm developers is
Currently have left the tox.ini with complexity at 20, need to add a second
commit (unrelated) to get the complexity down to 10, so it passes. We need to
raise a bug report to track that.
--
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/363682
Your team
I've addressed the comments, and squashed the commits to one.
--
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/363682
Your team Nagios Charm developers is subscribed to branch
hw-health-charm:master.
--
Mailing list: https://launchpad.net/~nagios-charmers
Review: Approve
lgtm
--
https://code.launchpad.net/~aieri/hw-health-charm/+git/hw-health-charm/+merge/363062
Your team Nagios Charm developers is subscribed to branch
hw-health-charm:master.
--
Mailing list: https://launchpad.net/~nagios-charmers
Post to :
Xav Paice has proposed merging ~xavpaice/hw-health-charm:lp1819733 into
hw-health-charm:master.
Requested reviews:
Nagios Charm developers (nagios-charmers)
For more details, see:
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/364345
--
Your team Nagios
Xav Paice has proposed merging ~xavpaice/hw-health-charm:fix_upgrade_hook into
hw-health-charm:master.
Requested reviews:
Nagios Charm developers (nagios-charmers)
For more details, see:
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/364355
--
Your team
Xav Paice has proposed merging ~xavpaice/hw-health-charm:fix_unittests into
hw-health-charm:master.
Requested reviews:
Nagios Charm developers (nagios-charmers)
For more details, see:
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/363403
--
Your team Nagios
Many thanks for that, I've updated the change accordingly. This does seem like
a better approach.
--
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/369219
Your team Nagios Charm developers is subscribed to branch
hw-health-charm:master.
--
Mailing list:
The proposal to merge ~xavpaice/hw-health-charm:fix_clear_sel_unittest into
hw-health-charm:master has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/369224
--
Your team Nagios Charm
Xav Paice has proposed merging ~xavpaice/hw-health-charm:fix_clear_sel_unittest
into hw-health-charm:master.
Commit message:
fix unit tests for clear-sel action
Requested reviews:
Canonical IS Reviewers (canonical-is-reviewers)
Nagios Charm developers (nagios-charmers)
For more details
The proposal to merge ~xavpaice/hw-health-charm:add_smart into
hw-health-charm:master has been updated.
Commit message changed to:
Add smartctl nrpe checks
Adds a Nagios plugin and nrpe check using it for smartctl checks against a list
of disks that support it. One check for all disks, report
Xav Paice has proposed merging ~xavpaice/hw-health-charm:fix_cron into
hw-health-charm:master.
Requested reviews:
Nagios Charm developers (nagios-charmers)
For more details, see:
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/367882
--
Your team Nagios Charm
Xav Paice has proposed merging ~xavpaice/hw-health-charm:fix_clear_sel into
hw-health-charm:master.
Requested reviews:
Nagios Charm developers (nagios-charmers)
For more details, see:
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/367880
--
Your team Nagios
** Merge proposal linked:
https://code.launchpad.net/~xavpaice/nagios-charm/+git/nagios-charm/+merge/366740
--
You received this bug notification because you are a member of Nagios
Charm developers, which is subscribed to Nagios Charm.
https://bugs.launchpad.net/bugs/1827159
Title:
Xav Paice has proposed merging ~xavpaice/nagios-charm:lp1827159 into
nagios-charm:master.
Commit message:
Add custom disk monitor for nagios host
Requested reviews:
James Hebden (ec0)
For more details, see:
https://code.launchpad.net/~xavpaice/nagios-charm/+git/nagios-charm/+merge/366741
Public bug reported:
I have two models on Openstack, both are in the same project, and same
controller. Using a cross model relation, I have related nrpe on model1
to Nagios on model2. The Nagios host has a floating IP address, the
machines in model1 (nrpe) do not.
When I added the relation,
The proposal to merge ~xavpaice/nagios-charm:lp1827159 into nagios-charm:master
has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~xavpaice/nagios-charm/+git/nagios-charm/+merge/366741
--
Your team Nagios Charm developers is subscribed to
Xav Paice has proposed merging
~xavpaice/nagios-charm:add-service_check_timeout_state into nagios-charm:master.
Requested reviews:
Nagios Charm developers (nagios-charmers)
For more details, see:
https://code.launchpad.net/~xavpaice/nagios-charm/+git/nagios-charm/+merge/369495
--
Your team
Xav Paice has proposed merging ~xavpaice/nagios-charm:pagerduty-defaults into
nagios-charm:master.
Requested reviews:
Nagios Charm developers (nagios-charmers)
For more details, see:
https://code.launchpad.net/~xavpaice/nagios-charm/+git/nagios-charm/+merge/369496
--
Your team Nagios Charm
Need to know if this is consistent with alerts going from CRITICAL to OK
without flapping. If they're going from CRITICAL to WARNING, or
flapping is detected, then the notification won't be sent.
Likely there's a better Pagerduty integration we can write that conveys
current status, but the
This charm needs to be re-worked to include a newer pytest framework
that is standard for the llama-charmers structure. That will remove the
Amulet tests, and replace with functional tests that are more complete
(and work on all the platforms supported). Marking wishlist since that
work is
** Changed in: charm-nagios
Status: New => In Progress
** Changed in: charm-nagios
Assignee: (unassigned) => Sandor Zeestraten (szeestraten)
** Changed in: charm-nagios
Importance: Undecided => Wishlist
--
You received this bug notification because you are a member of Nagios
The proposal to merge ~aieri/charm-nagios:bug/1843863 into charm-nagios:master
has been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~aieri/charm-nagios/+git/nagios-charm/+merge/378160
--
Your team Nagios Charm developers is subscribed to branch
** Changed in: charm-nagios
Importance: Undecided => Wishlist
** Changed in: charm-nagios
Status: New => Triaged
--
You received this bug notification because you are a member of Nagios
Charm developers, which is subscribed to Nagios Charm.
https://bugs.launchpad.net/bugs/1864970
Review: Needs Fixing
Many thanks for this patch.
I've made a few comments regarding the config options in config.yaml, around
documentation. The change itself seems good, and has tested OK via a manual
deployment test.
Diff comments:
> diff --git a/config.yaml b/config.yaml
> index
** Changed in: charm-nagios
Status: In Progress => Fix Committed
--
You received this bug notification because you are a member of Nagios
Charm developers, which is subscribed to Nagios Charm.
https://bugs.launchpad.net/bugs/1843863
Title:
Postfix installed listens on all interfaces by
** Changed in: charm-nagios
Status: In Progress => Fix Released
--
You received this bug notification because you are a member of Nagios
Charm developers, which is subscribed to Nagios Charm.
https://bugs.launchpad.net/bugs/1811997
Title:
Password(s) stored on file(s) readable by
Review: Approve
LGTM
--
https://code.launchpad.net/~giulio.cervera/charm-nagios/+git/nagios-charm/+merge/378449
Your team Nagios Charm developers is subscribed to branch charm-nagios:master.
--
Mailing list: https://launchpad.net/~nagios-charmers
Post to :
I'm unable to find a reproducer for this, it's likely either been fixed
by an update since the bug was reported, or I need to get some more info
on how to demonstrate this issue.
** Changed in: charm-nagios
Status: Triaged => Incomplete
--
You received this bug notification because you
The hostgroups config item only takes effect for when the hosts are
exported via export_nagios_definitions, this is not sent across
relations at this stage. We could clear that up by changing the
documentation, or better yet add the information to the relation data
and update Nagios to include
** Changed in: charm-nagios
Importance: Undecided => Wishlist
** Changed in: charm-nagios
Status: New => Triaged
--
You received this bug notification because you are a member of Nagios
Charm developers, which is subscribed to Nagios Charm.
https://bugs.launchpad.net/bugs/1864969
So far, I've been unable to reproduce Aurelien's reproducer, but I'm
also aware that the Nagios charm takes a while to complete all the
iterations of monitors-relation-changed. The hook does include a
service reload, and on my tests I have seen new hosts appear in the list
after some time,
Current versions of the charm are OK now without needing the extra
switch.
** Changed in: charm-nrpe
Status: New => Invalid
--
You received this bug notification because you are a member of Nagios
Charm developers, which is subscribed to Nagios Charm.
Xav Paice has proposed merging ~xavpaice/hw-health-charm:LP1814867 into
hw-health-charm:master.
Commit message:
Update README for tools resource info
Requested reviews:
Canonical IS Reviewers (canonical-is-reviewers)
Nagios Charm developers (nagios-charmers)
Related bugs:
Bug #1814867
Xav Paice has proposed merging ~xavpaice/hw-health-charm:LP1838562 into
hw-health-charm:master.
Requested reviews:
Nagios Charm developers (nagios-charmers)
Related bugs:
Bug #1838562 in hw-health-charm: "cron_ipmi_sensors.py can get blocked if PID
file not removed&quo
Updated the change to add the file paths in the error messages trapped.
--
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/378128
Your team Nagios Charm developers is subscribed to branch
hw-health-charm:master.
--
Mailing list:
Review: Approve
LGTM
--
https://code.launchpad.net/~peter-sabaini/hw-health-charm/+git/hw-health-charm/+merge/378151
Your team Nagios Charm developers is subscribed to branch
hw-health-charm:master.
--
Mailing list: https://launchpad.net/~nagios-charmers
Post to :
Xav Paice has proposed merging ~xavpaice/hw-health-charm:fix_unittests into
hw-health-charm:master.
Requested reviews:
Nagios Charm developers (nagios-charmers)
For more details, see:
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/379414
--
Your team Nagios
https://code.launchpad.net/~peter-sabaini/hw-health-charm/+git/hw-health-charm/+merge/378123
starts to address the getcwd() issues, I think we can complete that change
with these fixed too after a rebase.
--
Review: Needs Fixing
This change unfortunately needs rebasing against master, currently there's more
merge conflicts than Git can handle.
--
https://code.launchpad.net/~afreiberger/hw-health-charm/+git/hw-health-charm/+merge/374838
Your team Nagios Charm developers is subscribed to branch
The proposal to merge ~xavpaice/hw-health-charm:fix_unittests into
hw-health-charm:master has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/379414
--
Your team Nagios Charm developers
Xav Paice has proposed merging ~xavpaice/hw-health-charm:lp1855095 into
hw-health-charm:master.
Requested reviews:
Nagios Charm developers (nagios-charmers)
Related bugs:
Bug #1855095 in hw-health-charm: "hpsa driver not implemented"
https://bugs.launchpad.net/hw-health-charm/+b
commit message updated - I've left the code there since the addition is on the
roadmap and it'll save some time looking up vendor strings in the future.
--
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/379431
Your team Nagios Charm developers is subscribed to
The proposal to merge ~xavpaice/hw-health-charm:lp1855095 into
hw-health-charm:master has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~xavpaice/hw-health-charm/+git/hw-health-charm/+merge/379431
--
Your team Nagios Charm developers is
Review: Needs Fixing
Needs a review and update for the changes made to master since this was
written, this is a massive rewrite and necessary work, but needs some focus
from our engineering to complete.
--
https://code.launchpad.net/~woutervb/nagios-charm/+git/nagios-charm/+merge/364360
Your
I'd prefer to see a subordinate charm for this kind of option, adding the
packages and package config, plus dropping an extra .cfg in the Nagios config
dirs, to allow the extra contact. However, given that the charm includes
contacts-cfg.tmpl and that templates the file which defines the
Redirect to nagios-charm (not package)
** Package changed: nagios (Juju Charms Collection) => nagios-charm
** Changed in: nagios-charm
Status: New => Triaged
** Changed in: nagios-charm
Importance: Undecided => Wishlist
--
You received this bug notification because you are a member
Review: Approve
Trivial change, will merge without Canonical IS Reviewers input (been a couple
of weeks waiting now).
--
https://code.launchpad.net/~aieri/nagios-charm/+git/nagios-charm/+merge/378116
Your team Nagios Charm developers is subscribed to branch nagios-charm:master.
--
Mailing
The proposal to merge ~aieri/nagios-charm:bug/1843863 into nagios-charm:master
has been updated.
Status: Needs review => Approved
For more details, see:
https://code.launchpad.net/~aieri/nagios-charm/+git/nagios-charm/+merge/378160
--
Your team Nagios Charm developers is subscribed to
Review: Approve
Tested, lgtm. May need a rebase.
--
https://code.launchpad.net/~aieri/nagios-charm/+git/nagios-charm/+merge/378160
Your team Nagios Charm developers is subscribed to branch nagios-charm:master.
--
Mailing list: https://launchpad.net/~nagios-charmers
Post to :
Thanks for the contribution.
I'm concerned that we have the code to enable the module, but nothing to
actually deploy the module itself, meaning that this change alone doesn't
achieve much. Adding the module would be ideally put into a subordinate charm,
along with dropping a fresh config
The docs are pretty clear, this is an option for the main config file only, and
given the age of Nagios 3 it's unlikely to be something that'll change any time
soon.
Since we're in the middle of a release, I'll postpone this till after the
release and target for the next one.
--
** Merge proposal linked:
https://code.launchpad.net/~giulio.cervera/nagios-charm/+git/nagios-charm/+merge/378449
--
You received this bug notification because you are a member of Nagios
Charm developers, which is subscribed to Nagios Charm.
https://bugs.launchpad.net/bugs/1860669
Title:
** Changed in: charm-nagios
Importance: Undecided => Wishlist
** Changed in: charm-nagios
Status: New => Triaged
--
You received this bug notification because you are a member of Nagios
Charm developers, which is subscribed to Nagios Charm.
https://bugs.launchpad.net/bugs/1866624
Review: Needs Fixing
There's some merge conflict markers in hooks/upgrade-charm which need sorting,
and likely a rebase against master to ensure there's no further conflicts.
Other than that, lgtm.
--
https://code.launchpad.net/~szeestraten/charm-nagios/+git/nagios-charm/+merge/379943
Your
Xav Paice has proposed merging ~xavpaice/charm-nagios:bug/1819537 into
charm-nagios:master.
Requested reviews:
Nagios Charm developers (nagios-charmers)
Related bugs:
Bug #1819537 in Nagios Charm: "config-changed hook fails with
enable_livestatus enabled"
https://bugs.launchpad
Xav Paice has proposed merging ~xavpaice/charm-nagios:bug/1819537 into
charm-nagios:master.
Requested reviews:
Nagios Charm developers (nagios-charmers)
Related bugs:
Bug #1819537 in Nagios Charm: "config-changed hook fails with
enable_livestatus enabled"
https://bugs.launchpad
Reproducer:
juju deploy cs:nagios-38
wait for hooks to settle
juju config nagios enable_livestatus=true
2020-09-30 07:32:28 WARNING unit.nagios/0.config-changed logger.go:60 Traceback
(most recent call last):
2020-09-30 07:32:28 WARNING unit.nagios/0.config-changed logger.go:60 File
Ah... the functest has enable livestatus enabled at install, and hooks/install
does the package install for that.
--
https://code.launchpad.net/~xavpaice/charm-nagios/+git/nagios-charm/+merge/391667
Your team Nagios Charm developers is subscribed to branch charm-nagios:master.
--
Mailing list:
1 - 100 of 119 matches
Mail list logo