Skip to content

fix: Replace hardcoded Docker IP with host.docker.internal in Prometheus config - #81

Closed
theinterneti with Copilot wants to merge 10 commits into
agent/copilotfrom
copilot/sub-pr-80
Closed

fix: Replace hardcoded Docker IP with host.docker.internal in Prometheus config#81
theinterneti with Copilot wants to merge 10 commits into
agent/copilotfrom
copilot/sub-pr-80

Conversation

Copilot AI commented Nov 13, 2025 •
edited
Loading

Copy link
Copy Markdown
Contributor

Description

Addresses feedback from PR #80 regarding hardcoded 172.17.0.1 in Prometheus configuration. Replaced with host.docker.internal for cross-platform compatibility across Docker Desktop (Mac/Windows) and Linux environments.

Updated targets:

  • tta-live-metrics job (target + relabel replacement)
  • tta-primitives job
  • tta-observability job

Aligns configuration with existing tta-applications job pattern.

Type of Change

  • fix: Bug fix
  • feat: New feature
  • docs: Documentation update
  • refactor: Code refactoring
  • test: Test additions/improvements
  • chore: Maintenance/tooling

Quality Checklist

  • Code follows project style guidelines (Ruff, Pyright)
  • Self-review completed
  • Tests added/updated (if applicable)
  • All tests passing locally
  • Documentation updated (if applicable)
  • No secrets or sensitive data included
  • Commit messages follow Conventional Commits format

Testing

Configuration change only - validated YAML structure and verified no hardcoded IPs remain.

Unit Tests

  • Tests passing: N/A (config-only change)
  • Coverage: N/A

Integration Tests (if applicable)

  • Manual testing completed: YAML structure validated, grep verified no 172.17.0.1 instances remain

Test Evidence

$ grep -n "172.17.0.1" config/prometheus/prometheus.yml
✓ No more hardcoded 172.17.0.1 IPs found

$ grep -r "172.17.0.1" config/
✓ No hardcoded IPs in config directory

Related Issues

Addresses PR #80 feedback: #80 (comment)

Breaking Changes

None

Screenshots (if applicable)

N/A - Configuration file change

Deployment Notes

Requires Docker environment to support host.docker.internal. On Linux, this may require:

  • Docker 20.10+ with --add-host=host.docker.internal:host-gateway
  • Or manual /etc/hosts configuration

Reviewer Checklist

  • Code quality meets standards
  • Tests are comprehensive
  • Documentation is clear
  • Breaking changes are justified
  • Commit history is clean

✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

theinterneti and others added 9 commits November 11, 2025 14:24
@theinterneti
feat(observability): Session 1 complete - fix trace context propagati…
6d1916c 
…on and validate metrics

Session 1 Complete (3/3 tasks):
- Fix trace context propagation with root span wrapper pattern
- Validate core metrics export (7 metrics in Prometheus)
- Comprehensive validation suite (13/13 tests passed)

Changes:
- Add setup_metrics() function to observability_demo.py
- Add root span wrappers to all workflow.execute() calls
- Fix type safety for optional workflow_id field
- Create validation script and session documentation

Impact:
- Full distributed tracing now working (5-10 span hierarchies)
- Waterfall views available in Jaeger for bottleneck identification
- End-to-end observability validated and production-ready
- Foundation complete for Sessions 2-5 (dashboards, testing, docs)

Validation Results:
- Jaeger: 5+ spans per trace with perfect parent-child linking
- Prometheus: 60+ metric series exported on port 9464
- Services: All healthy (Jaeger, Prometheus, OTLP, Metrics)
- Tests: 13/13 passed (trace hierarchy, metrics, labels, health)

Deferred to Session 3:
- tta_llm_tokens_total metric (requires LLM primitive instrumentation)
- tta_router_decisions_total metric (requires Router instrumentation)

Documentation: OBSERVABILITY_SESSION1_COMPLETE.md
@theinterneti
feat(logseq): implement Knowledge Graph System v2.0
7bf64f4 
- Add comprehensive taxonomy (5 primitive types: C/P/D/I/S)
- Create production-ready template system (templates.md)
- Implement hierarchical namespace (TTA.dev/*)
- Add universal + type-specific property schema
- Create 4 example pages (10,500 lines total)
- Add complete migration guide for existing pages
- Document system architecture and usage

Deliverables:
- logseq/KNOWLEDGE_GRAPH_SYSTEM_README.md (630 lines)
- logseq/templates.md (530 lines, 5 templates)
- logseq/MIGRATION_GUIDE.md (550 lines)
- 4 example pages: TypeSafety, OrchestratorConfig, E2BPrimitive, ObservabilityStack
- LOGSEQ_KNOWLEDGE_GRAPH_IMPLEMENTATION_COMPLETE.md (completion report)

Ready for Cline to migrate existing 200+ pages and create [P] Primitive examples.

Note: Bypassing pre-commit validation as this commit is primarily documentation.
Code validation issues will be addressed in separate commit.
@theinterneti
Fix locale warnings in Git operations and VS Code
74be9b3 
- Added LC_ALL=C.UTF-8 and LANG=C.UTF-8 to pre-commit hook
- Added locale environment variables to VS Code terminal settings
- Added pragma: allow-asyncio to ParallelPrimitive (primitive implementation needs asyncio.gather)
- Fixed locale in shell profile (~/.zshrc, ~/.profile, systemd user environment)

This resolves the 'setlocale: LC_ALL: cannot change locale (en_US.UTF-8)' warning
that appeared in VS Code's Source Control panel and Git operations.
@theinterneti
feat: Add TTA.dev observability scripts and metrics server
e59afeb 
- Implemented Persistent Metrics Server for continuous sample workflow metrics.
- Created observability status check script to verify Docker and service health.
- Developed setup scripts for Grafana dashboards and observability infrastructure.
- Added professional observability setup script for production-grade monitoring.
- Introduced simple live metrics server with direct Prometheus integration.
- Created working metrics demo to generate and export metrics via Prometheus.
- Enhanced start metrics server script for persistent metrics export.
@theinterneti
feat: Universal LLM Architecture with budget-aware multi-provider sup…
68f3671 
…port

Implements comprehensive LLM integration supporting:

Core Features:
- UniversalLLMPrimitive base class for multi-provider operations
- Budget profiles: FREE (/usr/bin/zsh), CAREFUL (-50/month), UNLIMITED
- Auto-detect agentic coders (Copilot, Augment, Cline)
- Model routing based on complexity + budget
- Cost tracking with justification logging
- Quality threshold decisions (85% default)

Budget Management:
- FREE mode: Gemini/Kimi/DeepSeek only
- CAREFUL mode: 50/50 free/paid split with justification required
- UNLIMITED mode: Best model always, cost tracked but not limiting
- Track WHY paid chosen over free
- Alert at 80% budget threshold
- Budget enforcement with fallback to free

Architecture:
- Modality-agnostic (VS Code, CLI, GitHub, browser)
- Provider abstraction (OpenAI, Anthropic, Google, OpenRouter, HF)
- User control over budget decisions
- Empirical model selection framework

Documentation:
- UNIVERSAL_LLM_ARCHITECTURE.md - Complete design
- FREE_MODEL_SELECTION.md - Model recommendations guide
- UNIVERSAL_LLM_ARCHITECTURE_QUESTIONS.md - Requirements questionnaire
- UNIVERSAL_LLM_IMPLEMENTATION_PROGRESS.md - Implementation status

Package Structure:
- packages/tta-dev-integrations/llm/ - LLM primitives module
- Enums: UserBudgetProfile, CoderType, ModalityType, ModelTier
- Models: LLMRequest, LLMResponse, CostJustification
- Base: UniversalLLMPrimitive (abstract, awaiting coder implementations)

Next Steps:
- CopilotPrimitive (Claude Sonnet 3.5, complex work)
- ClinePrimitive (Gemini/Kimi/DeepSeek, free tier focus)
- AugmentPrimitive (Claude Sonnet 3.5, parallel comparison)

Based on user requirements:
- 50% free tier usage (Gemini, Kimi, DeepSeek)
- 50% paid usage (Claude for complex work)
- User in control of all budget decisions
- Vibe coder focused (FREE tier support critical)
@theinterneti
Update config/alertmanager/alertmanager.yml
157ad7d 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@theinterneti
Update config/alertmanager/alertmanager.yml
32d4828 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@theinterneti
Update config/prometheus/rules/recording_rules.yml
48a65a0 
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Initial plan
929d5e7
Copilot AI mentioned this pull request Nov 13, 2025
Closed
@theinterneti
fix: Replace hardcoded Docker IP with host.docker.internal for cross-…
845e90b 
…platform compatibility

Co-authored-by: theinterneti <169108167+theinterneti@users.noreply.github.com>
Copilot AI changed the title [WIP] Address feedback on Universal LLM Architecture PR fix: Replace hardcoded Docker IP with host.docker.internal in Prometheus config Nov 13, 2025
Copilot AI requested a review from theinterneti November 13, 2025 00:32
@theinterneti theinterneti added bug Something isn't working observability Observability, tracing, metrics, and monitoring P1 High priority - Should have for production quality help wanted Extra attention is needed labels Nov 13, 2025
@theinterneti

theinterneti commented Nov 13, 2025

Copy link
Copy Markdown
Owner

@app/copilot-swe-agent Thank you for this contribution. This PR aligns with our new framework-first direction and has been labeled for the current milestone. This PR has complex merge conflicts with main after the recent refactor and requires manual review.

@theinterneti
theinterneti deleted the copilot/sub-pr-80 branch November 16, 2025 20:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@theinterneti theinterneti Awaiting requested review from theinterneti

Assignees

@theinterneti theinterneti

Copilot code review Copilot

Labels

bug Something isn't working help wanted Extra attention is needed observability Observability, tracing, metrics, and monitoring P1 High priority - Should have for production quality

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@theinterneti