[Compass-ci] Re: [PATCH lkp-tests 3/6] iso2rootfs: add a switch to configure rootfs

...

diff --git a/jobs/iso2rootfs.yaml b/jobs/iso2rootfs.yaml index 4e8729997a00..c81d548b49ea 100644 --- a/jobs/iso2rootfs.yaml +++ b/jobs/iso2rootfs.yaml @@ -13,6 +13,8 @@ iso2rootfs:

# dailybuild related fields iso_url_file:

• ## 0 - config; 1- don't config

• config_rootfs:

  # submit test yaml related fields test_yaml:

diff --git a/tests/iso2rootfs b/tests/iso2rootfs index d42c38909f68..34c8e84d9d74 100755 --- a/tests/iso2rootfs +++ b/tests/iso2rootfs @@ -355,6 +355,7 @@ disable_selinux()

config_rootfs() {

• [ "${config_rootfs}" == "0" ] || return 0

                          ^^ is there need string "0" ?

yes, you're right, this is surely a good point.

because of we can't limit the user's input content of job.yaml, so use string instead of integer here. :)

-------- Thanks Yu Chuan

                          Thanks,
                          Baijing

...

config_dns_resolver disable_selinux } -- 2.23.0