Skip to content

Tone down logging for unknown folder types#7185

Merged
labkey-jeckels merged 1 commit intorelease25.11-SNAPSHOTfrom
25.11_fb_folderTypeLogging
Nov 13, 2025
Merged

Tone down logging for unknown folder types#7185
labkey-jeckels merged 1 commit intorelease25.11-SNAPSHOTfrom
25.11_fb_folderTypeLogging

Conversation

@labkey-jeckels
Copy link
Copy Markdown
Contributor

@labkey-jeckels labkey-jeckels commented Nov 6, 2025

Rationale

We can spam the log file with many redundant complaints about an unknown folder type. Real-world wxamples:

https://app.datadoghq.com/logs?query=%22no%20such%20folder%20type%22&agg_m=count&agg_m_source=base&agg_t=count&clustering_pattern_field_path=message&cols=host%2Cservice&messageDisplay=inline&refresh_mode=sliding&storage=hot&stream_sort=desc&viz=stream&from_ts=1761169394772&to_ts=1762465394772&live=true

Changes

  • After the first time hitting an unknown folder type, switch to DEBUG logging
  • Track by RowId instead of Container

Tasks 📍

  • Manual Testing @labkey-tchad
    • Create two folders with a folder type supplied by a non-critical module, like MS2 or Flow.
    • Remove that module from your deployment directory
    • Start up the server and visit those containers. You should see a single WARN message about the folder type
  • Needs Automation - N/A

@labkey-jeckels labkey-jeckels requested a review from a team November 6, 2025 21:48
@labkey-jeckels labkey-jeckels self-assigned this Nov 6, 2025
@labkey-jeckels labkey-jeckels merged commit 1999374 into release25.11-SNAPSHOT Nov 13, 2025
18 of 19 checks passed
@labkey-jeckels labkey-jeckels deleted the 25.11_fb_folderTypeLogging branch November 13, 2025 18:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants