virsh: domsetlaunchsecstate: report error if no options are passed

[libvirt PATCHv2 0/2] virsh: domsetlaunchsecstate: report error if no options are passed

Ján Tomko posted 2 patches 2 years, 3 months ago

Download series mbox

Test syntax-check failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1643309033.git.jtomko@redhat.com

src/qemu/qemu_driver.c | 33 +++++++++++++++++++--------------
tools/virsh-domain.c   |  4 +++-
2 files changed, 22 insertions(+), 15 deletions(-)

Expand all Fold all

[libvirt PATCHv2 0/2] virsh: domsetlaunchsecstate: report error if no options are passed

Posted by Ján Tomko 2 years, 3 months ago

Use a different approach that is hopefully more future-proof and also
add a check to the qemu driver, as suggested by Michal.

Ján Tomko (2):
  virsh: domsetlaunchsecstate: report error if no options are passed
  qemu: qemuDomainSetLaunchSecurityState: check for params presence

 src/qemu/qemu_driver.c | 33 +++++++++++++++++++--------------
 tools/virsh-domain.c   |  4 +++-
 2 files changed, 22 insertions(+), 15 deletions(-)

-- 
2.31.1

Re: [libvirt PATCHv2 0/2] virsh: domsetlaunchsecstate: report error if no options are passed

Posted by Michal Prívozník 2 years, 3 months ago

On 1/27/22 19:44, Ján Tomko wrote:
> Use a different approach that is hopefully more future-proof and also
> add a check to the qemu driver, as suggested by Michal.
> 
> Ján Tomko (2):
>   virsh: domsetlaunchsecstate: report error if no options are passed
>   qemu: qemuDomainSetLaunchSecurityState: check for params presence
> 
>  src/qemu/qemu_driver.c | 33 +++++++++++++++++++--------------
>  tools/virsh-domain.c   |  4 +++-
>  2 files changed, 22 insertions(+), 15 deletions(-)
> 

Reviewed-by: Michal Privoznik <mprivozn@redhat.com>

Michal