nanobox-io / golang-lvs Goto Github PK

Hi, Have been playing with this package to see if i can build a layer 4 load balancer with this.

This package assumes that ipvsadm is installed on the machine + there are enough privileges to get values returned. I could be wrong, but errors from lvs.Ipvs.Services are not returned when ipvsadm is missing on the host, this makes debugging a bit harder.

Example code:

package main

import (
	"fmt"

	lvs "github.com/nanobox-io/golang-lvs"
)

var listIpvs lvs.Ipvs

func main() {
	listIpvs.Save()
	output := listIpvs.Services

	fmt.Println(output)
}

Returns [] when either privileges are not there or ipvsadm is not installed.

Perhaps it is an idea to add requirements in the README or perhaps check in the code if ipvsadm / privileges are there.

Without being able to change the "backend/backendStdin/backendRun" or having defined interfaces for Ipvs, Service, etc. it is difficult to mock out this library. Would you be open to either exposing the backend variables or providing a way to set them or creating interfaces for the existing structs? I am happy to create a PR to help.

I have had to resort to creating a set of adapters around the library in order to mock it, however this is time consuming and only benefits me. So i'm happy to contribute in any way.

thanks.

Currently only supports tcp.