Re: [PROPOSAL] Change log4j2 default config to xml (or json)

2021-12-28 Thread Romain Manni-Bucau
Hmm, I kind of think it is a very biased statement Matt - not blaming, let me explain: > @Romain- I think heterogenous is a great goal— however, with complex hierarchal configurations, the “simplicity” of properties is lost in the structure. Default Log4j2 as properties is illegible by any UX/DX s

Re: [PROPOSAL] Change log4j2 default config to xml (or json)

2021-12-28 Thread Jean-Baptiste Onofre
Hi You mean org.ops4j.pax.logging config PID right (so today etc/org.ops4j.pax.logging.cfg). It’s already possible to refresh a log4j XML in org.ops4j.pax.logging. So basically, etc/org.ops4j.pax.logging.cfg just contain the location of the log4j.xml. I think it’s good enough. My point is th

Re: [PROPOSAL] Change log4j2 default config to xml (or json)

2021-12-28 Thread Matt Pavlovich
@JB- To be clear, the request is for the log4j2 config file to be in xml or json, not supporting json or xml log formats @Romain- I think heterogenous is a great goal— however, with complex hierarchal configurations, the “simplicity” of properties is lost in the structure. Default Log4j2 as pr

Re: [PROPOSAL] Change log4j2 default config to xml (or json)

2021-12-27 Thread Romain Manni-Bucau
My 2cts would be that log4j2 or any configuration in karaf should be homogeneous with other config files. Since OSGi is .cfg (enriched properties) by design, I think it is better to stick to this or something very close *by default*. Making the config formats heterogeneous will make your tooling he

Re: [PROPOSAL] Change log4j2 default config to xml (or json)

2021-12-27 Thread Jean-Baptiste Onofre
By the way, just a reminder: a good point about properties format in pax-logging-log4j2 service is that it doesn’t require extra dependency. Using xml/json format needs additional dependency/packages/bundles in the Karaf distribution. Just a side note. Regards JB > Le 27 déc. 2021 à 19:33, Mat

Re: [PROPOSAL] Change log4j2 default config to xml (or json)

2021-12-27 Thread JB Onofré
Hi I think it’s a personal standpoint. For me, properties format is fine but I understand people may like other format. First, if we want to change the format it would be only on a major release bump, meaning 4.4.x currently. Second, the log:* commands have to be updated because they expect

[PROPOSAL] Change log4j2 default config to xml (or json)

2021-12-27 Thread Matt Pavlovich
I’ve created a proposal JIRA KARAF-7307 (https://issues.apache.org/jira/browse/KARAF-7307 ) to track any specifics. As the subject mentions— I think it would be beneficial to users to change the default configuration for log4j2 to XML (or maybe