-
Notifications
You must be signed in to change notification settings - Fork 4.9k
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
[Autodiscovery] Ignore ErrInputNotFinished errors in autodiscover config checks #20305
[Autodiscovery] Ignore ErrInputNotFinished errors in autodiscover config checks #20305
Conversation
Signed-off-by: chrismark <chrismarkou92@gmail.com>
Pinging @elastic/integrations-platforms (Team:Platforms) |
Signed-off-by: chrismark <chrismarkou92@gmail.com>
ErrInputNotFinished
errors in autodiscover config checks
ErrInputNotFinished
errors in autodiscover config checks
💔 Tests FailedExpand to view the summary
Build stats
Test stats 🧪
Test errorsExpand to view the tests failures
Steps errorsExpand to view the steps failures
Log outputExpand to view the last 100 lines of log output
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for tackling this issue! It'd be good to have a test to cover this case. But we can leave it for a future PR.
Signed-off-by: chrismark <chrismarkou92@gmail.com>
Signed-off-by: chrismark <chrismarkou92@gmail.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Change looks great to me, minor comment on the placement of the errors.go. I don't think we will be reusing this, so probably input/file is a better place
Signed-off-by: chrismark <chrismarkou92@gmail.com>
…fig checks (elastic#20305) (cherry picked from commit 7a34167)
…fig checks (elastic#20305) (cherry picked from commit 7a34167)
…fig checks (elastic#20305) (cherry picked from commit 7a34167)
… errors in autodiscover config checks (#20338)
…ne-2.0 * upstream/master: (29 commits) Add an explicit system test for processes on unix systems (elastic#20320) [Autodiscovery] Ignore ErrInputNotFinished errors in autodiscover config checks (elastic#20305) [CI] Update README.md with CI references (elastic#20316) Add ECK doc links to Heartbeat docs (elastic#20284) [Filebeat] Add export tests to x-pack/filebeat (elastic#20156) feat(ci): support building docker images for PRs (elastic#20323) Update system tests dependency (elastic#20287) [Libbeat] Log debug message if the Kibana dashboard can not be imported from the archive (elastic#12211) (elastic#20150) [Filebeat][Gsuite] Transform all dates to timestamp with processor (elastic#20308) Infer types in Prometheus remote_write (elastic#19944) Remove unnecessary restarts of metricsets while using Node autodiscover (elastic#19974) docs: update changelog on master branch (elastic#20259) feat(ci): support storing artifacts for PRs in separate dirs (elastic#20282) [CI] Change upstream reference (elastic#20296) [Filebeat] Updates to Suricata module (elastic#20220) [docs] Fix Windows download link for agent (elastic#20258) [docs] Rename release highlights to what's new (elastic#20255) fix: update the display name of the multibranch job (elastic#20265) [Elastic Agent] Add basic protocol to control Elastic Agent. (elastic#20146) Cisco ASA: Fix message 106100 (elastic#20245) ...
…inished errors in autodiscover config checks (elastic#20338)
…inished errors in autodiscover config checks (elastic#20339) * [Autodiscovery] Ignore ErrInputNotFinished errors in autodiscover config checks (elastic#20305) (cherry picked from commit 8d2bc1f) * Update CHANGELOG.next.asciidoc
What does this PR do?
This PR ignores
ErrInputNotFinished
error occur in autodiscover stop/start process. This is required in order to avoid having stoped configs that never come back since the start event fails due to this error on first attempt if the previous state is not cleaned yet.Currently configs that fail due to
ErrInputNotFinished
are skipped atbeats/libbeat/autodiscover/autodiscover.go
Line 212 in fb52d26
beats/libbeat/autodiscover/autodiscover.go
Line 223 in fb52d26
beats/libbeat/autodiscover/autodiscover.go
Line 164 in fb52d26
Why is it important?
In order to resolve a permanent issue with updated Pods, which makes Filebeat stop collecting logs after a Pod is updated.
How to test this PR locally
team: qa
kubectl apply -f <manifest_filename>.yml