nnstreamer / TAOS-CI

:tangerine: TAOS-CI (a.k.a LightSys) is an lightweight and efficient CI system to achieve "Review less, merge faster" with a tool-based review system. It is completely compatible with all GitHub repositories although TAOS-CI is designed & implemented for NNStreamer repository. (Homepage: https://taos-ci.github.io/)
http://nnstreamer.mooo.com/TAOS-CI/ci/taos/
Apache License 2.0
117 stars 90 forks source link

[Plugin] Add plugin for yocto-kas build #702

Closed anyj0527 closed 3 years ago

anyj0527 commented 3 years ago

REF:

Please check: https://github.sec.samsung.net/yongjoo1-ahn/meta-neural-network-yongjoo/pull/7 https://github.sec.samsung.net/yongjoo1-ahn/meta-neural-network-yongjoo/pull/6

Signed-off-by: Yongjoo Ahn yongjoo1.ahn@samsung.com

Self evaluation: Run test: [X]Passed [ ]Failed [ ]Skipped

taos-ci commented 3 years ago

:memo: TAOS-CI Version: 1.5.20200925. Thank you for submitting PR #702. Please a submit 1commit/1PR (one commit per one PR) policy to get comments quickly from reviewers. Your PR must pass all verificiation processes of cibot before starting a review process from reviewers. If you are new member to join this project, please read manuals in documentation folder and wiki page. In order to monitor a progress status of your PR in more detail, visit http://nnsuite.mooo.com/.

leemgs commented 3 years ago

Please check: https://github.sec.samsung.net/yongjoo1-ahn/meta-neural-network-yongjoo/pull/7 https://github.sec.samsung.net/yongjoo1-ahn/meta-neural-network-yongjoo/pull/6

@anyj0527 , Could you upload the execution result (See below your work: two PRs) of the self-assessment here in order that I merge this PR finally? 🏎️

image

image

anyj0527 commented 3 years ago

@leemgs

For https://github.sec.samsung.net/yongjoo1-ahn/meta-neural-network-yongjoo/pull/6 :

You can check the build log for intended failure in ........../report/build_log_{PR_NUMBER}_yocto_kas_log.txt:

...
2021-02-02 17:48:15 - ERROR    - ERROR: ParseError at /var/www/html/meta-neural-network-yongjoo/TAOS-CI/ci/repo-workers/pr-checker/6-202102021748030.76346206665039-0f3124a09f6f3c91e65415e92be05c7ef63655c2/meta-neural-network-yongjoo/kas-build/meta-neural-network/recipes-nnstreamer/nnstreamer/nnstreamer_1.7.0.bb:10: unparsed line: 'This line fails CI build test'
2021-02-02 17:48:17 - INFO     - Parsing recipes...
2021-02-02 17:48:17 - INFO     - Summary: There was 1 ERROR message shown, returning a non-zero exit code.
2021-02-02 17:48:17 - ERROR    - Command "/var/www/html/meta-neural-network-yongjoo/TAOS-CI/ci/repo-workers/pr-checker/6-202102021748030.76346206665039-0f3124a09f6f3c91e65415e92be05c7ef63655c2/meta-neural-network-yongjoo/kas-build/build$ /var/www/html/meta-neural-network-yongjoo/TAOS-CI/ci/repo-workers/pr-checker/6-202102021748030.76346206665039-0f3124a09f6f3c91e65415e92be05c7ef63655c2/meta-neural-network-yongjoo/kas-build/poky/bitbake/bin/bitbake -k -c build core-image-minimal" failed
--- Error summary ---
ERROR: ParseError at /var/www/html/meta-neural-network-yongjoo/TAOS-CI/ci/repo-workers/pr-checker/6-202102021748030.76346206665039-0f3124a09f6f3c91e65415e92be05c7ef63655c2/meta-neural-network-yongjoo/kas-build/meta-neural-network/recipes-nnstreamer/nnstreamer/nnstreamer_1.7.0.bb:10: unparsed line: 'This line fails CI build test'

For https://github.sec.samsung.net/yongjoo1-ahn/meta-neural-network-yongjoo/pull/7 :

You can check the built packages and image in ........../meta-neural-network-yongjoo/kas-build/build/tmp/deploy/*:

$ cd ~/html/meta-neural-network-yongjoo/ci/repo-workers/pr-checker/7-202102050946360.75058197975159-ddf23f2b556b7653aa4c857d81a2f6d2c6d30221/meta-neural-network-yongjoo/kas-build/build/tmp/deploy/rpm/aarch64
$ ls | grep nnstreamer
nnstreamer-1.7.0+git0+d14c6892c2-r0.aarch64.rpm
nnstreamer-dbg-1.7.0+git0+d14c6892c2-r0.aarch64.rpm
nnstreamer-dev-1.7.0+git0+d14c6892c2-r0.aarch64.rpm
nnstreamer-src-1.7.0+git0+d14c6892c2-r0.aarch64.rpm
nnstreamer-staticdev-1.7.0+git0+d14c6892c2-r0.aarch64.rpm
nnstreamer-tensorflow-lite-1.7.0+git0+d14c6892c2-r0.aarch64.rpm
nnstreamer-unittest-1.7.0+git0+d14c6892c2-r0.aarch64.rpm

$ cd ~/html/meta-neural-network-yongjoo/ci/repo-workers/pr-checker/7-202102050946360.75058197975159-ddf23f2b556b7653aa4c857d81a2f6d2c6d30221/meta-neural-network-yongjoo/kas-build/build/tmp/deploy/images/qemuarm64
$ ls
core-image-minimal-qemuarm64-20210205004756.qemuboot.conf
core-image-minimal-qemuarm64-20210205004756.rootfs.ext4
core-image-minimal-qemuarm64-20210205004756.rootfs.manifest
core-image-minimal-qemuarm64-20210205004756.rootfs.tar.bz2
core-image-minimal-qemuarm64-20210205004756.testdata.json
core-image-minimal-qemuarm64.ext4@
core-image-minimal-qemuarm64.manifest@
core-image-minimal-qemuarm64.qemuboot.conf@
core-image-minimal-qemuarm64.tar.bz2@
core-image-minimal-qemuarm64.testdata.json@
Image@
Image--5.2.32+git0+bb2776d6be_73b12de4c8-r0-qemuarm64-20210128043933.bin
Image-qemuarm64.bin@
modules--5.2.32+git0+bb2776d6be_73b12de4c8-r0-qemuarm64-20210128043933.tgz
modules-qemuarm64.tgz@
leemgs commented 3 years ago

Please check: https://github.sec.samsung.net/yongjoo1-ahn/meta-neural-network-yongjoo/pull/7 https://github.sec.samsung.net/yongjoo1-ahn/meta-neural-network-yongjoo/pull/6

@anyj0527 , Could you upload the execution result (See below your work: two PRs) of the self-assessment here in order that I merge this PR finally? 🏎️

  • [FYI] The below screenshot shows I captured on Feb-05-2021 at 09:00.

image

image

@anyj0527 , PTAL. REMINDER V2.

anyj0527 commented 3 years ago

@leemgs

I don't know what exactly self assessment is. Does https://github.com/nnstreamer/TAOS-CI/pull/702#issuecomment-773716041 not enough? I could boot qemu with built image.

BTW I'm @anyj0527 not @helloahn . it's Hyoungjoo :)

leemgs commented 3 years ago

I don't know what exactly self assessment is.

The self-assessment means that you verified by testing in your local PC to avoid an integration issue due to incorrect PRs. [FYI] https://github.com/nnstreamer/TAOS-CI/blob/main/ci/doc/self-assessment-before-submitting-pr.md

Does #702 (comment) not enough? I could boot qemu with built image.

I checked that you added your locat test now at the below address. Acked-by: Geunsik Lim. https://github.com/nnstreamer/TAOS-CI/pull/702#issuecomment-773716041

BTW I'm @anyj0527 not @helloahn .

Pardon me. 😅 Replaced above. (s/helloahn/anyj0527)

leemgs commented 3 years ago

@anyj0527 , I like yocto-kas (new module). Could you append a summary on the yocto-kas module at the below wiki page?

anyj0527 commented 3 years ago

@leemgs Thanks! I added a summary.