-
Notifications
You must be signed in to change notification settings - Fork 324
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
chore: complete updates to build-images from msrv bump #12320
Conversation
Sidenote on this, any thoughts on having |
Interesting. So the check toolchain function would take the version from rust-toolchain? Think that makes sense. How often are we expected to bump the rust version? because it is a pain... But as per README.md in the build-images, someone with the dockerhub password (i can do it), can run ./bootstrap.sh ci. Then I need to rebuild and update the AMI's we use in CI. It's not done automatically in CI yet. |
I'm wanting to avoid doing it unnecessarily as well. We've followed the process of just bumping our MSRV in response to major dependencies bumping theirs (as this prevents us from publishing our crates). This should be less of an issue in future due to rust stabilising the msrv-aware resolver in the 2024 edition. |
I've tested CI with a local build of the new image and everything is building correctly. Let me know if there's anything extra that needs to be done before we publish a new image. |
@guipublic I spoke with Charlie on this today and we agreed that we're going to loosen the checks in Let's get this PR merged and update the sync PR so we can unblock the sync. |
The rust MSRV got bumped in #12298 but we didn't catch that we're still using a stale devbox image due to the check in
bootstrap.sh
not being updated properly. CI's passing due to cargo automatically installing the proper toolchain at runtime.I've completed the relevant changes to fix the build image /
./bootstrap.sh check
in this PR but I'm not up to date on how pushing an update to devbox should be done (or if CI still automatically pushes to dockerhub anymore).@charlielye @ludamad Could one of you help me with fixing this?