summaryrefslogtreecommitdiff
path: root/morphlib/buildcontroller.py
Commit message (Collapse)AuthorAgeFilesLines
* Make ./check check for long lines (with excptions); fix long linesLars Wirzenius2012-02-271-1/+1
|
* Fix the wait_for_workers() loop.Jannis Pohlmann2012-01-261-1/+1
| | | | | | | This loop turns out to be highly complex. Maybe adding 1-2 helper variables and converting it to a traditional loop rather than an abstract one based on list comprehension might help in making it less hard to understand.
* Add poor man's unit tests for controller and workers.Jannis Pohlmann2012-01-251-9/+9
|
* Report build errors in workers back to the controller.Jannis Pohlmann2012-01-241-0/+4
| | | | | | Also, distinguish between running "sudo" and "fakeroot" depending on whether or not a remote worker builds a system image. For some reason, sudo in SSH does not seem to work here, so we need to figure that out.
* Fix the loop that waits for workers to finish.Jannis Pohlmann2012-01-241-1/+1
| | | | | | | | Apparently, all(x for x in y) performs some sort of lazy evaluation, causing it to only check workers to complete until one of them is not complete. What we want is to check all of them, so we need to use the all([x for x in y]) notation, which will first generate a list and then evaluate that by iterating over it. Madness - but it works now!
* Avoid errors when a worker does not return any output.Jannis Pohlmann2012-01-241-2/+3
|
* Share functionality in BuildWorker, implement SSH, add name/ident concept.Jannis Pohlmann2012-01-241-0/+4
| | | | | | | | | | | | | All BuildWorker subclasses are likely to make use of the multiprocessing library anyway, so most of the functionality can be shared. All workers now have an ident (e.g. "user@hostname" or "local") which is used by the BuildController to generate enumerated names for them (e.g. "user@hostname-1", "user@hostname-2" or "local-1" and "local-2"), which makes it easier to identify who does what. The RemoteBuildWorker now runs "ssh HOSTNAME fakeroot morph" for building stuff remotely.
* Use 0.1s as the default timeout in wait_for_worker(), instead of 100s.Jannis Pohlmann2012-01-241-2/+1
|
* Add controller, worker classes and a new "build-distributed" command.Jannis Pohlmann2012-01-231-0/+119
This commit introduces four new classes: BuildController: * takes an app instance and a tempdir * allows to add BuildWorker objects * provides a build() method that takes a set of blobs and a build order that is then built by assigning work to the build workers as needed * the build() method takes care of polling the workers for their state, moving them between busy and idle states reliably, collect and print their output in a non-confusing order, and makes sure to wait for all workers to finish before processing the next group in the build order. * at this point, when waiting for one or more workers to become idle to assign them another blob to build, the controller always picks the worker that has been idling for the longest period of time. this can be changed later. BuildWorker: * base class for all worker classes * takes a name and an app instance * has a idle_since datetime property * provides a build() method that takes a Blob object and builds it in whatever way the subclasses implement it * provides a check_complete(timeout) method that checks whether the worker has finished building the blob yet or not LocalBuildWorker: * worker class for local builds that don't go through SSH * it uses morphlib.execute.Execute to run morph in a child process in build() * at the moment, this class executes "./morph" instead of "morph" as it assumes the user to run morph from its source tree. obviously, this will have to be fixed later. RemoteBuildWorker: * doesn't implement anything yet, will be used for distributing work to other machines running morph via SSH Notes: * At the moment, there is a degree of undesired redundancy when building a stratum in a worker, as this will cause the worker to rebuild all its dependencies. This will have to be fixed as it is avoidable and wastes a lot of time and processing power.