Improve GetRuntimeClient compatibility

[lbbniu]

What type of PR is this?

What this PR does / why we need it:

Which issue(s) this PR fixes:

Fixes #232 #251 #276

Special notes for your reviewer:

[github-actions]

🎊 PR Preview 76d5644 has been successfully built and deployed to https://gocrane-crane-preview-pr-273.surge.sh

🕐 Build time: 1.573s

_{🤖 By surge-preview}

[qmhu]

@chenkaiyue Please take a look

[chenkaiyue]

Thanks for your contributions. Have you tested in Kind or K3S, maybe you can paste test images here?

[github-actions]

🎉 Successfully Build Images.

Overview: https://finops.coding.net/public-artifacts/gocrane/crane/packages

Image	Pull Command
crane-agent:pr-273-6eef366	docker pull finops-docker.pkg.coding.net/gocrane/crane/crane-agent:pr-273-6eef366
dashboard:pr-273-6eef366	docker pull finops-docker.pkg.coding.net/gocrane/crane/dashboard:pr-273-6eef366
metric-adapter:pr-273-6eef366	docker pull finops-docker.pkg.coding.net/gocrane/crane/metric-adapter:pr-273-6eef366
craned:pr-273-6eef366	docker pull finops-docker.pkg.coding.net/gocrane/crane/craned:pr-273-6eef366

[zsnmwy]

I have some tests in finops-docker.pkg.coding.net/gocrane/crane/crane-agent:pr-273-c744466, k8s v1.23.5.

Connect container runtime endpoint successfully, but have some permission problems.

    1 connection.go:42] connect endpoint 'unix:///var/run/dockershim.sock', make sure you are running as root and the endpoint has been started
    1 connection.go:26] Connect using endpoint 'unix:///run/containerd/containerd.sock' with '2s' timeout
    1 connection.go:44] Connected successfully using endpoint: unix:///run/containerd/containerd.sock
    1 agent.go:157] Failed to get noderesource tsp configmap : configmaps "noderesource-tsp-template" is forbidden: User "system:serviceaccount:crane-system:crane-agent" cannot
    1 agent.go:202] Discover the presence of old noderesource tsp and try to contrast the changes: noderesource-demo
    1 agent.go:206] Discover the presence of old noderesource tsp and the Tsp rules have been changed: noderesource-demo
    1 request.go:936] Error in request: resource name may not be empty
    1 agent.go:210] Failed to update noderesource tsp noderesource-demo : resource name may not be empty
    1 agent.go:60] resource name may not be empty

[lbbniu]

k3s connect container runtime endpoint successfully:

[qmhu]

https://github.com/gocrane/helm-charts/pull/46/files is this pr related to this change?

[zsnmwy]

https://github.com/gocrane/helm-charts/pull/46/files is this pr related to this change?

yes
see: https://github.com/gocrane/crane/pull/273/files#diff-f915bb96af9fbe90557cb2b8ede3eb7903ddae1297dad168074182cd3e55eb2c

[lbbniu]

https://github.com/gocrane/helm-charts/pull/46/files is this pr related to this change?

ok

[lbbniu]

I have some tests in finops-docker.pkg.coding.net/gocrane/crane/crane-agent:pr-273-c744466, k8s v1.23.5.

Connect container runtime endpoint successfully, but have some permission problems.

    1 connection.go:42] connect endpoint 'unix:///var/run/dockershim.sock', make sure you are running as root and the endpoint has been started
    1 connection.go:26] Connect using endpoint 'unix:///run/containerd/containerd.sock' with '2s' timeout
    1 connection.go:44] Connected successfully using endpoint: unix:///run/containerd/containerd.sock
    1 agent.go:157] Failed to get noderesource tsp configmap : configmaps "noderesource-tsp-template" is forbidden: User "system:serviceaccount:crane-system:crane-agent" cannot
    1 agent.go:202] Discover the presence of old noderesource tsp and try to contrast the changes: noderesource-demo
    1 agent.go:206] Discover the presence of old noderesource tsp and the Tsp rules have been changed: noderesource-demo
    1 request.go:936] Error in request: resource name may not be empty
    1 agent.go:210] Failed to update noderesource tsp noderesource-demo : resource name may not be empty
    1 agent.go:60] resource name may not be empty

@qmhu @chenkaiyue Should this bug be fixed by this pr or fixed by a separate pr?

[lbbniu]

kind connect container runtime endpoint successfully:

[chenkaiyue]

Please squash these commits.

[lbbniu]

Please squash these commits.

Finish

[qmhu]

/LGTM

nice job!

[chenkaiyue]

/LGTM