Skip to content

fix: add a CI job to test the container - #150

Open
aryehb wants to merge 1 commit into
owasp-modsecurity:mainfrom
aryehb:feature/ci-test-dockerfile
Open

fix: add a CI job to test the container#150
aryehb wants to merge 1 commit into
owasp-modsecurity:mainfrom
aryehb:feature/ci-test-dockerfile

Conversation

@aryehb

@aryehb aryehb commented Aug 27, 2026

Copy link
Copy Markdown
Contributor
  • Build the image from the Dockerfile, serve the site with it, and request a page to confirm the container actually works.
  • Request the generated stylesheet as well as the home page.
    It is built by dart-sass and PostCSS inside the container, so serving it exercises the whole toolchain of the image rather than just Hugo.
  • Poll for the server instead of sleeping a fixed amount, and bail out early if the container has already exited.
  • Dump the container logs on failure to make a broken build diagnosable from the run alone.

* Build the image from the Dockerfile, serve the site with it, and
  request a page to confirm the container actually works.
* Request the generated stylesheet as well as the home page.
  It is built by `dart-sass` and PostCSS inside the container, so
  serving it exercises the whole toolchain of the image rather than just
  Hugo.
* Poll for the server instead of sleeping a fixed amount, and bail out
  early if the container has already exited.
* Dump the container logs on failure to make a broken build diagnosable
  from the run alone.
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.

1 participant