On Wed, Jan 29, 2025 at 05:40:27PM +0400, marcandre.lureau@xxxxxxxxxx wrote:
> From: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>
>
> Without an error message, it can be tedious to figure out failure to start.
>
> Signed-off-by: Marc-André Lureau <marcandre.lureau@xxxxxxxxxx>
> ---
> src/qemu/qemu_command.c | 3 +++
> 1 file changed, 3 insertions(+)
>
> diff --git a/src/qemu/qemu_command.c b/src/qemu/qemu_command.c
> index 1f20189e89..f08dba7988 100644
> --- a/src/qemu/qemu_command.c
> +++ b/src/qemu/qemu_command.c
> @@ -8421,6 +8421,9 @@ qemuBuildGraphicsCommandLine(virQEMUDriverConfig *cfg,
> break;
> case VIR_DOMAIN_GRAPHICS_TYPE_RDP:
> case VIR_DOMAIN_GRAPHICS_TYPE_DESKTOP:
> + virReportError(VIR_ERR_CONFIG_UNSUPPORTED,
> + _("graphics type '%1$s' not supported"),
> + virDomainGraphicsTypeToString(graphics->type));
> return -1;
We don't report such errors when building the command line as they should
have already been caught & reported by qemuValidateDomainDef, in this
case we have qemuValidateDomainDeviceDefGraphics which reports any
unsupported graphics types, based on the capabilities we probe. If that
check is wrong/incomplete, we should fix the validate method
> case VIR_DOMAIN_GRAPHICS_TYPE_LAST:
> default:
With regards,
Daniel
--
|: https://berrange.com -o- https://www.flickr.com/photos/dberrange :|
|: https://libvirt.org -o- https://fstop138.berrange.com :|
|: https://entangle-photo.org -o- https://www.instagram.com/dberrange :|
