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

Suggestion: citgm-smoker-lite job for PRs #1023

Open
BethGriggs opened this issue Nov 7, 2023 · 6 comments
Open

Suggestion: citgm-smoker-lite job for PRs #1023

BethGriggs opened this issue Nov 7, 2023 · 6 comments

Comments

@BethGriggs
Copy link
Member

This was briefly discussed in a Release WG meeting. A similar concept to https://ci.nodejs.org/view/All/job/node-test-commit-lite/, we could add a CITGM 'lite' job that only covers one or two most critical platforms (probably would make sense to be rhel8-x64 and osx11).

Running across all platforms takes a lot of time to both run, and interpret triage the results. Having a lite job running means we could increase our usage of CITGM on PRs. The more PRs we run it on, the earlier and easier it is to catch regressions, and may be easier for the wider collaborator base to keep up with the state.

(Perhaps eventually even a citgm-smoker-lite-daily job could make sense to cover the state over time - if there are volunteers to look at it.)

Releasers would still run CITGM and triage on the full set of platforms as part of the release process, but in theory they'd have an easier job as a number of issues are likely to have been caught by the lite job.

@BethGriggs
Copy link
Member Author

Happy to volunteer to set this job up if I can be granted access to do so. It doesn't seem like it'd require much more than duplicating the job and removing the other platforms.

@RafaelGSS
Copy link
Member

Would you believe it should be a default workflow for all PRs or an opt-in such as request-ci label? I think the latter is reasonable.

@targos
Copy link
Member

targos commented Nov 14, 2023

@BethGriggs I created https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker-lite/ as a copy of citgm-smoker and gave you permissions to configure it.

@BethGriggs
Copy link
Member Author

Would you believe it should be a default workflow for all PRs or an opt-in such as request-ci label? I think the latter is reasonable.

Probably makes sense to start with opt-in until we see how often it's used and how often it stays passing. Are you thinking of a separate request-citgm label? I'd be +1 to that.

@RafaelGSS
Copy link
Member

Are you thinking of a separate request-citgm label?

Yes


Why not use GitHub Actions instead of a Jenkins setup as it should only run on Linux, OSX, and Windows?

@BethGriggs
Copy link
Member Author

Why not use GitHub Actions instead of a Jenkins setup as it should only run on Linux, OSX, and Windows?

I feel there's probably some value to running the CITGM lite job using the same environments and machines that we use to run CITGM during releases. Otherwise we (releasers) may spend time debugging failures between the two environments - perhaps negating some of the benefit of running the lite job? I can't really quantify how frequently that would happen, though. But, I do know we've seen CITGM runs impacted by environment/machine in the past.

I also find the per platform per module breakdown of test results in https://ci.nodejs.org/view/Node.js-citgm/job/citgm-smoker/3342/testReport/ quite useful - but maybe we could replicate that easily in actions using --markdown output option or similar.

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

No branches or pull requests

3 participants