Code review comment for lp:~axwalk/juju-core/azure-provider-doc

Revision history for this message
William Reade (fwereade) wrote :

LGTM. Please make sure evilnick knows about this.

https://codereview.appspot.com/84470052/diff/1/doc/azure-provider.txt
File doc/azure-provider.txt (right):

https://codereview.appspot.com/84470052/diff/1/doc/azure-provider.txt#newcode101
doc/azure-provider.txt:101: default, we set
availability-sets-enabled=true for new environments.
I know you cover it later, but it would be good to have a single
sentence here, explicitly pointing out the tradeoff you're making by
setting availability-sets-enabled to false.

https://codereview.appspot.com/84470052/

« Back to merge proposal