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

Issue 5938050: Implement maas-name constraint

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years, 1 month ago by fwereade
Modified:
12 years, 1 month ago
Reviewers:
mp+99746
Visibility:
Public.

Description

Implement maas-name constraint Descended from lp:~julian-edwards/juju/name-constraints; but uses new-style constraints registration. Also uses a custom ubuntu-series constraint to forbid use of charms written for systems we can't yet provision with MAAS. https://code.launchpad.net/~fwereade/juju/maas-name-constraint/+merge/99746 Requires: https://code.launchpad.net/~fwereade/juju/constraints-get/+merge/99704 (do not edit description out of merge proposal)

Patch Set 1 #

Total comments: 4

Patch Set 2 : Implement maas-name constraint #

Unified diffs Side-by-side diffs Delta from patch set Stats (+171 lines, -22 lines) Patch
A [revision details] View 1 1 chunk +2 lines, -0 lines 0 comments Download
M juju/environment/config.py View 1 1 chunk +3 lines, -1 line 0 comments Download
M juju/environment/tests/test_config.py View 1 2 chunks +8 lines, -6 lines 0 comments Download
M juju/machine/constraints.py View 1 2 chunks +3 lines, -2 lines 0 comments Download
M juju/machine/tests/test_constraints.py View 1 chunk +11 lines, -0 lines 0 comments Download
M juju/providers/maas/launch.py View 1 chunk +1 line, -4 lines 0 comments Download
M juju/providers/maas/maas.py View 1 chunk +5 lines, -1 line 0 comments Download
M juju/providers/maas/provider.py View 1 chunk +15 lines, -0 lines 0 comments Download
M juju/providers/maas/tests/test_launch.py View 6 chunks +38 lines, -5 lines 0 comments Download
M juju/providers/maas/tests/test_maas.py View 2 chunks +38 lines, -1 line 0 comments Download
M juju/providers/maas/tests/test_provider.py View 2 chunks +39 lines, -1 line 0 comments Download
M juju/providers/maas/tests/testing.py View 1 chunk +8 lines, -1 line 0 comments Download

Messages

Total messages: 5
fwereade
Please take a look.
12 years, 1 month ago (2012-03-28 14:46:46 UTC) #1
allenap
On 2012/03/28 14:46:46, fwereade wrote: > Please take a look. Looks sensible to me.
12 years, 1 month ago (2012-03-28 15:09:20 UTC) #2
hazmat
LGTM https://codereview.appspot.com/5938050/diff/1/juju/environment/config.py File juju/environment/config.py (right): https://codereview.appspot.com/5938050/diff/1/juju/environment/config.py#newcode68 juju/environment/config.py:68: "default-series": Constant("precise")}, deserves a comment regarding maas limitation ...
12 years, 1 month ago (2012-03-28 18:10:47 UTC) #3
fwereade
https://codereview.appspot.com/5938050/diff/1/juju/environment/config.py File juju/environment/config.py (right): https://codereview.appspot.com/5938050/diff/1/juju/environment/config.py#newcode68 juju/environment/config.py:68: "default-series": Constant("precise")}, On 2012/03/28 18:10:47, hazmat wrote: > deserves ...
12 years, 1 month ago (2012-03-28 18:21:58 UTC) #4
fwereade
12 years, 1 month ago (2012-03-28 19:06:28 UTC) #5
*** Submitted:

Implement maas-name constraint

Descended from lp:~julian-edwards/juju/name-constraints; but uses new-style
constraints registration.

Also uses a custom ubuntu-series constraint to forbid use of charms written
for systems we can't yet provision with MAAS.

R=allenap, hazmat
CC=
https://codereview.appspot.com/5938050
Sign in to reply to this message.

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