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
Fix for ARM processors #3
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
docker-compose.yml
Outdated
test: ["CMD", "wget", "-nv", "-t1", "--spider", "http://localhost:8080/api/health"] | ||
test: | ||
[ | ||
"CMD", | ||
"wget", | ||
"-nv", | ||
"-t1", | ||
"--spider", | ||
"http://localhost:8080/api/health", | ||
] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do we even need this test now?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doesn't seem like it, it's gone
It might also be worth changing the port in https://github.com/ministryofjustice/hmpps-approved-premises-tools/blob/main/bin/start-server#L36 from |
Tilt doesn’t support dependencies so we have to remove them and manually wait until the community-api is up tilt-dev/tilt#2210 (comment)
cf46bc0
to
0accfa4
Compare
As community-api is the slowest service it make sense for us to wait for it to be up
Tilt was not working on my MBP - probably due to the emulation load the community-api was putting on the machine.
Thanks to the work of @pezholio here a multiplatform version of the community-api was created.
However the
ap-tools server start
script was still failing due to the dependencies between the services as mentioned in this issue comment. With these removed and with some patience whilst waiting for the community-api to come up, the issue is resolved.