Skip to content
This repository has been archived by the owner on Apr 27, 2021. It is now read-only.

Unit test balancer.lua #50

Closed
wants to merge 2 commits into from
Closed

Unit test balancer.lua #50

wants to merge 2 commits into from

Conversation

zrdaley
Copy link

@zrdaley zrdaley commented Apr 16, 2018

Currently no unit tests exist for ingress-nginx/rootfs/etc/nginx/lua/balancer.lua. This PR adds unit tests for the file.

Pre-requisites: #49

Related issues: https://github.com/Shopify/edgescale/issues/393

@zrdaley zrdaley force-pushed the test-balancer branch 2 times, most recently from 9ffa6a0 to 6084f38 Compare April 17, 2018 17:48
@zrdaley zrdaley changed the title Unit test balancer.lua [WIP] Unit test balancer.lua Apr 17, 2018
@zrdaley
Copy link
Author

zrdaley commented Apr 23, 2018

This is now being addressed in the upstream PR kubernetes#2410.

@zrdaley zrdaley closed this Apr 23, 2018
@zrdaley zrdaley deleted the test-balancer branch April 23, 2018 21:12
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant