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.

Sign up for GitHub

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

Jump to bottom

Does Multi-tenant Elastics fail if other plugins like tutor-forum are installed? #48

Closed
CodeWithEmad opened this issue Nov 20, 2023 · 0 comments · Fixed by #73
Closed

Does Multi-tenant Elastics fail if other plugins like tutor-forum are installed? #48

CodeWithEmad opened this issue Nov 20, 2023 · 0 comments · Fixed by #73
Assignees
@gabor-boros

Comments

@CodeWithEmad
Copy link
Member

CodeWithEmad commented Nov 20, 2023 •
edited
Loading

This plugin will only change the ELASTIC_SEARCH_CONFIG for LMS/CMS, but other applications like Forum will use elastic:

https://github.com/overhangio/tutor-forum/blob/fee67a8b66ec003967d58618d830f9b4d4ac211d/tutorforum/plugin.py#L27

and since tutor is responsible for the deployment, the forum plugin will be configured with
ELASTICSEARCH_SCHEME (default: "http")
ELASTICSEARCH_HOST (default: "elasticsearch")
ELASTICSEARCH_PORT (default: 9200)

tutor-discovery is also using elastic. What about that?
@CodeWithEmad CodeWithEmad changed the title Multi-tenant Elasticsearch will fail if other plugins like tutor-mfe Multi-tenant Elastics will fail if other plugins like tutor-mfe are installed? Nov 20, 2023
@CodeWithEmad CodeWithEmad changed the title Multi-tenant Elastics will fail if other plugins like tutor-mfe are installed? Does Multi-tenant Elastics fail if other plugins like tutor-mfe are installed? Nov 20, 2023
@CodeWithEmad CodeWithEmad changed the title Does Multi-tenant Elastics fail if other plugins like tutor-mfe are installed? Does Multi-tenant Elastics fail if other plugins like tutor-forum are installed? Nov 28, 2023
@gabor-boros gabor-boros self-assigned this May 3, 2024
@gabor-boros gabor-boros mentioned this issue May 3, 2024
Merged
@gabor-boros gabor-boros mentioned this issue Jun 7, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@gabor-boros gabor-boros

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix: ensure ES/OS cluster setup for shared usage openedx/openedx-k8s-harmony
2 participants
@gabor-boros @CodeWithEmad