* [Buildroot] [PATH v2 1/1] docs/manual: add qemu defconfigs gitlab test
@ 2020-09-15 10:38 Jugurtha BELKALEM
2020-09-15 14:51 ` Thomas Petazzoni
0 siblings, 1 reply; 4+ messages in thread
From: Jugurtha BELKALEM @ 2020-09-15 10:38 UTC (permalink / raw)
To: buildroot
Buildroot supports testing qemu defconfigs on gitlab since [1].
[1] : commit 0c7935063851accf3db4c9eb2fc4f11c3da0c391.
Signed-off-by: Jugurtha BELKALEM <jugurtha.belkalem@smile.fr>
---
Changes v1 -> v2:
- remove drive=hd0 from doc.
docs/manual/adding-board-support.txt | 18 ++++++++++++++++++
1 file changed, 18 insertions(+)
diff --git a/docs/manual/adding-board-support.txt b/docs/manual/adding-board-support.txt
index 33ed709535..6c7eb93ce2 100644
--- a/docs/manual/adding-board-support.txt
+++ b/docs/manual/adding-board-support.txt
@@ -46,3 +46,21 @@ create a directory +board/<manufacturer>+ and a subdirectory
+board/<manufacturer>/<boardname>+. You can then store your patches
and configurations in these directories, and reference them from the main
Buildroot configuration. Refer to xref:customize[] for more details.
+
+=== Adding qemu defconfig gitlab test
+
+Buildroot provides a test script (support/scripts/boot-qemu-image.py)
+to validate all defconfigs under +board/qemu/. Every supported qemu
+architecture should append the comment : # qemu_ARCH_defconfig
+at the end of its command line entry in board/qemu/ARCH/readme.txt (for
+example aarch64-virt includes : # qemu_aarch64_virt_defconfig).
+Once a week, every defconfig is tested under gitlab's pipelines :
+* Launches a qemu instance.
+* Enters root credentials and checks if login shell is available.
+* Powers off the instance.
+The job fails if one of above steps returns unsuccessful status code.
+to trigger qemu defconfigs test jobs:
+
+---------------------
+ $ git push gitlab HEAD:<name>_defconfig
+---------------------
--
2.11.0
^ permalink raw reply related [flat|nested] 4+ messages in thread
* [Buildroot] [PATH v2 1/1] docs/manual: add qemu defconfigs gitlab test
2020-09-15 10:38 [Buildroot] [PATH v2 1/1] docs/manual: add qemu defconfigs gitlab test Jugurtha BELKALEM
@ 2020-09-15 14:51 ` Thomas Petazzoni
2022-01-08 10:56 ` Romain Naour
0 siblings, 1 reply; 4+ messages in thread
From: Thomas Petazzoni @ 2020-09-15 14:51 UTC (permalink / raw)
To: buildroot
On Tue, 15 Sep 2020 12:38:06 +0200
Jugurtha BELKALEM <jugurtha.belkalem@smile.fr> wrote:
> +=== Adding qemu defconfig gitlab test
> +
> +Buildroot provides a test script (support/scripts/boot-qemu-image.py)
> +to validate all defconfigs under +board/qemu/. Every supported qemu
> +architecture should append the comment : # qemu_ARCH_defconfig
> +at the end of its command line entry in board/qemu/ARCH/readme.txt (for
> +example aarch64-virt includes : # qemu_aarch64_virt_defconfig).
> +Once a week, every defconfig is tested under gitlab's pipelines :
> +* Launches a qemu instance.
> +* Enters root credentials and checks if login shell is available.
> +* Powers off the instance.
> +The job fails if one of above steps returns unsuccessful status code.
> +to trigger qemu defconfigs test jobs:
> +
> +---------------------
> + $ git push gitlab HEAD:<name>_defconfig
> +---------------------
While I understand the idea of documenting this, I am wondering if the
Buildroot manual is really the right place for this. Indeed, this bit
of documentation is really only useful for the very specific/narrow use
case of qemu defconfigs.
Should this be in board/qemu/readme.txt ? I'm not sure.
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Buildroot] [PATH v2 1/1] docs/manual: add qemu defconfigs gitlab test
2020-09-15 14:51 ` Thomas Petazzoni
@ 2022-01-08 10:56 ` Romain Naour
2022-01-08 17:52 ` Thomas Petazzoni
0 siblings, 1 reply; 4+ messages in thread
From: Romain Naour @ 2022-01-08 10:56 UTC (permalink / raw)
To: Thomas Petazzoni, Jugurtha BELKALEM; +Cc: buildroot
Hello Thomas,
Le 15/09/2020 à 16:51, Thomas Petazzoni a écrit :
> On Tue, 15 Sep 2020 12:38:06 +0200
> Jugurtha BELKALEM <jugurtha.belkalem@smile.fr> wrote:
>
>> +=== Adding qemu defconfig gitlab test
>> +
>> +Buildroot provides a test script (support/scripts/boot-qemu-image.py)
>> +to validate all defconfigs under +board/qemu/. Every supported qemu
>> +architecture should append the comment : # qemu_ARCH_defconfig
>> +at the end of its command line entry in board/qemu/ARCH/readme.txt (for
>> +example aarch64-virt includes : # qemu_aarch64_virt_defconfig).
>> +Once a week, every defconfig is tested under gitlab's pipelines :
>> +* Launches a qemu instance.
>> +* Enters root credentials and checks if login shell is available.
>> +* Powers off the instance.
>> +The job fails if one of above steps returns unsuccessful status code.
>> +to trigger qemu defconfigs test jobs:
>> +
>> +---------------------
>> + $ git push gitlab HEAD:<name>_defconfig
>> +---------------------
>
> While I understand the idea of documenting this, I am wondering if the
> Buildroot manual is really the right place for this. Indeed, this bit
> of documentation is really only useful for the very specific/narrow use
> case of qemu defconfigs.
>
> Should this be in board/qemu/readme.txt ? I'm not sure.
Maybe the manual needs a new chapter about the gitlab-ci in general and document
all pipeline trigger supported.
For a new qemu defconfigs, we need to take care of:
- define "# qemu_ARCH_defconfig" tag in the file board/qemu/ARCH/readme.txt
- Use board/qemu/post-image.sh post image script and provide the name of the
defconfig as argument of this script:
BR2_ROOTFS_POST_IMAGE_SCRIPT="board/qemu/post-image.sh"
BR2_ROOTFS_POST_SCRIPT_ARGS="$(BR2_DEFCONFIG)"
- Enable host-qemu to use qemu built by Buildroot for the host:
BR2_PACKAGE_HOST_QEMU=y
BR2_PACKAGE_HOST_QEMU_SYSTEM_MODE=y
Being in the manual would promote gitlab-ci testing with Qemu.
Best regards,
Romain
>
> Thomas
>
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 4+ messages in thread
* Re: [Buildroot] [PATH v2 1/1] docs/manual: add qemu defconfigs gitlab test
2022-01-08 10:56 ` Romain Naour
@ 2022-01-08 17:52 ` Thomas Petazzoni
0 siblings, 0 replies; 4+ messages in thread
From: Thomas Petazzoni @ 2022-01-08 17:52 UTC (permalink / raw)
To: Romain Naour; +Cc: Jugurtha BELKALEM, buildroot
Hello Romain,
On Sat, 8 Jan 2022 11:56:54 +0100
Romain Naour <romain.naour@smile.fr> wrote:
> Maybe the manual needs a new chapter about the gitlab-ci in general and document
> all pipeline trigger supported.
>
> For a new qemu defconfigs, we need to take care of:
>
> - define "# qemu_ARCH_defconfig" tag in the file board/qemu/ARCH/readme.txt
>
> - Use board/qemu/post-image.sh post image script and provide the name of the
> defconfig as argument of this script:
> BR2_ROOTFS_POST_IMAGE_SCRIPT="board/qemu/post-image.sh"
> BR2_ROOTFS_POST_SCRIPT_ARGS="$(BR2_DEFCONFIG)"
>
> - Enable host-qemu to use qemu built by Buildroot for the host:
> BR2_PACKAGE_HOST_QEMU=y
> BR2_PACKAGE_HOST_QEMU_SYSTEM_MODE=y
>
> Being in the manual would promote gitlab-ci testing with Qemu.
Yes, I agree! We need a single place in the chapter with all the Gitlab
CI details. Like the test-pkg integration, testing of defconfigs,
testing of runtime tests, etc.
I'll mark this patch as Changes Requested, because we don't want just a
section/chapter specifically for the qemu defconfig testing in Gitlab
CI, but a broader section;
Please note that Gitlab CI testing of runtime tests is already
documented in the manual, so it will have to be refactored.
Best regards,
Thomas
--
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2022-01-08 17:52 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2020-09-15 10:38 [Buildroot] [PATH v2 1/1] docs/manual: add qemu defconfigs gitlab test Jugurtha BELKALEM
2020-09-15 14:51 ` Thomas Petazzoni
2022-01-08 10:56 ` Romain Naour
2022-01-08 17:52 ` Thomas Petazzoni
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.