plugins icon indicating copy to clipboard operation
plugins copied to clipboard

new(plugins/k8smeta): update k8smeta plugin to require plugin API version 3.9.0.

Open FedeDP opened this issue 9 months ago • 11 comments

What type of PR is this?

/kind cleanup /kind feature

Any specific area of the project related to this PR?

/area plugins

What this PR does / why we need it:

Implemented the "suggested output fields feature", by suggesting k8smeta.pod.name and k8smeta.ns.name as output fields.

Also, entirely avoid the proc scan, instead relying on the listening CAPability to initially loop over the thread table to attach pod_uid to threads. Following the above, hostProc initConfig key is now deprecated and unused.

Moved the plugin to 0.3.0 too (given the minimum plugin API version changes).

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

FedeDP avatar Feb 13 '25 09:02 FedeDP

Rules files suggestions

github-actions[bot] avatar Feb 13 '25 10:02 github-actions[bot]

Rules files suggestions

github-actions[bot] avatar Feb 13 '25 10:02 github-actions[bot]

Rules files suggestions

github-actions[bot] avatar Feb 13 '25 11:02 github-actions[bot]

Rules files suggestions

github-actions[bot] avatar Feb 13 '25 13:02 github-actions[bot]

/hold

FedeDP avatar Feb 13 '25 13:02 FedeDP

Rules files suggestions

github-actions[bot] avatar Feb 13 '25 13:02 github-actions[bot]

@leogr pushed changes to address your review! Thanks!

FedeDP avatar Feb 13 '25 14:02 FedeDP

Rules files suggestions

github-actions[bot] avatar Feb 13 '25 15:02 github-actions[bot]

Also, we might want to use the provided logger by the sdk, like i did here for the container plugin: https://github.com/FedeDP/container_plugin/pull/26

This would allow use to drop the spdlog dep, to have better unified logging, and to drop the verbosity init config key, since we would then use the verbosity used for libs logger (https://github.com/falcosecurity/libs/blob/3d1d4a930d3a16f3e72ab62876ba0837c02d2ec5/userspace/libsinsp/plugin.cpp#L79).

I will make the change in its own commit.

FedeDP avatar Feb 28 '25 08:02 FedeDP

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: 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:

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

poiana avatar Mar 10 '25 09:03 poiana

I will make the change in its own commit.

Won't do that here since it would be a bit too big. We can fix it later if needed.

FedeDP avatar Mar 20 '25 14:03 FedeDP

/unhold Let's go!

FedeDP avatar Apr 07 '25 08:04 FedeDP