-
Notifications
You must be signed in to change notification settings - Fork 40k
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
Rewrite monitoring.sh end-to-end test in Go #4186
Comments
I was planning to tackle this one and in the process try to find a good Go replacement for calling external Will assign it to myself and see if I can get close to a proof of concept by end of this week or beginning of the next... |
I have some pieces of monitoring.sh written in go as part of an integration Filipe I am happy to collaborate with you if that would help! |
I think one option for this test is actually to just |
+1 to that approach. That will test the cluster initialization logic as On Thu, Feb 5, 2015 at 1:21 PM, Zach Loafman notifications@github.com
|
@filbranden: Any progress to report here? I was looking at items to include in tomorrow's Fix-It and this came up. |
@filbranden: I can work on this one if you are busy. |
Assigned the issue to me. I might not get to this by tomorrow. So I will remove the "Fix-It" milestone for now. |
@vishh Any update on this one? |
Haven't gotten to this one yet. On Mon, Mar 2, 2015 at 3:32 PM, Jeff Grafton notifications@github.com
|
Part of umbrella issue #3130. cc @satnam6502 @filbranden
And cc @vishh
The text was updated successfully, but these errors were encountered: