Code review comment for lp:~hazmat/pyjuju/states-with-principals

Revision history for this message
Gustavo Niemeyer (niemeyer) wrote :

On 2012/04/04 17:43:36, hazmat wrote:
> The mir failed in part because of lack of appropriate security, and
the
> security concerns remain an issue to adoption.

A broken implementation is a much more serious concern for adoption.

> for reference this was the original spec

https://code.launchpad.net/%7Ehazmat/juju/security-specification/+merge/63921

"Needs Fixing on 2011-06-09"

> what i'm asking for is direction wrt to your comments.. i've updated
the
> security branches and cleaned them up. If you'd like me to bring it up
on
> list, i'm happy to push the spec there. If its something that should
be
> dropped.. well then pls say that.. at this point clearly i'm not too
> concerned about wasting time effort on dead code.

Do you want me to say one more time? Sure: I think it is a bad idea to
be merging massive changes for 12.04 at this stage. I would not do it. I
would not be merging all those significant changes right before a
release. It's supposed to be stable. Let's please not ship crack in
12.04.

https://codereview.appspot.com/5966076/

« Back to merge proposal