Add ALL level

[geoand]

Fixes: quarkusio/quarkus#37598

[jamezp]

Interesting, would it not inherit the level from java.util.logging.Level? We also don't have OFF.

[jamezp]

Actually, I'm assuming any JUL levels would fail. I don't know enough about GraalVM to know how the inheritance works though.

[geoand]

It's not real inheritance though because the fields are static

[jamezp]

Yeah. What generally happens is Level.parse(value) is used. That method is only in the java.util.logging.Level and it's static so it's also not real inheritance. I'm not sure how Quarkus resolves levels though.

[geoand]

Actually, I'm assuming any JUL levels would fail. I don't know enough about GraalVM to know how the inheritance works though.

Yeah, that's true. Do you want me to add OFF as well?

[geoand]

I'm not sure how Quarkus resolves levels though.

We use org.jboss.logmanager.Logger#setLevel.

[jamezp]

I'm not sure how Quarkus resolves levels though.

We use org.jboss.logmanager.Logger#setLevel.

How is the Level created though? Do you use Level.parse()? The Logger.setLevel() requires a java.util.logging.Level argument so the level would need to be created before that method is invoked.

[geoand]

How is the Level created though? Do you use Level.parse()? The Logger.setLevel() requires a java.util.logging.Level argument so the level would need to be created before that method is invoked.

We use LogContext.getLogContext().getLevelForName(value.toUpperCase(Locale.ROOT));

[jamezp]

That's really strange then. That should definitely work. @dmlloyd Do you have any ideas?

I don't understand where the org.jboss.logmanager.Level.ALL would come from as the LogContext.getLogContext().getLevelForName("ALL") should return java.util.logging.Level.All.

[dmlloyd]

It definitely should work as it is. We have tests for it too IIRC, so maybe something GraalVM-specific is happening?

[dmlloyd]

This ended up being fixed by quarkusio/quarkus#37873 - it was an error in how the field was registered to the native image. Problem solved!