plainbox session restoring doesn't restore generated jobs if run crashes while running a generated job

Bug #1219065 reported by Daniel Manrique
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
PlainBox (Toolkit)
Fix Released
Medium
Unassigned

Bug Description

- Modify plainbox/stubbox/jobs/multilevel.txt.in, and add a "sleep 20" in multilevel_1 command (to give us time to kill plainbox)
- Run as follows:

plainbox -c stub run -w plainbox/stubbox/data/whitelists/stub1.whitelist

- When you hit multilevel_1 and it's waiting, open another terminal window and kill plainbox (I looked at the pid on ps wwuxa |grep plainbox and killed -9 by specific pid)

- Rerun plainbox the same way

- Select "Yes" to restore the session

This result shows multilevel_1 and multilevel_2 disappeared:

===============================[ Analyzing Jobs ]===============================
Do you want to resume the previous session [Y/n]? y
Estimated duration cannot be determined for automated jobs.
Estimated duration cannot be determined for manual jobs.
==============================[ Running All Jobs ]==============================
==================================[ Results ]===================================
stub/dependency/bad: not-supported
stub/false: fail
stub/multilevel: pass
stub/requirement/bad: not-supported
stub/requirement/good: pass
stub/true: pass
stub_package: pass

Whereas the output should be closer to this (this is letting multilevel_1 complete on a non-restored session run):

==================================[ Results ]===================================
stub/dependency/bad: not-supported
stub/false: fail
stub/multilevel: pass
stub/multilevel_1: pass
stub/multilevel_2: pass
stub/requirement/bad: not-supported
stub/requirement/good: pass
stub/true: pass
stub_package: pass

Zygmunt Krynicki (zyga)
Changed in checkbox:
importance: Undecided → Medium
tags: added: plainbox
Zygmunt Krynicki (zyga)
Changed in checkbox:
assignee: nobody → Zygmunt Krynicki (zkrynicki)
Zygmunt Krynicki (zyga)
tags: added: aging
Zygmunt Krynicki (zyga)
Changed in checkbox:
milestone: none → plainbox-0.5
Zygmunt Krynicki (zyga)
tags: added: may-be-fixed
Revision history for this message
Zygmunt Krynicki (zyga) wrote :

I'm marking as incomplete. I don't think it's happening anymore but I haven't researched this thoroughly.

Changed in checkbox:
status: New → Incomplete
milestone: plainbox-0.5 → plainbox-0.6
Zygmunt Krynicki (zyga)
Changed in plainbox:
milestone: none → 0.6
assignee: nobody → Zygmunt Krynicki (zkrynicki)
importance: Undecided → Medium
status: New → Incomplete
Zygmunt Krynicki (zyga)
Changed in checkbox:
milestone: plainbox-0.6 → none
Zygmunt Krynicki (zyga)
no longer affects: checkbox
Zygmunt Krynicki (zyga)
tags: removed: plainbox
Changed in plainbox:
assignee: Zygmunt Krynicki (zkrynicki) → nobody
Changed in plainbox:
milestone: 0.6 → 0.7
Changed in plainbox:
milestone: 0.7 → 0.10
Revision history for this message
Daniel Manrique (roadmr) wrote :

OK, I retried the repro case and it's not happening anymore. Marking as Fix Released :D

Changed in plainbox:
status: Incomplete → 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.