Skip to content

ti_eclecticiq: Add CEL resource.tracer limit - #10232

Merged
kcreddy merged 2 commits into
elastic:mainfrom
kcreddy:eclectiq
Jun 25, 2024
Merged

ti_eclecticiq: Add CEL resource.tracer limit#10232
kcreddy merged 2 commits into
elastic:mainfrom
kcreddy:eclectiq

Conversation

@kcreddy

@kcreddy kcreddy commented Jun 25, 2024

Copy link
Copy Markdown
Contributor

Proposed commit message

Add CEL resource.tracer limit.

We have seen cases where the response size exceeds the default 
`resource.tracer.maxsize` 1 MB and responses are dropped. 
Add `resource.tracer.maxsize` and `resource.tracer.maxbackups` 
to capture large responses and avoid loss of visibility.

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.

How to test this PR locally

@kcreddy kcreddy added Integration:ti_eclecticiq EclecticIQ (Partner supported) enhancement New feature or request labels Jun 25, 2024
@elasticmachine

Copy link
Copy Markdown

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine

Copy link
Copy Markdown

💚 Build Succeeded

@elastic-sonarqube

Copy link
Copy Markdown

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

@kcreddy
kcreddy marked this pull request as ready for review June 25, 2024 06:41
@kcreddy
kcreddy requested a review from a team as a code owner June 25, 2024 06:41
@kcreddy kcreddy added the Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] label Jun 25, 2024
@elasticmachine

Copy link
Copy Markdown

Pinging @elastic/security-service-integrations (Team:Security-Service Integrations)

@kcreddy kcreddy self-assigned this Jun 25, 2024
@kcreddy
kcreddy merged commit cb08534 into elastic:main Jun 25, 2024
@elasticmachine

Copy link
Copy Markdown

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

@kcreddy
kcreddy deleted the eclectiq branch February 7, 2025 09:11
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:ti_eclecticiq EclecticIQ (Partner supported) Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

3 participants