Skip to content
Build check

feat: node ipam controller #192

Sign in to view logs

feat: node ipam controller

feat: node ipam controller #192

Triggered via pull request June 28, 2024 12:51
@sergelogvinovsergelogvinov
opened #176
sergelogvinov:nodeipam
Status Failure
Total duration 2m 37s
Artifacts –

build-test.yaml

on: pull_request
Build
2m 27s
Build
Fit to window
Zoom out
Zoom in

Annotations

10 errors
Build: cmd/talos-cloud-controller-manager/nodeipamcontroller.go#L44
Comment should end in a period (godot)
Build: cmd/talos-cloud-controller-manager/nodeipamcontroller.go#L46
Comment should end in a period (godot)
Build: cmd/talos-cloud-controller-manager/nodeipamcontroller.go#L62
Error return value of `nodeIpamController.nodeIPAMControllerOptions.ApplyTo` is not checked (errcheck)
Build: cmd/talos-cloud-controller-manager/nodeipamcontroller.go#L69
Comment should end in a period (godot)
Build: cmd/talos-cloud-controller-manager/nodeipamcontroller.go#L143
return with no blank line before (nlreturn)
Build: cmd/talos-cloud-controller-manager/nodeipamcontroller.go#L149
Comment should end in a period (godot)
Build: cmd/talos-cloud-controller-manager/nodeipamcontroller.go#L160
Error return value of `netutils.IsDualStackCIDRs` is not checked (errcheck)
Build: cmd/talos-cloud-controller-manager/nodeipamcontroller.go#L168
unnecessary leading newline (whitespace)
Build: cmd/talos-cloud-controller-manager/nodeipamcontroller.go#L180
return with no blank line before (nlreturn)
Build: cmd/talos-cloud-controller-manager/nodeipamcontroller.go#L197
if statements should only be cuddled with assignments (wsl)