-
Notifications
You must be signed in to change notification settings - Fork 8
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 test: check detached container #337
base: dev
Are you sure you want to change the base?
Conversation
Check if the detached container is correctly running before to test the libreant interface. In the case the container encounter some error during startup the exit code and the last log lines will be printed.
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.
good!
.docker/test_all.sh
Outdated
docker run --rm -p 5000:5000 -d --name "${PREFIX}${os}" "${PREFIX}${os}" | ||
if ! with_backoff test_libreant_homepage; then | ||
docker run -p 5000:5000 -d --name "${PREFIX}${os}" "${PREFIX}${os}" | ||
check_detatched_container "${PREFIX}${os}" || exit 1 |
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.
wrong indentation?
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.
yeap there was various indentation errors. Fixed
.docker/test_all.sh
Outdated
echo "Container state: " 1>&2 | ||
docker container inspect $c_name --format=" Status: {{.State.Status}}" 1>&2 | ||
docker container inspect $c_name --format=" ExitCode: {{.State.ExitCode}}" 1>&2 | ||
echo "Container's log tail:" 1>&2 |
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.
saxon genitive is for people only, not for things. Container last log lines:
would be better
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.
Fixed
Check if the detached container is correctly running before to test the
libreant interface.
In the case the container encounter some error during startup the exit
code and the last log lines will be printed.