Skip to content

Conversation

@irozzo-1A
Copy link
Contributor

What type of PR is this?

Uncomment one (or more) /kind <> lines:

/kind bug

/kind cleanup

/kind design

/kind documentation

/kind failing-test

/kind feature

/kind release

Any specific area of the project related to this PR?

Uncomment one (or more) /area <> lines:

/area build

/area engine

/area tests

/area proposals

/area CI

What this PR does / why we need it:

Emit a warning when a rule uses the deprecated "evt.dir" field in output.

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

feat(engine): emit warning when a rule containing the `evt.dir` field in output is encountered

Emit a warning when a rule uses the deprecated "evt.dir" field in output.

Signed-off-by: Iacopo Rozzo <[email protected]>
@github-actions
Copy link

This PR may bring feature or behavior changes in the Falco engine and may require the engine version to be bumped.

Please double check userspace/engine/falco_engine_version.h file. See versioning for FALCO_ENGINE_VERSION.

/hold

@irozzo-1A
Copy link
Contributor Author

ref: #3690

@poiana
Copy link
Contributor

poiana commented Oct 13, 2025

LGTM label has been added.

Git tree hash: 820d5f8c3f1036333002106bcb23dd81faa29d2a

@leogr
Copy link
Member

leogr commented Oct 13, 2025

This PR may bring feature or behavior changes in the Falco engine and may require the engine version to be bumped.

Please double check userspace/engine/falco_engine_version.h file. See versioning for FALCO_ENGINE_VERSION.

/hold

false positive
/hold cancel

@irozzo-1A irozzo-1A marked this pull request as ready for review October 13, 2025 16:24
@poiana poiana requested a review from LucaGuerra October 13, 2025 16:24
@ekoops
Copy link
Contributor

ekoops commented Oct 14, 2025

/milestone 0.42.0

@poiana poiana added this to the 0.42.0 milestone Oct 14, 2025
Copy link
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

@github-project-automation github-project-automation bot moved this from Todo to In progress in Falco Roadmap Oct 14, 2025
@poiana
Copy link
Contributor

poiana commented Oct 14, 2025

[APPROVALNOTIFIER] This PR is APPROVED

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

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:

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 1717a98 into falcosecurity:master Oct 14, 2025
34 checks passed
@github-project-automation github-project-automation bot moved this from In progress to Done in Falco Roadmap Oct 14, 2025
@leogr leogr mentioned this pull request Oct 14, 2025
89 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

5 participants