Skip to content

feat: add smoke test inputs to CI workflow#600

Open
DeepDiver1975 wants to merge 2 commits intomasterfrom
feat/smoke-test
Open

feat: add smoke test inputs to CI workflow#600
DeepDiver1975 wants to merge 2 commits intomasterfrom
feat/smoke-test

Conversation

@DeepDiver1975
Copy link
Copy Markdown
Contributor

@DeepDiver1975 DeepDiver1975 commented Apr 23, 2026

Summary

Behaviour per matrix entry

Version HTTP 200 check Version assertion
10.16.2 yes .versionstring == 10.16.2
10.16.1 yes .versionstring == 10.16.1
10.15.3 yes .versionstring == 10.15.3
11.0.0-prealpha yes skipped

Why the smoke test was failing

The smoke-test step runs under bash -e -o pipefail. Docker's userland proxy binds the host port and accepts TCP connections immediately, before Apache inside the container is ready. ownCloud's first-run SQLite init takes ~60–90 s, so the very first curl attempt receives a connection reset (exit code 56 = CURLE_RECV_ERROR). With set -e, that aborts the script immediately — the EXIT trap then calls docker stop explaining the characteristic 10 s gap before the CI error. Fixed in owncloud-docker/ubuntu#265 by adding || STATUS="000" so curl network errors continue the polling loop rather than aborting.

🤖 Generated with Claude Code

DeepDiver1975 and others added 2 commits May 6, 2026 09:28
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
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