← Back to issue list

Detect `InspectorsConfig` and `SessionInspectorsConfig` are not in sync

View original Github issue

Metadata

Project
fetch-service
Number
#485
Type
issue
State
closed
Author
upils
Labels
Type: Enhancement Status: Triaged
Created
Updated
Closed

Current evaluation

Implemented a reflection-based configuration synchronization check in commit db064e0, matching the issue title and validating corresponding fields and types.

Resolution: closed resolved

Reason: Commit db064e0, titled “feat: Detect `InspectorsConfig` and `SessionInspectorsConfig` are not in sync,” adds the requested synchronization validation. The repository includes the reflection-based test comparing the global and session override inspector configurations.

Evaluated by: mr-cal-bot

No scores available.

Issue body

### What needs to get done Implement a check (with reflection?) to ensure `InspectorsConfig` and `SessionInspectorsConfig` are consistent. ### Why it needs to get done `SessionInspectorsConfig` is used to override the global configuration held in a `InspectorsConfig` object. These structures must define the same fields to make sure new configurations added in `InspectorsConfig` can be overridden in `SessionInspectorsConfig`. See https://github.com/canonical/fetch-service/pull/480#discussion_r2455368154

Evaluation history

Date Model Scores Action Summary
stealth/space-bunny-alpha — closed resolved Implemented a reflection-based configuration synchronization check in commit db064e0, matching the issue title and validating corresponding fields and types.
stealth/space-bunny-alpha — closed stale Closed without implementation after the consistency check was triaged as FETCH-47; no resolving PR or commit is linked.
qwen/qwen3.6-35b-a3b — — Closed without implementation. The enhancement to verify InspectorsConfig and SessionInspectorsConfig field synchronization was triaged and logged in Jira but never addressed or merged, indicating it was abandoned or deferred.

Update history

Date Change
created

Related work

  • Likely Fixed By: fetch-service#db064e0 (confidence 98%)

    The commit message directly matches the issue, and its implementation adds the requested reflection check.

Related issues

Issue Project State Summary Similarity
#490 feat: Detect `InspectorsConfig` and `SessionInspectorsConfig` are not in sync fetch-service merged Merged PR #490 in commit db064e0, adding consistency checks for InspectorsConfig and SessionInspectorsConfig and fixing issue #485 (FETCH-47).
79%