Thursday, 2018-06-07

openstackgerritjunbo.li proposed openstack/cinder master: Add debug logs for weigher filters  https://review.openstack.org/57112400:10
*** bigdogstl has joined #openstack-cinder00:15
*** kien-ha has joined #openstack-cinder00:17
*** itlinux has joined #openstack-cinder00:20
*** bigdogstl has quit IRC00:23
*** threestrands has quit IRC00:25
*** jmlowe has quit IRC00:26
*** jmlowe has joined #openstack-cinder00:27
*** threestrands has joined #openstack-cinder00:28
*** wanghao has joined #openstack-cinder00:34
openstackgerritjunbo.li proposed openstack/python-cinderclient master: Use api version 3 for functional test  https://review.openstack.org/48366200:39
openstackgerritDongcan Ye proposed openstack/cinder master: api-ref: Fix some params in volume types  https://review.openstack.org/57276200:54
*** itlinux has quit IRC00:58
*** wanghao_ has joined #openstack-cinder01:00
*** liuzz has joined #openstack-cinder01:00
*** itlinux has joined #openstack-cinder01:00
*** jiaopengju has joined #openstack-cinder01:01
*** Nel1x has joined #openstack-cinder01:02
*** jiaopengju has quit IRC01:02
*** jiaopengju has joined #openstack-cinder01:02
*** wanghao has quit IRC01:02
*** zhaochao has joined #openstack-cinder01:02
*** jiaopengju has quit IRC01:02
*** jiaopengju has joined #openstack-cinder01:03
*** jiaopengju has quit IRC01:04
*** jiaope___ has joined #openstack-cinder01:04
*** Dinesh_Bhor has joined #openstack-cinder01:04
*** wanghao has joined #openstack-cinder01:05
*** wanghao_ has quit IRC01:09
*** felipemonteiro has joined #openstack-cinder01:12
*** felipemonteiro_ has joined #openstack-cinder01:14
*** felipemonteiro has quit IRC01:18
*** felipemonteiro_ has quit IRC01:24
*** dalgaaf has quit IRC01:27
*** dalgaaf has joined #openstack-cinder01:28
*** threestrands has quit IRC01:35
*** threestrands has joined #openstack-cinder01:35
*** threestrands has quit IRC01:36
*** threestrands has joined #openstack-cinder01:37
*** threestrands has quit IRC01:38
*** threestrands has joined #openstack-cinder01:38
*** threestrands has quit IRC01:38
*** threestrands has joined #openstack-cinder01:38
*** threestrands has quit IRC01:39
*** threestrands has joined #openstack-cinder01:40
*** threestrands has quit IRC01:40
*** threestrands has joined #openstack-cinder01:40
*** threestrands has quit IRC01:41
*** edmondsw has joined #openstack-cinder01:42
*** kaisers has joined #openstack-cinder01:47
*** edmondsw has quit IRC01:47
*** dpawlik has joined #openstack-cinder01:49
*** kaisers has quit IRC01:53
*** dpawlik has quit IRC01:53
*** wanghao has quit IRC01:53
*** r-daneel has joined #openstack-cinder01:56
*** jiaope___ has quit IRC02:10
*** threestrands has joined #openstack-cinder02:11
*** threestrands has quit IRC02:11
*** threestrands has joined #openstack-cinder02:11
*** jiaopengju has joined #openstack-cinder02:13
*** mriedem has quit IRC02:52
*** psachin has joined #openstack-cinder02:52
*** wanghao has joined #openstack-cinder02:53
*** jiaopengju has quit IRC02:57
*** markstur has joined #openstack-cinder02:57
*** jiaopeng_ has joined #openstack-cinder02:59
*** lifeless_ has joined #openstack-cinder03:03
*** BrinZhang has joined #openstack-cinder03:03
*** lifeless has quit IRC03:03
*** jiaopeng_ has quit IRC03:04
*** boris_42_ has quit IRC03:05
*** kien-ha has quit IRC03:05
*** threestrands has quit IRC03:13
*** markstur has quit IRC03:22
*** Nel1x has quit IRC03:26
*** edmondsw has joined #openstack-cinder03:30
*** edmondsw has quit IRC03:35
*** wanghao has quit IRC03:40
*** lpetrut has joined #openstack-cinder03:42
*** prometheanfire has left #openstack-cinder03:47
*** kaisers has joined #openstack-cinder03:48
*** dpawlik has joined #openstack-cinder03:49
*** kaisers has quit IRC03:53
*** dpawlik has quit IRC03:54
*** lpetrut has quit IRC04:14
*** Nel1x has joined #openstack-cinder04:18
*** Nel1x has quit IRC04:25
*** neha_alhat has joined #openstack-cinder04:27
*** links has joined #openstack-cinder04:28
neha_alhattommylikehu: Hi, In requirements project upper-constraints for keystoneauth1 is 3.7.0 [1] and lower-contraints is 3.4.0 [2]. [1]: https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L47104:32
neha_alhattommylikehu: [2]: https://github.com/openstack/requirements/blob/master/lower-constraints.txt#L47204:32
neha_alhattommylikehu: Currently for cinder in requirements.txt release version for keystoneauth1 is >=3.4.0. Need to change this to latest release of keystoneauth1 i.e. 3.7.0. For that what can be done?04:34
*** markstur has joined #openstack-cinder04:48
*** markstur has quit IRC04:53
*** mvk has joined #openstack-cinder04:54
*** Dinesh_Bhor has quit IRC05:00
*** wanghao has joined #openstack-cinder05:06
*** chhagarw has joined #openstack-cinder05:06
*** pgadiya has joined #openstack-cinder05:08
*** pgadiya has quit IRC05:08
*** kaisers has joined #openstack-cinder05:13
*** edmondsw has joined #openstack-cinder05:19
*** vivsoni has quit IRC05:19
*** vivsoni has joined #openstack-cinder05:19
*** e0ne has joined #openstack-cinder05:23
*** edmondsw has quit IRC05:23
*** e0ne has quit IRC05:25
*** markstur has joined #openstack-cinder05:27
*** markstur has quit IRC05:27
*** Dinesh_Bhor has joined #openstack-cinder05:30
openstackgerritVishakha Agarwal proposed openstack/cinder master: Cloning image fails results duplicate cache entry  https://review.openstack.org/56747305:32
*** neha_alhat has quit IRC05:49
*** dpawlik has joined #openstack-cinder05:50
*** Luzi has joined #openstack-cinder05:51
*** Dinesh__Bhor has joined #openstack-cinder05:52
*** Dinesh_Bhor has quit IRC05:53
*** abishop has joined #openstack-cinder05:53
*** dpawlik has quit IRC05:55
*** hamdyk has joined #openstack-cinder06:05
*** lvdombrkr has joined #openstack-cinder06:05
*** whoami-rajat__ has joined #openstack-cinder06:09
*** sar has joined #openstack-cinder06:10
*** whoami-rajat_ has quit IRC06:12
openstackgerritPooja Jadhav proposed openstack/cinder master: V3 jsonschema validation: Volumes  https://review.openstack.org/57309306:15
*** Dinesh__Bhor has quit IRC06:23
*** lpetrut has joined #openstack-cinder06:23
*** Dinesh_Bhor has joined #openstack-cinder06:26
*** bigdogstl has joined #openstack-cinder06:29
*** dklyle has quit IRC06:29
*** namnh has joined #openstack-cinder06:31
*** e0ne has joined #openstack-cinder06:31
*** bigdogstl has quit IRC06:33
*** pcaruana has joined #openstack-cinder06:35
*** abishop has quit IRC06:35
*** abhishekk has joined #openstack-cinder06:36
*** abhishekk is now known as akekane|sick06:36
*** akekane|sick is now known as akekane|away06:37
*** akekane|away is now known as abhishekk|away06:37
*** e0ne has quit IRC06:41
openstackgerritYong Huang proposed openstack/cinder master: Empty option value maybe cause Unity driver failed to initialize  https://review.openstack.org/57309906:43
openstackgerritYong Huang proposed openstack/cinder master: Empty option value maybe cause Unity driver failed to initialize  https://review.openstack.org/57309906:58
*** kaisers_ has joined #openstack-cinder07:02
*** AlexeyAbashkin has joined #openstack-cinder07:04
*** moshele has joined #openstack-cinder07:04
*** kaisers has quit IRC07:06
*** edmondsw has joined #openstack-cinder07:07
*** gkadam has joined #openstack-cinder07:07
*** e0ne has joined #openstack-cinder07:10
*** armaan has joined #openstack-cinder07:10
*** e0ne has quit IRC07:11
*** edmondsw has quit IRC07:11
tommylikehuhey neha_alhat please read the section: Updating the minimum version of a dependency¶07:13
tommylikehu here https://docs.openstack.org/project-team-guide/dependency-management.html07:13
tommylikehuit looks like you need to update our local requirements.txt as well as lower-constraints.txt file07:14
*** jistr|mtgs is now known as jistr07:22
*** Dinesh_Bhor has quit IRC07:26
*** sar has quit IRC07:28
*** Dinesh_Bhor has joined #openstack-cinder07:31
*** dklyle has joined #openstack-cinder07:32
*** Alexey_Abashkin has joined #openstack-cinder07:34
*** AlexeyAbashkin has quit IRC07:35
*** Alexey_Abashkin is now known as AlexeyAbashkin07:35
*** lvdombrkr89 has joined #openstack-cinder07:43
*** lvdombrkr has quit IRC07:45
*** AlexeyAbashkin has quit IRC07:46
*** dpawlik has joined #openstack-cinder07:49
*** dpawlik has quit IRC07:54
wanghaotommylikehu: tommy, would you help to review this patch: https://review.openstack.org/#/c/533564/?07:55
*** AlexeyAbashkin has joined #openstack-cinder07:56
wanghaowe proposed to merge this in R.  really need more review.07:56
tommylikehuwanghao: hao!07:57
wanghaotommylikehu: thanks :)07:58
*** alexchadin has joined #openstack-cinder08:04
*** moshele has quit IRC08:09
*** moshele has joined #openstack-cinder08:10
*** AlexeyAbashkin has quit IRC08:23
*** dpawlik has joined #openstack-cinder08:25
*** AlexeyAbashkin has joined #openstack-cinder08:26
*** dpawlik has quit IRC08:30
*** moshele has quit IRC08:33
*** AlexeyAbashkin has quit IRC08:33
*** AlexeyAbashkin has joined #openstack-cinder08:38
*** AlexeyAbashkin has quit IRC08:45
tommylikehuping wanghao08:49
tommylikehufor your patch: https://review.openstack.org/#/c/533564/ here, if I understand you correctly, the biggest issue is we will have two sets of volume transfer APIs, one is volume_transfer and another is os-volume-transfer.08:51
tommylikehuwanghao: I personally would prefer to keep the old one and leave as it is:)08:51
wanghaotommylikehu: yes, now we will have old one and new one,   Sean suggested we should move old os-volume-transfer to cinder V3 api.08:53
wanghaotommylikehu: so I build the new old, and keep the old one for back-compatible08:53
wanghaonew old/new one08:54
*** edmondsw has joined #openstack-cinder08:55
tommylikehuwanghao:  thanks I will check the comment log08:55
openstackgerritNeha Alhat proposed openstack/cinder master: Update api-ref and release notes for schema validation  https://review.openstack.org/57313708:56
wanghaotommylikehu: sure :)08:59
*** edmondsw has quit IRC09:00
*** wanghao has quit IRC09:01
*** wanghao has joined #openstack-cinder09:01
*** wanghao has quit IRC09:01
*** wanghao has joined #openstack-cinder09:02
*** wanghao has quit IRC09:02
*** wanghao has joined #openstack-cinder09:02
*** wanghao has quit IRC09:03
*** wanghao has joined #openstack-cinder09:03
tommylikehusmcginnis, wanghao: if that so, I guess we need to update the api document to keep them both while mark the old ones as deprecated?09:05
wanghaotommylikehu: yeah, agree09:06
openstackgerritXiangfei Zhu proposed openstack/cinder master: VMware: Add file open mode to support python3  https://review.openstack.org/57314309:10
pooja-jadhavtommylikehu: Hi09:14
*** pooja-jadhav is now known as pooja_jadhav09:14
tommylikehupooja_jadhav:  hi09:15
pooja_jadhavtommylikehu: Most of the test cases failing from tempest like "tempest.api.volume.test_volumes_negative.VolumesNegativeTest.test_update_volume_with_nonexistent_volume_id" on my patch. because its passing empty body {} and after schema validation its failing.09:16
*** lifeless_ has quit IRC09:17
pooja_jadhavtommylikehu: As {} is invalid body for update volume, so should we fix tempest tests?09:17
*** lifeless has joined #openstack-cinder09:18
tommylikehupooja_jadhav:  yes, we should, the thing that matters is  where should we fix this, if it's a intended api behaviour change we should update the tempest, otherwise we should update our service.09:26
*** armaan has quit IRC09:27
*** armaan has joined #openstack-cinder09:28
pooja_jadhavtommylikehu: thanks :)09:28
*** moshele has joined #openstack-cinder09:30
*** AlexeyAbashkin has joined #openstack-cinder09:31
*** wanghao has quit IRC09:31
*** wanghao has joined #openstack-cinder09:32
*** wanghao has quit IRC09:32
*** wanghao has joined #openstack-cinder09:32
*** wanghao has quit IRC09:33
*** wanghao has joined #openstack-cinder09:33
*** wanghao has quit IRC09:34
*** wanghao has joined #openstack-cinder09:34
*** wanghao has quit IRC09:35
*** wanghao has joined #openstack-cinder09:35
*** wanghao has quit IRC09:35
*** wanghao has joined #openstack-cinder09:36
*** wanghao has quit IRC09:36
*** wanghao has joined #openstack-cinder09:36
*** wanghao has quit IRC09:37
*** hamdykhader has joined #openstack-cinder09:37
*** hamdyk has quit IRC09:39
*** links has quit IRC09:48
*** dklyle has quit IRC09:50
*** david-lyle has joined #openstack-cinder09:50
*** AlexeyAbashkin has quit IRC09:50
*** david-lyle has quit IRC09:52
*** dklyle has joined #openstack-cinder09:52
*** dklyle has quit IRC09:53
*** david-lyle has joined #openstack-cinder09:53
*** david-lyle has quit IRC09:53
*** BrinZhang has quit IRC09:54
*** e0ne has joined #openstack-cinder09:56
*** AlexeyAbashkin has joined #openstack-cinder10:02
*** dklyle has joined #openstack-cinder10:04
*** links has joined #openstack-cinder10:05
*** AlexeyAbashkin has quit IRC10:13
*** Alexey_Abashkin has joined #openstack-cinder10:13
openstackgerritVishakha Agarwal proposed openstack/cinder master: Cloning image fails results duplicate cache entry  https://review.openstack.org/56747310:14
*** namnh has quit IRC10:15
*** Alexey_Abashkin is now known as AlexeyAbashkin10:16
*** Dinesh_Bhor has quit IRC10:17
*** AlexeyAbashkin has quit IRC10:23
*** AlexeyAbashkin has joined #openstack-cinder10:25
*** ircuser-1 has quit IRC10:27
*** ircuser-1 has joined #openstack-cinder10:28
*** bigdogstl has joined #openstack-cinder10:29
*** bigdogstl has quit IRC10:34
*** Alexey_Abashkin has joined #openstack-cinder10:38
*** AlexeyAbashkin has quit IRC10:39
*** Alexey_Abashkin is now known as AlexeyAbashkin10:39
*** links has quit IRC10:39
*** abhishekk|away has quit IRC10:42
*** edmondsw has joined #openstack-cinder10:43
*** edmondsw has quit IRC10:47
*** links has joined #openstack-cinder10:56
*** dpawlik has joined #openstack-cinder11:00
*** AlexeyAbashkin has quit IRC11:01
*** dpawlik has quit IRC11:05
*** kaisers_ has quit IRC11:05
*** kaisers has joined #openstack-cinder11:06
*** wanghao has joined #openstack-cinder11:08
*** edmondsw has joined #openstack-cinder11:22
*** dustins has joined #openstack-cinder11:38
*** lvdombrkr89 has quit IRC11:46
*** lvdombrkr89 has joined #openstack-cinder11:47
openstackgerritjunbo.li proposed openstack/cinder master: Add check to the flake8 job  https://review.openstack.org/57319811:49
*** AlexeyAbashkin has joined #openstack-cinder11:51
*** dpawlik has joined #openstack-cinder11:52
*** alexchadin has quit IRC11:56
*** alexchadin has joined #openstack-cinder11:57
*** dpawlik has quit IRC11:57
*** alexchadin has quit IRC11:57
*** alexchadin has joined #openstack-cinder11:57
*** alexchadin has quit IRC11:58
*** dustins has quit IRC12:01
*** lvdombrkr has joined #openstack-cinder12:06
*** eharney has quit IRC12:08
*** lvdombrkr89 has quit IRC12:08
*** dustins has joined #openstack-cinder12:09
*** lvdombrkr has quit IRC12:09
*** lvdombrkr has joined #openstack-cinder12:09
*** nicolasbock has joined #openstack-cinder12:11
*** hamdykhader has quit IRC12:11
*** adisky__ has quit IRC12:12
*** armaan has quit IRC12:12
*** armaan has joined #openstack-cinder12:13
*** dave-mccowan has joined #openstack-cinder12:14
*** dave-mcc_ has joined #openstack-cinder12:16
*** armaan has quit IRC12:17
*** dave-mccowan has quit IRC12:18
*** lvdombrkr89 has joined #openstack-cinder12:24
*** lvdombrkr has quit IRC12:27
*** liverpooler has joined #openstack-cinder12:30
*** hemna_ has joined #openstack-cinder12:31
*** lvdombrkr89 has quit IRC12:34
*** alexchadin has joined #openstack-cinder12:34
*** lvdombrkr has joined #openstack-cinder12:34
*** ygl has joined #openstack-cinder12:37
*** lvdombrkr89 has joined #openstack-cinder12:38
*** mriedem has joined #openstack-cinder12:38
*** alexchadin has quit IRC12:39
*** wanghao has quit IRC12:39
*** lseki has joined #openstack-cinder12:40
*** lvdombrkr has quit IRC12:41
*** lvdombrkr89 has quit IRC12:44
mriedemdoes cinder have a concept of a lock api for volumes?12:46
e0nemriedem: hi. what do you mean?12:47
mriedemnvm, reading https://review.openstack.org/#/c/572790/7/nova/compute/api.py@4131 i guess we shouldn't need one - when swapping a volume, nova 'begins detaching' the source volume, so a user shouldn't be able to do things with that volume while it's being detached12:47
*** lvdombrkr has joined #openstack-cinder12:47
mriedemright?12:48
mriedemi'm trying to avoid outright blocking a swap volume from a multiattach volume if it has <=1 read/write attachment,12:48
mriedemthere is a concern in there that swap volume can take a long time and while that's happening, the volume could be attached to another server and cause data corruption,12:48
mriedembut we change the attach state of the old volume we're swapping from at the start of the operation, so you shouldn't be able to do stuff to that volume while nova is detaching it12:49
mriedemhttp://git.openstack.org/cgit/openstack/cinder/tree/cinder/volume/api.py#n72312:50
yglhi all12:50
ygli am not able to attach a cinder volume to an instance12:50
ygli am using queens OSA install12:50
*** enriquetaso has left #openstack-cinder12:50
*** lvdombrkr has quit IRC12:51
*** pchavva has joined #openstack-cinder12:51
yglcan someone help me please12:52
*** rcernin has quit IRC12:56
pooja_jadhavtommylikehu: Hi12:57
e0nemriedem: sorry, I'm not sure that I can help you with it:(12:58
*** tpsilva has joined #openstack-cinder13:02
*** lvdombrkr has joined #openstack-cinder13:05
*** eharney has joined #openstack-cinder13:07
*** AlexeyAbashkin has quit IRC13:08
*** Alexey_Abashkin has joined #openstack-cinder13:08
*** gnufied has joined #openstack-cinder13:08
*** lvdombrkr has quit IRC13:09
*** lvdombrkr has joined #openstack-cinder13:09
*** Alexey_Abashkin is now known as AlexeyAbashkin13:10
*** alexchadin has joined #openstack-cinder13:13
openstackgerritMerged openstack/python-brick-cinderclient-ext master: Fix build error due to doc warning and l-c  https://review.openstack.org/57197813:13
openstackgerritMerged openstack/python-brick-cinderclient-ext master: Switch to using stestr  https://review.openstack.org/57196913:13
*** lvdombrkr89 has joined #openstack-cinder13:17
*** abhishekk has joined #openstack-cinder13:18
*** lvdombrkr has quit IRC13:20
*** ygl has quit IRC13:21
*** lvdombrkr89 has quit IRC13:22
*** alexchad_ has joined #openstack-cinder13:23
*** alexchad_ is now known as alexchadin_13:23
*** alexchadin has quit IRC13:24
*** ygl has joined #openstack-cinder13:28
yglcan someone help me with my cinder issue please13:30
yglI am unable to attach a volume to an instance13:30
yglthese r the logs13:30
yglhttp://paste.openstack.org/show/722880/13:30
*** openstackgerrit has quit IRC13:34
smcginnisygl: Sorry, I haven't seen that one. Any logs earlier than the paste that give you any clues?13:35
yglsmcginnis: no13:35
yglsmcginnis: i am able to create volumes though without any issue13:36
yglsmcginnis: i cant notice any error logs in cinder13:38
smcginnisygl: Probably not errors, but there should be some logging of what was going on that might give some clues why it didn't find the device.13:39
*** dave-mcc_ has quit IRC13:41
yglsmcginnis: ii can  see these http://paste.openstack.org/show/722894/13:41
yglsmcginnis: can u see those logs ?13:42
*** AlexeyAbashkin has quit IRC13:43
*** dave-mccowan has joined #openstack-cinder13:44
*** dave-mccowan has quit IRC13:46
*** openstackgerrit has joined #openstack-cinder13:48
openstackgerritEric Harney proposed openstack/cinder master: Tests: Remove some ovo FutureWarnings about UUIDs  https://review.openstack.org/57323613:48
*** dave-mccowan has joined #openstack-cinder13:48
*** dave-mcc_ has joined #openstack-cinder13:50
*** AlexeyAbashkin has joined #openstack-cinder13:50
*** armaan has joined #openstack-cinder13:52
*** moshele has quit IRC13:52
*** dave-mccowan has quit IRC13:53
*** dpawlik has joined #openstack-cinder13:53
*** throwsb1 has joined #openstack-cinder13:54
openstackgerritMerged openstack/os-brick master: fix tox python3 overrides  https://review.openstack.org/57291413:55
smcginnisygl: That doesn't seem complete, but at least shows the connection info looks correct. Maybe check the system logs on that host to see if there are any iSCSI related errors.13:55
smcginnisygl: Which OS is this?13:56
yglsmcginnis: I found some logs from the compute node where this instance is launched13:57
yglsmcginnis: i will paste it noe13:57
*** dpawlik has quit IRC13:58
yglsmcginnis: check this logs from the compute  http://paste.openstack.org/show/722899/14:01
*** Alexey_Abashkin has joined #openstack-cinder14:02
*** AlexeyAbashkin has quit IRC14:02
*** Alexey_Abashkin is now known as AlexeyAbashkin14:02
*** gkadam has quit IRC14:03
*** gkadam has joined #openstack-cinder14:04
*** lvdombrkr has joined #openstack-cinder14:05
yglsmcginnis: got any idea from those compute logs ?14:06
smcginnisygl: Something is wrong with your iSCSI setup on the compute node: 'iscsiadm: Cannot modify node.session.scan. Invalid param name.\niscsiadm: Could not execute operation on all records: invalid parameter'14:07
yglsmcginnis: how to check this manually ?14:07
yglsmcginnis: i am also suspecting the same14:07
yglsmcginnis: iscsiadm ('--op', 'update', '-n', 'node.session.scan', '-v', 'manual'): stdout= stderr=iscsiadm: Cannot modify node.session.scan. Invalid param name.14:08
*** lvdombrkr has quit IRC14:10
*** lvdombrkr has joined #openstack-cinder14:11
*** felipemonteiro has joined #openstack-cinder14:15
*** felipemonteiro_ has joined #openstack-cinder14:15
*** lvdombrkr has quit IRC14:16
*** alexchadin has joined #openstack-cinder14:17
*** alexchadin_ has quit IRC14:18
*** felipemonteiro has quit IRC14:20
*** ygl has quit IRC14:21
*** links has quit IRC14:23
*** itlinux has quit IRC14:26
*** Alexey_Abashkin has joined #openstack-cinder14:38
*** AlexeyAbashkin has quit IRC14:41
*** Alexey_Abashkin is now known as AlexeyAbashkin14:41
*** rajinir has joined #openstack-cinder14:43
openstackgerritGaudenz Steinlin proposed openstack/cinder stable/ocata: Fix for volume from image snapshot free space issue  https://review.openstack.org/57170814:45
*** moshele has joined #openstack-cinder14:46
*** chhavi__ has joined #openstack-cinder14:48
*** chhagarw has quit IRC14:49
*** ganso has joined #openstack-cinder14:50
*** lifeless_ has joined #openstack-cinder14:53
*** lifeless has quit IRC14:53
*** markstur has joined #openstack-cinder14:56
*** markstur has quit IRC14:56
*** markstur has joined #openstack-cinder14:56
*** alexchadin has quit IRC14:56
*** abhishekk has quit IRC15:00
openstackgerritGaudenz Steinlin proposed openstack/cinder stable/ocata: Move disk space check to run when needed  https://review.openstack.org/57170915:00
*** luizbag has joined #openstack-cinder15:01
*** dpawlik has joined #openstack-cinder15:01
*** Luzi has quit IRC15:01
*** zhaochao has quit IRC15:01
*** moshele has quit IRC15:05
*** dpawlik has quit IRC15:05
*** erlon has joined #openstack-cinder15:06
*** itlinux has joined #openstack-cinder15:14
*** moshele has joined #openstack-cinder15:14
*** imacdonn has quit IRC15:20
*** imacdonn has joined #openstack-cinder15:20
*** Alexey_Abashkin has joined #openstack-cinder15:20
*** AlexeyAbashkin has quit IRC15:22
*** Alexey_Abashkin is now known as AlexeyAbashkin15:22
*** AlexeyAbashkin has quit IRC15:22
*** AlexeyAbashkin has joined #openstack-cinder15:23
-openstackstatus- NOTICE: Zuul update for Ansible 2.5 in progress. Scheduler crashed as unexpected side effect of pip upgrade. Will be back and running shortly.15:29
*** moshele has quit IRC15:31
*** Alexey_Abashkin has joined #openstack-cinder15:37
*** AlexeyAbashkin has quit IRC15:37
*** Alexey_Abashkin is now known as AlexeyAbashkin15:37
*** abhishekk has joined #openstack-cinder15:47
*** AlexeyAbashkin has quit IRC15:53
*** dpawlik has joined #openstack-cinder15:58
*** dpawlik has quit IRC16:02
*** moshele has joined #openstack-cinder16:06
*** abhishekk is now known as abhishekk|dinner16:06
*** moshele has quit IRC16:08
-openstackstatus- NOTICE: The zuul upgrade to ansible 2.5 is complete and zuul is running again. Changes uploaded or approved between 15:25 and 15:45 will need to be rechecked. Please report any problems in #openstack-infra16:09
*** diablo_rojo has quit IRC16:10
*** lifeless has joined #openstack-cinder16:16
_alastor_smcginnis: Do you know of a way to get the nova block_device_mapping_v2 API call to use create_cloned_volume instead of create_volume plus a manual copy?16:16
*** lifeless_ has quit IRC16:16
_alastor_*slow copy16:16
smcginnis[m]alastor: no, but maybe mriedem might.16:18
_alastor_smcginnis[m]: alrighty, I'll check over in the nova channel16:19
*** bigdogstl has joined #openstack-cinder16:20
mriedemi'm here16:20
mriedembut i'm not sure i'm speaking your language16:20
mriedemwhat is create_cloned_volume?16:20
_alastor_mriedem: The cinder clone call which is different from the cinder blank volume create call16:21
_alastor_mriedem: Essentially I'm looking for a way to bypass a slow volume copy with the fast backend-assisted clone for a block device mapping16:22
_alastor_mriedem: So if I specify "source_type": "volume" and "destination_type": "volume" but both source and destination will be hosted on the same backend, then it will allow the backend to perform the heavy lifting for the data copy16:24
*** crose has joined #openstack-cinder16:25
_alastor_mriedem: And ideally specifying "source_type": "image" would work similarly since Cinder has the "clone_image" driver API for backend assisted image copies16:26
patrickeast^ cinder should do that on its own, iirc it tries to call the drivers optimized image copy methods before falling back to the slow path16:27
_alastor_patrickeast: I thought so too, but I can't seem to get a fast clone using block_device_mapping_v2, there is always a long copy that requires increasing the rpc timeout16:29
_alastor_patrickeast: which makes me think Nova isn't calling the same way Cinder would16:29
mriedem_alastor_: source type and dest type volume means you're providing nova with an existing volume and nova won't create anything, it will just attach that volume to the server that gets created16:29
mriedemsource type = image  and dest type = volume means nova creates an image-backed volume from the given image and then attaches it to the server16:29
patrickeast_alastor_: gotcha, afaik the problem you're describing is because of the timeout from nova->cinder, when you create one from just cinders apis it will wait long enough to complete16:30
mriedemif you want source_type=image and destination_type=volume with special fast image clone stuff, nova isn't orchestrating that, and you can do it by creating the image-backed volume first and then provide that volume to nova during server create16:30
patrickeastit is the primary reason for the image-volume cache in cinder, and the cinder glance backend (although I'm not sure that one still works)16:31
patrickeastmriedem: +1 for just making 2 api calls16:31
mriedemalso, searching for 'clone' in https://developer.openstack.org/api-ref/block-storage/v3/ yields nothing16:31
*** crose has quit IRC16:32
_alastor_mriedem: It's a driver interface not a public facing one, but I'm unfamiliar with how Nova calls Cinder for these things16:32
mriedemif volume clones are just a backend optimization in cinder, transparent to the api user (nova in this case) then i'm not sure what we're talking about wrt bfv16:32
smcginnisIt's just a create volume with a source volume specified.16:32
mriedem_alastor_: https://github.com/openstack/nova/blob/master/nova/virt/block_device.py16:33
mriedemlooking for "volume_api.create"16:33
mriedemnova creates volumes from images, snapshots, and blank (empty)16:33
mriedemthere is no source_type=volume destination_type=volume + clone wizardry in nova16:34
*** lifeless_ has joined #openstack-cinder16:35
_alastor_mriedem: ok, that at least gives me a good starting point.  Thanks16:36
*** lifeless has quit IRC16:36
*** dklyle has quit IRC16:36
_alastor_patrickeast: My customer is doing that thing where they really don't want to make two API calls because that would require changing their custom framework16:37
mriedemof course16:37
_alastor_patrickeast: But I'm probably just going to have to tell them "tough luck"16:37
mriedemlike bfv with a non-default volume type16:37
patrickeastsounds like its time to just crank up the build timeouts :D16:38
patrickeast_alastor_: or turn on the image cache and pre-seed it with one create from image from cinder out of band from the framework... should be fast after that16:38
_alastor_patrickeast: Yeah, I'll look into that.  I've had difficulties with the image cache before, but it's been a while16:40
*** felipemonteiro_ has quit IRC16:42
*** felipemonteiro_ has joined #openstack-cinder16:42
*** pocketprotector has quit IRC16:45
*** tbarron has quit IRC16:46
*** lpetrut has quit IRC16:49
*** pocketprotector has joined #openstack-cinder16:50
*** Swimingly has joined #openstack-cinder16:50
*** Swimingly has joined #openstack-cinder16:50
*** tbarron has joined #openstack-cinder16:50
*** e0ne has quit IRC16:55
*** diablo_rojo has joined #openstack-cinder17:05
*** lpetrut has joined #openstack-cinder17:08
*** r-daneel has quit IRC17:10
*** gkadam has quit IRC17:14
*** abhishekk|dinner is now known as abhishekk17:16
openstackgerritRajini Karthik proposed openstack/cinder master: Dell EMC: SC FC Driver wwns api response in lower  https://review.openstack.org/57337417:21
*** lpetrut has quit IRC17:22
*** itlinux has quit IRC17:30
*** boris_42_ has joined #openstack-cinder17:32
*** psachin has quit IRC17:32
*** openstackgerrit has quit IRC17:34
*** lpetrut has joined #openstack-cinder17:36
*** bigdogstl has quit IRC17:37
mriedemwho are the pure storage people around, simondodsley_? https://review.openstack.org/#/c/340168/17:39
mriedemtoday is spec freeze day so someone needs to update that spec17:39
mriedempatrickeast: ^?17:39
patrickeastmriedem: simons been driving that one17:39
*** dustins_ has joined #openstack-cinder17:39
mriedemare you able to drive an internal ping to him?17:40
patrickeastalready on it17:40
mriedemthanks17:40
*** dustins has quit IRC17:42
*** chhavi__ has quit IRC17:45
*** felipemonteiro__ has joined #openstack-cinder17:48
*** felipemonteiro_ has quit IRC17:52
*** e0ne has joined #openstack-cinder17:57
*** armaan_ has joined #openstack-cinder17:58
*** armaan has quit IRC17:58
*** r-daneel has joined #openstack-cinder18:09
*** openstackgerrit has joined #openstack-cinder18:18
openstackgerritVipin Balachandran proposed openstack/cinder master: VMware: Add file open mode to support python3  https://review.openstack.org/57314318:18
*** mriedem has quit IRC18:20
*** abhishekk has quit IRC18:28
*** alkhodos has joined #openstack-cinder18:29
*** lpetrut has quit IRC18:31
*** diablo_rojo has quit IRC18:33
*** diablo_rojo has joined #openstack-cinder18:34
*** lpetrut has joined #openstack-cinder18:36
alkhodossmcginnis: & jungleboyj Hi guys, could you please review this patch again, passed gate after a bugged test https://review.openstack.org/#/c/513143/18:37
jungleboyjalkhodos:  Shoot.  I need that to merge before I can cut the Milestone-218:38
jungleboyjalkhodos:  Looks good to me.  smcginnis can you +A if it looks good to you?18:39
*** lpetrut has quit IRC18:40
eharneyjungleboyj: smcginnis: added some feedback..18:40
*** lpetrut has joined #openstack-cinder18:40
jungleboyjalkhodos:  ^^18:42
*** amrith has joined #openstack-cinder18:42
amrithsmcginnis hello!18:43
alkhodoseharney: responded in the patch18:43
*** e0ne has quit IRC18:43
eharneyhow is an NFS qcow2 option related to iSCSI drivers?18:44
eharneyalkhodos: ^18:44
*** spotz has quit IRC18:45
*** e0ne has joined #openstack-cinder18:45
alkhodoseharney: ah sorry, didn't quite understand the first comment. This is from another patch, you are right...18:46
*** diablo_rojo has quit IRC18:46
*** spotz has joined #openstack-cinder18:46
*** armaan_ has quit IRC18:48
*** jaypipes has joined #openstack-cinder18:50
*** gkadam has joined #openstack-cinder18:51
jaypipesamrith: yo.18:52
amrithsmcginnis moving here so jay can join us18:52
amrithsorry, I was looking for leakypipes18:52
openstackgerritAlexey Khodos proposed openstack/cinder master: Add NexentaEdge drivers  https://review.openstack.org/51314318:52
smcginnisjungleboyj, eharney: Got your answer on that driver patch?18:53
smcginnisalkhodos: New patch coming then?18:53
eharneythe update looks good18:54
alkhodossmcginnis: just pushed18:54
smcginnisamrith, jaypipes: I remember talking to a Heat person about an issue with many simultaneous attach operations.18:54
smcginnisamrith, jaypipes: It was in channel some time ago, but can't remember specific time frames to be able to track it down.18:54
amrithsmcginnis I'm looking to see if my bouncer logs #openstack-cinder18:55
smcginnisamrith, jaypipes: But if memory serves, they found an issue with the way Heat was doing things and were going to address it there.18:55
smcginnisalkhodos: Thanks, looked right past the patch notification without registering it. :)18:55
jungleboyjeharney:  Good catch.18:56
smcginnisalkhodos: Would be great to have CI results posted on that. Any idea how long that would take?18:56
jungleboyjsmcginnis:  Also I assume I should wait to cut MS-2 until this merges or is that not such a big deal?18:57
alkhodossmcginnis: looks like there are a couple of runs in queue, so I'd say about 2 hours18:57
smcginnisjungleboyj: Since it passed third party CI previously and this update just removes an unused config option, we're probably safe just assuming CI will come back happy.18:59
smcginnisjungleboyj: And if not, get on alkhodos until it does. :)18:59
jungleboyjOk.  Sounds reasonable.  Do you want to +A and I will watch for it to merge?18:59
smcginnisjungleboyj: Yeah, just double checked to make sure it really wasn't being references anywhere. Looks safe.19:00
smcginniseharney: That sound OK to you? ^19:00
amriththanks smcginnis I don't have cinder in my bouncer; I'll try and search online and ask the heat folks19:01
eharneysmcginnis: yes19:01
smcginnisOK, I'll just wait for it to pass the check queue, then I'll approve it unless someone else gets to it first.19:02
*** lpetrut has quit IRC19:02
alkhodossmcginnis: seems it will take some time for zuul to pass :)19:04
smcginnisalkhodos: Yeah, but we need it to get through and merge before Jay can cut the m2 release anyway.19:04
alkhodossmcginnis: yep, just saying... I'll update you guys as soon as zuul check is passed19:06
smcginnisalkhodos: Great, that will help. Thanks.19:07
*** mvk has quit IRC19:18
*** ctengel has joined #openstack-cinder19:19
*** dustins_ has quit IRC19:22
*** eharney has quit IRC19:22
*** felipemonteiro__ has quit IRC19:26
*** ctengel has quit IRC19:26
*** lpetrut has joined #openstack-cinder19:27
*** moshele has joined #openstack-cinder19:29
*** diablo_rojo has joined #openstack-cinder19:29
*** moshele has quit IRC19:31
*** ctengel has joined #openstack-cinder19:34
*** boris_42_ has quit IRC19:36
*** ctengel has quit IRC19:42
*** pchavva has quit IRC19:52
*** pchavva has joined #openstack-cinder19:54
*** luizbag has quit IRC19:56
*** eharney has joined #openstack-cinder20:04
*** felipemonteiro has joined #openstack-cinder20:09
*** gkadam has quit IRC20:11
openstackgerritMerged openstack/cinder master: Tests: Remove some ovo FutureWarnings about UUIDs  https://review.openstack.org/57323620:14
*** itlinux has joined #openstack-cinder20:15
*** lpetrut has quit IRC20:15
*** felipemonteiro_ has joined #openstack-cinder20:22
*** felipemonteiro has quit IRC20:25
alkhodossmcginnis: looks like 1 zuul test failed: http://logs.openstack.org/43/513143/18/check/openstack-tox-lower-constraints/097edee/20:30
alkhodosnot sure why though20:30
*** e0ne has quit IRC20:32
*** e0ne has joined #openstack-cinder20:33
*** kaisers has quit IRC20:33
jungleboyjalkhodos:  *Sigh*20:33
alkhodosjungleboyj: I've even triggered the CI manually to speed things up, but...20:33
alkhodosjungleboyj: do you know if it's another broken test or?20:34
jungleboyjIt looks unrelated.20:34
jungleboyjalkhodos:  It is ok.  That is a non-voting job.20:35
jungleboyjOh wait.  No it isn't.20:35
alkhodosjungleboyj: it is not marked as non-voting20:36
jungleboyjMy mistake I was looking at the DRBD one accidentally.20:36
*** pcaruana has quit IRC20:39
*** ckonstanski has joined #openstack-cinder20:42
smcginnisalkhodos: It's all those datacore warning crap.20:43
smcginnisWhich... why the heck is that even running on a lower-constraints check??20:43
*** e0ne has quit IRC20:44
alkhodosI guess there is no point in running recheck before it's fixed?20:44
*** e0ne has joined #openstack-cinder20:45
*** mriedem1 has joined #openstack-cinder20:47
*** armaan has joined #openstack-cinder20:47
*** e0ne has quit IRC20:49
jungleboyjsmcginnis:  Yeah, I was wondering that.20:49
*** kaisers has joined #openstack-cinder20:50
smcginnisSo the lower-constraints job just runs unit tests, but using the constraints from lower-constraints.txt.20:53
smcginnisAnd the issue here is we are hitting the stestr failure we had seen in the past where a test outputs too much text due to all the junk being emitted by the datacore tests.20:54
jungleboyjOh, that is why that failure looks familiar.20:54
smcginnisYeah, we never did figure out a way around that other than to avoid logging too much.20:55
jungleboyj:-(20:55
smcginnisFix coming...20:55
*** dustins has joined #openstack-cinder20:57
*** throwsb1 has quit IRC20:57
*** mriedem1 is now known as mriedem20:58
alkhodossmcginnis: unrelated, but CI already reported for that patch21:01
jungleboyjWow, CI passing before Zuul.21:01
jungleboyjThat is a first.  ;-)21:01
alkhodosjungleboyj: :)21:03
*** erlon has quit IRC21:04
jungleboyjsmcginnis:  A fix for the upper constraints problem?21:07
smcginnisA fix for the DataCore driver unit test causing stestr to blow up during the lower-constraints job.21:08
jungleboyjOk.21:08
smcginnisalkhodos: Why don't you recheck that patch and see. It shouldn't fail every time, so I think you've just had bad luck. Again.21:10
*** itlinux has quit IRC21:15
*** bigdogstl has joined #openstack-cinder21:16
smcginnisalkhodos: I've rechecked it.21:19
openstackgerritSean McGinnis proposed openstack/cinder master: Mock log output from DatCore drivers  https://review.openstack.org/57344621:20
smcginnisjungleboyj: If you want to push that, it should make sure we don't hit the failure again.21:20
smcginnisjungleboyj: Might be OK to wait for normal review though.21:20
smcginnisThis isn't a problem specific to the Nexenta driver patch. Since we haven't been hitting this regularly with other patches, I'm fairly confident a recheck will pass for that one.21:21
*** nicolasbock has quit IRC21:22
jungleboyjSince Nexenta already rechecked I will wait.  I will push this once the check votes.21:22
smcginnis++21:22
smcginnisOh, quick nit.21:22
jungleboyjWhat?21:22
openstackgerritSean McGinnis proposed openstack/cinder master: Mock log output from DatCore drivers  https://review.openstack.org/57344621:22
smcginnisSo trivial I would have skipped it, but figured since I _just_ submitted the patch, might as well catch it quick.21:23
jungleboyjOk.  Works for me.  :-)21:23
*** moshele has joined #openstack-cinder21:24
*** armaan has quit IRC21:25
*** moshele has quit IRC21:28
*** felipemonteiro_ has quit IRC21:28
*** felipemonteiro_ has joined #openstack-cinder21:29
*** lseki has quit IRC21:29
smcginnisLower-constraints job passed on that patch this time. Assuming we don't hit some other random failure, we should be good.21:31
* smcginnis knocks on wood21:31
*** pchavva has quit IRC21:32
smcginnisjungleboyj: And now I noticed I misspelled DataCore (DatCore) in the commit message.21:33
smcginnisjungleboyj: You want me to fix that?21:33
*** rajinir has quit IRC21:34
jungleboyjsmcginnis:   Probably should.21:36
jungleboyjSad part is I looked at that twice and missed it.21:37
openstackgerritSean McGinnis proposed openstack/cinder master: Mock log output from DataCore drivers  https://review.openstack.org/57344621:38
*** bigdogstl has quit IRC21:38
*** edmondsw has quit IRC21:38
*** dustins has quit IRC21:46
*** bigdogstl has joined #openstack-cinder21:57
*** throwsb1 has joined #openstack-cinder21:57
alkhodossmcginnis: thanks for running the recheck, fingers crossed ;)22:01
smcginnisalkhodos: It's looking good so far.22:01
*** bigdogstl has quit IRC22:03
*** lifeless_ has quit IRC22:03
*** lifeless has joined #openstack-cinder22:04
*** bigdogstl has joined #openstack-cinder22:14
*** bigdogstl has quit IRC22:19
*** eharney has quit IRC22:19
*** ganso has quit IRC22:20
*** rcernin has joined #openstack-cinder22:24
*** mchlumsky has quit IRC22:34
*** edmondsw has joined #openstack-cinder22:36
*** edmondsw has quit IRC22:41
*** bigdogstl has joined #openstack-cinder22:47
*** felipemonteiro_ has quit IRC22:48
*** felipemonteiro has joined #openstack-cinder22:49
*** diablo_rojo has quit IRC22:50
*** bigdogstl has quit IRC22:52
*** mriedem is now known as mriedem_afk22:52
*** dklyle has joined #openstack-cinder22:56
alkhodossmcginnis: finally got +1!23:00
*** bigdogstl has joined #openstack-cinder23:01
*** bigdogstl has quit IRC23:09
*** bigdogstl has joined #openstack-cinder23:10
*** bigdogstl has quit IRC23:14
*** felipemonteiro_ has joined #openstack-cinder23:21
*** markstur has quit IRC23:22
*** felipemonteiro has quit IRC23:22
*** throwsb1 has quit IRC23:24
*** markstur has joined #openstack-cinder23:28
*** bigdogstl has joined #openstack-cinder23:30
openstackgerritMerged openstack/cinder master: VMAX driver - Enable multiattach support  https://review.openstack.org/55678023:30
*** bigdogstl has quit IRC23:35
*** markstur has quit IRC23:35
*** hemna_ has quit IRC23:38
*** diablo_rojo has joined #openstack-cinder23:43
*** bigdogstl has joined #openstack-cinder23:49
*** diablo_rojo has quit IRC23:51

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!