Skip to content

feat(bpf-lsm): add tty information to telemetry #2029

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

feat(bpf-lsm): add tty information to telemetry #2029

wants to merge 1 commit into from

Conversation

Ayush9026
Copy link

@Ayush9026 Ayush9026 commented Apr 13, 2025 •
edited
Loading

Purpose of PR?:

Fixes #2022

  • Added tty struct handling to BPF-LSM telemetry
  • Matches existing system monitor functionality
  • Modified shared.h to include tty information

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

@Ayush9026 Ayush9026 force-pushed the feat/2022-add-tty-info-bpf-lsm branch from 3f69bb5 to e488d4a Compare April 13, 2025 09:00
@Aryan-sharma11 Aryan-sharma11 requested review from Aryan-sharma11 and daemon1024 and removed request for Aryan-sharma11 May 12, 2025 04:02
@Aryan-sharma11 Aryan-sharma11 force-pushed the feat/2022-add-tty-info-bpf-lsm branch from e488d4a to 1ea1440 Compare May 12, 2025 04:02
@Aryan-sharma11
Copy link
Member

@Ayush9026, kindly share the telemetry output for the above implementation.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@daemon1024 daemon1024 Awaiting requested review from daemon1024

@Aryan-sharma11 Aryan-sharma11 Awaiting requested review from Aryan-sharma11

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Add tty information to bpf-lsm generated telemetry
2 participants
@Ayush9026 @Aryan-sharma11