Skip to content

Commit

Permalink
Refactor CVMS client and agent main to improve function signatures an…
Browse files Browse the repository at this point in the history
…d variable naming for clarity

Signed-off-by: Sammy Oina <[email protected]>
  • Loading branch information
SammyOina committed Feb 14, 2025
1 parent b6f1ffc commit 1c2e8fe
Show file tree
Hide file tree
Showing 2 changed files with 8 additions and 8 deletions.
4 changes: 2 additions & 2 deletions agent/cvms/api/grpc/client.go
Original file line number Diff line number Diff line change
Expand Up @@ -208,14 +208,14 @@ func (client *CVMSClient) processIncomingMessage(ctx context.Context, req *cvms.
case *cvms.ServerStreamMessage_StopComputation:
go client.handleStopComputation(ctx, mes)
case *cvms.ServerStreamMessage_AgentStateReq:
client.handleAgentStateReq(ctx, mes)
client.handleAgentStateReq(mes)
default:
return errUnknonwMessageType
}
return nil
}

func (client *CVMSClient) handleAgentStateReq(ctx context.Context, mes *cvms.ServerStreamMessage_AgentStateReq) {
func (client *CVMSClient) handleAgentStateReq(mes *cvms.ServerStreamMessage_AgentStateReq) {
state := client.svc.State()

msg := &cvms.ClientStreamMessage_AgentStateRes{
Expand Down
12 changes: 6 additions & 6 deletions cmd/agent/main.go
Original file line number Diff line number Diff line change
Expand Up @@ -19,13 +19,13 @@ import (
"github.com/ultravioletrs/cocos/agent"
"github.com/ultravioletrs/cocos/agent/api"
"github.com/ultravioletrs/cocos/agent/cvms"
cvmapi "github.com/ultravioletrs/cocos/agent/cvms/api/grpc"
cvmsapi "github.com/ultravioletrs/cocos/agent/cvms/api/grpc"
"github.com/ultravioletrs/cocos/agent/cvms/server"
"github.com/ultravioletrs/cocos/agent/events"
agentlogger "github.com/ultravioletrs/cocos/internal/logger"
"github.com/ultravioletrs/cocos/pkg/attestation/quoteprovider"
pkggrpc "github.com/ultravioletrs/cocos/pkg/clients/grpc"
cvmgrpc "github.com/ultravioletrs/cocos/pkg/clients/grpc/cvm"
cvmsgrpc "github.com/ultravioletrs/cocos/pkg/clients/grpc/cvm"
"golang.org/x/sync/errgroup"
)

Expand Down Expand Up @@ -86,7 +86,7 @@ func main() {
return
}

cvmGRPCClient, cvmClient, err := cvmgrpc.NewCVMClient(cvmGrpcConfig)
cvmGRPCClient, cvmsClient, err := cvmsgrpc.NewCVMClient(cvmGrpcConfig)
if err != nil {
logger.Error(err.Error())
exitCode = 1
Expand All @@ -95,7 +95,7 @@ func main() {
defer cvmGRPCClient.Close()

reconnectFn := func(ctx context.Context) (cvms.Service_ProcessClient, error) {
_, newClient, err := cvmgrpc.NewCVMClient(cvmGrpcConfig)
_, newClient, err := cvmsgrpc.NewCVMClient(cvmGrpcConfig)
if err != nil {
return nil, err
}
Expand All @@ -104,7 +104,7 @@ func main() {
return newClient.Process(ctx)
}

pc, err := cvmClient.Process(ctx)
pc, err := cvmsClient.Process(ctx)
if err != nil {
logger.Error(err.Error())
exitCode = 1
Expand All @@ -119,7 +119,7 @@ func main() {
return
}

mc := cvmapi.NewClient(pc, svc, eventsLogsQueue, logger, server.NewServer(logger, svc), storageDir, reconnectFn)
mc := cvmsapi.NewClient(pc, svc, eventsLogsQueue, logger, server.NewServer(logger, svc), storageDir, reconnectFn)

g.Go(func() error {
ch := make(chan os.Signal, 1)
Expand Down

0 comments on commit 1c2e8fe

Please sign in to comment.