Code review comment for lp:~hazmat/pyjuju/cli-placement-restore

Revision history for this message
Adam Gandelman (gandelman-a) wrote :

Hi--

Moving "--placement=local" from argv to "placement: local" in environments.yaml seems trivial enough, however, my specific use case often has me dealing with many different formulas, some of which are destroyed and re-deployed many times in one sitting. Having to to parse and modify environments.yaml many times over between 'juju deploy' to switch to and from 'local' placement policy seems needlessly tedious. Having to put together my own wrappers around 'juju deploy' and environments.yaml to do this for me and emulate what '--placement=local' once provided seems even more suboptimal.

We understand its a hack but we're using it as a workaround to Bug #806241. Without it, Juju goes from something that might be useful "sometime in the future" given our hardware constraints to something that is extremely useful *now* and *central* to our QA and integration testing of specific software stacks. IMHO, making it more difficult to use due to user facing changes like this (especially this late in the cycle) would be unfortunate.

« Back to merge proposal