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

Streamline our CI and testing setup #2587

Open
YJDoc2 opened this issue Dec 22, 2023 · 0 comments
Open

Streamline our CI and testing setup #2587

YJDoc2 opened this issue Dec 22, 2023 · 0 comments

Comments

@YJDoc2
Copy link
Collaborator

YJDoc2 commented Dec 22, 2023

After merge of #2536 , we now have build and testing for x86,aarch64 , libc and gnu with various combinations. These roughly launch 27 CI runs, as can be seen in the commit of #2584 , which are a lot of checks unless we are ready to merge.
For example, in #2585 , which change docs, but still runs all PRs.

Even with adding conditional checks to run the build/test CI only on code changes, 27 CI are a lot to run on each commit in PR unless we are ready to merge (i.e. code is reviewed and changes are done etc.) . What I have in mind is to have a label based trigger, so when a specific label (eg. run-tests) is added to PR, only then the checks are run. (We'll also have to consider dependabot PRs for automatic merges). We can keep some basic checks to run on each commit such as linting and formatting, maybe a single build for x86+gnu or such, so that we know the commit is at least compiling correctly.

More suggestions are welcome!

cc : @containers/youki-maintainers 👀

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

1 participant