-
Notifications
You must be signed in to change notification settings - Fork 454
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
[m3nsch] support generating new unique metrics #1397
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1397 +/- ##
========================================
- Coverage 70.7% 70.7% -0.1%
========================================
Files 827 827
Lines 71238 71233 -5
========================================
- Hits 50411 50386 -25
- Misses 17521 17541 +20
Partials 3306 3306
Continue to review full report at Codecov.
|
Codecov Report
@@ Coverage Diff @@
## master #1397 +/- ##
========================================
+ Coverage 70.8% 70.8% +<.1%
========================================
Files 832 832
Lines 71400 71404 +4
========================================
+ Hits 50577 50596 +19
+ Misses 17516 17505 -11
+ Partials 3307 3303 -4
Continue to review full report at Codecov.
|
960fb22
to
40ec4e1
Compare
@@ -1,3 +1,5 @@ | |||
//+build never |
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.
lol why?
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.
It was causing lint errors or some similar build error when I was testing and I wanted to ignore it, I'll loop back and try to figure out what that was
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 save the one question
Looks like something's off with my approach here. The below screenshot is from runs with Admittedly I don't want to sink a lot of time into this if the general spirit is accomplished, so may merge as is after a bit more testing. |
@schallert yep the behaviour you're seeing is expected. The agent has a bunch of "workers" (goroutines) which are assigned a set of metrics to generate traffic for every time the workload changes - 1, 2. Subsequently, in each worker's event loop they just use the metrics they're assigned to create writes - 3. Your change is only going thru the code-path at 1/2. i.e. You're only creating new uniques when the workload is updated. You'll need to update the logic to be triggered during 3 if you want the uniques to be consistently generated. |
`m3nsch` supports generating load for a defined set of metrics, but does not currently support generating new uniques over time. This allows specifying a percentage of generated metrics that will be unique.
40ec4e1
to
cd41dff
Compare
m3nsch
supports generating load for a defined set of metrics, but doesnot currently support generating new uniques over time. This allows
specifying a percentage of generated metrics that will be unique.