[virt-tools-list] [virt-manager][PATCH 2/3] Remove default channel for s390x.

2015-11-03 Thread Kevin Zhao
The Qemu guest agent change for 390x,remove default channel device for s390x and regenerate the test output.Modified the clitest xml. --- tests/cli-test-xml/compare/virt-install-s390x-KVMIBM.xml | 4 virtinst/guest.py| 2 ++ 2 files changed, 2 insertion

[virt-tools-list] [virt-manager][PATCH 3/3] Modify the machine type for s390x.All machine types begin with "s390x-ccw" will by set to the default machine type "s390-ccw-virtio".

2015-11-03 Thread Kevin Zhao
--- virtinst/capabilities.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/virtinst/capabilities.py b/virtinst/capabilities.py index 4fa1724..10e7ea0 100644 --- a/virtinst/capabilities.py +++ b/virtinst/capabilities.py @@ -364,7 +364,7 @@ class _CapsInfo(object):

[virt-tools-list] [virt-manager][PATCH 1/3] Add a capatilities-xml for s390x, and add a clitest for it.

2015-11-03 Thread Kevin Zhao
Add a capatilities xml file for s390x ,the capatility for IBM Distro called KVMIBM,mainly focus on a KVM hypervious on S390x. Also add a clitest by using virt-install. --- tests/capabilities-xml/kvm-s390x-KVMIBM.xml| 82 ++ .../compare/virt-install-s390x-KVMIBM.xml

[virt-tools-list] [PATCH virt-viewer v2] Add some multimonitor documentation

2015-11-03 Thread Jonathon Jongsma
These two files describe some of the behavior and requirements for virt-viewer in fullscreen multimonitor mode --- Changes since v1: - settings: G: specify that 'sparse' guest display configurations are invalid - requirements: D4: client doesn't restrict initial size, but WM might docs/multim

Re: [virt-tools-list] [PATCH virt-viewer] Add some multimonitor documentation

2015-11-03 Thread Jonathon Jongsma
Ah, thanks for the reminder. Looks like Pavel suggested some changes and I never got around to making them. Will submit a new version. On Mon, 2015-10-26 at 13:18 +0100, Fabiano Fidêncio wrote: > Can we revisit this one? > > On Thu, Jun 4, 2015 at 5:26 PM, Jonathon Jongsma > wrote: > > These tw

Re: [virt-tools-list] [PATCH virt-viewer v7 3/3] session-spice: Disable extra displays in fullscreen mode

2015-11-03 Thread Jonathon Jongsma
Hi Pavel, Sorry for the delay. This series looks good to me now. ACK. On Tue, 2015-10-20 at 15:13 +0200, Pavel Grunt wrote: > When running in fullscreen it is possible to end up in a situation > where we have more displays enabled than monitors. This can happen > if displays that were enabled i

Re: [virt-tools-list] [PATCH 0/3] qemu/kvm: dmidecode improvements

2015-11-03 Thread Andrew Jones
On Tue, Nov 03, 2015 at 09:59:08AM +, Richard W.M. Jones wrote: > Thanks for these patches. I have pushed them upstream. > > Do we have a BZ for this? We'll need one if we are going to add these > changes for RHEL 7.3. Yup https://bugzilla.redhat.com/show_bug.cgi?id=1275349 Thanks, drew >

Re: [virt-tools-list] [virt-manager PATCH] virt-install: always enable pae for xen hvm 64bit guest

2015-11-03 Thread Pavel Hrdina
On Tue, Nov 03, 2015 at 10:04:36AM -0500, Cole Robinson wrote: > On 11/03/2015 07:15 AM, Pavel Hrdina wrote: > > According to xen documentation 64bit guest has to have pae enabled in > > order to be able to run 64bit OS. > > > > Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1267160 > > >

Re: [virt-tools-list] [virt-manager PATCH] virt-install: always enable pae for xen hvm 64bit guest

2015-11-03 Thread Cole Robinson
On 11/03/2015 07:15 AM, Pavel Hrdina wrote: > According to xen documentation 64bit guest has to have pae enabled in > order to be able to run 64bit OS. > > Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1267160 > > Signed-off-by: Pavel Hrdina > --- > tests/cli-test-xml/compare/virt-insta

Re: [virt-tools-list] [virt-manager PATCH 0/2] some follow-up patches for virt-clone

2015-11-03 Thread Cole Robinson
On 11/03/2015 05:57 AM, Pavel Hrdina wrote: > Pavel Hrdina (2): > virt-clone: fix pylint error > tests: introduce a test for removing unix channel path for virt-clone > > tests/cli-test-xml/compare/virt-clone-clone-auto1.xml | 5 + > tests/testdriver.xml |

Re: [virt-tools-list] [PATCH 1/5] Call intltoolize after autoreconf

2015-11-03 Thread Fabiano Fidêncio
On Tue, Nov 3, 2015 at 12:14 PM, Eduardo Lima (Etrunko) wrote: > On 02/11/15 04:08, Fabiano Fidêncio wrote: >> Hey! >> >> On Thu, Oct 29, 2015 at 9:40 PM, Eduardo Lima (Etrunko) >> wrote: >>> After a calling 'git clean -d -x -f' I got this error: >>> >>> ln: failed to create symbolic link ‘m4/int

Re: [virt-tools-list] [PATCH 3/5] Update gitignore

2015-11-03 Thread Christophe Fergeau
Hey, On Tue, Nov 03, 2015 at 09:38:24AM -0200, Eduardo Lima (Etrunko) wrote: > On 02/11/15 08:36, Christophe Fergeau wrote: > > On Thu, Oct 29, 2015 at 06:40:38PM -0200, Eduardo Lima (Etrunko) wrote: > >> Signed-off-by: Eduardo Lima (Etrunko) > >> --- > >> Makefile.am | 5 + > >> git.mk

[virt-tools-list] [virt-manager PATCH] virt-install: always enable pae for xen hvm 64bit guest

2015-11-03 Thread Pavel Hrdina
According to xen documentation 64bit guest has to have pae enabled in order to be able to run 64bit OS. Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1267160 Signed-off-by: Pavel Hrdina --- tests/cli-test-xml/compare/virt-install-xen-hvm.xml | 1 + virtinst/guest.py

[virt-tools-list] [virt-manager PATCH 2/2] tests: introduce a test for removing unix channel path for virt-clone

2015-11-03 Thread Pavel Hrdina
Commit 3a33f341 fixed an issue, where we should remove source path for unix channel in order to let libvirt generate new one. Add a test case to cover this change. Signed-off-by: Pavel Hrdina --- tests/cli-test-xml/compare/virt-clone-clone-auto1.xml | 5 + tests/testdriver.xml

Re: [virt-tools-list] Some cosmetic fixes

2015-11-03 Thread Eduardo Lima (Etrunko)
On 02/11/15 04:26, Fabiano Fidêncio wrote: > Hey, > > On Thu, Oct 29, 2015 at 9:40 PM, Eduardo Lima (Etrunko) > wrote: >> While working on adding support to GtkApplication for virt-viewer and >> remote-viewer, I fixed some small things. I figured it was time to >> flush them to avoid those to pil

Re: [virt-tools-list] [PATCH 3/5] Update gitignore

2015-11-03 Thread Eduardo Lima (Etrunko)
On 02/11/15 08:36, Christophe Fergeau wrote: > On Thu, Oct 29, 2015 at 06:40:38PM -0200, Eduardo Lima (Etrunko) wrote: >> Signed-off-by: Eduardo Lima (Etrunko) >> --- >> Makefile.am | 5 + >> git.mk | 2 +- >> m4/.gitignore | 0 >> src/Makefile.am | 8 >> 4 files chang

Re: [virt-tools-list] [PATCH 1/5] Call intltoolize after autoreconf

2015-11-03 Thread Eduardo Lima (Etrunko)
On 02/11/15 04:08, Fabiano Fidêncio wrote: > Hey! > > On Thu, Oct 29, 2015 at 9:40 PM, Eduardo Lima (Etrunko) > wrote: >> After a calling 'git clean -d -x -f' I got this error: >> >> ln: failed to create symbolic link ‘m4/intltool.m4’: No such file or >> directory >> cp: cannot create regular fi

[virt-tools-list] [virt-manager PATCH 0/2] some follow-up patches for virt-clone

2015-11-03 Thread Pavel Hrdina
Pavel Hrdina (2): virt-clone: fix pylint error tests: introduce a test for removing unix channel path for virt-clone tests/cli-test-xml/compare/virt-clone-clone-auto1.xml | 5 + tests/testdriver.xml | 4 virtinst/cloner.py

[virt-tools-list] [virt-manager PATCH 1/2] virt-clone: fix pylint error

2015-11-03 Thread Pavel Hrdina
Signed-off-by: Pavel Hrdina --- virtinst/cloner.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/virtinst/cloner.py b/virtinst/cloner.py index 468e559..764388a 100644 --- a/virtinst/cloner.py +++ b/virtinst/cloner.py @@ -418,7 +418,7 @@ class Cloner(object): # new g

[virt-tools-list] [virt-manager PATCH] virt-install: report warning for cpuset=auto on non-NUMA host

2015-11-03 Thread Pavel Hrdina
Currently we report an error but continue with installation, we should report warning instead of error. Resolves: https://bugzilla.redhat.com/show_bug.cgi?id=1263903 Signed-off-by: Pavel Hrdina --- Pushed as trivial. virtinst/cli.py | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff

Re: [virt-tools-list] [PATCH 0/3] qemu/kvm: dmidecode improvements

2015-11-03 Thread Richard W.M. Jones
Thanks for these patches. I have pushed them upstream. Do we have a BZ for this? We'll need one if we are going to add these changes for RHEL 7.3. Rich. -- Richard Jones, Virtualization Group, Red Hat http://people.redhat.com/~rjones Read my programming and virtualization blog: http://rwmj.wo