[ovs-dev] [PATCH] xenserver: Restore original InterfaceReconfigure*.py on uninstall.

Justin Pettit jpettit at nicira.com
Mon Apr 19 18:43:32 UTC 2010


Thanks for fixing this!  Looks good.

--Justin


On Apr 19, 2010, at 11:40 AM, Ben Pfaff wrote:

> The %post script fragment in the RPM spec was moving aside the original
> InterfaceReconfigure{,Bridge,Vswitch}.py scripts, but the %postun was
> not restoring them.  This commit restores them in %postun.
> 
> Without this change, installing the openvswitch RPM on a stock XenServer
> and then uninstalling it breaks XenServer networking.
> 
> Bug #2624.
> Reported-by: Ram Jothikumar <rjothikumar at nicira.com>
> Signed-off-by: Ben Pfaff <blp at nicira.com>
> ---
> xenserver/openvswitch-xen.spec |    3 +++
> 1 files changed, 3 insertions(+), 0 deletions(-)
> 
> diff --git a/xenserver/openvswitch-xen.spec b/xenserver/openvswitch-xen.spec
> index d70b73a..c796a1b 100644
> --- a/xenserver/openvswitch-xen.spec
> +++ b/xenserver/openvswitch-xen.spec
> @@ -293,6 +293,9 @@ if [ "$1" = "0" ]; then     # $1 = 1 for upgrade
>     # Restore original XenServer scripts
>     for f in \
>         /opt/xensource/libexec/interface-reconfigure \
> +        /opt/xensource/libexec/InterfaceReconfigure.py \
> +        /opt/xensource/libexec/InterfaceReconfigureBridge.py \
> +        /opt/xensource/libexec/InterfaceReconfigureVswitch.py \
>         /etc/xensource/scripts/vif \
>         /usr/sbin/xen-bugtool \
>         /usr/sbin/brctl
> -- 
> 1.6.6.1
> 
> 
> _______________________________________________
> dev mailing list
> dev at openvswitch.org
> http://openvswitch.org/mailman/listinfo/dev_openvswitch.org





More information about the dev mailing list