Skip to content

fix test_data for vapps tests#128

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

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

Conversation

@shwstppr

Copy link
Copy Markdown
Member

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

@shwstppr

Copy link
Copy Markdown
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

yadvr commented Mar 24, 2021

Copy link
Copy Markdown
Contributor

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

@shwstppr

Copy link
Copy Markdown
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
Copy Markdown
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

yadvr commented Mar 24, 2021

Copy link
Copy Markdown
Contributor

@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

nvazquez commented Apr 7, 2021

Copy link
Copy Markdown
Member

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

nvazquez commented Apr 7, 2021

Copy link
Copy Markdown
Member

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

@nvazquez nvazquez reopened this Apr 7, 2021
@nvazquez

nvazquez commented Apr 8, 2021

Copy link
Copy Markdown
Member

@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