Closed ArneTR closed 4 months ago
Hi @ArneTR Thanks a lot for the update, this looks great !
About the PR: Yes I noticed this before, but I'm happy with the G-Actions summary print already (eg)
Just 1 question: About the machine-uuid
, I understand I can set this on a secret, but then it will be the same whatever the machine the G-Action is running on, is it expected ? what is the actual real use from Eco-CI of the machine-uuid
?
ty for the notes. I updated the PR
Also just a heads up on a new update we have done in the dashboard. I believe this might be interesting for you :)
https://github.com/green-coding-solutions/green-metrics-tool/pull/823
Hey @gmaze ,
we have massively slimmed our Eco-CI tool and wanted to bring you the lastest update via PR.
What I changed in your workflow
machine-uuid
to be set. I set again a secret as a pre-filled value. You will have to setsecrets.CARBONDB_MACHINE_UUID_PYTESTS_UPSTREAM
andsecrets.CARBONDB_MACHINE_UUID_PYTESTS
in your repo config. As usual: Just pick a random UUIDsend-data: true
, which was already by default before, but I thought it is nice to be explicit herestart-measurement
. I removed the superflous mentionsAlso we have done some improvements to the SCI calculation and are now showing more granular data.
In essence though the speed gain should be the biggest win. We went down from ~30 seconds to ~1-2 seconds of overhead for a typical 5-7 step workflow.
PR-Comments
I was not sure if you did not want the integration for PR-Comments or just did not know about it, but I added it as example to this PR. What it does is that it will show you the details of a workflow energy and cabon cost in the PR as comment. See screenshot attached. Please remove if you had that not activated explicitely
Using this feature requires "permissions -> pull_request -> write" to be set, which allows Eco-CI to write a comment in the PR. If you have second thoughts about the permissions you can find details here what it entails: https://docs.github.com/en/rest/authentication/permissions-required-for-github-apps?apiVersion=2022-11-28#repository-permissions-for-pull-requests
Hope you like the updated Eco-CI and happy for your feedback!!