Skip to content

fix: use internal network hostnames and update docs for devpod#388

Merged
nazia-datainn merged 15 commits into
developfrom
fix/devcontainer-network-hostnames
Dec 27, 2025
Merged

fix: use internal network hostnames and update docs for devpod#388
nazia-datainn merged 15 commits into
developfrom
fix/devcontainer-network-hostnames

Conversation

@MohsinHashmi-DataInn
Copy link
Copy Markdown
Contributor

Summary

  • Use internal network hostnames (db, redis) instead of localhost for health checks
  • Update documentation to remove references to deleted scripts (setup-user.sh, generate-hosts.sh)
  • Update README.md and NEW-USER-ONBOARDING.md for DevPod-based workflow

Changes

  • .devcontainer/post-start.sh: Use db and redis hostnames for health checks
  • .devcontainer/post-create.sh: Use internal hostnames in generated config
  • .devcontainer/README.md: Update multi-user setup instructions for DevPod
  • .devcontainer/NEW-USER-ONBOARDING.md: Update for DevPod workflow
  • .devcontainer/proxy/install-traefik-service.sh: Update next steps

Test plan

  • Verify health checks work in devcontainer
  • Verify Traefik integration on dev server
  • Review documentation for accuracy

🤖 Generated with Claude Code

mohsin-wiser and others added 15 commits December 25, 2025 21:07
Configure testing-library for React 18+ and suppress harmless act()
warnings that occur due to async state updates completing after test cleanup.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Upgrade devcontainer database to PostgreSQL 18.1 (released Sept 2025).

Key benefits:
- 3x performance with new Async I/O subsystem
- UUIDv7 function for timestamp-ordered UUIDs
- Virtual generated columns
- Skip scan on multicolumn B-tree indexes
- OAuth 2.0 authentication support

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Adds `init: true` to the devcontainer service which uses tini as PID 1.
This properly reaps zombie processes that accumulate from DevPod sessions.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Adds instructions for configuring username-based container names
when multiple developers share the same remote server.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Upgrade PostgreSQL from 16 to 18
- Add init: true to prevent zombie processes

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- Change pg_isready from localhost to db hostname
- Change redis-cli from localhost to redis hostname
- Update application-local.properties to use db/redis hostnames
- Update README.md troubleshooting commands
- Update proxy docker-compose comments for DevPod workflow

With the explicit internal network, services must use their
hostnames (db, redis) instead of localhost for communication.

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- remove references to deleted setup-user.sh and generate-hosts.sh
- update README.md multi-user section to use DevPod commands
- update NEW-USER-ONBOARDING.md for DevPod-based workflow
- update install-traefik-service.sh next steps

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- update idle-shutdown.sh to find DevPod workspace compose files
- update README.md architecture diagram for internal network
- fix service connectivity table to use db/redis hostnames
- remove obsolete hostname troubleshooting section
- remove non-functional port forwarding for db/redis
- update volume check command for user-specific naming

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
- update README.md volumes section with correct naming convention
- add timeout (60s) to PostgreSQL/Redis health check loops
- fix DEV_USER fallback consistency in post-start.sh
- add nip.io route for Traefik dashboard

🤖 Generated with [Claude Code](https://claude.com/claude-code)

Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
Copy link
Copy Markdown
Contributor

@nazia-datainn nazia-datainn left a comment

Choose a reason for hiding this comment

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

gtg

@nazia-datainn nazia-datainn merged commit a66c8e6 into develop Dec 27, 2025
19 checks passed
@nazia-datainn nazia-datainn deleted the fix/devcontainer-network-hostnames branch December 27, 2025 05:43
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.

3 participants