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

Patch for riscv64 - 8.5.1 #2720

Merged
merged 3 commits into from Jul 19, 2022
Merged

Patch for riscv64 - 8.5.1 #2720

merged 3 commits into from Jul 19, 2022

Conversation

ghost
Copy link

@ghost ghost commented Jul 18, 2022

Fix for falling GH action https://github.com/lf-edge/eve/runs/7363194652?check_suite_focus=true#step:13:78
Ver: EVE 8.5.1
Fix for GH action with Memory report

giggsoff added 3 commits July 18, 2022 13:25
We have problems with patches for our images and it would be nice to
have an information about problems before hitting them. So I add target
test-images-patches and call it in test target.

Signed-off-by: Petr Fedchenkov <giggsoff@gmail.com>
(cherry picked from commit b6f9eaf)
Ensure that all templates can be applied to rootfs

Signed-off-by: Petr Fedchenkov <giggsoff@gmail.com>
(cherry picked from commit 191c435)
We did not define matrix for report job and failing now. In general,
separate jobs runs on separate runners, so it looks useless to measure
options of another runner. Let remove separate report job and add
another report step running always (regardless of previous errors).

Signed-off-by: Petr Fedchenkov <giggsoff@gmail.com>
(cherry picked from commit 46887d3)
@ghost ghost requested review from eriknordmark and rvs as code owners July 18, 2022 11:28
Copy link
Contributor

@rvs rvs left a comment

Choose a reason for hiding this comment

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

LGTM

@eriknordmark eriknordmark merged commit b2c01d4 into lf-edge:8.5 Jul 19, 2022
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.

3 participants