Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(84)

Issue 90470044: provider/maas: support maas-name directives

Can't Edit
Can't Publish+Mail
Start Review
Created:
9 years, 11 months ago by axw
Modified:
9 years, 11 months ago
Reviewers:
fwereade, mp+216844
Visibility:
Public.

Description

provider/maas: support maas-name directives We treat all MAAS placement directives as maas-name, and pass it on as a constraint to the acquire-node API. Partially fixes lp:1237709 https://code.launchpad.net/~axwalk/juju-core/lp1237709-maasname/+merge/216844 (do not edit description out of merge proposal)

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+43 lines, -15 lines) Patch
A [revision details] View 1 chunk +2 lines, -0 lines 0 comments Download
M provider/maas/environ.go View 3 chunks +7 lines, -6 lines 0 comments Download
M provider/maas/environ_whitebox_test.go View 3 chunks +34 lines, -9 lines 0 comments Download

Messages

Total messages: 2
axw
Please take a look.
9 years, 11 months ago (2014-04-23 09:39:20 UTC) #1
fwereade
9 years, 11 months ago (2014-04-23 09:46:55 UTC) #2
LGTM.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b