feat: add Container Stats extension v1.0.1#1052
feat: add Container Stats extension v1.0.1#1052djhenry wants to merge 1 commit intopodman-desktop:mainfrom
Conversation
djhenry
requested review from
MarsKubeX and
feloy
and removed request for
a team
|
Warning Rate limit exceeded
⌛ How to resolve this issue?After the wait time has elapsed, a review can be triggered using the We recommend that you space out your commits to avoid hitting the rate limit. 🚦 How do rate limits work?CodeRabbit enforces hourly rate limits for each developer per organization. Our paid plans have higher rate limits than the trial, open-source and free plans. In all cases, we re-allow further reviews after a brief timeout. Please see our FAQ for further information. ⛔ Files ignored due to path filters (1)
📒 Files selected for processing (3)
📝 WalkthroughWalkthroughAdds a new Podman Desktop extension entry for Container Stats (extension-stats) by publisher dhenry, version 1.0.1: metadata registered in the extensions catalog (duplicated entry), plus an Apache-2.0 LICENSE and a comprehensive README under the extension's path. Changes
Estimated code review effort🎯 2 (Simple) | ⏱️ ~12 minutes 🚥 Pre-merge checks | ✅ 3✅ Passed checks (3 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
🧹 Nitpick comments (1)
static/api/extensions/dhenry/extension-stats/1.0.1/README.md (1)
43-49: Specify languages for fenced code blocks.markdownlint flags missing language identifiers; add
text(or a better fit) to both blocks.💡 Suggested fix
- ``` + ```text ghcr.io/djhenry/extension-stats:latest ```-``` +```text extension-stats/ ├── packages/ │ ├── shared/ # Shared types and utilities │ ├── backend/ # Extension backend (Node.js) │ └── frontend/ # Dashboard UI (Svelte 5) ├── docs/ # Architecture and development specs └── .github/workflows/ # CI/CD pipeline -``` +```Also applies to: 91-99
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@static/api/extensions/dhenry/extension-stats/1.0.1/README.md` around lines 43 - 49, The fenced code blocks in README.md are missing language identifiers which markdownlint flags; update both fenced blocks by adding an appropriate language tag (e.g., add ```text before the line "ghcr.io/djhenry/extension-stats:latest" and add ```text before the ASCII tree that begins "extension-stats/") so both code fences include a language specifier and then close them with ``` as currently present.
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Nitpick comments:
In `@static/api/extensions/dhenry/extension-stats/1.0.1/README.md`:
- Around line 43-49: The fenced code blocks in README.md are missing language
identifiers which markdownlint flags; update both fenced blocks by adding an
appropriate language tag (e.g., add ```text before the line
"ghcr.io/djhenry/extension-stats:latest" and add ```text before the ASCII tree
that begins "extension-stats/") so both code fences include a language specifier
and then close them with ``` as currently present.
ℹ️ Review info
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
static/api/extensions/dhenry/extension-stats/1.0.1/icon.pngis excluded by!**/*.png
📒 Files selected for processing (3)
static/api/extensions.jsonstatic/api/extensions/dhenry/extension-stats/1.0.1/LICENSEstatic/api/extensions/dhenry/extension-stats/1.0.1/README.md
djhenry
force-pushed
the
add-extension-stats
branch
from
8366a28 to
c53d753
Compare
djhenry
changed the title
There was a problem hiding this comment.
Actionable comments posted: 2
🧹 Nitpick comments (1)
static/api/extensions/dhenry/extension-stats/1.0.1/README.md (1)
142-145: Consider a more neutral placeholder in the commit example.“amazing feature” is informal; swapping it for a more neutral placeholder keeps the doc tone consistent.
🤖 Prompt for AI Agents
Verify each finding against the current code and only fix it if needed. In `@static/api/extensions/dhenry/extension-stats/1.0.1/README.md` around lines 142 - 145, Replace the informal commit example "feat: add amazing feature" with a neutral placeholder; update the example commit line in the README to use a generic token such as "feat: add <feature-name>" or "feat: add your-feature" so the commit message remains professional and consistent with the document tone (locate the example list lines showing the git commands and change the commit message text accordingly).
🤖 Prompt for all review comments with AI agents
Verify each finding against the current code and only fix it if needed.
Inline comments:
In `@static/api/extensions/dhenry/extension-stats/1.0.1/README.md`:
- Around line 41-49: The README has fenced code blocks without language
identifiers; update each fenced block that shows the OCI image and the directory
tree to include a language tag (for example use "text") so markdownlint stops
flagging them — specifically add the language identifier to the block containing
"ghcr.io/djhenry/extension-stats:latest" and to the block showing the
"extension-stats/" directory tree (also apply the same change to the similar
blocks referenced at the later section around lines 89-99); keep the content
unchanged, only prepend the opening ``` with the chosen language (e.g.,
```text).
- Around line 37-51: The README.md in the extension bundle references local
relative docs (e.g., INSTALL-SIMPLE.md, INSTALLATION.md,
docs/ARCHITECTURE-1_0_0.md) which will break when installed from the OCI
registry; update those references in README.md to use absolute upstream
repository URLs (or alternatively include the referenced files in the catalog
bundle) so the links resolve outside the source tree—search for the strings
"INSTALL-SIMPLE.md", "INSTALLATION.md", and "docs/ARCHITECTURE-1_0_0.md" in
README.md and replace them with full HTTPS links to the files in the GitHub repo
or add the missing files to the bundle.
---
Nitpick comments:
In `@static/api/extensions/dhenry/extension-stats/1.0.1/README.md`:
- Around line 142-145: Replace the informal commit example "feat: add amazing
feature" with a neutral placeholder; update the example commit line in the
README to use a generic token such as "feat: add <feature-name>" or "feat: add
your-feature" so the commit message remains professional and consistent with the
document tone (locate the example list lines showing the git commands and change
the commit message text accordingly).
ℹ️ Review info
Configuration used: Organization UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (1)
static/api/extensions/dhenry/extension-stats/1.0.1/icon.pngis excluded by!**/*.png
📒 Files selected for processing (3)
static/api/extensions.jsonstatic/api/extensions/dhenry/extension-stats/1.0.1/LICENSEstatic/api/extensions/dhenry/extension-stats/1.0.1/README.md
Real-time container and host resource monitoring dashboard providing per-container CPU, memory, network I/O, block I/O, and PID metrics along with host system overview. OCI image: ghcr.io/djhenry/extension-stats:1.0.1 Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com> Signed-off-by: David Henry <dhenry@redhat.com>
djhenry
force-pushed
the
add-extension-stats
branch
from
c53d753 to
36fc2e8
Compare
There was a problem hiding this comment.
Hey, that's an interesting extension, but I'm wondering if we would not duplicate the data, currently in Containers > Details we have an existing component displaying the CPU / Memory usage of the component
Containers -> Details
Containers Stats extension
Similarly, the Host System Overview seems pretty familiar with the Settings > Resources page where we display usage for a given machine / connection when available.
The Containers > Details page is pretty limited, and for example does not allow you to graph, so maybe this is something we may want to have improved inside Podman Desktop, but would need to be investigate and study!
Hi Axel, thanks for your feedback! 
|
Current we don't have a global view, you can check containers usage by opening a container details (This can be done by clicking the name of the container in the table). |
2 participants
Summary
dhenry/extension-statsv1.0.1)ghcr.io/djhenry/extension-stats:1.0.1Files added
static/api/extensions/dhenry/extension-stats/1.0.1/icon.pngstatic/api/extensions/dhenry/extension-stats/1.0.1/README.mdstatic/api/extensions/dhenry/extension-stats/1.0.1/LICENSEstatic/api/extensions.jsonwith new extension entryVerification
verify_extensions.shpasses all checks (files present, version/name/publisher match OCI image)