From f3a9a835d32ceeecaefac70fb9b77272e914f18c Mon Sep 17 00:00:00 2001
From: David Nalley <david@gnsa.us>
Date: Fri, 5 Oct 2012 16:26:52 -0400
Subject: [PATCH] Removing symlinks for CloudStack-248 and CloudStack-209 The
 problem that is described in both of those bugs is the deletion of files
 installed by cloud-scripts.

What is happening is that instead of fixing the paths to scripts in places
where they are called, we tried to create a symlink in a %post section in
the RPM so that there was a symlink to the new directory for the scripts.
That does work (in new installs) but the problem that arises is that when
RPM is setting up the transation it doesn't know about the symlink (it's
in a %post, the symlinked directory is unowned from RPMs perspective, or
rather it is only owned by the cloud-agent-scripts package, which will
be removed.

So what happens is that cloud-agent-scripts puts things in /foo - we
come along to upgrade to 4.0 and that means we use cloud-scripts -
which puts things in /bar - so we install things into /bar (/foo still
exists at this point) then in a %post (and for the record, RPM doesn't
know what happens in a %pre, %post, %preun, or %postun - they are outside
the transation) we delete /foo and then create a symlink from /foo to /bar.
Then we get to the transaction part where we are ready to remove
cloud-agent-scripts - so it's time to delete /foo - except /foo is now a
symlink to /bar and thus we wipe out the contents of /foo and /bar in one
fell swoop.

To quote Bill Notting from Red Hat who pointed this out:
15:43 < notting> rpm decides what to do with all the files (update, remove,
or write new file) before the transaction. if the directory layout changes
*during* the transaction, weird shit happens.
---
 cloud.spec | 15 ---------------
 1 file changed, 15 deletions(-)

diff --git a/cloud.spec b/cloud.spec
index 3c6bf0e5985..d004331fe02 100644
--- a/cloud.spec
+++ b/cloud.spec
@@ -392,21 +392,6 @@ if [ -x /etc/sysconfig/modules/kvm.modules ] ; then
     /bin/sh /etc/sysconfig/modules/kvm.modules
 fi
 
-%post scripts
-if [ -d %{_libdir}/%{name}/agent ]; then
-  rm -fr %{_libdir}/%{name}/agent
-fi
-# Symlink for backwards compatibility
-if [ ! -h %{_libdir}/%{name}/agent ]; then
-  ln -s %{_libdir}/%{name}/common %{_libdir}/%{name}/agent
-fi
-
-%postun scripts
-# For uninstallation, remove symlink
-if [ "$1" == "0" ]; then
-  rm -f %{_libdir}/%{name}/agent
-fi
-
 %post client
 if [ "$1" == "1" ] ; then
     /sbin/chkconfig --add %{name}-management > /dev/null 2>&1 || true