Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

🔥 Remove Filestream integration #4609

Merged
merged 2 commits into from
Nov 9, 2022
Merged

Conversation

kpollich
Copy link
Member

@kpollich kpollich commented Nov 9, 2022

Remove Filestream integration because it was never intended to be published on EPR 😶

@kpollich kpollich added the Team:Ecosystem Label for the Packages Ecosystem team [elastic/ecosystem] label Nov 9, 2022
@kpollich kpollich requested a review from mrodm November 9, 2022 18:12
@kpollich kpollich self-assigned this Nov 9, 2022
@kpollich kpollich requested a review from a team as a code owner November 9, 2022 18:12
@elasticmachine
Copy link

elasticmachine commented Nov 9, 2022

💚 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 preview

Expand to view the summary

Build stats

  • Start Time: 2022-11-09T18:26:49.604+0000

  • Duration: 5 min 24 sec

🤖 GitHub comments

Expand to view the GitHub comments

To re-run your PR in the CI, just comment with:

  • /test : Re-trigger the build.

Copy link
Member

@cmacknz cmacknz left a comment

Choose a reason for hiding this comment

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

Thanks for correcting this

@kpollich kpollich merged commit 7b04924 into elastic:main Nov 9, 2022
@kpollich kpollich deleted the remove-filestream branch November 9, 2022 18:34
@Busbar2297
Copy link

Question, when it is gonna be released, it is very cool as I mistakenly deployed it and works great

@kpollich
Copy link
Member Author

Hi @Busbar2297 - we're prioritizing migrating the underlying Beats inputs for all integrations using the log input to the filestream input in the future, so there won't be a separate filestream integration to install. This will be an under-the-hood change to the custom logs integration.

You can track this overall effort here: #2518. Though we haven't prioritized this quite yet, it is on our roadmap internally.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Team:Ecosystem Label for the Packages Ecosystem team [elastic/ecosystem]
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants