Skip to content

Register Edera zone plugin#1070

Merged
poiana merged 1 commit into
falcosecurity:mainfrom
bleggett:bleggett/add-edera-plugin
Nov 24, 2025
Merged

Register Edera zone plugin#1070
poiana merged 1 commit into
falcosecurity:mainfrom
bleggett:bleggett/add-edera-plugin

Conversation

@bleggett
Copy link
Copy Markdown
Contributor

What type of PR is this?
/kind documentation

Any specific area of the project related to this PR?
/area plugins
/area registry
/area documentation

What this PR does / why we need it:

Adds the Edera Falco plugin to the Falco plugin registry.

Docs

Special notes for your reviewer:

I've proactively assigned 26 as the plugin ID as it's the next in the list, let me know if it should change.

@poiana
Copy link
Copy Markdown
Contributor

poiana commented Nov 17, 2025

Welcome @bleggett! It looks like this is your first PR to falcosecurity/plugins 🎉

@bleggett
Copy link
Copy Markdown
Contributor Author

The registry.yaml checker seems to be having a GOVERSION mismatch issue

@bleggett
Copy link
Copy Markdown
Contributor Author

The registry.yaml checker seems to be having a GOVERSION mismatch issue

#1071 Should fix

@irozzo-1A
Copy link
Copy Markdown
Contributor

The registry.yaml checker seems to be having a GOVERSION mismatch issue

#1071 Should fix

@bleggett It's in, you can rebase 😉

@irozzo-1A irozzo-1A self-assigned this Nov 19, 2025
Signed-off-by: Benjamin Leggett <benjamin@edera.io>
@bleggett bleggett force-pushed the bleggett/add-edera-plugin branch from d22412a to 85cf7bb Compare November 20, 2025 16:29
@bleggett
Copy link
Copy Markdown
Contributor Author

@irozzo-1A rebased, ty!

@irozzo-1A irozzo-1A self-requested a review November 24, 2025 08:46
Copy link
Copy Markdown
Contributor

@irozzo-1A irozzo-1A left a comment

Choose a reason for hiding this comment

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

/lgtm

@poiana poiana added the lgtm label Nov 24, 2025
@poiana
Copy link
Copy Markdown
Contributor

poiana commented Nov 24, 2025

LGTM label has been added.

DetailsGit tree hash: bbf72f26fa4ab1e1d56c9775c8ff60df73b42cd1

Copy link
Copy Markdown
Contributor

@ekoops ekoops 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
Copy Markdown
Contributor

poiana commented Nov 24, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: bleggett, ekoops, irozzo-1A

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

The pull request process is described here

Details Needs approval from an approver in each of these files:

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

@poiana poiana merged commit 237ece2 into falcosecurity:main Nov 24, 2025
12 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants