lhumina_code/hero_compute
20
0
Fork 0
Code Issues 12 Pull requests 2 Projects Releases 8 Packages Wiki Activity Actions

update: show node component versions in UI sidebar with hover details #81

Merged
nabil_salah merged 6 commits from development_show_node_versions into development 2026-04-08 12:24:11 +00:00
Conversation 5 Commits 6 Files changed 6 +91 -62
nabil_salah commented 2026-04-07 14:40:42 +00:00
Member
Copy link

Summary

  • Merged version info into /status endpoint — no separate /version route
  • Component versions (hero_compute, my_hypervisor, hero_proc, cloud-hypervisor, mycelium) read from cached central config ( ~/hero/var/config.toml )
  • Added cloud-hypervisor and mycelium to service health checks
  • Sidebar displays hero_compute version; hover card shows all component versions
  • Version info refreshes every 2s via health poll

Changes

  • crates/hero_compute_ui/Cargo.toml: Add toml dependency for config parsing
  • crates/hero_compute_ui/src/server.rs: Merge version data into detailed_health_handler, remove version_handler
  • crates/hero_compute_ui/static/js/dashboard.js: Populate version card from /status response, remove /version fetch
  • crates/hero_compute_ui/static/css/dashboard.css: Version hover card styles
  • crates/hero_compute_ui/templates/partials/sidebar.html: Version hover card markup

Test plan

  • Verify sidebar displays correct hero_compute version
  • Verify hover card shows all 5 component versions

Resolves #76

image

image

## Summary - Merged version info into `/status` endpoint — no separate `/version` route - Component versions (hero_compute, my_hypervisor, hero_proc, cloud-hypervisor, mycelium) read from cached central config ( `~/hero/var/config.toml` ) - Added cloud-hypervisor and mycelium to service health checks - Sidebar displays hero_compute version; hover card shows all component versions - Version info refreshes every 2s via health poll ## Changes - `crates/hero_compute_ui/Cargo.toml`: Add toml dependency for config parsing - `crates/hero_compute_ui/src/server.rs`: Merge version data into detailed_health_handler, remove version_handler - `crates/hero_compute_ui/static/js/dashboard.js:` Populate version card from /status response, remove /version fetch - `crates/hero_compute_ui/static/css/dashboard.css`: Version hover card styles - `crates/hero_compute_ui/templates/partials/sidebar.html`: Version hover card markup ## Test plan - [x] Verify sidebar displays correct hero_compute version - [x] Verify hover card shows all 5 component versions Resolves #76 ![image](/attachments/8ee182dd-5cc1-44a3-bab3-6b51256a1d40) ![image](/attachments/ea6a4177-6bf0-4ee5-bdf5-a303ae79b785)
image.png
15 KiB
image.png
94 KiB
fix(configure): handle Ubuntu 22 virtiofsd setup and complete issue #62
All checks were successful
Test / test (push) Successful in 1m27s
Details
Test / test (pull_request) Successful in 1m49s
Details
b13dfb0451 
- Complete issue #62 by making configure.sh install qemu fallback packages and ensure virtiofsd is available in PATH via symlink check/warn flow.

This commit is the second part of the fix.
The first part was done in:
a711a138b2

Signed-off-by: Nabil-Salah <nabil.salah203@gmail.com>
feat(ui): display component versions in sidebar with hover tooltip
All checks were successful
Test / test (push) Successful in 1m24s
Details
Test / test (pull_request) Successful in 1m48s
Details
7ec370bffd 
- Backend: /version endpoint returns structured JSON with all 5 component versions
  (hero_compute, my_hypervisor, hero_proc, cloud-hypervisor, mycelium)
- hero_compute uses compile-time version + git hash (env!() macros)
- External components queried via shell commands with 5s timeout
- Missing binaries display "not installed"
- Frontend: sidebar shows hero_compute version, hover shows all components
- Closes #76

Signed-off-by: Nabil-Salah <nabil.salah203@gmail.com>
mahmoud commented 2026-04-08 07:21:46 +00:00
Owner
Copy link

Good job, I think it might be better if we used a UI card that displays on hover instead of using the element title property, for example, see the status on the navbar

Good job, I think it might be better if we used a UI card that displays on hover instead of using the element title property, for example, see the status on the navbar
mahmoud requested changes 2026-04-08 07:38:01 +00:00
Dismissed
SKILL.md Outdated
@ -0,0 +1,1032 @@
---
mahmoud commented 2026-04-08 07:32:11 +00:00
Owner
Copy link

This file should not be committed

This file should not be committed
crates/hero_compute_ui/build.rs Outdated
@ -1,4 +1,17 @@
fn main() {
// Embed git hash at compile time so it works in production (no .git dir).
let git_hash = std::process::Command::new("git")
mahmoud commented 2026-04-08 07:37:55 +00:00
Owner
Copy link

Let's use a better way, i would suggest that we remove the hash since we already have the release version

Let's use a better way, i would suggest that we remove the hash since we already have the release version
refactor: remove hash viewing and use a UI card that displays on hover
All checks were successful
Test / test (pull_request) Successful in 1m24s
Details
Test / test (push) Successful in 1m56s
Details
bfbab6c9dc 
Signed-off-by: Nabil-Salah <nabil.salah203@gmail.com>
fix: overlay components
All checks were successful
Test / test (pull_request) Successful in 3m15s
Details
Test / test (push) Successful in 3m16s
Details
2bcb414af7 
Signed-off-by: Nabil-Salah <nabil.salah203@gmail.com>
nabil_salah force-pushed development_show_node_versions from 2bcb414af7
All checks were successful
Test / test (pull_request) Successful in 3m15s
Details
Test / test (push) Successful in 3m16s
Details
to 785fbe3212
All checks were successful
Test / test (push) Successful in 2m25s
Details
Test / test (pull_request) Successful in 2m26s
Details
2026-04-08 10:25:22 +00:00 Compare
mahmoud commented 2026-04-08 10:58:16 +00:00
Owner
Copy link

As we now have the central config, i would recommend that we use it to get the versions, no need for the hash

As we now have the central config, i would recommend that we use it to get the versions, no need for the hash
feat: use central config to fetch versions and remove: version handler
Some checks failed
Test / test (pull_request) Failing after 1m19s
Details
Test / test (push) Failing after 1m45s
Details
0e960f8218 
Signed-off-by: Nabil-Salah <nabil.salah203@gmail.com>
fix: hero compute to get the version from central config
Some checks failed
Test / test (pull_request) Failing after 1m25s
Details
Test / test (push) Failing after 1m52s
Details
c1d916826f 
Signed-off-by: Nabil-Salah <nabil.salah203@gmail.com>
mahmoud approved these changes 2026-04-08 12:17:06 +00:00
Dismissed
mahmoud dismissed mahmoud's review 2026-04-08 12:17:32 +00:00
fix: code fmt
All checks were successful
Test / test (pull_request) Successful in 1m26s
Details
Test / test (push) Successful in 1m55s
Details
0480a4f779 
Signed-off-by: Nabil-Salah <nabil.salah203@gmail.com>
mahmoud approved these changes 2026-04-08 12:23:51 +00:00
nabil_salah merged commit 670770c8ca into development 2026-04-08 12:24:11 +00:00
nabil_salah deleted branch development_show_node_versions 2026-04-08 12:24:11 +00:00
Sign in to join this conversation.
Reviewers
No reviewers
mahmoud
Labels
Clear labels   
prio_critical

   
prio_low

   
type_bug

   
type_contact

   
type_issue

   
type_lead

   
type_question

   
type_story

   
type_task
No labels
prio_critical
prio_low
type_bug
type_contact
type_issue
type_lead
type_question
type_story
type_task
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
lhumina_code/hero_compute!81
No description provided.