Skip to content

Conversation

@kgeller
Copy link
Contributor

@kgeller kgeller commented Jan 12, 2024

Proposed commit message

Adding the ability for users to set custom yaml config for windows datastream winlog inputs.

Checklist

  • I have reviewed tips for building integrations and this pull request is aligned with them.
  • I have verified that all data streams collect metrics or logs.
  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.

Author's Checklist

  • [ ]

How to test this PR locally

Related issues

Screenshots

Screenshot 2024-01-12 at 3 33 28 PM

@kgeller kgeller added the enhancement New feature or request label Jan 12, 2024
@kgeller kgeller self-assigned this Jan 12, 2024
@elasticmachine
Copy link

elasticmachine commented Jan 12, 2024

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@kgeller kgeller marked this pull request as ready for review January 12, 2024 21:00
@kgeller kgeller requested review from a team as code owners January 12, 2024 21:00
@pierrehilbert pierrehilbert added the Team:Elastic-Agent Platform - Ingest - Agent [elastic/elastic-agent] label Jan 14, 2024
@elasticmachine
Copy link

Pinging @elastic/elastic-agent (Team:Elastic-Agent)

{{#if processors.length}}
{{processors}}
{{/if}}
{{custom}}
Copy link
Contributor

Choose a reason for hiding this comment

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

Final new lines where missing?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I guess so?

All I did was manually drop the {{custom}} on the last line of the files to match what we do in winlog (https://github.com/kgeller/integrations/blob/main/packages/winlog/agent/input/winlog.yml.hbs#L32)

@elasticmachine
Copy link

💚 Build Succeeded

cc @kgeller

@kgeller kgeller merged commit 3840a7f into elastic:main Jan 16, 2024
@kgeller kgeller deleted the windows-custom-configs branch January 16, 2024 15:38
@elasticmachine
Copy link

Package windows - 1.44.0 containing this change is available at https://epr.elastic.co/search?package=windows

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

Labels

enhancement New feature or request Integration:windows Windows Team:Elastic-Agent Platform - Ingest - Agent [elastic/elastic-agent]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants