[PATCH] short fix on jboss module

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Hi all, 
	I found a short fix on jboss module.

Before now it was impossible to use jboss module if jboss was launched 
without options (like listening interface and/or instance name), because
the return type contains a None.

I sent patch inline in mail (afaik mailman like to eat patch
attachments) which correct this problem.

Luca

#########--------------

>From 9d5087c65fd81cb378bd390ec58e364a8a695093 Mon Sep 17 00:00:00 2001
From: Luca Foppiano <lfoppiano@xxxxxxxxxxxxxxxxxxxx>
Date: Sun, 7 Sep 2008 12:00:23 +0200
Subject: [PATCH] short fix on jboss module.
 It was not able to work if jboss is launched without options,
 jboss module fails because tried to transfer through xmlrpc a
 None type, which failed.

---
 func/minion/modules/jboss.py |    4 ++--
 1 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/func/minion/modules/jboss.py b/func/minion/modules/jboss.py
index a7a3fa2..dc09091 100644
--- a/func/minion/modules/jboss.py
+++ b/func/minion/modules/jboss.py
@@ -45,12 +45,12 @@ class JBoss(func_module.FuncModule):
                     if items.__contains__("-c"):
                         instance = items[items.index("-c")+1]
                     else:
-                        instance = None
+                        instance = ""

                     if items.__contains__("-b"):
                         address = items[items.index("-b")+1]
                     else:
-                        address = None
+                        address = ""

                     output.append((int(items[0]),instance,address,[]))

-- 
1.5.5.1

###########-------------

-- 
Today is Setting Orange, the 31st day of Bureaucracy in the YOLD 3174

kernel panic: write-only-memory (/dev/wom0) capacity exceeded.

_______________________________________________
Func-list mailing list
Func-list@xxxxxxxxxx
https://www.redhat.com/mailman/listinfo/func-list

[Index of Archives]     [Fedora Users]     [Linux Networking]     [Fedora Legacy List]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]

  Powered by Linux