Skip to content

chore(monitor): change iov check for RHEL and newer kernels in system monitor#2583

Merged
Aryan-sharma11 merged 2 commits into
kubearmor:mainfrom
charankamal20:chore/add-iov-check-for-rhel
Jun 26, 2026
Merged

chore(monitor): change iov check for RHEL and newer kernels in system monitor#2583
Aryan-sharma11 merged 2 commits into
kubearmor:mainfrom
charankamal20:chore/add-iov-check-for-rhel

Conversation

@charankamal20

Copy link
Copy Markdown
Contributor

Purpose of PR?:
Add shadow struct for compatibility btwn RHEL and mainstream kernel. Referenced from enforcer.bpf.c

Fixes #

Does this PR introduce a breaking change?

If the changes in this PR are manually verified, list down the scenarios covered::

Additional information for reviewer? :
Mention if this PR is part of any design or a continuation of previous PRs

Checklist:

  • Bug fix. Fixes #
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • PR Title follows the convention of <type>(<scope>): <subject>
  • Commit has unit tests
  • Commit has integration tests

Comment thread KubeArmor/BPF/system_monitor.c
@charankamal20 charankamal20 force-pushed the chore/add-iov-check-for-rhel branch from 99570cf to 11f7363 Compare May 11, 2026 04:33
@soorya-knox soorya-knox requested a review from achrefbensaad May 15, 2026 06:23
@charankamal20 charankamal20 force-pushed the chore/add-iov-check-for-rhel branch from 11f7363 to d56884d Compare June 2, 2026 05:28
@charankamal20 charankamal20 self-assigned this Jun 2, 2026
@charankamal20 charankamal20 force-pushed the chore/add-iov-check-for-rhel branch from 392350a to 1b56151 Compare June 8, 2026 05:36
@Aryan-sharma11 Aryan-sharma11 self-requested a review June 9, 2026 03:47

@achrefbensaad achrefbensaad left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tested on centos and the code is working as expected

[root@kubearmor-pr-2583 ~]# uname -a
Linux kubearmor-pr-2583 5.14.0-701.el9.x86_64 #1 SMP PREEMPT_DYNAMIC Mon May 4 09:10:57 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
[root@kubearmor-pr-2583 ~]# cat /etc/os-release 
NAME="CentOS Stream"
VERSION="9"
ID="centos"
ID_LIKE="rhel fedora"
VERSION_ID="9"
PLATFORM_ID="platform:el9"
PRETTY_NAME="CentOS Stream 9"
ANSI_COLOR="0;31"
LOGO="fedora-logo-icon"
CPE_NAME="cpe:/o:centos:centos:9"
HOME_URL="https://centos.org/"
BUG_REPORT_URL="https://issues.redhat.com/"
REDHAT_SUPPORT_PRODUCT="Red Hat Enterprise Linux 9"
REDHAT_SUPPORT_PRODUCT_VERSION="CentOS Stream"
[root@kubearmor-pr-2583 ~]# 

Latest KubeArmor fails
image
image

This Change fixes the issue

@charankamal20 please do the following:

  • Rebase this PR
  • Handle DCO

We can merge after that

@charankamal20 charankamal20 force-pushed the chore/add-iov-check-for-rhel branch from c25a64d to 09bdadc Compare June 12, 2026 09:48
@charankamal20

Copy link
Copy Markdown
Contributor Author

Tested on centos and the code is working as expected

[root@kubearmor-pr-2583 ~]# uname -a
Linux kubearmor-pr-2583 5.14.0-701.el9.x86_64 #1 SMP PREEMPT_DYNAMIC Mon May 4 09:10:57 UTC 2026 x86_64 x86_64 x86_64 GNU/Linux
[root@kubearmor-pr-2583 ~]# cat /etc/os-release 
NAME="CentOS Stream"
VERSION="9"
ID="centos"
ID_LIKE="rhel fedora"
VERSION_ID="9"
PLATFORM_ID="platform:el9"
PRETTY_NAME="CentOS Stream 9"
ANSI_COLOR="0;31"
LOGO="fedora-logo-icon"
CPE_NAME="cpe:/o:centos:centos:9"
HOME_URL="https://centos.org/"
BUG_REPORT_URL="https://issues.redhat.com/"
REDHAT_SUPPORT_PRODUCT="Red Hat Enterprise Linux 9"
REDHAT_SUPPORT_PRODUCT_VERSION="CentOS Stream"
[root@kubearmor-pr-2583 ~]# 

Latest KubeArmor fails image image

This Change fixes the issue

@charankamal20 please do the following:

* Rebase this PR

* Handle DCO

We can merge after that

@achrefbensaad rebased the PR.

@achrefbensaad achrefbensaad left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@Aryan-sharma11 Aryan-sharma11 force-pushed the chore/add-iov-check-for-rhel branch from 09bdadc to c4865de Compare June 15, 2026 10:34
@charankamal20 charankamal20 force-pushed the chore/add-iov-check-for-rhel branch from c4865de to 683e4b3 Compare June 16, 2026 06:21
@Aryan-sharma11 Aryan-sharma11 force-pushed the chore/add-iov-check-for-rhel branch from 683e4b3 to 7ba6dab Compare June 26, 2026 04:17
@Aryan-sharma11 Aryan-sharma11 merged commit 25519c1 into kubearmor:main Jun 26, 2026
18 of 19 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants