update(cmake): bump libs and driver to latest master

[jasondellaluce]

What type of PR is this?

/kind cleanup

Any specific area of the project related to this PR?

/area build

/area tests

What this PR does / why we need it:

Just updating libs and fix tests to catch warnings with new output messages.

Which issue(s) this PR fixes:

Special notes for your reviewer:

Does this PR introduce a user-facing change?:

NONE

[jasondellaluce]

/milestone 0.39.0

[poiana]

LGTM label has been added.

Git tree hash: 61dc250820589c04df0699b569d851a423f0ba96

[LucaGuerra]

@jasondellaluce seems like this isn't building because of a missing metrics method, do you have a patch for it?

[LucaGuerra]

falcosecurity/libs#1971

[leogr]

👍

[poiana]

LGTM label has been added.

Git tree hash: fed86b5e1e4ed4295b6399ff14a8b9569cc84e49

[FedeDP]

Updated once again to bring in the changes from falcosecurity/libs#1960

[FedeDP]

We need to fix the windows CI that is complaining about zlib; most probably caused by this change: falcosecurity/libs#1940 or falcosecurity/libs#1899 :/

[FedeDP]

Incorporated the fix by @geraldcombs in the latest commit.

[FedeDP]

@Molter73 how is this building without your #3265 🤣

[FedeDP]

Failing test:

  TestFalco_Legacy_FalcoEventGenerator

    legacy_test.go:2688: 
        	Error Trace:	/home/runner/work/_actions/falcosecurity/testing/main/legacy_test.go:2688
        	Error:      	Should not be zero, but was 0
        	Test:       	TestFalco_Legacy_FalcoEventGenerator
    legacy_test.go:2691: 
        	Error Trace:	/home/runner/work/_actions/falcosecurity/testing/main/legacy_test.go:2691
        	Error:      	Not equal: 
        	            	expected: 1
        	            	actual  : 0
        	Test:       	TestFalco_Legacy_FalcoEventGenerator

and we also have 4 tests that are receiving heapbuffer overflow errors on x86:

        	            	==9743==ERROR: AddressSanitizer: heap-buffer-overflow on address 0x6020000ed340 at pc 0x000000b0c73e bp 0x7ffda518c300 sp 0x7ffda518c2f0

[Molter73]

@Molter73 how is this building without your #3265 🤣

I honestly have no idea, #3265 was failing to compile too, I checked.

I must have messed up a rebase or something before merging falcosecurity/libs#1920, the new_metric method is duplicated in 2 different classes. 🤦🏻‍♂️

https://github.com/falcosecurity/libs/blob/3ce0a2ddb41de2e5e446da0882f0aca836805c46/userspace/libsinsp/metrics_collector.h#L339-L350

https://github.com/falcosecurity/libs/blob/3ce0a2ddb41de2e5e446da0882f0aca836805c46/userspace/libsinsp/metrics_collector.h#L248-L259

[LucaGuerra]

@Molter73 after falcosecurity/libs#1971 it built just fine locally for me 🤔

[Molter73]

@Molter73 after falcosecurity/libs#1971 it built just fine locally for me 🤔

Yeah, I messed up and left in a method that I should've removed and now it's duplicated, I'll open a PR and clean it up.

I don't know if we want to rush this in though, since it's a minor cleanup, let me know what you guys think.

[FedeDP]

No problem @Molter73, take your time! I think it would be great to clean that up for libs 0.18.0 (by the end of august), but no hurry.

[LucaGuerra]

Opened an issue for the test failure: falcosecurity/libs#2018

[FedeDP]

Latest commit should fix CI; moreover it accounts for API breaks from:

• cleanup(sinsp)!: add set_static_container method libs#2016
• new(scap, sinsp)!: linux_hostinfo platform for use with non-syscall source plugins libs#1969

[FedeDP]

Tests passed!!! 🚀

[FedeDP]

/approve

[poiana]

LGTM label has been added.

Git tree hash: 6249f4fc5c369ef6bef06b92269e1d3dc05a571f

[poiana]

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: FedeDP, jasondellaluce

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 [FedeDP,jasondellaluce]

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

[LucaGuerra]

We did it! 🚀 😅