Skip to content

UUID migration#4614

Open
prakhar-singh1928 wants to merge 2 commits intomainfrom
dev_int_120_uuid
Open

UUID migration#4614
prakhar-singh1928 wants to merge 2 commits intomainfrom
dev_int_120_uuid

Conversation

@prakhar-singh1928
Copy link
Copy Markdown
Collaborator

@prakhar-singh1928 prakhar-singh1928 commented May 6, 2026

🔗 Related Issue


Partially Closes: https://github.ibm.com/contextforge-org/mcp-context-forge/issues/120
Container should run with UID > 10000 to avoid host conflicts

📝 Summary

Migrates container UID from 1001 to 10001 to avoid conflicts with host system users and improve security posture. This change affects all container deployments (Docker, Kubernetes, Helm) and includes comprehensive permission handling for test environments.

Key Changes:

  • Updated Containerfile.lite to use UID/GID 10001
  • Updated Helm chart security contexts (deployment, migration job)
  • Updated Docker Compose files across all profiles
  • Fixed test migration scripts with robust permission handling
  • Created comprehensive migration guide for existing deployments
  • Updated MCP server templates and documentation

🏷️ Type of Change

  • Bug fix
  • Feature / Enhancement
  • Documentation
  • Refactor
  • Chore (deps, CI, tooling)
  • Other (describe below)

🧪 Verification

Check Command Status
Lint suite make lint ✅ PASS
Unit tests make test ✅ PASS
Coverage ≥ 80% make coverage ✅ PASS

✅ Checklist

  • Code formatted (make black isort pre-commit)
  • Tests added/updated for changes
  • Documentation updated (if applicable)
  • No secrets or credentials committed

📓 Notes (optional)

Screenshots, design decisions, or additional context.

Signed-off-by: prakhar-singh1928 <prakhar.singh1928@ibm.com>
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.

1 participant