Skip to content
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

docs(proposal): new artifacts distribution proposal #2304

Merged
merged 3 commits into from
Dec 14, 2022

Conversation

leogr
Copy link
Member

@leogr leogr commented Nov 28, 2022

What type of PR is this?

/kind documentation

Any specific area of the project related to this PR?

/area proposals

What this PR does / why we need it:

This proposal aims to formalize the revamped way adopted by Falcosecurity to distribute artifacts. In fact, it reflects some topics widely discussed in the last few months, plus suggestions for some obvious action items to move forward.

Furthermore, most of the points reflect some already ongoing efforts, such as the revamp of falcoctl.

Any feedback is appreciated

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

/milestone 0.34.0

Does this PR introduce a user-facing change?:

NONE

@leogr leogr changed the title wip: docs(proposal): new artifacts distribution proposal docs(proposal): new artifacts distribution proposal Dec 5, 2022
@leogr
Copy link
Member Author

leogr commented Dec 5, 2022

Dear @falcosecurity/core-maintainers

As discussed during the last community call, here's my proposal :)
So, PTAL 🙏

🤗

Co-authored-by: Federico Di Pierro <[email protected]>
Signed-off-by: Leonardo Grasso <[email protected]>
Copy link
Member

@Andreagit97 Andreagit97 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/approve

@FedeDP
Copy link
Contributor

FedeDP commented Dec 13, 2022

/hold

Copy link
Contributor

@FedeDP FedeDP left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/approve

@poiana
Copy link
Contributor

poiana commented Dec 13, 2022

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Andreagit97, FedeDP, leogr

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Needs approval from an approver in each of these files:
  • OWNERS [Andreagit97,FedeDP,leogr]

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@leogr
Copy link
Member Author

leogr commented Dec 14, 2022

As discussed in the current community call, it seems to me that nobody disagrees with this proposal, and we would like to deliver some action items within Falco 0.34, so

/hold cancel

🥳

@poiana poiana merged commit 9c04622 into master Dec 14, 2022
@poiana poiana deleted the docs/proposal/new-artifacts-distribution branch December 14, 2022 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants