Skip to content

ci: move tests/clippy to GH workflows #126

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

ci: move tests/clippy to GH workflows #126

wants to merge 1 commit into from

Conversation

dancrossnyc
Copy link
Contributor

No description provided.

@dancrossnyc dancrossnyc requested a review from daym April 29, 2025 16:37
@jclulow
Copy link

jclulow commented Apr 29, 2025

What's the goal here? I think we should continue to run the tests on Helios?

@dancrossnyc
Copy link
Contributor Author

Just slimming things down.

We exercise this on helios via buildomat through amd-host-image-bulider; it seems wasteful of limited buildomat resources to also do amd-efs specifically there (granted, this is pretty small), especially since there's nothing illumos specific in here.

@jclulow
Copy link

jclulow commented Apr 29, 2025

We're not constrained on the buildomat resources this is using, though, just on the bare metal machines that things like control plane integration tests use. I'd much rather we discover an issue here, rather than in the CI for another repository.

@dancrossnyc
Copy link
Contributor Author

We're not constrained on the buildomat resources this is using, though, just on the bare metal machines that things like control plane integration tests use. I'd much rather we discover an issue here, rather than in the CI for another repository.

I don't know what issues we would discover through buildomat that we wouldn't similarly discover in the GH workflow. This is just a library, and not one that exercises the system in a particularly weird way where we'd expect to see any helios-specific issues.

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.

2 participants