fabric8-services / fabric8-jenkins-proxy

Apache License 2.0
1 stars 16 forks source link

Fix #278 Implement a requestid identifier to correlate queries #279

Closed chmouel closed 6 years ago

chmouel commented 6 years ago

Let's use this to pass to idler so we can correlate the action between idler and proxy.

We change the logging facility from Debug to Info for the Idle/Unidle but leave it as Debug for State so we don't fill up too much the logs with noise.

Only testing was added for the new function. Writting tests for the whole clients would need quite a bit of refactoring to use intefaces.

centos-ci commented 6 years ago

Can one of the admins verify this patch?

chmouel commented 6 years ago

Trying to test this locally with a modified idler and proxy proven to be too hacky and buggy, i'll merge this since it's harmless and will be able to develop idler path after