On Fri, Jun 23, 2023 at 8:36 AM Josh McKenzie wrote:
> How frequently do we see failures in tests (straight failure, flakes, etc)
> that are due to the versions of dependencies (JDK or python) that we're
> running the tests with?
>
> Regardless of 1, if the answer to 2 is "almost never", then th
Sounds like you were assuming the effort would require 3 Ekaterina?
Thank you Josh.
Yes, you are totally right. I forgot for a moment that the new scripts are
layered. Valid point about 2.
Now the question is how fast 2 will be to run it and how many people would
actually use it for running
I'll take engagement. Even if it's broadly disagreement. :D
(Full disclosure, I have a weak opinion here held weakly):
> Wouldn’t we recommend people to use the test images the project CI use? Thus
> using in testing the versions we use? I would assume the repeatable CI will
> still expect test
I like having the latest patch release always brought in for testing and CI for the JDK versions explicitly supported. So for this release, 11/17.On Jun 22, 2023, at 5:42 PM, Jeremiah Jordan wrote:
Yes. -1 on forcing the patch release version, and possibly minor version, for anything that is
Yes. -1 on forcing the patch release version, and possibly minor version,
for anything that is not absolutely necessary to do so. Especially for
things like Java or Python version, I would hope we just install the latest
Java 8, Java 11, or Java 17 JDK for the platform the image is built from
an
Wouldn’t we recommend people to use the test images the project CI use?
Thus using in testing the versions we use? I would assume the repeatable CI
will still expect test images the way we have now?
(I hope I did not misunderstand the question)
I also share similar worries with Brandon
On Thu, 22
On Thu, Jun 22, 2023 at 4:23 PM Josh McKenzie wrote:
>
> 2. Anyone concerned about us being specific about versions in
> requirements.txt in the dtest repo?
My concern with this is atrophy; we'll set the version once and when
finally forced to update, find that a lot of other things must also be
