All of lore.kernel.org
 help / color / mirror / Atom feed
* [Buildroot] [PATCH] support/scripts/generate-gitlab-ci-yml: make it possible to test multiple defconfigs
@ 2021-12-29 14:38 Thomas Petazzoni
  2021-12-30 16:44 ` Romain Naour
  2021-12-30 17:42 ` Yann E. MORIN
  0 siblings, 2 replies; 3+ messages in thread
From: Thomas Petazzoni @ 2021-12-29 14:38 UTC (permalink / raw)
  To: Buildroot List; +Cc: Romain Naour, Thomas Petazzoni

Already supported:

 - Pushing a branch called "<foo>-defconfigs" tests all defconfigs.

 - Pushing a branch called "<foo>-defconfig-<defconfig-name>" will
   test one particular defconfig

This commit adds support for:

 - Pushing a branch called "<foo>-defconfigs-<pattern>" which will
   test all defconfigs whose name start with the pattern. For example
   "<foo>-defconfigs-qemu_" will test all Qemu defconfigs

Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
---
Ideally, this should be documented in the Buildroot manual, but I'm
not sure where it should be documented.
---
 support/scripts/generate-gitlab-ci-yml | 5 +++++
 1 file changed, 5 insertions(+)

diff --git a/support/scripts/generate-gitlab-ci-yml b/support/scripts/generate-gitlab-ci-yml
index 8aaa456b03..3e0c68091b 100755
--- a/support/scripts/generate-gitlab-ci-yml
+++ b/support/scripts/generate-gitlab-ci-yml
@@ -66,6 +66,11 @@ gen_tests() {
           (*-defconfigs)
             do_defconfigs=base
             ;;
+	  (*-defconfigs-*)
+            pattern=$(echo ${CI_COMMIT_REF_NAME} | sed 's%[^\-]*-defconfigs-\(.*\)%\1%')
+            defconfigs=( $(cd configs; LC_ALL=C ls -1 | grep ^${pattern}) )
+            do_defconfigs=base
+            ;;
           (*-*_defconfig)
             defconfigs=( "${CI_COMMIT_REF_NAME##*-}" )
             do_defconfigs=base
-- 
2.31.1

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply related	[flat|nested] 3+ messages in thread

* Re: [Buildroot] [PATCH] support/scripts/generate-gitlab-ci-yml: make it possible to test multiple defconfigs
  2021-12-29 14:38 [Buildroot] [PATCH] support/scripts/generate-gitlab-ci-yml: make it possible to test multiple defconfigs Thomas Petazzoni
@ 2021-12-30 16:44 ` Romain Naour
  2021-12-30 17:42 ` Yann E. MORIN
  1 sibling, 0 replies; 3+ messages in thread
From: Romain Naour @ 2021-12-30 16:44 UTC (permalink / raw)
  To: Thomas Petazzoni, Buildroot List

Hello Thomas,

Le 29/12/2021 à 15:38, Thomas Petazzoni a écrit :
> Already supported:
> 
>  - Pushing a branch called "<foo>-defconfigs" tests all defconfigs.
> 
>  - Pushing a branch called "<foo>-defconfig-<defconfig-name>" will
>    test one particular defconfig
> 
> This commit adds support for:
> 
>  - Pushing a branch called "<foo>-defconfigs-<pattern>" which will
>    test all defconfigs whose name start with the pattern. For example
>    "<foo>-defconfigs-qemu_" will test all Qemu defconfigs

I'm doing a shameless local hack in generate-gitlab-ci-yml to keep only qemu*
defconfig for testing in gitlab:

defconfigs=( $(cd configs; LC_ALL=C ls -1 qemu*_defconfig) )

> 
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>
> ---
> Ideally, this should be documented in the Buildroot manual, but I'm
> not sure where it should be documented.

Probably a new chapter after "22.7. Using the run-tests framework".

"22.8. Testing using Buildroot's gitlab-ci"

> ---
>  support/scripts/generate-gitlab-ci-yml | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/support/scripts/generate-gitlab-ci-yml b/support/scripts/generate-gitlab-ci-yml
> index 8aaa456b03..3e0c68091b 100755
> --- a/support/scripts/generate-gitlab-ci-yml
> +++ b/support/scripts/generate-gitlab-ci-yml
> @@ -66,6 +66,11 @@ gen_tests() {
>            (*-defconfigs)
>              do_defconfigs=base
>              ;;
> +	  (*-defconfigs-*)
> +            pattern=$(echo ${CI_COMMIT_REF_NAME} | sed 's%[^\-]*-defconfigs-\(.*\)%\1%')
> +            defconfigs=( $(cd configs; LC_ALL=C ls -1 | grep ^${pattern}) )
> +            do_defconfigs=base
> +            ;;

I'll use this patch next time I'll test qemu defconfig.

Best regards,
Romain


>            (*-*_defconfig)
>              defconfigs=( "${CI_COMMIT_REF_NAME##*-}" )
>              do_defconfigs=base
> 

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 3+ messages in thread

* Re: [Buildroot] [PATCH] support/scripts/generate-gitlab-ci-yml: make it possible to test multiple defconfigs
  2021-12-29 14:38 [Buildroot] [PATCH] support/scripts/generate-gitlab-ci-yml: make it possible to test multiple defconfigs Thomas Petazzoni
  2021-12-30 16:44 ` Romain Naour
@ 2021-12-30 17:42 ` Yann E. MORIN
  1 sibling, 0 replies; 3+ messages in thread
From: Yann E. MORIN @ 2021-12-30 17:42 UTC (permalink / raw)
  To: Thomas Petazzoni; +Cc: Romain Naour, Buildroot List

Thomas, All,

On 2021-12-29 15:38 +0100, Thomas Petazzoni spake thusly:
> Already supported:
> 
>  - Pushing a branch called "<foo>-defconfigs" tests all defconfigs.
> 
>  - Pushing a branch called "<foo>-defconfig-<defconfig-name>" will
>    test one particular defconfig
> 
> This commit adds support for:
> 
>  - Pushing a branch called "<foo>-defconfigs-<pattern>" which will
>    test all defconfigs whose name start with the pattern. For example
>    "<foo>-defconfigs-qemu_" will test all Qemu defconfigs
> 
> Signed-off-by: Thomas Petazzoni <thomas.petazzoni@bootlin.com>

Applied to master, thanks.

Regards,
Yann E. MORIN.

> ---
> Ideally, this should be documented in the Buildroot manual, but I'm
> not sure where it should be documented.
> ---
>  support/scripts/generate-gitlab-ci-yml | 5 +++++
>  1 file changed, 5 insertions(+)
> 
> diff --git a/support/scripts/generate-gitlab-ci-yml b/support/scripts/generate-gitlab-ci-yml
> index 8aaa456b03..3e0c68091b 100755
> --- a/support/scripts/generate-gitlab-ci-yml
> +++ b/support/scripts/generate-gitlab-ci-yml
> @@ -66,6 +66,11 @@ gen_tests() {
>            (*-defconfigs)
>              do_defconfigs=base
>              ;;
> +	  (*-defconfigs-*)
> +            pattern=$(echo ${CI_COMMIT_REF_NAME} | sed 's%[^\-]*-defconfigs-\(.*\)%\1%')
> +            defconfigs=( $(cd configs; LC_ALL=C ls -1 | grep ^${pattern}) )
> +            do_defconfigs=base
> +            ;;
>            (*-*_defconfig)
>              defconfigs=( "${CI_COMMIT_REF_NAME##*-}" )
>              do_defconfigs=base
> -- 
> 2.31.1
> 
> _______________________________________________
> buildroot mailing list
> buildroot@buildroot.org
> https://lists.buildroot.org/mailman/listinfo/buildroot

-- 
.-----------------.--------------------.------------------.--------------------.
|  Yann E. MORIN  | Real-Time Embedded | /"\ ASCII RIBBON | Erics' conspiracy: |
| +33 662 376 056 | Software  Designer | \ / CAMPAIGN     |  ___               |
| +33 561 099 427 `------------.-------:  X  AGAINST      |  \e/  There is no  |
| http://ymorin.is-a-geek.org/ | _/*\_ | / \ HTML MAIL    |   v   conspiracy.  |
'------------------------------^-------^------------------^--------------------'
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

^ permalink raw reply	[flat|nested] 3+ messages in thread

end of thread, other threads:[~2021-12-30 17:42 UTC | newest]

Thread overview: 3+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2021-12-29 14:38 [Buildroot] [PATCH] support/scripts/generate-gitlab-ci-yml: make it possible to test multiple defconfigs Thomas Petazzoni
2021-12-30 16:44 ` Romain Naour
2021-12-30 17:42 ` Yann E. MORIN

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.