[email protected]<mailto:[email protected]> ________________________________ From: Mitul Kothari <[email protected]> Sent: Tuesday, May 21, 2024 3:35 PM To: Ravi Singh <[email protected]>; Richard W.M. Jones <[email protected]> Cc: [email protected] <[email protected]>; Aswad Kulkarni <[email protected]>; Shailesh Marathe <[email protected]>; Sumit Dighe <[email protected]>; Brad Boyer <[email protected]>; Saket Pusalkar <[email protected]>; Gaurang Agnihotri <[email protected]>; Satyajit Gorhe parlikar <[email protected]>; John Cronin <[email protected]> Subject: Re: [External] Re: [Libguestfs] [PATCH] VxFS Filesystem support to libguestfs
Hi Richard, Just checking if this change has been merged yet as a patch to libguestfs? Regards, Mitul From: Ravi Singh <[email protected]> Date: Thursday, 2 November 2023 at 11:15 PM To: Richard W.M. Jones <[email protected]> Cc: [email protected] <[email protected]>, Aswad Kulkarni <[email protected]>, Shailesh Marathe <[email protected]>, Sumit Dighe <[email protected]>, Mitul Kothari <[email protected]>, Brad Boyer <[email protected]>, Saket Pusalkar <[email protected]>, Gaurang Agnihotri <[email protected]>, Satyajit Gorhe parlikar <[email protected]>, John Cronin <[email protected]> Subject: Re: [External] Re: [Libguestfs] [PATCH] VxFS Filesystem support to libguestfs Hi Richard, I wanted to inform you that we have implemented the changes as per your suggestion to move the logic for starting the Veritas Filesystem related services from the daemon file to appliance/init script. We have thoroughly tested the updated version on our end to ensure its functionality and reliability. Additionally, we have made the necessary modifications to the copyright, following the recommendations provided by our legal team. Please let us know if you require any further assistance. Thank you for your continued collaboration. We look forward to your response. Thanks, Ravi/Gaurang ________________________________ From: Ravi Singh <[email protected]> Sent: Thursday, September 21, 2023 9:36 PM To: Richard W.M. Jones <[email protected]> Cc: [email protected] <[email protected]>; Aswad Kulkarni <[email protected]>; Shailesh Marathe <[email protected]>; Sumit Dighe <[email protected]>; Mitul Kothari <[email protected]>; Brad Boyer <[email protected]>; Saket Pusalkar <[email protected]>; Gaurang Agnihotri <[email protected]>; Satyajit Gorhe parlikar <[email protected]>; John Cronin <[email protected]> Subject: Re: [External] Re: [Libguestfs] [PATCH] VxFS Filesystem support to libguestfs Hi Richard, Thank you for making these changes. We appreciate your insights. We will promptly address the issues you have mentioned, thoroughly test them. We will keep you updated and send you the patch once we have completed the testing. Thanks, Ravi/Gaurang ________________________________ From: Richard W.M. Jones <[email protected]> Sent: Tuesday, September 19, 2023 4:30 PM To: Ravi Singh <[email protected]> Cc: [email protected] <[email protected]>; Aswad Kulkarni <[email protected]>; Shailesh Marathe <[email protected]>; Sumit Dighe <[email protected]>; Mitul Kothari <[email protected]>; Brad Boyer <[email protected]>; Saket Pusalkar <[email protected]>; Gaurang Agnihotri <[email protected]>; Satyajit Gorhe parlikar <[email protected]>; John Cronin <[email protected]> Subject: Re: [External] Re: [Libguestfs] [PATCH] VxFS Filesystem support to libguestfs CAUTION: This email originated from outside the organization. Do not click links or open attachments unless you recognize the sender and know the content is safe. If you believe this is a phishing email, use the Report to Cybersecurity icon in Outlook. A new patch is attached. This is only build tested so far. To build with this patch I had to do: ./configure --enable-werror \ --with-supermin-packager-config=$PWD/localyum.conf where localyum.conf contains: ---------------------------------------------------------------------- [local] name=local baseurl=file:///root/dvd1-redhatlinux/rhel9_x86_64/rpms/ enabled=1 gpgcheck=0 ---------------------------------------------------------------------- (adjust the path to point to the Veritas DVD mount point) I made various changes in this patch: - Add copyright and license notice to list_vxvm.ml & vxfs.c. Please ensure all files have copyright and license notices, and if necessary get them checked by your legal. - Change vm.ml{,i} -> list_vxvm.ml{,i} - In packagelist.in, move package names to their own section. - Remove perl/python packages as perl/python is not present in the appliance. - Fix commit message. - Line length under 80 columns in a few places. - Function names at start of line. - List SOURCES_MLI alphabetically. - daemon/listfs.ml: Remove comment about has_vxvm. - do_vxfs_start -> vxfs_start (since do_* is reserved prefix in daemon). - Skip vxfs_start if !vxdisk available, to save start up time. - Add a prototype for vxfs_start. - Don't use reply_with_error in vxfs_start, as we're not called during protocol context. - Use only one *err. - Add daemon/list_vxvm.mli & daemon/vxvm_type.mli to .gitignore vxfs_start has several issues which need to be addressed: - Why does it return if a service (eg. veki) is not available? That would prevent all further initialization. - I think this would be better done as part of appliance/init, and delete this function completely. Rich. -- Richard Jones, Virtualization Group, Red Hat https://nam12.safelinks.protection.outlook.com/?url=http%3A%2F%2Fpeople.redhat.com%2F~rjones&data=05%7C01%7Cravi.singh%40veritas.com%7C6df598eb4c58442388ef08dbb8ffa410%7Cfc8e13c0422c4c55b3eaca318e6cac32%7C0%7C0%7C638307180357901926%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=oNbYsYG%2B0kJX%2FSUyTgba10nH8762rEvcEWqN1n4tlHs%3D&reserved=0<http://people.redhat.com/~rjones> Read my programming and virtualization blog: https://nam12.safelinks.protection.outlook.com/?url=http%3A%2F%2Frwmj.wordpress.com%2F&data=05%7C01%7Cravi.singh%40veritas.com%7C6df598eb4c58442388ef08dbb8ffa410%7Cfc8e13c0422c4c55b3eaca318e6cac32%7C0%7C0%7C638307180357901926%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=LP%2BLKvr3i%2Bc%2BTbIln8uIdakIrWhbYyOW8reELLnIvqE%3D&reserved=0<http://rwmj.wordpress.com/> libguestfs lets you edit virtual machines. Supports shell scripting, bindings from many languages. https://nam12.safelinks.protection.outlook.com/?url=http%3A%2F%2Flibguestfs.org%2F&data=05%7C01%7Cravi.singh%40veritas.com%7C6df598eb4c58442388ef08dbb8ffa410%7Cfc8e13c0422c4c55b3eaca318e6cac32%7C0%7C0%7C638307180357901926%7CUnknown%7CTWFpbGZsb3d8eyJWIjoiMC4wLjAwMDAiLCJQIjoiV2luMzIiLCJBTiI6Ik1haWwiLCJXVCI6Mn0%3D%7C3000%7C%7C%7C&sdata=IFYqpYgPgyXOcW%2BLHkKww9nIwPm4oiOiyjrnw3IpT5o%3D&reserved=0<http://libguestfs.org/>
_______________________________________________ Libguestfs mailing list -- [email protected] To unsubscribe send an email to [email protected]
