[PATCH lkp-tests] sbin/submit: fix -m when submit job failed

[Why] When use "submit -m", if submit job failed, the monitor is still run. [Error] submit iperf.yaml failed, got job_id=0, error: Failed to verify the account. Please check your configuration query=>{"job_id":[]} connect to ws://localhost:11310/filter Signed-off-by: Wu Zhende <wuzhende666@163.com> --- sbin/submit | 1 + 1 file changed, 1 insertion(+) diff --git a/sbin/submit b/sbin/submit index 6766e704a..05b3e86dd 100755 --- a/sbin/submit +++ b/sbin/submit @@ -162,6 +162,7 @@ ARGV.each do |jobfile| job_ids << msg['job_id'].to_s puts("submit #{jobfile}, got job_id=#{msg['job_id']}") else + opt_monitor = false puts("submit #{jobfile} failed, got job_id=#{msg['job_id']}, error: #{msg['message']}") end end -- 2.23.0

On Mon, Dec 14, 2020 at 09:33:56AM +0800, Wu Zhende wrote:
[Why] When use "submit -m", if submit job failed, the monitor is still run.
[Error] submit iperf.yaml failed, got job_id=0, error: Failed to verify the account. Please check your configuration
How about show which configurations the user need to config? Thanks, Luan Shengde
query=>{"job_id":[]} connect to ws://localhost:11310/filter
Signed-off-by: Wu Zhende <wuzhende666@163.com> --- sbin/submit | 1 + 1 file changed, 1 insertion(+)
diff --git a/sbin/submit b/sbin/submit index 6766e704a..05b3e86dd 100755 --- a/sbin/submit +++ b/sbin/submit @@ -162,6 +162,7 @@ ARGV.each do |jobfile| job_ids << msg['job_id'].to_s puts("submit #{jobfile}, got job_id=#{msg['job_id']}") else + opt_monitor = false puts("submit #{jobfile} failed, got job_id=#{msg['job_id']}, error: #{msg['message']}") end end -- 2.23.0

On Mon, Dec 14, 2020 at 09:38:17AM +0800, Luan Shengde wrote:
On Mon, Dec 14, 2020 at 09:33:56AM +0800, Wu Zhende wrote:
[Why] When use "submit -m", if submit job failed, the monitor is still run.
[Error] submit iperf.yaml failed, got job_id=0, error: Failed to verify the account. Please check your configuration
How about show which configurations the user need to config?
May show user the document URL. Then exit. Thanks, Fengguang
participants (3)
-
Luan Shengde
-
Wu Fengguang
-
Wu Zhende