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.

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

Implement work-queue in operator #803

Merged
merged 54 commits into from Feb 5, 2018

Conversation

Projects
None yet
2 participants
@diptadas
Copy link
Contributor

commented Jan 9, 2018

No description provided.

@diptadas diptadas requested a review from tamalsaha Jan 9, 2018

@diptadas diptadas force-pushed the workqueue branch from 6ddef70 to 9815aed Jan 10, 2018

@diptadas diptadas force-pushed the workqueue branch 5 times, most recently from 2967e0f to 92cbb8f Jan 15, 2018

@tamalsaha tamalsaha force-pushed the workqueue branch from cf378fc to ab91071 Jan 29, 2018

@tamalsaha

This comment has been minimized.

Copy link
Member

commented Jan 29, 2018

@tamalsaha tamalsaha force-pushed the workqueue branch from 383d9c4 to fac3763 Jan 30, 2018

@tamalsaha tamalsaha force-pushed the workqueue branch from ecbb926 to e51d62f Feb 3, 2018

@tamalsaha

This comment has been minimized.

Copy link
Member

commented Feb 5, 2018

[Fail] IngressWithTLSAuth Create Required Auth [It] Should response HTTP 
/home/tamal/go/src/github.com/appscode/voyager/test/e2e/ingress_tlsauth.go:194

[Fail] IngressTCP Proxy Protocol With version 2 [It] Should test decoded proxy-protocol header 
/home/tamal/go/src/github.com/appscode/voyager/test/e2e/ingress_tcp.go:328

Ran 83 of 90 Specs in 2602.112 seconds
FAIL! -- 81 Passed | 2 Failed | 0 Pending | 7 Skipped --- FAIL: TestE2E (2603.02s)
@tamalsaha

This comment has been minimized.

Copy link
Member

commented Feb 5, 2018

I retried the failed tests and those passed.

@tamalsaha tamalsaha merged commit bc0ff50 into master Feb 5, 2018

@tamalsaha tamalsaha deleted the workqueue branch Feb 5, 2018

@tamalsaha

This comment has been minimized.

Copy link
Member

commented Feb 5, 2018

I re-run all the tests using the informer version of the haproxy-controller that is included in the HAProxy container. All tests passed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.