Bug 1313588

Summary:	ovirt host broke local iscsi server
Product:	[oVirt] vdsm	Reporter:	Badalyan Vyacheslav <v.badalyan>
Component:	Services	Assignee:	Nir Soffer <nsoffer>
Status:	CLOSED CURRENTRELEASE	QA Contact:
Severity:	medium	Docs Contact:
Priority:	unspecified
Version:	4.18.0	CC:	amureini, bugs, eedri, nsoffer, v.badalyan, ylavi
Target Milestone:	ovirt-4.1.1	Flags:	rule-engine: ovirt-4.1+ rule-engine: ovirt-4.2+
Target Release:	4.19.6
Hardware:	x86_64
OS:	Linux
Whiteboard:
Fixed In Version:		Doc Type:	Bug Fix
Doc Text:		Story Points:	---
Clone Of:		Environment:
Last Closed:	2017-03-19 14:56:44 UTC	Type:	Bug
Regression:	---	Mount Type:	---
Documentation:	---	CRM:
Verified Versions:		Category:	---
oVirt Team:	Storage	RHEL 7.3 requirements from Atomic Host:
Cloudforms Team:	---	Target Upstream Version:
Embargoed:
Bug Depends On:	1374545
Bug Blocks:

Description Badalyan Vyacheslav 2016-03-02 00:51:24 UTC

Description of problem:

If we have iscsi and host on one server we get confuguration issue after reload.

How reproducible:
Alwoays

Steps to Reproduce:
1. Create LVM partition
2. Add it to targetcli
3. install ovirt host
4. add iscsi to storagedomain
5. Reoot host

Actual results:

vsdm create LVM partition in iscsi that allready LVM partition.
After reboot LVM see recurced partition throuth multitarget and DM (iscis->LVM->PV-VG->DomainLVM) 
Targertcli can't accesss to it becouse Device is Busy.

You may add filer to LVM and targetcli and also delete some DM devices, but with filter vsdm dont see domain after reboot.

Expected results:
All must work normal. All small companyes have many services in one server.

Comment 1 Badalyan Vyacheslav 2016-03-02 00:52:17 UTC

*** Bug 1313587 has been marked as a duplicate of this bug. ***

Comment 2 Nir Soffer 2016-03-02 09:09:20 UTC

(In reply to Badalyan Vyacheslav from comment #0)
> Description of problem:
> 
> If we have iscsi and host on one server we get confuguration issue after
> reload.
> 
> How reproducible:
> Alwoays
> 
> Steps to Reproduce:
> 1. Create LVM partition

What do you mean?

Pleas share the output of lsblk and fdisk (list partition table) before
installing the host and after it.

> 2. Add it to targetcli
> 3. install ovirt host
> 4. add iscsi to storagedomain

Did you add iscsi storage domain using a device used by targetcli?

> 5. Reoot host
> 
> Actual results:
> 
> vsdm create LVM partition in iscsi that allready LVM partition.

Vdsm does not create LVM partitions. Vdsm create a VG from the multipath
devices you selected in engine UI.

If you selected a device which is already used, Engine will warn you
and you have to confirm the operation. If you confirmed, you cannot
complain that ovirt did what you asked for :-)

Please also attach engine and vdsm logs showing the time when you created this
domain.

Comment 3 Badalyan Vyacheslav 2016-03-13 17:13:17 UTC

I remove iscsi and go to NFS.

1. I was have VG - centos and LV iscsi
2. I add LV iscsi to targetcli as backpain.
3. I conect Engine to iscisi. 
4. Engine create VG in iscisi.
5. I reboot
6. LVM or multitarget or dm found VG-LV-VG-LV
7. targetcli can't start iscsi becouse "device is busy"
8. Result - Storage is DOWN.

I also try don't create LVM and use "clear" partition. Result same! If iscsi and host its one HOST, then VG partitions found before targetcli. Result - "device is busy" and iscsi is broken. 

It's very popular case use Storage in host wehere work VM. I have raid in SSD with 4GB/s speed and bonding with 4 e1000 can't use all perfomance. Bandwidth, MTU and many other things.

Why you create LVM on iscsi? Many chip Storages use linux and targetcli and may have some issue. Why you don't create storage domain like in NFS in file mode? Why need create VG and LV? Using LVM will broke all iscsi that use LVM or Device Partition.

Comment 4 Yaniv Kaul 2016-11-21 11:03:33 UTC

Nir - again another auto-activation variant? Sounds quite similar to what Lev had in ovirt-system-tests, btw.

Comment 5 Nir Soffer 2017-02-14 11:36:57 UTC

We disabled lvmetad service, so this may work for you now.

Can you test with current 4.1.1 nightly?

Comment 6 Nir Soffer 2017-02-27 15:13:51 UTC

We believe this issue should be solved in 4.1.1, since we disabled lvmetad.

Comment 7 Eyal Edri 2017-03-19 08:07:49 UTC

No patches attached to the bug, there is not way for the bot to know if this is in a build or not.

please make sure to attach the relevant patches from gerrit.

Comment 8 Allon Mureinik 2017-03-19 08:51:49 UTC

(In reply to Eyal Edri from comment #7)
> No patches attached to the bug, there is not way for the bot to know if this
> is in a build or not.
> 
> please make sure to attach the relevant patches from gerrit.

This seems to have been fixed by https://gerrit.ovirt.org/#/c/71674/, which is part of 4.19.6.
Moving to ON_QA

Comment 9 Yaniv Lavi 2017-03-19 14:56:44 UTC

This should be fixed in 4.1.1, please reopen if you still have a issue.