cloud-provider-openstack icon indicating copy to clipboard operation
cloud-provider-openstack copied to clipboard

[all]: openstack API debugger shows `client.go:128` for all calls

Open kayrus opened this issue 2 years ago • 3 comments

Is this a BUG REPORT or FEATURE REQUEST?:

/kind bug

What happened:

I noticed that all API debug logs have client.go:128 prefix from the https://github.com/kubernetes/cloud-provider-openstack/issues/2295#issuecomment-1634366621 logs

What you expected to happen:

Logs should show the actual place of the call, e.g. in 1.25 it is:

I0713 11:50:50.686815       1 openstack_volumes.go:82]   "volumes_links": [
I0713 11:50:50.686831       1 openstack_volumes.go:82]     {
I0713 11:50:50.686845       1 openstack_volumes.go:82]       "href": "https://cinder-v3/v3/4d0477a019414ee08ad0fd11b777eee2/volumes/detail?limit=2\u0026marker=faf60902-227c-4812-ac35-fb5f1aea7e7f",
I0713 11:50:50.686863       1 openstack_volumes.go:82]       "rel": "next"
I0713 11:50:50.686877       1 openstack_volumes.go:82]     }
I0713 11:50:50.686892       1 openstack_volumes.go:82]   ]
I0713 11:50:50.686909       1 openstack_volumes.go:82] }

How to reproduce it:

n/a

Anything else we need to know?:

https://github.com/kubernetes/cloud-provider-openstack/pull/867

Environment:

  • openstack-cloud-controller-manager(or other related binary) version:
  • OpenStack version:
  • Others:

kayrus avatar Jul 14 '23 08:07 kayrus

good catch~

you mean #867 might be the root cause or it's reference that previously it works fine?

jichenjc avatar Jul 17 '23 03:07 jichenjc

@jichenjc it's a reference for the initial implementation. I don't know at what point it stopped to work, but for sure it works fine in 1.25.x

kayrus avatar Jul 17 '23 09:07 kayrus

/assign

majorchork avatar Oct 19 '23 00:10 majorchork