Skip to content

Commit

Permalink
Fix pylint errors
Browse files Browse the repository at this point in the history
Signed-off-by: Ian Hoang <[email protected]>
  • Loading branch information
Ian Hoang committed Jan 26, 2024
1 parent 53aec57 commit 8240d8f
Showing 1 changed file with 4 additions and 3 deletions.
7 changes: 4 additions & 3 deletions osbenchmark/workload/loader.py
Original file line number Diff line number Diff line change
Expand Up @@ -1022,17 +1022,18 @@ def read(self, workload_name, workload_spec_file, mapping_dir):
msg += " Lines containing the error:\n\n{}\n\n".format("\n".join(erroneous_lines))
msg += "The complete workload has been written to '{}' for diagnosis. \n\n".format(tmp.name)
console_message = f"Suggestion: Verify that [{workload_name}] workload has correctly formatted JSON files and " + \
f"Jinja Templates. For Jinja2 errors, consider using a live Jinja2 parser. " + \
"Jinja Templates. For Jinja2 errors, consider using a live Jinja2 parser. " + \
f"See common workload formatting errors:{WorkloadFileReader.COMMON_WORKLOAD_FORMAT_ERRORS}"
msg += console_message
raise WorkloadSyntaxError(msg)

except Exception as e:
# TypeErrors get logged here
self.logger.exception("Could not load [%s].", workload_spec_file)
msg = "Could not load '{}'. The complete workload has been written to '{}' for diagnosis. \n\n".format(workload_spec_file, tmp.name)
msg = "Could not load '{}'. The complete workload has been written to '{}' for diagnosis. \n\n".format(
workload_spec_file, tmp.name)
console_message = f"Suggestion: Verify that [{workload_name}] workload has correctly formatted JSON files and " + \
f"Jinja Templates. For Jinja2 errors, consider using a live Jinja2 parser. " + \
"Jinja Templates. For Jinja2 errors, consider using a live Jinja2 parser. " + \
f"See common workload formatting errors:{WorkloadFileReader.COMMON_WORKLOAD_FORMAT_ERRORS}"
msg += console_message
# Convert to string early on to avoid serialization errors with Jinja exceptions.
Expand Down

0 comments on commit 8240d8f

Please sign in to comment.