Skip to content
Go

PLT-1.1 adding breakout config code #18291

Sign in to view logs

PLT-1.1 adding breakout config code

PLT-1.1 adding breakout config code #18291

Re-run triggered December 20, 2024 22:28
@dploredplore
#3648
b4firex:plt-1.1.v2
Status Success
Total duration 29m 9s
Artifacts –

go.yml

on: pull_request
build
2m 8s
build
test
3m 18s
test
Static Analysis
28m 57s
Static Analysis
Fit to window
Zoom out
Zoom in

Annotations

10 errors and 3 warnings
Static Analysis: internal/otg_helpers/otg_config_helpers/otgconfighelpers.go#L96
parameter 't' seems to be unused, consider removing or renaming it as _
Static Analysis: feature/platform/tests/breakout_configuration/breakout_base_test.go#L1
don't use an underscore in package name
Static Analysis: feature/platform/tests/breakout_configuration/breakout_base_test.go#L224
if block ends with a return statement, so drop this else and outdent its block
Static Analysis: feature/platform/tests/breakout_configuration/breakout_test.go#L31
should omit type bool from declaration of var foundExpectedInterfaceFlag; it will be inferred from the right-hand side
Static Analysis: feature/platform/tests/breakout_configuration/breakout_test.go#L255
this block is empty, you can remove it
Static Analysis: feature/platform/tests/breakout_configuration/breakout_test.go#L1
don't use an underscore in package name
Static Analysis: feature/platform/tests/breakout_configuration/breakout_test.go#L98
var ateIpAddress should be ateIPAddress
Static Analysis: feature/platform/tests/breakout_configuration/breakout_test.go#L104
var gwIp should be gwIP
Static Analysis: feature/platform/tests/breakout_configuration/breakout_test.go#L105
var dutIpAddress should be dutIPAddress
Static Analysis: feature/platform/tests/breakout_configuration/breakout_test.go#L156
struct field dutIntfIp should be dutIntfIP
build
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
test
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636
Static Analysis
ubuntu-latest pipelines will use ubuntu-24.04 soon. For more details, see https://github.com/actions/runner-images/issues/10636