[PATCH v2 0/3] Allow use of JSON in domain configuration files

Elliott Mitchell posted 3 patches 1 year, 12 months ago
Patches applied successfully (tree, apply log)
git fetch https://gitlab.com/xen-project/patchew/xen tags/patchew/cover.1651285313.git.ehem+xen@m5p.com
Test gitlab-ci failed
tools/xl/xl.h           |   6 ++-
tools/xl/xl_cmdtable.c  |  14 ++---
tools/xl/xl_vmcontrol.c | 114 +++++++++++++++++++++-------------------
3 files changed, 73 insertions(+), 61 deletions(-)
[PATCH v2 0/3] Allow use of JSON in domain configuration files
Posted by Elliott Mitchell 1 year, 12 months ago
While the traditional domain configuration file format works acceptably,
I can see uses for having full JSON support.  As such add "-j" and "-J"
to `xl create` to specify format.  The traditional format is the current
default.

While attempting this, it came up that options for `xl create` aren't in
a consistent order.  I'm concerned about moving the VNC options apart,
but the others have been sorted.

Due to one issue in the previous round I ended up examing the dom_info
variable in main_create().  The situation there is a bit tangled.  There
were shadow variables for everything in dom_info.  Unfortunately the
short-hand serves to confuse, so I believe the appropriate action is to
remove the shadows.  Appears .dry_run had effectively been deprecated,
but not fully removed; as such now fully remove it.

Rename everything "format" to "config_format".

Elliott Mitchell (3):
  tools/xl: Sort create command options
  tools/xl: Use sparse init for dom_info, remove duplicate vars
  tools/xl: Allow specifying JSON for domain configuration file format

 tools/xl/xl.h           |   6 ++-
 tools/xl/xl_cmdtable.c  |  14 ++---
 tools/xl/xl_vmcontrol.c | 114 +++++++++++++++++++++-------------------
 3 files changed, 73 insertions(+), 61 deletions(-)

-- 
2.30.2