...
Announcements will be posted to Slack in #tutor-maintenance
Recordings and Transcripts
Info |
---|
All public Working Group meetings follow the Recording Policy for Open edX Meetings |
https://drive.google.com/drive/folders/1jHyN3DMxwfNEgPxiUJOJS3Q3mb0Z0uMx1hdlItbQ6RK8iHeOZXahpo_vIo5offr_h
Agenda and Notes
We meet every other week. Our schedule and video conferencing link is on the Working Group Calendar. The meeting is usually hosted by Kyle McCormick .
You can add topics below ahead of time, or add them during the meeting.
2024-
...
11-
...
18
...
2024-11-04
WordPress plugin for commerce
verdaccio.Official plugin? Core? Not ready? Need more time to evaluate?wordpress
Speed of change to microfrontends makes it difficult to want to contribute
robots.txt for most MFEs
Caddyfile could add a robots.txt like this to the MFE container, with varying restrictions based on the MFEs within the container:
Code Block # robots.txt {% for app in MFE_APPS %} {% if app['no-crawl'] % } User-agent: * Disallow: /{app} {% edif % } {% endfor %} User-agent: * Disallow: /learning/course/*/progress # Don't crawl learner progress pages
2024-10-21
(if you’re watching the recording, jump to ~0:13 for the beginning of conversation)
Kyle/Hina: Tour of https://github.com/overhangio/tutor-indigo
Max: Mini update on RG’s use of verdaccio to mitigate the fact that npm-install is regularly timing out / getting rate-limited
2024-09-23
AI
https://github.com/kdmccormick/tutor/pull/39
Kyle will work on this in November
Database migrations are slow
2024-08-26
tutor-legacy-assets idea
too many MFEs
agreed
we’ll use existing MFEs for new pages
tutor-contrib-verdaccio update: not conclusive
ubuntu upgrade – maintainers will need to update their dockerfiles
note: open edx upstream will just test on ubuntu-latest going forward
2024-08-12
Follow-up on tutor-contrib-verdaccio.
Official plugin? Core? Not ready? Need more time to evaluate?
The plugin doesn’t work for Docker Desktop but only for Orbstack - need to check the docker.internal for docker desktop
We will continue the conversation in the discuss and tutor-mfe issue
We tutor maintainers should keep an eye on DEPR tickets and make sure plugins get deprecated if the upstream code gets deprecated
2024-07-29
Moises: https://docs.docker.com/reference/dockerfile/#run---mounttypessh I wanted to use the
RUN --mount=type=ssh
instruction to install private packages. This command needs an additional flag to the build command that I can add via theDOCKER_BUILD_COMMAND
filter. The problem is that the--ssh=default=$SSH_AUTH_SOCK
flag gets escaped by shlex. As a workaround is possible to useos.environ.get("SSH_AUTH_SOCK")
but some people may not want to print those values to the stdout.what type of packages?
Zameel: Installing npm packages from a private github repo
cloned with personal access token, installed locally
didn’t need to use
--mount=type=ssh
Dawoud: is it
--ssh=default=$SSH_AUTH_SOCK
or--ssh default=$SSH_AUTH_SOCK
?Kyle:
--ssh=default=$SSH_AUTH_SOCK
– what is getting escaped?Max: we are also interested in this answer
Moisés González do you have a POC to try?
RG is interested in helping to implement or test the proposal
In our non-tutor deployments we extensively used SSH key for private repos cloning
Max’s verdaccio plugin: https://discuss.openedx.org/t/local-npm-registry-for-mfe-builds/13526
Issues
doesn’t work without
host.docker.internal
hostname
Important insight- timing
very good time for low parallelism (max 2) – 12min
worst time for high parallellism (no max) – 27min
Regis
Caching npm deps – good.
Regis runs verdaccio on a remote server.
Getting issues with timeouts/connection.
not sure if connection issue is b/w verdaccio and tutor or verdaccio and npm
Running a container before a build – not supported in tutor naturally
Doesn’t make sense to have separate verdaccios for nightly vs master
Decouple launching verdaccio instance vs configuring NPM_REGISTRY
starting verdaccio could be managed by docker, not docker-compose
tutor local do verdaccio-start
?
Max - similarly, don’t like idea of having a Tutor service that’s not truly a tutor service
Max - why a separate machine for verdaccio?
regis - didn’t know about
host.docker.internal
Max - it doesn’t seem to be resource intensive for his machine
Max’s coworker disagrees
Regis - it uses …. MB memory and … MB disk space. Not very intensive in either case
What does the api look like?
Regis:
tutor local do verdaccio start
would start verdaccio. would print an instruction to run
tutor config save --set NPM_REGISTRY=host.docker.internal:$PORT
Kyle:
tutor verdaccio start
, same as aboveMax:
tutor npm-mirror start
, same as aboveMax: plugin or core?
Regis: try out a couple weeks , and see whether to add it to core?
Kyle: worried about resource usage depending on different machines, want it to be optional
Max: parallellism, why does unbounded parallelism slow it down?
Zameel - non-latin languages and mysql 5.7
course outline was not getting saved to mysql
mysql was using latin-1 instead of utf-8
Zameel: https://discuss.openedx.org/t/load-missing-lesson-information-in-mysql-table/10034/11
Regis: In Redwood (v18), default collation changed from utf8 to utf8mb4
Zameel: could we have tutor docs warning about this?
Zameel: to fix, we made a mysql backup, changed encoding from latin-1 to utf-8. encoding, not character set.
Dawoud: when tutor configures mysql, we pass in args to ensure it’s utf8: https://github.com/overhangio/tutor/blob/palm/tutor/templates/local/docker-compose.yml#L43 . But these are not passed to the managed service.
Zameel: mysql 5.7 by default uses
Kyle: “bare metal” edx-platform docs: https://github.com/openedx/edx-platform?tab=readme-ov-file#bare-metal-advanced
a note there that flags must be passed in to configure mysql to use utf-8 and utf8mb4 , rather than latin-1
not neccessary for mysql 8, though.
Nihad: https://discuss.openedx.org/t/unable-to-import-open-edx-demo-course-in-redwood-release/13455 ,
...