Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

toolbox: reset session when invalidated by the vmx #829

Merged
merged 1 commit into from
Aug 29, 2017

Conversation

dougm
Copy link
Member

@dougm dougm commented Aug 29, 2017

Seen cases where ESX is under heavy load and the toolbox session
cookie is invalidated by the vmx.

This change will backoff the polling loop and also re-open the message
channel to get a new session cookie.

On the guest side, this resulted in a flood of:
Message: Unable to send a message over the communication channel 0
Message: Unable to poll for messages over the communication channel 0

On the vmx site (vmware.log), this resulted in a flood of:
GuestMsg: Channel 0, Protocol error, state: 0
GuestMsg: channel 0: wrong cookie, discarding message.

Seen cases where ESX is under heavy load and the toolbox session
cookie is invalidated by the vmx.

This change will backoff the polling loop and also re-open the message
channel to get a new session cookie.

On the guest side, this resulted in a flood of:
   Message: Unable to send a message over the communication channel 0
   Message: Unable to poll for messages over the communication channel 0

On the vmx site (vmware.log), this resulted in a flood of:
   GuestMsg: Channel 0, Protocol error, state: 0
   GuestMsg: channel 0: wrong cookie, discarding message.
@dougm dougm merged commit 6ce8848 into vmware:master Aug 29, 2017
dougm added a commit to dougm/vic that referenced this pull request Sep 4, 2017
Cherry pick of vmware/govmomi#829
master branch has this change via the full govmomi update in vmware#6144
dougm added a commit to dougm/vic that referenced this pull request Sep 4, 2017
Cherry pick of vmware/govmomi#829

master branch has this change via the full govmomi update in vmware#6144

Fixes vmware#6152
mdubya66 pushed a commit to vmware/vic that referenced this pull request Sep 5, 2017
Cherry pick of vmware/govmomi#829

master branch has this change via the full govmomi update in #6144

Fixes #6152
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants