Skip to content

Conversation

@amotl
Copy link
Contributor

@amotl amotl commented Dec 3, 2022

Dear Giorgio,

this is another improvement to the CI configuration, in order to optimize resource usage. I hope you like it.

With kind regards,
Andreas.

On subsequent invocations of the test suite on CI, when no dependencies
have been changed, the process can make full use of the package cache,
thus speeding up test runs, and skipping redundant downloads.

When any dependency file changes, the cache will be busted.
@amotl amotl mentioned this pull request Dec 3, 2022
- When pushing to the same branch on a PR, cancel running jobs of the
  same kind.
- Allow running the job manually from the UI, using `workflow_dispatch`.
@sonarqubecloud
Copy link

sonarqubecloud bot commented Dec 3, 2022

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@mindflayer mindflayer merged commit 5313158 into mindflayer:master Dec 3, 2022
@mindflayer
Copy link
Owner

Awesome, thank you!

@amotl
Copy link
Contributor Author

amotl commented Dec 3, 2022

At [1], it now says "pip cache is not found", and at [2], you see that 13 MB worth of cache content have been saved. You may want to manually re-run the same workflow now, to see that the cache is properly re-used.

[1] https://github.com/mindflayer/python-mocket/actions/runs/3610529308/jobs/6084381221#step:3:15
[2] https://github.com/mindflayer/python-mocket/actions/runs/3610529308/jobs/6084381221#step:13:4

@amotl
Copy link
Contributor Author

amotl commented Dec 3, 2022

Resource usage

Cold cache: https://github.com/mindflayer/python-mocket/actions/runs/3610529308/usage
Warm cache: https://github.com/mindflayer/python-mocket/actions/runs/3610554332/usage

Thoughts

Almost all jobs show speedups, but the PyPy job experiences a significant slowdown. I hope it's only a fluke. Let's monitor this.

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.

2 participants