-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
Fix and unskip integration test suite #97144
Fix and unskip integration test suite #97144
Conversation
Pinging @elastic/kibana-presentation (Team:Presentation) |
@elasticmachine merge upstream |
@@ -230,7 +230,7 @@ export class MonitoringPlugin | |||
if (this.cluster) { | |||
this.cluster.close(); | |||
} | |||
if (this.licenseService) { | |||
if (this.licenseService && this.licenseService.stop) { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's necessary because the server crashes if it calls stop
before start
.
@elasticmachine merge upstream |
@elasticmachine merge upstream |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Code only review - makes sense and LGTM!
💚 Build SucceededMetrics [docs]
History
To update your PR or re-run it, just comment with: |
💔 Backport failed
To backport manually run: |
Co-authored-by: Kibana Machine <[email protected]> # Conflicts: # src/core/server/saved_objects/migrationsv2/integration_tests/migration_7.7.2_xpack_100k.test.ts
Summary
For some reason, was waiting until plugin start to register the embeddable factory server side. This was causing migrations to fail on startup.
This moves the registration to happen in setup, and unskips the test suite in question.