Skip to content

travis: Fix failing list vms & snapshots tests#6124

Closed
Pearl1594 wants to merge 1 commit into
apache:mainfrom
shapeblue:fix-list-resources-tests
Closed

travis: Fix failing list vms & snapshots tests#6124
Pearl1594 wants to merge 1 commit into
apache:mainfrom
shapeblue:fix-list-resources-tests

Conversation

@Pearl1594

@Pearl1594 Pearl1594 commented Mar 16, 2022

Copy link
Copy Markdown
Contributor

Description

This PR fixes issues in the number of resources returned (VMs/Snapshots) for the corresponding list commands. This pertains to the change in behavior with listall & isrecusive (#6045).

Types of changes

  • Breaking change (fix or feature that would cause existing functionality to change)
  • New feature (non-breaking change which adds functionality)
  • Bug fix (non-breaking change which fixes an issue)
  • Enhancement (improves an existing feature and functionality)
  • Cleanup (Code refactoring and cleanup, that may add test cases)

Feature/Enhancement Scale or Bug Severity

Bug Severity

  • BLOCKER
  • Critical
  • Major
  • Minor
  • Trivial

Screenshots (if appropriate):

How Has This Been Tested?

nosetests --with-xunit --xunit-file=results.xml --with-marvin --marvin-config=setup/dev/advanced.cfg  -s -a tags=xx --hypervisor=simulator test/integration/component/test_acl_listsnapshot.py 

==== Marvin Init Started ====

=== Marvin Parse Config Successful ===

=== Marvin Setting TestData Successful===

==== Log Folder Path: /tmp/MarvinLogs/Mar_15_2022_15_55_59_V8ZQH8. All logs will be available here ====

=== Marvin Init Logging Successful===

==== Marvin Init Successful ====
=== TestName: test_listSnapshot_as_domainadmin_domainid_listall_true | Status : SUCCESS ===

=== TestName: test_listSnapshot_as_domainadmin_listall_true_rec_false | Status : SUCCESS ===




[16:46:11] pearl@pearl-XPS-15-7590:~/lab/shapeblue/cloudstack$ nosetests --with-xunit --xunit-file=results.xml --with-marvin --marvin-config=setup/dev/advanced.cfg  -s -a tags=xx --hypervisor=simulator test/integration/component/test_acl_listvm.py

==== Marvin Init Started ====

=== Marvin Parse Config Successful ===

=== Marvin Setting TestData Successful===

==== Log Folder Path: /tmp/MarvinLogs/Mar_15_2022_16_49_53_7T5J5R. All logs will be available here ====

=== Marvin Init Logging Successful===

==== Marvin Init Successful ====
=== TestName: test_listVM_as_domainadmin_domainid_listall_true | Status : SUCCESS ===

=== TestName: test_listVM_as_domainadmin_listall_true_rec_false | Status : SUCCESS ===

=== TestName: test_listVM_as_rootadmin_domainid_listall_true | Status : SUCCESS ===




$ nosetests --with-xunit --xunit-file=results.xml --with-marvin --marvin-config=setup/dev/advanced.cfg  -s -a tags=xx --hypervisor=simulator test/integration/component/test_acl_listvolume.py 

==== Marvin Init Started ====

=== Marvin Parse Config Successful ===

=== Marvin Setting TestData Successful===

==== Log Folder Path: /tmp/MarvinLogs/Mar_16_2022_17_54_29_RSQTC1. All logs will be available here ====

=== Marvin Init Logging Successful===

==== Marvin Init Successful ====
=== TestName: test_listVolume_as_domainadmin_domainid_listall_true | Status : SUCCESS ===

=== TestName: test_listVolume_as_domainadmin_listall_true_rec_false | Status : SUCCESS ===

=== TestName: test_listVolume_as_rootadmin_domainid_listall_true | Status : SUCCESS ===

===final results are now copied to: /tmp/MarvinLogs/test_acl_listvolume_3J1DO0===


@Pearl1594

Copy link
Copy Markdown
Contributor Author

@blueorangutan package

@blueorangutan

Copy link
Copy Markdown

@Pearl1594 a Jenkins job has been kicked to build packages. It will be bundled with KVM, XenServer and VMware SystemVM templates. I'll keep you posted as I make progress.

@blueorangutan

Copy link
Copy Markdown

Packaging result: ✔️ el7 ✔️ el8 ✔️ debian ✔️ suse15. SL-JID 2902

@nvazquez

Copy link
Copy Markdown
Contributor

@blueorangutan test

@blueorangutan

Copy link
Copy Markdown

@nvazquez a Trillian-Jenkins test job (centos7 mgmt + kvm-centos7) has been kicked to run smoke tests

@Pearl1594 Pearl1594 force-pushed the fix-list-resources-tests branch from 7297453 to 528110b Compare March 16, 2022 12:36
@nvazquez

Copy link
Copy Markdown
Contributor

Closing in favour of #6126

@nvazquez nvazquez closed this Mar 16, 2022
@yadvr yadvr removed this from the 4.17.0.0 milestone Mar 24, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

No open projects
Status: Done

Development

Successfully merging this pull request may close these issues.

4 participants