Re: [virt-tools-list] virt-manager 2.2.1 does nothing when I click “New Virtual Machine”

2019-10-10 Thread Cole Robinson
On 10/10/19 2:39 PM, Michael wrote: Hi All, Newbie with virt-manager, 20+ yrs w/ Linux. I tried the Debian 9 (stretch) default install (virt-manager 1.4.0-5), but it wasn’t working [“python 3.4 or later is required, your's is sys.version_info(major=2, minor=7, micro=13, releaselevel='final', se

Re: [virt-tools-list] Checking on plans for cloud-init

2019-11-01 Thread Cole Robinson
On 10/23/19 9:21 AM, Christian Ehrhardt wrote: > Hi, > I was seeing activity on cloud-init for virtinst ending up in a > 'cloudinit' branch that still is in the project [1]. I know that this > was due to a GSOC [2] project and wanted to thank all of you for that. > But according to [3] I'd guess th

Re: [virt-tools-list] Saved VM state not part of snapshot?

2019-11-12 Thread Cole Robinson
On 10/25/19 2:28 PM, Michael Weiser wrote: > Hello, > > as an end user I today realized and was quite surprised that the saved > VM state of qemu/KVM VMs in virt-manager/libvirt does not seem to be > part of internal qcow2 snapshots. Is that understanding correct? > > The libvirt Wiki says as muc

Re: [virt-tools-list] [PATCH 1/1] connection: Avoid repeated default pool creation attempts

2019-11-12 Thread Cole Robinson
On 10/25/19 3:27 PM, Michael Weiser wrote: > During startup virtinst.StoragePool.build_default_pool() tries to > determine whether the default storage pool already exists. Because > events have not yet been processed, the list of existing storage pools > is still empty. Therefore it seems as if it

Re: [virt-tools-list] Saved VM state not part of snapshot?

2019-11-14 Thread Cole Robinson
On 11/14/19 10:11 AM, Michael Weiser wrote: > Hello Cole, > > thanks for getting back to me. > > On Tue, Nov 12, 2019 at 12:08:20PM -0500, Cole Robinson wrote: > >>> - create a snapshot of this suspended state including disk and memory >> Since the VM

Re: [virt-tools-list] Guest time sync on snapshot resume

2019-11-17 Thread Cole Robinson
On 11/14/19 10:30 AM, Michael Weiser wrote: > Hello, > > I expect this pops up every couple of months and apologise if indeed it > does. But I couldn't find a discussion specific to virt-manager and so > it goes: > > When working with snapshots, guests will have outdated date and time > upon resu

Re: [virt-tools-list] Saved VM state not part of snapshot?

2019-11-19 Thread Cole Robinson
On 11/19/19 11:15 AM, Michael Weiser wrote: > Hello Cole, > > On Thu, Nov 14, 2019 at 10:43:58AM -0500, Cole Robinson wrote: >>> Instead of handling the symptoms, might it be easier to try and make the >>> saved memory state become part of the snapshot then? I could th

[virt-tools-list] virt-install and cloud-init, feedback wanted

2019-11-20 Thread Cole Robinson
Hi all. The purpose of this mail is to get some feedback on pending cloud-init support in virt-install. If you're on the CC list here, I either pulled your email from a cloud-init discussion on the the virt-tools-list mailing list, or from the CC list of this virt-install bug: RFE: Provide cloud-i

Re: [virt-tools-list] virt-install and cloud-init, feedback wanted

2019-11-21 Thread Cole Robinson
Thanks for the feedback Dusty! I think Dan's response covered the tricky bit, I'll cover some response about ssh keys elsewhere in the thread. One bit here though: On 11/20/19 8:18 PM, Dusty Mabe wrote: > > > On 11/20/19 5:49 PM, Cole Robinson wrote: >> Hi all. The p

Re: [virt-tools-list] virt-install and cloud-init, feedback wanted

2019-11-21 Thread Cole Robinson
On 11/21/19 10:49 AM, Ryan Harper wrote: > * Cole Robinson [2019-11-20 17:48]: >> Hi all. The purpose of this mail is to get some feedback on pending >> cloud-init support in virt-install. If you're on the CC list here, I >> either pulled your email from a cloud-init dis

Re: [virt-tools-list] virt-install and cloud-init, feedback wanted

2019-11-21 Thread Cole Robinson
On 11/21/19 5:28 AM, Daniel P. Berrangé wrote: > On Wed, Nov 20, 2019 at 05:49:45PM -0500, Cole Robinson wrote: >> Hi all. The purpose of this mail is to get some feedback on pending >> cloud-init support in virt-install. If you're on the CC list here, I >> either pull

Re: [virt-tools-list] virt-install and cloud-init, feedback wanted

2019-11-22 Thread Cole Robinson
On 11/21/19 9:51 PM, Dusty Mabe wrote: > > > On 11/21/19 5:34 AM, Daniel P. Berrangé wrote: >> On Wed, Nov 20, 2019 at 08:18:01PM -0500, Dusty Mabe wrote: >>> >>> >>> On 11/20/19 5:49 PM, Cole Robinson wrote: > >>>> * Any other ideas, t

Re: [virt-tools-list] virt-install and cloud-init, feedback wanted

2019-11-22 Thread Cole Robinson
On 11/22/19 5:12 AM, Daniel P. Berrangé wrote: > On Thu, Nov 21, 2019 at 07:44:38PM -0500, Cole Robinson wrote: >> On 11/21/19 5:28 AM, Daniel P. Berrangé wrote: >>> One option is to take the lazy approach and always enable cloud init >>> if we are given a pre-built di

Re: [virt-tools-list] [virt-manager PATCH v2] controller: don't set default controller index

2019-12-03 Thread Cole Robinson
root-port \ > $VM > > Signed-off-by: Pavel Hrdina Reviewed-by: Cole Robinson - Cole ___ virt-tools-list mailing list virt-tools-list@redhat.com https://www.redhat.com/mailman/listinfo/virt-tools-list

Re: [virt-tools-list] virt-manager test suite hangs on FreeBSD

2019-12-11 Thread Cole Robinson
On 12/11/19 5:22 AM, Andrea Bolognani wrote: > In the libvirt-jenkins-ci repo, which contains the configuration for > the CI environment used by libvirt and related project, in the > virt-manager specific configuration we've had this comment for a > while: > > # libxml2's Python 3 bindings don't

Re: [virt-tools-list] virt-manager test suite hangs on FreeBSD

2019-12-11 Thread Cole Robinson
On 12/11/19 7:40 AM, Andrea Bolognani wrote: > On Wed, 2019-12-11 at 07:28 -0500, Cole Robinson wrote: >> On 12/11/19 5:22 AM, Andrea Bolognani wrote: >>> I have no idea how to debug this thing. Can an actual virt-manager >>> developer jump in? I'll gladly pro

Re: [virt-tools-list] virt-manager test suite hangs on FreeBSD

2019-12-13 Thread Cole Robinson
On 12/12/19 5:17 AM, Daniel P. Berrangé wrote: > On Thu, Dec 12, 2019 at 09:47:42AM +0100, Andrea Bolognani wrote: >> On Wed, 2019-12-11 at 18:01 -0500, Cole Robinson wrote: >>> On 12/11/19 7:40 AM, Andrea Bolognani wrote: >>>> On Wed, 2019-12-11 at 07:28 -0500, Cole

Re: [virt-tools-list] [virt-manager][PATCH] domcapabilities: Whitelist upstream qemu edk2 paths

2019-12-17 Thread Cole Robinson
On 12/14/19 6:48 AM, Michal Privoznik wrote: > Upstream qemu installs > /usr/share/qemu/edk2-$ARCH-(?secure-)code.fd FW images. Whitelist > them too. > > Signed-off-by: Michal Privoznik Reviewed-by: Cole Robinson And pushed - Cole ___

Re: [virt-tools-list] [RFC PATCH v3 0/7] Guest time sync on snapshot resume

2019-12-17 Thread Cole Robinson
On 12/9/19 4:53 PM, Michael Weiser wrote: > Hello Cole, > > this is the third iteration of implementing guest time sync as per > your suggestions. The only change compared to v2 is in the way the last > patch blocks the toggled signal. > > These patches do: > * add method _set_time() calling the

Re: [virt-tools-list] Need virt-install to boot Ubuntu netboot initrd

2019-12-23 Thread Cole Robinson
On 12/23/19 12:07 PM, Orion Poplawski wrote: > I'm trying to use koan with cobbler to install a Ubuntu 16.04 VM. koan > invokes virt-install with: > > 'virt-install', '--connect', 'qemu:///system', '--name', 'vmub16', '--ram', > '2000', '--vcpus', '1', '--graphics', 'spice', '--virt-type', 'kvm',

Re: [virt-tools-list] [virt-manager PATCH 2/2] virtManager: object: domain: Async set guest time

2019-12-23 Thread Cole Robinson
timeout > to 30 seconds. > I pushed patch one. Some comments here > Signed-off-by: Michael Weiser > Suggested-by: Cole Robinson > --- > virtManager/object/domain.py | 49 > 1 file changed, 38 insertions(+), 11 deletions(-) > &g

Re: [virt-tools-list] [virt-manager RFC PATCH 0/2] Warn about saved state behaviour with snapshots

2019-12-23 Thread Cole Robinson
On 12/19/19 4:52 PM, Michael Weiser wrote: > Hello Cole, > > I've done a bit of work on the issue of saved memory state not becoming > part of snapshots: > > On 10/25/19 2:28 PM, Michael Weiser wrote: > >> * Run/Revert of a snapshot should be rejected if the VM is in the >> 'Saved' state, aka ha

Re: [virt-tools-list] [virt-manager RFC PATCH 2/2] details: snapshots: Warn of saved state on creation

2019-12-23 Thread Cole Robinson
On 12/19/19 4:52 PM, Michael Weiser wrote: > Add a confirmation dialog when creating a new snapshot if there is saved > memory state so the user is aware that memory state will not be part of > the snapshot. > > Signed-off-by: Michael Weiser > Suggested-by: Cole Robinson &g

Re: [virt-tools-list] [virt-manager PATCH] Install the AppStream file to the canonical location

2020-01-14 Thread Cole Robinson
o Reviewed-by: Cole Robinson and pushed Thanks, Cole ___ virt-tools-list mailing list virt-tools-list@redhat.com https://www.redhat.com/mailman/listinfo/virt-tools-list

Re: [virt-tools-list] [virt-manager PATCH] Avoid string puzzle

2020-01-14 Thread Cole Robinson
Toscano Reviewed-by: Cole Robinson and pushed Thanks, Cole ___ virt-tools-list mailing list virt-tools-list@redhat.com https://www.redhat.com/mailman/listinfo/virt-tools-list

Re: [virt-tools-list] [virt-manager] [PATCH] details: Fix showing the firmware type in case of firmware auto selection

2020-01-14 Thread Cole Robinson
On 1/9/20 11:14 PM, Lin Ma wrote: > For a shutoff VM, If user select uefi firmware auto selection, e.g. > > ... > > > Its firmware information is set to 'BIOS' in details, This is incorrect. > This fixes it. > > Signed-off-by: Lin Ma > --- > virtManager/details/details.py | 7 +-- > virti

Re: [virt-tools-list] [virt-manager PATCH v2 0/6] Set guest time asynchronously

2020-01-14 Thread Cole Robinson
On 1/3/20 3:12 PM, Michael Weiser wrote: > Hello Cole, > > here's an updated series that reworks the time setting into a separate > thread. > >> while self._thread and self._thread.isAlive(): >>self._do_cancel = True >>self._do_cancel = False > > I've replaced that co

Re: [virt-tools-list] [PATCH] virtManager: Folder sharing implementation for SPICE session

2020-01-14 Thread Cole Robinson
On 12/28/19 7:12 AM, Jitao Lu wrote: > * This implements folder sharing for the built-in Spice client, tested >working with Win10 guest. > * The basic idea is taken from virt-viewer. > > Signed-off-by: Jitao Lu > --- > Feel free to improve the patch! > Nice work! This looks pretty good. S

Re: [virt-tools-list] [virt-manager PATCH v2] details: Fix showing the firmware type in case of firmware auto selection

2020-01-15 Thread Cole Robinson
_uefi(): > +firmware = 'UEFI' > +else: > +firmware = domcaps.label_for_firmware_path( > +self.vm.get_xmlobj().os.loader) > if self.widget("overview-firmware").is_visible(): > uiutil.set_li

Re: ERROR Could not find an installable distribution at ....

2020-02-13 Thread Cole Robinson
On 2/13/20 12:18 PM, Kaushal Shriyan wrote: > Hi, > > I am trying to set up alpine Linux Guest OS using the below command > > #virt-install --name=mfecdrupalalphine > --file=/linuxkvmguestosdisk/var/lib/libvirt/images/mfecdrupalalphine.img > --file-size=100 --nonsparse --vcpus=2 --ram=6096 --netw

Re: ERROR Could not find an installable distribution at ....

2020-02-13 Thread Cole Robinson
On 2/13/20 2:24 PM, Kaushal Shriyan wrote: > > > On Thu, Feb 13, 2020 at 11:01 PM Cole Robinson <mailto:crobi...@redhat.com>> wrote: > > On 2/13/20 12:18 PM, Kaushal Shriyan wrote: > > Hi, > > > > I am trying to set up alp

Re: Adding an "Enable Launch Security" checkbox to the Memory Details dialog

2020-03-27 Thread Cole Robinson
CCing Erik who knows more about that launchSecurity/sev than I do On 3/27/20 11:44 AM, Charles Arnold wrote: > What is the opinion of adding a checkbox called "Enable Launch > Security" under the 'Current allocation' and 'Maximum allocation' boxes > on the Details->Memory dialog? It would only be

Re: [PATCH virt-manager] rng: add builtin rng backend model

2020-04-21 Thread Cole Robinson
On 4/19/20 9:39 AM, Han Han wrote: > The builtin rng backend uses getrandom syscall to generate random, no > external rng source needed, introduced from libvirt v6.1.0. > Thanks, pushed now - Cole

Re: [virt-manager PATCH] disk: Support disk io mode io_uring

2020-04-30 Thread Cole Robinson
On 4/26/20 11:55 PM, Han Han wrote: > Support the io_uring io mode, introduced since Libvirt v6.3.0. > > Signed-off-by: Han Han > --- > man/virt-install.pod | 2 +- > virtinst/devices/disk.py | 1 + > 2 files changed, 2 insertions(+), 1 deletion(-) > > diff --git a/man/virt-install.pod b/ma

Re: Black Screen with Spice and OpenGL acceleration in virt-manager with QEMU/KVM

2020-05-06 Thread Cole Robinson
On 5/1/20 7:28 AM, Filip Banák wrote: > Hello. > > I just want to use an Arch Linux guest with OpenGL 3D acceleration on > with Spice. > It works with Spice when the OpenGL acceleration is disabled. > But when I enable it (with vitio graphics with accel3d yes of course), I > only see a black scree

Re: Black Screen with Spice and OpenGL acceleration in virt-manager with QEMU/KVM

2020-05-06 Thread Cole Robinson
orks even like that but I can use it only > blindly through the keyboard, because the screen is just black. > > > Dňa st 6. 5. 2020, 20:14 Cole Robinson <mailto:crobi...@redhat.com>> napísal(a): > > On 5/1/20 7:28 AM, Filip Banák wrote: > > Hello. &g

Re: Plans for the new Ubuntu installer and a new release?

2020-07-04 Thread Cole Robinson
On 5/28/20 2:35 AM, Christian Ehrhardt wrote: > Hi, > I was writing a very minor patch [1] clearing the remaining Ubuntu > Delta. And then realized I was adding way too much content into the PR > text that doesn't match the context there. > That is better as a ML submission I guess :-) > > > On o

Re: [PATCH v2 0/7] change default TPM for ppc64 + pylint fixes

2020-07-04 Thread Cole Robinson
On 6/3/20 9:14 AM, Daniel Henrique Barboza wrote: > Changes in v2: > - a fellow brazilian programmer informed me in private that I was > using 'forth' instead of 'fourth' all around the commit message > of patch 06. Let's avoid the irony of all the typos in a codespell > fix series. > > v1 link:

Re: [virt-manager PATCH v2 0/2] Mention io_uring support

2020-07-04 Thread Cole Robinson
On 5/7/20 3:45 AM, Han Han wrote: > v1: > https://www.redhat.com/archives/virt-tools-list/2020-April/msg00019.html > > Diff from v1: Remove unused constant. > > Han Han (2): > disk: Remove unused constant IO_MODE_THREADS > docs: Add disk io mode io_uring > > man/virt-install.pod | 2 +-

Re: [virt-manager PATCH v2 0/5] Improve translations

2020-07-07 Thread Cole Robinson
On 7/7/20 4:53 PM, Pino Toscano wrote: > This patch series improve the handling of translations. > > Split the current virt-manager catalog in two: > - a virt-manager one, containing only the messages for the GUI; its > translations are still build and installed as usual > - a virt-manager-meta

Re: [virt-manager PATCH v4 0/9] Improve translations

2020-07-12 Thread Cole Robinson
On 7/8/20 10:54 AM, Pino Toscano wrote: > This patch series improve the handling of translations. > > Use the gettext tools to: > - extract the messages from all the various sources > - merge old translations with the new catalog > - build translations as mo files > - merge translations to desktop

Re: [PATCH] virtinst: add support for configuring the IOMMU

2020-07-12 Thread Cole Robinson
On 7/7/20 3:55 AM, Menno Lageman wrote: > Add a --iommu option to configure IOMMU parameters as described in > https://libvirt.org/formatdomain.html#elementsIommu > > E.g. 'virt-install --iommu model=intel,driver.aw_bits=48,driver.iotlb=on ...' > will generate the following domain XML: > > >

Re: [virt-manager RFC PATCH 3/3] Add GUI support for enabling Secure Encrypted Virtualization

2020-07-12 Thread Cole Robinson
Sorry for the long review delay, I've been hoarding virt-manager bits in a folder until I could find the time. I pushed patch #1 The interconnected sev + iommu=on + locked memory requirement scares me. Let's say user clicks the sev checkbox but it the VM fails to boot for some reason. They unclic

Re: [virt-manager PATCH 00/21] Various i18n fixes

2020-07-13 Thread Cole Robinson
On 7/13/20 4:25 AM, Pino Toscano wrote: > This patch series tries to remove some of the i18n issues, mostly > untranslatable strings, and string puzzles. > > Not all the issues are fixed: there are still some string puzzles > around (for example the labels for disks in the VM details); they will >

Re: [virt-manager PATCH v4 0/9] Improve translations

2020-07-13 Thread Cole Robinson
On 7/13/20 4:58 AM, Pino Toscano wrote: > On Sunday, 12 July 2020 15:12:24 CEST Cole Robinson wrote: >> On 7/8/20 10:54 AM, Pino Toscano wrote: >>> This patch series improve the handling of translations. >>> >>> Use the gettext tools to: >>> - extr

Re: [virt-manager PATCH 00/16] Various i18n fixes, part #2

2020-07-14 Thread Cole Robinson
On 7/14/20 3:41 AM, Pino Toscano wrote: > This patch series remove some more of the i18n issues, mostly > untranslatable strings, string puzzles, and plural forms. > > Also a couple of bugs related to i18n are fixed. > Thanks! I pushed with this squashed into the details.py controller patch, so

Re: [virt-manager PATCH 1/2] ci: commit the catalog only if changed

2020-07-14 Thread Cole Robinson
On 7/14/20 9:08 AM, Pino Toscano wrote: > Compare the newly generated catalog with the old one: if it actually > changed (i.e. there are changes beside the creation date), then push it > to the 'translations' branch. Thanks, I pushed these two - Cole

Re: [virt-manager PATCH 0/3] Various i18n fixes, part #3

2020-07-15 Thread Cole Robinson
On 7/15/20 8:48 AM, Pino Toscano wrote: > This patch series remove some more of the i18n issues, mostly > untranslatable strings, and string puzzles. > > Pino Toscano (3): > i18n: add missing placeholder name in string > i18n: create single strings for texts/messages > i18n: translate UI str

Re: [virt-manager PATCH] bash completion: remove shebang

2020-07-20 Thread Cole Robinson
On 7/19/20 6:10 AM, Pino Toscano wrote: > The bash completion file is not a script to be executed directly, but > it is sourced by bash. Hence, remove its shebang. > > Signed-off-by: Pino Toscano > --- > data/bash-completion.sh.in | 2 -- > 1 file changed, 2 deletions(-) Thanks, pushed now - C

Re: [virt-manager PATCH] about: use the standard license type

2020-07-20 Thread Cole Robinson
On 7/18/20 4:09 AM, Pino Toscano wrote: > Use the standard gpl-2-0 license type of GtkAboutDialog, instead of the > custom license text: this way, the dialog will show a translated text > with the license type, and a link to the full license text. > > Signed-off-by: Pino Toscano Thanks, pushed n

Re: [PATCH] Correct two translations for zh_CN

2020-08-05 Thread Cole Robinson
On 8/5/20 4:32 AM, Pavel Hrdina wrote: > On Wed, Aug 05, 2020 at 12:44:24PM +0800, Huacai Chen wrote: >> Hi, Pavel, >> >> On Mon, Aug 3, 2020 at 8:06 PM Pavel Hrdina wrote: >>> >>> On Mon, Aug 03, 2020 at 04:45:03PM +0800, Huacai Chen wrote: There is no "Error" in "Refresh snapshot list" and

Re: [PATCH] Correct two translations for zh_CN

2020-08-12 Thread Cole Robinson
On 8/6/20 1:44 AM, Huacai Chen wrote: > Hi, Daniel, > > On Wed, Aug 5, 2020 at 10:57 PM Daniel P. Berrangé > wrote: >> >> On Wed, Aug 05, 2020 at 10:42:22AM -0400, Cole Robinson wrote: >>> On 8/5/20 4:32 AM, Pavel Hrdina wrote: >>>> On Wed, Aug 05

Re: [PATCH] virtManager: Folder sharing implementation for SPICE session

2020-08-12 Thread Cole Robinson
On 7/25/20 9:30 AM, Jitao Lu wrote: > * This implements folder sharing for the built-in Spice client, tested >working with Win10 guest. > * The basic idea is taken from virt-viewer. > > Signed-off-by: Jitao Lu > --- > ui/spicewebdav.ui | 129 + >

Re: [PATCH] Correct two translations for zh_CN

2020-08-13 Thread Cole Robinson
On 8/13/20 12:21 AM, Huacai Chen wrote: > Hi, Cole, > > On Thu, Aug 13, 2020 at 2:07 AM Cole Robinson wrote: >> >> On 8/6/20 1:44 AM, Huacai Chen wrote: >>> Hi, Daniel, >>> >>> On Wed, Aug 5, 2020 at 10:57 PM Daniel P. Berrangé >>>

Re: [PATCH] virtManager: Folder sharing implementation for SPICE session

2020-08-13 Thread Cole Robinson
to expose more/all spice/vnc options, and virt-manager would only cover a common subset, to save us from having to maintain a duplicate feature set. Thanks, Cole > 閸︼拷 2020/8/13 娑撳﹤宕�4:29, Cole Robinson 閸愭瑩浜�: >> On 7/25/20 9:30 AM, Jitao Lu wrote: >>>   * This implements folder sh

Re: [PATCH] virtManager: Folder sharing implementation for SPICE session

2020-08-13 Thread Cole Robinson
ot;? I think you can watch > over "port-opened" [0] for that. > > [0] > https://www.spice-space.org/api/spice-gtk/SpicePortChannel.html#SpicePortChannel--port-opened > No, what we mean is some notification from spice that there is actually a webdav agent listening/ready in

Re: [virt-manager PATCH] osdict: extend non-EOL distros range to 10 years

2020-08-18 Thread Cole Robinson
On 8/18/20 12:47 PM, Pino Toscano wrote: > Commit d52d9885c85623b8d924dbf0aceecb08b33e9122 added a logic to > consider as EOL a distribution with no EOL date set and release date > earlier than 5 years from the current day. This was done because there > were (and still are, even if fewer now) many

Re: [virt-manager PATCH 0/3] Improve Mageia tree detection

2020-08-31 Thread Cole Robinson
On 8/26/20 6:44 AM, Pino Toscano wrote: > Detect the version, and slightly improve the tests for it. > > Pino Toscano (3): > tests: tweak mageia detection > urldetect: add version detection for Mageia > tests: add Mageia 5 detection test > > tests/data/urldetect/mageia/5/VERSION |

ANNOUNCE: virt-manager 3.0.0 released

2020-09-15 Thread Cole Robinson
thina Plaskasoviti, Cole Robinson) - The virt-convert tool has been removed. Please use virt-v2v instead - A handful of UI XML configuration options have been removed. The XML editor can be used instead. For a larger discussion see this thread: https://www.redhat.com/archives/virt-tools-list/2019

Re: [virt-manager PATCH 8/9] i18n: use a single string for error message

2020-09-17 Thread Cole Robinson
On 9/17/20 2:44 AM, Pino Toscano wrote: > Signed-off-by: Pino Toscano > --- > virtManager/virtmanager.py | 5 +++-- > 1 file changed, 3 insertions(+), 2 deletions(-) > > diff --git a/virtManager/virtmanager.py b/virtManager/virtmanager.py > index ee7d4d89..2c8ccbb8 100755 > --- a/virtManager/vir

Re: [virt-manager PATCH 0/9] Various i18n fixes, part #4

2020-09-17 Thread Cole Robinson
On 9/17/20 2:43 AM, Pino Toscano wrote: > This patch series remove some more of the i18n issues, mostly > untranslatable strings, and string puzzles. > > Pino Toscano (9): > i18n: fix string puzzle in volume creation dialog > i18n: improve title of connection dialog > i18n: fix string puzzle

Re: [virt-manager PATCH 2/9] i18n: improve title of connection dialog

2020-09-17 Thread Cole Robinson
On 9/17/20 2:43 AM, Pino Toscano wrote: > Instead of using the title of the dialog and prepending the connection > label, create a new title as a single string. This way it is possible to > translate this title as single sentence. > > Signed-off-by: Pino Toscano > --- > virtManager/host.py | 5 +

virt-install: changing default --os-variant behavior

2020-09-20 Thread Cole Robinson
Hi virt-tools-list + CCd libosinfo developers too Use of virt-install without a detected guest OS, and without manually specified --os-variant, is a never ending source of bugs and user confusion and poorly performing VM configs. This mail is intended to start a discussion about how to improve tha

Re: virt-install: changing default --os-variant behavior

2020-09-22 Thread Cole Robinson
On 9/20/20 4:46 PM, Peter Crowther wrote: > On Sun, 20 Sep 2020 at 21:10, Cole Robinson <mailto:crobi...@redhat.com>> wrote: > [...] > > 2) Default to --os-variant detect=on,name=. 'give me > virtio' is representative of what most virt-install users

Re: virt-install: changing default --os-variant behavior

2020-09-22 Thread Cole Robinson
On 9/22/20 11:59 AM, Peter Crowther wrote: > On Tue, 22 Sep 2020 at 16:48, Cole Robinson <mailto:crobi...@redhat.com>> wrote: > > On 9/20/20 4:46 PM, Peter Crowther wrote: > > On Sun, 20 Sep 2020 at 21:10, Cole Robinson <mailto:crobi...@redhat

Re: virt-install: changing default --os-variant behavior

2020-09-22 Thread Cole Robinson
On 9/22/20 11:53 AM, Pino Toscano wrote: > On Sunday, 20 September 2020 22:09:54 CEST Cole Robinson wrote: >> Use of virt-install without a detected guest OS, and without manually >> specified --os-variant, is a never ending source of bugs and user >> confusion and poorly

Re: virt-install: changing default --os-variant behavior

2020-09-23 Thread Cole Robinson
On 9/21/20 4:01 PM, Charles Arnold wrote: > On Sun, 2020-09-20 at 16:09 -0400, Cole Robinson wrote: >> Hi virt-tools-list + CCd libosinfo developers too >> >> Use of virt-install without a detected guest OS, and without manually >> specified --os-variant, is a never end

ANNOUNCE: virt-manager 3.1.0 released

2020-09-30 Thread Cole Robinson
I'm happy to announce the release of virt-manager 3.1.0! virt-manager is a desktop application for managing KVM, Xen, and LXC virtualization via libvirt. The release can be downloaded from: http://virt-manager.org/download/ This release includes: - Return to using qcow2 sparse by default with

Re: virt-install: changing default --os-variant behavior

2020-10-21 Thread Cole Robinson
Following on from this I submitted a linux2020 entry to osinfo-db: https://gitlab.com/libosinfo/osinfo-db/-/merge_requests/237 Thanks, Cole On 9/20/20 4:09 PM, Cole Robinson wrote: > Hi virt-tools-list + CCd libosinfo developers too > > Use of virt-install without a detected gues

Re: [virt-manager PATCH] Add /usr/sbin to search path for non-root user

2020-11-03 Thread Cole Robinson
On 10/23/20 5:20 PM, Charles Arnold wrote: > On a fresh install where no previous connection has been made a > non-root user starting virt-manager won't find the installed libvirtd > because the location is not in the user path. > > Signed-off-by: Charles Arnold > --- > virtManager/lib/connectau

ANNOUNCE: virt-manager 3.2.0 released

2020-11-16 Thread Cole Robinson
ANNOUNCE: virt-manager 3.2.0 released I'm happy to announce the release of virt-manager 3.2.0! virt-manager is a desktop application for managing KVM, Xen, and LXC virtualization via libvirt. The release can be downloaded from: http://virt-manager.org/download/ This release includes: - Slim d

Re: [virt-manager PATCH 0/3] Create pool: show only available types

2020-12-07 Thread Cole Robinson
On 11/24/20 9:21 AM, Pino Toscano wrote: > Hi, > > this series adds a minimal StoragePoolCapabilities handling using the > virConnect.getStoragePoolCapabilities libvirt API; this is used to > filter the available pool types in the "Create pool" dialog, so it does > not offer anymore pool types tha

Re: [virt-manager PATCH 0/3] Create pool: show only available types

2020-12-09 Thread Cole Robinson
bvirt XML, or you are copy+pasting XML from an online source, in which case having clicky (and limited) UI doesn't add much over the XML editor or the command line. - Cole > Cheers, > > - Peter > > On Mon, 7 Dec 2020 at 19:39, Cole Robinson <mailto:crobi...@redhat.com&g

Re: [virt-manager PATCH 0/3] Create pool: show only available types

2020-12-09 Thread Cole Robinson
On 12/8/20 9:09 AM, Pino Toscano wrote: > Hi Cole, > > I'm reordering the paragraphs of your answer to ease mine. > > On Monday, 7 December 2020 20:39:29 CET Cole Robinson wrote: >> Can you explain your motivation a bit: Has this caused you issues >> before? &g

Re: [PATCH] virtmanager: Fixed the legacy firmware installation error

2021-01-04 Thread Cole Robinson
On 12/22/20 3:24 AM, Cho, Yu-Chen wrote: > When we choose /usr/share/qemu/bios.bin or /usr/share/qemu/bios-256k.bin > in Customize configuration before install, show this error message: > > Unable to complete install: 'operation failed: unable to find any master var > store for loader: /usr/share

Re: [PATCH] virtmanager: Fixed the legacy firmware installation error

2021-01-11 Thread Cole Robinson
On 1/5/21 2:34 AM, Cho, Yu-Chen wrote: > When we choose /usr/share/qemu/bios.bin or /usr/share/qemu/bios-256k.bin > in Customize configuration before install, show this error message: > > Unable to complete install: 'operation failed: unable to find any master var > store for loader: /usr/share/q

Re: [PATCH] virtmanager: Fixed the legacy firmware installation error

2021-01-12 Thread Cole Robinson
On 1/11/21 10:52 PM, Al Cho wrote: > Hi , > > Yes, you are right, I only try git am and pytest after 2nd version, and > miss it. > Fixing it again. > > bios.bin/bios-256k.bin is from qemu-seabios package in openSUSE/SLE > those also can find in seabios-bin package in fedora/RedHat EL. > Yes

Re: [PATCH] virtmanager: Fixed the legacy firmware installation error

2021-01-13 Thread Cole Robinson
On 1/13/21 1:40 AM, Al Cho wrote: > Hi, > > That's before the last step during install new VM, > and there is a checkbutton "Customize Configure before install" > and Apply after choose Frimware bios.bin or bios-256k.bin, Click "Begin > Installation" > show up the error message, list virt-manager

Re: [virt-manager PATCH 0/2] Couple of bhyve related improvements

2021-02-08 Thread Cole Robinson
On 2/6/21 8:18 AM, Roman Bogorodskiy wrote: > Roman Bogorodskiy (2): > virtinst: prefer SATA bus for bhyve > virtinst: bhyve: properly configure loader > > virtinst/connection.py | 2 ++ > virtinst/devices/disk.py | 3 +++ > virtinst/domain/os.py| 16 > 3 files change

Re: [virt-manager PATCH 2/2] virtinst: bhyve: properly configure loader

2021-02-08 Thread Cole Robinson
On 2/6/21 8:18 AM, Roman Bogorodskiy wrote: > Bhyve requires explicit loader configuration. So query > domain capabilities, try to find the "official" > firmware and configure all the necessary loader options. > > Signed-off-by: Roman Bogorodskiy > --- > virtinst/domain/os.py | 16 ++

Re: [virt-manager PATCH] virtinst: fix error message format string

2021-02-10 Thread Cole Robinson
On 2/10/21 8:54 AM, Roman Bogorodskiy wrote: > Fix a regression introduced by commit 71f034d6b where > format string expects kwarg "domain", but "vm" is passed instead. > > Signed-off-by: Roman Bogorodskiy > --- > virtinst/virtxml.py | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > d

Re: [virt-manager PATCH 0/2] Couple of bhyve related improvements

2021-02-10 Thread Cole Robinson
On 2/9/21 10:51 AM, Roman Bogorodskiy wrote: > Cole Robinson wrote: > >> On 2/6/21 8:18 AM, Roman Bogorodskiy wrote: >>> Roman Bogorodskiy (2): >>> virtinst: prefer SATA bus for bhyve >>> virtinst: bhyve: properly configure loader >>> >

Re: [virt-manager PATCH v2 0/4] Couple of bhyve related improvements

2021-02-15 Thread Cole Robinson
On 2/11/21 10:41 AM, Roman Bogorodskiy wrote: > Changes from v1: > > - Added tests/test_cli.py testing, > - Reworked loader commit, > - Added nmdm console configuration commit. > > Roman Bogorodskiy (4): > tests: cli: start testing bhyve > virtinst: prefer SATA bus for bhyve > virtinst:

Re: [virt-manager PATCH 0/2] Couple of bhyve related improvements

2021-02-15 Thread Cole Robinson
On 2/11/21 6:34 AM, Roman Bogorodskiy wrote: > Cole Robinson wrote: > >> On 2/9/21 10:51 AM, Roman Bogorodskiy wrote: >>> Cole Robinson wrote: >>> >>>> On 2/6/21 8:18 AM, Roman Bogorodskiy wrote: >>>>> Roman Bogorodskiy (2): >>&g

Re: [virt-manager PATCH 2/2] virtinst: bhyve: properly configure loader

2021-02-15 Thread Cole Robinson
On 2/11/21 11:05 AM, Roman Bogorodskiy wrote: > Cole Robinson wrote: > >> On 2/6/21 8:18 AM, Roman Bogorodskiy wrote: >>> Bhyve requires explicit loader configuration. So query >>> domain capabilities, try to find the "official" >>> firmwa

Re: [PATCH v2] virtmanager: Fix legacy firmware installation error

2021-04-06 Thread Cole Robinson
On 3/10/21 4:02 PM, Charles Arnold wrote: > When we choose /usr/share/qemu/bios.bin or /usr/share/qemu/bios-256k.bin > in Customize configuration before install, show this error message: > > Unable to complete install: 'operation failed: unable to find any master var > store for loader: /usr/shar

Re: [PATCH 1/3] Fix forgetting password from keyring

2021-04-06 Thread Cole Robinson
On 3/21/21 1:44 PM, w...@torlan.ru wrote: > From: WGH > > First, Item.Delete never accepted any arguments, so this code likely > never worked. > > Second, Item.Delete might return a Prompt object, which client > is supposed to call if keyring wants to confirm deletion. > --- > virtManager/lib/k

Re: [PATCH virt-install v1 1/3] virt-install: add mediated device

2021-05-18 Thread Cole Robinson
On 4/14/21 11:18 AM, Shalini Chellathurai Saroja wrote: > Add support to install a virtual server with passed-through mediated > device. Mediated device can be created using vGPU attached to > vfio_pci driver or DASD attached to vfio_ccw driver or APQNs attached > to vfio_ap driver. > > Signed-off

Re: [PATCH virt-install v1 2/3] virt-xml: add support for mediated devices

2021-05-18 Thread Cole Robinson
On 4/14/21 11:18 AM, Shalini Chellathurai Saroja wrote: > Provide support to add/remove mdev in a guest domain, which is in > shut-off or running state (hotplug/unplug). Also support update of > already existing mdev device, when the guest domain is in shut-off > state. Please note that libvirt doe

Re: [PATCH virt-install v1 3/3] virtxml: prevent defining same hostdev again

2021-05-18 Thread Cole Robinson
On 4/14/21 11:18 AM, Shalini Chellathurai Saroja wrote: > Currently, it is possible to add same device multiple times, when the > guest domain is in shut-off state. This patch prevents this unexpected > behavior for all hostdev devices, including mdev devices. > If this is an invalid config, thes

Re: [PATCH virt-install] virtinst: Support rotation_rate attrib in target

2021-05-19 Thread Cole Robinson
On 5/18/21 2:36 AM, Han Han wrote: > Support rotation_rate attrib which is introduced since libvirt v7.3. > > Signed-off-by: Han Han Thanks! But you need to extend tests/test_cli.py --disk examples as well otherwise test suite is failing (but I didn't confirm) Thanks, Cole > --- > virtinst/cl

Re: [RFC virt-manager] virt-manager: enable mdev support

2021-05-23 Thread Cole Robinson
On 5/5/21 11:42 AM, Shalini Chellathurai Saroja wrote: > Enable virt-manager GUI to support add, edit, remove, hot-plug and > hot-unplug of mediated devices (like DASDs, APQNs and PCIs) in virtual > server. > > It is not possible to edit mdev when a virtual server is in > running state, as this is

Re: [PATCH v2] virtinst: Support rotation_rate attrib in target

2021-05-23 Thread Cole Robinson
On 5/20/21 2:48 AM, Han Han wrote: > Support rotation_rate attrib which is introduced since libvirt v7.3. > > Signed-off-by: Han Han > --- > Diff from v1: Add tests for the new options in virtinst > --- > tests/data/cli/compare/virt-install-many-devices.xml | 4 ++-- > tests/test_cli.py

Re: [PATCH 0/2] Support transient disk in virtinst

2021-05-26 Thread Cole Robinson
On 5/25/21 9:22 PM, Han Han wrote: > > Han Han (2): > virtinst: Implement support for transient disks > virtinst: Support transient.shareBacking option > > tests/data/cli/compare/virt-install-many-devices.xml | 2 ++ > tests/test_cli.py| 4 ++-- > virtinst

Re: [PATCH] cli: --feature: Support kvm.poll-control.state

2021-05-28 Thread Cole Robinson
On 5/27/21 12:01 AM, Han Han wrote: > Implement kvm.poll-control.state to `virt-install --feature`. It requires > libvirt >= v6.10.0. > > Signed-off-by: Han Han > --- > tests/data/cli/compare/virt-install-singleton-config-3.xml | 1 + > tests/test_cli.py

Re: Fwd: Re: [PATCH virt-install v1 3/3] virtxml: prevent defining same hostdev again

2021-05-28 Thread Cole Robinson
On 5/28/21 8:06 AM, Shalini Chellathurai Saroja wrote: > > On 5/19/21 12:49 AM, Cole Robinson wrote: >> On 4/14/21 11:18 AM, Shalini Chellathurai Saroja wrote: >>> Currently, it is possible to add same device multiple times, when the >>> guest domain is in shut-off s

Re: [PATCH virt-manager v2 0/4] Enable MDEV support

2021-06-01 Thread Cole Robinson
On 5/31/21 3:54 PM, Shalini Chellathurai Saroja wrote: > Enable support for mediated devices in virt-xml tool and virt-manager > GUI tool. > Thanks! I pushed 1-3 with a small tweak to patch #1 to keep code coverage at 100%. In order to get the parent naming stuff in addhardware.py to trigger I n

Re: [PATCH virt-manager v2 0/4] Enable MDEV support

2021-06-02 Thread Cole Robinson
On 6/2/21 9:34 AM, Shalini Chellathurai Saroja wrote: > > On 6/1/21 11:13 PM, Cole Robinson wrote: >> On 5/31/21 3:54 PM, Shalini Chellathurai Saroja wrote: >>> Enable support for mediated devices in virt-xml tool and virt-manager >>> GUI tool. >>> >&g

Re: ERROR internal error: Unexpected enum value 0 for virDomainDeviceAddressType

2021-06-07 Thread Cole Robinson
On 6/4/21 5:40 AM, Mario Marietto wrote: > Hello. > > I want to emulate the raspberry pi 3 on my Jetson nano (aarch64) using > virt-install. I'm learning how to do that by reading heavily here : > > https://github.com/dhruvvyas90/qemu-rpi-kernel >

Re: [PATCH virt-manager v3] virt-manager: enable MDEV support

2021-06-08 Thread Cole Robinson
On 6/7/21 5:34 AM, Shalini Chellathurai Saroja wrote: > Enable virt-manager GUI to support add, edit, remove, hot-plug and > hot-unplug of mediated devices (like DASDs, APQNs and PCIs) in virtual > server. > > It is not possible to edit MDEV when a virtual server is in > running state, as this is

<    3   4   5   6   7   8   9   10   11   12   >