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

PR actions failing #626

Closed
squakez opened this issue Feb 10, 2021 · 2 comments · Fixed by #646
Closed

PR actions failing #626

squakez opened this issue Feb 10, 2021 · 2 comments · Fixed by #646

Comments

@squakez
Copy link
Contributor

squakez commented Feb 10, 2021 •
edited
Loading

PRs are failing because No such image: testcontainers/ryuk:0.3.0. It seems related to this issue.

This should be fixed by using testcontainers version 1.15.1 instead of 1.15.0. Dependency is updated in camel 3.8.0 but not in 3.7.0, so likely we should have this fixed when bumping the camel version.
@squakez squakez mentioned this issue Feb 10, 2021
Merged
@lburgazzoli
Copy link
Contributor

You can bump the version in this project, see: apache/camel-quarkus@e59a7bd

squakez added a commit to squakez/camel-k-runtime that referenced this issue Feb 10, 2021
@squakez
fix(test): bump testcontainers to 1.5.1
b328078 
Ref apache#626
@squakez squakez mentioned this issue Feb 10, 2021
Merged
lburgazzoli pushed a commit that referenced this issue Feb 10, 2021
@squakez @lburgazzoli
fix(test): bump testcontainers to 1.5.1
965bb71 
Ref #626
@squakez
Copy link
Contributor Author

squakez commented Feb 10, 2021

This was fixed via #627. However I'd keep it open in order to remove the overridden dependencies once we bump camel-quarkus and camel version to versions that are using the updated testcontainers.

@lburgazzoli lburgazzoli mentioned this issue Mar 29, 2021
Merged
orpiske pushed a commit that referenced this issue May 24, 2021
@squakez @orpiske
fix(test): bump testcontainers to 1.5.1
e2e40ea 
Ref #626
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Upgrade to camel-quarkus 1.8.0, camel 3.9.0 lburgazzoli/apache-camel-k-runtime
2 participants
@lburgazzoli @squakez