promethues_scaler: add url in error information

[surki]

Without the query its hard to debug which query is causing the error when there are hundreds of scalers.

Checklist

• Commits are signed with Developer Certificate of Origin (DCO - learn more)
• Tests have been added
• A PR is opened to update our Helm chart (repo) (if applicable, ie. when deployment manifests are modified)
• A PR is opened to update the documentation on (repo) (if applicable)
• Changelog has been updated and is aligned with our changelog requirements

[zroubalik]

I wonder whether it is not a privacy problem to expose the full url in the logs. Let me get back to you next week.

[tomkerkhove]

I wonder whether it is not a privacy problem to expose the full url in the logs. Let me get back to you next week.

I don't think it hurts; if you have access to the logs then it's fair to say you are a cluster admin; no? I do get the concern though.

[JorTurFer]

I don't think it hurts; if you have access to the logs then it's fair to say you are a cluster admin; no? I do get the concern though.

I wouldn't assume that people with access to logs are admins... ie, In Docplanner all devs have access to all the logs in the clusters but not to secrets or configs. I think that the url itself is not a secret (at this point in code, that variable contains only the url, not the url with user and password) but maybe I'm wrong

[zroubalik]

Would following feature help?
#3419

[surki]

Would following feature help? #3419

yes, that should be good enough (as long as full name is there, namespace + name)

I will close this, thanks.