Code review comment for lp:~rconradharris/nova/xs-snap-return-image-id-before-snapshot

Revision history for this message
Rick Harris (rconradharris) wrote :

Eday: Oops, fixed now, thanks.

Soren: Right, is_public does mean everyone can see it, for now, until we build the ownership/sharing modeling into Glance (in Cactus). The idea with is_public=True was something like:

Nobody should be using this right now since we haven't addressed security (at all), let's just make it a little easier on the devs since the newly created snapshot will appear in the image-listing as they'd expect.

For now, I've changed it to is_public=False, so there shouldn't be anymore confusion.

« Back to merge proposal