Skip to content

fix(loki): send the logs to the arbutus loki instance - #158

Merged
tjzegmott merged 4 commits into
mainfrom
updated-loki-config-and-dep
Sep 2, 2026
Merged

fix(loki): send the logs to the arbutus loki instance#158
tjzegmott merged 4 commits into
mainfrom
updated-loki-config-and-dep

Conversation

@tjzegmott

@tjzegmott tjzegmott commented Sep 1, 2026

Copy link
Copy Markdown
Contributor

This pull request updates logging configuration for Loki integration and improves how logging tags and headers are handled in the codebase. It also updates some dependencies and configuration values. The most important changes are grouped below:

Loki Logging Configuration Improvements:

  • Refactored the logic for updating Loki logging tags in workflow/cli/run.py to avoid overwriting other logging configuration, ensuring only the relevant tags are updated and preserving other settings like headers.
  • Modified the Loki configuration in workflow/lifecycle/configure.py to also retrieve and use custom HTTP headers (such as X-Scope-OrgID) when initializing the Loki logging handler. [1] [2]
  • Updated the default Loki endpoint URL and added default Loki headers (X-Scope-OrgID: chime) in the workflow/workspaces/chimefrb.yml workspace configuration. [1] [2]

Dependency Updates:

  • Replaced the python-logging-loki dependency with a specific git revision of python-logging-loki-v2 in pyproject.toml, likely to support new features such as custom headers.

Minor Configuration Cleanups:

  • Minor formatting changes in pyproject.toml for consistency in the packages and env sections. [1] [2]

CI

  • Expanded python matrix up to 3.14
  • Refactor the installation step

@tjzegmott
tjzegmott force-pushed the updated-loki-config-and-dep branch from 17be584 to cd49f45 Compare September 2, 2026 18:50
@tjzegmott
tjzegmott requested a review from Afrokk September 2, 2026 18:58
@tjzegmott
tjzegmott merged commit e51744a into main Sep 2, 2026
14 checks passed
@tjzegmott
tjzegmott deleted the updated-loki-config-and-dep branch September 2, 2026 19:50
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