[libvirt] [PATCH] virfile: added GPFS as shared fs
Ján Tomko
jtomko at redhat.com
Tue Feb 26 09:03:23 UTC 2019
On Tue, Feb 26, 2019 at 09:08:27AM +0100, Peter Krempa wrote:
>On Mon, Feb 25, 2019 at 19:19:03 +0100, Diego Michelotto wrote:
>> From: dmichelotto <diego.michelotto at cnaf.infn.it>
>
>With your permission I'll remove the above line when applying the patch
>so that your authorship of the patch shows up as:
>
>Author: Diego Michelotto <diego.michelotto at cnaf.infn.it>
>
>instead of
>
>Author: dmichelotto <diego.michelotto at cnaf.infn.it>
>
>>
>> Added GPFS as shared file system recognized during live migration
>> security checks.
>>
>> GPFS is 'IBM General Parallel File System' also called
>> 'IBM Spectrum Scale'
>>
>> BUG: https://bugzilla.redhat.com/show_bug.cgi?id=1679528
>>
>> Signed-off-by: Diego Michelotto <diego.michelotto at cnaf.infn.it>
>> ---
>> src/util/virfile.c | 9 ++++++++-
>> src/util/virfile.h | 1 +
>> tests/virfiledata/mounts3.txt | 1 +
>> tests/virfilemock.c | 5 +++++
>> tests/virfiletest.c | 1 +
>> 5 files changed, 16 insertions(+), 1 deletion(-)
>
>One thing which might be worth exploring is whether GPFS is also
>migratable with cache != none and thus should be also added to
>virStorageFileIsClusterFS.
>
>ACK.
>
>Depending on the angle of view this can be viewed both as a bug and as a
>feature.
Bug, since we consider this a blocker for migration.
Also, probably a regression introduced by commit ed11e9cd95bd9cae6cdfab14ae0936930bbb63e6
qemuMigrationSrcIsSafe: Check local storage more thoroughly
>Since we are feature freeze, is anybody objecting in pushing
>this right away? (obviously I'm not going to push it right now)
>
I object to calling this a feature. Feel free to postpone if you deem it
too risky or invasive.
Jano
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 488 bytes
Desc: not available
URL: <http://listman.redhat.com/archives/libvir-list/attachments/20190226/de9936dc/attachment-0001.sig>
More information about the libvir-list
mailing list