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

Remove webui from Saturn L2 node #285

Closed
2 tasks
bajtos opened this issue Nov 23, 2022 · 1 comment
Closed
2 tasks

Remove webui from Saturn L2 node #285

bajtos opened this issue Nov 23, 2022 · 1 comment
Labels
code-cleanup Tasks making our codebase easier to maintain

Comments

@bajtos
Copy link
Member

bajtos commented Nov 23, 2022

L2-node is bundling filecoin-saturn/node-webui and serving it over HTTP, so that people running L2 nodes (e.g. inside Station) can access the dashboard.

This was a very early implementation of Saturn GUI for the Station. The Station was rendering node-webui served by L2-node inside an iframe. A lot has changed since then, the Station no longer renders any webui from L2-node.

I think we can safely remove node-webui from L2-node:

  • It was a quick hack to begin with.
  • If we want L2 nodes to show webui, then we should start with UX research to find out what kind of dashboard makes the most sense for L2 operators. It is likely to be different from a dashboard for L1 operators.

Our developer docs for the Station hasn’t been updated after we stopped rendering Saturn webui, so it illustrates well the original architecture: https://github.com/filecoin-station/filecoin-station/blob/main/docs/DEVELOPING.md

We don't have a production need for the L2 single node stats webui currently. Once L2s provide value outside of Station, people might consider running L2s in headless mode, and then might want to see the stats UI.

Tasks

@bajtos bajtos added the code-cleanup Tasks making our codebase easier to maintain label Nov 23, 2022
@bajtos
Copy link
Member Author

bajtos commented Jan 8, 2024

Opened #1239 to update our Developer Guide.

I think nobody cares about Saturn's L2 node anymore. Closing this issue as not planned.

@bajtos bajtos closed this as not planned Won't fix, can't repro, duplicate, stale Jan 8, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
code-cleanup Tasks making our codebase easier to maintain
Projects
None yet
Development

No branches or pull requests

1 participant