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@xxxxxxxxxxxx>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@xxxxxxxxxxxx> instead of Author: dmichelotto <diego.michelotto@xxxxxxxxxxxx>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@xxxxxxxxxxxx> --- 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
Attachment:
signature.asc
Description: PGP signature
-- libvir-list mailing list libvir-list@xxxxxxxxxx https://www.redhat.com/mailman/listinfo/libvir-list
