-
Notifications
You must be signed in to change notification settings - Fork 277
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
[RPM M1] (Re-open) Update build workflow to build based on distribution (OpenSearch) #1629
[RPM M1] (Re-open) Update build workflow to build based on distribution (OpenSearch) #1629
Conversation
Signed-off-by: Peter Zhu <[email protected]>
Codecov Report
@@ Coverage Diff @@
## main #1629 +/- ##
============================================
+ Coverage 94.67% 94.68% +0.01%
- Complexity 13 14 +1
============================================
Files 163 163
Lines 3437 3446 +9
Branches 21 21
============================================
+ Hits 3254 3263 +9
Misses 180 180
Partials 3 3
Continue to review full report at Codecov.
|
Will add more tests tomorrow after all the changes in #1619. |
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.
This looks good! Just needs tests.
Signed-off-by: Peter Zhu <[email protected]>
Signed-off-by: Peter Zhu <[email protected]>
Signed-off-by: Peter Zhu <[email protected]>
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.
Good work.
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.
LGTM
Signed-off-by: Peter Zhu <[email protected]>
The dashboard part still needs some changes to reflect this addition, will issue new PRs on this. |
@dblock just so you know OSD code does not build rpm correctly: opensearch-project/OpenSearch-Dashboards#1259 |
Signed-off-by: Peter Zhu [email protected]
Description
Update build workflow to build based on distribution (OpenSearch).
You can now use
--distribution
to build tar/zip/rpm for OpenSearch-min.Not completely bound to
platform
anymore.This would also setup the stage for bundled RPM creation in assemble_workflow (#2073)
Issues Resolved
Part of #1543
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.