Skip to content

Conversation

@djs55
Copy link
Collaborator

@djs55 djs55 commented Oct 28, 2013

Previously we used 'hotplug-status' for Vbds (good) but out own
custom path in '/xapi' for Vifs (bad). This makes xenopsd work with
both hotplug script versions.

Signed-off-by: David Scott [email protected]

Previously we used 'hotplug-status' for Vbds (good) but out own
custom path in '/xapi' for Vifs (bad). This makes xenopsd work with
both hotplug script versions.

Signed-off-by: David Scott <[email protected]>
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

compatability?

@robhoes
Copy link
Member

robhoes commented Oct 28, 2013

I guess this would be a good improvement for xl as well?

@djs55
Copy link
Collaborator Author

djs55 commented Oct 28, 2013

My spelling is getting worse.

I think this makes sense for xl too. I'm trying to remember why we used
"hotplug-status" for one type of device and not for the other. I'm a bit
suspicious.

On Mon, Oct 28, 2013 at 5:52 PM, Rob Hoes [email protected] wrote:

I guess this would be a good improvement for xl as well?


Reply to this email directly or view it on GitHubhttps://github.com//pull/40#issuecomment-27237130
.

Dave Scott

@robhoes
Copy link
Member

robhoes commented Jan 20, 2014

I think I just merged this as part of @euanh's pull request...

@jonludlam
Copy link
Collaborator

@djs55 @robhoes what's the status of this? Has it been merged via euanh's PR?

@robhoes
Copy link
Member

robhoes commented Jun 19, 2014

This is already in master.

@robhoes robhoes closed this Jun 19, 2014
johnelse pushed a commit to xenserver/xenopsd that referenced this pull request Apr 7, 2016
Fix up one of the previously imported hunks
psafont pushed a commit to psafont/xenopsd that referenced this pull request Jul 5, 2021
CA-195440: Mark VM stuck only once balloon-driver has published featur…
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants