Ottomata has submitted this change and it was merged.
Change subject: Two fixes to rolematcher.py
......................................................................
Two fixes to rolematcher.py
Change-Id: I13b481707feafac3bd09188ae80a07ff6bc8b8a8
---
M files/misc/PacketLossLogtailer.py
M files/misc/rolematcher.py
2 files changed, 3 insertions(+), 3 deletions(-)
Approvals:
Ottomata: Verified; Looks good to me, approved
jenkins-bot: Verified
diff --git a/files/misc/PacketLossLogtailer.py
b/files/misc/PacketLossLogtailer.py
index 4c2c2f9..42e3d99 100644
--- a/files/misc/PacketLossLogtailer.py
+++ b/files/misc/PacketLossLogtailer.py
@@ -27,12 +27,12 @@
def __init__(self):
'''This function should initialize any data structures or variables
needed for the internal state of the line parser.'''
+ self.last_update_time = time.time()
self.reset_state()
self.lock = threading.RLock()
# a list of rolematchers which are simple object to determine the role
of a particular server
# this list is obtained from crawling noc.wikimedia.org/pybal and
parse the available configurations
self.day_in_seconds = 24 * 60 * 60
- self.last_update_time = time.time()
self.matchers = rolematcher.init()
# this is what will match the packet loss lines
# packet loss format :
diff --git a/files/misc/rolematcher.py b/files/misc/rolematcher.py
index 1a4dd6d..db3024e 100644
--- a/files/misc/rolematcher.py
+++ b/files/misc/rolematcher.py
@@ -204,11 +204,11 @@
if __name__ == '__main__':
- if len(sys.argv) != 1:
+ if len(sys.argv) != 2:
print 'Please specify path to packetloss log file, call this file only
for testing purposes.'
sys.exit(-1)
else:
- path = sys.argv[0]
+ path = sys.argv[1]
matchers = init()
line_matcher = re.compile('^\[(?P<date>[^]]+)\] (?P<server>[^ ]+) lost:
\((?P<percentloss>[^ ]+) \+\/- (?P<margin>[^)]+)\)%')
--
To view, visit https://gerrit.wikimedia.org/r/63668
To unsubscribe, visit https://gerrit.wikimedia.org/r/settings
Gerrit-MessageType: merged
Gerrit-Change-Id: I13b481707feafac3bd09188ae80a07ff6bc8b8a8
Gerrit-PatchSet: 2
Gerrit-Project: operations/puppet
Gerrit-Branch: production
Gerrit-Owner: Diederik <[email protected]>
Gerrit-Reviewer: Ottomata <[email protected]>
Gerrit-Reviewer: jenkins-bot
_______________________________________________
MediaWiki-commits mailing list
[email protected]
https://lists.wikimedia.org/mailman/listinfo/mediawiki-commits
