Skip to content
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.

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

[receiver/journald] adds namespace support #36031

Merged

Conversation

fuero
Copy link
Contributor

@fuero fuero commented Oct 28, 2024

Description

Adds support for querying journald namespaces
See https://www.freedesktop.org/software/systemd/man/latest/systemd-journald.service.html#Journal%20Namespaces for details

Testing

Adds tests for correctly generating the journalctl command.

Tested on a real world example as well with the resulting binary

Documentation

Adds description of the setting the PR introduces

@fuero fuero force-pushed the feature/journald-namespace-support branch from 2faefa6 to 814dbfe Compare October 29, 2024 16:04
@djaglowski djaglowski merged commit 426e0a4 into open-telemetry:main Oct 29, 2024
158 checks passed
@github-actions github-actions bot added this to the next release milestone Oct 29, 2024
zzhlogin pushed a commit to zzhlogin/opentelemetry-collector-contrib-aws that referenced this pull request Nov 12, 2024
#### Description

Adds support for querying journald namespaces
See
https://www.freedesktop.org/software/systemd/man/latest/systemd-journald.service.html#Journal%20Namespaces
for details

#### Testing

Adds tests for correctly generating the `journalctl` command.

Tested on a real world example as well with the resulting binary

#### Documentation

Adds description of the setting the PR introduces
sbylica-splunk pushed a commit to sbylica-splunk/opentelemetry-collector-contrib that referenced this pull request Dec 17, 2024
#### Description

Adds support for querying journald namespaces
See
https://www.freedesktop.org/software/systemd/man/latest/systemd-journald.service.html#Journal%20Namespaces
for details

#### Testing

Adds tests for correctly generating the `journalctl` command.

Tested on a real world example as well with the resulting binary

#### Documentation

Adds description of the setting the PR introduces
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.

3 participants