Skip to content

Commit

Permalink
coredump-report: Fix OSError for ulimit command
Browse files Browse the repository at this point in the history
The ulimit needs to be executed through shell but passing cmd as list
preventing it to be executed and getting OSError. The command_output func
sets shell to True in popen call if cmd is a string object.

Signed-off-by: Prashant D <pdhange@redhat.com>
  • Loading branch information
Prashant D committed Oct 9, 2017
1 parent ebb4b97 commit 6c17e46
Showing 1 changed file with 1 addition and 1 deletion.
2 changes: 1 addition & 1 deletion glusterhealth/reports/coredump.py
Original file line number Diff line number Diff line change
Expand Up @@ -13,7 +13,7 @@
from utils import command_output, CommandError

def report_coredump(ctx):
cmd = ["ulimit", "-c"]
cmd = "ulimit -c"
try:
out = command_output(cmd)
if out.strip() == "unlimited":
Expand Down

0 comments on commit 6c17e46

Please sign in to comment.