[8.x](backport #6485) [k8s] fix hints stream missing ids #6487
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
![mergify[bot]](https://avatars.githubusercontent.com/in/10562?s=60&v=4){kind=small}
* fix: add required ids in hints input streams of type filestream * feat: disable hints default_container_logs when kubernetes integration container logs is enabled (cherry picked from commit 7ad0456)
mergify
bot
requested review from
pkoutsovasilis and
michel-laterman
and removed request for
a team
|
The beats change was breaking and will only go into 9.0.0, but I think we should fix this in all the maintained branches in agent anyway. |
swiatekm
approved these changes
Jan 7, 2025
|
pkoutsovasilis
approved these changes
Jan 7, 2025
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
What does this PR do?
After this PR got merged our k8s CI test started failing. After investigating I realised that all the hint-based inputs are lacking streams IDs in the respective filestream inputs resulting in all of them registering with an empty ID and thus colliding. Also, during this investigation, I realised that by default when hints are enabled are capturing the logs of containers that do not feature a hints-related annotation. Thus I updated the Helm chart to explicitly disable the former when both the contailer logs of kubernetes integration and hints are enabled.
A fix should also happen in the actual job that fabricates and introduces these templates
Why is it important?
Because it fixes an actual bug and it brings the CI back to green
Checklist
Disruptive User Impact
None
How to test this PR locally
Related issues
N/A
This is an automatic backport of pull request #6485 done by [Mergify](https://mergify.com).