Refactor to support "agent manager" pattern #1172
Triggered via pull request
September 23, 2024 19:31
kthomas
Status
Failure
Total duration
7m 16s
Artifacts
–
ltb.yml
on: pull_request
lint
21s
build
7m 6s
Matrix: test
Annotations
14 errors and 1 warning
|
lint:
internal/node/internal-nats/config_template.go#L1
: # github.com/synadia-io/nex/internal/node/internal-nats [github.com/synadia-io/nex/internal/node/internal-nats.test]
|
|
lint:
internal/node/internal-nats/internal_nats_server_test.go#L91
not enough arguments in call to server.StoreFileForID
|
|
lint:
internal/node/agent_manager.go#L325
Error return value of `a.Stop` is not checked (errcheck)
|
|
lint:
internal/node/processmanager/firecracker_procman.go#L296
func `(*FirecrackerProcessManager).resetCNI` is unused (unused)
|
|
lint:
internal/node/agent_manager.go#L322
loopclosure: loop variable id captured by func literal (govet)
|
|
lint
issues found
|
|
test (ubuntu-latest):
internal/node/internal-nats/internal_nats_server_test.go#L91
not enough arguments in call to server.StoreFileForID
|
|
test (ubuntu-latest)
Process completed with exit code 1.
|
|
test (macos-latest)
not enough arguments in call to NewSpawningProcessManager
|
|
test (macos-latest)
Process completed with exit code 1.
|
|
test (windows-latest)
not enough arguments in call to NewSpawningProcessManager
|
|
test (windows-latest)
Process completed with exit code 1.
|
|
build:
internal/node/processmanager/process_mgr_nosandbox.go#L22
not enough arguments in call to NewSpawningProcessManager
|
|
build
The process '/opt/hostedtoolcache/goreleaser-action/1.26.2/x64/goreleaser' failed with exit code 1
|
|
build
You are using 'latest' as default version. Will lock to '~> v1'.
|