Closed ityuhui closed 4 years ago
Some comments. We should consider adding valgrind
testing to catch memory leaks.
Thank you @brendandburns. I have addressed your comments. Thank you for your finding about these memory leaks.
And I also have run valgrind
to check memory and logged two issues #13,#15
I will add the valgrind memory check to Github/Actions sometimes later
/lgtm /approve
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: brendandburns, ityuhui
The full list of commands accepted by this bot can be found here.
The pull request process is described here
support
exec
for kube config ( https://kubernetes.io/docs/reference/access-authn-authz/authentication/#configuration )/cc @brendandburns