[WIP] initial module federation support #3799
Open
+1,381
−494
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
https://issues.redhat.com/browse/RHOAIENG-19055
Description
This PR adds support for module federation but has no default configuration. I will be preparing an on cluster demo alongside more instructions on running a local setup. But for now the intent is to ensure that regular dev and production envs continue to work as intended. I don't expect a reviewer to test module federation at this time.
MODULE_FEDERATION_CONFIG
env var JSON value to add module federation supportMODULE_FEDERATION_CONFIG
is configured. This proxy is used to route frontend requests for both the federated modules and their API requests.start:dev:mf
which will include download type definitions from modules. I'm still experimenting with what is the best approach for working with types.How Has This Been Tested?
I tested this with a local instance of model registry running which was modified to expose some APIs. Then in ODH used the following configuration.
ODH was modified to consume the exposed API and render a portion of the model registry in the UI.
Test Impact
N/A
Request review criteria:
Self checklist (all need to be checked):
If you have UI changes:
After the PR is posted & before it merges:
main