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

Add additional GH hosted runners to the smoke tests #1279

Merged
merged 17 commits into from
Aug 3, 2023

Conversation

directionless
Copy link
Contributor

@directionless directionless commented Aug 2, 2023

Relates to: #1276

This adds additional runners to the Exec Test section, and expands it to use both launcher --version and launcher doctor.

Because it expands the smoke test targets, I've also pulled the Upload into it's own job. That way we don't run upload for each smoke test.

And to support doctor we also need to have osquery installed. So I've added a launcher subcommand to support that

James-Pickett
James-Pickett previously approved these changes Aug 2, 2023
@directionless directionless marked this pull request as ready for review August 3, 2023 03:03
James-Pickett
James-Pickett previously approved these changes Aug 3, 2023
Copy link
Contributor

@James-Pickett James-Pickett left a comment

Choose a reason for hiding this comment

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

Nice! I see the ubuntu Store Artifacts job is failing for some reason, once that is fixed, I think this is good.

.github/workflows/go.yml Outdated Show resolved Hide resolved
.github/workflows/go.yml Outdated Show resolved Hide resolved
RebeccaMahany
RebeccaMahany previously approved these changes Aug 3, 2023
@directionless directionless merged commit 97f81d9 into kolide:main Aug 3, 2023
24 checks passed
@directionless directionless deleted the seph/more-os branch August 3, 2023 15:04
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