-
Notifications
You must be signed in to change notification settings - Fork 35
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
Make AppDefinition
s more powerful in landing pages
#349
Comments
Hi @iyannsch ,
This is true. However, the current approach and landing page are more of a starting point rather than a production-ready page. IMO for this use case the current approach with the Helm chart is sufficient.
Similar to above, the current approach is IMO sufficient for the provided landing page but not optimal for more elaborate landing pages.
I'm not sure if this is required for the example landing page but it certainly would be helpful to enable custom landing pages to query available app definitions. The operator should not be involved in this, though. The operator does not and will not offer any directly accessible API. It only handles custom resources (i.e. app definitions, sessions, workspaces). If you are interested, you can find more information on the operator pattern in the Kubernetes documentation. Instead, an endpoint could be added to the REST service to expose available app definitions: The REST service is the external entry point to interact with Theia Cloud. For instance, the landing page calls it to launch a new session.
Adding an explicit property Regarding the key-value pairs. This should already be possible via the |
Thanks for your thoughts, I agree with all of your aspects! The current state of Theia (Cloud) is already awesome and one cannot implement everything in the first run - I just wanted to hint at possible options to make it even better from an outside perspective. Looking forward to hearing decisions or takeaways from you internal discussion about it :) |
We'll discuss this next week as some project members are currently on vacation :) |
Hi @iyannsch, |
Awesome! 🔥 |
### App definition endpoint - Add AppDefinitionResource with endpoint to list all app definitions. The endpoint is restricted to authenticated users but is still available in anonymous mode ### Sensitive data redaction - Add `SensitiveData` annotation to mark properties that should not be serialized publicly by Jackson. - Add a corresponding serializer and serializer modifiert for Jackson and register the modifier in the service - Add unit tests for the serializer ### Javascript API - Update openapi.json from service - Regenerate api code - Add AppDefinitions namespace with function to list app definitions ### Testing Page - Add button to get app definitions - Apply formatting rules to App.tsx by saving it - Minor fix in example keycloak URL to remove obsolete `/auth` ### Misc - Add mockito dependency to the common maven module
### App definition endpoint - Add AppDefinitionResource with endpoint to list all app definitions. The endpoint is restricted to authenticated users but is still available in anonymous mode ### Sensitive data redaction - Add `SensitiveData` annotation to mark properties that should not be serialized publicly by Jackson. - Add a corresponding serializer and serializer modifiert for Jackson and register the modifier in the service - Add unit tests for the serializer ### Javascript API - Update openapi.json from service - Regenerate api code - Add AppDefinitions namespace with function to list app definitions ### Testing Page - Add button to get app definitions - Apply formatting rules to App.tsx by saving it - Minor fix in example keycloak URL to remove obsolete `/auth` ### Misc - Add mockito dependency to the common maven module
Is your feature request related to a problem? Please describe.
Currently, a landing page only knows about those appDefinitons that are configured for use in the helm chart under
landingPage.additionalApps
. This introduces two problematic aspects and leaves a lot of potential aside.Administrative effort
When a new
AppDefinition
is introduced to the cluster, it's not available whatsoever. An administrator has to make manually add it to the helm chart using the resource's exactname
and decide on alabel
. This process is cumbersome and prone to typos.Loss of information
While the landing page and the respective section in the helm chart only accept a
name
andlabel
per additional app, theAppDefinition
in K8s offers way more information. Some of it might not be useful for the landing page, other might provide value (resource limits, image name). Due to the limitation of having to manually linklandingPage.additionalApps
with the existing resources, information is lost.Describe the solution you'd like
We are currently building a landing page suited for multiple use cases introducing a lot of complexity here. For that it would be very helpful to make the
AppDefinitions
more powerful and push more decision making power to the landing page. I think of the following features:Auto-Discovering suitable AppDefinitions
Instead of having to manually configure
additionalApps
in the helm chart, I propose the landing page to automatically discover suitableAppDefinition
s in the cluster/namespace and load them alongside their complete properties. This would solve both problems described above.In the current Theia Cloud setup, the
operator
already has API access to K8s and could provide the landing page with internal information about available definitions. This introduces the problem that all existing definitions would be visible to a landing page.Add key-value configuration of AppDefinitions
The
AppDefinition
resource could be enlarged even further by add a key-value configuration mechanism utilized during manifest creation. Following the argumentation above, I would propose to add ashowAtLandingpage: false
entry that could decide whether it's auto-discoverable.Apart from this advantage, the mechanism also introduces powerful logic into landing pages, e.g., by multiplexing in respect to path parameters. We could automatically start a blueprint whenever it's provided via the
?appDef
path parameter but show a well-designed overview of all available apps when started without. This overview could be parametrized by (exemplary) key-value information likeDescribe alternatives you've considered
We could fork both the
theia-cloud-helm
andtheia-cloud
repos and enlarge the set of configurable information passed between them (landingPage.additionalApps
). This would unlock all user-facing advantages but still require considerable manual efforts and potential for errors.Cluster provider
No response
Additional information
No response
The text was updated successfully, but these errors were encountered: