Sample integration of Eco-CI to Falco Pipelines #3121
Conversation
|
Adding the "do-not-merge/release-note-label-needed" label because no release-note block was detected, please follow our release note process to remove it. Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository. |
|
Welcome @ArneTR! It looks like this is your first PR to falcosecurity/falco 🎉 |
There was a problem hiding this comment.
This is awesome, thank you!
Question: We created a dedicated repo here: https://github.com/falcosecurity/cncf-green-review-testing perhaps we should expand that repo to be "All Things Green Review"? However, we don't yet have any CI really setup in that repo.
CC @leogr @maxgio92 @LucaGuerra awaiting guidance on how we would like to expand our infra in this regards. Thanks.
adding @FedeDP |
|
Hi @ArneTR, which parts of the CI pipeline this measurement applies to? In any case generally I think that reports that improve the observability around the resource consumption like other reports per CI run I think is right to have them declared in the same repo where the tracked pipelines are declared @incertum. WDYT? |
|
I would vote for the other repo https://github.com/falcosecurity/cncf-green-review-testing as it may end up becoming many new pipelines. Aka keep this repos' CI pipelines scoped to our core falco release and test pipelines. Also the other repo would have faster dev and update cycles as it requires only one approve. |
|
Hey @maxgio92 I have posted the details in the linked issue, but I guess it would have been better to also post them here :) Here it goes:
Eco-CI is integrated here by encapsulating all the steps that are originally done in the Scorecard workflow. By design it cannot capture the boot up of the VM and / or the removal. Eco-CI only starts measuring as the first step of a Github / Gitlab workflow. A bit more details how it works you can find here: https://github.com/green-coding-solutions/eco-ci-energy-estimation I also would like to add that this is mainly a Draft PR to illustrate the concept and advance the discussin in the CNCF TAG ENV group. Happy to extend this PR to a full fledged one though or even move it rather to a different repor where you see a try-out integration of Eco-CI more applicable. |
|
/milestone TBD |
|
Issues go stale after 90d of inactivity. Mark the issue as fresh with Stale issues rot after an additional 30d of inactivity and eventually close. If this issue is safe to close now please do so with Provide feedback via https://github.com/falcosecurity/community. /lifecycle stale |
|
/remove-lifecycle stale |
|
Issues go stale after 90d of inactivity. Mark the issue as fresh with Stale issues rot after an additional 30d of inactivity and eventually close. If this issue is safe to close now please do so with Provide feedback via https://github.com/falcosecurity/community. /lifecycle stale |
|
Any news on this one? |
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: ArneTR The full list of commands accepted by this bot can be found here.
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
|
Oh my, seems like I missed to make the DCO. I rebased the branched and added it. Also updated Eco-CI and pinned the version, as you also do it with the other GitHub extensions you use. For questions please follow up. All description and infos I posted in the message from April are still current. |
Hey @ArneTR I guess your changes are out from the branch due to a wrong rebase (and for this reason, GitHub closed this PR). May you fix it or open a new PR pls? |
This is a draft PR to illustrate an integration of an energy / carbon measurement plugin. Discussed in cncf-tags/green-reviews-tooling#55
What type of PR is this?
/kind feature
Any specific area of the project related to this PR?
/area CI