Simplify mapped Ubuntu/Unraid OS labels in dashboard UI#4
Conversation
Agent-Logs-Url: https://github.com/cbulock/server-dashboard/sessions/7f52f8e1-f454-4ede-874c-6e2296eea7c8 Co-authored-by: cbulock <176519+cbulock@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
This PR simplifies OS label rendering in the dashboard by shortening the mapped Ubuntu/Unraid strings and removing appended kernel version details from mapped output.
Changes:
- Updated
formatKernelMappedOsto return only the mapped label (no appendedLinux <kernel>). - Simplified Unraid kernel-to-OS mappings for certain exact kernel versions to return
x-family labels. - Replaced Ubuntu “kernel family” mapping labels with direct
Ubuntu <release> LTSlabels.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
web/src/kernelMappings.js |
Updates Ubuntu/Unraid kernel mapping labels to be more concise. |
web/src/App.vue |
Changes mapped OS formatting to stop appending the kernel version to mapped labels. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| "5.4": "Ubuntu 20.04 LTS", | ||
| "5.15": "Ubuntu 22.04 LTS", | ||
| "6.2": "Ubuntu 22.04 LTS", | ||
| "6.5": "Ubuntu 22.04 LTS", | ||
| "6.8": "Ubuntu 24.04 LTS", |
There was a problem hiding this comment.
Implemented in commit 54a47ed.
Changes made:
- Parse Ubuntu release directly from
osDescriptionwhen present (e.g.,Ubuntu 24.04.3 LTS->Ubuntu 24.04 LTS) to avoid incorrect kernel-only inference. - Updated ambiguous Ubuntu kernel fallbacks to non-single-release labels (
5.15 -> Ubuntu 20.04/22.04 LTS,6.2/6.5 -> Ubuntu 22.04+).
Screenshot: /tmp/os-label-ui.png
| "6.6.78": "Unraid OS 7.0.1", | ||
| "6.12.24": "Unraid OS 7.1.0-7.1.4", | ||
| "6.12.54": "Unraid OS 7.2.0-7.2.4", | ||
| "6.12.24": "Unraid OS 7.1.x", | ||
| "6.12.54": "Unraid OS 7.2.x", | ||
| "6.12.85": "Unraid OS 7.2.5", | ||
| "6.12.87": "Unraid OS 7.2.6", |
There was a problem hiding this comment.
Implemented in commit dd9d3d9.
I removed the 6.12 series fallback mapping so unlisted 6.12.* kernels no longer resolve to the ambiguous Unraid OS 7.1.x-7.2.x label.
Screenshot: /tmp/os-label-ui-unraid-612.png
Agent-Logs-Url: https://github.com/cbulock/server-dashboard/sessions/131b465d-d313-4501-b5f5-1f3d2f88f931 Co-authored-by: cbulock <176519+cbulock@users.noreply.github.com>
|
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Agent-Logs-Url: https://github.com/cbulock/server-dashboard/sessions/e5d1683f-279c-405c-9d50-eab2e188369b Co-authored-by: cbulock <176519+cbulock@users.noreply.github.com>
@copilot can you continue this work but just ignoring whatever you were trying to do font-wise? |
|
Just as a heads up, I was blocked by some firewall rules while working on your feedback. Expand below for details. Warning Firewall rules blocked me from connecting to one or more addresses (expand for details)I tried to connect to the following addresses, but was blocked by firewall rules:
If you need me to access, download, or install something from one of these locations, you can either:
|
Yes — I continued without any font-related changes and completed the mapping update in Current behavior keeps concise labels and removes the ambiguous Unraid Screenshot: |
Uh oh!
There was an error while loading. Please reload this page.