[Ovirt-devel] [PATCH server 01/10] converted storage volume controller to use the service layer

[Scott Seago]

Alan Pevec wrote:
> On Wed, May 20, 2009 at 5:52 PM, David Lutterkort <lutter at redhat.com 
> <mailto:lutter at redhat.com>> wrote:
>
>     On Wed, 2009-05-20 at 11:18 -0400, Scott Seago wrote:
>
>     > I'm not a big fan of auto-creating this sort of thing in the
>     'new' form,
>     > but it seemed like things would be a lot more convoluted if we
>     made the
>     > form work with or without an existing VG. Also, in this case
>     there's not
>     > really any user-visible effect of the empty VG since we don't
>     show the
>     > LVM VG in the UI.
>
>     Yeah, not a big deal, just something we need to be careful about.
>
>
> What about double-click or DoS attack protection? Wouldn't those 
> orphaned objects pollute the database?
>
Not in this case, no -- the LVM pool is only created on an iSCSI volume 
if it doesn't already exist -- if it does exist, the current one is 
used. Also, in either case, it's not orphaned, as it's associated with 
the user-selected storage volume.

Scott