Skip to content

feat(infra): add ktop Kubernetes monitoring tool#28

Open
ldamasio wants to merge 4 commits intomainfrom
claude/implement-ktop-robon-YUX2W
Open

feat(infra): add ktop Kubernetes monitoring tool#28
ldamasio wants to merge 4 commits intomainfrom
claude/implement-ktop-robon-YUX2W

Conversation

@ldamasio
Copy link
Owner

Implement ktop as a top-style resource monitoring tool for the k3s cluster, following the same patterns established for K9s.

Added:

  • Makefile targets: ktop, ktop-ns, ktop-preview
  • Helper scripts: infra/scripts/ktop-ns.sh, infra/scripts/ktop-preview.sh
  • Comprehensive documentation: infra/KTOP-OPERATIONS.md
  • Shell aliases: infra/ktop.aliases.example.sh

Updated:

  • CLAUDE.md: Added ktop to infrastructure tools and commands
  • docs/INDEX.md: Added ktop operations guide reference
  • infra/README.md: Added ktop to Operations & Debugging section

ktop complements K9s by providing quick resource monitoring (CPU, memory) while K9s remains the primary tool for debugging and cluster operations.

See: https://github.com/vladimirvivien/ktop

Summary

  • Briefly explain the problem and the proposed solution.

Changes

  • Key changes in this PR:

Screenshots (optional)

Tests

  • How to verify:
    • cd apps/backend/monolith && ./bin/dj test
    • cd apps/frontend && npm ci && npm test
  • Added/updated tests:

Migrations

  • No schema changes
  • Includes schema migrations
  • Notes:

Backward Compatibility

  • Does this PR introduce breaking changes? Explain mitigation/migration strategy.

Security & Data

  • Any secrets, credentials, or PII involved? If so, describe handling.
  • External calls guarded by flags? (e.g., TRADING_ENABLED=False in dev)

Checklist

  • Scope is focused and documented
  • Code follows project conventions (see docs/DEVELOPER.md)
  • Migrations created (if needed) and rationale explained
  • Tests added/updated and passing locally
  • Docs updated (README/DEVELOPER/MIGRATION_GUIDE as applicable)

Implement ktop as a top-style resource monitoring tool for the k3s cluster,
following the same patterns established for K9s.

Added:
- Makefile targets: ktop, ktop-ns, ktop-preview
- Helper scripts: infra/scripts/ktop-ns.sh, infra/scripts/ktop-preview.sh
- Comprehensive documentation: infra/KTOP-OPERATIONS.md
- Shell aliases: infra/ktop.aliases.example.sh

Updated:
- CLAUDE.md: Added ktop to infrastructure tools and commands
- docs/INDEX.md: Added ktop operations guide reference
- infra/README.md: Added ktop to Operations & Debugging section

ktop complements K9s by providing quick resource monitoring (CPU, memory)
while K9s remains the primary tool for debugging and cluster operations.

See: https://github.com/vladimirvivien/ktop
The client field was missing `client = models.ForeignKey(` causing
an IndentationError in the CI pipeline.
Changed duplicate 'offset' parameter to 'limit' in
TradingIntentRepositoryPort.list_by_client method.
Added missing 'if not client:' and 'return Response(' in
create_trading_intent view.
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

Successfully merging this pull request may close these issues.

2 participants