-
Notifications
You must be signed in to change notification settings - Fork 59
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update Dependencies - Part 2 #103
Conversation
{lvl: logrus.InfoLevel, msg: "config.metrics.influx.host: "}, | ||
{lvl: logrus.InfoLevel, msg: "config.metrics.influx.database: "}, | ||
{lvl: logrus.InfoLevel, msg: "config.metrics.influx.measurement: "}, | ||
{lvl: logrus.InfoLevel, msg: "config.metrics.influx.align_timestamps: false"}, |
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.
I restructured the code to put each log expectation on its own line, which made it easier for me to read and understand.
Measurement string `mapstructure:"measurement"` | ||
Username string `mapstructure:"username"` | ||
Password string `mapstructure:"password"` | ||
AlignTimestamps bool `mapstructure:"align_timestamps"` |
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.
Should these two new values have to be added to func setConfigDefaults(v *viper.Viper)
?
@@ -10,7 +10,7 @@ import ( | |||
influxdb "github.com/vrischmann/go-metrics-influxdb" | |||
) | |||
|
|||
var TenSeconds time.Duration = time.Second * 10 | |||
const TenSeconds time.Duration = time.Second * 10 |
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.
👍
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.
Local testing looks good
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
github.com/prometheus/client_golang
github.com/rcrowley/go-metrics
github.com/vrischmann/go-metrics-influxdb
All metrics related dependencies have been updated to the same versions we are using in Prebid Server. We haven't seen any issues with these updates in Prebid Server, so I expect no issues for Prebid Cache. This does include breaking changes / new required fields for Influx metrics. Please see the details on the Prebid Server release notes: https://github.com/prebid/prebid-server/releases/tag/v0.198.0
github.com/sirupsen/logrus
We were 6 releases behind. The release notes indicate mostly just bug fixes.
github.com/stretchr/testify
We were only 1 minor release behind. This is used exclusively for tests and tests still pass.