Skip to content

[github-actions] Made it so tests also run on intel #32

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

Merged

Conversation

danbugs
Copy link
Contributor

@danbugs danbugs commented Nov 7, 2024

Note: I only modified the dep_rust matrix which will run the tests to assess differences between amd and intel. In the future, we might want to consider revisiting our matrix setup.

@ludfjig
Copy link
Contributor

ludfjig commented Nov 7, 2024

Could we add intel to benchmarking too? And include them in github release artifacts?

@simongdavies
Copy link
Contributor

@danbugs when this merges you will need to update the branch protection rules required checks as there are now new checks and some of the existing ones don't exist

@danbugs danbugs force-pushed the add-intel-runners-to-build-matrix branch from 1019713 to f060b76 Compare November 7, 2024 19:18
Copy link
Contributor

@ludfjig ludfjig left a comment

Choose a reason for hiding this comment

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

lgtm with some minor questions

@danbugs danbugs force-pushed the add-intel-runners-to-build-matrix branch 3 times, most recently from f060b76 to 7f3d83a Compare November 7, 2024 19:54
Copy link
Contributor

@ludfjig ludfjig left a comment

Choose a reason for hiding this comment

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

LGTM

@danbugs danbugs enabled auto-merge (rebase) November 7, 2024 21:01
auto-merge was automatically disabled November 7, 2024 21:03

Base branch requires signed commits

@danbugs danbugs force-pushed the add-intel-runners-to-build-matrix branch from 73b5512 to b0b9e5f Compare November 7, 2024 21:40
Note: I only modified the dep_rust matrix which will run the tests to assess differences between amd and intel. In the future, we might want to consider revisiting our matrix setup.

Signed-off-by: danbugs <[email protected]>
@danbugs danbugs force-pushed the add-intel-runners-to-build-matrix branch from b0b9e5f to 2a43eca Compare November 7, 2024 21:50
@danbugs danbugs enabled auto-merge (rebase) November 7, 2024 21:51
auto-merge was automatically disabled November 7, 2024 22:07

Base branch requires signed commits

@danbugs danbugs merged commit 3bf7baa into hyperlight-dev:main Nov 7, 2024
21 checks passed
@danbugs danbugs deleted the add-intel-runners-to-build-matrix branch November 7, 2024 22:25
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