Bug in recent change re: thruk-agent check rendering

Bug #2035403 reported by Paul Goins
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Thruk External Agent Charm
New
Undecided
Unassigned

Bug Description

In commit ef1b77238970e0ac91a7542a6555e980a82183b3, part of the change performed there introduced use of the -p flag on the check_http call.

Unfortunately, this change assumed that there was a port number explicitly specified in the "url" config parameter - and breaks in the case of URLs with an implicit HTTP or HTTPS port.

This can be worked around via explicitly adding a :80 for HTTP and :443 for HTTPS "url" config values.

Regardless, this change was backwards-incompatible. While I don't think it should be fully backed out, I think the port should be made optional in some way so that the implicit case, which we've depended on previously, will continue to work.

Revision history for this message
Paul Goins (vultaire) wrote (last edit ):

Sorry; wrong charm - thruk-external-agent is where this needs to be filed. (Removed nrpe charm from list of affected projects.)

no longer affects: charm-nrpe
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.