[Compass-ci] Re: [PATCH lkp-tests 1/3] lib/job.rb: load configuration files

On Wed, Dec 16, 2020 at 11:07:23AM +0800, Wu Fengguang wrote:

On Tue, Dec 15, 2020 at 04:09:33PM +0800, Liu Yinsi wrote:

...

scheduler need to read administrator account configuration info, such as account autosubmit.

Signed-off-by: Liu Yinsi liuyinsi@163.com

lib/job.rb | 1 + 1 file changed, 1 insertion(+)

diff --git a/lib/job.rb b/lib/job.rb index b41d55a7c..e6a96fd1f 100755 --- a/lib/job.rb +++ b/lib/job.rb @@ -427,6 +427,7 @@ class Job end

Dir.glob(["/etc/compass-ci/defaults/*.yaml",

•          "/etc/compass-ci/account/*.yaml",
  

Only defaults/* are to be loaded as job defaults, into _every_ job.

If you want that behavior, can add

   /etc/compass-ci/defaults/admin-account.yaml

Otherwise if only a small fraction of the system need this configuration, consider create

   /etc/compass-ci/admin-account.yaml

It looks fine to have

/etc/compass-ci/account/autosubmit.yaml

Just don't do this patch.

Thanks, Fengguang

...

          "#{ENV['HOME']}/.config/compass-ci/defaults/*.yaml"]).each do |file|
  load_one_defaults(file, @job)
end

-- 2.23.0