Postal message order is wrong
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
ubuntu-push (Ubuntu) |
New
|
Undecided
|
Unassigned |
Bug Description
Reproduce:
Sometimes
What happens:
-------
FAIL: <autogenerated>:66: ualPostalSuite.
[LOG] 36.85478 DEBUG GetAll got: map[string]
[LOG] 36.85478 DEBUG using helper helpId (exec: bar) for app com.example.
[LOG] 36.85478 DEBUG current helper input backlog has grown to 1 entries.
[LOG] 36.85478 DEBUG using helper (exec: lhex) for app _classic-app
[LOG] 36.85478 INFO helper output: {"message"
[LOG] 36.85478 DEBUG copying backlog to avoid wasting too much space (1/2 used)
[LOG] 36.85478 DEBUG current helper input backlog has shrunk to 1 entries.
[LOG] 36.85478 DEBUG using helper helpId (exec: bar) for app com.example.
[LOG] 36.85478 DEBUG current helper input backlog has shrunk to 0 entries.
[LOG] 36.85478 INFO helpId helper output: {"message"
[LOG] 36.85478 INFO helpId helper output: {"message"
[LOG] 36.85478 DEBUG msgHandler did not present the notification
[LOG] 36.85478 DEBUG msgHandler did not present the notification
[LOG] 36.85478 DEBUG msgHandler did not present the notification
postal_test.go:422:
c.Check(
... obtained string = "{\"moon\":1}"
... expected string = "{\"world\":1}"
postal_test.go:423:
c.Check(
... obtained string = "{\"world\":1}"
... expected string = "{\"moon\":1}"
postal_test.go:424:
c.Check(
... obtained []string = []string{"m2", "m1"}
... expected []string = []string{"m1", "m2"}
** (process:7672): WARNING **: a message with id '0' already exists
** (process:7672): WARNING **: a message with id '0' already exists
OOPS: 103 passed, 1 FAILED
--- FAIL: TestService (0.09s)
FAIL
FAIL launchpad.
What should happen:
It needs to pass. Samuele proposed this patch [1].
Related branches
- Simon Fels: Approve
-
Diff: 11 lines (+1/-0)1 file modifiedclient/service/postal_test.go (+1/-0)
- Samuele Pedroni: Approve
-
Diff: 71 lines (+10/-4)5 files modifiedclient/service/postal_test.go (+0/-1)
client/session/session_test.go (+1/-1)
debian/rules (+2/-1)
launch_helper/kindpool.go (+4/-0)
launch_helper/kindpool_test.go (+3/-1)
- Ubuntu Push Hackers: Pending requested
-
Diff: 113 lines (+12/-10)9 files modifiedbus/connectivity/connectivity_test.go (+1/-1)
bus/networkmanager/networkmanager.go (+1/-1)
client/session/session_test.go (+1/-1)
debian/rules (+2/-1)
debian/source/format (+0/-1)
docs/example-server/.bzrignore (+0/-1)
docs/example-server/.gitignore (+0/-3)
launch_helper/kindpool.go (+4/-0)
launch_helper/kindpool_test.go (+3/-1)
summary: |
- Postal message order test is super flaky + Postal message order is wrong |