Code review comment for lp:~thomir-deactivatedaccount/adt-cloud-worker/trunk-use-even-more-beef

Revision history for this message
Celso Providelo (cprov) wrote :

Thomi,

Yes, I continue to stand for code changes instead of config changes, but in this case it's a cloud-dependent option, i.e PS4 gets cpu2-ram2-disk50 but bootstack/canonistack get m1.small (or similar). We cannot hardcode it.

And despite of the fact that all the logic behind flavor discovering we landed being correct (custom -> default -> first) we never used it. So, that's why I'm suggesting it do be dropped.

« Back to merge proposal