Fellow Bazel contributors (and Java Team members),It would be nice if I'm wrong about that) To save people a click, the failing error is:
I've discussed this with a few of you but I want this to go to the widest possible audience right now. We still have no idea why Bazel Java tests are failing [1] and it seems to indicate an actual problem with Bazel, as opposed to just a flaky test. (
Unrecognized VM option 'UseParallelOldGC'
(Which has been patched out of the code [2] so I have no idea where it's coming from)
So, if we cannot fix this, I'll have to remove Bazel from Debian testing before the bookworm release. Meaning Bazel will NOT be in the next stable Debian release.
1 March 2023 is the last day to make an upload that will definitely make it in before the Hard Freeze.
So, I'm asking all interested people to please please take a look and see if you can figure out how to fix either the tests or the suspected underlying issue.
Pull Requests from your own branch in the bazel-bootstrap repository [3] are preferred but I'll take patches or even "you should look at line xxx in this one source file."
Please reply to this thread with any ideas or suggestions. Hopefully with all our eyes on this we can make this really frustrating bug a bit more shallow.
Thanks in advance!!
-Olek
[1] https://salsa.debian.org/bazel-team/bazel-bootstrap/-/jobs/3862692
[2] https://salsa.debian.org/bazel-team/bazel-bootstrap/-/blob/master/debian/patches/fix_JDK-16_test_failures.patch
[3] https://salsa.debian.org/bazel-team/bazel-bootstrap
I *have* temporarily disabled the Java tests [1]. As you said, to not
hold back the package due to a possibly flaky test. But my concern is
that this is actually an issue with how Bazel uses Java rather than a
flaky test. Again, I'd be happy to be wrong about that. :)
If the test passes regularly, and just sometimes fails under the same conditions, I would consider disabling it in autopkgtest rather than have it not make it into bookworm. The perfect is the enemy of the good.
Hi hc,
Thanks for helping to start the conversation!
On February 21, 2023 2:38:14 PM UTC, Hans-Christoph Steiner <hans@at.or.at> wrote:
If the test passes regularly, and just sometimes fails under the same conditions, I would consider disabling it in autopkgtest rather than haveit not make it into bookworm. The perfect is the enemy of the good.
Unfortunately, the test fails pretty regularly now. This behavior started when we moved to OpenJDK17. That makes sense since 'UseParallelOldGC' was valid prior to that.
I *have* temporarily disabled the Java tests [1]. As you said, to not hold back the package due to a possibly flaky test. But my concern is that this
is actually an issue with how Bazel uses Java rather than a flaky test. Again, I'd be happy to be wrong about that. :)
-Olek
[1] https://salsa.debian.org/bazel-team/bazel-bootstrap/-/commit/1c90621d7b9b5d44d01b91dabb04645a20273ca8
--
Mailing list: https://launchpad.net/~bazel-team
Post to : bazel-team@lists.launchpad.net
Unsubscribe : https://launchpad.net/~bazel-team
More help : https://help.launchpad.net/ListHelp
Sysop: | Keyop |
---|---|
Location: | Huddersfield, West Yorkshire, UK |
Users: | 300 |
Nodes: | 16 (2 / 14) |
Uptime: | 28:43:12 |
Calls: | 6,707 |
Files: | 12,239 |
Messages: | 5,352,813 |