On Sat, Feb 27, 2021 at 05:04:15PM +0800, Wu Fengguang wrote:

Why not just get the whole job? (to reduce a special API) It's no much difference in performance POV. Do you have security concerns?

Before there is no secret field, I just return stats, now we may support a get whole job API.

I will rename the API and return the whole job.

Thanks, Xueliang

Thanks, Fengguang

On Thu, Feb 25, 2021 at 02:15:32PM +0800, Cao Xueliang wrote:

...

We can expose assist-result host/port to the SUT, then the SUT can get the job stats.

Signed-off-by: Cao Xueliang caoxl78320@163.com

container/assist-result/routes.rb | 11 +++++++++++ container/assist-result/views/get_job_stats.rb | 10 ++++++++++ 2 files changed, 21 insertions(+) create mode 100644 container/assist-result/views/get_job_stats.rb

diff --git a/container/assist-result/routes.rb b/container/assist-result/routes.rb index 077a427..02956aa 100755 --- a/container/assist-result/routes.rb +++ b/container/assist-result/routes.rb @@ -7,6 +7,7 @@ require 'json' require 'sinatra'

require_relative './views/get_job_yaml' +require_relative './views/get_job_stats' require_relative './views/check_job_credible'

configure do @@ -34,3 +35,13 @@ post '/check_job_credible' do

return [200, {'credible' => result}.to_json] end

+get '/get_job_stats/:job_id' do

• begin
• result = get_job_stats(params[:job_id])
• rescue StandardError => e
• return [400, e.backtrace.inspect]
• end
• return [200, result.to_json]

+end diff --git a/container/assist-result/views/get_job_stats.rb b/container/assist-result/views/get_job_stats.rb new file mode 100644 index 0000000..d294df3 --- /dev/null +++ b/container/assist-result/views/get_job_stats.rb @@ -0,0 +1,10 @@ +# SPDX-License-Identifier: MulanPSL-2.0+ +# Copyright (c) 2020 Huawei Technologies Co., Ltd. All rights reserved. +# frozen_string_literal: true

+require_relative "#{ENV['CCI_SRC']}/lib/es_query"

+def get_job_stats(job_id)

• content = ESQuery.new.query_by_id(job_id)
• return content['stats']

+end

2.23.0