Skip to content

Commit

Permalink
CA-219763: If the PV drivers have been detected, raise an update
Browse files Browse the repository at this point in the history
Since this is an update to a VIF or VBD, the usual logic only causes
an update for the relevant device. Thus xapi didn't know to look at
the VM again, and hence might not update the pv_drivers_detected
field. This commit adds an explicit update to the VM if the
pv_drivers_detected field changes from false to true.

Signed-off-by: Jon Ludlam <[email protected]>
  • Loading branch information
jonludlam committed Aug 30, 2016
1 parent 02a8cd2 commit 34e6eed
Showing 1 changed file with 2 additions and 1 deletion.
3 changes: 2 additions & 1 deletion xc/xenops_server_xen.ml
Original file line number Diff line number Diff line change
Expand Up @@ -2648,7 +2648,8 @@ module Actions = struct
if value = "4" (* connected *) then begin
let non_persistent = { non_persistent with VmExtra.pv_drivers_detected = true } in
debug "VM = %s; found PV driver evidence on %s (value = %s)" vm path value;
DB.write vm { VmExtra.persistent; non_persistent }
DB.write vm { VmExtra.persistent; non_persistent };
Updates.add (Dynamic.Vm vm) internal_updates
end
with Xs_protocol.Enoent _ ->
warn "Watch event on %s fired but couldn't read from it" path;
Expand Down

0 comments on commit 34e6eed

Please sign in to comment.