falcosecurity / libs

libsinsp, libscap, the kernel module driver, and the eBPF driver sources
https://falcosecurity.github.io/libs/
Apache License 2.0
227 stars 162 forks source link

[PROPOSAL] Remove udig engine support #1368

Closed Andreagit97 closed 10 months ago

Andreagit97 commented 1 year ago

Motivation

In the last maintainer call, we discussed whether it is the case to remove the udig engine from libscap. The main reason behind this proposal is that today the udig engine is not maintained and broken and unfortunately we have no manpower to maintain it...Removing it will clean and reduce the size of our codebase so I'm in favor of this change.

Feature

Remove the udig engine from libscap

WDYT? @falcosecurity/libs-maintainers

You can vote as always with a +1 or -1

This issue is not just for maintainers but also for all adopters to jump in with concerns or blockers, even if as I said udig is broken so I don't think we have adopters for it

FedeDP commented 1 year ago

+1 from me! Removing dead code is always good to lower maintenance cost.

gnosek commented 1 year ago

+1. We can always bring it back (as a first class citizen, unlike now) if there's community interest

incertum commented 1 year ago

+1 and thanks for driving this!

leogr commented 1 year ago

+1

hbrueckner commented 1 year ago

+1 reducing technical debt is always great! Thanks for driving this!

incertum commented 10 months ago

@Andreagit97 keep for 0.14.0? Don't see a PR up yet or push to 0.15.0?

Andreagit97 commented 10 months ago

it's better to have it to avoid conflicts with the release branch in the future :/ this is the PR https://github.com/falcosecurity/libs/pull/1485 a big no-op