-
Notifications
You must be signed in to change notification settings - Fork 27
Load balancer example check improvement #82
Comments
@uablrek can you please check if we can have a more sophisticated check? |
Yes I will try. Thanks for fixing the ci problems, I can't say I have understood those. Now when it works I can try to extend the testing. My problem has been that the base-image for NSE lacks every tcp test tool I could think of, only "ping" exists, so test from within the NSE pod is hard. |
Oh, you can install whatever you need even in the check script if you want to. If it is Alpine based it is a matter of |
Tried to add install of
But unfortunately any "apt install" fails in the "${VPP_AGENT}" image;
I tried a number of packages and all fails. I will try something else. |
So the images get distributed with cleaned package cache to make time lighter weight. The typical way to make this work is like this:
|
Seems to work fine. Thanks! |
Fixes networkservicemesh#82 Signed-off-by: Lars Ekman <[email protected]>
Fixes networkservicemesh#82 Signed-off-by: Lars Ekman <[email protected]>
Fixes networkservicemesh#82 Signed-off-by: Lars Ekman <[email protected]>
The load balancer example check script is not verifying the traffic is passing and distributed by the balancer. We need a more sophisticated check that will ensure the balancer is operating as expected.
The text was updated successfully, but these errors were encountered: