init script check for sysfs prevents hald to work correctly in openvz on hardy

Bug #399629 reported by Guido Nickels
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
hal (Ubuntu)
Fix Released
Undecided
Martin Pitt

Bug Description

Hi!

The script /etc/init.d/hal does some checks on service start for inotify support, dbus and sysfs stuff.

Unfortunately, the sysfs check just looks for directory /sys/kernel. This directory is not available in openvz containers. However, other parts of sysfs (/sys/class, /sys/devices) are in place. Maybe that sysfs check can be changed to look for one of these instead?

One could ask why to use hald in openvz environment at all, but this problem breaks support for canonical's landscape-client on openvz as it depends on running hald.

Regards,

Guido

Revision history for this message
Martin Pitt (pitti) wrote :

Fixed in bzr.

Changed in hal (Ubuntu):
assignee: nobody → Martin Pitt (pitti)
status: New → Fix Committed
Revision history for this message
Martin Pitt (pitti) wrote :
Revision history for this message
Launchpad Janitor (janitor) wrote :
Download full text (3.1 KiB)

This bug was fixed in the package hal - 0.5.13-1ubuntu1

---------------
hal (0.5.13-1ubuntu1) karmic; urgency=low

  * Merge with Debian experimental and svn head, to minimize our Delta.
    Drop MacBook and smartdimmer brightness support, since hal is being
    deprecated, and already not used for brightness changes any more.
    Remaining Ubuntu changes:
   - Enable support for X11 input hotplugging:
     + debian/rules: Do not remove 10-x11-input.fdi, we want to install it by
       default.
     + Drop 10-x11-input.fdi from debian/hal.examples, since we install
       it by default.
   - debian/rules: Disable support for keymaps, now done by udev.
   - debian/rules: Disable support for ACL management, now done by udev. Drop
     "acl" dependency.

hal (0.5.13-2) UNRELEASED; urgency=low

  [ Martin Pitt ]
  * debian/control: Drop libsmbios-dev from build depends. Kernel 2.6.30
    provides dell_laptop kernel module which exports these interfaces into
    standard "generic" ones. Drop recommends for libsmbios-bin.
  * debian/control: Bump libblkid-dev build dep to >= 2.15 as done by
    upstream recently; 1.43 was a synthetic version number before the
    util-linux release number was determined.
  * debian/hal.init: Check for /sys/devices, not /sys/kernel. The latter
    doesn't exist in OpenVZ. (LP: #399629)

hal (0.5.13-1) experimental; urgency=low

  * New upstream release.
    - Fix crash when assembling certain MD devices. (Closes: #522136)
    - Fixes for GNU/kFreeBSD.
    - Set default udev_prefix to ${exec_prefix}/lib in configure.in.
    - Define new (sub-)namespace(s) and policy for smart card readers.
    - Ignore ADD events if there is already a device with same sysfs_path.
    - probe-volume: remove needless check for empty fstype.
    - Use $libexecdir not $libdir/hal to define private hal directory.
    - Various startup speedups.
    - Add hal-set-property --direct.
    - addon-acpi.c: Support acpid.
    - 10-x11-input.fdi: Add tablets.
    - Add input.keys capability to input buttons.
    - Add support for devices on the SSB and ibme bus.
  * debian/patches/11-direct-hal-set-propery.patch
    - Removed, merged upstream.
  * debian/patches/23_addon_acpi.patch
    - Removed, merged upstream.
  * Add support for GNU/kFreeBSD. (Closes: #528383)
    Thanks to Aurelien Jarno and Cyril Brulebois for the patch(es).
  * debian/control
    - Change Build-Depends from libvolume-id-dev to libblkid-dev (>= 1.43).
    - Bump Standards-Version to 3.8.2. No further changes.
    - Wrap hal dependencies.
    - Demote pm-utils to Recommends now that pm-utils is in the laptop and
      desktop task and Recommends are installed by default. (Closes: #477913)
  * debian/hal.init
    - Refactor safety checks into a separate function and make sure they are
      also run on restart. (Closes: #537251)
    - Use s-s-d's --retry option on stop which allows us to safely remove the
      ugly "sleep 1" between start and stop within restart.
  * debian/patches/20_cpufreq_warning_message_fix.patch
    - Fix a misleading warning message in ondemand_set_consider_nice().
      (Closes: #532999)

 -- Martin Pitt <email address hidden> Wed, 29 Jul 2009...

Read more...

Changed in hal (Ubuntu):
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.