LGTM.
I think we actually need to target whatever branch Dave Cheney brings out for the 1.14 series, but we can land this in trunk and then land it in 1.14 when we are able to.
https://codereview.appspot.com/13464043/diff/1/provider/azure/config.go File provider/azure/config.go (right):
https://codereview.appspot.com/13464043/diff/1/provider/azure/config.go#newcode151 provider/azure/config.go:151: default-series: precise Because the global DefaultSeries is precise, I would put this in a comment.
https://codereview.appspot.com/13464043/
« Back to merge proposal
LGTM.
I think we actually need to target whatever branch Dave Cheney brings
out for the 1.14 series, but we can land this in trunk and then land it
in 1.14 when we are able to.
https:/ /codereview. appspot. com/13464043/ diff/1/ provider/ azure/config. go azure/config. go (right):
File provider/
https:/ /codereview. appspot. com/13464043/ diff/1/ provider/ azure/config. go#newcode151 azure/config. go:151: default-series: precise
provider/
Because the global DefaultSeries is precise, I would put this in a
comment.
https:/ /codereview. appspot. com/13464043/