Bug 1463263 - remove ISO domain creation from OST
Summary: remove ISO domain creation from OST
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: ovirt-system-tests
Classification: Community
Component: General
Version: 2.0
Hardware: x86_64
OS: Linux
unspecified
medium
Target Milestone: ---
: ---
Assignee: Dafna Ron
QA Contact: Pavel Stehlik
URL:
Whiteboard:
Depends On:
Blocks:
TreeView+ depends on / blocked
 
Reported: 2017-06-20 13:03 UTC by Dafna Ron
Modified: 2018-03-20 15:31 UTC (History)
4 users (show)

Fixed In Version:
Doc Type: If docs needed, set a value
Doc Text:
Clone Of:
Environment:
Last Closed: 2018-03-20 15:31:03 UTC
oVirt Team: ---
Embargoed:
dron: planning_ack?
dron: devel_ack?


Attachments (Terms of Use)


Links
System ID Private Priority Status Summary Last Updated
oVirt gerrit 79329 0 None None None 2017-07-12 12:33:04 UTC
oVirt gerrit 79342 0 None None None 2017-07-12 14:48:08 UTC

Description Dafna Ron 2017-06-20 13:03:31 UTC
Description of problem:

There's a test that gets stuck on mount to iso domain. 

We have an open bug on this, the mount to the ISO domain gets stuck and never returns:
call mount with (u'192.168.201.3:/exports/nfs/iso', u'/rhev/data-center/mnt/192.168.201.3:_exports_nfs_iso') {'vfstype': 'nfs', 'mntOpts': 'soft,nosharecache,timeo=600,retrans=6,nfsvers=3', 'timeout': None, 'cgroup': None}


since we no longer actually use the iso domain so we should remove this domain from our tests. 

link to failed job: http://jenkins.ovirt.org/job/test-repo_ovirt_experimental_master/7264

Comment 4 Yaniv Kaul 2017-07-11 01:00:21 UTC
I'd begin with skip.

Comment 5 Dafna Ron 2017-07-11 18:50:35 UTC
If I am reading the code correctly, the add secondary domain test is creating more than just the iso domain:

http://pastebin.test.redhat.com/502050

I can't skip the test by adding raise SkipTest without stopping creating the rest of the domains. 

I can comment out the functools.partial(add_iso_storage_domain, prefix), which would disable it but I am not sure this is the correct solution 

any suggestions?

Comment 6 Eyal Edri 2017-07-12 06:43:43 UTC
Can we split the test into multiple tests, so each storage domain would be added it its own test? that way we'll only disable the adding of iso domain?

Comment 7 Yaniv Kaul 2017-07-12 09:32:45 UTC
(In reply to Eyal Edri from comment #6)
> Can we split the test into multiple tests, so each storage domain would be
> added it its own test? that way we'll only disable the adding of iso domain?

We can, but it is slow. Just comment out the call.

Comment 8 Dafna Ron 2017-07-12 11:37:31 UTC
https://gerrit.ovirt.org/#/c/79329/

Comment 9 Eyal Edri 2017-07-12 12:20:04 UTC
Dafna, we use the 'external tracker' to attached oVirt patches to bugs.
Can you use the 'oVirt Gerrit' tracker and attach the fix to the bug?


Note You need to log in before you can comment on or make changes to this bug.