[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