make kubectl delete pod busybox -o json work

[grosser]

I want to atomically delete an object and know it's last configuration.
DELETE api/v1/pod/NAME already returns the json, so please allow passing it back to the user.

@kubernetes/sig-cli-feature-requests

[k8s-ci-robot]

@grosser: Reiterating the mentions to trigger a notification:
@kubernetes/sig-cli-feature-requests

In response to this:

I want to atomically delete an object and know it's last configuration.
DELETE api/v1/pod/NAME already returns the json, so please allow passing it back to the user.

@kubernetes/sig-cli-feature-requests

Instructions for interacting with me using PR comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the kubernetes/test-infra repository.

[liggitt]

Not all delete methods return object json, so kubectl could not implement this consistently

[grosser]

Which ones don't ? I'd still prefer if it was possible for the ones where it returns Json.

…

On Sat, May 25, 2019, 16:36 Jordan Liggitt ***@***.***> wrote: Not all delete methods return object json, so kubectl could not implement this consistently — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#78343?email_source=notifications&email_token=AAACYZ3P65PEYXRNF572R2DPXHERBA5CNFSM4HPVBTD2YY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGODWH27ZQ#issuecomment-495955942>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AAACYZYRXLFLNR6QSFYGMYLPXHERBANCNFSM4HPVBTDQ> .

[liggitt]

Which ones don't ?

these don't:

• APIService
• AuditSink
• CertificateSigningRequest
• ClusterRole
• ClusterRoleBinding
• ConfigMap
• ControllerRevision
• CronJob
• CustomResourceDefinition (and custom resources)
• DaemonSet
• Deployment
• Endpoint
• Event
• HorizontalPodAutoscaler
• Ingress
• Job
• Lease
• LimitRange
• MutatingWebhookConfiguration
• NetworkPolicy
• Node
• PodDisruptionBudget
• PodPreset
• PriorityClass
• ReplicaSet
• ReplicationController
• Role
• RoleBinding
• RuntimeClass
• Secret
• Service
• StatefulSet
• ValidatingWebhookConfiguration

these do:

• CSIDriver
• CSINode
• Namespace
• PersistentVolume
• PersistentVolumeClaim
• Pod
• PodSecurityPolicy
• ResourceQuota
• ServiceAccount
• StorageClass
• VolumeAttachment

I'd still prefer if it was possible for the ones where it returns Json.

the wait behavior associated with delete makes it fairly difficult to do... since we can't do it consistently across resources, I'm not hugely in favor of it.

[grosser]

oh boy, did not expect it to be that complicated :(