There exists some config that cannot be
shared across all providers. For example,
the port for ipmitool is generally 623, but
is generally 16992 for intelAMT. Allowing
per provider config enables customizing this
type of data.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>

This allows the provider to specify all config it needs.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>

This makes New functions cleaner.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>

Add functional arguments for optional Ipmi struct
values. Add additional struct values. Make cipher
suite 3 the first suite tried. 3 is more commonly
used then 17 so most machines will not try with 17 first.
Add debug logging of ipmitool command line options.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>

The new function signature does not return
an error.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>

This uniforms providers to using func args.
Updates the client to work with this.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>

Fix basic auth test and example.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>

The registering of ipmitool provider doesnt
ignore the error when its constructor is called.
This means that systems without ipmitool will see
that the ipmitool provider does not get registered.
Previously it would always be registered.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>

Allow for a 10 millisecond error of margin.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>

Previously it was not.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>

This makes it more flexible and follows the pattern the
other providers are using.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>

gofish is an implementation detail that shouldnt
be exposed like this.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>

Clean up.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>

Extra precaution for race conditions.

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>

`-race` flag was added to `make test` and requires CGO.
go: -race requires cgo; enable cgo by setting CGO_ENABLED=1

Signed-off-by: Jacob Weinstock <jakobweinstock@gmail.com>