Landscape Client

Monitor doesn't work with gir1.0-gudev-1.0 installed

Bug #747498 reported by Thomas Herve
12
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Landscape Client
Fix Released
High
Thomas Herve
Landscape Client 11.04
landscape-client (Ubuntu)
Fix Released
Undecided
Chuck Short
Karmic
Won't Fix
Undecided
Unassigned
Lucid
Fix Released
Undecided
Unassigned
Maverick
Fix Released
Undecided
Unassigned
Natty
Fix Released
Undecided
Chuck Short

Bug Description

The changes (in GIR API I think?) breaks it this way:

Traceback (most recent call last):
  File "/usr/bin/landscape-monitor", line 8, in <module>
    run(sys.argv)
  File "/usr/lib/python2.6/dist-packages/landscape/monitor/service.py", line 63, in run
    run_landscape_service(MonitorConfiguration, MonitorService, args)
  File "/usr/lib/python2.6/dist-packages/landscape/service.py", line 82, in run_landscape_service
    service = service_class(configuration)
  File "/usr/lib/python2.6/dist-packages/landscape/monitor/service.py", line 26, in __init__
    self.plugins = self.get_plugins()
  File "/usr/lib/python2.6/dist-packages/landscape/monitor/service.py", line 34, in get_plugins
    for plugin_name in self.config.plugin_factories]
  File "/usr/lib/python2.6/dist-packages/landscape/monitor/mountinfo.py", line 42, in __init__
    self._gudev_client = GUdev.Client.new([])
  File "/usr/lib/pymodules/python2.6/gtk-2.0/gi/module.py", line 88, in __getattr__
    self.__class__.__name__, name))

Revision history for this message
Thomas Herve (therve) wrote :

Fixed in r322.

Changed in landscape-client:
status: New → Fix Committed
Chuck Short (zulcss)
Changed in landscape-client (Ubuntu Natty):
assignee: nobody → Chuck Short (zulcss)
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

This is the patch, from Thomas Herve.

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

This bug was fixed in the package landscape-client - 11.02-0ubuntu0.11.04.1

---------------
landscape-client (11.02-0ubuntu0.11.04.1) natty; urgency=low

  * debian/control, debian/rules: Add quilt
  * debian/patches/fix-landscape-monitor.patch: Fix landscape
    monitoring with gir1.0-gudev-1.0 installed. (LP: #747498)
 -- Chuck Short <email address hidden> Fri, 08 Apr 2011 09:46:24 -0400

Changed in landscape-client (Ubuntu Natty):
status: New → Fix Released
Revision history for this message
Clint Byrum (clint-fewbar) wrote :

Chuck, can you please add a clear test case for this so we can verify it for SRU?

Leaving in -proposed queue until SRU information is added.

Revision history for this message
Andreas Hasenack (ahasenack) wrote :

Clint, this page describes all the tests: https://wiki.canonical.com/Landscape/LandscapeDocumentation/ClientSRUTests/11.02

It's canonical only, sorry. The specific test for this bug is simple and I paste it below:
"""
* gir1.0-gudev test
  * for lucid, maverick and natty, make sure landscape-client can be restarted and keeps running if the gir1.0-gudev package is also installed (bug #747498)
"""

That test report is from my own private testing, before this was uploaded to -proposed. My intention is to go over them all again, but basically just checking that the packages in -proposed have the same build as the ones I tested before uploading.

Revision history for this message
Clint Byrum (clint-fewbar) wrote : Please test proposed package

Accepted landscape-client into lucid-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in landscape-client (Ubuntu Lucid):
status: New → Fix Committed
tags: added: verification-needed
Changed in landscape-client (Ubuntu Karmic):
status: New → Fix Committed
Revision history for this message
Clint Byrum (clint-fewbar) wrote :

Accepted landscape-client into karmic-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

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

Accepted landscape-client into maverick-proposed, the package will build now and be available in a few hours. Please test and give feedback here. See https://wiki.ubuntu.com/Testing/EnableProposed for documentation how to enable and use -proposed. Thank you in advance!

Changed in landscape-client (Ubuntu Maverick):
status: New → Fix Committed
Revision history for this message
Martin Pitt (pitti) wrote :

This package has been in karmic-proposed for some time without verification. I removed it as karmic is end-of-life now.

Changed in landscape-client (Ubuntu Karmic):
status: Fix Committed → Won't Fix
Revision history for this message
Andreas Hasenack (ahasenack) wrote :

Confirmed working for lucid and maverick with their respective proposed packages:
- 11.02-0ubuntu0.10.04.1
- 11.02-0ubuntu0.10.10.1

Martin Pitt (pitti)
tags: added: verification-done
removed: verification-needed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package landscape-client - 11.02-0ubuntu0.10.10.1

---------------
landscape-client (11.02-0ubuntu0.10.10.1) maverick-proposed; urgency=low

  * debian/control, debian/rules: Add quilt
  * debian/patches/fix-landscape-monitor.patch: Fix landscape
    monitoring with gir1.0-gudev-1.0 installed. (LP: #747498)

landscape-client (11.02-0ubuntu0.10.10.0) maverick-proposed; urgency=low

  * New upstream version (LP: #727324)

    - Exit gracefully instead of crashing when the filesystem is
      read-only (LP: #649997).

    - Drop hal requirement (LP: #708502).

    - Enable HTTP compression in Curl (LP: #297623).

    - Explicitly name log files that need to be rotated (LP: #634236).

    - Assorted test suite fixes

    - Use a better load check for the sysinfo wrapper, taking into account the
      number of cores (LP: #643565).

    - Add an option to bootstrap cloud instances using cloud-init
      (LP: #701972).

    - Fix packaging for Natty (LP: #688115).

    - Force deletion of all the persist data for the monitoring plugins at
      resynchronization, instead of relying each one of them to do
      (LP: #688161).

    - Don't send the mount-activity message to the server anymore
      (LP: #688514).

    - Workaround a new behavior in NetworkManager where getfqdn would report
      localhost instead of useful hostname (LP: #649142).

landscape-client (1.5.5.1-0ubuntu0.10.10.0) maverick; urgency=low

  * The client network plugin would send erroneous data if a network
    interface was removed (and its kernel module removed as well) and
    then readded (LP: #641264).
 -- Chuck Short <email address hidden> Tue, 12 Apr 2011 13:56:38 -0400

Changed in landscape-client (Ubuntu Maverick):
status: Fix Committed → Fix Released
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package landscape-client - 11.02-0ubuntu0.10.04.1

---------------
landscape-client (11.02-0ubuntu0.10.04.1) lucid-proposed; urgency=low

  * debian/control, debian/rules: Add quilt
  * debian/patches/fix-landscape-monitor.patch: Fix landscape:
    monitoring with gir1.0-gudev-1.0 installed. (LP: #747498)

landscape-client (11.02-0ubuntu0.10.04.0) lucid-proposed; urgency=low

  * New upstream version (LP: #727324)

    - Exit gracefully instead of crashing when the filesystem is
      read-only (LP: #649997).

    - Drop hal requirement (LP: #708502).

    - Enable HTTP compression in Curl (LP: #297623).

    - Explicitly name log files that need to be rotated (LP: #634236).

    - Assorted test suite fixes

    - Use a better load check for the sysinfo wrapper, taking into account the
      number of cores (LP: #643565).

    - Add an option to bootstrap cloud instances using cloud-init
      (LP: #701972).

    - Fix packaging for Natty (LP: #688115).

    - Force deletion of all the persist data for the monitoring plugins at
      resynchronization, instead of relying each one of them to do
      (LP: #688161).

    - Don't send the mount-activity message to the server anymore
      (LP: #688514).

    - Workaround a new behavior in NetworkManager where getfqdn would report
      localhost instead of useful hostname (LP: #649142).
 -- Chuck Short <email address hidden> Tue, 12 Apr 2011 14:38:20 -0400

Changed in landscape-client (Ubuntu Lucid):
status: Fix Committed → Fix Released
Andreas Hasenack (ahasenack)
Changed in landscape-client:
status: Fix Committed → 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.