InfuseAI / primehub

open-source MLOps platform
https://docs.primehub.io
Apache License 2.0
389 stars 39 forks source link

fix: show GPU resource, unnecessary semicolon #653

Closed jimmyliao closed 2 years ago

jimmyliao commented 2 years ago

Signed-off-by: Jimmy Liao jimmyliao@jimmyliao.net

PR checklist

What type of PR is this? Remove unnecessary semicolon on GPU resource grep

What this PR does / why we need it:

Before:

gpu=$(echo $resource | grep 'nvidia.com/gpu'; | sed 's/[^0-9]*//g')
-bash: command substitution: line 39: syntax error near unexpected token `|'
-bash: command substitution: line 39: `echo $resource | grep 'nvidia.com/gpu'; | sed 's/[^0-9]*//g')'

Which issue(s) this PR fixes:

Special notes for your reviewer:

Does this PR introduce a user-facing change?: