Skip to content

ci: prefer docker daemon over podman and increase test timeouts - #110

Merged
matthyx merged 1 commit into
mainfrom
ci/prepull-pulsar-image
Aug 19, 2026
Merged

matthyx merged 1 commit into
mainfrom
ci/prepull-pulsar-image

Conversation

@matthyx

@matthyx matthyx commented Aug 19, 2026 •

Copy link
Copy Markdown
Contributor

Summary

  • Updated pulsar.sh and pulsar_stop.sh scripts to check if the Docker daemon is available (docker info) before falling back to podman.
  • Increased Pulsar startup health check attempts from 40 / 30 to 120 (up to 4 minutes) in pulsar/connector/suite_test.go and pulsar/test/suite.go.
  • Increased go test timeout to 500s in pr.yaml and release.yaml.

Root Cause

GitHub Actions Ubuntu runners have Podman pre-installed. Because pulsar.sh used which podman, it selected rootless Podman instead of the active Docker daemon. Rootless Podman port forwarding on CI runners failed to route localhost:8081 to the container, causing tests to wait endlessly until failure.

@coderabbitai

coderabbitai Bot commented Aug 19, 2026 •

Copy link
Copy Markdown

Important

  • 🔍 Trigger review

This repository does not receive automatic reviews because it has fewer than 10 stars.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 5d5fa91f-15f7-4377-861d-ced5a3389054


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@matthyx matthyx changed the title ci: pre-pull Pulsar Docker image before running tests ci: increase test timeout to 300s in workflows Aug 19, 2026
@matthyx
matthyx force-pushed the ci/prepull-pulsar-image branch 2 times, most recently from 5c6f865 to 0ed85ee Compare August 19, 2026 06:55
@matthyx matthyx changed the title ci: increase test timeout to 300s in workflows ci: increase pulsar startup retry attempts and test timeouts Aug 19, 2026
@matthyx
matthyx force-pushed the ci/prepull-pulsar-image branch from 0ed85ee to 16cab83 Compare August 19, 2026 08:19
@matthyx matthyx changed the title ci: increase pulsar startup retry attempts and test timeouts ci: prefer docker daemon over podman and increase test timeouts Aug 19, 2026
@matthyx
matthyx force-pushed the ci/prepull-pulsar-image branch 2 times, most recently from a070009 to aa910f0 Compare August 19, 2026 08:34
… and add test diagnostics

Signed-off-by: Matthias Bertschy <matthias.bertschy@gmail.com>
@matthyx
matthyx force-pushed the ci/prepull-pulsar-image branch from aa910f0 to 30d24f6 Compare August 19, 2026 08:41
@github-actions

Copy link
Copy Markdown

Summary:

  • License scan: failure
  • Credentials scan: skipped
  • Vulnerabilities scan: skipped
  • Unit test: success
  • Go linting: failure

@matthyx
matthyx merged commit 45688af into main Aug 19, 2026
8 checks passed
@matthyx
matthyx deleted the ci/prepull-pulsar-image branch August 19, 2026 10:00
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