Skip to content

Fix kbn/optimizer tests#70827

Merged
spalger merged 4 commits intoelastic:masterfrom
spalger:fix/kbn-optimizer-tests
Jul 6, 2020
Merged

Fix kbn/optimizer tests#70827
spalger merged 4 commits intoelastic:masterfrom
spalger:fix/kbn-optimizer-tests

Conversation

@spalger
Copy link
Contributor

@spalger spalger commented Jul 6, 2020

Fixes #70753
Fixes #70762
Fixes #70764

The @kbn/optimizer captures stdio from workers and makes it available on it's main event stream. The workers started randomly complaining:

Browserslist: caniuse-lite is outdated. Please run the following command: `yarn upgrade`

This was triggering new events in the stream and caused assertions to fail. To prevent that from being an issue now and in the future I'm filtering out worker stdio events from the stream.

@spalger spalger added Team:Operations Kibana-Operations Team v8.0.0 release_note:skip Skip the PR/issue when compiling release notes 7.9.0 v7.8.1 labels Jul 6, 2020
@kibanamachine
Copy link
Contributor

💚 Build Succeeded

Build metrics

✅ unchanged

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

@spalger spalger marked this pull request as ready for review July 6, 2020 17:36
@spalger spalger requested a review from a team as a code owner July 6, 2020 17:36
@elasticmachine
Copy link
Contributor

Pinging @elastic/kibana-operations (Team:Operations)

@spalger spalger merged commit 4492531 into elastic:master Jul 6, 2020
@spalger spalger deleted the fix/kbn-optimizer-tests branch July 6, 2020 18:13
spalger added a commit to spalger/kibana that referenced this pull request Jul 6, 2020
Co-authored-by: spalger <spalger@users.noreply.github.com>
# Conflicts:
#	packages/kbn-optimizer/src/integration_tests/__snapshots__/basic_optimization.test.ts.snap
spalger added a commit to spalger/kibana that referenced this pull request Jul 6, 2020
Co-authored-by: spalger <spalger@users.noreply.github.com>
# Conflicts:
#	packages/kbn-optimizer/src/integration_tests/__snapshots__/basic_optimization.test.ts.snap
@spalger spalger added v7.9.0 and removed 7.9.0 labels Jul 6, 2020
spalger added a commit that referenced this pull request Jul 6, 2020
Co-authored-by: spalger <spalger@users.noreply.github.com>
spalger added a commit that referenced this pull request Jul 6, 2020
Co-authored-by: spalger <spalger@users.noreply.github.com>
# Conflicts:
#	packages/kbn-optimizer/src/integration_tests/__snapshots__/basic_optimization.test.ts.snap
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment