Skip to content

Conversation

@avidflyer17
Copy link
Owner

Summary

  • restore load average, CPU, memory, disk and top process sections in web dashboard

Testing

  • ./tests/run.sh

https://chatgpt.com/codex/tasks/task_e_68b05fa6cd98832d9617c4f9c0f96c5c

@avidflyer17 avidflyer17 merged commit 4dcbb3c into dev Aug 28, 2025
1 check passed
@avidflyer17 avidflyer17 deleted the codex/corriger-sections-disparues-suite-refonte branch August 28, 2025 13:59
Copy link

@chatgpt-codex-connector chatgpt-codex-connector bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Codex Review: Here are some suggestions.

Reply with @codex fix comments to fix any unresolved comments.

About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you open a pull request for review, mark a draft as ready, or comment "@codex review". If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex fix this CI failure" or "@codex address that feedback".

Comment on lines +110 to +113
<h2 id="loadTitle"><i class="fa-solid fa-gauge-high heading-icon"></i>Charge moyenne</h2>
<div id="loadCard" class="card" aria-labelledby="loadTitle">
<div id="loadAvg" class="load-container">
<div class="load-main">

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

[P1] Populate restored metric sections from audit data

The new load, CPU, memory, disk and top‑process blocks added here never get populated. loadAndRender in audits/scripts/viewer.js (around lines 469‑475) still only calls renderInfo, renderServices and renderDocker, even though each audit JSON contains load_average, cpu, memory, disks, top_cpu and top_mem keys. As a result these restored sections render only placeholders (-- or empty lists) and the dashboard still lacks the metrics the commit intends to bring back. Consider wiring the viewer script to parse those fields and render into the new elements.

Useful? React with 👍 / 👎.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants