ppa-dev-tools

Tests result duplicated

Bug #2038649 reported by Sebastien Bacher
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ppa-dev-tools
Fix Committed
High
Bryce Harrington

Bug Description

Using 0.5.0 deb from the ppa

$ ppa tests --arch=amd64 ppa:seb128/update-manager-sru
* Triggers:
  - Source update-manager/1:22.04.16: Published
    + Trigger basic @amd64♻️ Trigger all-proposed @amd64💍
  - Source update-manager/1:20.04.10.17: Published
    + Trigger basic @amd64♻️ Trigger all-proposed @amd64💍
* Results:
  - update-manager/1:20.04.10.17
    + ✅ update-manager on focal for amd64 @ 06.10.23 10:52:59 Log️ 🗒️
...
  - update-manager/1:20.04.10.17
    + ✅ update-manager on focal for amd64 @ 06.10.23 10:52:59 Log️ 🗒️
* Running: (none)
* Waiting: (none)

There should be only one result for focal, also the summary isn't include the bionic results (which are correctly listed if '-r bionic' is added)

Related branches

ppa-dev-tools:fix-lp2038649-tests-result-duplicated
Athos Ribeiro (community): Approve
PpaDevTools Developers: Pending requested
Canonical Server Reporter: Pending requested
Diff: 79 lines (+37/-8)
2 files modified
ppa/result.py (+6/-7)
tests/test_result.py (+31/-1)
Revision history for this message
Bryce Harrington (bryce) wrote :

Confirmed the behavior, and indeed with bug LP: #2038651 fixed, it actually produced *triple* the results it should.

Problem was it was double-looping over the results. Thanks for reporting this, I'm surprised it didn't get caught earlier.

Changed in ppa-dev-tools:
status: New → In Progress
assignee: nobody → Bryce Harrington (bryce)
importance: Undecided → High
Revision history for this message
Bryce Harrington (bryce) wrote :

Fixed output:

$ ppa tests --arch=amd64 ppa:seb128/update-manager-sru
* Triggers:
  - Source update-manager/1:22.04.16: Published
    + Trigger basic @amd64♻️ Trigger all-proposed @amd64💍
  - Source update-manager/1:20.04.10.17: Published
    + Trigger basic @amd64♻️ Trigger all-proposed @amd64💍
  - Source update-manager/1:18.04.11.20: Published
    + Trigger basic @amd64♻️ Trigger all-proposed @amd64💍
* Results:
  - update-manager/1:20.04.10.17
    + ✅ update-manager on focal for amd64 @ 06.10.23 10:52:59 Log️ 🗒️
  - update-manager/1:22.04.16
    + ✅ update-manager on jammy for amd64 @ 06.10.23 10:54:55 Log️ 🗒️
  - update-manager/1:18.04.11.20
    + ✅ update-manager on bionic for amd64 @ 06.10.23 08:48:34 Log️ 🗒️
* Running: (none)
* Waiting: (none)

Revision history for this message
Sebastien Bacher (seb128) wrote :

Thanks for the quick fix!

Bryce Harrington (bryce)
Changed in ppa-dev-tools:
status: In Progress → Fix Committed
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.