Skip to content

chore(plugins/container): headers cleanup#888

Merged
poiana merged 3 commits intofalcosecurity:mainfrom
deepskyblue86:cleanup/container-headers
Jul 17, 2025
Merged

chore(plugins/container): headers cleanup#888
poiana merged 3 commits intofalcosecurity:mainfrom
deepskyblue86:cleanup/container-headers

Conversation

@deepskyblue86
Copy link
Member

What type of PR is this?

/kind cleanup

Any specific area of the project related to this PR?

/area plugins

What this PR does / why we need it:
Headers cleanup in the container plugin

Which issue(s) this PR fixes:

Fixes #

Special notes for your reviewer:

@github-actions
Copy link

Rules files suggestions

@poiana
Copy link
Contributor

poiana commented Jul 17, 2025

LGTM label has been added.

DetailsGit tree hash: 2619a7f45ead166c65c99ad917d01e6bc314077d

@deepskyblue86 deepskyblue86 force-pushed the cleanup/container-headers branch from b381337 to 33507ac Compare July 17, 2025 13:23
@poiana poiana removed the lgtm label Jul 17, 2025
@poiana poiana requested a review from gnosek July 17, 2025 13:23
@github-actions
Copy link

Rules files suggestions

Signed-off-by: Angelo Puglisi <angelopuglisi86@gmail.com>
Signed-off-by: Angelo Puglisi <angelopuglisi86@gmail.com>
m_container_ip and m_container_user are already exposed in table.cpp

Signed-off-by: Angelo Puglisi <angelopuglisi86@gmail.com>
@deepskyblue86 deepskyblue86 force-pushed the cleanup/container-headers branch from 33507ac to a75479c Compare July 17, 2025 13:44
Copy link
Contributor

@jasondellaluce jasondellaluce 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
Contributor

poiana commented Jul 17, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: deepskyblue86, gnosek, jasondellaluce

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
Copy link
Contributor

poiana commented Jul 17, 2025

LGTM label has been added.

DetailsGit tree hash: 344a4c8aeb4d821a2a339211f3ec82a954d06b65

@github-actions
Copy link

Rules files suggestions

@poiana poiana merged commit 6e02f91 into falcosecurity:main Jul 17, 2025
24 checks passed
@deepskyblue86 deepskyblue86 deleted the cleanup/container-headers branch July 18, 2025 07:46
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