search

kernelci / kernelci-core

Core KernelCI tools
https://kernelci.org
GNU Lesser General Public License v2.1
104 stars 97 forks source link

config/v4l2-decoder-conformance: Replace dots in LAVA test set/case names #2599

Closed laura-nao closed 2 months ago

laura-nao commented 3 months ago

Dots are used in KCIDB to denote test hierarchy; this can lead to misinterpretation when they're used in LAVA test sets and case names. Modify the fluster parser script to replace dots with dashes to prevent this issue.

JenySadadia commented 3 months ago

Thanks a lot @laura-nao for the fix. Let's wait for the results on staging in the next run.

laura-nao commented 3 months ago

Thanks a lot @laura-nao for the fix. Let's wait for the results on staging in the next run.

@JenySadadia on second thought, in order to test the changes we'll need to generate a new rootfs image (and I guess update the link in the test configuration on kernelci-pipeline as well).

Launched rootfs build for testing: https://github.com/kernelci/kernelci-core/actions/runs/9775413201

laura-nao commented 3 months ago

@JenySadadia I opened https://github.com/kernelci/kernelci-pipeline/pull/675 to test the changes on staging. I'll check the results in a while.

laura-nao commented 2 months ago

Seems to be working fine on staging - see e.g.: https://staging.kernelci.org:9000/viewer?node_id=668b02578e761c46b24e6b11

I think this can be merged @JenySadadia , then we can build the definitive version of the rootfs for both staging and production.