clusterpedia icon indicating copy to clipboard operation
clusterpedia copied to clipboard

`binding-apiserver` could not delete pods

Open huiwq1990 opened this issue 2 years ago • 3 comments

What happened?

When delete pod in subcluster, binding-apiserver could not delete pods.

binding-apiserver error logs: image kubectl --cluster clusterpedia get pod image

What did you expect to happen?

binding-apiserver could delete pods in cache.

How can we reproduce it (as minimally and precisely as possible)?

Use latest clusterpedia version.

Anything else we need to know?

I fixed it by modify pkg/synchromanager/clustersynchro/resource_synchro.go which pass real object to delete event.

image

Clusterpedia Version

~ kubectl exec -it -n clusterpedia-system clusterpedia-binding-apiserver-d9dbc5b56-lzrb5 -- binding-apiserver --version
Clusterpedia v0.7.0

Host Kubernetes version

~ kubectl version
WARNING: This version information is deprecated and will be replaced with the output from kubectl version --short.  Use --output=yaml|json to get the full version.
Client Version: version.Info{Major:"1", Minor:"27", GitVersion:"v1.27.3", GitCommit:"25b4e43193bcda6c7328a6d147b1fb73a33f1598", GitTreeState:"clean", BuildDate:"2023-06-14T09:47:38Z", GoVersion:"go1.20.5", Compiler:"gc", Platform:"darwin/arm64"}
Kustomize Version: v5.0.1
Server Version: version.Info{Major:"1", Minor:"27", GitVersion:"v1.27.3", GitCommit:"25b4e43193bcda6c7328a6d147b1fb73a33f1598", GitTreeState:"clean", BuildDate:"2023-06-15T00:38:14Z", GoVersion:"go1.20.5", Compiler:"gc", Platform:"linux/arm64"}

huiwq1990 avatar Sep 12 '23 12:09 huiwq1990

Hi @huiwq1990, Thanks for opening an issue! We will look into it as soon as possible.

Details

Instructions for interacting with me using comments are available here. If you have questions or suggestions related to my behavior, please file an issue against the gh-ci-bot repository.

clusterpedia-bot avatar Sep 12 '23 12:09 clusterpedia-bot

@wuyingjun-lucky @xyz2277 could you take a look at this, please?

Iceber avatar Sep 20 '23 02:09 Iceber

https://github.com/clusterpedia-io/clusterpedia/pull/592#pullrequestreview-1737075720

But https://github.com/clusterpedia-io/clusterpedia/pull/592 also requires the memory storage layer to support PartialObjectMetadata, which of course may require other issues and pr to resolve

Also I think that not getting the resource version when deleting may have an effect on https://github.com/clusterpedia-io/clusterpedia/pull/593 as well.

Iceber avatar Nov 20 '23 03:11 Iceber