codozor / fwkeeper

46%

main: 45%

LAST BUILD ON BRANCH feat/improve-config-validation

branch: feat/improve-config-validation

CHANGE BRANCH

x

Reset

• feat/improve-config-validation
• feat/error-classification-pod-lifecycle
• feat/forwarder-integration-tests
• feat/tests
• main

Build # 19613479581

Build Type

Pull #8

github

Committed by SebastienLaurent-CF

Commit Message

✨ feat: Improve configuration reload error messages

Enhance error reporting when configuration reload fails to provide clear,
actionable feedback to developers instead of silently ignoring errors.

## Changes

**internal/app/runner.go**
- Replaced vague error message "Failed to reload config, continuing with current configuration"
- Added explicit file path to error context with structured logging
- New message: "Configuration reload failed - keeping previous configuration. Fix the configuration file and try again"
- Helps developers quickly understand:
  * That their config changes were rejected
  * Which file has the problem
  * What action to take (fix and try again)

## Benefits

✅ Immediate feedback when config reload fails
✅ Clear indication of which file has the problem
✅ Actionable error message (not silent failure)
✅ Better developer experience when iterating on configuration
✅ All 129 tests passing with zero regressions

## Testing

- All config validation tests pass
- Config loading and validation works as expected
- Error messages are clear and informative

Pull Request Pull Request #8: ✨ feat: Improve configuration reload error messages

Run Details

0 of 4 new or added lines in 1 file covered. (0.0%)

16 existing lines in 1 file now uncovered.

369 of 798 relevant lines covered (46.24%)

0.51 hits per line