Skip to content

[aws] add ignore_failure: true for uri_parts processor in s3access data stream#17915

Merged
kaiyan-sheng merged 2 commits intomainfrom
s3access_uri
Mar 26, 2026
Merged

[aws] add ignore_failure: true for uri_parts processor in s3access data stream#17915
kaiyan-sheng merged 2 commits intomainfrom
s3access_uri

Conversation

@kaiyan-sheng
Copy link
Contributor

@kaiyan-sheng kaiyan-sheng commented Mar 20, 2026

Proposed commit message

S3 access log can include request paths that uri_parts processor cannot parse, such as /[[...optional]]. This change sets ignore_failure: true on the uri_parts step so parsing continues even though uri_parts failed.

With this change, aws.s3access.request_uri and related fields remain available, and ECS url.* is only populated when the path is parseable.

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.
  • I have verified that any added dashboard complies with Kibana's Dashboard good practices

@kaiyan-sheng kaiyan-sheng requested a review from a team as a code owner March 20, 2026 04:30
@kaiyan-sheng kaiyan-sheng requested review from a team as code owners March 20, 2026 04:45
@elastic-vault-github-plugin-prod

🚀 Benchmarks report

To see the full report comment with /test benchmark fullreport

@elasticmachine
Copy link

💚 Build Succeeded

@andrewkroh andrewkroh added Integration:aws AWS Team:obs-ds-hosted-services Observability Hosted Services team [elastic/obs-ds-hosted-services] labels Mar 20, 2026
@kaiyan-sheng
Copy link
Contributor Author

@claude Review the PR please

@kaiyan-sheng kaiyan-sheng merged commit 775aafd into main Mar 26, 2026
9 checks passed
@kaiyan-sheng kaiyan-sheng deleted the s3access_uri branch March 26, 2026 02:49
@elastic-vault-github-plugin-prod

Package aws - 6.4.1 containing this change is available at https://epr.elastic.co/package/aws/6.4.1/

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

Labels

Integration:aws AWS Team:obs-ds-hosted-services Observability Hosted Services team [elastic/obs-ds-hosted-services]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants