Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docker compose error #402

Closed
imerelli opened this issue Sep 7, 2022 · 7 comments
Closed

docker compose error #402

imerelli opened this issue Sep 7, 2022 · 7 comments

Comments

@imerelli
Copy link

imerelli commented Sep 7, 2022

Hi, I'm trying to install sregistry to manage my singularity images.
I'm using centos 7 and Docker version 20.10.17, build 100c701
(which is the actual default from the docked repository).
As you probably know docker compose is now part of docker.

I have this error

docker compose up -d
(root) Additional property db is not allowed

which I guess is due to some incompatibility of docker
compose file with my version of docker/docker compose.

Can you help me in solving this problem?
Is an update scheduled to new versions of docker?
Are there new tools I can use to manage singularity images instead of this?

Thanks

@vsoch
Copy link
Member

vsoch commented Sep 7, 2022

We probably just need to update the compose file for the current version! It looks like all of the current top level images should be moved under services, with possibly a version too,eg.,https://docs.docker.com/compose/compose-file/

If you want to try that I can make suggestions, and when we stumble on something working can do a PR!

@imerelli
Copy link
Author

imerelli commented Sep 7, 2022

I'm not an expert of docker compose...
if you can update the installation files etc etc
I would be more than happy to test the new installation procedure.

@vsoch
Copy link
Member

vsoch commented Sep 7, 2022

you don't need to be an expert! It's literally just yaml, and one level of nesting. Here is the example they have:

image

The only missing piece for the files here is the top level "services" to wrap what we already have.

@imerelli
Copy link
Author

imerelli commented Sep 9, 2022

As suggested, I was able to install sregistry just modifying the docker-compose.yaml file by placing everything under the "services:" top level tag.

@vsoch
Copy link
Member

vsoch commented Sep 9, 2022

woohoo! If you'd like to pull request with these changes (in the root docker-compose.yaml and the one in the https folder) it would be hugely appreciated. If not, I can make the same changes and ask for a quick review.

And see - you totally rocked it!

@vsoch
Copy link
Member

vsoch commented Sep 17, 2022

This will be closed by #405 where I've updated instructions (docs) and the docker-compse.yml recipes.

@cpeel
Copy link

cpeel commented Oct 9, 2022

#405 was merged, we should be able to close this issue.

@vsoch vsoch closed this as completed Oct 9, 2022
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

No branches or pull requests

3 participants