pyjuju

Merge lp:~fwereade/pyjuju/require-default-series into lp:pyjuju

require-default-series
Merge into trunk

Proposed by William Reade on 2011-10-03

Status:

Merged

Approved by:

Kapil Thangavelu on 2011-10-04

Approved revision:

377

Merged at revision:

381

Proposed branch:

lp:~fwereade/pyjuju/require-default-series

Merge into:

lp:pyjuju

Diff against target:

0 lines

To merge this branch:

bzr merge lp:~fwereade/pyjuju/require-default-series

Undecided

Fix Released

Link a bug report

Reviewer	Date Requested	Status
Kapil Thangavelu (community)		Approve on 2011-10-04
Gustavo Niemeyer	2011-10-03	Approve on 2011-10-03
Review via email: mp+77878@code.launchpad.net

Revision history for this message

Gustavo Niemeyer (niemeyer) wrote on 2011-10-03:

[1]

Tricky!

I want to disagree with this because it's not the correct long term solution,
but at the same time you have a point. This will probably yield much better
short term results within the next 6 months.

Will ask Kapil to have a look at this as well.

review: Approve

Revision history for this message

Gustavo Niemeyer (niemeyer) wrote on 2011-10-03:

Please note this has lp:~fwereade/juju/use-remote-formulas as a pre-req.

Revision history for this message

Kapil Thangavelu (hazmat) wrote on 2011-10-04:

+1, explicit over implicit, portable over platform specific. This is simple and gives control to the user. otoh, it wants to imply homogeneous environments, and has no validation of series. Still a nice win over what is replacing.

niemeyer, thanks for mentioning the pre-req.

review: Approve

lp:~fwereade/pyjuju/require-default-series updated on 2011-10-04