winterhazel commented on PR #12124:
URL: https://github.com/apache/cloudstack/pull/12124#issuecomment-4731162052

   > Hello @winterhazel @DaanHoogland @Pearl1594
   > 
   > The test result looks good.
   > 
   > > [SF] Trillian test result (tid-16299) Environment: kvm-ol8 (x2), zone: 
Advanced Networking with Mgmt server ol8 Total time taken: 49044 seconds Marvin 
logs: 
https://github.com/blueorangutan/acs-prs/releases/download/trillian/pr12124-t16299-kvm-ol8.zip
 Smoke tests completed. 151 look OK, 0 have errors, 0 did not run Only failed 
and skipped tests results shown below:
   > > Test     Result  Time (s)        Test File
   > 
   > i am not sure how to resolve these two :/ <img alt="image" width="908" 
height="144" 
src="https://private-user-images.githubusercontent.com/96632094/608368571-6d20cac9-4076-4424-b27c-c7acb80081a0.png?jwt=eyJ0eXAiOiJKV1QiLCJhbGciOiJIUzI1NiJ9.eyJpc3MiOiJnaXRodWIuY29tIiwiYXVkIjoicmF3LmdpdGh1YnVzZXJjb250ZW50LmNvbSIsImtleSI6ImtleTUiLCJleHAiOjE3ODE3MDI0NjMsIm5iZiI6MTc4MTcwMjE2MywicGF0aCI6Ii85NjYzMjA5NC82MDgzNjg1NzEtNmQyMGNhYzktNDA3Ni00NDI0LWIyN2MtYzdhY2I4MDA4MWEwLnBuZz9YLUFtei1BbGdvcml0aG09QVdTNC1ITUFDLVNIQTI1NiZYLUFtei1DcmVkZW50aWFsPUFLSUFWQ09EWUxTQTUzUFFLNFpBJTJGMjAyNjA2MTclMkZ1cy1lYXN0LTElMkZzMyUyRmF3czRfcmVxdWVzdCZYLUFtei1EYXRlPTIwMjYwNjE3VDEzMTYwM1omWC1BbXotRXhwaXJlcz0zMDAmWC1BbXotU2lnbmF0dXJlPTIyNjc5MmQ2OTcyZjZlMGQyYTcwNmNhYWZjYWZhMzljZmM2Mzk5OTZkZGJkZjMwNmY2MjNlNTdiNjBkODcxZjAmWC1BbXotU2lnbmVkSGVhZGVycz1ob3N0JnJlc3BvbnNlLWNvbnRlbnQtdHlwZT1pbWFnZSUyRnBuZyJ9.9cy436sK7OCq-DwM3-uH4ZmBOCWHVHdfjyCzL11JDGs";>
   
   @mhkadhum the first failure is caused by the lack of unit tests (coverage 
under 18%) for these changes. Passing this check is not a requirement for 
merging, but of course it would be good to have tests to prevent future 
regressions, if you would like to add some. The second one does not seem 
related to your changes, but I triggered a rerun.


-- 
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