fix: resolve registry image conflict in docker-compose (#114)
Issue: links directive was failing on podman - Removed unsupported 'links' directive for Podman compatibility <!-- Provide a brief summary of your changes --> It was failing on Podman because Podman doesn't support the links directive that Docker Compose does, and we already using depends_on, which establishes the dependency relationship between services; we can simply remove the links section from your docker-compose.yml file. ## Motivation and Context docker-compose -links deprecated #107 Fix the defect by removing link. ## How Has This Been Tested? Yes. Test by running on both. For Docker run. docker build -t registry . docker compose up For Podman podman build -t registry . podman compose up ## Breaking Changes Yes for podman. ## Types of changes <!-- What types of changes does your code introduce? Put an `x` in all the boxes that apply: --> - [ x] Bug fix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ x] Breaking change (fix or feature that would cause existing functionality to change) - [ ] Documentation update ## Checklist <!-- Go over all the following points, and put an `x` in all the boxes that apply. --> - [x ] I have read the [MCP Documentation](https://modelcontextprotocol.io) - [ x] My code follows the repository's style guidelines - [ x] New and existing tests pass locally - [ x] I have added appropriate error handling - [x ] I have added or updated documentation as needed ## Additional context It is related to issue #107 Co-authored-by: Avinash Sridhar <sridharavinash@users.noreply.github.com> Co-authored-by: adam jones <domdomegg+git@gmail.com>
This commit is contained in:
parent
26b85d7158
commit
502b628ac8
1 changed files with 0 additions and 2 deletions
|
|
@ -4,8 +4,6 @@ services:
|
|||
container_name: registry
|
||||
build:
|
||||
dockerfile: Dockerfile
|
||||
links:
|
||||
- mongodb
|
||||
depends_on:
|
||||
- mongodb
|
||||
environment:
|
||||
|
|
|
|||
Loading…
Reference in a new issue