JoaoJandre commented on PR #7131:
URL: https://github.com/apache/cloudstack/pull/7131#issuecomment-1697756005

   I looked through the errors on the tests, and none of the errors seem 
related to this PR:
   
   test_vm_lifecycle_unmanage_import.py:
   ```
   marvin.cloudstackException.CloudstackAPIException: Execute cmd: 
deployvirtualmachine failed, due to: errorCode: 431, errorText:Unable to deploy 
VM as UserData is provided while deploying the VM, but there is no support for 
UserData service in the default network Test L2 
Network/1e22f7d0-a41b-4c89-b50f-8b688f8b52e3.\n']
   ```
   
   test_volumes.py:
   ```
   AssertionError: 'FAILED' != 'SUCCESS'\n- FAILED\n+ SUCCESS\n : b'Check if 
promised disk size actually available'\n"
   ```
   
   test_host_maintenance.py:
   ```
   deployvirtualmachine failed, due to: errorCode: 431, errorText:Unable to 
deploy VM as UserData is provided while deploying the VM, but there is no 
support for UserData service in the default network Test L2 
Network/7d554e3b-a2d6-4bc5-ab72-e82374555059.\n']
   ```
   
   test_metrics_api.py:
   ```
   errorcode : 530, errortext : 'Unable to start a VM 
[9db40464-835e-4e9a-8494-e95b98f9b3b6] due to [Unable to create a deployment 
for VM instance 
{"id":255,"instanceName":"i-2-255-VM","type":"User","uuid":"9db40464-835e-4e9a-8494-e95b98f9b3b6"
   ```
   
   test_vm_autoscaling.py:
   ```
   marvin.cloudstackException.CloudstackAPIException: Execute cmd: 
autoscalevmprofile failed, due to: errorCode: 530, errorText:Internal error 
executing command, please contact your system administrator\n'
   ```
   
   test_vm_schedule.py:
   ```
    'AssertionError: True is not false : VM did not switch states after 
schedule time\n'
   ```
   
   test_host_maintenance.py:
   ```
   Execute cmd: deployvirtualmachine failed, due to: errorCode: 431, 
errorText:Unable to deploy VM as UserData is provided while deploying the VM, 
but there is no support for UserData service in the default network Test L2 
Network/00c6366e-8af5-47c8-82ee-73ead3e28c1c.\n']
   ```
   
   test_hostha_kvm.py:
   ```
   Exception: Template download failed exception
   ```
   
   
   About #7912, it might fix the CI problem (whether it's the best approach or 
not, we can discuss there). However, I don't think that this PR should depend 
on #7912, everything is already working and delaying it is very costly, I have 
already spend hours and hours fixing conflicts. This PR has been ready for a 
long time, it has passed all the projects requirements to be accepted. If the 
CI implemented by Trillian is calling inner packages of the application 
directly, it should adapt to changes in dependencies, not the other way around.


-- 
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: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]

Reply via email to