Skip to content

[elasticsearch] Add timeout configuration support #11479

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 4 commits into from
Oct 21, 2024
Merged

[elasticsearch] Add timeout configuration support #11479

merged 4 commits into from
Oct 21, 2024

Conversation

consulthys
Copy link
Contributor

@consulthys consulthys commented Oct 21, 2024 •
edited
Loading

Proposed commit message

The timeout option is available when configuring the Metricbeat elasticsearch module. This PR adds the possibility of configuring the timeout option for the corresponding elasticsearch integration.

Checklist

  • I have added an entry to my package's changelog.yml file.
  • I have verified that Kibana version constraints are current according to guidelines.

Related issues

Closes #11467

@consulthys consulthys added Integration:elasticsearch Elasticsearch Team:Stack Monitoring Stack Monitoring team [elastic/stack-monitoring] labels Oct 21, 2024
@consulthys consulthys self-assigned this Oct 21, 2024
@consulthys consulthys requested a review from a team as a code owner October 21, 2024 11:21
@consulthys consulthys enabled auto-merge (squash) October 21, 2024 11:25
@andrewkroh andrewkroh added the enhancement New feature or request label Oct 21, 2024
sachinfrayne
sachinfrayne approved these changes Oct 21, 2024
View reviewed changes
Copy link

@sachinfrayne sachinfrayne left a comment

Choose a reason for hiding this comment

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

Other than my understanding of why you need text for a timeout, this LGTM!

@consulthys
Copy link
Contributor Author

/test

@elastic-sonarqube Elastic SonarQube
Copy link

Quality Gate passed Quality Gate passed

Issues
0 New issues
0 Fixed issues
0 Accepted issues

Measures
0 Security Hotspots
100.0% Coverage on New Code
0.0% Duplication on New Code

See analysis details on SonarQube

@elastic-vault-github-plugin-prod
Copy link

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@consulthys consulthys merged commit d592efa into main Oct 21, 2024
5 checks passed
@elasticmachine
Copy link

💚 Build Succeeded

History

cc @consulthys

@consulthys consulthys deleted the fix-stack-mon-es-timeout branch October 21, 2024 13:43
@elastic-vault-github-plugin-prod
Copy link

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

harnish-crest-data pushed a commit to chavdaharnish/integrations that referenced this pull request Feb 4, 2025
@consulthys
[elasticsearch] Add timeout configuration support (elastic#11479)
b948364 
Add timeout configuration support
harnish-crest-data pushed a commit to chavdaharnish/integrations that referenced this pull request Feb 5, 2025
@consulthys
[elasticsearch] Add timeout configuration support (elastic#11479)
282eeed 
Add timeout configuration support
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@sachinfrayne sachinfrayne sachinfrayne approved these changes

Assignees

@consulthys consulthys

Labels
enhancement New feature or request Integration:elasticsearch Elasticsearch Team:Stack Monitoring Stack Monitoring team [elastic/stack-monitoring]
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

[Elasticsearch integration] timeout option
4 participants
@consulthys @elasticmachine @sachinfrayne @andrewkroh