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

Make build image verbose #99

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

bzed
Copy link
Contributor

@bzed bzed commented Dec 22, 2023

Debugging the current build failure was completely impossible without some debug output

  • adding logs from coraza and haproxy to the output
  • tidying the docker compose file
  • using high ports only as port 80 seems to make issues (!?)

This includes #98 as its just necessary.

@sts
Copy link
Contributor

sts commented Dec 23, 2023

LGTM!

bzed added 2 commits March 1, 2024 18:57
To avoid the e2e test blocking a worker until it is killed, ensure that
curl returns (and fails) if it was not able to get a reply in a sane
amount of time.
On my recent docker installation the image build process failed similar
to what github shows, so after a lot of debugging this is the result,
including log outputs of haproxy and coraza at the end.
@bzed bzed force-pushed the make_build_image_verbose branch from 3a006fd to 1c0a02f Compare March 1, 2024 17:58
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

Successfully merging this pull request may close these issues.

3 participants