openmicroscopy icon indicating copy to clipboard operation
openmicroscopy copied to clipboard

Add documentation for the omero.logging properties

Open sbesson opened this issue 1 year ago • 3 comments

Initial step towards fixing #6225

All these properties are defined in OMERO.py so changing the default value in this file has no effect but allows for the configuration to be included in the reference documentation

sbesson avatar Jun 19 '24 08:06 sbesson

Looks good. Text makes sense.

So this PR is purely to update the docs at https://omero.readthedocs.io/en/latest/sysadmins/config.html? Trivial point: I noticed there that all booleans tend to be lowercase false whereas you have a mix (although I think it doesn't matter since they are all lowercased when comparing, but just for consistency).

will-moore avatar Jun 21 '24 10:06 will-moore

So this PR is purely to update the docs at https://omero.readthedocs.io/en/latest/sysadmins/config.html?

Absolutely, the default logging values are defined in https://github.com/ome/omero-py/blob/8d8108de02104c654741d65f0ecbbf11c85e329d/src/omero/util/init.py#L32-L38 so modifying etc/omero.properties has no functional effect.

But it is currently undocumented that the server logs can be configured using the standard omero config approach and this PR aims to update the documentation to include these properties.

Trivial point: I noticed there that all booleans tend to be lowercase false whereas you have a mix (although I think it doesn't matter since they are all lowercased when comparing, but just for consistency).

Pushed b73a17c5f0e8f7b3f8f6d3b036152ce73fd0638f to use the current lowercase convention consistently

sbesson avatar Jun 21 '24 10:06 sbesson

While using these properties as part of the review of https://github.com/ome/omero-py/pull/417, one important aspect worth noting is that the omero.logging.level property only affects the Python servers (like Processor or Tables). This is consistent with what is described in https://omero.readthedocs.io/en/stable/developers/logging.html.

The last commit should update the property description to be explicit about this.

sbesson avatar Jun 26 '24 11:06 sbesson