Skip to content

Conversation

andsel
Copy link
Contributor

@andsel andsel commented Aug 1, 2025

Release notes

[rn:skip]

What does this PR do?

Removed agent field from collector, becuase not used

Why is it important/What is the impact to the user?

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 made corresponding change to the default configuration files (and/or docker env variables)
  • [ ] I have added tests that prove my fix is effective or that my feature works

Author's Checklist

  • [ ]

How to test this PR locally

https://buildkite.com/elastic/logstash-exhaustive-tests-pipeline/builds/2205

Related issues

@andsel andsel self-assigned this Aug 1, 2025
@github-actions
Copy link
Contributor

github-actions bot commented Aug 1, 2025

🤖 GitHub comments

Expand to view the GitHub comments

Just comment with:

  • run docs-build : Re-trigger the docs validation. (use unformatted text in the comment!)

@andsel andsel changed the title Removed agent field from collector Remove agent field from metric collector Aug 1, 2025
@mergify
Copy link
Contributor

mergify bot commented Aug 1, 2025

This pull request does not have a backport label. Could you fix it @andsel? 🙏
To fixup this pull request, you need to add the backport labels for the needed
branches, such as:

  • backport-8./d is the label to automatically backport to the 8./d branch. /d is the digit.
  • If no backport is necessary, please add the backport-skip label

@andsel andsel mentioned this pull request Aug 1, 2025
2 tasks
@andsel andsel force-pushed the fix/remove_unused_field_in_collector branch from c322178 to aaa187b Compare August 1, 2025 09:49
@elastic-sonarqube
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarQube

@andsel andsel marked this pull request as ready for review August 1, 2025 10:39
Copy link
Member

@jsvd jsvd left a comment

Choose a reason for hiding this comment

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

LGTM, this was initially introduced by 23dcf46, but logstash-core/lib/logstash/api/lib/app/command.rb no longer exists.

@andsel andsel merged commit f7dd12a into elastic:main Aug 4, 2025
12 checks passed
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