Juju Charm Tools

helpers/sh/net.sh function ch_get_ip doesn't works for CNAME records

Bug #1303825 reported by Jorge Niedbalski
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Juju Charm Tools
Fix Released
Undecided
Unassigned

Bug Description

[Impact]
* The function ch_get_ip() located on helpers/sh/net.sh function doesn't works for hosts that resolves to CNAME entries.
* This causes that any charm using this helper function fails with unexpected results.

[Test Case]
$ source helpers/sh/net.sh
$ ch_get_ip cname.host.com
- nothing returned

See original description

Related branches

lp:~niedbalski/charm-tools/fix-lp-1303825
Marco Ceppi (community): Approve
Diff: 12 lines (+1/-1)
1 file modified
helpers/sh/net.sh (+1/-1)
Jorge Niedbalski (niedbalski)
description: updated
Jorge Niedbalski (niedbalski)
Changed in charm-tools:
assignee: nobody → Jorge Niedbalski (niedbalski)
status: New → In Progress
Revision history for this message
Jorge Niedbalski (niedbalski) wrote :

Proposed merge request

Changed in charm-tools:
assignee: Jorge Niedbalski (niedbalski) → nobody
status: In Progress → New
Revision history for this message
Marco Ceppi (marcoceppi) wrote :

I'm curious why this is being patched, this version of charm-helpers is no longer supported, in fact it's no longer being built outside of precise. We've begun recommending people either factor these out or embed them directly. If you need this fixed for a specific charm it's better to simply embed that file in that charm rather than relying on the packaged version. We're working on moving everything to the lp:charm-helpers project going forward.

Revision history for this message
Jorge Niedbalski (niedbalski) wrote :

Hello marcoceppi,

In fact i followed your recommendation and I embedded this package into the nrpe charm ( merge request sent bug #1303862 ), but i think is good to have that changes into upstream for further references on related errors.

Marco Ceppi (marcoceppi)
Changed in charm-tools:
status: New → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

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