Ubuntu
sugar-presence-service package

Sugar Presence Service fails to connect to jabber server

Bug #284557 reported by Morgan Collett on 2008-10-16

This bug affects 1 person

Affects		Status	Importance	Assigned to	Milestone
	sugar-presence-service (Ubuntu)	Fix Released	Undecided	Unassigned

Bug Description

Binary package hint: sugar-presence-service

Sugar Presence Service fails to connect to a jabber server, with the error in presenceservice.log being:
1224179059.306887 DEBUG s-p-s.psutils: Error getting NetworkManager devices: org.freedesktop.DBus.Error.UnknownMethod: Method "getDevices" with signature "" on interface "org.freedesktop.NetworkManager" doesn't exist

This is because PS (Presence Service) works with NetworkManager 0.6.x, and not 0.7 which is in Intrepid. NM 0.7 has a different D-Bus interface for this call.

The failing call is to watch NM to see when we have a valid IP address. There is a fallback to manually check the IP address, but it isn't called if NM is seen on the session bus, so I have a patch to disable the initial check for NM.

debdiff on its way.

Related branches

lp:ubuntu/karmic/sugar-presence-service

Bug Watch Updater (bug-watch-updater) on 2008-10-16

Changed in sugar-presence-service:
status:	Unknown → New

Revision history for this message

Morgan Collett (morgan) wrote on 2008-10-16:

sugar-presence-service.debdiff Edit (2.5 KiB, text/plain)

Revision history for this message

Morgan Collett (morgan) wrote on 2008-10-16:

debdiff attached.

Changed in sugar-presence-service:
status:	New → Confirmed

Revision history for this message

James Westby (james-w) wrote on 2008-10-16:

Hi Morgan,

How different is the 0.7 interface for this call? Are there any other
issues with using NM 0.7?

Thanks,

James

Revision history for this message

James Westby (james-w) wrote on 2008-10-16:

Ah, I see it is described in the upstream bug. I'll probably
sponsor this then, but it would be nice to have the real fix.

Thanks,

James

Revision history for this message

Morgan Collett (morgan) wrote on 2008-10-17:

The 0.7 interface is very different. The upstream Sugar team intend to fix this soon but it's been pending for a long time so I don't have confidence we'll have a real fix before intrepid releases.

I am watching the progress, so if they come up with a fix I'll log a bug to revert this patch and apply that one.

In the mean time, this is quite critical to get into intrepid as collaboration is one of the core features of Sugar.

Revision history for this message

Launchpad Janitor (janitor) wrote on 2008-10-17:

This bug was fixed in the package sugar-presence-service - 0.82.0-1ubuntu1

---------------
sugar-presence-service (0.82.0-1ubuntu1) intrepid; urgency=low

* NM 0.7 not supported, so don't watch it for IP address changes,
rather use the fallback (LP: #284557)

-- Morgan Collett <email address hidden> Thu, 16 Oct 2008 20:21:53 +0200

Changed in sugar-presence-service:
status:	Confirmed → Fix Released

Revision history for this message

Morgan Collett (morgan) wrote on 2008-10-17:

I don't know what happened, but this got released somehow without the patch applied to the actual file in the source. I see the patch in the diff.gz but if I download the deb, psutil.py isn't patched.

(Also, I didn't see anyone actually approve this for release???)

Revision history for this message

Guillaume Desmottes (cassidy) wrote on 2008-10-17:

Morgan: do you think it would be worth to merge this patch upstream as well?

Revision history for this message

Morgan Collett (morgan) wrote on 2008-10-17:

Guillaume: it's the workaround from #6248 and only applies to disable watching for NM. I think upstream should wait for the real patch. Marco (or Simon?) is planning on working on NM 0.7 for Sugar next week - we should see what they do, and get a patch suitable for 0.82.x.

Revision history for this message

James Westby (james-w) wrote on 2008-10-17:

#10

Hi,

Bugfixes don't need approval at this stage. I think I know what is
wrong with the upload, so I'll fix it up. I assumed you had tested the
fix, which I shouldn't do, sorry.

Thanks,

James

Bug Watch Updater (bug-watch-updater) on 2008-11-20