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

Problem with upgrading database #5544

Closed
2 tasks done
efcy opened this issue Jan 2, 2023 · 3 comments · Fixed by #5545
Closed
2 tasks done

Problem with upgrading database #5544

efcy opened this issue Jan 2, 2023 · 3 comments · Fixed by #5545
Assignees
Labels
question Further information is requested

Comments

@efcy
Copy link
Contributor

efcy commented Jan 2, 2023

My actions before raising this issue

I wanted to upgrade the database as described in https://opencv.github.io/cvat/docs/administration/advanced/upgrade_guide/#how-to-upgrade-postgresql-database-base-image

I ran into an error because I have an override file similar to what can be found in the docs at https://opencv.github.io/cvat/docs/administration/basics/installation/#share-path

The error came from the command docker compose up -d cvat_db because the version of the docker-compose files are different to the override file.

Possible Solution

Simply add the line version: "3.3" to all the other docker-compose files.

@nmanovic
Copy link
Contributor

nmanovic commented Jan 3, 2023

@StellaASchlotter , please look at #5524. Do you use docker-compose v2? Could you please remove version: "3.3" from override docker compose file?

@SpecLad , could you please advise?

@nmanovic nmanovic added the question Further information is requested label Jan 3, 2023
@efcy
Copy link
Contributor Author

efcy commented Jan 3, 2023

Removing the version in the override file works for me. I suggest removing the version in the docs at https://opencv.github.io/cvat/docs/administration/basics/installation/#share-path so that there is no confusion again.

@SpecLad
Copy link
Contributor

SpecLad commented Jan 3, 2023

Thank you for the report. I will fix the docs.

nmanovic pushed a commit that referenced this issue Jan 3, 2023
I was supposed to do this in #5524 when I was removing them everywhere
else, but I didn't think to search the documentation. Compose override
files must have the same format as the base file, otherwise Docker
Compose rejects them.

Fixes #5544.
mikhail-treskin pushed a commit to retailnext/cvat that referenced this issue Jul 1, 2023
…t-ai#5545)

I was supposed to do this in cvat-ai#5524 when I was removing them everywhere
else, but I didn't think to search the documentation. Compose override
files must have the same format as the base file, otherwise Docker
Compose rejects them.

Fixes cvat-ai#5544.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants