Skip to content

[loganalyzer] create_msg_regex condition fix #65

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
lguohan merged 1 commit into from
Dec 13, 2016
Merged

[loganalyzer] create_msg_regex condition fix #65

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
15 changes: 8 additions & 7 deletions ansible/roles/test/files/tools/loganalyzer/loganalyzer.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -168,7 +168,7 @@ def create_msg_regex(self, file_lsit):
'''
messages_regex = []

if file_lsit is None or len(file_lsit):
if file_lsit is None or (0 == len(file_lsit)):
return None

for filename in file_lsit:
Expand Down Expand Up @@ -306,7 +306,8 @@ def analyze_file(self, log_file_path, match_messages_regex, ignore_messages_rege
break

if in_analysis_range :
if self.line_matches(rev_line, match_messages_regex, ignore_messages_regex):
if self.line_matches(rev_line, match_messages_regex, ignore_messages_regex):
self.print_diagnostic_message('matching string: %s' % rev_line)
matching_lines.append(rev_line)

if (not found_start_marker):
Expand Down Expand Up @@ -532,7 +533,10 @@ def main(argv):
return 0
elif (action == "analyze"):
match_file_list = match_files_in.split(tokenizer);
ignore_file_list = ignore_files_in.split(tokenizer);

ignore_file_list = []
if ignore_files_in :
ignore_file_list = ignore_files_in.split(tokenizer);

analyzer.place_marker(log_file_list, analyzer.create_end_marker())

Expand All @@ -551,7 +555,4 @@ def main(argv):
#---------------------------------------------------------------------

if __name__ == "__main__":
main(sys.argv[1:])



main(sys.argv[1:])