On Thu, Apr 15, 2021 at 09:59:02AM +0800, Li Yuanchao wrote:

On Thu, Apr 15, 2021 at 09:37:08AM +0800, Lin Jiaxin wrote:

...

Assigning a null value to upstream_repo is meaningless. And when use "submit upstream_repo=xxx rpmbuild-pkg.yaml" will make upstream_repo be null. Unless we put "upstream_repo=xxx" behind. Or declare it in yaml.

Because the argvs in submit command have a priority that the right one would have higher priority, and would cover the left one if they are the same.

So upstream_repo is null because there is a upstream_repo in rpmbuild-pkg.yaml and it's null, it covers the one left of it.

Yes. So i remove it for better availablity.

Thanks, Jiaxin

Thanks, Yuanchao

...

[example] linjiaxin@crystal ~%/home/linjiaxin/lkp-tests/sbin/submit upstream_repo="a/azk/azk" upstream_url=https://github.com/azukiapp/azk pkgbuild_repo=archlinux/aur-a/azk-git testbox=dc-16g os=openeuler os_version=20.03-LTS os_mount=container os_arch=aarch64 docker_image=openeuler:20.03-LTS repo_name=aur rpmbuild-pkg.yaml upstream_branch=master upstream_commit=e72adb039e4dd64fd6ed81d82fd597c1d4efdf14 upstream_dir=upstream submit_id=d18fabcd-6302-464c-aa99-3ed9e610704b submit /home/linjiaxin/lkp-tests/jobs/rpmbuild-pkg.yaml, got job id=crystal.1594407

linjiaxin@crystal ~% es-find id=crystal.1594407 | grep upstream_repo "upstream_repo": null,

Signed-off-by: Lin Jiaxin ljx.joe@qq.com

jobs/rpmbuild-pkg.yaml | 1 - 1 file changed, 1 deletion(-)

diff --git a/jobs/rpmbuild-pkg.yaml b/jobs/rpmbuild-pkg.yaml index a70590af..5860ba02 100644 --- a/jobs/rpmbuild-pkg.yaml +++ b/jobs/rpmbuild-pkg.yaml @@ -1,5 +1,4 @@ suite: rpmbuild-pkg category: functional

-upstream_repo: rpmbuild-pkg: -- 2.23.0