[BUG] Delete PVC should return PVC not status

[drewmullen]

Describe the bug

Deleting a persistentvolumeclaim should return the pvc object, not Status:

harvester/api/openapi-spec/swagger.json

Lines 410 to 425 in bf5847a

	"responses": {
	"200": {
	"description": "OK",
	"content": {
	"application/json": {
	"schema": {
	"$ref": "#/components/schemas/k8s.io.v1.Status"
	}
	},
	"application/yaml": {
	"schema": {
	"$ref": "#/components/schemas/k8s.io.v1.Status"
	}
	}
	}
	},

This API comes from the k8s API which documents the appropriate return type: https://raw.githubusercontent.com/kubernetes/kubernetes/master/api/openapi-spec/swagger.json

[drewmullen]

highly suspect:

harvester/pkg/genswagger/rest/webservices.go

Lines 162 to 175 in 5ab43f0

	if slice.ContainsString(actions, http.MethodDelete) {
	ws.Route(addDeleteParams(
	ws.DELETE(ResourcePath(resource, namespaced)).
	Produces(mime.MIME_JSON, mime.MIME_YAML).
	Consumes(mime.MIME_JSON, mime.MIME_YAML).
	Operation("deleteNamespaced"+objKind).
	To(Noop).
	Reads(metav1.DeleteOptions{}).Writes(metav1.Status{}).
	Doc("Delete a "+objKind+" object.").
	Metadata("kind", objKind).
	Returns(http.StatusOK, "OK", metav1.Status{}).
	Returns(http.StatusUnauthorized, "Unauthorized", ""), ws,
	))
	}

[Vicente-Cheng]

Hi @drewmullen
Thanks for raising this!

We would check your PR with both issues #5781, #5528

Hi @Yu-Jack, @m-ildefons, could you help to check the PR #5782?
Thanks!

[drewmullen]

Thanks for accepting! Can you please confirm, I think the “reopen” was on accident? @bk201