Skip checking resources when `--wait=false` is specified
- Dominant language
- Go
- Stars
- 1.1k
- Forks
- 134
- PR merge metrics
- No merged PRs in 30d
Description
**What steps did you take:**
I currently use Kapp as a CI tool to manage lots of YAML files. I used `--wait=false` when I deleted the app because sometimes deleting custom resources will take a long time.
**What happened:**
kapp exits with non-zero code which makes CI fail.
```
$ kapp delete -a app1 --wait=false -y
Target cluster 'https://127.0.0.1:6443' (nodes: firgavin)
Changes
Namespace Name Kind Age Op Op st. Wait to Rs Ri
default simple-app Deployment 22s delete - - ok -
^ simple-app Service 22s delete - - ok -
Op: 0 create, 2 delete, 0 update, 0 noop, 0 exists
Wait to: 0 reconcile, 0 delete, 2 noop
11:18:59AM: ---- applying 2 changes [0/2 done] ----
11:18:59AM: delete deployment/simple-app (apps/v1) namespace: default
11:18:59AM: delete service/simple-app (v1) namespace: default
11:18:59AM: ---- waiting on 2 changes [0/2 done] ----
11:18:59AM: ok: noop service/simple-app (v1) namespace: default
11:18:59AM: ok: noop deployment/simple-app (apps/v1) namespace: default
11:18:59AM: ---- applying complete [2/2 done] ----
11:18:59AM: ---- waiting complete [2/2 done] ----
kapp: Error: Expected all resources to be gone, but found: endpointslice/simple-app-vp2dw (discovery.k8s.io/v1) namespace: default, pod/simple-app-64c66864f5-g9sb8 (v1) namespace: default, replicaset/simple-app-64c66864f5 (apps/v1) namespace: default
```
**What did you expect:**
Kapp could skip checking resources when `--wait=false` is specified.
**Anything else you would like to add:**
I did some research and I found that kapp checks the existence of related resources after applying changes. But resources will be deleted eventually. See https://github.com/vmware-tanzu/carvel-kapp/blob/v0.52.0/pkg/kapp/cmd/app/delete.go#L159.
It would be great if kapp could default to skipping checking resources when `--wait=false` is specified or add a flag to control this logic. And if that makes sense, I'd like to help implement this ;)
**Environment:**
- kapp version (use `kapp --version`): v0.52.0
- OS (e.g. from `/etc/os-release`): Ubuntu 20.04.4 LTS
- Kubernetes version (use `kubectl version`): v1.23.6+k3s1
---
Vote on this request
This is an invitation to the community to vote on issues, to help us prioritize our backlog. Use the "smiley face" up to the right of this comment to vote.
👍 "I would like to see this addressed as soon as possible"
👎 "There are other more important things to focus on right now"
We are also happy to receive and review Pull Requests if you want to help working on this issue.
Contributor guide
Assessment
This issue has not been assessed yet.