Code review comment for lp:~javier.collado/checkbox/bug990075

Revision history for this message
Javier Collado (javier.collado) wrote :

The problem is that there are some dependencies that are not explicitly set in the jobs.

For example, there are many jobs that are expected to be executed before suspend/suspend_advanced, but they haven't been added as a dependency because that will make its list of dependencies really long.

One way to workaround this issue would be to reorder the whitelist directly to make sure that the ordering is the expected one, but this will also require to be reviewed every time a job is added, so it will be easy to break the whitelist inadvertently.

Ideally, there should be some midpoint in which a whitelist that is ordered almost nicely can be reordered without breaking any of these hidden dependencies against the suspend/suspend_advanced job.

« Back to merge proposal