-
Notifications
You must be signed in to change notification settings - Fork 272
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
Automate publishing plugins zips to artifacts repository #184
Comments
[Triage] Plugin releases as zips will be available for only distribution builds, no out of band releases. |
[Grooming] We need to design this thing so that it installed like the native plugin, contrary to ODFE setup where user need the direct link to the zips. |
ProposalThere are 2 ways to accomplish this -
Approach 1This approach would require a change in Following are the contents file generated on the fly at the time of build - (22-01-31 22:23:03) <0> [/tmp/tmpuj_b8b1g/OpenSearch]
% find . -name plugins.txt
./server/build/generated-resources/plugins.txt
./server/build/resources/main/plugins.txt
(22-01-31 22:23:14) <0> [/tmp/tmpuj_b8b1g/OpenSearch]
% cat ./server/build/generated-resources/plugins.txt
analysis-icu
analysis-kuromoji
analysis-nori
analysis-phonetic
analysis-smartcn
analysis-stempel
analysis-ukrainian
discovery-azure-classic
discovery-ec2
discovery-gce
ingest-attachment
mapper-annotated-text
mapper-murmur3
mapper-size
repository-azure
repository-gcs
repository-hdfs
repository-s3
store-smb
transport-nio%
(22-01-31 22:23:20) <0> [/tmp/tmpuj_b8b1g/OpenSearch]
% cat ./server/build/resources/main/plugins.txt
analysis-icu
analysis-kuromoji
analysis-nori
analysis-phonetic
analysis-smartcn
analysis-stempel
analysis-ukrainian
discovery-azure-classic
discovery-ec2
discovery-gce
ingest-attachment
mapper-annotated-text
mapper-murmur3
mapper-size
repository-azure
repository-gcs
repository-hdfs
repository-s3
store-smb
transport-nio% Approach 2Second approach, we can provide urls for these plugins for download (would require updating existing documentation, example link from the plugins listed here). Question for discussion - Which approach do should we follow? |
For # 2. The plugins are already available at ci.opensearch.org and we can install it now using the below command and the plugin (zip) artifacts links are present inside the manifest of every distribution.
I prefer a clean solution for plugin installation (both native and standalone plugin) using same plugin install command for both native and standalone plugins. |
@VachaShah @mch2 @adnapibar @nknize Please provide your inputs on this proposal. |
tl;dr For bcw we need to know the released version of the plugin - what's that URL on To enable downloading by name of both plugins, I would modify OpenSearch not to hard-code the plugin names, but to ship with a configurable set of plugin repositories and default to |
Awaiting response from teams to proceed with this issue further. |
Hey @VachaShah can this issue be closed also just a thought once this META ISSUE is resolved, we can expect the plugins to be in maven repo, we can fetch them during runtime using maven coordinates for BWC tests during runtime. With this we need not publish again to artifacts.opensearch.org. |
Since we publish several ZIPs to artifacts.opensearch.org, we should also publish the other zips there. The maven use-case is very specific for those that use maven as their dependency/build system. |
I'm closing this issue here as we now publish plugin zips to maven. We could also find plugin zips in our |
We should be able to predictably get older releases of opensearch project plugins from an artifact repository. This would be needed to introduce a better way for scalability to test bwc for all releases. This came up in opensearch-project/anomaly-detection#158.
Acceptance Criteria:
Plugin zips should be downloadable from htts://artifacts.opensearch.org/......
The text was updated successfully, but these errors were encountered: