Skip to content

[aws.cloudtrail] Ingest Pipeline Improvments - #20403

Draft
brijesh-elastic wants to merge 4 commits into
elastic:mainfrom
brijesh-elastic:aws_cloudtrail-ingest_pipeline-improvements
Draft

[aws.cloudtrail] Ingest Pipeline Improvments#20403
brijesh-elastic wants to merge 4 commits into
elastic:mainfrom
brijesh-elastic:aws_cloudtrail-ingest_pipeline-improvements

Conversation

@brijesh-elastic

Copy link
Copy Markdown
Contributor

Proposed commit message

aws.cloudtrail: ingest pipeline improvments

This PR replaced every `ignore_failure: true` with an `ignore_missing` flag or condition guard where
failure was expected and an explicit `on_failure` handler reporting to `error.message` where
it wasn't, so routine cases no longer throw per document and real faults surface instead of
being silently swallowed.

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

How to test this PR locally

  • Clone integrations repo.
  • Install elastic package locally.
  • Start elastic stack using elastic-package.
  • Move to integrations/packages/aws directory.
  • Run the following command to run tests.

elastic-package test -v

Related issues

…r condition guard where failure was expected and an explicit `on_failure` handler reporting to `error.message` where it wasn't, so routine cases no longer throw per document and real faults surface instead of being silently swallowed
@brijesh-elastic brijesh-elastic self-assigned this Jul 30, 2026
@brijesh-elastic brijesh-elastic added documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. enhancement New feature or request Integration:aws AWS Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations] Team:SDE-Crest Crest developers on the Security Integrations team [elastic/sit-crest-contractors] labels Jul 30, 2026
@elastic-vault-github-plugin-prod

Copy link
Copy Markdown
Contributor

✅ All changelog entries have the correct PR link.

@infra-vault-gh-plugin-prod

infra-vault-gh-plugin-prod Bot commented Aug 7, 2026

Copy link
Copy Markdown

💔 Build Failed

Failed CI Steps

History

cc @brijesh-elastic

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

Labels

documentation Improvements or additions to documentation. Applied to PRs that modify *.md files. enhancement New feature or request Integration:aws AWS Team:SDE-Crest Crest developers on the Security Integrations team [elastic/sit-crest-contractors] Team:Security-Service Integrations Security Service Integrations team [elastic/security-service-integrations]

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant