-
Notifications
You must be signed in to change notification settings - Fork 8.2k
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 the base package from the default packages #70437
Conversation
As the base assets will be shipped by ES directly, the base package is not needed anymore. elastic/elasticsearch#57629 In the future we might reintroduce it to update the installed assets.
Pinging @elastic/ingest-management (Team:Ingest Management) |
This PR should go green / be working as soon as the most recent snapshots are available from Elasticsearch. |
@jonathan-buttner For awareness |
@ruflin This would go in 7.x as well, 7.x PR is merged, I have added a reminder to myself to make sure its merged on monday |
💛 Build succeeded, but was flaky
Test FailuresKibana Pipeline / kibana-xpack-agent / X-Pack API Integration Tests.x-pack/test/api_integration/apis/endpoint/resolver·ts.apis Endpoint plugin Resolver related events route endpoint events should not find any eventsStandard Out
Stack Trace
Kibana Pipeline / kibana-xpack-agent / X-Pack API Integration Tests.x-pack/test/api_integration/apis/endpoint/resolver·ts.apis Endpoint plugin Resolver "after all" hook in "Resolver"Standard Out
Stack Trace
Build metrics
To update your PR or re-run it, just comment with: |
@elasticmachine merge upstream |
Well that test failure is discouraging 🤔 removing the base package shouldn't have anything to do with that... I'll look into it |
@jonathan-buttner I'll assigned you as "Assignee" to this PR so you can follow up. |
As the base assets will be shipped by ES directly, the base package is not needed anymore. elastic/elasticsearch#57629 In the future we might reintroduce it to update the installed assets. Co-authored-by: Nicolas Ruflin <spam@ruflin.com>
As the base assets will be shipped by ES directly, the base package is not needed anymore. elastic/elasticsearch#57629
In the future we might reintroduce it to update the installed assets.