falcosecurity / evolution

Evolution process of The Falco Project
Apache License 2.0
48 stars 37 forks source link

Donate maxgio92/peribolos-syncer #269

Closed maxgio92 closed 1 year ago

maxgio92 commented 1 year ago

Repository: https://github.com/maxgio92/peribolos-syncer

Motivation

There are two declarative sources of truth about people of the GitHub organization:

the first honored by GitHub access control, and the latter by Prow automation's access control (like through the verify-owners plugin).

The tool can be used in CI to trigger, after the merge of PRs which bring changes to the owners, synchronization of the Peribolos configuration, via pull request.

Additional context

leogr commented 1 year ago

I love it!

+1 from me

maxgio92 commented 1 year ago

cc @falcosecurity/core-maintainers WDYT?

If approved I'm going to propose the repository settings, thanks.

cpanato commented 1 year ago

+1

incertum commented 1 year ago

Thanks a lot @maxgio92 ❤️ !

+1

maxgio92 commented 1 year ago

Thank you all :-)

maxgio92 commented 1 year ago

If anyone would like to help let me know

cpanato commented 1 year ago

If anyone would like to help let me know

let me know what you need :)

maxgio92 commented 1 year ago

Thanks @cpanato! Right now the tool is pretty simple and it's doing the work, but maintainers are more than welcome :-)

maxgio92 commented 1 year ago

Oh, obviously the code and code coverage have space for improvement!

FedeDP commented 1 year ago

Very nice and great work! +1 from me! Thank you Max!

Andreagit97 commented 1 year ago

great work! +1 from me!

jasondellaluce commented 1 year ago

That's great Max! +1

leogr commented 1 year ago

Hey @maxgio92

Are you ready to move it to Falcosecurity?

Let me know if you need any support! I will be happy to help

maxgio92 commented 1 year ago

Thank you all! ❤️

Yep @leogr :-) Thank you.

I just unlocked https://github.com/falcosecurity/test-infra/pull/1168 🚀

leogr commented 1 year ago

Repo trasfered https://github.com/falcosecurity/peribolos-syncer We will set up @poiana and the rest soon.

But, we first need maintainers for the project. Who volunteers? :smile_cat:

Ofc, count me in :)

maxgio92 commented 1 year ago

I'm going to propose the owners here: https://github.com/falcosecurity/peribolos-syncer/pull/5 It's work in progress, so anyone who wants to join can comment there :-)

leogr commented 1 year ago

Hey @maxgio92

Is this complete?

maxgio92 commented 1 year ago

Yep @leogr! 🥰 Thanks for noting this. I'm going to close it

maxgio92 commented 1 year ago

/close

poiana commented 1 year ago

@maxgio92: Closing this issue.

In response to [this](https://github.com/falcosecurity/evolution/issues/269#issuecomment-1622190500): >/close Instructions for interacting with me using PR comments are available [here](https://git.k8s.io/community/contributors/guide/pull-requests.md). If you have questions or suggestions related to my behavior, please file an issue against the [kubernetes/test-infra](https://github.com/kubernetes/test-infra/issues/new?title=Prow%20issue:) repository.