[Pkg-ganeti-devel] [ganeti] 03/05: Drop patches already merged upstream
Apollon Oikonomopoulos
apoikos-guest at alioth.debian.org
Wed Sep 4 11:21:44 UTC 2013
This is an automated email from the git hooks/post-receive script.
apoikos-guest pushed a commit to branch experimental
in repository ganeti.
commit 26ce0632209c31b60c78b5a88536f6058b97060f
Author: Apollon Oikonomopoulos <apoikos at gmail.com>
Date: Fri Aug 30 13:29:02 2013 +0300
Drop patches already merged upstream
• 0001-Use-FQDN-to-check-master-node-status.patch
• disable-TestVerifyRestrictedCmdDirectory.patch
- fix merged upstream
---
...0001-Use-FQDN-to-check-master-node-status.patch | 59 --------------------
.../disable-TestVerifyRestrictedCmdDirectory.patch | 28 ----------
debian/patches/series | 2 -
3 files changed, 89 deletions(-)
diff --git a/debian/patches/0001-Use-FQDN-to-check-master-node-status.patch b/debian/patches/0001-Use-FQDN-to-check-master-node-status.patch
deleted file mode 100644
index 836b2e2..0000000
--- a/debian/patches/0001-Use-FQDN-to-check-master-node-status.patch
+++ /dev/null
@@ -1,59 +0,0 @@
-From ecff332f0af01dc49a72175eab4b290bf275a753 Mon Sep 17 00:00:00 2001
-From: Thomas Thrainer <thomasth at google.com>
-Date: Tue, 13 Aug 2013 09:50:42 +0200
-Subject: [PATCH] Use FQDN to check master node status
-
-The master node name in SS conf is stored as FQDN, so also use the FQDN
-on each node to check if it is the master node.
-
-This fixes issue 551.
-
-Signed-off-by: Thomas Thrainer <thomasth at google.com>
-Reviewed-by: Guido Trotter <ultrotter at google.com>
----
- src/Ganeti/Daemon.hs | 15 +++++++++++++--
- 1 file changed, 13 insertions(+), 2 deletions(-)
-
-diff --git a/src/Ganeti/Daemon.hs b/src/Ganeti/Daemon.hs
-index 6f9db4a..ada57eb 100644
---- a/src/Ganeti/Daemon.hs
-+++ b/src/Ganeti/Daemon.hs
-@@ -47,7 +47,7 @@ module Ganeti.Daemon
-
- import Control.Exception
- import Control.Monad
--import Data.Maybe (fromMaybe)
-+import Data.Maybe (fromMaybe, listToMaybe)
- import Data.Word
- import GHC.IO.Handle (hDuplicateTo)
- import Network.BSD (getHostName)
-@@ -297,6 +297,17 @@ parseAddress opts defport = do
- vClusterHostNameEnvVar :: String
- vClusterHostNameEnvVar = "GANETI_HOSTNAME"
-
-+getFQDN :: IO String
-+getFQDN = do
-+ hostname <- getHostName
-+ addrInfos <- Socket.getAddrInfo Nothing (Just hostname) Nothing
-+ let address = listToMaybe addrInfos >>= (Just . Socket.addrAddress)
-+ case address of
-+ Just a -> do
-+ fqdn <- liftM fst $ Socket.getNameInfo [] True False a
-+ return (fromMaybe hostname fqdn)
-+ Nothing -> return hostname
-+
- -- | Returns if the current node is the master node.
- isMaster :: IO Bool
- isMaster = do
-@@ -307,7 +318,7 @@ isMaster = do
- ioErrorToNothing
- curNode <- case vcluster_node of
- Just node_name -> return node_name
-- Nothing -> getHostName
-+ Nothing -> getFQDN
- masterNode <- Ssconf.getMasterNode Nothing
- case masterNode of
- Ok n -> return (curNode == n)
---
-1.7.10.4
-
diff --git a/debian/patches/disable-TestVerifyRestrictedCmdDirectory.patch b/debian/patches/disable-TestVerifyRestrictedCmdDirectory.patch
deleted file mode 100644
index 24b44c7..0000000
--- a/debian/patches/disable-TestVerifyRestrictedCmdDirectory.patch
+++ /dev/null
@@ -1,28 +0,0 @@
-Author: Apollon Oikonomopoulos <apoikos at gmail.com>
-Subject: Disable a test in ganeti.backend_unittest.py
-
-Disable the TestVerifyRestrictedCmdDirectory.testNormal, as it fails on the
-Debian buildds. We know that the code is good, since it builds correctly in a
-pristine chroot.
-
-Forwarded: No
-Last-Update: 2013-07-28
---- a/test/py/ganeti.backend_unittest.py
-+++ b/test/py/ganeti.backend_unittest.py
-@@ -164,16 +164,6 @@
- self.assertFalse(status)
- self.assertTrue(msg.endswith("is not a directory"))
-
-- def testNormal(self):
-- tmpname = utils.PathJoin(self.tmpdir, "foobar")
-- os.mkdir(tmpname)
-- self.assertTrue(os.path.isdir(tmpname))
-- (status, msg) = \
-- backend._VerifyRestrictedCmdDirectory(tmpname,
-- _owner=_DefRestrictedCmdOwner())
-- self.assertTrue(status)
-- self.assertTrue(msg is None)
--
-
- class TestVerifyRestrictedCmd(unittest.TestCase):
- def setUp(self):
diff --git a/debian/patches/series b/debian/patches/series
index 8f8378f..7011cb7 100644
--- a/debian/patches/series
+++ b/debian/patches/series
@@ -1,5 +1,3 @@
fix-startup-with-old-config.patch
cfgupgrade12-remove-old-ssconf.patch
0003-Disable-local-checks-during-build.patch
-disable-TestVerifyRestrictedCmdDirectory.patch
-0001-Use-FQDN-to-check-master-node-status.patch
--
Alioth's /usr/local/bin/git-commit-notice on /srv/git.debian.org/git/pkg-ganeti/ganeti.git
More information about the Pkg-ganeti-devel
mailing list