-
Notifications
You must be signed in to change notification settings - Fork 112
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 publish in spi build.gradle #1207
Add publish in spi build.gradle #1207
Conversation
9035634
to
848a717
Compare
848a717
to
3ef1f9c
Compare
Signed-off-by: bowenlan-amzn <[email protected]>
3ef1f9c
to
f0f18c6
Compare
Signed-off-by: bowenlan-amzn <[email protected]>
Thanks @bowenlan-amzn can you please also confirm if there is no change in publishing of the zip file ?
|
I run @prudhvigodithi and a quick question, do we need to backport this to any branch other than 2.x? |
reference: opensearch-project/job-scheduler#586 Signed-off-by: bowenlan-amzn <[email protected]>
Thanks @bowenlan-amzn, ya as long as there no change in publishing of the zip file we should be good, also just 2.x as the gradle custom plugin used |
214aae5
into
opensearch-project:main
(cherry picked from commit 214aae5) Signed-off-by: github-actions[bot] <github-actions[bot]@users.noreply.github.com>
Description
Publish the spi to maven
Can be consumed as
compileOnly "org.opensearch:opensearch-index-management-spi:${opensearch_version}"
Example commit of extend ISM opensearch-project/cross-cluster-replication@3853901
Test
Test locally by running
./gradlew publishShadowPublicationToStagingRepository
Reference
opensearch-project/alerting#1562
opensearch-project/alerting#1604
https://github.com/opensearch-project/opensearch-plugins/blob/main/BUILDING.md#include-checksums-in-maven-publications
Check List
--signoff
.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.