search

aeolusproject / conductor

Aeolus Conductor
http://www.aeolusproject.org/
Apache License 2.0
32 stars 27 forks source link

build all does not ignore disabled providers #376

Open mtaylor opened 11 years ago

mtaylor commented 11 years ago

set up a tim / conductor system with three providers, when I disabled two if them and clicked on the image build all button, all three providers had image builds attempted which should not be the case.

Copying message from Tim Issue: https://github.com/aeolus-incubator/tim/issues/101

n1zyy commented 11 years ago

The build step is at the ProviderType level, not the Provider level. You shouldn't be able to push to a disabled Provider, but Build All doesn't do that.

Should we decline to build target images for a ProviderType where all associated Providers are disabled? If so, should we not show that type in the build UI?