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

Update docker dev setup and docs #6263

Merged
merged 12 commits into from
Mar 19, 2024
Merged

Update docker dev setup and docs #6263

merged 12 commits into from
Mar 19, 2024

Conversation

mahesh-naxa
Copy link
Collaborator

@mahesh-naxa mahesh-naxa commented Feb 28, 2024

What this covers?

What it Doesn't?

@robsavoye
Copy link
Collaborator

I get this error with this PR:

tm-migration-1 | PermissionError: [Errno 13] Permission denied: 'logs'
service "tm-migration" didn't complete successfully: exit 1

@robsavoye
Copy link
Collaborator

Part of my problem turned out to be related to the version of docker-compose. With the new version, this branch is much closer to working. The front end comes up, but the backend fails with this message:
tm-db-1 | 2024-03-12 04:09:27.604 UTC [181] FATAL: role "-d" does not exist

@dakotabenjamin
Copy link
Member

This PR adds a workflow for building docker images in the GH container registry: #6272

@robsavoye
Copy link
Collaborator

Since I approved the PR that got swagger working, does this need to be updated to work with that ?

@robsavoye
Copy link
Collaborator

I get this now when testing this PR.

338.4 npm ERR! path /usr/src/app/node_modules/@sentry/cli
338.4 npm ERR! command failed
338.4 npm ERR! command sh -c -- node ./scripts/install.js
338.4 npm ERR! [sentry-cli] Downloading from https://downloads.sentry-cdn.com/sentry-cli/2.21.1/sentry-cli-Linux-x86_64
338.4 npm ERR! Error: EACCES: permission denied, mkdir '/root/.npm/sentry-cli'
338.4
338.4 npm ERR! A complete log of this run can be found in:
338.4 npm ERR! /root/.npm/_logs/2024-03-14T07_30_22_362Z-debug-0.log

As the container names forces container to use the fixed container names, it conflicts if you have multiple tm setup
Copy link

Quality Gate Failed Quality Gate failed

Failed conditions
1 Security Hotspot

See analysis details on SonarCloud

@mahesh-naxa
Copy link
Collaborator Author

@robsavoye can you merge this.
I will cover issues with dockerfile (sonarcloud) in a seperate PR.

@robsavoye robsavoye merged commit 635e118 into develop Mar 19, 2024
8 of 9 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Docker-compose fails to work Improve docker to use alternate ports Build issues with docker-compose
3 participants