Skip to content

[Fleet] Fix data stream API when event.ingested is null#135817

Merged
kpollich merged 2 commits intoelastic:mainfrom
kpollich:fix-missing-data-stream-data
Jul 6, 2022
Merged

[Fleet] Fix data stream API when event.ingested is null#135817
kpollich merged 2 commits intoelastic:mainfrom
kpollich:fix-missing-data-stream-data

Conversation

@kpollich
Copy link
Copy Markdown
Member

@kpollich kpollich commented Jul 6, 2022

Summary

Closes elastic/elastic-agent#654

I don't actually have a great way to test this, so I'm working on getting a cloud deploy set up to attempt to reproduce the issue linked above. Putting this PR up so we can deploy it to cloud as well to compare.

@kpollich kpollich added release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team ci:deploy-cloud v8.4.0 labels Jul 6, 2022
@kpollich kpollich self-assigned this Jul 6, 2022
@kpollich kpollich requested a review from a team as a code owner July 6, 2022 13:47
@elasticmachine
Copy link
Copy Markdown
Contributor

Pinging @elastic/fleet (Team:Fleet)

@kpollich
Copy link
Copy Markdown
Member Author

kpollich commented Jul 6, 2022

Issue is present on my 8.4 snapshot QA cloud deploy:
image

Copy link
Copy Markdown
Contributor

@juliaElastic juliaElastic left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kpollich
Copy link
Copy Markdown
Member Author

kpollich commented Jul 6, 2022

@elasticmachine merge upstream

@kibana-ci
Copy link
Copy Markdown

💚 Build Succeeded

Metrics [docs]

✅ unchanged

History

To update your PR or re-run it, just comment with:
@elasticmachine merge upstream

cc @kpollich

@kpollich
Copy link
Copy Markdown
Member Author

kpollich commented Jul 6, 2022

Issue appears to be resolved in the cloud deployment off of 8.4-SNAPSHOT for this PR

image

Going to merge this.

@kpollich kpollich merged commit f15355a into elastic:main Jul 6, 2022
@kibanamachine kibanamachine added the backport:skip This PR does not require backporting label Jul 6, 2022
@kpollich kpollich deleted the fix-missing-data-stream-data branch July 6, 2022 16:50
@mdelapenya
Copy link
Copy Markdown
Contributor

Thanks @kpollich for creating #135858 to not forget about the tests 👏

@tylersmalley tylersmalley added ci:cloud-deploy Create or update a Cloud deployment and removed ci:deploy-cloud labels Aug 17, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

backport:skip This PR does not require backporting ci:cloud-deploy Create or update a Cloud deployment release_note:skip Skip the PR/issue when compiling release notes Team:Fleet Team label for Observability Data Collection Fleet team v8.4.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Fleet]: No data available under Data Streams for installed Agents.

7 participants