Skip to content

workflows: replace poky-altcfg with nodistro and fix lava-test-plans#1671

Open
quaresmajose wants to merge 2 commits intoqualcomm-linux:masterfrom
quaresmajose:poky-altcfg
Open

workflows: replace poky-altcfg with nodistro and fix lava-test-plans#1671
quaresmajose wants to merge 2 commits intoqualcomm-linux:masterfrom
quaresmajose:poky-altcfg

Conversation

@quaresmajose
Copy link
Contributor

We name some of the ci task with the distro name, so better to use the nodistro since we are no longer use the poky-altcfg.

@mwasilew
Copy link
Contributor

mwasilew commented Mar 3, 2026

@qcom-anilyada This PR would replace #1643. Is that OK with you? I'm still going to hold off until the S3 migration is complete to avoid possible issues with merging 2 changes at the same time

ricardosalveti
ricardosalveti previously approved these changes Mar 3, 2026
@quaresmajose
Copy link
Contributor Author

@qcom-anilyada This PR would replace #1643. Is that OK with you? I'm still going to hold off until the S3 migration is complete to avoid possible issues with merging 2 changes at the same time

I can also hold on and wait for the #1643.

@quaresmajose quaresmajose marked this pull request as draft March 3, 2026 18:48
@mwasilew
Copy link
Contributor

mwasilew commented Mar 3, 2026

#1643 is just some earlier commit in lava-test-plans. So this PR will contain it.

@ricardosalveti
Copy link
Contributor

Please rebase.

@qcom-anilyada
Copy link
Contributor

if this gets merged, I can close PR#1643. This will have the updated reference that will include the changes in PR#1643

@mwasilew
Copy link
Contributor

mwasilew commented Mar 4, 2026

@mwasilew
Copy link
Contributor

mwasilew commented Mar 4, 2026

@quaresmajose bump of the lava-test-plans ref will cause quite a few tests jobs to fail. I'm not sure how to deal with the situation since newer version is required

@quaresmajose
Copy link
Contributor Author

@quaresmajose bump of the lava-test-plans ref will cause quite a few tests jobs to fail. I'm not sure how to deal with the situation since newer version is required

This isn't critical and we can wait a little while. Lets merge #1643 first which includes fewer changes, we can also revert qualcomm-linux/lava-test-plans#28 if that helps.

@mwasilew
Copy link
Contributor

mwasilew commented Mar 4, 2026

#1643 suffers from the same issues. They're not related to "nodistro" change, that's the problem.

@mwasilew
Copy link
Contributor

mwasilew commented Mar 6, 2026

@quaresmajose could you use this commit from lava-test-plans d5b19bcc4d038a40849fe3de0847d4277d5c1305? I added proper tags there.

Changelog:

d5b19bc Merge pull request qualcomm-linux#29 from mwasilew/add-bt-tag
e11b9f7 testcases: add has-bt tag to pre-merge-bt test
61d4549 Merge pull request qualcomm-linux#28 from quaresmajose/nodistro
fc85c0f testplans: replace poky-altcfg with nodistro
1d9ed81 Merge pull request qualcomm-linux#26 from mwasilew/remove-unused-tests
08d74c4 Remove "metadata" from master template
3300dd6 Remove unused tests
5e2441f Merge pull request qualcomm-linux#22 from qcom-anilyada/test/add-graphics-display
6ef3323 testcases: Add display-gfx test plan
2d52e81 testcases: Refactor BT suite to use loop-based test definitions
8a9beea Merge pull request qualcomm-linux#15 from qcom-anilyada/feature/optimizing-job-time
06f7c0f Merge pull request qualcomm-linux#21 from qcom-anilyada/test/add_testcase
043ac29 testcases: Add fastrpc_test to pre-merge-basic suite
d010bff testcases: Reduce timeout values for faster test feedback
960ba04 Merge pull request qualcomm-linux#24 from mattface/db410c-fastboot-timeout
74a30b3 Merge pull request qualcomm-linux#17 from mwasilew/expected-tests
d74c066 Allow target_deploy_timeout to be overridden and increase it for db410c
1859f1b pre-merge: add list of expected tests
6f79a24 Merge pull request qualcomm-linux#16 from qcom-anilyada/test/iq9075evk
af42fe7 devices: iq-9075-evk: exclude AudioRecord test

Signed-off-by: Jose Quaresma <jose.quaresma@oss.qualcomm.com>
We name some of the ci task with the distro name, so better
to use the nodistro since we are no longer use the poky-altcfg.

Signed-off-by: Jose Quaresma <jose.quaresma@oss.qualcomm.com>
@quaresmajose
Copy link
Contributor Author

@quaresmajose could you use this commit from lava-test-plans d5b19bcc4d038a40849fe3de0847d4277d5c1305? I added proper tags there.

Done. Thanks

@quaresmajose quaresmajose marked this pull request as ready for review March 6, 2026 14:29
@mwasilew
Copy link
Contributor

mwasilew commented Mar 6, 2026

https://github.com/qualcomm-linux/meta-qcom/actions/runs/22763259673/job/66042232981
There are some failures. @rsalveti @lumag either we merge it like this and point out the issues to the test team (and potentially kernel team) or we need to come up with some other plan. I propose to merge it. It will affect the PRs as the tests will now fail with unrelated issues.

Copy link
Contributor

@lumag lumag left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

NAK from my side. We have enough failing test runs. I defeinitely don't wan't to have the need to distinguish "this test run failed expectedly, while this test run failed on the non-expected failure",

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants