Return existing reservation if podRef matched - rebase #837
Triggered via pull request
September 11, 2023 20:00
Status
Failure
Total duration
39m 10s
Artifacts
–
Annotations
2 errors and 12 warnings
test (1.19.x, ubuntu-latest)
Process completed with exit code 1.
|
e2e test
Process completed with exit code 1.
|
test (1.19.x, ubuntu-latest)
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-go@v2, actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|
test (1.19.x, ubuntu-latest)
Unexpected input(s) 'name', valid inputs are ['entryPoint', 'args', 'config', 'exclude', 'path']
|
test (1.19.x, ubuntu-latest):
pkg/api/whereabouts.cni.cncf.io/register.go#L14
should have a package comment
|
test (1.19.x, ubuntu-latest):
pkg/storage/kubernetes/client.go#L1
should have a package comment
|
test (1.19.x, ubuntu-latest):
pkg/storage/kubernetes/ipam.go#L71
type name will be used as kubernetes.KubernetesIPAM by other packages, and that stutters; consider calling this IPAM
|
test (1.19.x, ubuntu-latest):
pkg/storage/kubernetes/ipam.go#L78
exported const UnnamedNetwork should have comment or be unexported
|
test (1.19.x, ubuntu-latest):
pkg/reconciler/ip.go#L1
should have a package comment
|
test (1.19.x, ubuntu-latest):
pkg/reconciler/iploop.go#L118
should replace retries += 1 with retries++
|
test (1.19.x, ubuntu-latest):
pkg/reconciler/wrappedPod.go#L93
func isIpOnPod should be isIPOnPod
|
test (1.19.x, ubuntu-latest):
pkg/reconciler/ip.go#L14
exported function ReconcileIPs should have comment or be unexported
|
test (1.19.x, ubuntu-latest):
pkg/storage/kubernetes/ipam.go#L80
exported type PoolIdentifier should have comment or be unexported
|
e2e test
The following actions uses node12 which is deprecated and will be forced to run on node16: actions/setup-go@v1, actions/checkout@v2. For more info: https://github.blog/changelog/2023-06-13-github-actions-all-actions-will-run-on-node16-instead-of-node12-by-default/
|