Fix lint issues and enable test check

aws · Oct 12, 2020 · 1c1d4b9 · 1c1d4b9
1 parent e96164e
commit 1c1d4b9
Show file tree

Hide file tree

Showing 14 changed files with 14 additions and 14 deletions.
diff --git a/.travis.yml b/.travis.yml
@@ -32,7 +32,7 @@ install:
 script:
   - make check-format
   - make build-linux
-  - make lint LINT_FLAGS=
+  - make lint
   - make vet
   - make unit-test
 

diff --git a/cmd/routed-eni-cni-plugin/driver/driver.go b/cmd/routed-eni-cni-plugin/driver/driver.go
@@ -11,7 +11,7 @@
 // express or implied. See the License for the specific language governing
 // permissions and limitations under the License.
 
-// CNI network driver setting up iptables, routes and rules
+// Package driver is the CNI network driver setting up iptables, routes and rules
 package driver
 
 import (

diff --git a/pkg/awsutils/awsutils.go b/pkg/awsutils/awsutils.go
@@ -11,7 +11,7 @@
 // express or implied. See the License for the specific language governing
 // permissions and limitations under the License.
 
-// Util package to interact with EC2
+// Package awsutils is a utility package for calling EC2 or IMDS
 package awsutils
 
 import (

diff --git a/pkg/cri/cri.go b/pkg/cri/cri.go
@@ -11,7 +11,7 @@
 // express or implied. See the License for the specific language governing
 // permissions and limitations under the License.
 
-// Package to get running sandboxes from the CRI socket
+// Package cri is used to get running sandboxes from the CRI socket
 package cri
 
 import (

diff --git a/pkg/grpcwrapper/client.go b/pkg/grpcwrapper/client.go
@@ -11,7 +11,7 @@
 // express or implied. See the License for the specific language governing
 // permissions and limitations under the License.
 
-// Wrapper for the ipamd client Dial interface
+// Package grpcwrapper is a wrapper for the ipamd client Dial interface
 package grpcwrapper
 
 import (

diff --git a/pkg/ipwrapper/ip.go b/pkg/ipwrapper/ip.go
@@ -11,7 +11,7 @@
 // express or implied. See the License for the specific language governing
 // permissions and limitations under the License.
 
-// Wrapper interface for the containernetworking ip plugin
+// Package ipwrapper is a wrapper interface for the containernetworking ip plugin
 package ipwrapper
 
 import (

diff --git a/pkg/netlinkwrapper/netlink.go b/pkg/netlinkwrapper/netlink.go
@@ -11,7 +11,7 @@
 // express or implied. See the License for the specific language governing
 // permissions and limitations under the License.
 
-// Wrapper methods for the netlink package
+// Package netlinkwrapper is a wrapper methods for the netlink package
 package netlinkwrapper
 
 import (

diff --git a/pkg/networkutils/network.go b/pkg/networkutils/network.go
@@ -11,7 +11,7 @@
 // express or implied. See the License for the specific language governing
 // permissions and limitations under the License.
 
-// Collection of iptables and netlink functions
+// Package networkutils is a collection of iptables and netlink functions
 package networkutils
 
 import (

diff --git a/pkg/nswrapper/ns.go b/pkg/nswrapper/ns.go
@@ -11,7 +11,7 @@
 // express or implied. See the License for the specific language governing
 // permissions and limitations under the License.
 
-// Wrapper interface for the containernetworking ns plugin
+// Package nswrapper is a wrapper interface for the containernetworking ns plugin
 package nswrapper
 
 import (

diff --git a/pkg/procsyswrapper/procsys.go b/pkg/procsyswrapper/procsys.go
@@ -11,7 +11,7 @@
 // express or implied. See the License for the specific language governing
 // permissions and limitations under the License.
 
-// Wrapper package for doing /proc/sys calls
+// Package procsyswrapper contains helper functions for doing /proc/sys calls
 package procsyswrapper
 
 import (

diff --git a/pkg/rpcwrapper/client.go b/pkg/rpcwrapper/client.go
@@ -11,7 +11,7 @@
 // express or implied. See the License for the specific language governing
 // permissions and limitations under the License.
 
-// Wrapper for the CNI IPAMD pod connection
+// Package rpcwrapper is a wrapper for the CNI IPAMD pod connection
 package rpcwrapper
 
 import (

diff --git a/pkg/typeswrapper/client.go b/pkg/typeswrapper/client.go
@@ -11,7 +11,7 @@
 // express or implied. See the License for the specific language governing
 // permissions and limitations under the License.
 
-// Wrapper interface for containernetworking types package
+// Package typeswrapper is a wrapper interface for containernetworking types package
 package typeswrapper
 
 import (

diff --git a/pkg/utils/logger/logger.go b/pkg/utils/logger/logger.go
@@ -11,7 +11,7 @@
 // express or implied. See the License for the specific language governing
 // permissions and limitations under the License.
 
-// CNI Logger interface, using zap
+// Package logger is the CNI Logger interface, using zap
 package logger
 
 //Log is global variable so that log functions can be directly accessed

diff --git a/pkg/utils/retry/retry.go b/pkg/utils/retry/retry.go
@@ -11,7 +11,7 @@
 // express or implied. See the License for the specific language governing
 // permissions and limitations under the License.
 
-// Retry with backoff implementation
+// Package retry is a retry with backoff implementation
 package retry
 
 import (