Skip to content
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

Clarify plugin repo naming conventions #27

Merged
merged 1 commit into from
Mar 22, 2022

Conversation

lukas-vlcek
Copy link
Contributor

Signed-off-by: Lukáš Vlček lukas.vlcek@aiven.io

Description

OpenSearch plugins can "live" either inside the OpenSearch organization or outside it. This is the difference between official plugins and 3rd party plugins that are maintained in independent repositories.

Currently, these two types of plugins are not recommended to share exactly the same repo naming conventions (this might change but not in the near-term future). If needed, the 3rd party plugin authors should also follow up with several extra checks in making sure the formal processes are correctly aligned with the community needs (such as Code of Conduct, Security Policy, Trademarks and Attributions).

Issues Resolved

Closes #23

Check List

  • Commits are signed per the DCO using --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.

@lukas-vlcek lukas-vlcek changed the title Clarify plugin naming conventions Clarify plugin repo naming conventions Feb 25, 2022
@lukas-vlcek
Copy link
Contributor Author

lukas-vlcek commented Feb 25, 2022

In the PR I briefly mention that versioning policy should be documented for (3rd party) plugins but no specific instructions/examples are provided.
Now I noticed there is #8 but it is still open so no definitive policy has been adopted or recommended yet (meaning we still can not provide any specific info about versioning in the documentation).

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@lukas-vlcek lukas-vlcek force-pushed the naming_conventions branch 2 times, most recently from f91a0c7 to 1bb8913 Compare February 25, 2022 18:52

Depending on the plugin relationship with the OpenSearch organization we currently recommend the following naming conventions and optional follow-up checks:

### Official plugins
Copy link
Member

Choose a reason for hiding this comment

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

I don't love the name "Official". How about "Plugins Released with OpenSearch", of which there are 2 kinds, native and non-native?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I was following naming pattern from InstallPluginCommand.java, see the JavaDoc and var name (also in online docs). Shall we rename in the install tool source code as well?

I would use "Native" and "Thirdparty" plugins. WDYT?

Copy link
Member

Choose a reason for hiding this comment

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

I see. That can be a separate PR. Let's leave it this way here, consider following up?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

consider following up?

Yes, naming should be addressed in different PR.

README.md Outdated Show resolved Hide resolved
@dblock dblock requested review from reta and AmiStrn March 7, 2022 20:53
OpenSearch plugins can "live" either inside the OpenSearch organization or outside it. This is the difference between official plugins and 3rd party plugins that are maintained in independent repositories.

Currently, these two types of plugins are not recommended to share exactly the same repo naming conventions (this might change but not in the near-term future). If needed, the 3rd party plugin authors should also follow up with several extra checks in making sure the formal processes are correctly aligned with the community needs (such as Code of Conduct, Security Policy, Trademarks and Attributions).

Closes opensearch-project#23

Signed-off-by: Lukáš Vlček <lukas.vlcek@aiven.io>
@dblock dblock merged commit 9cd46a2 into opensearch-project:main Mar 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[PROPOSAL] Plugin naming conventions do not work well for community plugins
3 participants