← Back to team overview

ubuntu-phone team mailing list archive

Re: GMail notifications and account-polld

 

Hi,

Thank you for your reply. :)

Am Mo, 20. Okt, 2014 um 3:53 schrieb John Lenton <john.lenton@xxxxxxxxxxxxx>:
On 20 Oct 2014 06:14, "Niklas Wenzel" <nikwen.developer@xxxxxxxxx> wrote:

Thanks a lot. This works perfectly for me. Please allow me to ask one more question. Can I restart the account-polld process somehow after installing the new package or do I always need to reboot?

You can restart account-polld by entering “restart account-polld”.

Ah, I previously ran that command prefixed with sudo and that didn't work.


Regarding the issues I found: Often, when there is an issue with polling the mail server (which happens quite often, partially due to a bug in the go language), account-polld will stop polling the server for this account until the next reboot.

If something goes wrong with an account it should stop polling for a
few polls, but not forever, so if you are seeing that it would
certainly be a bug. You can trigger polls manually over dbus:

gdbus call --session -d com.ubuntu.AccountPolld -o
/com/ubuntu/AccountPolld -m com.ubuntu.AccountPolld.Poll

Well, without my changes it does stop for me. Due to the following line it never tries to connect again: http://bazaar.launchpad.net/~ubuntu-push-hackers/account-polld/trunk/view/head:/cmd/account-polld/account_manager.go#L118

Oh, that's great. That will speed development up a lot.


What “bug in the go language”?

I'm using this launchpad branch to push my changes to my Nexus 4 for building the packages I test: https://code.launchpad.net/~nikwen/account-polld/bug-fixes

taking a look...

Whilst the current version works around a major bug I encountered, it's definitely not ready yet.

from a quick glance at the branch, I'm assuming what you're calling
the major bug is it not resetting the failure count on an account if
the account is updated?

The major issue was related to the line I linked to above. When an account failed, it never tried to authenticate again.



Can you please be more precise and/or specific when describing these things?

I'll keep working on it and will submit the patch with a better commit message later. The only problem is that account-polld runs in five minute intervals so debugging is quite time-consuming...

 Thanks again. :)

 Niklas

Am So, 19. Okt, 2014 um 10:18 schrieb John Lenton <john.lenton@xxxxxxxxxxxxx>:

Care to share what you see going wrong, and what you think will fix it? I'm interested in what you found. The easiest way to build it on the phone is to get the build dependencies through apt, for which yes you need a writeable image. Once you've got that, the easiest (but not quickest) way of building it is just "dpkg-buildpackage" (you might want to install fakeroot though). It's also possible to build it by hand, but it's a faff. On 19 October 2014 17:11, Niklas Wenzel <nikwen.developer@xxxxxxxxx> wrote:

Hi, Ever since GMail notificiations were introduced, they haven't been working well for me on my Nexus 4. I've reinstalled the system quite a few times since then (partly to test whether it would change the working status of the push notifications) but it didn't help. About one and a half months ago, I even sent a mail (well, actually two mails) to the package maintainer of the account-polld package but I never got an answer. Therefore, I decided to look into the problem myself now. When observing log files today, I noticed that it wasn't just one bug which kept notifications from working but multiple ones. I know where the problem lies in the code and I think that I will be able to figure out how to fix it. What I don't know, however, is how to test my changes. How can I compile and test the account-polld package on my Nexus 4? Does it require a writeable image? Thank you very much in advance. :) Kind regards, Niklas -- Mailing list: https://launchpad.net/~ubuntu-phone Post to : ubuntu-phone@xxxxxxxxxxxxxxxxxxx Unsubscribe : https://launchpad.net/~ubuntu-phone More help : https://help.launchpad.net/ListHelp

Follow ups

References