1
100%
master: 100%

Ran 22 Oct 2016 09:56AM UTC

Files 102

Run time 4s

Badge

Embed ▾

Committed 22 Oct 2016 09:43AM UTC coverage: 72.346% (-0.01%) from 72.359%

Job # 3519.1

Build Type

push

travis-ci

Committed by

coolo

Commit Message

Don't free worker before artefacts are uploaded (POO #14378) (#958)

0b78fc5f broke artefact uploading on job cancellation. Because
the job `cancel()` method now frees the worker (so far as the
schema is concerned) immediately after telling the worker to
cancel the job, by the time the worker process reaches the
artefact upload bits of `_stop_job()`, the schema job instance
has no worker, and when the worker process tries to upload the
artefacts, the schema job instance `create_artefact()` method
throws a 404 because so far as it knows, it no longer has a
worker.

The change was intended to mitigate a problem caused by the
dbus deadlock issue, where the server never actually handled
the `set_done` API call the worker process sends at the end
of `_stop_job`, so the worker was never freed - but as the dbus
deadlock issue has now been addressed, we shouldn't need to
worry about that any more.

Run Details

4552 of 6292 relevant lines covered (72.35%)

108.63 hits per line

os-autoinst / openQA / 3519 / 1
100%
master: 100%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 3519.1

os-autoinst / openQA / 3519 / 1 100% master: 100%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 3519.1

os-autoinst / openQA / 3519 / 1
100%
master: 100%

README BADGES
x