Re: [PATCH] storagepoolxml2argvtest: Populate test entries for macOS

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

 



On Wed, Oct 28, 2020 at 09:52:47PM +0100, Andrea Bolognani wrote:
> On Mon, 2020-10-26 at 00:54 +0300, Roman Bolshakov wrote:
> >  tests/storagepoolxml2argvdata/pool-fs-darwin.argv               | 1 +
> >  tests/storagepoolxml2argvdata/pool-netfs-auto-darwin.argv       | 1 +
> >  tests/storagepoolxml2argvdata/pool-netfs-cifs-darwin.argv       | 1 +
> >  tests/storagepoolxml2argvdata/pool-netfs-darwin.argv            | 1 +
> >  tests/storagepoolxml2argvdata/pool-netfs-gluster-darwin.argv    | 1 +
> >  .../storagepoolxml2argvdata/pool-netfs-protocol-ver-darwin.argv | 1 +
> >  tests/storagepoolxml2argvtest.c                                 | 2 ++
> >  7 files changed, 8 insertions(+)
> >  create mode 100644 tests/storagepoolxml2argvdata/pool-fs-darwin.argv
> >  create mode 100644 tests/storagepoolxml2argvdata/pool-netfs-auto-darwin.argv
> >  create mode 100644 tests/storagepoolxml2argvdata/pool-netfs-cifs-darwin.argv
> >  create mode 100644 tests/storagepoolxml2argvdata/pool-netfs-darwin.argv
> >  create mode 100644 tests/storagepoolxml2argvdata/pool-netfs-gluster-darwin.argv
> >  create mode 100644 tests/storagepoolxml2argvdata/pool-netfs-protocol-ver-darwin.argv
> 
> This is unnecessarily complicated: all you need is
> 
>   diff --git a/tests/storagepoolxml2argvdata/pool-netfs-gluster.argv b/tests/storagepoolxml2argvdata/pool-netfs-gluster.argv
>   index 4303d514ef..97be9cbeb3 100644
>   --- a/tests/storagepoolxml2argvdata/pool-netfs-gluster.argv
>   +++ b/tests/storagepoolxml2argvdata/pool-netfs-gluster.argv
>   @@ -1 +1 @@
>   -mount -t glusterfs example.com:/volume -o direct-io-mode=1 /mnt/gluster
>   +mount -t glusterfs example.com:/volume /mnt/gluster -o direct-io-mode=1
> 
> If you dig through the git history, you'll see that when
> default_mount_opts was added to src/storage/storage_util.c, one of
> the virCommandAddArgList() calls used to build the mount command line
> has also seen its arguments reordered; however, since the per-OS
> split of output files was performed as part of the same patch instead
> of in a separate one, the existing output file was not updated
> accordingly.
> 
> Since you're correcting a mistake introduced in an old commit, you
> can include
> 
>   Fixes: f00cde7f1133fee96dc13a80d7f402c704346974
> 
> in your commit message.
> 

Hi Andrea,

Initially I updated only pool-netfs-gluster.argv but I wasn't sure if
I'm doing this right or wrong. Thanks for the commit reference, the hunk
proofs your point:

@@ -4278,8 +4305,8 @@ virStorageBackendFileSystemMountGlusterArgs(virCommandPtr cmd,
     const char *fmt;
 
     fmt = virStoragePoolFormatFileSystemNetTypeToString(def->source.format);
-    virCommandAddArgList(cmd, "-t", fmt, src, "-o", "direct-io-mode=1",
-                         def->target.path, NULL);
+    virCommandAddArgList(cmd, "-t", fmt, src, def->target.path, NULL);
+    virStorageBackendFileSystemMountAddOptions(cmd, "direct-io-mode=1");
 }

Regards,
Roman




[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]

  Powered by Linux