Skip to content
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.

Sign up for GitHub

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

Jump to bottom

e2e test failing consistently: external load balancer #7680

Closed
ghost opened this issue May 3, 2015 · 0 comments · Fixed by #7681
Closed

e2e test failing consistently: external load balancer #7680

ghost opened this issue May 3, 2015 · 0 comments · Fixed by #7681
Assignees
@a-robinson
Labels
priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now.
Milestone
v1.0

Comments

@ghost
Copy link

ghost commented May 3, 2015

Since this PR got merged, the ELB e2e test, which was consistently succeeding, is now consistently failing:

Revert #7145 now that #7609 is in, and fix the tests that were relying on it. (details)
Commit 890ff7b by arob
Revert #7145 now that #7609 is in, and fix the tests that were relying on it.
The file was modified pkg/cloudprovider/servicecontroller/servicecontroller.go
The file was modified pkg/cloudprovider/cloud.go
The file was modified pkg/cloudprovider/servicecontroller/servicecontroller_test.go

I'm going to roll it back to confirm.
@ghost ghost added priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now. team/cluster labels May 3, 2015
@ghost ghost assigned a-robinson May 3, 2015
@ghost ghost added this to the v1.0 milestone May 3, 2015
@ghost ghost mentioned this issue May 3, 2015
Merged
@ghost ghost closed this as completed in #7681 May 3, 2015
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@a-robinson a-robinson

Labels
priority/critical-urgent Highest priority. Must be actively worked on as someone's top priority right now.
Projects
None yet
Milestone
v1.0
Development

Successfully merging a pull request may close this issue.

Revert "Revert #7145 now that #7609 is in, and fix the tests that were r... kubernetes/kubernetes
1 participant
@a-robinson