Conversation
e9190c8
to
574e0d6
Compare
@idoqo CI is failning |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Let's create feature build to make sure that nothing was broken
aa4f36d
to
7f46339
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
@@ -55,7 +55,7 @@ func NewMySQLExplainAction(id string, params *agentpb.StartActionRequest_MySQLEx | |||
if params.TlsFiles != nil && params.TlsFiles.Files != nil { | |||
err := tlshelpers.RegisterMySQLCerts(params.TlsFiles.Files) | |||
if err != nil { | |||
log.Error(err) | |||
logrus.Error(err) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not sure if the move to logrus brings any UX change on binary output front. Let me know if this switch is okay.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah I don't think it does either. This was the only place we used it and it wasn't customized in any way. Prometheus' common/log used to be a wrapper around logrus and has been deprecated for a while, it just got removed recently (prometheus/common#306)
527d811
to
beb37fa
Compare
@idoqo please fix merge conflicts |
Actually, |
Updates prometheus client due to CVE-2022-21698 which causes failure in WhiteSource security checks.
Build: SUBMODULES-2366