search

carvel-dev / kapp

kapp is a simple deployment tool focused on the concept of "Kubernetes application" — a set of resources with the same label
https://carvel.dev/kapp
Apache License 2.0
899 stars 108 forks source link

figure out better error/path-forward when user provides resource with uid/creationTimestamp filled in #125

Open cppforlife opened 4 years ago

cppforlife commented 4 years ago

currently kapp will panic due to use of ExistingResource() returning nil: https://github.com/k14s/kapp/blob/develop/pkg/kapp/clusterapply/changes_view.go#L115-L116

iamhsk commented 3 years ago

@cppforlife Could you share how to reproduce this? In the code snippet you provided, don't see where it would panic.