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

feat: auto-mount platform packages at build & run time (very WIP) #36

Conversation

kdmccormick
Copy link
Owner

Super kludgey proof-of-concept of openedx-unsupported/wg-developer-experience#177 (it hardcodes my own Tutor root).

@regisb
Copy link

regisb commented Oct 5, 2023

This is almost to the letter what I also got working on this feature ;) Among the remaining TODOs is making the list of prefixes/regexes extensible via a filter.

Would you like to open a PR or should I do it?

@kdmccormick
Copy link
Owner Author

@regisb I'm a little swamped right now... if you have something close to ready, go for it!

kdmccormick and others added 17 commits October 5, 2023 10:44
…end-sources' into kdmccormick/mount-packages
On Oct. 10, py2neo package was abruptly removed from pypi, GitHub, and
the py2neo website now displays just a super funny meme: https://py2neo.org/

Yes, we should get rid of that dependency, but we are still supposed to
support existing users. So we install py2neo from our fork.
On macOS, building the "openedx-dev" Docker image resulted in an image that required more than 600 GB of disk space. This was due to the `adduser` command which was called with a user ID of 2x10⁹ (on macOS only). This resulted in a very large /var/log/faillog file, hence the image size.

Related upstream discussion: moby/moby#5419
Close openedx-unsupported/wg-developer-experience#178
@kdmccormick
Copy link
Owner Author

closed in favor of overhangio#915

@kdmccormick kdmccormick deleted the kdmccormick/mount-packages branch October 16, 2023 20:23
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.

4 participants