/juju

provider/azure: don't log unknown sizes as errors #5462

Merged
merged 1 commit into from May 27, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
2 participants
@axw @jujubot
@axw
Member

axw commented May 26, 2016 •
Edited 1 time

It's not an error if we don't know a VM size. The
sizes that Azure supports grow over time, and the
Juju code will need to be updated eventually to
support them completely. In the mean time we just
say the unknown ones are more expensive than the
ones we know about, so they're not chosen without
explicit request.

(Review request: http://reviews.vapour.ws/r/4906/)
@axw
provider/azure: don't log unknown sizes as errors 
It's not an error if we don't know a VM size. The
sizes that Azure supports grow over time, and the
Juju code will need to be updated eventually to
support them completely. In the mean time we just
say the unknown ones are more expensive than the
ones we know about, so they're not chosen without
explicit request.
7a85355
@axw
Member

axw commented May 27, 2016

$$fixes-1586217$$
@jujubot
Contributor

jujubot commented May 27, 2016

Status: merge request accepted. Url: http://juju-ci.vapour.ws:8080/job/github-merge-juju

@jujubot jujubot merged commit 7c6b3da into juju:master May 27, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment