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

[Filebeat] Add experimental to journald input docs, rename two fields #28065

Merged
merged 4 commits into from
Sep 28, 2021

Conversation

andrewkroh
Copy link
Member

What does this PR do?

Add the experimental tag to journald input docs. This matches the code which logs that input is experimental.

  • Also fix the regexp condition in the example config.
  • Rename field journald.process.capabilites to journald.process.capabilities to fix spelling.
  • Rename field log.syslog.facility.name to log.syslog.facility.code because the value is numeric rather than the facility name.

Why is it important?

Set expectations of stability for the input. There will be one config change coming in the future.

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

Mark the docs as experimental to match the code.

Also fix the regexp condition in the example config.
There was a typo in the word `capabilities` for `journald.process.capabilities`.

And the mapping for syslog facility should have been to `log.syslog.facility.code` because the
value is a number rather than a name.
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Sep 22, 2021
@elasticmachine
Copy link
Collaborator

Pinging @elastic/agent (Team:Agent)

@elasticmachine
Copy link
Collaborator

Pinging @elastic/security-external-integrations (Team:Security-External Integrations)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Sep 22, 2021
@elasticmachine
Copy link
Collaborator

elasticmachine commented Sep 22, 2021

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview preview

Expand to view the summary

Build stats

  • Start Time: 2021-09-22T16:35:11.430+0000

  • Duration: 82 min 3 sec

  • Commit: 99a1e91

Test stats 🧪

Test Results
Failed 0
Passed 7137
Skipped 1120
Total 8257

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 7137
Skipped 1120
Total 8257

🤖 GitHub comments

To re-run your PR in the CI, just comment with:

  • /test

  • jenkins run the tests

@andrewkroh andrewkroh changed the title [Filebeat] Add experimental to journald docs, rename two fields [Filebeat] Add experimental to journald input docs, rename two fields Sep 22, 2021
@andrewkroh andrewkroh merged commit fe84ec7 into elastic:master Sep 28, 2021
mergify bot pushed a commit that referenced this pull request Sep 28, 2021
…#28065)

* Add the experimental tag to journald input

Mark the docs as experimental to match the code.

Also fix the regexp condition in the example config.

* Fix journald field mappings

There was a typo in the word `capabilities` for `journald.process.capabilities`.

And the mapping for syslog facility should have been to `log.syslog.facility.code` because the
value is a number rather than a name.

(cherry picked from commit fe84ec7)

# Conflicts:
#	journalbeat/include/fields.go
andrewkroh added a commit that referenced this pull request Sep 28, 2021
…#28065)

* Add the experimental tag to journald input

Mark the docs as experimental to match the code.

Also fix the regexp condition in the example config.

* Fix journald field mappings

There was a typo in the word `capabilities` for `journald.process.capabilities`.

And the mapping for syslog facility should have been to `log.syslog.facility.code` because the
value is a number rather than a name.
andrewkroh added a commit that referenced this pull request Sep 30, 2021
…#28065) (#28160)

* Add the experimental tag to journald input

Mark the docs as experimental to match the code.

Also fix the regexp condition in the example config.

* Fix journald field mappings

There was a typo in the word `capabilities` for `journald.process.capabilities`.

And the mapping for syslog facility should have been to `log.syslog.facility.code` because the
value is a number rather than a name.

Co-authored-by: Andrew Kroh <andrew.kroh@elastic.co>
Icedroid pushed a commit to Icedroid/beats that referenced this pull request Nov 1, 2021
…elastic#28065)

* Add the experimental tag to journald input

Mark the docs as experimental to match the code.

Also fix the regexp condition in the example config.

* Fix journald field mappings

There was a typo in the word `capabilities` for `journald.process.capabilities`.

And the mapping for syslog facility should have been to `log.syslog.facility.code` because the
value is a number rather than a name.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport-v7.16.0 Automated backport with mergify breaking change Filebeat Filebeat Journalbeat Team:Elastic-Agent Label for the Agent team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants