Enable monitoring flag is only available from GraalVM 22.3 onwards

[galderz]

Fixes #37825

[gsmet]

@zakkak do we still support 22.2 in recent versions? I thought we moved away from it but it might be me misremembering.

[galderz]

@gsmet If we don't support 22.2 any more, we should up the minimum version. Right now there's nothing stopping people using it.

[zakkak]

@zakkak do we still support 22.2 in recent versions?

We don't even support it in older versions! 22.2 was released on Jul 27, 2022 and reached EOL in October 2022.

@gsmet If we don't support 22.2 any more, we should up the minimum version. Right now there's nothing stopping people using it.

+1, the question is which version should we up the minimum to?

22.3 is still supported till April 2024, however AFAIK there are no workflows testing Quarkus main with 22.3, we only test against Quarkus 2.13.

The sensible thing to do would be to up the minimum to 23.0, but it might be worth keeping 22.3 just for doing cross-version comparisons. Thoughts?

[zakkak]

LGTM, but is probably not worth merging. 22.2 is not supported since Oct 2022, we should better up the minimum supported version as discussed in the comments.

[gsmet]

@galderz where did you see we still support 22.2?

[zakkak]

@galderz where did you see we still support 22.2?

@gsmet see

quarkus/core/deployment/src/main/java/io/quarkus/deployment/pkg/steps/GraalVM.java

Line 159 in 519b3ea

public static final Version MINIMUM = VERSION_22_2_0;

This is used to let users know if they are using an unsupported version or not. Right now using 22.2 doesn't show any warnings/errors.

[quarkus-bot]

Failing Jobs - Building 7723401

Status	Name	Step	Failures	Logs	Raw logs	Build scan
✖	Native Tests - Security2	Update Docker Client User Agent	⚠️ Check →	Logs	Raw logs	🚧

You can also consult the Develocity build scans.

[galderz]

LGTM, but is probably not worth merging. 22.2 is not supported till Oct 2022, we should better up the minimum supported version as discussed in the comments.

Fair enough. Feel free to close this and send a new PR updating minimum GraalVM version when agreed.