[Compass-ci] Re: [PATCH compass-ci] delimiter: add common account info to bisect job

On Sat, Nov 28, 2020 at 04:09:05PM +0800, Xu Xijian wrote:

On Sat, Nov 28, 2020 at 04:04:37PM +0800, Cao Xueliang wrote:

...

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

src/delimiter/utils.rb | 12 ++++++++++-- 1 file changed, 10 insertions(+), 2 deletions(-)

diff --git a/src/delimiter/utils.rb b/src/delimiter/utils.rb index 679c8d9..2be99b5 100644 --- a/src/delimiter/utils.rb +++ b/src/delimiter/utils.rb @@ -88,16 +88,24 @@ module Utils return 'good' end

• def get_account_info

•  ESQuery.new(index: 'accounts').query_by_id('compass-ci@qq.com')
  

• end

• def init_job_content(job_id) es = ESQuery.new job = es.query_by_id(job_id) raise "es query job id: #{job_id} failed!" unless job

•  account_info = get_account_info
  

•  raise "query compass-ci@qq.com account info failed!" unless account_info
  

• job['bad_job_id'] = job_id
  job['suite'] = 'bisect'
  

•  job['my_email'] = account_info['my_email']
  

•  job['my_name'] = account_info['my_name']
  

•  job['my_uuid'] = account_info['my_uuid']
  

Suggest display like this:

job['bad_job_id'] = job_id job['suite'] = 'bisect' job['my_email'] = account_info['my_email'] job['my_name'] = account_info['my_name'] job['my_uuid'] = account_info['my_uuid']

I think to adjust the order will be better than this alignment.

Thanks, Xueliang

Thanks, Xijian