-
Notifications
You must be signed in to change notification settings - Fork 1.3k
Config parsing errors should include document + line context #2927
Copy link
Copy link
Open
Labels
priority: p1Important issue which blocks shipping the next release. Will be fixed prior to next release.Important issue which blocks shipping the next release. Will be fixed prior to next release.type: feature request‘Nice-to-have’ improvement, new feature or different behavior or design.‘Nice-to-have’ improvement, new feature or different behavior or design.
Description
Title:
Config parse errors should include document + line/column
Prerequisites
- I searched the current open issues.
What are you trying to do that currently feels hard or impossible?
Quickly locate config errors in large multi‑doc YAML files.
Clear end goal (specific use‑case)
When the config is missing kind or name, I want the error to show doc index + line/column, so I can fix it without hunting.
Suggested Solution(s)
- Include doc index and line/column in errors from
UnmarshalResourceConfig. - If
nameis present but other fields are invalid, include the name in the
error. - Apply similar context to env‑var parsing errors.
Alternatives Considered
Manual search / binary search of YAML — not great.
Additional Details
Code pointers:
internal/server/config.go→UnmarshalResourceConfigcmd/internal/config.go→ParseConfig/ConvertConfig
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
priority: p1Important issue which blocks shipping the next release. Will be fixed prior to next release.Important issue which blocks shipping the next release. Will be fixed prior to next release.type: feature request‘Nice-to-have’ improvement, new feature or different behavior or design.‘Nice-to-have’ improvement, new feature or different behavior or design.