Skip to content

Commit 7d018f4

Browse files
askyrieSongZhen0704
authored andcommitted
fix: remove illegal fields from sync reponse
1 parent 5a32d38 commit 7d018f4

1 file changed

Lines changed: 15 additions & 6 deletions

File tree

  • server/controller/trisolaris/services/grpc/agentsynchronize

server/controller/trisolaris/services/grpc/agentsynchronize/sync_push.go

Lines changed: 15 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -409,16 +409,25 @@ func (e *AgentEvent) Sync(ctx context.Context, in *api.SyncRequest) (*api.SyncRe
409409
syncResponse.NewGrpcBufferSize = &grpcBufferSize
410410
return &syncResponse, nil
411411
}
412+
// When the controller determines that the agent needs to adjust `current_grpc_buffer_size`,
413+
// this value is set to true. At this time, only the following fields in the message are valid:
414+
// - status
415+
// - user_config
416+
// - revision
417+
// - self_update_url
418+
// - containers
419+
// - dynamic_config
420+
// - only_partial_fields
421+
// - new_grpc_buffer_size
412422
return &api.SyncResponse{
413423
Status: &STATUS_SUCCESS,
414-
OnlyPartialFields: proto.Bool(true),
415-
NewGrpcBufferSize: &grpcBufferSize,
416424
UserConfig: proto.String(userConfigData),
417-
DynamicConfig: dynamicConfig,
418-
Containers: containers,
419-
SelfUpdateUrl: proto.String(selfUpdateURL),
420425
Revision: proto.String(upgradeRevision),
421-
CustomAppConfig: &customAppConfig,
426+
SelfUpdateUrl: proto.String(selfUpdateURL),
427+
Containers: containers,
428+
DynamicConfig: dynamicConfig,
429+
OnlyPartialFields: proto.Bool(true),
430+
NewGrpcBufferSize: &grpcBufferSize,
422431
}, nil
423432
}
424433

0 commit comments

Comments
 (0)