Re: [PR] conditional import in case we are using xcpng 8.3 [cloudstack]

["prashanthr2 (via GitHub)" <gi...@apache.org>]

prashanthr2 commented on PR #13692:
URL: https://github.com/apache/cloudstack/pull/13692#issuecomment-5293118620

   @DaanHoogland @Pearl1594 
   
   The latest commit has fixed the issue with SystemVM's with fully patched xcp-ng 8.3
   
   However, we see certain operations failing
   
   1. Creating template from volume
   2. Create template from snapshot.
   
   Failure seen in the xensource.log on xcp-ng is below
   
   ```
   Aug 14 12:00:38 ref-trl-12214-x-mol9-prashanth-reddy-xs1 xapi: [error||6364 :::80|dispatch:host.call_plugin D:eb5bec0c5ac1|backtrace] host.call_plugin R:c6658c7122b8 failed with exception Server_error(XENAPI_PLUGIN_FAILURE, [ non-zero exit; ; Traceback (most recent call last):\x0A  File "/etc/xapi.d/plugins/vmopsSnapshot", line 34, in <module>\x0A    import lvhdutil\x0AModuleNotFoundError: No module named 'lvhdutil'\x0A ])
   Aug 14 12:00:38 ref-trl-12214-x-mol9-prashanth-reddy-xs1 xapi: [error||6364 :::80|dispatch:host.call_plugin D:eb5bec0c5ac1|backtrace] Raised Server_error(XENAPI_PLUGIN_FAILURE, [ non-zero exit; ; Traceback (most recent call last):\x0A  File "/etc/xapi.d/plugins/vmopsSnapshot", line 34, in <module>\x0A    import lvhdutil\x0AModuleNotFoundError: No module named 'lvhdutil'\x0A ])
   
   ```
   
   I think we need to have guard fix in vmopsSnapshot as well.
   
   
   Note: this is only fix for xcp-ng 8.3 with SM module reverted. For future versions of xcp-ng ( where lvhdutil is removed), we need fix as suggested here https://github.com/apache/cloudstack/issues/13292#issuecomment-4683013183


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: commits-unsubscribe@cloudstack.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org