-
Notifications
You must be signed in to change notification settings - Fork 90
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
Add verify binary installation workflow and add loose to bootstrapping #891
Add verify binary installation workflow and add loose to bootstrapping #891
Conversation
Signed-off-by: Derek Ho <[email protected]>
@@ -63,7 +63,7 @@ jobs: | |||
- name: Bootstrap OpenSearch Dashboards with plugin | |||
run: | | |||
cd OpenSearch-Dashboards/plugins/alerting-dashboards-plugin | |||
yarn osd bootstrap | |||
yarn osd bootstrap --single-version=loose |
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.
More context here: opensearch-project/OpenSearch-Dashboards#5561
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #891 +/- ##
==========================================
- Coverage 48.70% 48.69% -0.01%
==========================================
Files 236 236
Lines 6852 6845 -7
Branches 997 996 -1
==========================================
- Hits 3337 3333 -4
+ Misses 3512 3509 -3
Partials 3 3 ☔ View full report in Codecov by Sentry. |
Signed-off-by: Derek Ho <[email protected]>
Signed-off-by: Derek Ho <[email protected]>
@@ -42,7 +42,7 @@ jobs: | |||
cd ./OpenSearch-Dashboards/ | |||
su `id -un 1000` -c "source $NVM_DIR/nvm.sh && nvm use && node -v && yarn -v && | |||
cd ./plugins/alerting-dashboards-plugin && | |||
whoami && yarn osd bootstrap && yarn build && yarn run test:jest --coverage" | |||
whoami && yarn osd bootstrap --single-version=loose && yarn build && yarn run test:jest --coverage" |
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.
Is the --single-version=loose
argument also needed on line 87?
Signed-off-by: Derek Ho <[email protected]>
…rds-plugin into build-and-test-workflow
…rds-plugin into build-and-test-workflow
#891) * Add verify binary installation workflow and add loose to bootstrapping Signed-off-by: Derek Ho <[email protected]> * Fix built zip name Signed-off-by: Derek Ho <[email protected]> * Remove references to security Signed-off-by: Derek Ho <[email protected]> * Add loose Signed-off-by: Derek Ho <[email protected]> --------- Signed-off-by: Derek Ho <[email protected]> (cherry picked from commit 06fa155) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Description
I have observed some issues within other plugins of issues being only caught at run time. This is because several things can go wrong during the build process, which may not be caught in a dev setup. This adds a workflow to verify that building and installing into OSD works on every PR. It also adds loose bootstrapping to resolve cypress conflicts after OSD added a dependency recently.
Related issues:
opensearch-project/security-dashboards-plugin#1709
opensearch-project/security-analytics-dashboards-plugin#875
Issues Resolved
[List any issues this PR will resolve]
Check List
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.