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

Deploy OpenEdX Forum V2 to mitx #2894

Open
feoh opened this issue Jan 10, 2025 · 11 comments · Fixed by #2910
Open

Deploy OpenEdX Forum V2 to mitx #2894

feoh opened this issue Jan 10, 2025 · 11 comments · Fixed by #2910
Assignees

Comments

@feoh
Copy link
Contributor

feoh commented Jan 10, 2025

Description/Context

Overall

Changes that will require new infrastructure:

  • new python service to replace Ruby service
  • mysql instead of mongo
  • feature flags
  • data migration?

Deployment

  • Split forum v2 and ultimately all openedx deploy into their own, non ol-infrastructure repository
  • Write Dockerfile
  • Create docker-packer-pulumi pipeline
@feoh
Copy link
Contributor Author

feoh commented Jan 13, 2025

@feoh feoh self-assigned this Jan 14, 2025
@feoh
Copy link
Contributor Author

feoh commented Jan 15, 2025

Forked the repo and took a very uneducated swing at a Dockerfile, but I'm missing too much context for this to be a likely hit.

@feoh
Copy link
Contributor Author

feoh commented Jan 16, 2025

Enabled in mitx-ci:

app@5cf73745591e:~/edx-platform$ ./manage.py lms waffle_flag --create --everyone discussions.enable_forum_v2

Creating flag: discussions.enable_forum_v2
Setting name: discussions.enable_forum_v2
Setting everyone: True
Setting percent: None
Setting superusers: False
Setting staff: False
Setting authenticated: False
Setting group(s): []
Setting user(s): set()
Setting rollout: False
Setting testing: False
app@5cf73745591e:~/edx-pl

@feoh
Copy link
Contributor Author

feoh commented Jan 16, 2025

Enable mysql backend as per @blarghmatey:

app@5cf73745591e:~/edx-platform$ ./manage.py lms waffle_flag --create --everyone forum_v2.enable_mysql_backend

Creating flag: forum_v2.enable_mysql_backend
Setting name: forum_v2.enable_mysql_backend
Setting everyone: True
Setting percent: None
Setting superusers: False
Setting staff: False
Setting authenticated: False
Setting group(s): []
Setting user(s): set()
Setting rollout: False
Setting testing: False

@feoh
Copy link
Contributor Author

feoh commented Jan 17, 2025

From @blarghmatey:

We need to provide the settings expected by the plugin which you can find at https://github.com/openedx/forum/blob/master/forum/settings/common.py

In particular, we need to set the MongoDB values, as well as setting the search backend to forum.search.es.ElasticsearchBackend

@feoh
Copy link
Contributor Author

feoh commented Jan 17, 2025

@feoh
Copy link
Contributor Author

feoh commented Jan 22, 2025

I figured out the necessary syntax for a couple of the configurables by interpolating the example found here: https://github.com/openedx/forum/blob/342ad3b59344cb998159eb4ee580b39785888a62/forum/settings/test.py#L68 with the forum v1 info Tobias pointed me at previously.

Hopefully I managed to actually connect with the ball, if not make a base hit :)

@pdpinch
Copy link
Member

pdpinch commented Jan 23, 2025

Where/how can I see this in action? Is it on https://lms-ci.mitx.mit.edu/ ?

@feoh
Copy link
Contributor Author

feoh commented Jan 28, 2025

We found an upstream bug in validating this, reported at openedx/forum#155

@feoh feoh reopened this Jan 28, 2025
@feoh
Copy link
Contributor Author

feoh commented Jan 28, 2025

Not even sure this will work going forward but worth a try. This is the URL that throws the exception:

https://courses-qa.mitxonline.mit.edu/courses/course-v1:MITx+10.50.CH01x+1T2023/discussion/forum/

@feoh
Copy link
Contributor Author

feoh commented Jan 28, 2025

Where/how can I see this in action? Is it on https://lms-ci.mitx.mit.edu/ ?

MITx Online QA.

See my URL above for an exact path to reproduce, but I'm also not sure if that URL encodes my user etc as we went through a bunch of steps to get there.

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 a pull request may close this issue.

2 participants