Ticket #11952 (closed defect: fixed)

Opened 2 years ago

Last modified 2 years ago

XO-1.75 os14 manifest fail on subsequent run

Reported by: Quozl Owned by: dsd
Priority: high Milestone: 12.1.0
Component: not assigned Version: Development build as of this date
Keywords: Cc:
Action Needed: test in build Verified: no
Deployments affected: Blocked By:
Blocking:

Description

if the TS tag is set to RUNIN, and the first 24-hour loop of runin passes, the subsequent will fail with a manifest error identifying wtmp as having changed.

however, there had been no human login attempt, not even a debug shell.

regression over 11.3.1.

breaks documented usage.

Check fs version 14 with hash f4791264eacd95c8537b70e4db9b2b31ffe39f3f8364b5339c3a3f46b9eda182
Loop 000000 : Contents manifest failure at line 319
Last file examined: wtmp
FAIL: fscheck: 1339971263 : 1694.35 : code=1
Loop 000001 : 

Change History

Changed 2 years ago by dsd

  • owner set to dsd

Changed 2 years ago by Quozl

As far as I can tell, this problem does not break use of release in mass production ... because an fs-update would be done before a repeated runin. But I have asked.

Changed 2 years ago by dsd

It could also suggest other problems, so I'll treat this with high priority and hopefully look at it soon. Thanks for reporting.

Changed 2 years ago by Quozl

  • priority changed from normal to high

Factory has replied. Has moderate mass production impact. Test operator is given opportunity to re-run RUNIN by factory scripting. fs-update is a significant time cost.

Changed 2 years ago by dsd

  • next_action changed from diagnose to add to build

Fixed in olpc-os-builder da79b55. This was a race condition that was always happening, but seemed to be hit by runin every time.

Changed 2 years ago by dsd

  • next_action changed from add to build to test in build

Test in 12.1.0 build 16.

Changed 2 years ago by Quozl

Seems to work fine on XO-1.5, also restores our ability to re-run runin despite having logged in to Sugar ... which was something we lost along the way without recognising this race as the cause.

Changed 2 years ago by dsd

  • status changed from new to closed
  • resolution set to fixed

Thanks for testing.

Note: See TracTickets for help on using tickets.