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.

Sign up for GitHub

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

Jump to bottom

fix: readiness hook back off #2718

Merged
merged 1 commit into from
Aug 12, 2024
Merged

fix: readiness hook back off #2718

merged 1 commit into from
Aug 12, 2024

Conversation

stevenh
Copy link
Collaborator

@stevenh stevenh commented Aug 8, 2024

Fix the definition of readiness hook back off which had the wrong MaxElapsedTime resulting in some flaky tests.

@stevenh
fix: readiness hook back off
c58bf3c 
Fix the definition of readiness hook back off which had the wrong
MaxElapsedTime resulting in some flaky tests.
@stevenh stevenh requested a review from a team as a code owner August 8, 2024 18:53
@netlify Netlify
Copy link

netlify bot commented Aug 8, 2024 •
edited
Loading

✅ Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit c58bf3c
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-go/deploys/66b514152ccf3000083aaa48
😎 Deploy Preview https://deploy-preview-2718--testcontainers-go.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@stevenh stevenh mentioned this pull request Aug 8, 2024
Closed
@mdelapenya mdelapenya self-assigned this Aug 12, 2024
@mdelapenya mdelapenya added the bug An issue with the library label Aug 12, 2024
@mdelapenya mdelapenya merged commit 8dbd4cd into testcontainers:main Aug 12, 2024
111 of 112 checks passed
mdelapenya added a commit that referenced this pull request Aug 12, 2024
@mdelapenya
Merge branch 'main' into v1
372e67f 
* main:
  fix: readiness hook back off (#2718)
  fix: skip unhealthy (#2717)
  fix: filter context errors from log production (#2715)
@mdelapenya mdelapenya mentioned this pull request Aug 22, 2024
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mdelapenya mdelapenya mdelapenya left review comments

Assignees

@mdelapenya mdelapenya

Labels
bug An issue with the library
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@stevenh @mdelapenya