X-Git-Url: http://git.onelab.eu/?a=blobdiff_plain;f=sfa%2Futil%2FrspecHelper.py;h=89f15afdb7d42901aac4ecf2c953df133a56c994;hb=db091e73c33c373b7f6c2c96bd2caf6a2acf0178;hp=f6c860b4e4b35eb12d499453ace178b682afb6b2;hpb=cfeb502877d73623a039217f77627f3eb91f273c;p=sfa.git diff --git a/sfa/util/rspecHelper.py b/sfa/util/rspecHelper.py index f6c860b4..89f15afd 100755 --- a/sfa/util/rspecHelper.py +++ b/sfa/util/rspecHelper.py @@ -8,7 +8,7 @@ from StringIO import StringIO from optparse import OptionParser from sfa.util.faults import * -from sfa.util.sfalogging import sfa_logger +from sfa.util.sfalogging import logger def merge_rspecs(rspecs): """ @@ -24,29 +24,31 @@ def merge_rspecs(rspecs): try: known_networks[network.get('name')]=True except: - sfa_logger().error("merge_rspecs: cannot register network with no name in rspec") + logger.error("merge_rspecs: cannot register network with no name in rspec") pass def is_registered_network (network): try: return network.get('name') in known_networks except: - sfa_logger().error("merge_rspecs: cannot retrieve network with no name in rspec") + logger.error("merge_rspecs: cannot retrieve network with no name in rspec") return False # the resulting tree rspec = None for input_rspec in rspecs: + # ignore empty strings as returned with used call_ids + if not input_rspec: continue try: tree = etree.parse(StringIO(input_rspec)) except etree.XMLSyntaxError: # consider failing silently here - sfa_logger().log_exc("merge_rspecs, parse error") + logger.log_exc("merge_rspecs, parse error") message = str(sys.exc_info()[1]) + ' with ' + input_rspec raise InvalidRSpec(message) root = tree.getroot() if not root.get("type") in ["SFA"]: - sfa_logger().error("merge_rspecs: unexpected type for rspec root, %s"%root.get('type')) + logger.error("merge_rspecs: unexpected type for rspec root, %s"%root.get('type')) continue if rspec == None: # we scan the first input, register all networks