Skip to content

Commit

Permalink
Merge pull request #18527 from mvdbeek/strip_unicode_null_tool_stdio_…
Browse files Browse the repository at this point in the history
…pulsar

[24.1] Strip unicode null from tool stdio
  • Loading branch information
dannon authored Jul 11, 2024
2 parents ce36118 + 2b2a4f8 commit dc0ffcd
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions lib/galaxy/jobs/runners/pulsar.py
Original file line number Diff line number Diff line change
Expand Up @@ -57,6 +57,7 @@
galaxy_directory,
specs,
string_as_bool_or_none,
unicodify,
)

log = logging.getLogger(__name__)
Expand Down Expand Up @@ -649,8 +650,8 @@ def finish_job(self, job_state: JobState):
client = self.get_client_from_state(job_state)
run_results = client.full_status()
remote_metadata_directory = run_results.get("metadata_directory", None)
tool_stdout = run_results.get("stdout", "")
tool_stderr = run_results.get("stderr", "")
tool_stdout = unicodify(run_results.get("stdout", ""), strip_null=True)
tool_stderr = unicodify(run_results.get("stderr", ""), strip_null=True)
job_stdout = run_results.get("job_stdout")
job_stderr = run_results.get("job_stderr")
exit_code = run_results.get("returncode")
Expand Down

0 comments on commit dc0ffcd

Please sign in to comment.