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

Consolidation of Services Overview #14

Open
JayRovacsek opened this issue Apr 19, 2024 · 3 comments
Open

Consolidation of Services Overview #14

JayRovacsek opened this issue Apr 19, 2024 · 3 comments

Comments

@JayRovacsek
Copy link
Contributor

JayRovacsek commented Apr 19, 2024

Is there any objection to the project considering a consolidation of the services overview rendered in the main.svg output?
I've got two options that could be considered below;

Minimise Current Output (replace behaviour)

I propose we could shift from the current view to one in which all hosts running the same service are listed under the service and details or info elements are visible only in the per-host card to reduce information on the one screen

First example of current view:
20240420_08h16m59s_grim

Compared to a proposed:
20240420_08h16m59s_grim_proposed

Second example of current view (that includes elements of detailed information):
20240420_08h17m19s_grim

Compared to a proposed:
20240420_08h17m19s_grim_proposed

Add Minimised Output (extend behaviour)

I propose we could apply the above option, however output the contents not over main.svg but in a minimal.svg or alike - upside of-course is non-breaking behaviour, however it might lead to increased eval/build times if issues such as #6 are caused by the number and size of images.

An addendum to increasing the default outputs could be increased drv options to build from; such as:

nix build .\#topology.x86_64-linux.config.output
# vs
nix build .\#topology.x86_64-linux.config.minimal-output
# or 
nix build .\#topology.x86_64-linux.config.output.passthru.minimal

Thanks in advance!

@IogaMaster
Copy link
Contributor

I second this

@oddlama
Copy link
Owner

oddlama commented Apr 21, 2024

No objection at all. Consolidating these is a great way to reduce the waste of space. I actually wouldn't create a new output for this, just change the default.

I think consolidation in your first example is pretty much always better than not consolidating it. If there's info involved like in the second example, we could also consolidate it but still show the info behind the hostname? My initial intention for the overview was to have a quick way to see all the relevant domains.

host1            https://nextcloud.example.com
some-other-host  https://service.example.com

What do you think?

@JayRovacsek
Copy link
Contributor Author

That sounds great @oddlama - sorry for the slow reply, if I get a moment to create the PR I'll link it here as well 😁

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

No branches or pull requests

3 participants