Re: [PATCH 0/2] Fix network names with quotes

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Tue, Jun 09, 2015 at 07:44:36PM +0530, Shivaprasad bhat wrote:
> On Mon, Jun 8, 2015 at 7:09 PM, Michal Privoznik <mprivozn@xxxxxxxxxx> wrote:
> > On 01.06.2015 10:06, Shivaprasad G Bhat wrote:
> >> The following series implements...
> >>
> >> ---
> >>
> >> Shivaprasad G Bhat (2):
> >>       fix domaincommon.rng to accept network name with quotes
> >>       escape quotes for dsmasq conf contents
> >>
> >>
> >>  docs/schemas/domaincommon.rng                      |    2 +-
> >>  src/util/virdnsmasq.c                              |   25 +++++++++++++++----
> >>  src/util/virpidfile.c                              |   15 ++++++++----
> >>  .../nat-network-name-with-quotes.conf              |   20 +++++++++++++++
> >>  .../nat-network-name-with-quotes.xml               |   26 ++++++++++++++++++++
> >>  tests/networkxml2conftest.c                        |    1 +
> >>  6 files changed, 79 insertions(+), 10 deletions(-)
> >>  create mode 100644 tests/networkxml2confdata/nat-network-name-with-quotes.conf
> >>  create mode 100644 tests/networkxml2confdata/nat-network-name-with-quotes.xml
> >>
> >
> >
> > There's nothing wrong with the patches. I'm just curious, what's the use
> > case? I always thought that name should be something simple. On the
> > other hand, we do something similar with domain names IIRC.
> >
> 
> Hi Michal,
> 
> I too am not sure if the client apps use the quotes. My tester
> reported it having issues as he was trying
> combinations including non-english language characters.
> 
> I saw the quotes being handled diligently everywhere except here. So
> posted the patches fixing them.

I was able to define a guest using quotes in the name, and start it
successfully. So on that basis, I think we should allow it for networks
and other object types whereever possible, for sake of consistency.

IOW, I don't care whether there's an explicit use case mentioned - it
is justiable based on existing practice :-)

$ virsh list
 Id    Name                           State
----------------------------------------------------
 3     ser"foo"ial                    running

$ ps -axuwf | grep qemu

berrange 18386  0.3  0.2 1398604 48240 ?       Sl   15:26   0:03 /usr/bin/qemu-system-x86_64 -machine accel=kvm -name ser"foo"ial -S -machine pc-i440fx-1.4,accel=tcg,usb=off -cpu SandyBridge,+erms,+smep,+fsgsbase,+rdrand,+f16c,+osxsave,+pcid,+pdcm,+xtpr,+tm2,+est,+smx,+vmx,+ds_cpl,+monitor,+dtes64,+pbe,+tm,+ht,+ss,+acpi,+ds,+vm

$ ls $XDG_RUNTIME_DIR/libvirt/qemu/run/
ser"foo"ial.pid  ser"foo"ial.xml

Regards,
Daniel
-- 
|: http://berrange.com      -o-    http://www.flickr.com/photos/dberrange/ :|
|: http://libvirt.org              -o-             http://virt-manager.org :|
|: http://autobuild.org       -o-         http://search.cpan.org/~danberr/ :|
|: http://entangle-photo.org       -o-       http://live.gnome.org/gtk-vnc :|

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




[Index of Archives]     [Virt Tools]     [Libvirt Users]     [Lib OS Info]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite News]     [KDE Users]     [Fedora Tools]