1. Patchew
  2. Libvirt
  3. View series

[libvirt] [PATCH v2 00/15] Split and enhancement of virsh completer

Michal Privoznik posted 15 patches 4 years, 8 months ago
Test syntax-check passed
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/cover.1565166420.git.mprivozn@redhat.com
tools/Makefile.am                  |   11 +
tools/virsh-completer-checkpoint.c |   78 +++
tools/virsh-completer-checkpoint.h |   27 +
tools/virsh-completer-domain.c     |  314 +++++++++
tools/virsh-completer-domain.h     |   55 ++
tools/virsh-completer-host.c       |  148 ++++
tools/virsh-completer-host.h       |   31 +
tools/virsh-completer-interface.c  |   67 ++
tools/virsh-completer-interface.h  |   27 +
tools/virsh-completer-network.c    |  145 ++++
tools/virsh-completer-network.h    |   35 +
tools/virsh-completer-nodedev.c    |  117 ++++
tools/virsh-completer-nodedev.h    |   35 +
tools/virsh-completer-nwfilter.c   |  105 +++
tools/virsh-completer-nwfilter.h   |   31 +
tools/virsh-completer-pool.c       |  120 ++++
tools/virsh-completer-pool.h       |   35 +
tools/virsh-completer-secret.c     |   91 +++
tools/virsh-completer-secret.h     |   31 +
tools/virsh-completer-snapshot.c   |   73 ++
tools/virsh-completer-snapshot.h   |   27 +
tools/virsh-completer-volume.c     |   73 ++
tools/virsh-completer-volume.h     |   28 +
tools/virsh-completer.c            | 1028 +---------------------------
tools/virsh-completer.h            |  114 +--
tools/virsh-nodedev.c              |   16 +-
tools/virsh-nodedev.h              |    6 +-
tools/virsh-pool.c                 |    1 +
28 files changed, 1730 insertions(+), 1139 deletions(-)
create mode 100644 tools/virsh-completer-checkpoint.c
create mode 100644 tools/virsh-completer-checkpoint.h
create mode 100644 tools/virsh-completer-domain.c
create mode 100644 tools/virsh-completer-domain.h
create mode 100644 tools/virsh-completer-host.c
create mode 100644 tools/virsh-completer-host.h
create mode 100644 tools/virsh-completer-interface.c
create mode 100644 tools/virsh-completer-interface.h
create mode 100644 tools/virsh-completer-network.c
create mode 100644 tools/virsh-completer-network.h
create mode 100644 tools/virsh-completer-nodedev.c
create mode 100644 tools/virsh-completer-nodedev.h
create mode 100644 tools/virsh-completer-nwfilter.c
create mode 100644 tools/virsh-completer-nwfilter.h
create mode 100644 tools/virsh-completer-pool.c
create mode 100644 tools/virsh-completer-pool.h
create mode 100644 tools/virsh-completer-secret.c
create mode 100644 tools/virsh-completer-secret.h
create mode 100644 tools/virsh-completer-snapshot.c
create mode 100644 tools/virsh-completer-snapshot.h
create mode 100644 tools/virsh-completer-volume.c
create mode 100644 tools/virsh-completer-volume.h
[libvirt] [PATCH v2 00/15] Split and enhancement of virsh completer
Posted by Michal Privoznik 4 years, 8 months ago 
v2 of:

https://www.redhat.com/archives/libvir-list/2019-July/msg01225.html

diff to v1:
- Rebase and adapt to new checkpoint completer

Michal Prívozník (15):
  tools: s/Nodedev/NodeDevice/
  tools: Expose virshCommaStringListComplete()
  tools: Separate domain related completers into a file
  tools: Separate storage pool related completers into a file
  tools: Separate storage volume related completers into a file
  tools: Separate interface related completers into a file
  tools: Separate network related completers into a file
  tools: Separate nodedev related completers into a file
  tools: Separate nwfilter related completers into a file
  tools: Separate secret related completers into a file
  tools: Separate snapshot related completers into a file
  tools: Separate host related completers into a file
  tools: Separate checkpoint related completers into a file
  virsh-completer: Drop needless #include
  virsh: Introduce virshPoolTypeCompleter

 tools/Makefile.am                  |   11 +
 tools/virsh-completer-checkpoint.c |   78 +++
 tools/virsh-completer-checkpoint.h |   27 +
 tools/virsh-completer-domain.c     |  314 +++++++++
 tools/virsh-completer-domain.h     |   55 ++
 tools/virsh-completer-host.c       |  148 ++++
 tools/virsh-completer-host.h       |   31 +
 tools/virsh-completer-interface.c  |   67 ++
 tools/virsh-completer-interface.h  |   27 +
 tools/virsh-completer-network.c    |  145 ++++
 tools/virsh-completer-network.h    |   35 +
 tools/virsh-completer-nodedev.c    |  117 ++++
 tools/virsh-completer-nodedev.h    |   35 +
 tools/virsh-completer-nwfilter.c   |  105 +++
 tools/virsh-completer-nwfilter.h   |   31 +
 tools/virsh-completer-pool.c       |  120 ++++
 tools/virsh-completer-pool.h       |   35 +
 tools/virsh-completer-secret.c     |   91 +++
 tools/virsh-completer-secret.h     |   31 +
 tools/virsh-completer-snapshot.c   |   73 ++
 tools/virsh-completer-snapshot.h   |   27 +
 tools/virsh-completer-volume.c     |   73 ++
 tools/virsh-completer-volume.h     |   28 +
 tools/virsh-completer.c            | 1028 +---------------------------
 tools/virsh-completer.h            |  114 +--
 tools/virsh-nodedev.c              |   16 +-
 tools/virsh-nodedev.h              |    6 +-
 tools/virsh-pool.c                 |    1 +
 28 files changed, 1730 insertions(+), 1139 deletions(-)
 create mode 100644 tools/virsh-completer-checkpoint.c
 create mode 100644 tools/virsh-completer-checkpoint.h
 create mode 100644 tools/virsh-completer-domain.c
 create mode 100644 tools/virsh-completer-domain.h
 create mode 100644 tools/virsh-completer-host.c
 create mode 100644 tools/virsh-completer-host.h
 create mode 100644 tools/virsh-completer-interface.c
 create mode 100644 tools/virsh-completer-interface.h
 create mode 100644 tools/virsh-completer-network.c
 create mode 100644 tools/virsh-completer-network.h
 create mode 100644 tools/virsh-completer-nodedev.c
 create mode 100644 tools/virsh-completer-nodedev.h
 create mode 100644 tools/virsh-completer-nwfilter.c
 create mode 100644 tools/virsh-completer-nwfilter.h
 create mode 100644 tools/virsh-completer-pool.c
 create mode 100644 tools/virsh-completer-pool.h
 create mode 100644 tools/virsh-completer-secret.c
 create mode 100644 tools/virsh-completer-secret.h
 create mode 100644 tools/virsh-completer-snapshot.c
 create mode 100644 tools/virsh-completer-snapshot.h
 create mode 100644 tools/virsh-completer-volume.c
 create mode 100644 tools/virsh-completer-volume.h

-- 
2.21.0

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH v2 00/15] Split and enhancement of virsh completer
Posted by Jonathon Jongsma 4 years, 8 months ago 
On Wed, 2019-08-07 at 10:30 +0200, Michal Privoznik wrote:
> v2 of:
> 
> https://www.redhat.com/archives/libvir-list/2019-July/msg01225.html
> 
> diff to v1:
> - Rebase and adapt to new checkpoint completer
> 
> Michal Prívozník (15):
>   tools: s/Nodedev/NodeDevice/
>   tools: Expose virshCommaStringListComplete()
>   tools: Separate domain related completers into a file
>   tools: Separate storage pool related completers into a file
>   tools: Separate storage volume related completers into a file
>   tools: Separate interface related completers into a file
>   tools: Separate network related completers into a file
>   tools: Separate nodedev related completers into a file
>   tools: Separate nwfilter related completers into a file
>   tools: Separate secret related completers into a file
>   tools: Separate snapshot related completers into a file
>   tools: Separate host related completers into a file
>   tools: Separate checkpoint related completers into a file
>   virsh-completer: Drop needless #include
>   virsh: Introduce virshPoolTypeCompleter
> 
>  tools/Makefile.am                  |   11 +
>  tools/virsh-completer-checkpoint.c |   78 +++
>  tools/virsh-completer-checkpoint.h |   27 +
>  tools/virsh-completer-domain.c     |  314 +++++++++
>  tools/virsh-completer-domain.h     |   55 ++
>  tools/virsh-completer-host.c       |  148 ++++
>  tools/virsh-completer-host.h       |   31 +
>  tools/virsh-completer-interface.c  |   67 ++
>  tools/virsh-completer-interface.h  |   27 +
>  tools/virsh-completer-network.c    |  145 ++++
>  tools/virsh-completer-network.h    |   35 +
>  tools/virsh-completer-nodedev.c    |  117 ++++
>  tools/virsh-completer-nodedev.h    |   35 +
>  tools/virsh-completer-nwfilter.c   |  105 +++
>  tools/virsh-completer-nwfilter.h   |   31 +
>  tools/virsh-completer-pool.c       |  120 ++++
>  tools/virsh-completer-pool.h       |   35 +
>  tools/virsh-completer-secret.c     |   91 +++
>  tools/virsh-completer-secret.h     |   31 +
>  tools/virsh-completer-snapshot.c   |   73 ++
>  tools/virsh-completer-snapshot.h   |   27 +
>  tools/virsh-completer-volume.c     |   73 ++
>  tools/virsh-completer-volume.h     |   28 +
>  tools/virsh-completer.c            | 1028 +-------------------------
> --
>  tools/virsh-completer.h            |  114 +--
>  tools/virsh-nodedev.c              |   16 +-
>  tools/virsh-nodedev.h              |    6 +-
>  tools/virsh-pool.c                 |    1 +
>  28 files changed, 1730 insertions(+), 1139 deletions(-)
>  create mode 100644 tools/virsh-completer-checkpoint.c
>  create mode 100644 tools/virsh-completer-checkpoint.h
>  create mode 100644 tools/virsh-completer-domain.c
>  create mode 100644 tools/virsh-completer-domain.h
>  create mode 100644 tools/virsh-completer-host.c
>  create mode 100644 tools/virsh-completer-host.h
>  create mode 100644 tools/virsh-completer-interface.c
>  create mode 100644 tools/virsh-completer-interface.h
>  create mode 100644 tools/virsh-completer-network.c
>  create mode 100644 tools/virsh-completer-network.h
>  create mode 100644 tools/virsh-completer-nodedev.c
>  create mode 100644 tools/virsh-completer-nodedev.h
>  create mode 100644 tools/virsh-completer-nwfilter.c
>  create mode 100644 tools/virsh-completer-nwfilter.h
>  create mode 100644 tools/virsh-completer-pool.c
>  create mode 100644 tools/virsh-completer-pool.h
>  create mode 100644 tools/virsh-completer-secret.c
>  create mode 100644 tools/virsh-completer-secret.h
>  create mode 100644 tools/virsh-completer-snapshot.c
>  create mode 100644 tools/virsh-completer-snapshot.h
>  create mode 100644 tools/virsh-completer-volume.c
>  create mode 100644 tools/virsh-completer-volume.h
> 


Series looks good to me, though you may want to wait for somebody with
more experience to properly ack it.

Reviewed-by: Jonathon Jongsma <jjongsma@redhat.com>

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list
Re: [libvirt] [PATCH v2 00/15] Split and enhancement of virsh completer
Posted by Michal Privoznik 4 years, 8 months ago 
On 8/8/19 6:54 PM, Jonathon Jongsma wrote:
>
> Series looks good to me, though you may want to wait for somebody with
> more experience to properly ack it.
> 
> Reviewed-by: Jonathon Jongsma <jjongsma@redhat.com>
> 

That's okay. I've taken your R-b and pushed it. These are simple enough 
so if there was any problem, it would be obvious.

Thanks!
Michal

--
libvir-list mailing list
libvir-list@redhat.com
https://www.redhat.com/mailman/listinfo/libvir-list

Patchew 2.1-devel-0870f84

© 2016 - 2024 Red Hat, Inc.