Still Failing: libvirt/libvirt#3208 (master - 5253207)

[Travis CI]

Build Update for libvirt/libvirt
-------------------------------------

Build: #3208
Status: Still Failing

Duration: 47 mins and 51 secs
Commit: 5253207 (master)
Author: Nikolay Shirokovskiy
Message: qemu: remove redundant needReply argument of qemuAgentCommand

needReply added in [1] looks redundant. Indeed it is set to false only
when mon->await_event is set too (the only exception qemuAgentFSTrim
which is mistaken).

However it fixes the issue when qemuAgentCommand exits on error path and
mon->await_event is not reset. Let's instead reset mon->await_event properly.

Also remove "Woken up by event" debug message as it can be misleading.
We can get it also if monitor is closed due to serial changed event
currently. Anyway both qemuAgentClose and qemuAgentNotifyEvent log
itself.

[1] qemu: make sure agent returns error when required data are missing

Signed-off-by: Nikolay Shirokovskiy <nshirokovskiy at virtuozzo.com>
Signed-off-by: Michal Privoznik <mprivozn at redhat.com>
Reviewed-by: Michal Privoznik <mprivozn at redhat.com>

View the changeset: https://github.com/libvirt/libvirt/compare/a30078cb8326...52532073d807

View the full build log and details: https://travis-ci.org/github/libvirt/libvirt/builds/661632533?utm_medium=notification&utm_source=email

--

You can unsubscribe from build emails from the libvirt/libvirt repository going to https://travis-ci.org/account/preferences/unsubscribe?repository=4872032&utm_medium=notification&utm_source=email.
Or unsubscribe from *all* email updating your settings at https://travis-ci.org/account/preferences/unsubscribe?utm_medium=notification&utm_source=email.
Or configure specific recipients for build notifications in your .travis.yml file. See https://docs.travis-ci.com/user/notifications.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://listman.redhat.com/archives/libvirt-ci/attachments/20200312/f702e507/attachment.htm>