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.

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

ci: yocto-check-layer: run KAS with debug #91

Merged
merged 1 commit into from
Nov 26, 2024

Conversation

ndechesne
Copy link
Contributor

For some reasons, kas does not checkout what we expect. The lock yaml files contain the most recent commits from poky and meta-qcom, which is expected since we run kas dump --update, however the downstream kas commands checkout the latest commits in the local mirror, not what we set in the lock files.

Let's add some debug to observe what is happening during the build.

When using download-artifact action, the path attribute is the
destination path, and it was wrongly set to ci/*.lock.yaml. As a
result, the lock files were extracted in meta-qcom-hwe/ci/*.lock.yaml/
folder instead of meta-qcom-hwe/ci.

Signed-off-by: Nicolas Dechesne <[email protected]>
Copy link
Contributor

@quaresmajose quaresmajose left a comment

Choose a reason for hiding this comment

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

LGTM,
I have tried something like that in my first approach. but then I got confused after the review.
#36 (comment)

Thanks for fixing

Copy link
Contributor

@ricardosalveti ricardosalveti left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for fixing this.

@ndechesne ndechesne merged commit 626d56b into qualcomm-linux:main Nov 26, 2024
8 of 9 checks passed
@ndechesne ndechesne deleted the test-kas branch November 26, 2024 14:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

3 participants