← Back to issue list

feat: support per-session inspector conf

View original Github issue

Metadata

Project
fetch-service
Number
#480
Type
pull request
State
merged
Author
upils
Labels
—
Created
Updated
Closed

Current evaluation

Merged: Added per-session inspector configuration support. Approved by two reviewers and passed all CI checks. Proceeded despite 14 unresolved comments and a known spread test failure from an external store bug. Closes FETCH-31.

Suggested action: —

No scores available.

Issue body

- [ ] Have you signed the [CLA](http://www.ubuntu.com/legal/contributors/)? ----- Support configuring inspectors on a per-session basis. Adopt an approach that will be able to evolve into something similar to how pebble deals with merging configurations. FETCH-31

Evaluation history

Date Model Scores Action Summary
qwen/qwen3.6-35b-a3b — — Merged: Added per-session inspector configuration support. Approved by two reviewers and passed all CI checks. Proceeded despite 14 unresolved comments and a known spread test failure from an external store bug. Closes FETCH-31.

Update history

Date Change
created

Related issues

Issue Project State Summary Similarity
#837 feat: "support" existing fetch-service sessions craft-application merged Merged incremental changes adding CRAFT_USE_EXTERNAL_FETCH_SERVICE and CRAFT_PROXY_CERT environment variables for external fetch-service sessions. Approved by reviewers and passing CI, this serves as a foundational step toward full external session management.
72%
#490 feat: Detect `InspectorsConfig` and `SessionInspectorsConfig` are not in sync fetch-service merged Merged a feature detecting synchronization mismatches between InspectorsConfig and SessionInspectorsConfig. Resolves issue 485. Approved by two reviewers with all CI checks passing. Changes span 17 files.
72%
#493 docs: session inspectors conf fetch-service merged Merged documentation update for per-session inspectors configuration. Aligns JSON comment spacing for rendered output and resolves issue #488. Approved by two reviewers and passed CI checks.
72%
#490 feat: allow strict/permissive fetch-service sessions craft-application merged Merged after approval and CI checks, adding mandatory strict or permissive session arguments to the --use-fetch-service flag. Resolves issue #487.
72%
#272 feat: add fetch-service version to session metadata fetch-service merged Merged after approval from two reviewers and passing CI. Introduces a generator key to session metadata to explicitly track the fetch-service version, resolving issue #258.
71%
#371 feat: add session-aware logging support fetch-service merged 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.
70%
#450 feat: add inspector for the store resolve API fetch-service merged Merged feature adding a stateless inspector for the store resolve revisions API. Renamed the existing info API to prevent ambiguity. Approved by two reviewers and passed all CI checks.
70%