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.

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

Use latest Docker images for integration tests. #30

Merged
merged 7 commits into from
Feb 1, 2024

Conversation

lthibault
Copy link
Contributor

@lthibault lthibault commented Jan 31, 2024

Use flashbots/suave-execution-geth:lastest for suave-enabled chain.
Use latest image for flashbots/suave-geth.

This will allow suapp-examples to be pulled into the CI pipeline of suave-geth and suave-execution-geth.

@lthibault lthibault added the enhancement New feature or request label Jan 31, 2024
@lthibault lthibault requested a review from dmarzzz January 31, 2024 22:16
@lthibault lthibault self-assigned this Jan 31, 2024
@@ -28,7 +28,6 @@ lt: lint test

.PHONY: run-integration
run-integration:
go run examples/app-ofa-private/main.go
Copy link
Contributor Author

@lthibault lthibault Feb 1, 2024

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This test is broken and was causing the entire pipeline to fail. Removing it for now.

@lthibault lthibault marked this pull request as ready for review February 1, 2024 20:55
Copy link
Member

@dmarzzz dmarzzz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM but am no docker expert, can you create a github issue to re-enable the flaky private OFA test plz?

@lthibault
Copy link
Contributor Author

LGTM but am no docker expert

No worries. Both of the images get rebuilt by the CI pipeline for suave-geth and suave-exec, respectively. This change ensures the latest image is used, rather than hard-coding a specific version.

@lthibault lthibault merged commit de7103c into main Feb 1, 2024
2 checks passed
@lthibault lthibault deleted the feat/use-docker-latest branch February 1, 2024 21:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants