feat: add session-aware logging support
Metadata
Current evaluation
Merged PR #371 at commit ffb88ea, adding session-aware logs that prefix session events with session IDs and resolve fetch-service#135.
Resolution: closed resolved
Reason: PR #371 was merged into fetch-service as commit ffb88ea. Its implementation prefixes session-context log messages with the session ID and explicitly states that it fixes #135.
Evaluated by: mr-cal-bot
No scores available.
Issue body
Log messages from events happening in the context of a session are
prefixed with the session ID. This makes log messages more readable
when multiple sessions are running concurrently.
Fixes #135
Signed-off-by: Claudio Matsuoka <claudio.matsuoka@canonical.com>
- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)?
-----
Evaluation history
| Date | Model | Scores | Action | Summary |
|---|---|---|---|---|
| stealth/space-bunny-alpha | — | closed resolved | Merged PR #371 at commit ffb88ea, adding session-aware logs that prefix session events with session IDs and resolve fetch-service#135. | |
| stealth/space-bunny-alpha | — | closed resolved | Session-aware logging was merged, prefixing session event logs with session IDs to distinguish concurrent sessions. The PR explicitly fixes #135. | |
| qwen/qwen3.6-35b-a3b | — | — | Merged feature adding session ID prefixes to log messages for improved readability during concurrent sessions. Approved by two reviewers, resolves issue #135, and updates 54 files. |
Update history
| Date | Change |
|---|---|
| created |
Related work
-
Related To:
canonical/fetch-service#135
(confidence 100%)
Original request to include session IDs in relevant logs, explicitly fixed by PR #371.
Related issues
| Issue | Project | State | Summary | Similarity |
|---|---|---|---|---|
| #272 feat: add fetch-service version to session metadata | fetch-service | merged | Merged PR #272 added a generator field with the fetch-service version to session metadata, resolving fetch-service#258; merged as commit a0e826f. | |
| #456 feat: add fetch-service integration | craft-application | merged | Merged in PR #456, adding fetch-service lifecycle, sessions, certificates, build-instance configuration, and environment integration across 18 files. | |
| #135 Mention session_id in all relevant log entries | fetch-service | closed | Session-aware logging was added in commit ffb88ea via PR #371, addressing session-ID context in logs. Issue #135 was closed without a maintainer explanation. |