search

kernelci / kernelci-core

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

config/runtime/tests/tast: add `setup` test set #2583

Closed JenySadadia closed 3 months ago

JenySadadia commented 3 months ago

Remove an individual setup test suite and add setup as test set inside tast test suite. This will cover tests like checking CrOS version (os-release) and creating tast tarball (tast-tarball).

JenySadadia commented 3 months ago

The new hierarchy for tast job will be:

graph TD
    A[tast-power-x86-intel] --> B(setup)
    A[tast-power-x86-intel] --> C(tast)
    C --> D[setup]
    D --> I[os-release]
    D --> J[tast-tarball]
    C -->  E[test1]
    C --> F[test2]
    B -->  G[login]
    B --> H[kernelmsg]

Tested OK on staging: https://staging.kernelci.org:9000/viewer?search=parent%3D667974a8799d51fba3e12951

JenySadadia commented 3 months ago

LGTM

Great work on visualizing changes in hierarchy - it significantly improves overall PR readability. It should definitely be added to test case/maestro developer docs 👍

Thank you :)

It should definitely be added to test case/maestro developer docs 👍

Yes, that's the next step.