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

Discovery: How could we include new MFEs in releases without slowing down Tutor builds? #152

Closed
kdmccormick opened this issue Apr 18, 2022 · 3 comments
Milestone

Comments

@kdmccormick
Copy link
Member

kdmccormick commented Apr 18, 2022

Context

The tutor-mfe plugin is our method for including micro-frontends to the named releases. It currently builds four MFEs into its Docker image by default, leaving the end user the ability to build in extra MFEs. Each additional MFE significantly increases the plugin's build time and image size.

We need Open edX releases to work "out-of-the-box", so all MFEs that are required to in order for the platform to work need to be built into the tutor-mfe image by default. As more MFEs become default and old frontends removed, the build time and size of the MFE image will continue to balloon.

In this Nutmeg issue, we discussed whether the Authn MFE should be included in Nutmeg. One of the arguments against including it was that it'd increase tutor-mfe build time and size. Going forward, we'd like if this didn't need to be such a consideration.

The solution to this issue may be related to the dynamic MFE config idea which is being floated at the frontend WG. There also may be salient points in the recent MFE retrospective.

Acceptance

Investigate the problem and list potential solution(s).

@kdmccormick
Copy link
Member Author

Since this is issue is a longer-term one and will require some discovery, I think it'd be fine to table it to post-Nutmeg.

@arbrandes
Copy link
Contributor

eduNext is working on this, as outlined in this exporation document. (In particular, Epic 1.)

@BbrSofiane BbrSofiane removed this from the Nutmeg Release milestone May 27, 2022
@BbrSofiane BbrSofiane added this to the Olive Release Candidate milestone Aug 13, 2022
@kdmccormick
Copy link
Member Author

Late update: This is being solved in time for Olive (🤞🏻) with:

Closing this since the discovery work has been done for a while.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

No branches or pull requests

3 participants