cargo-bisect-rustc icon indicating copy to clipboard operation
cargo-bisect-rustc copied to clipboard

prefer using toolchains already installed

Open matthiaskrgr opened this issue 2 years ago • 2 comments

Let's say we would naturally try to bisect a change between the 1st and the 30th of a month, the bisect might go something like this:

check 30th

check 1st

check 15th

check 7th

check 11th

check 9th

check 10th <- regression

For each of these steps we might have to download a new nightly.

Now, if we already have, from a previous bisection, nightlies of the 14th, 6th, 11th and 8th installed, I believe we could just use these to already limit our search space so that the total number of nightlies we have to download is reduced to "between 8th and 11th"?

Should save a bunch of bandwith and time.

matthiaskrgr avatar Jul 16 '23 13:07 matthiaskrgr

@matthiaskrgr Maybe I am misreading this issue, but wasn't the --preserve parameter exactly for this (i.e. do not delete nightlies after finishing the bisection)? Or did you mean something else at the time?

(this issue is a bit old so I'm not sure about its status)

apiraino avatar Feb 19 '24 21:02 apiraino

No, given I have a bisection from "01.01.2024 -${today}" And I run it on monday with --preserve, I download all the needed toolchains and find the commit. Then, I run the same bisection AGAIN except today is now friday, it would basically have to re-download most of the toolchains if the respective "middles" of the bisections shift even by one, to a later day.

I want CBR to be smart enough to notice "we already have some toolchain installed" and use these as initial testing-points even if they do not perfectly match the testing points of a "new/fresh" bisection.

matthiaskrgr avatar Feb 19 '24 22:02 matthiaskrgr