Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

workflows: add boot testing to CI builds #96

Merged
merged 1 commit into from
Dec 5, 2024
Merged

workflows: add boot testing to CI builds #96

merged 1 commit into from
Dec 5, 2024

Conversation

mwasilew
Copy link
Contributor

No description provided.

ndechesne
ndechesne reviewed Nov 29, 2024
View reviewed changes
.github/workflows/build-yocto.yml Show resolved Hide resolved
.github/workflows/build-yocto.yml Show resolved Hide resolved
.github/workflows/build-yocto.yml Show resolved Hide resolved
ci/lava/qcs6490-rb3gen2-core-kit.yml Outdated Show resolved Hide resolved
@mwasilew mwasilew force-pushed the nightly-boot-test branch 10 times, most recently from 879c0d5 to ed2785a Compare December 2, 2024 17:01
@mwasilew mwasilew changed the title workflows: add boot testing to nightly builds workflows: add boot testing to CI builds Dec 2, 2024
@mwasilew
workflows: add boot testing to CI builds
b47de44 
Boot testing is disabled for PR builds because of security reasons.

Signed-off-by: Milosz Wasilewski <[email protected]>
@mwasilew
Copy link
Contributor Author

mwasilew commented Dec 3, 2024

I have a working version now. Example workflow here: https://github.com/quic-yocto/meta-qcom-hwe/actions/runs/12140562626/job/33851381293
The build still doesn't boot, that's why the boot test job fails.

I'll delete the nightly-boot-test branch once this PR is merged.

@quic-vkraleti quic-vkraleti merged commit c0e01e3 into qualcomm-linux:main Dec 5, 2024
9 checks passed
@mwasilew
Copy link
Contributor Author

mwasilew commented Dec 5, 2024

Thanks for merging this PR.
Right now there is just one board in the LAB that works. I'm trying to fix the problem, but it may take some time.

@ndechesne
Copy link
Contributor

hmm. why did we merge a workflow which we know does not work (yet)? that seems counter intuitive to me.

@mwasilew
Copy link
Contributor Author

mwasilew commented Dec 5, 2024

@ndechesne why do you say it doesn't work?

@ndechesne
Copy link
Contributor

I noticed the last push to main failed, but the failure is the compile job, and got confused. However I am not able to find a boot-test that completed successfully. Do you expect the boot-test to pass right now?

@mwasilew
Copy link
Contributor Author

mwasilew commented Dec 5, 2024

Yes, the problem was that 2 devices in the lab are effectively "bricked". I disabled them until there is a solution. Here is example boot test that worked: https://lava.infra.foundries.io/scheduler/job/56965.

@mwasilew mwasilew mentioned this pull request Dec 9, 2024
Open
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ndechesne ndechesne ndechesne left review comments

@quic-vkraleti quic-vkraleti quic-vkraleti approved these changes

@ricardosalveti ricardosalveti ricardosalveti approved these changes

@sbanerjee-quic sbanerjee-quic Awaiting requested review from sbanerjee-quic sbanerjee-quic is a code owner

Assignees
No one assigned
Labels
None yet
Projects
Mainline Qualcomm Linux
Status: Done
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@mwasilew @ndechesne @ricardosalveti @quic-vkraleti