apps-cli-plugin
apps-cli-plugin copied to clipboard
As an app dev, reviewing messages in `...workload get` output, I can see the fully qualified reference to the resource which emitted the error
implementing this change is dependent upon updates to cartographer
There's an ask for them to add a field to their status which provides the fully qualified reference to the resource that's emitting any given message (error or otherwise) which the apps plugin displays in workload get
output.
Description of problem
There's currently no obvious and deterministic way for a developer to associate a message that's displayed in the "Messages" section of workload get
output with a specific resource.
The absence of this association makes it difficult for a developer to troubleshoot errors when they occur.
Proposed solution
Include the fully qualified reference to the emitting resource in the message that's displayed.
@danfein - please review/update/approve
Example of a current message
Examples
TO DO - provide multiple examples rather than just one Current message:
Workload [MissingValueAtPath]: waiting to read value [.status.latestImage] from resource [image.kpack.io/petc] in namespace [default]
New version of message:
Workload [image.kpack.io/petc]: waiting to read value [.status.latestImage] from resource [image.kpack.io/petc] in namespace [default]
Current message:
Deliverable [HealthyConditionRule]: field value: True, message: kapp: Error: Applying update service/petc (serving.knative.dev/v1) namespace: default:
Updating resource service/petc (serving.knative.dev/v1) namespace: default:
API server says: admission webhook "validation.webhook.serving.knative.dev" denied the request: validation failed: "PORT" is a reserved environment variable: spec.template.spec.containers[0].env[0].name (reason: BadRequest)
New version of message:
PENDING