-
Notifications
You must be signed in to change notification settings - Fork 72
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
Upgrade cluster_tools to v1.0.8 in shard.yml #2200
base: main
Are you sure you want to change the base?
Conversation
Please don't merge till the new cluster_tools version is published (See cnf-testsuite/cluster_tools#29) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
@collivier the reason for GH actions failure is that you need to transitively change the cluster_tools to your version in all the shards and make that a part of your pull request (purely for the purposes of testing). This is quite a painful process. |
@svteb yes. Thank you. I'm on it now. |
It includes the fix stopping passing stdin to the container. Signed-off-by: Cédric Ollivier <[email protected]>
74d1f05
to
b1dcc93
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm
Please wait the upstream change and then I will slightly update this one |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
upstream change merged, cluster-tools v1.0.8 released. Please adapt the PR accordingly.
Description
It includes the fix stopping passing stdin to the container.
How has this been tested: