ztunnel icon indicating copy to clipboard operation
ztunnel copied to clipboard

Bump rust version to 1.78

Open howardjohn opened this issue 1 year ago • 5 comments
trafficstars

This matches the new docker image

howardjohn avatar May 08 '24 19:05 howardjohn

@howardjohn: The following test failed, say /retest to rerun all failed tests or /retest-required to rerun all mandatory failed tests:

Test name Commit Details Required Rerun command
test_ztunnel d8b206f43dea4b4fd456eb272cbfd614606f60cf link true /test test

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes-sigs/prow repository. I understand the commands that are listed here.

istio-testing avatar May 08 '24 19:05 istio-testing

maybe we shouldn't dothis since we don't require 1.78, and this will make it fail on 1.77?

howardjohn avatar May 08 '24 19:05 howardjohn

Yeah I think we need to track our CI version/build-tools version exactly in the Cargo.toml

bleggett avatar May 08 '24 20:05 bleggett

rust-version is the 'The minimal supported Rust version.' We run fine on 1.77, do we really want to say you MUST have 1.78?

Build tools is fine since that is managed. I don't personally care since I always use latest. But seems a bit excessive

howardjohn avatar May 08 '24 20:05 howardjohn

rust-version is the 'The minimal supported Rust version.' We run fine on 1.77, do we really want to say you MUST have 1.78?

I think our min version can only be the min version we actually build and test with.

People are free to try and change it and set it lower, but our floor sort of has to be what we use in CI, we can't make strong claims about any other version.

bleggett avatar May 09 '24 19:05 bleggett