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

Install analysis-icu plugin for OC17+ #4

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

wsmirnow
Copy link
Contributor

Opencast 17+ needs analysis-icu OpenSearch plugin installed.

Comment on lines 39 to 48
- name: Install OpenSearch analysis-icu plugin
notify: Restart opensearch
ansible.builtin.command:
argv:
- /usr/share/opensearch/bin/opensearch-plugin
- install
- --silent
- analysis-icu
creates: /usr/share/opensearch/plugins/analysis-icu
when: opencast_version_major | default(16) | int >= 17
Copy link
Contributor

Choose a reason for hiding this comment

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

Don't do this. Instead, in line 4, install opensearch-plugin-analysis-icu

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Will update this PR

Copy link
Contributor Author

Choose a reason for hiding this comment

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

There are no opensearch plugins in Debian repo :(

apt search opensearch
Sorting... Done
Full Text Search... Done
libwww-opensearch-perl/stable 0.17-2 all
  search OpenSearch compatible web sites

opensearch/17.x 1.3.16 amd64
  An open source distributed and RESTful search engine

opensearch-dist/17.x 1.3.3-1 amd64
  OpenSearch is a community-driven, Apache 2.0-licensed open source

Copy link
Contributor

@lkiesow lkiesow Dec 17, 2024

Choose a reason for hiding this comment

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

Maybe it's still in the testing repo? AFAIK Greg wanted to package it. Let's ask him.
For the RPMs, it's in the 17 and 18 stable repos.

Copy link
Contributor

Choose a reason for hiding this comment

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

Talked to Greg. He said the Debian packages should be available later today or tomorrow.

@wsmirnow wsmirnow force-pushed the install-analysis-icu-plugin-for-oc17 branch from 80e0038 to 6bb423c Compare December 18, 2024 08:18
@wsmirnow
Copy link
Contributor Author

The requested changes are implemented. For now the Debian molecule tests will fail. But when the plugin is available in the Debian repo, we should continue with the review.

@wsmirnow wsmirnow force-pushed the install-analysis-icu-plugin-for-oc17 branch from 6bb423c to 2405a4e Compare December 18, 2024 08:21
Opencast 17+ needs analysis-icu OpenSearch plugin installed.
@wsmirnow wsmirnow force-pushed the install-analysis-icu-plugin-for-oc17 branch from 2405a4e to c4fab75 Compare December 19, 2024 09:11
@wsmirnow
Copy link
Contributor Author

Debian Plugin package is available now. The PR is ready for the next round :)

@wsmirnow wsmirnow requested a review from lkiesow December 19, 2024 09:16
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.

2 participants