Skip to content

Cherry-pick #20370 to 7.9: Check expand_event_list_from_field when json in map[string]interface{} format#20373

Merged
kaiyan-sheng merged 2 commits intoelastic:7.9from
kaiyan-sheng:backport_20370_7.9
Jul 31, 2020
Merged

Cherry-pick #20370 to 7.9: Check expand_event_list_from_field when json in map[string]interface{} format#20373
kaiyan-sheng merged 2 commits intoelastic:7.9from
kaiyan-sheng:backport_20370_7.9

Conversation

@kaiyan-sheng
Copy link
Copy Markdown
Contributor

@kaiyan-sheng kaiyan-sheng commented Jul 31, 2020 •
edited by zube bot
Loading

Cherry-pick of PR #20370 to 7.9 branch. Original message:

What does this PR do?

When json is in map[string]interface{} format, s3 input should also check for expand_event_list_from_field config parameter.
With this PR, cloudtrail fileset is working again:
Screen Shot 2020-07-30 at 7 11 09 PM

Checklist

  • My code follows the style guidelines of this project
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have made corresponding change to the default configuration files
  • I have added tests that prove my fix is effective or that my feature works
  • I have added an entry in CHANGELOG.next.asciidoc or CHANGELOG-developer.next.asciidoc.

@kaiyan-sheng kaiyan-sheng added [zube]: In Review backport Team:Platforms Label for the Integrations - Platforms team labels Jul 31, 2020
@botelastic botelastic bot added the needs_team Indicates that the issue/PR needs a Team:* label label Jul 31, 2020
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Jul 31, 2020

Pinging @elastic/integrations-platforms (Team:Platforms)

@botelastic botelastic bot removed the needs_team Indicates that the issue/PR needs a Team:* label label Jul 31, 2020
@kaiyan-sheng kaiyan-sheng self-assigned this Jul 31, 2020
@elasticmachine
Copy link
Copy Markdown
Contributor

elasticmachine commented Jul 31, 2020 •
edited by jenkins-beats-ci bot
Loading

💚 Build Succeeded

Pipeline View Test View Changes Artifacts preview

Expand to view the summary

Build stats

  • Build Cause: [Pull request #20373 updated]

  • Start Time: 2020-07-31T03:04:10.538+0000

  • Duration: 57 min 57 sec

Test stats 🧪

Test Results
Failed 0
Passed 2434
Skipped 385
Total 2819

Steps errors

Expand to view the steps failures

  • Name: Install Go 1.14.4

    • Description: .ci/scripts/install-go.sh

    • Duration: 1 min 57 sec

    • Start Time: 2020-07-31T03:29:28.764+0000

    • log

@kaiyan-sheng kaiyan-sheng merged commit bef2263 into elastic:7.9 Jul 31, 2020
@kaiyan-sheng kaiyan-sheng deleted the backport_20370_7.9 branch July 31, 2020 13:06
@zube zube bot removed the [zube]: Done label Oct 30, 2020
leweafan pushed a commit to leweafan/beats that referenced this pull request Apr 28, 2023
Cherry-pick elastic#20370 to 7.9: Check expand_event_list_from_field …
ddd9f21 
…when json in map[string]interface{} format (elastic#20373)

(cherry picked from commit c39dde3)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ChrsMark ChrsMark ChrsMark approved these changes

Assignees

@kaiyan-sheng kaiyan-sheng

Labels

backport Team:Platforms Label for the Integrations - Platforms team

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@kaiyan-sheng @elasticmachine @ChrsMark