error in udev rules for hyperv net rules

Bug #1361272 reported by Ben Howard
10
This bug affects 1 person
Affects Status Importance Assigned to Milestone
systemd (Ubuntu)
Fix Released
High
Unassigned
Trusty
Fix Released
Undecided
Unassigned
Utopic
Fix Released
High
Unassigned
udev (Ubuntu)
Invalid
Undecided
Unassigned
Precise
Fix Released
Undecided
Unassigned

Bug Description

There is an error in the syntax where the MAC address does not match if the alpha characters are upper case rather than all lower case. For example, these rules are not matched properly:

ENV{MATCHADDR}=="00:0c:29:*|00:50:56:*|00:05:69:*|00:1C:14:*" -> should be 00:1c:14:*
ENV{MATCHADDR}=="00:12:5A:*" -> should be "00:12:5a:*"
ENV{MATCHADDR}=="00:17:FA:*" -> should be "00:17:fa:*"
ENV{MATCHADDR}=="00:50:F2:*" -> should be "00:50:f2:*"
ENV{MATCHADDR}=="Dc:B4:c4:*" -> should be "dc:b4:c4:*"

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

This has been confirmed by Microsoft.

Martin Pitt (pitti)
affects: ubuntu → systemd (Ubuntu)
Changed in systemd (Ubuntu):
status: Confirmed → Triaged
Revision history for this message
Ben Howard (darkmuggle-deactivatedaccount) wrote :
Revision history for this message
Ben Howard (darkmuggle-deactivatedaccount) wrote :
Revision history for this message
Ben Howard (darkmuggle-deactivatedaccount) wrote :
tags: added: patch
Martin Pitt (pitti)
no longer affects: systemd (Ubuntu Precise)
no longer affects: udev (Ubuntu Utopic)
no longer affects: udev (Ubuntu Trusty)
Changed in udev (Ubuntu):
status: New → Invalid
Changed in udev (Ubuntu Precise):
status: New → Triaged
Changed in systemd (Ubuntu Trusty):
status: New → Triaged
Revision history for this message
Martin Pitt (pitti) wrote :

Committed to Debian packaging git for utopic: http://anonscm.debian.org/gitweb/?p=pkg-systemd/systemd.git;a=commitdiff;h=e8f7d56

Ben, you missed one more captial letter, I fixed that in the commit above. I'll do the same for the SRUs.

Changed in systemd (Ubuntu Utopic):
status: Triaged → Fix Committed
Revision history for this message
Martin Pitt (pitti) wrote :

trusty/precise SRUs uploaded to review queue. These debdiffs were fine already. Note that the previous trusty SRU hasn't been released to trusty-updates yet, so reviewing/accepting that is blocked for a few days.

Changed in systemd (Ubuntu Trusty):
status: Triaged → In Progress
Changed in udev (Ubuntu Precise):
status: Triaged → In Progress
Revision history for this message
Chris J Arges (arges) wrote : Please test proposed package

Hello Ben, or anyone else affected,

Accepted udev into precise-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/udev/175-0ubuntu9.6 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Changed in udev (Ubuntu Precise):
status: In Progress → Fix Committed
tags: added: verification-needed
Revision history for this message
Chris J Arges (arges) wrote :

Hello Ben, or anyone else affected,

Accepted systemd into trusty-proposed. The package will build now and be available at http://launchpad.net/ubuntu/+source/systemd/204-5ubuntu20.6 in a few hours, and then in the -proposed repository.

Please help us by testing this new package. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Your feedback will aid us getting this update out to other Ubuntu users.

If this package fixes the bug for you, please add a comment to this bug, mentioning the version of the package you tested, and change the tag from verification-needed to verification-done. If it does not fix the bug for you, please add a comment stating that, and change the tag to verification-failed. In either case, details of your testing will help us make a better decision.

Further information regarding the verification process can be found at https://wiki.ubuntu.com/QATeam/PerformingSRUVerification . Thank you in advance!

Changed in systemd (Ubuntu Trusty):
status: In Progress → Fix Committed
Revision history for this message
Ben Howard (darkmuggle-deactivatedaccount) wrote :

Fix confirmed. Marking as 'verification-done'.

tags: added: verification-done
removed: verification-needed
Changed in udev (Ubuntu Precise):
assignee: nobody → Ben Howard (utlemming)
Changed in systemd (Ubuntu Trusty):
assignee: nobody → Ben Howard (utlemming)
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package systemd - 208-8ubuntu2

---------------
systemd (208-8ubuntu2) utopic; urgency=medium

  [ Ben Howard ]
  * 75-persistent-net-generator.rules: Fix matches of HyperV. (LP: #1361272)

  [ Martin Pitt ]
  * Add new autopkgtest boot_and_services which reboots the testbed with
    systemd as init and verifies that the most critical services like lightdm,
    D-BUS, NetworkManager, and cron work. (To be extended in the future)
  * Fix udev path in udev(7). (LP: #1357782)
  * path_id: suppress ID_PATH for devices with an unknown parent device type.
    (LP: #1321816)
  * udev-builtin-keyboard: Allow numeric key codes. (LP: #1247584)
 -- Martin Pitt <email address hidden> Mon, 08 Sep 2014 14:44:35 +0200

Changed in systemd (Ubuntu Utopic):
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package udev - 175-0ubuntu9.6

---------------
udev (175-0ubuntu9.6) precise; urgency=medium

  * Fix syntax of HyperV net rules (LP: #1361272).
 -- Ben Howard <email address hidden> Mon, 25 Aug 2014 11:23:38 -0600

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

The verification of the Stable Release Update for udev 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 regressions.

Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package systemd - 204-5ubuntu20.6

---------------
systemd (204-5ubuntu20.6) trusty; urgency=medium

  * Fix syntax of HyperV net rules (LP: #1361272).
 -- Ben Howard <email address hidden> Mon, 25 Aug 2014 11:02:20 -0600

Changed in systemd (Ubuntu Trusty):
status: Fix Committed → Fix Released
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.