-
Notifications
You must be signed in to change notification settings - Fork 42
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Choosing the correct worker for a project #42
Comments
It is something that indeed is on the TODO list, and shall go via this image attribute. |
Yeah, I noticed that was supported for the |
I'd say the best is for LibVirtWorker to have some of its parameter rendereable so that they can be chosen according to the properties. |
I would also expect some support for Windows workers. So far, I only see Linux based ones. How hard would it be to implement an 'os' attribute (or a set of worker properties) in the bbtravis.yml ? Like this is currently supported in travis, although only for linux and macos. |
I plan to make a fork of buildbot_travis adding a "labels" attribute to Workers, in order to have a node selection mechanism "à la" Jenkins. But this requires to also change the Worker class in buildbot, and add it a "labels" attribute as well. Is there a way I can avoid this ? |
In buildbot, the way you can do this is to implement and nextWorker callback. You can always add a attribute to your standard buildbot workers, and then look at this attribute in your nextWorker. |
I planned to use canStartBuild, but nextWorker will do as well. The problem is that checkConfig (buildbot.worker.base.AbstractWorker) returns an error on the unexpected "labels" argument I added (in the UI, cfg.yml, and createWorkerConfigWorker). I cannot see how to avoid modifying buildbot itself to make that pass. |
I can't find a way to map a project to specific worker or worker types. Is this on the TODO list?
I see that travisyml.py has an image attribute, but it's currently not used as far as I can see? Is this meant to handle this in the future?
The text was updated successfully, but these errors were encountered: