seed-new-release does not copy forward neutral test results

Bug #2018950 reported by Brian Murray
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Auto Package Testing
Fix Released
Medium
Unassigned

Bug Description

During the opening of Mantic vorlon noticed that there were no results for debspawn on armhf and that the first result appears on 2023-05-04 which is sometime after the opening and britney being initialized.

https://autopkgtest.ubuntu.com/packages/debspawn/mantic/armhf

The seed-new-release code should be examined to see if neutral results are copied forward and if not fixed. If it is fixed then we could copy the results forward for any packages which fall into this category. Additionally, we probably don't need to check and see if tests for the packages have run since the archive opening because this would just add an earlier entry to the result set.

Tags: adt-295

Related branches

Revision history for this message
Brian Murray (brian-murray) wrote :

seed-new-release has the following:

 94 # get passing result per package/arch from database
 95 db_con = sqlite3.connect(args.results_db)
 96 for (package, arch, run_id) in db_con.execute(
 97 "SELECT package, arch, MAX(run_id) "
 98 "FROM test, result "
 99 "WHERE test.id = result.test_id AND release = '%s' "
100 " AND (exitcode = 0 OR exitcode = 2 "
101 " OR triggers = 'migration-reference/0') "
102 "GROUP BY package, arch" % args.old_release):

and autopgktest-web/webcontrol/browse.cgi has:

 88 if code in (8,):
 89 return 'neutral'

So no, neutral results are not copied forward.

Changed in auto-package-testing:
status: New → Triaged
importance: Undecided → Medium
summary: - seed-new-release may not copy forward neutral test results
+ seed-new-release does not copy forward neutral test results
tags: added: adt-295
Revision history for this message
Tim Andersson (andersson123) wrote :
Revision history for this message
Tim Andersson (andersson123) wrote :

I don't think we need to make any changes to browse.cgi right?

Revision history for this message
Brian Murray (brian-murray) wrote :

It's fixed in git but an updated charm with this change has not been deployed in production so I am setting this to Fix Committed.

Changed in auto-package-testing:
status: Triaged → Fix Committed
Changed in auto-package-testing:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.