Skip to content

*: support TLS/mTLS on the peer and client #1433

*: support TLS/mTLS on the peer and client

*: support TLS/mTLS on the peer and client #1433

Triggered via pull request March 17, 2024 09:44
Status Cancelled
Total duration 2m 53s
Artifacts

pr_build_and_test.yaml

on: pull_request
Fit to window
Zoom out
Zoom in

Annotations

12 errors and 12 warnings
Maelstrom: maelstrom/main.go#L191
cannot use grpcProvider (variable of type *maelstromGrpcProvider) as container.GrpcProvider value in argument to server.NewWithGrpcProvider: *maelstromGrpcProvider does not implement container.GrpcProvider (wrong type for method StartGrpcServer)
Maelstrom
Process completed with exit code 2.
Run chaos-mesh
Canceling since a higher priority waiting request for 'Build-refs/pull/443/merge' exists
Run chaos-mesh
The operation was canceled.
Build & Test
Canceling since a higher priority waiting request for 'Build-refs/pull/443/merge' exists
Build & Test: maelstrom/coordinator_rpc_client.go#L1
: # github.com/streamnative/oxia/maelstrom
Build & Test: maelstrom/main.go#L191
cannot use grpcProvider (variable of type *maelstromGrpcProvider) as container.GrpcProvider value in argument to server.NewWithGrpcProvider: *maelstromGrpcProvider does not implement container.GrpcProvider (wrong type for method StartGrpcServer)
Build & Test: common/security/tls.go#L78
G402: TLS InsecureSkipVerify may be true. (gosec)
Build & Test: common/security/tls.go#L94
ineffectual assignment to bPem (ineffassign)
Build & Test: tests/security/tls/tls_encryption_test.go#L157
ineffectual assignment to err (ineffassign)
Build & Test: tests/security/tls/tls_encryption_test.go#L196
ineffectual assignment to err (ineffassign)
Build & Test
issues found
Maelstrom
Node.js 16 actions are deprecated. Please update the following actions to use Node.js 20: actions/checkout@v3, actions/setup-go@v3, actions/setup-java@v3, actions/upload-artifact@v3. For more information see: https://github.blog/changelog/2023-09-22-github-actions-transitioning-from-node-16-to-node-20/.
Maelstrom
No files were found with the provided path: maelstrom/store. No artifacts will be uploaded.
Build & Test: common/security/tls.go#L25
var-naming: type TlsOption should be TLSOption (revive)
Build & Test: common/security/tls.go#L47
var-naming: var InvalidTlsCertFile should be InvalidTLSCertFile (revive)
Build & Test: common/security/tls.go#L48
var-naming: var InvalidTlsKeyFile should be InvalidTLSKeyFile (revive)
Build & Test: common/security/tls.go#L105
var-naming: method MakeClientTlsConf should be MakeClientTLSConf (revive)
Build & Test: common/security/tls.go#L126
var-naming: method MakeServerTlsConf should be MakeServerTLSConf (revive)
Build & Test: common/container/container.go#L46
import-shadowing: The name 'tls' shadows an import name (revive)
Build & Test: common/container/container.go#L54
import-shadowing: The name 'tls' shadows an import name (revive)
Build & Test: common/container/container.go#L66
import-shadowing: The name 'tls' shadows an import name (revive)
Build & Test: oxia/options.go#L48
var-naming: var ErrInvalidOptionTls should be ErrInvalidOptionTLS (revive)
Build & Test: tests/security/tls/tls_encryption_test.go#L32
var-naming: func getPeerTlsOption should be getPeerTLSOption (revive)