Showing 52 of 52 total issues
Avoid deeply nested control flow statements. Open
Open
if err != nil {
glog.V(2).Infof("error decoding private key #%d from transport profile %s", i, rule.Transport)
} else {
privateKey = k
}
Your code does not pass gofmt in 1 place. Go fmt your code! Open
Open
// +build !linux
- Exclude checks
type name will be used as fake.FakeDomain by other packages, and that stutters; consider calling this Domain Open
Open
type FakeDomain struct {
- Exclude checks
type name will be used as fake.FakeSecret by other packages, and that stutters; consider calling this Secret Open
Open
type FakeSecret struct {
- Exclude checks
method getEmulatorUidGid should be getEmulatorUIDGid Open
Open
func (fs *realFileSystem) getEmulatorUidGid() (int, int, error) {
- Exclude checks
type name will be used as fake.FakeStoragePool by other packages, and that stutters; consider calling this StoragePool Open
Open
type FakeStoragePool struct {
- Exclude checks
Your code does not pass gofmt in 1 place. Go fmt your code! Open
Open
/*
- Exclude checks
Your code does not pass gofmt in 1 place. Go fmt your code! Open
Open
// +build linux
- Exclude checks
Your code does not pass gofmt in 1 place. Go fmt your code! Open
Open
// +build !linux
- Exclude checks
should replace nextToUseHostdevNo += 1 with nextToUseHostdevNo++ Open
Open
nextToUseHostdevNo += 1
- Exclude checks
don't use ALL_CAPS in Go names; use CamelCase Open
Open
FAMILY_ALL = netlink.FAMILY_ALL
- Exclude checks
don't use ALL_CAPS in Go names; use CamelCase Open
Open
FAMILY_ALL = 0
- Exclude checks
Your code does not pass gofmt in 1 place. Go fmt your code! Open
Open
/*
- Exclude checks
exported func NewClient returns unexported type *cni.client, which can be annoying to use Open
Open
func NewClient(pluginsDir, configsDir string) (*client, error) {
- Exclude checks
don't use ALL_CAPS in Go names; use CamelCase Open
Open
RTPROT_KERNEL = syscall.RTPROT_KERNEL
- Exclude checks
exported function GenerateCert should have comment or be unexported Open
Open
func GenerateCert(t *testing.T, isCA bool, host string, signer *x509.Certificate, key *rsa.PrivateKey) (*x509.Certificate, *rsa.PrivateKey) {
- Exclude checks
Your code does not pass gofmt in 1 place. Go fmt your code! Open
Open
/*
- Exclude checks
Your code does not pass gofmt in 1 place. Go fmt your code! Open
Open
// +build !linux
- Exclude checks
Your code does not pass gofmt in 1 place. Go fmt your code! Open
Open
/*
- Exclude checks
don't use ALL_CAPS in Go names; use CamelCase Open
Open
FAMILY_V4 = netlink.FAMILY_V4
- Exclude checks