-
Notifications
You must be signed in to change notification settings - Fork 1
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
Support check.test requests (CMC-1506) #139
Conversation
poller/session.go
Outdated
|
||
newCheck, err := check.NewCheckParsed(s.ctx, *req.Params) | ||
if err != nil { | ||
resp := protocol.NewErrorResponse(f, 1, "Unable to interpret the given check for testing") |
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.
@rphillips is there a strategy from virgo agent to define appropriate error code values?
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.
es := poller.NewSession(context.Background(), eleConn, reconciler, cfg) | ||
defer func() { time.Sleep(5 * time.Millisecond) }() |
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 was seeing some sporadic unit test failures here...but pushing so I can switch to working on ping/ICMP
…dd-check-test # Conflicts: # check/checkresult.go
@@ -172,7 +174,11 @@ type MetricTVU struct { | |||
Unit string `json:"u"` | |||
} | |||
|
|||
type MetricsPostRequestParams struct { | |||
func (m *MetricTVU) String() string { | |||
return fmt.Sprintf("{t=%v, v=%v, u=%v}", m.Type, m.Value, m.Unit) |
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.
nice
TODO