-
Notifications
You must be signed in to change notification settings - Fork 5.9k
fix(ci): add step to remove docker images on linux-arm64 #2804
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
Conversation
f2c2706
to
d947579
Compare
.github/workflows/ci.yaml
Outdated
- name: Prune docker images | ||
run: docker system prune -af |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'll remove this after it runs once. I think it just needs to do a quick cleanup and then in general, it should be enough to run it at the end!
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
oh makes sense!
This adds a step to remove both the release packages and the test artifacts as a step at the end of test in ci.yaml.
This adds a step to remove the docker images and hopefully free up space in the linux-arm64 part of ci.
049dee3
to
90807e1
Compare
This PR fixes #2480 .
linux-arm64
ci job (thanks to @code-asher for the tip, @vapurrmaid for the refactor!)test
jobtest:standalone-release
I suggest changing the extension away from the Python because it's notorious for breaking in code-server. A theme extension changes less frequently and is much more reliable for testing.