openconfig / gribigo

Go implementation of gRIBI.
Apache License 2.0
16 stars 24 forks source link

Gribi client throw `cannot remove pending operation 1, could not dequeue operation 1, unknown operation` #130

Closed yushiyushi closed 2 years ago

yushiyushi commented 2 years ago

if FIB_BACK feature toggle is turned on, the client will throw cannot remove pending operation 1, could not dequeue operation 1, unknown operation error

reason: the server side will send both FIB_BACK and RIB_BACK response to client, at client side, each operation has one unique operation ID, the pending operation will be removed from pending operation queue by checking response's operation ID, normally, the server will return RIB_BACK, and then FIB_BACK, after response with RIB_BACK is received, the operation is removed from pending operation queue, when it comes to response with FIB_BACK, it can not find the operation, so the client throws this error

this is also the reason why client does not return responses with FIB_BACK.

xw-g commented 2 years ago

131 for reference.

xw-g commented 2 years ago

Thanks for fixing the issue @yushiyushi !