Skip to content

fix test_data for vapps tests#128

Merged
yadvr merged 1 commit intosofia-labfrom
fix-vapps-test-data
Apr 9, 2021
Merged

fix test_data for vapps tests#128
yadvr merged 1 commit intosofia-labfrom
fix-vapps-test-data

Conversation

@shwstppr
Copy link
Member

Adds test data for vApps tests (test_vm_life_cycle.py).

@shwstppr
Copy link
Member Author

@nvazquez vApps tests in test_vm_life_cycle.py were failing on Trillian as test_data was not having the required details.

2021-03-10 01:17:30,654 - CRITICAL - EXCEPTION: test_01_unmanage_vm_cycle: ['Traceback (most recent call last):\n', '  File "/usr/lib/python2.7/site-packages/nose/suite.py", line 209, in run\n    self.setUp()\n', '  File "/usr/lib/python2.7/site-packages/nose/suite.py", line 292, in setUp\n    self.setupContext(ancestor)\n', '  File "/usr/lib/python2.7/site-packages/nose/suite.py", line 315, in setupContext\n    try_run(context, names)\n', '  File "/usr/lib/python2.7/site-packages/nose/util.py", line 471, in try_run\n    return func()\n', '  File "/marvin/tests/smoke/test_vm_life_cycle.py", line 1752, in setUpClass\n    cls.services[\'test_ovf_templates\'],\n', "KeyError: 'test_ovf_templates'\n"]

I added the changes and have copied http://10.2.0.4/marvin/machina-2dd-iso.ova from http://10.5.1.128/templates/machina-2dd-iso.ova.

2021-03-10 08:28:35,071 - CRITICAL - EXCEPTION: None: ['Traceback (most recent call last):\n', '  File "/usr/lib/python2.7/site-packages/nose/suite.py", line 209, in run\n    self.setUp()\n', '  File "/usr/lib/python2.7/site-packages/nose/suite.py", line 292, in setUp\n    self.setupContext(ancestor)\n', '  File "/usr/lib/python2.7/site-packages/nose/suite.py", line 315, in setupContext\n    try_run(context, names)\n', '  File "/usr/lib/python2.7/site-packages/nose/util.py", line 471, in try_run\n    return func()\n', '  File "/marvin/tests/smoke/atest.py", line 97, in setUpClass\n    assert False, "get_test_ovf_templates() failed to return templates"\n', 'AssertionError: get_test_ovf_templates() failed to return templates\n']

Can you please check if the OVF template is correct or do we need to change get_test_ovf_templates method.

PS: Before bf597e5 Trillian was having a different version of test_vm_life_cycle.py

@yadvr
Copy link
Member

yadvr commented Mar 24, 2021

@shwstppr is this ready for merging? cc @Pearl1594 @davidjumani

@shwstppr
Copy link
Member Author

@rhtyd I think issue is not there on Sofia lab. I'll check and reopen (with changes) if needed.

@shwstppr shwstppr closed this Mar 24, 2021
@shwstppr
Copy link
Member Author

Verified issue is still there.
While this fixes the failing tests, @nvazquez need to verify template to be used, https://github.com/shapeblue/Trillian/pull/128/files#diff-27fd2cc7d23dbc532fb65312275e4dbb61bec62418aa849ecaa82296f2d40c51R1020

@shwstppr shwstppr reopened this Mar 24, 2021
@shwstppr shwstppr changed the base branch from master to sofia-lab March 24, 2021 07:17
@shwstppr shwstppr changed the base branch from sofia-lab to master March 24, 2021 07:17
Signed-off-by: Abhishek Kumar <abhishek.mrt22@gmail.com>
@shwstppr shwstppr force-pushed the fix-vapps-test-data branch from bb00d42 to 7af0e95 Compare March 24, 2021 07:20
@yadvr
Copy link
Member

yadvr commented Mar 24, 2021

@shwstppr can you check your branch - does it need rebasing, there are many additional commits than your change

@shwstppr shwstppr changed the base branch from master to sofia-lab March 24, 2021 07:30
@shwstppr shwstppr marked this pull request as ready for review April 7, 2021 11:43
@nvazquez
Copy link
Member

nvazquez commented Apr 7, 2021

Thanks @shwstppr, I've copied the .ova from '10.0.3.122/other_templates' to '10.0.3.122/marvin/' and placed your code on a marvin box '/marvin/test_data.py' file. Needed to fix the test as well, will be adding the code as well.

Test the following for all found ovf templates: ... === TestName: test_01_vapps_vm_cycle | Status : SUCCESS ===
ok

@nvazquez nvazquez closed this Apr 7, 2021
@nvazquez
Copy link
Member

nvazquez commented Apr 7, 2021

PR containing the marvin tests changes: apache/cloudstack#4738

@nvazquez nvazquez reopened this Apr 7, 2021
@nvazquez
Copy link
Member

nvazquez commented Apr 8, 2021

@shwstppr we might consider rebasing the branch vmware-sofia-lab instead of sofia-lab

@yadvr yadvr merged commit 3ae11ec into sofia-lab Apr 9, 2021
@DaanHoogland DaanHoogland deleted the fix-vapps-test-data branch December 29, 2022 08:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants