On Tue, Jan 31, 2012 at 01:53:49PM -0800, Gurucharan Shetty wrote:
> + case ${OVSDHCPINTERFACES} in
> + ${OVSINTF#ifcfg-} | \
> + "${OVSINTF#ifcfg-},"* | \
> + *",${OVSINTF#ifcfg-}" | \
> + *",${OVSINTF#ifcfg-},"*)
As an Old Fart ;-) I'd want to say, that this could be written
as:
case ",${OVSDHCPINTERFACES}," in
*",${OVSINTF#ifcfg-},"*)
And instead of comma-separated list OVSDHCPINTERFACES could be
space separated list (as in $BOND_IFACES), and instead of
OVSINTF=${CONFIG} we could use OVSINTF=${DEVICE}, this way we
don't need to strip the "ifcfg-" prefix.
I was also thinking about my proposal to specify all Port
configuration in Bridge config file. Advantage of this method is
when you ifup a Bridge, all Ports will be automatically added to
this Bridge. Disadvantage is when you ifdown/ifup a Port, it
will not be added back to Bridge. This could be serious issue
for, like, PPP-based devices. Since current implementation
satisfies everyone, let it be.
--
Regards, --
Sir Raorn. --- http://thousandsofhate.blogspot.com/
signature.asc
Description: Digital signature
_______________________________________________ dev mailing list [email protected] http://openvswitch.org/mailman/listinfo/dev
