Merge lp:~nikwen/account-polld/directly-poll-with-new-account-data-fix into lp:~ubuntu-push-hackers/account-polld/trunk

[Niklas Wenzel]

Existing accounts which have new account data will now directly poll the server. Otherwise it would wait 5 minutes (until Poll() is called again), recognize it failed before setting the new account data (penaltyCount != 0) and wait 5 minutes again until polling finally happens in the next invokation of Poll().

[Niklas Wenzel (nikwen)]

This possibly also fixes a multithreading issue which may occur in the monitorAccounts() function as without this fix it wouldn't wait for the Poll() calls in "case data := <-watcher.C:".

[Niklas Wenzel (nikwen)]

Ok, please don't merge this yet. I found an error in the logs. It seems like there is some problem in the code which starts the authentication process again. It looks like this happened because the reference to the variable 'account' got updated in the meantime. I'll look into this tomorrow.

2014/10/20 16:02:38 Poll for account 2 has failed: Token expired
2014/10/20 16:02:38 Ending poll for account 2
2014/10/20 16:02:38 Poll for account 3 has failed: Token expired
2014/10/20 16:02:38 Ending poll for account 3
2014/10/20 16:02:40 New account data for existing account with id 2
2014/10/20 16:02:40 Starting poll for account 2
2014/10/20 16:02:40 Polling account 2
2014/10/20 16:02:41 Account 2 has 0 updates to report
2014/10/20 16:02:41 Poll for account 2 was successful
2014/10/20 16:02:41 Ending poll for account 2
2014/10/20 16:02:47 Creating account with id 2 for com.ubuntu.developer.webapps.webapp-gmail_webapp-gmail
2014/10/20 16:02:47 gmail plugin 2: last state loaded from storage
2014/10/20 16:02:47 Starting poll for account 2
2014/10/20 16:02:47 Polling account 2
2014/10/20 16:02:48 Account 2 has 0 updates to report
2014/10/20 16:02:48 Poll for account 2 was successful
2014/10/20 16:02:48 Ending poll for account 2
2014/10/20 16:03:08 Creating account with id 2 for com.ubuntu.developer.webapps.webapp-gmail_webapp-gmail
2014/10/20 16:03:08 gmail plugin 2: last state loaded from storage
2014/10/20 16:03:08 Starting poll for account 2
2014/10/20 16:03:08 Polling account 2
2014/10/20 16:03:09 Account 2 has 0 updates to report
2014/10/20 16:03:09 Poll for account 2 was successful
2014/10/20 16:03:09 Ending poll for account 2

[Niklas Wenzel (nikwen)]

The solution for this problem is described here: https://code.google.com/p/go-wiki/wiki/CommonMistakes#Using_Closures_with_Goroutines

You see, it's been the first time I've ever used Go.

[Niklas Wenzel (nikwen)]

I'll have this running in the background for some time today to see if the fix resolves the issue. It should, however, do in my opinion.

[Niklas Wenzel (nikwen)]

I can confirm that the issue is fixed now. :)

[John Lenton (chipaca)]

This is a rather big hammer for the fix you're doing. Let me dig into it a bit.

review: Needs Fixing

[Niklas Wenzel (nikwen)]

As stated here, this MP still needs to be adjusted: https://code.launchpad.net/~nikwen/account-polld/authenticate-again-fix/+merge/238900/comments/591702

[Niklas Wenzel (nikwen)]

review: Needs Fixing

[Niklas Wenzel (nikwen)]

Sorry, I changed the status on the wrong merge proposal.

[Niklas Wenzel (nikwen)]

Ok, now it's ready to be merged again. :)

[Niklas Wenzel (nikwen)]

review: Approve

=== modified file 'cmd/account-polld/main.go'
--- cmd/account-polld/main.go	2014-10-02 13:58:49 +0000
+++ cmd/account-polld/main.go	2014-12-02 20:11:53 +0000
@@ -1,6 +1,7 @@
 /*
  Copyright 2014 Canonical Ltd.
  Authors: Sergio Schvezov <[e-mail]>
+          Niklas Wenzel <[e-mail]>
 
  This program is free software: you can redistribute it and/or modify it
  under the terms of the GNU General Public License version 3, as published
@@ -107,7 +108,9 @@
 			if account, ok := mgr[data.AccountId]; ok {
 				if data.Enabled {
 					log.Println("New account data for existing account with id", data.AccountId)
+					account.penaltyCount = 0
 					account.updateAuthData(data)
+					account.Poll(false)
 				} else {
 					account.Delete()
 					delete(mgr, data.AccountId)