LGTM

On Mon, May 12, 2014 at 6:08 PM, 'Klaus Aehlig' via ganeti-devel <
[email protected]> wrote:

> This is the last task currently done by masterd, so makeing
> luxid taking this over, we can get rid of masterd.
>
> Signed-off-by: Klaus Aehlig <[email protected]>
> ---
>  src/Ganeti/Query/Server.hs | 19 ++++++++++++++++++-
>  1 file changed, 18 insertions(+), 1 deletion(-)
>
> diff --git a/src/Ganeti/Query/Server.hs b/src/Ganeti/Query/Server.hs
> index 8952d4e..eb85fa5 100644
> --- a/src/Ganeti/Query/Server.hs
> +++ b/src/Ganeti/Query/Server.hs
> @@ -32,7 +32,7 @@ module Ganeti.Query.Server
>  import Control.Applicative
>  import Control.Concurrent
>  import Control.Exception
> -import Control.Monad (forever, when, zipWithM, liftM)
> +import Control.Monad (forever, when, zipWithM, liftM, void)
>  import Control.Monad.IO.Class
>  import Data.Bits (bitSize)
>  import qualified Data.Set as Set (toList)
> @@ -434,6 +434,21 @@ luxiHandler cfg = U.Handler { U.hParse         =
> decodeLuxiCall
>  -- | Type alias for prepMain results
>  type PrepResult = (Server, IORef (Result ConfigData), JQStatus)
>
> +-- | Activate the master IP address.
> +activateMasterIP :: IO (Result ())
> +activateMasterIP = runResultT $ do
> +  liftIO $ logDebug "Activating master IP address"
> +  conf_file <- liftIO Path.clusterConfFile
> +  config <- mkResultT $ Config.loadConfig conf_file
> +  let mnp = Config.getMasterNetworkParameters config
> +      masters = Config.getMasterNodes config
> +      ems = clusterUseExternalMipScript $ configCluster config
> +  liftIO . logDebug $ "Master IP params: " ++ show mnp
> +  res <- liftIO . executeRpcCall masters $ RpcCallNodeActivateMasterIp
> mnp ems
> +  _ <- liftIO $ logRpcErrors res
> +  liftIO $ logDebug "finished activating master IP address"
> +  return ()
> +
>  -- | Check function for luxid.
>  checkMain :: CheckFn ()
>  checkMain _ = return $ Right ()
> @@ -465,6 +480,8 @@ main _ _ (server, cref, jq) = do
>
>    _ <- P.installHandler P.sigCHLD P.Ignore Nothing
>
> +  _ <- forkIO . void $ activateMasterIP
> +
>    finally
>      (forever $ U.listener (luxiHandler (qlock, jq, creader)) server)
>      (closeServer server)
> --
> 1.9.1.423.g4596e3a
>
>

Reply via email to