chiark / gitweb /
transaction: rework merging with installed jobs
authorMichal Schmidt <mschmidt@redhat.com>
Fri, 20 Apr 2012 07:38:43 +0000 (09:38 +0200)
committerMichal Schmidt <mschmidt@redhat.com>
Fri, 20 Apr 2012 15:12:29 +0000 (17:12 +0200)
commit656bbffc6c45bdd8d5c28a96ca948ba16c546547
tree72ae472a5d5bb05b7b5c8cfcceefbfce0898537b
parent05d576f1f77699ea5c2e5ed0e04451b14dfc45a0
transaction: rework merging with installed jobs

Previously transactions could reference installed jobs. It made some issues
difficult to fix.

This sets new rules for jobs:
A job cannot be both a member of a transaction and installed. When jobs are
created, they are linked to a transaction. The whole transaction is constructed
(with merging of jobs within, etc.). When it's complete, all the jobs are
unlinked from it one by one and let to install themselves. It is during the
installation when merging with previously installed jobs (from older
transactions) is contemplated.

Merging with installed jobs has different rules than merging within a
transaction:
 - An installed conflicting job gets cancelled. It cannot be simply deleted,
   because someone might be waiting for its completion on DBus.
 - An installed, but still waiting, job can be safely merged into.
 - An installed and running job can be tricky. For some job types it is safe to
   just merge. For the other types we merge anyway, but put the job back into
   JOB_WAITING to allow it to run again. This may be suboptimal, but it is not
   currently possible to have more than one installed job for a unit.

Note this also fixes a bug where the anchor job could be deleted during merging
within the transaction.
src/core/job.c
src/core/job.h
src/core/transaction.c