udev-finish.conf fails to merge temporary rules files

Bug #1093731 reported by Yutani
18
This bug affects 2 people
Affects Status Importance Assigned to Milestone
udev (Ubuntu)
Fix Released
Medium
Unassigned

Bug Description

I removed /etc/udev/rules.d/70-persistent-net.rules and rebooted the machine in order to generate a new rules file, but couldn't get a complete 70-persistent-net.rules. A rule about eth0 was written in /run/udev/tmp--70-persitent-net.rules and didn't merged onto /etc/udev/rules.d/70-persistent-net.rules, which udev-finish.conf is
expected to do.

This is probably because the tmp dir path specified in udev-finish.conf is different from that in /lib/udev/rule_generator.functions.

/lib/udev/rule_generator.functions:
        RUNDIR=$(udevadm info --run)
        local tmp_rules_file="$RUNDIR/tmp-rules--${RULES_FILE##*/}"

/etc/init/udev-finish.conf:
    # Copy any rules generated while the root filesystem was read-only
    for file in /dev/.udev/tmp-rules--*
    do
        [ -e "$file" ] || continue
        cat "$file" >> "/etc/udev/rules.d/${file##*tmp-rules--}"

In my environment, "udevadm info --run" returns "/run/udev". I think udev-finish.conf should use $(udevadm info --run) instead of "/dev/.udev/".

---
Description: Ubuntu 12.04.1 LTS
Release: 12.04
udev:
  Installed: 175-0ubuntu9.1
  Candidate: 175-0ubuntu9.2

Tags: patch

Related branches

Yutani (hiroaki-yutani)
description: updated
Revision history for this message
Yutani (hiroaki-yutani) wrote :
Revision history for this message
Launchpad Janitor (janitor) wrote :

Status changed to 'Confirmed' because the bug affects multiple users.

Changed in udev (Ubuntu):
status: New → Confirmed
Revision history for this message
Ubuntu Foundations Team Bug Bot (crichton) wrote :

The attachment "udev-finish-rundir.patch" of this bug report has been identified as being a patch. The ubuntu-reviewers team has been subscribed to the bug report so that they can review the patch. In the event that this is in fact not a patch you can resolve this situation by removing the tag 'patch' from the bug report and editing the attachment so that it is not flagged as a patch. Additionally, if you are member of the ubuntu-reviewers team please also unsubscribe the team from this bug report.

[This is an automated message performed by a Launchpad user owned by Brian Murray. Please contact him regarding any issues with the action taken in this bug report.]

tags: added: patch
Revision history for this message
Brian Murray (brian-murray) wrote :

Martin - could you have a look at this?

Changed in udev (Ubuntu):
importance: Undecided → Medium
Revision history for this message
Martin Pitt (pitti) wrote :

Thank you! Sponsoring.

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

This bug was fixed in the package udev - 175-0ubuntu15

---------------
udev (175-0ubuntu15) raring; urgency=low

  * debian/udev.udev-finish.upstart: Use the correct rundir path.
    (LP: #1093731)
 -- Hiroaki Yutani <email address hidden> Fri, 04 Jan 2013 07:43:06 +0100

Changed in udev (Ubuntu):
status: Confirmed → 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.