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

Rust 1.66.0 #1597

Merged
merged 2 commits into from
Jan 11, 2023
Merged

Rust 1.66.0 #1597

merged 2 commits into from
Jan 11, 2023

Conversation

ojeda
Copy link
Contributor

@ojeda ojeda commented Dec 26, 2022

Rust 1.66.0 will be the next version supported by the kernel [1].

Therefore, add rustc-1.66 as a new build environment alongside rustc-1.62 (which can be removed later on when the new version hits mainline), and then move rust-for-linux_rust to it.

Link: Rust-for-Linux/linux#947 [1]
Signed-off-by: Miguel Ojeda ojeda@kernel.org

Rust 1.66.0 will be the next version supported by the kernel [1].

Therefore, add `rustc-1.66` as a new build environment alongside
`rustc-1.62` (which can be removed later on when the new version
hits mainline).

Link: Rust-for-Linux/linux#947 [1]
Signed-off-by: Miguel Ojeda <ojeda@kernel.org>
The `rust` branch of `rust-for-linux` is moving to Rust 1.66.0 [1],
therefore, use the new build environment for it.

Link: Rust-for-Linux/linux#947 [1]
Signed-off-by: Miguel Ojeda <ojeda@kernel.org>
@aliceinwire aliceinwire self-assigned this Dec 27, 2022
@aliceinwire
Copy link
Member

Related: #1580

@aliceinwire
Copy link
Member

currently testing this.

@ojeda
Copy link
Contributor Author

ojeda commented Dec 27, 2022

Thanks a lot!

@10ne1
Copy link
Contributor

10ne1 commented Dec 29, 2022

LGTM

Reviewed-by: Adrian Ratiu adrian.ratiu@collabora.com

@aliceinwire
Copy link
Member

aliceinwire commented Dec 31, 2022

[writing also here] @gctucker or @mgalka could you enable rustc-1.66 docker image on KernelCI for testing? I cannot test this on staging a starting a test job get stuck because it cannot find the related docker image.

nuclearcat added a commit to nuclearcat/kernelci-deploy that referenced this pull request Jan 6, 2023
As defined in recent PR kernelci/kernelci-core#1597
we need docker image with rustcc 1.66 built on staging first.

Signed-off-by: Denys Fedoryshchenko <denys.f@collabora.com>
@nuclearcat
Copy link
Member

@ojeda
Copy link
Contributor Author

ojeda commented Jan 9, 2023

I guess -next not supposed to be buildable by 1.66?

Yeah. Right now, no branch is buildable by 1.66. If we merge Rust-for-Linux/linux#947, then rust-for-linux_rust will be, but at the same time, it will stop being buildable by 1.62. That is why I haven't merged it yet (i.e. the one in the Rust for Linux repo) -- whenever you are ready we can do it. I am not sure which is the best order (or at once) to do so for KernelCI. On the Rust for Linux side, we can do it at any time (for that branch).

Then, after that is merged and we check it is OK, my plan is to submit it to the mailing list (and a matching PR here) and soon after put it into rust-for-linux_rust-next and merge that new PR here. Finally, repeat the same for mainline.

But the above is just what I came up with... If you think KernelCI could handle these "flag day" upgrades in a better way, it would be nice. Ideally, KernelCI could fetch the right environment as given by scripts/min-tool-version.sh (that way we can prepare KernelCI in advance), but I assume that is non-trivial unless the system is designed to do things like bisections anyway.

@nuclearcat
Copy link
Member

@aliceinwire maybe we can merge it then? So Rust-for-Linux/linux#947 will be merged too.
Generally rust-1.66 looks ok and working, everything prepared and tested what possible.

@gctucker
Copy link
Contributor

I think it's best if KernelCI is always upgrading rust toolchains to the latest used by Rust-for-Linux, to provide CI as early as possible. As Rust support in the kernel is still in active development, it's expected that things will break in linux-next etc. It's better if KernelCI shows problems that then get fixed, in the end what really matters is that it helps Rust support in the kernel to get tested and working rather than a quiet sea of green results on linux.kernelci.org.

@ojeda
Copy link
Contributor Author

ojeda commented Jan 10, 2023

Yeah, I think it is fine, so let's merge both branches and see if everything goes well. Thanks a lot!

@aliceinwire
Copy link
Member

aliceinwire commented Jan 11, 2023

@nuclearcat yes, I was waiting for mgalka to add the rust 1.66 docker build image. Doing last tests with rust 1.66 before merging

@aliceinwire aliceinwire merged commit d90d6b4 into kernelci:main Jan 11, 2023
@ojeda ojeda deleted the rust-1.66 branch January 11, 2023 09:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants