Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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
Add metrics to simulator #706
Add metrics to simulator #706
Changes from 16 commits
414a7cf
41f94ec
c65a8e7
bf74d46
0ed532e
cedb3b0
461c2c6
750ed9f
5c521d7
3655c66
7b0f663
f225dfb
be68d42
173f847
a3b6020
2fef11e
c8f3da5
775e5e9
ca504d3
7395f60
c9b9604
35480f8
867b81e
4511cc5
616d9c8
131ab7d
5262218
46d1208
6666a59
c30bf85
2048ffa
99e6f27
d7d09ff
875048e
cce96a7
59a2d29
b07cfa9
710a222
d919724
63731b7
9821242
2ad7ad3
7856705
649daca
2ffda7a
e4fb332
8849b75
8e7bbdc
f7d3850
c81454e
2fb7c3b
1773e13
f3867e5
3c4f54a
e918d4d
14e1d55
ee390f5
cc51ea9
2825310
85cc2f8
ea1e609
2651d76
92bb91f
aa464fb
c3d777b
d049a98
0fcf0d7
ebe20cd
11d4295
33cbe16
5c80926
710bed0
e1cf6c3
8b87e94
a2ff6f9
d26845c
File filter
Filter by extension
Conversations
Jump to
There are no files selected for viewing
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.
If we're repeating the same logic for multiple metrics it should be broken out into its own function.
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.
yup
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.
Is there a better way to do this than creating a regex over the body?
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.
thanks
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.
the metrics API just returns a huge block of text so I have to search for exactly what I want
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.
thanks
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.
thought it would be nicer from a user reading the logs to see it 1 -indexed
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.
Fair point haha
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.
why are we returning an error from a deferred function that we're not checking?
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.
fixed
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 didn't even know you could do this in Golang, it seems pretty useful in contexts like this!