Skip to content

NETOBSERV-1377: Add controller to deploy netobserv network policy #1423

NETOBSERV-1377: Add controller to deploy netobserv network policy

NETOBSERV-1377: Add controller to deploy netobserv network policy #1423

Triggered via pull request June 25, 2024 14:51
Status Failure
Total duration 2m 3s
Artifacts

pull_request.yml

on: pull_request
Checking bundle up-to-date
1m 47s
Checking bundle up-to-date
Matrix: Build, lint, test
Fit to window
Zoom out
Zoom in

Annotations

7 errors and 2 warnings
Checking bundle up-to-date
Process completed with exit code 1.
Build, lint, test (1.22): controllers/networkpolicy/np_controller_test.go#L107
if-return: redundant if ...; err != nil check, just return error instead. (revive)
Build, lint, test (1.22): controllers/networkpolicy/np_controller.go#L90
if-return: redundant if ...; err != nil check, just return error instead. (revive)
Build, lint, test (1.22): controllers/networkpolicy/np_controller_test.go#L32
var `updateCR` is unused (unused)
Build, lint, test (1.22): controllers/networkpolicy/np_controller_test.go#L46
const `otherNamespace` is unused (unused)
Build, lint, test (1.22): controllers/networkpolicy/np_objects.go#L13
S1002: should omit comparison to bool constant, can be simplified to `!*desired.Spec.NetworkPolicy.Deploy` (gosimple)
Build, lint, test (1.22)
Process completed with exit code 2.
Checking bundle up-to-date
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Build, lint, test (1.22)
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/setup-go@v3, actions/checkout@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.