Skip to content
This repository has been archived by the owner on Aug 2, 2022. It is now read-only.

Fix UT workflow to only run bootstrap once #210

Merged
merged 2 commits into from
Jun 9, 2020

Conversation

ohltyler
Copy link
Contributor

@ohltyler ohltyler commented Jun 9, 2020

Issue #, if available: #75

Description of changes:

This PR fixes the bug where running yarn kbn bootstrap would occasionally fail the first time and would have to be ran twice (on both the GitHub runners and on a local dev environment).

Specific changes:

  • Removed the extra bootstrap step in the UT workflow
  • Upgraded the yarn.lock by running yarn upgrade to upgrade all dependencies. This fixes the bootstrap failing issue.
  • Updated test snapshots after upgrading dependencies.

Notes

  • Confirmed all UT pass
  • Confirmed UT workflow passes (see this run)
  • Confirmed the plugin loads and functions fine after upgrading

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@ohltyler ohltyler linked an issue Jun 9, 2020 that may be closed by this pull request
@yizheliu-amazon
Copy link
Contributor

can you quickly do small smoke test to verify if UI is working without anything broken? looks like many dependencies are upgraded.

@ohltyler
Copy link
Contributor Author

ohltyler commented Jun 9, 2020

can you quickly do small smoke test to verify if UI is working without anything broken? looks like many dependencies are upgraded.

yes, I've done this. don't see anything broken

@yizheliu-amazon
Copy link
Contributor

can you quickly do small smoke test to verify if UI is working without anything broken? looks like many dependencies are upgraded.

yes, I've done this. don't see anything broken

Cool.

Copy link
Contributor

@ylwu-amzn ylwu-amzn left a comment

Choose a reason for hiding this comment

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

LGTM. Thanks for the fix.

@ohltyler ohltyler merged commit 3ac448e into opendistro-for-elasticsearch:master Jun 9, 2020
@ohltyler ohltyler deleted the fixBootstrap branch June 9, 2020 21:49
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Unit test workflow failing when running yarn kbn bootstrap
3 participants