[SRU] regression: walinuxagent incorrectly sets unsupported dhcp parameter

Bug #1308974 reported by Ben Howard
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
walinuxagent (Ubuntu)
Invalid
Medium
Unassigned
Precise
Fix Released
Medium
Unassigned

Bug Description

SRU Justification

Impact: The fix for bug #1268050 on 12.04 switches the dhcpclient.conf configuration from "<hostname>" to gethostname(). This fix, while valid for later versions of isc-dhcp-client, in invalid for 12.04. Therefore, this fix should be reverted and remediated.

Fix: Revert previous patch

Test case:
- Launch new Azure instance with serial of 20140408
- Install new package
- Confirm that /etc/dhcp/dhclient.conf has 'send hostname "<hostname>"'

Revision history for this message
Ben Howard (darkmuggle-deactivatedaccount) wrote :
summary: - regression: walinuxagent incorrectly sets unsupported dhcp parameter
+ [SRU] regression: walinuxagent incorrectly sets unsupported dhcp
+ parameter
Revision history for this message
Ben Howard (darkmuggle-deactivatedaccount) wrote :

Tested patch, and confirmed it removes the regression.

description: updated
Revision history for this message
Ben Howard (darkmuggle-deactivatedaccount) wrote :

Upload fix, pending SRU.

Changed in walinuxagent (Ubuntu Precise):
status: New → In Progress
importance: Undecided → Medium
assignee: nobody → Ben Howard (utlemming)
Changed in walinuxagent (Ubuntu):
status: In Progress → Invalid
tags: added: precise regression-update
Revision history for this message
Stephen A. Zarkos (stevez) wrote :

Hi Ben,

Is it necessary to revert the patch to /usr/sbin/waagent itself rather than just the packaging? The original issue was about users calling 'waagent -deprovision', so my concern is that this issue might pop up again if we revert that patch.

As far as I can tell the fix to waagent is working as expected. The agent currently calls UpdateAndPublishHostName()->UpdateAndPublishHostNameCommon() only if "Provisioning.MonitorHostName=y" in waagent.conf (it is set to 'n' on 12.04), or if we run -deprovision. If it sees ether send '<hostname>' or 'gethostname()' it will not touch the dhclient.conf file. The problem here is that dhclient.conf is misconfigured after even the first provision.

Thanks,
Steve

Revision history for this message
Ben Howard (darkmuggle-deactivatedaccount) wrote :

Confirmed Stephen's comment with packaging changes. The attached debdiff fixes the problem.

I've confirmed Stephen's assessment and confirmed the fix via packaging changes. Uploaded new version for SRU.

Revision history for this message
Ben Howard (darkmuggle-deactivatedaccount) wrote :
Robert C Jennings (rcj)
description: updated
Revision history for this message
Robert C Jennings (rcj) wrote :

Verified with version 1.3.2-0ubuntu4~12.04.7 on Azure. Installation of package resulted in 'send host-name "<hostname>";' in /etc/dhcp/dhclient.conf.

tags: added: verification-done
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package walinuxagent - 1.3.2-0ubuntu4~12.04.7

---------------
walinuxagent (1.3.2-0ubuntu4~12.04.7) precise; urgency=medium

  * Revert of regression introduced with LP: #1268050: use compatible
    DHCP send hostname (LP: #1308974)
 -- Ben Howard <email address hidden> Thu, 17 Apr 2014 11:51:14 -0600

Changed in walinuxagent (Ubuntu Precise):
status: In Progress → Fix Released
Revision history for this message
Chris J Arges (arges) wrote : Update Released

The verification of the Stable Release Update for walinuxagent has completed successfully and the package has now been released to -updates. Subsequently, the Ubuntu Stable Release Updates Team is being unsubscribed and will not receive messages about this bug report. In the event that you encounter a regression using the package from -updates please report a new bug using ubuntu-bug and tag the bug report regression-update so we can easily find any regresssions.

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.