Skip to content

Conversation

@fmigneault
Copy link
Member

@fmigneault fmigneault commented Sep 19, 2025

Changes

  • Bump Docker base image to Python 3.12.
  • Fix time-zone aware datetime parsing for log/db entries comparison
  • Fix multiple log record lookup doing an additional index leading to invalid DB extraction, of datetime, last update status, and invocation counts.
  • Fix ReadTimeout not handled as ConnectionError during service check.

Related

mishaschwartz
mishaschwartz previously approved these changes Sep 19, 2025
Copy link
Collaborator

@mishaschwartz mishaschwartz left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This makes sense. Thanks for updating that.

Since this just parses the logs from the proxy component. Do we ever have a case where the proxy logs aren't generating TZ-aware timestamps?

@github-actions github-actions bot added the ci label Sep 19, 2025
@github-actions github-actions bot added the api label Sep 19, 2025
@fmigneault
Copy link
Member Author

Since this just parses the logs from the proxy component. Do we ever have a case where the proxy logs aren't generating TZ-aware timestamps?

Yes. I think it is possible for historical reasons, or when the produced entry by the service itself is not fully explicit about its timezone.

@codecov
Copy link

codecov bot commented Sep 23, 2025

Codecov Report

❌ Patch coverage is 83.33333% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 83.91%. Comparing base (8f7df0d) to head (16eece7).
⚠️ Report is 15 commits behind head on master.

Files with missing lines Patch % Lines
canarieapi/api.py 71.42% 0 Missing and 2 partials ⚠️
canarieapi/logparser.py 88.88% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #53      +/-   ##
==========================================
+ Coverage   80.76%   83.91%   +3.14%     
==========================================
  Files          11       11              
  Lines         572      578       +6     
  Branches       63       64       +1     
==========================================
+ Hits          462      485      +23     
+ Misses         80       64      -16     
+ Partials       30       29       -1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@fmigneault fmigneault merged commit 1c25036 into master Sep 23, 2025
17 checks passed
fmigneault added a commit to bird-house/birdhouse-deploy that referenced this pull request Sep 23, 2025
## Overview

Update `CanarieAPI==1.0.3` default version.

## Changes

**Non-breaking changes**
- Canarie-API: Update to
[`1.0.3`](https://github.com/Ouranosinc/CanarieAPI/releases/tag/1.0.3)
to fix log parsing.
  - Ensure that timezone-aware date-time are considered.
- Fix statistics retrieval to obtain service/request invocation counts.

**Breaking changes**
- n/a

## Related Issue / Discussion

- Relates to Ouranosinc/CanarieAPI#53
- Relates to Ouranosinc/CanarieAPI#21).

## CI Operations

birdhouse_daccs_configs_branch: master
birdhouse_skip_ci: false
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Monitoring and access logs values not updating

2 participants