-
Notifications
You must be signed in to change notification settings - Fork 19
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
Run CI jobs in container #1136
Comments
demisx
added
discussion
Topic for Discussion at a Community Call
tech debt
ci
Continuous integration
labels
Feb 24, 2023
@wilwade @saraswatpuneet I think this is the next logical step in Frequency CI evolution. Let me know what your thoughts are on this. |
Here is my plan of attack. So far we are running two types of executors:
|
The |
demisx
added a commit
that referenced
this issue
Mar 7, 2023
# Goal The goal of this PR is to move jobs to docker executor in the following workflows: 1. Verify PR Commit 2. Run Post PR Merge Actions Part of #1136
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently, we execute all jobs directly on self-hosted runners. This leads to a side effect where each runner has state left over from the previously executed job. We try to clean it up in the common "Reset Runner" action right now, but it has its own drawbacks:
In order to make CI process more reliable, I recommend considering running each job in a container, so each runner starts off with a clean state every single time. There is also another benefit of running in a container - we can create and use our own images with all preinstalled tools and libraries. Makes CI even faster.
Tasks:
HOME: /root
env var?root
torunner
userReferences:
The text was updated successfully, but these errors were encountered: