Skip to content

chore(ci): do not fail fast main tests in different Go versions#2210

Merged
mdelapenya merged 2 commits intotestcontainers:mainfrom
mdelapenya:ci-refinements
Feb 14, 2024
Merged

chore(ci): do not fail fast main tests in different Go versions#2210
mdelapenya merged 2 commits intotestcontainers:mainfrom
mdelapenya:ci-refinements

Conversation

@mdelapenya
Copy link
Copy Markdown
Member

What does this PR do?

This PR removes the fail-fast situation when a CI build fails, making all the build in the following matrices to continue:

  • main job
  • reaper-off
  • rootless docker

Why is it important?

Many times the failure could be caused for some flakiness in that particular build, and its sibling could not be affected.

We run all the tests in the two current releases of Go, so if it fails in Go 1.20.x we still want to continue the execution for Go 1.21.x

@mdelapenya mdelapenya added the chore Changes that do not impact the existing functionality label Feb 14, 2024
@mdelapenya mdelapenya requested a review from a team as a code owner February 14, 2024 21:10
@mdelapenya mdelapenya self-assigned this Feb 14, 2024
@netlify
Copy link
Copy Markdown

netlify bot commented Feb 14, 2024

Deploy Preview for testcontainers-go ready!

Name Link
🔨 Latest commit efea2b7
🔍 Latest deploy log https://app.netlify.com/sites/testcontainers-go/deploys/65cd3f4f61be580008e148b1
😎 Deploy Preview https://deploy-preview-2210--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.

@mdelapenya mdelapenya merged commit 4bf9515 into testcontainers:main Feb 14, 2024
mdelapenya added a commit to Chris-AS1/testcontainers-go that referenced this pull request Feb 14, 2024
* main:
  chore(ci): do not fail fast main tests in different Go versions (testcontainers#2210)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore Changes that do not impact the existing functionality

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant