[libvirt] [PATCH] storage pool discovery
David Lively
dlively at virtualiron.com
Mon Aug 25 14:33:45 UTC 2008
On Fri, 2008-08-22 at 16:15 +0100, Daniel P. Berrange wrote:
> On Fri, Aug 22, 2008 at 10:58:54AM -0400, David Lively wrote:
> > As long as we're on the subject of naming (and before it's too late),
> > it's been bothering me that we keep calling this "storage pool
> > discovery". To me, "storage source discovery" seems more accurate
> > (because they're not pools until we define libvirt pools based on the
> > sources). So I'd prefer renaming the various *Discover[Storage]Pools*
> > functions (and support structs) introduced in this patch to
> > *Discover[Storage]Sources*. I was just sticking with the
> > originally-proposed names to avoid confusion. What do you all think?
>
> That sounds like a reasonable idea to me.
[Sorry to harp on the naming issue. But names are important, and we
can't change them once they're in the API ...]
After making the change I suggested above, it now feels a little strange
because "Pool" is gone from the name. I'm starting to think
"*Discover[Storage]PoolSources*" is the only good choice. It's rather
long, but makes it clear we're talking about storage pool sources (as
opposed to "storage sources", which feels a little ambiguous, or
"storage pools" which isn't accurate since they're not (yet) pools).
Sound ok?
Dave
More information about the libvir-list
mailing list