Skip to content

[7.x](backport #26267) Add log.flags and object metadata to aws-s3 input events#26298

Merged
andrewkroh merged 1 commit into7.xfrom
mergify/bp/7.x/pr-26267
Jun 14, 2021
Merged

[7.x](backport #26267) Add log.flags and object metadata to aws-s3 input events#26298
andrewkroh merged 1 commit into7.xfrom
mergify/bp/7.x/pr-26267

Conversation

@mergify
Copy link
Copy Markdown
Contributor

@mergify mergify bot commented Jun 14, 2021

This is an automatic backport of pull request #26267 done by Mergify.


Mergify commands and options

More conditions and actions can be found in the documentation.

You can also trigger Mergify actions by commenting on this pull request:

  • @Mergifyio refresh will re-evaluate the rules
  • @Mergifyio rebase will rebase this PR on its base branch
  • @Mergifyio update will merge the base branch into this PR
  • @Mergifyio backport <destination> will backport this PR on <destination> branch

Additionally, on Mergify dashboard you can:

  • look at your merge queues
  • generate the Mergify configuration with the config editor.

Finally, you can contact us on https://mergify.io/

* Add log.flags and object metadata to aws-s3 input events

This adds the log.flags field created by the line readers to aws-s3 events. log.flags contains metadata like `multiline` and `truncated` to indicate how the data was processed.

This also adds a config option to include S3 object metadata in the event if it exists. The use case for me was to get the Last-Modified timestamp for cases where the log does not have a timestamp or it cannot be parsed. Then this can be used as a fallback.

* Pass metadata using s3Info struct to avoid adding new func params

(cherry picked from commit c25fca8)
@mergify mergify bot added the backport label Jun 14, 2021
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Jun 14, 2021
@botelastic
Copy link
Copy Markdown

botelastic bot commented Jun 14, 2021

This pull request doesn't have a Team:<team> label.

@elasticmachine
Copy link
Copy Markdown
Contributor

💚 Build Succeeded

the below badges are clickable and redirect to their specific view in the CI or DOCS
Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: Pull request #26298 opened

  • Start Time: 2021-06-14T17:05:27.046+0000

  • Duration: 109 min 43 sec

  • Commit: 3e47a2f

Test stats 🧪

Test Results
Failed 0
Passed 14015
Skipped 2306
Total 16321

Trends 🧪

Image of Build Times

Image of Tests

💚 Flaky test report

Tests succeeded.

Expand to view the summary

Test stats 🧪

Test Results
Failed 0
Passed 14015
Skipped 2306
Total 16321

@andrewkroh andrewkroh merged commit 8944a69 into 7.x Jun 14, 2021
@mergify mergify bot deleted the mergify/bp/7.x/pr-26267 branch June 14, 2021 19:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport needs_team Indicates that the issue/PR needs a Team:* label

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants