go vet + golint 100%

main.go

@@ -37,7 +37,7 @@ func main() {
 	systemdDbus, err := dbus.New()
 	check(err)
 
-	systemd := &system.Systemd{systemdDbus}
+	systemd := &system.Systemd{Conn: systemdDbus}
 
 	system := system.System{
 		Filesystem: &file.Atomic{},

pkg/backoff/backoff_test.go

@@ -19,7 +19,7 @@ func TestEmptyBackoff(t *testing.T) {
 }
 
 func TestJitteredBackoff(t *testing.T) {
-	seq := backoff.Backoff{[]int{1, 2, 4, 8}}
+	seq := backoff.Backoff{Seq: []int{1, 2, 4, 8}}
 
 	if seq.Duration(1) == seq.Duration(1) {
 		t.Error("Jitter should ensure calls are not equal")

pkg/eks/cluster.go

@@ -13,7 +13,7 @@ import (
 	"github.com/aws/aws-sdk-go/service/eks/eksiface"
 )
 
-var b = backoff.Backoff{[]int{1, 2, 4, 8, 16, 32, 64}}
+var b = backoff.Backoff{Seq: []int{1, 2, 4, 8, 16, 32, 64}}
 
 // Cluster returns the named EKS cluster.
 //

pkg/file/atomic.go

@@ -10,6 +10,7 @@ import (
 	"syscall"
 )
 
+//Atomic exposes an interface to atomicly write config files to the filesystem
 type Atomic struct{}
 
 // Sync atomicly writes data to a file at the given path with the given permissions

pkg/system/system.go

@@ -11,24 +11,28 @@ import (
 	"text/template"
 )
 
-type Filesystem interface {
+type filesystem interface {
 	Sync(io.Reader, string, os.FileMode) error
 }
 
-type Init interface {
+type initsystem interface {
 	EnsureRunning(string) error
 }
 
-type Hostname interface {
+type hostname interface {
 	SetHostname(string) error
 }
 
+// System represents the system we are configuring and
+// should be created with the interfaces to interact with it
 type System struct {
-	Filesystem Filesystem
-	Init       Init
-	Hostname   Hostname
+	Filesystem filesystem
+	Init       initsystem
+	Hostname   hostname
 }
 
+// Configure configures the system to connect to the EKS cluster given the node
+// and cluster metadata provided as arguments
 func (s System) Configure(n *node.Node, cluster *eks.Cluster) error {
 	if err := s.Hostname.SetHostname(*n.PrivateDnsName); err != nil {
 		return err
@@ -81,7 +85,7 @@ func base64decode(v string) (string, error) {
 type config struct {
 	template   *template.Template
 	path       string
-	filesystem Filesystem
+	filesystem filesystem
 }
 
 func (c config) write(data interface{}) error {

pkg/system/systemd_test.go

@@ -48,7 +48,7 @@ func TestEnsureRunning(t *testing.T) {
 	for _, tC := range testCases {
 		t.Run(tC.desc, func(t *testing.T) {
 			d := &fakeDbusConn{}
-			s := &system.Systemd{d}
+			s := &system.Systemd{Conn: d}
 
 			err := s.EnsureRunning(tC.unit)
 			if err != nil {
@@ -105,7 +105,7 @@ func TestErrorHandling(t *testing.T) {
 			errs := make(map[string]error)
 			errs[tC.name] = tC.err
 			d := &fakeDbusConn{errors: errs}
-			s := &system.Systemd{d}
+			s := &system.Systemd{Conn: d}
 			err := s.EnsureRunning("kubelet.service")
 			if err != tC.err {
 				t.Errorf("Got error: %v, expected %v", err, tC.err)