nagaboinaramgopal opened a new pull request, #14040: URL: https://github.com/apache/cloudstack/pull/14040
### Description When sending the next queued request threw AgentUnavailableException, the request was cancelled but _currentSequence was still set to that dead sequence. No answer ever arrives for a cancelled command, so sendNext was never driven again and every later in-sequence command to the host queued behind it and timed out, until the attache was rebuilt. Advance _currentSequence only on a successful send, and on failure move on to the next queued request. ### Types of changes - [x] Bug fix (non-breaking change which fixes an issue) ### Feature/Enhancement Scale or Bug Severity #### Bug Severity - [x] Minor ### How Has This Been Tested? Added a unit test where the first queued command fails to send and asserts the next queued command is dispatched and becomes the current sequence, instead of stalling on the failed one. Also built the standard packages and deployed on a KVM advanced zone. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
