Skip to content

Increase timeout when testing Docker image#10924

Merged
hashhar merged 1 commit intotrinodb:masterfrom
hashhar:hashhar/debug-docker-image
Feb 3, 2022
Merged

Increase timeout when testing Docker image#10924
hashhar merged 1 commit intotrinodb:masterfrom
hashhar:hashhar/debug-docker-image

Conversation

@hashhar
Copy link
Copy Markdown
Member

@hashhar hashhar commented Feb 3, 2022

The ARM64 Docker image runs under emulation and hence can take time to
start up. The existing timeout of ~150 seconds is too close to the
amount of time it takes for the image to start and hence introduces
occasional flakiness.

We also make available the container's logs in timeouts to help debug if
needed.

@hashhar hashhar added the WIP label Feb 3, 2022
@cla-bot cla-bot bot added the cla-signed label Feb 3, 2022
@hashhar hashhar force-pushed the hashhar/debug-docker-image branch from bfa7768 to cd5738b Compare February 3, 2022 06:52
@hashhar
Copy link
Copy Markdown
Member Author

hashhar commented Feb 3, 2022

Ah, it was flaky earlier too. We have a timeout of ~150 seconds to wait for docker image to start.
Even before Docker image change it fails sometimes locally if run enough times for the ARM64 image (since it runs using emulation on CI).

I'll update the timeout to give more breathing room.

The ARM64 Docker image runs under emulation and hence can take time to
start up. The existing timeout of ~150 seconds is too close to the
amount of time it takes for the image to start and hence introduces
occasional flakiness.

We also make available the container's logs in timeouts to help debug if
needed.
@hashhar hashhar force-pushed the hashhar/debug-docker-image branch from cd5738b to 8c70bd4 Compare February 3, 2022 08:02
@hashhar hashhar changed the title [TMP] Debug docker image test flakiness Increase timeout when testing Docker image Feb 3, 2022
@hashhar hashhar removed the WIP label Feb 3, 2022
@hashhar hashhar marked this pull request as ready for review February 3, 2022 08:03
@hashhar hashhar requested review from ebyhr and sopel39 February 3, 2022 08:03
@hashhar hashhar merged commit 1f1177b into trinodb:master Feb 3, 2022
@hashhar hashhar deleted the hashhar/debug-docker-image branch February 3, 2022 10:01
@github-actions github-actions bot added this to the 370 milestone Feb 3, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Development

Successfully merging this pull request may close these issues.

2 participants