Thursday, 2016-01-07

chris_morrelljgriffith sept 2014 https://review.openstack.org/#/c/112977/00:00
patrickeastchris_morrell: hey, sorry was in a meeting00:02
patrickeastchris_morrell: so the cinder side landed a while ago, and support in nova landed for glance images but didn't support it on cinder volumes00:03
patrickeastchris_morrell: the last piece of the puzzle is thing thing https://review.openstack.org/#/c/205726/00:03
patrickeastchris_morrell: it basically extends what was added from https://review.openstack.org/#/c/112977/ to do trim on cinder volume attachments00:03
chris_morrellpatrickeast ah, I'll dig through that review then. Thanks for getting back to me.00:04
patrickeastchris_morrell: np, lemmie know if you have any other questions about it, happy to help00:06
chris_morrellpatrickeast with all the heavy lifting done by you guys, I think our(solidfire) driver just needs a flag set during initialize connection, which seems simple enough.00:07
*** dims_ has quit IRC00:07
patrickeastchris_morrell: yep, that or just use the generic cinder.conf setting i added https://review.openstack.org/#/c/258747/00:08
patrickeastchris_morrell: more reviewers on that nova one is probably helpful though too to get some momentum... stupid thing has been in review for like 6 months now, am going to be sad if we get stuck behind the nova feature freeze again00:09
chris_morrellpatrickeast looking at it now, but I'm just a little newbie figuring out how all this stuff works together :D00:10
patrickeasthaha00:10
patrickeastchris_morrell: sometimes i doubt anyone really knows how all this junk works together :P00:11
chris_morrellpatrickeast heh, well here's to hoping that I'll at least have some semblance of an idea soonish. There's so much information scattered all over the place, I swear I spend more time working my google skills than reading/writing code.00:13
*** jwcroppe has quit IRC00:13
chris_morrellpatrickeast I'll take a look at the reviews you linked later tonight, thanks for the help00:14
*** angela-s has quit IRC00:16
*** akerr_away has quit IRC00:23
*** lcurtis has quit IRC00:29
*** alonma has joined #openstack-cinder00:29
*** haomaiwa_ has joined #openstack-cinder00:33
*** alonma has quit IRC00:33
*** haomaiwa_ has quit IRC00:38
patrickeasthey uh, any cores around that can help this lil fella? https://review.openstack.org/#/c/217940/ its a tiny bug fix thats been up for a looong time with -1's saying they are waiting for the ci to pass.... and it has passed successfully00:40
asselinwow...people really -1 waiting for ci to pass?00:41
*** alonma has joined #openstack-cinder00:41
patrickeastasselin: heh every once and while00:42
patrickeastthen apparently they don't come back and change it when it does00:42
*** jwcroppe has joined #openstack-cinder00:42
jgriffithjust IBM'ers, which is ironic :)00:42
patrickeasthaha00:42
*** zhangjn has joined #openstack-cinder00:43
patrickeastjgriffith: thanks!00:43
jgriffithany time00:44
*** alonma has quit IRC00:46
hemnaasselin, yah I've done that a few times, because a few patches have gotten +A'd before the critical CI even reports.00:46
asselinoh ok...I see that making sense00:47
patrickeasthemna: makes sense for lots of stuff that is only tested on 3rd party ci's, but the come back and adjust review is an important step00:47
openstackgerritmelissaml proposed openstack/cinder: Wrong usage of "an" in the mesages:  https://review.openstack.org/26449400:48
jgriffithdon't think I need to wait for CI00:49
jgriffith^00:49
hemnapatrickeast, yup00:49
patrickeastall abort the review stats train00:50
asselinwow...really...a patch to fix a minor typo  in a private method ^^?00:50
* patrickeast makes train noises00:50
jgriffith:)00:50
patrickeastaboard**00:50
jgriffithFree pass to the next OpenStack summit... *Granted*00:51
patrickeasthaha00:51
asselinand vote :)00:51
jgriffithasselin: in a doc-string no less00:51
jgriffithoh yeah... voting!!!00:51
asselincinder citizenship00:51
*** alonma has joined #openstack-cinder00:51
* jgriffith needs him some commits in other projects00:51
jgriffithvoting is fun!00:52
patrickeastif you hurry there might still be some a->an bugs left!00:52
jgriffithpatrickeast: I'm grepping on "it's" and "its"00:53
patrickeasthaha00:53
jgriffiththat's much more beneficial00:53
jgriffithand harder to push back on :)00:53
patrickeasttruth00:53
patrickeastanyone remember if there was a bug or bp for removing db calls in the cg methods of drivers?00:54
patrickeastfixing it in mine now but don't remember where the thing is to associate with it00:54
patrickeastguess i can just make a new one00:54
*** zhangjn has quit IRC00:55
*** alonma has quit IRC00:56
*** dims has joined #openstack-cinder00:56
*** alonma has joined #openstack-cinder00:59
*** bardia has quit IRC01:00
mtaninosmcginnis: hi, Do you have vendor driver review list for Mitaka?01:01
*** cheneydc has joined #openstack-cinder01:02
mtaninosmcginnis: I know only spec review list here. https://etherpad.openstack.org/p/mitaka-cinder-spec-review-tracking01:02
*** alonma has quit IRC01:04
*** vinayp has joined #openstack-cinder01:04
*** vinayp has quit IRC01:05
*** vinayp has joined #openstack-cinder01:06
*** zhangjn has joined #openstack-cinder01:06
*** EinstCrazy has joined #openstack-cinder01:07
*** gouthamr_ has joined #openstack-cinder01:07
*** alonma has joined #openstack-cinder01:08
*** merooney has joined #openstack-cinder01:10
*** gouthamr has quit IRC01:10
*** alonma has quit IRC01:13
*** boris-42 has quit IRC01:13
*** alonma has joined #openstack-cinder01:17
*** alonma has quit IRC01:21
*** alonma has joined #openstack-cinder01:23
*** cheneydc has quit IRC01:28
*** alonma has quit IRC01:28
*** cheneydc has joined #openstack-cinder01:29
*** alonma has joined #openstack-cinder01:30
*** alonma has quit IRC01:35
*** Lee1092 has joined #openstack-cinder01:35
*** asselin has quit IRC01:36
*** alonma has joined #openstack-cinder01:36
*** alonma has quit IRC01:40
*** Thelo has joined #openstack-cinder01:41
*** alonma has joined #openstack-cinder01:43
*** garthb_ has quit IRC01:45
*** leeantho has quit IRC01:45
*** alonma has quit IRC01:47
*** mtanino has quit IRC01:47
*** Thelo has quit IRC01:49
*** Thelo has joined #openstack-cinder01:49
*** vinayp has quit IRC01:50
*** alonma has joined #openstack-cinder01:50
*** alonma has quit IRC01:55
*** alonma has joined #openstack-cinder01:58
*** julim has joined #openstack-cinder02:00
openstackgerritxiaoqin proposed openstack/cinder: Storwize/SVC: Volume manage using source-name  https://review.openstack.org/25225902:01
*** alonma has quit IRC02:02
openstackgerritLiucheng Jiang proposed openstack/cinder: Huawei: Implement v2 replication (managed)  https://review.openstack.org/26124602:13
*** mriedem has quit IRC02:13
*** s-moriya has quit IRC02:13
openstackgerritchenzongliang proposed openstack/cinder: Huawei: Support huawei consistency group  https://review.openstack.org/26093002:15
*** zhangjn has quit IRC02:19
*** salv-orl_ has quit IRC02:23
*** salv-orlando has joined #openstack-cinder02:23
*** haomaiwang has joined #openstack-cinder02:26
*** haomaiw__ has joined #openstack-cinder02:29
*** haomaiwang has quit IRC02:31
*** houming has joined #openstack-cinder02:33
*** vinayp has joined #openstack-cinder02:37
*** zhangjn has joined #openstack-cinder02:38
*** esker has joined #openstack-cinder02:40
*** natarej_ has joined #openstack-cinder02:40
*** natarej_ has quit IRC02:44
*** Fdaisuke has joined #openstack-cinder02:48
*** sheel has joined #openstack-cinder02:57
*** alonma has joined #openstack-cinder02:58
*** dims has quit IRC02:58
*** mragupat has joined #openstack-cinder02:58
*** haomaiw__ has quit IRC03:01
*** haomaiwang has joined #openstack-cinder03:01
*** alonma has quit IRC03:02
*** boris-42 has joined #openstack-cinder03:02
*** alonma has joined #openstack-cinder03:08
*** links has joined #openstack-cinder03:10
*** links has quit IRC03:10
*** alonma has quit IRC03:12
*** edmondsw has quit IRC03:12
*** links has joined #openstack-cinder03:12
*** alonma has joined #openstack-cinder03:14
*** alonma has quit IRC03:19
*** salv-orlando has quit IRC03:26
*** salv-orlando has joined #openstack-cinder03:27
*** esker has quit IRC03:35
*** esker has joined #openstack-cinder03:37
*** zhangjn has quit IRC03:40
*** esker has quit IRC03:41
*** ChubYann has quit IRC03:42
*** [1]Thelo has joined #openstack-cinder03:52
*** Thelo has quit IRC03:55
*** [1]Thelo is now known as Thelo03:55
*** mudassirlatif has quit IRC03:56
*** sheel has quit IRC03:59
*** haomaiwang has quit IRC04:01
*** haomaiwang has joined #openstack-cinder04:01
*** gouthamr_ has quit IRC04:05
*** gouthamr has joined #openstack-cinder04:06
*** esker has joined #openstack-cinder04:06
*** mragupat has quit IRC04:08
*** salv-orl_ has joined #openstack-cinder04:09
*** esker has quit IRC04:11
*** salv-orlando has quit IRC04:11
*** lprice has joined #openstack-cinder04:13
*** alonma has joined #openstack-cinder04:14
*** shyama has joined #openstack-cinder04:16
*** esker has joined #openstack-cinder04:17
*** alonma has quit IRC04:19
*** coolsvap|away is now known as coolsvap04:21
*** esker has quit IRC04:22
*** alonma has joined #openstack-cinder04:26
*** gouthamr_ has joined #openstack-cinder04:28
*** alonma has quit IRC04:30
*** gouthamr has quit IRC04:31
*** harlowja_at_home has joined #openstack-cinder04:45
*** dave-mccowan has quit IRC04:50
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/26441404:52
*** shyama has quit IRC04:55
*** shyama has joined #openstack-cinder04:58
*** haomaiwang has quit IRC05:01
*** haomaiwa_ has joined #openstack-cinder05:01
*** esker has joined #openstack-cinder05:03
*** esker has quit IRC05:08
*** smoriya_ has joined #openstack-cinder05:13
*** lprice has quit IRC05:14
*** EinstCrazy has quit IRC05:14
*** EinstCrazy has joined #openstack-cinder05:14
*** EinstCra_ has joined #openstack-cinder05:20
*** mragupat has joined #openstack-cinder05:21
*** shyama has quit IRC05:23
*** zhangjn has joined #openstack-cinder05:23
*** harlowja_at_home has quit IRC05:23
*** EinstCrazy has quit IRC05:23
*** shausy has joined #openstack-cinder05:26
*** shausy has quit IRC05:26
*** alonma has joined #openstack-cinder05:27
*** gouthamr_ has quit IRC05:30
*** shausy has joined #openstack-cinder05:30
openstackgerritKazumasa Nomura proposed openstack/cinder: Refactor Hitachi Block Storage Driver  https://review.openstack.org/25523905:30
*** alonma has quit IRC05:31
*** alonma has joined #openstack-cinder05:34
*** zhangjn_ has joined #openstack-cinder05:34
*** zhangjn has quit IRC05:36
*** zhangjn_ has quit IRC05:38
*** alonma has quit IRC05:38
*** chhavi has joined #openstack-cinder05:41
*** alonma has joined #openstack-cinder05:41
openstackgerritVinay Prasad M proposed openstack/os-brick: Closes Bug : 1493350 Code Change to convert lunID/deviceID to hex as per SCSI report LUNs standard if greater than 255  https://review.openstack.org/26457805:42
*** tsekiyama has joined #openstack-cinder05:45
*** zhangjn has joined #openstack-cinder05:45
*** alonma has quit IRC05:46
*** chhavi has quit IRC05:46
*** alonma has joined #openstack-cinder05:48
*** zhangjn has quit IRC05:51
*** alonma has quit IRC05:52
*** zhangjn has joined #openstack-cinder05:54
*** rajinir has quit IRC05:55
*** zhangjn has quit IRC05:55
*** esker has joined #openstack-cinder05:58
*** zhangjn has joined #openstack-cinder06:00
*** alonma has joined #openstack-cinder06:00
*** haomaiwa_ has quit IRC06:01
*** zhangjn has quit IRC06:01
*** 7JTABHHZB has joined #openstack-cinder06:01
*** zhangjn has joined #openstack-cinder06:02
*** shyama has joined #openstack-cinder06:04
*** alonma has quit IRC06:05
*** tsekiyama has quit IRC06:06
*** alonma has joined #openstack-cinder06:07
*** chhavi has joined #openstack-cinder06:08
*** salv-orl_ has quit IRC06:08
*** salv-orlando has joined #openstack-cinder06:08
*** markvoelker has quit IRC06:11
*** alonma has quit IRC06:11
*** esker has quit IRC06:18
*** alonma has joined #openstack-cinder06:18
*** merooney has quit IRC06:21
*** alonma has quit IRC06:22
*** chhavi has quit IRC06:22
*** ankit_ag has joined #openstack-cinder06:24
*** zhangjn has quit IRC06:25
*** kazu has joined #openstack-cinder06:30
*** mragupat has quit IRC06:30
*** nkrinner has joined #openstack-cinder06:35
*** zhangjn has joined #openstack-cinder06:35
*** zhangjn has quit IRC06:36
*** zhangjn has joined #openstack-cinder06:37
*** PsionTheory has quit IRC06:38
openstackgerritSwapnil Kulkarni (coolsvap) proposed openstack/os-brick: Use assertTrue/False instead of assertEqual(T/F)  https://review.openstack.org/26460006:39
*** kazu has quit IRC06:41
*** zhangjn has quit IRC06:44
*** harlowja_at_home has joined #openstack-cinder06:44
*** sheel has joined #openstack-cinder06:45
*** vgridnev has joined #openstack-cinder06:48
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/26441406:51
*** tongli has quit IRC06:51
*** coolsvap is now known as coolsvap|away06:56
*** tongli has joined #openstack-cinder06:58
*** esker has joined #openstack-cinder07:00
*** 7JTABHHZB has quit IRC07:01
*** zhangjn has joined #openstack-cinder07:04
*** esker has quit IRC07:05
*** knomura has joined #openstack-cinder07:07
*** haomaiwang has joined #openstack-cinder07:07
*** knomura has quit IRC07:08
*** ankit_ag has quit IRC07:10
*** EinstCrazy has joined #openstack-cinder07:11
*** EinstCra_ has quit IRC07:11
*** markvoelker has joined #openstack-cinder07:11
*** ankit_ag has joined #openstack-cinder07:15
*** markvoelker has quit IRC07:16
*** belmoreira has joined #openstack-cinder07:26
*** EinstCrazy has quit IRC07:29
*** EinstCrazy has joined #openstack-cinder07:29
*** harlowja_at_home has quit IRC07:31
openstackgerritLiuNanke proposed openstack/python-cinderclient: Remove openstack-common.conf  https://review.openstack.org/26464407:32
*** chhavi has joined #openstack-cinder07:33
*** isaacb has joined #openstack-cinder07:39
*** coolsvap|away is now known as coolsvap07:41
*** anshul has joined #openstack-cinder07:45
*** deepakcs has joined #openstack-cinder07:46
*** pots2 has joined #openstack-cinder07:48
*** pots4 has joined #openstack-cinder07:49
*** lpetrut has joined #openstack-cinder07:50
*** pots3 has quit IRC07:51
*** pots has quit IRC07:51
*** liverpooler has joined #openstack-cinder07:58
*** haomaiwang has quit IRC08:01
*** haomaiwang has joined #openstack-cinder08:01
*** zhangjn has quit IRC08:03
*** salv-orlando has quit IRC08:04
flip214DuncanT: available for a short (non-technical) question, please?08:04
*** salv-orlando has joined #openstack-cinder08:04
*** zhangjn has joined #openstack-cinder08:05
*** alonma has joined #openstack-cinder08:06
*** pots4 has quit IRC08:08
*** pots2 has quit IRC08:08
*** pots has joined #openstack-cinder08:08
openstackgerritChangBo Guo(gcb) proposed openstack/cinder: test:clean up wrong value in method CONF.set_override  https://review.openstack.org/25332808:09
*** pots2 has joined #openstack-cinder08:10
*** alonma has quit IRC08:10
*** isaacb has quit IRC08:14
flip214hemna: can you help with https://review.openstack.org/#/c/256292/3, please? thanks a lot.08:14
*** alonma has joined #openstack-cinder08:18
*** alonma has quit IRC08:22
*** alonma has joined #openstack-cinder08:29
*** salv-orlando has quit IRC08:30
*** salv-orlando has joined #openstack-cinder08:30
*** alonma has quit IRC08:31
*** alonma has joined #openstack-cinder08:31
*** isaacb has joined #openstack-cinder08:37
*** markus_z has joined #openstack-cinder08:40
*** itzdilip has joined #openstack-cinder08:40
*** mkoderer_cloud has joined #openstack-cinder08:48
openstackgerritKeiichi KII proposed openstack/os-brick: remove unneeded in_use flag checking for multipath  https://review.openstack.org/23039108:49
*** esker has joined #openstack-cinder08:53
openstackgerritVinay Prasad M proposed openstack/os-brick:  Closes Bug : 1493350     Code Change to convert lunID/deviceID to hex as per SCSI report LUNs standard if greater than 255  https://review.openstack.org/26467108:54
*** chhavi has quit IRC08:57
*** esker has quit IRC08:57
*** markus_z has quit IRC08:58
*** jordanP has joined #openstack-cinder08:59
*** haomaiwang has quit IRC09:01
*** haomaiwang has joined #openstack-cinder09:01
openstackgerritSilvan Kaiser proposed openstack/cinder: Add Quobyte policy tag support to Quobyte Cinder driver  https://review.openstack.org/25470809:01
*** markus_z has joined #openstack-cinder09:02
*** shausy has quit IRC09:06
*** lpetrut has quit IRC09:06
*** shausy has joined #openstack-cinder09:06
*** vgridnev has quit IRC09:06
*** haomaiwang has quit IRC09:08
*** haomaiwang has joined #openstack-cinder09:09
openstackgerritKeiichi KII proposed openstack/os-brick: remove unneeded in_use flag checking for multipath  https://review.openstack.org/26467809:12
*** markvoelker has joined #openstack-cinder09:12
*** klkumar has joined #openstack-cinder09:16
*** markvoelker has quit IRC09:17
*** vgridnev has joined #openstack-cinder09:20
*** vgridnev has quit IRC09:25
*** itzdilip has quit IRC09:29
*** haomaiwang has quit IRC09:35
*** manous has joined #openstack-cinder09:40
*** jistr has joined #openstack-cinder09:41
*** zhangjn has quit IRC09:42
openstackgerritMannu proposed openstack/cinder: volume create with unicode name results in HTTP 500 fix  https://review.openstack.org/26129009:43
*** alonma has quit IRC09:45
*** haomaiwa_ has joined #openstack-cinder09:45
*** alonma has joined #openstack-cinder09:45
*** tongli has quit IRC09:48
*** alonma has quit IRC09:50
*** alonma has joined #openstack-cinder09:51
*** alonma has quit IRC09:55
*** ndipanov has joined #openstack-cinder09:57
*** alonma has joined #openstack-cinder09:57
openstackgerritchenzongliang proposed openstack/cinder: Huawei: Refactor driver for the second time  https://review.openstack.org/25692009:58
*** haomaiwa_ has quit IRC09:58
openstackgerritSzymon Wróblewski proposed openstack/cinder: Update initialize_connection to use versionedobjects  https://review.openstack.org/23889409:59
*** zhangjn has joined #openstack-cinder09:59
*** alonma has quit IRC10:02
*** cheneydc has quit IRC10:03
*** alonma has joined #openstack-cinder10:04
*** EinstCrazy has quit IRC10:07
*** haomaiwang has joined #openstack-cinder10:07
*** alonma has quit IRC10:08
*** salv-orl_ has joined #openstack-cinder10:08
*** alonma has joined #openstack-cinder10:10
*** salv-orlando has quit IRC10:11
*** shyama has quit IRC10:13
*** alonma has quit IRC10:14
*** yhayashi has quit IRC10:14
*** alonma has joined #openstack-cinder10:16
*** ankit_ag has quit IRC10:19
*** alonma has quit IRC10:20
*** alonma has joined #openstack-cinder10:22
*** alonma has quit IRC10:27
*** alonma has joined #openstack-cinder10:29
*** haomaiwang has quit IRC10:32
*** alonma has quit IRC10:34
*** alonma has joined #openstack-cinder10:36
*** apoorvad has joined #openstack-cinder10:40
*** alonma has quit IRC10:40
*** esker has joined #openstack-cinder10:41
*** alonma has joined #openstack-cinder10:42
openstackgerritVinay Prasad M proposed openstack/os-brick:  Closes Bug : 1493350     Code Change to convert lunID/deviceID to hex as per SCSI report LUNs standard if greater than 255  https://review.openstack.org/26467110:43
openstackgerritKazumasa Nomura proposed openstack/cinder: Refactor Hitachi Block Storage Driver  https://review.openstack.org/25523910:43
*** haomaiwang has joined #openstack-cinder10:45
*** esker has quit IRC10:45
*** alonma has quit IRC10:46
*** alonma has joined #openstack-cinder10:48
*** lpetrut has joined #openstack-cinder10:49
*** ociuhandu has quit IRC10:51
*** alonma has quit IRC10:52
*** alonma has joined #openstack-cinder10:55
*** haomaiwang has quit IRC10:57
openstackgerritMerged openstack/cinder: Wrong usage of "an" in the mesages: an service  https://review.openstack.org/26412110:59
*** alonma has quit IRC10:59
*** chhavi has joined #openstack-cinder11:00
*** alonma has joined #openstack-cinder11:01
*** alonma has quit IRC11:06
*** alonma has joined #openstack-cinder11:07
*** salv-orl_ has quit IRC11:07
*** salv-orlando has joined #openstack-cinder11:08
*** haomaiwang has joined #openstack-cinder11:08
*** jordanP has quit IRC11:09
*** dims has joined #openstack-cinder11:10
*** coolsvap is now known as coolsvap|away11:11
*** alonma has quit IRC11:12
*** markvoelker has joined #openstack-cinder11:13
*** alonma has joined #openstack-cinder11:13
*** lprice has joined #openstack-cinder11:15
*** alonma has quit IRC11:17
*** markvoelker has quit IRC11:18
*** ociuhandu has joined #openstack-cinder11:19
*** alonma has joined #openstack-cinder11:19
*** haomaiwang has quit IRC11:21
*** alonma has quit IRC11:23
*** haomaiwang has joined #openstack-cinder11:24
*** alonma has joined #openstack-cinder11:30
*** alonma has quit IRC11:34
*** haomaiwang has quit IRC11:36
*** alonma has joined #openstack-cinder11:36
*** alonma has quit IRC11:37
*** alonma has joined #openstack-cinder11:37
*** sheel has quit IRC11:38
*** 16WAAH8AU has joined #openstack-cinder11:41
*** zhangjn has quit IRC11:49
*** 16WAAH8AU has quit IRC11:53
*** haomaiwa_ has joined #openstack-cinder11:54
*** esker has joined #openstack-cinder11:56
*** ankit_ag has joined #openstack-cinder11:58
*** zhangjn has joined #openstack-cinder11:58
*** zhangjn has quit IRC11:58
*** zhangjn has joined #openstack-cinder11:59
*** zhangjn has quit IRC11:59
*** zhangjn has joined #openstack-cinder11:59
*** apoorvad has quit IRC12:00
*** zhangjn has quit IRC12:00
*** esker has quit IRC12:00
*** haomaiwa_ has quit IRC12:01
*** zhangjn has joined #openstack-cinder12:01
*** haomaiwang has joined #openstack-cinder12:06
*** haomaiwang has quit IRC12:13
*** haomaiw__ has joined #openstack-cinder12:13
*** markvoelker has joined #openstack-cinder12:14
*** apoorvad has joined #openstack-cinder12:17
*** esker has joined #openstack-cinder12:17
*** kaisers has quit IRC12:17
*** markvoelker has quit IRC12:18
openstackgerritMichal Dulko proposed openstack/cinder-specs: Update online-schema-upgrades spec  https://review.openstack.org/26474412:21
*** apoorvad has quit IRC12:22
*** esker has quit IRC12:22
*** alonma has quit IRC12:22
*** JoseMello has joined #openstack-cinder12:22
*** alonma has joined #openstack-cinder12:23
openstackgerritWilson Liu proposed openstack/cinder: Huawei: Add manage/unmanage volume support  https://review.openstack.org/24041312:26
*** martyturner has joined #openstack-cinder12:27
*** alonma has quit IRC12:27
openstackgerritWilson Liu proposed openstack/cinder: Huawei: Balanced FC port selection when zoning  https://review.openstack.org/26115112:31
*** kaisers has joined #openstack-cinder12:33
*** raildo-afk is now known as raildo12:34
*** haomaiw__ has quit IRC12:35
*** alonma has joined #openstack-cinder12:36
*** cfriesen__ has joined #openstack-cinder12:36
*** haomaiwa_ has joined #openstack-cinder12:38
*** apoorvad has joined #openstack-cinder12:39
cfriesen__question...does anyone know why iscsi_write_cache option defaults to "on" given that we're likely already going to be caching those writes up in the instance?12:40
*** alonma has quit IRC12:40
cfriesen__I'm seeing issues where heavy write load from the guests is consuming the host writeback cache to the point where the rootfs disk access is impaired12:41
cfriesen__I'm testing with iscsi_write_cache turned off, in the hopes that it'll fix that problem.12:42
*** alonma has joined #openstack-cinder12:42
*** apoorvad has quit IRC12:43
*** deepakcs has quit IRC12:44
*** alonma has quit IRC12:47
*** zhangjn has quit IRC12:48
*** alonma has joined #openstack-cinder12:48
*** resker has joined #openstack-cinder12:50
*** delattec has joined #openstack-cinder12:50
*** haomaiwa_ has quit IRC12:51
*** cheneydc has joined #openstack-cinder12:51
*** cdelatte has quit IRC12:52
*** gcb has quit IRC12:52
*** resker has quit IRC12:52
*** alonma has quit IRC12:53
openstackgerritKazumasa Nomura proposed openstack/cinder: Refactor Hitachi Block Storage Driver  https://review.openstack.org/25523912:54
*** alonma has joined #openstack-cinder12:55
*** dave-mccowan has joined #openstack-cinder12:56
*** porrua has joined #openstack-cinder12:56
*** houming has quit IRC12:58
*** haomaiwang has joined #openstack-cinder12:58
*** alonma has quit IRC12:59
*** smoriya_ has quit IRC12:59
*** haomaiwang has quit IRC13:01
*** alonma has joined #openstack-cinder13:01
*** cknight has joined #openstack-cinder13:01
*** haomaiwa_ has joined #openstack-cinder13:01
*** salv-orlando has quit IRC13:03
*** salv-orlando has joined #openstack-cinder13:03
*** alonma has quit IRC13:05
*** alonma has joined #openstack-cinder13:05
*** zhangjn has joined #openstack-cinder13:06
openstackgerrithuanan proposed openstack/cinder: Huawei: Add manage/unmanage snapshot support  https://review.openstack.org/26115613:07
*** links has quit IRC13:07
*** cfriesen__ has quit IRC13:08
*** haomaiwa_ has quit IRC13:09
*** cfriesen__ has joined #openstack-cinder13:09
*** manous has quit IRC13:10
*** manous has joined #openstack-cinder13:11
*** shyama has joined #openstack-cinder13:14
*** Thelo has quit IRC13:16
*** lprice has quit IRC13:16
*** haomaiwa_ has joined #openstack-cinder13:16
*** Thelo has joined #openstack-cinder13:16
*** alonma has quit IRC13:17
*** alonma has joined #openstack-cinder13:17
*** markvoelker has joined #openstack-cinder13:18
*** haomaiwa_ has quit IRC13:18
*** erlon has joined #openstack-cinder13:22
*** edmondsw has joined #openstack-cinder13:22
*** klkumar has quit IRC13:31
*** cfriesen__ has quit IRC13:33
*** apoorvad has joined #openstack-cinder13:34
*** cfriesen__ has joined #openstack-cinder13:34
*** gouthamr has joined #openstack-cinder13:35
*** ndipanov has quit IRC13:36
*** apoorvad has quit IRC13:38
*** diablo_rojo has joined #openstack-cinder13:38
*** cheneydc has quit IRC13:41
*** haomaiwang has joined #openstack-cinder13:46
*** jordanP has joined #openstack-cinder13:47
*** akerr has joined #openstack-cinder13:48
*** [1]Thelo has joined #openstack-cinder13:53
*** Thelo has quit IRC13:56
*** [1]Thelo is now known as Thelo13:56
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/26441413:59
*** boichev has joined #openstack-cinder14:00
*** haomaiwang has quit IRC14:01
*** 5EXAAOC18 has joined #openstack-cinder14:01
*** EinstCrazy has joined #openstack-cinder14:01
*** rlrossit has joined #openstack-cinder14:02
*** ankit_ag has quit IRC14:02
*** xyang1 has joined #openstack-cinder14:03
*** shausy has quit IRC14:03
*** shausy has joined #openstack-cinder14:04
*** anshul has quit IRC14:05
*** delatte has joined #openstack-cinder14:05
*** delattec has quit IRC14:08
rlrossitdulek: question for you: did you have to put in https://review.openstack.org/#/c/247763/15/cinder/objects/volume.py L138-170 because you were getting attribute errors when trying to set those values?14:08
*** EinstCrazy has quit IRC14:09
dulekrlrossit: So before objects there were volume_metadata in SQLAlchemy model.14:10
dulekrlrossit: thangp renamed it in object to metadata.14:10
dulekrlrossit: Now we're facing issues in the drivers (but not only), because they are referencing old names.14:11
rlrossitdulek: why did you add those properties though? instead of just letting the versioned object handle it?14:11
*** esker has joined #openstack-cinder14:11
dulekrlrossit: Rename the attrs in drivers then!14:11
rlrossit(what I'm getting at is I think I found an o.vo bug)14:11
*** mkoderer_cloud has quit IRC14:11
dulekrlrossit: But unfortunately - we're in the middle of transition, so sometimes it's an object, sometimes a SQLA model.14:12
rlrossitoh so you just did properties to handle renaming?14:12
dulekrlrossit: Yeah, aliases to keep compatibility with SQLA models.14:12
rlrossithmmm... I swear what I've been seeing is a bug with o.vo, and I'm looking for something else to help me confirm it14:13
dulekrlrossit: Okay, go on. :)14:13
*** gcb_ has joined #openstack-cinder14:14
dulekrlrossit: So what've you observed?14:14
rlrossitdulek: I'm trying to get rid of the dict compat for nova, and in https://github.com/openstack/nova/blob/master/nova/objects/aggregate.py#L38 we have an obj_extra_fields that is blowing up when I try to use setattr on the object14:15
rlrossit(this isn't even a dict compat thing, not even that works)14:15
rlrossitso I'm amazed this has lived for so long14:15
rlrossitI'm getting AttributeError: can't set attribute, so I'm wondering if that's a problem with the properties o.vo automatically sets for the fields (but apparently not obj_extra_fields) on register14:16
dulekrlrossit: Ha, interesting… But I have added unit tests that are setting and reading from these obj_extra_fields I've added.14:17
*** vgridnev has joined #openstack-cinder14:17
dulekrlrossit: https://review.openstack.org/#/c/247763/15/cinder/tests/unit/objects/test_volume.py14:17
dulekrlrossit: So this seems to work in Cinder.14:17
*** mriedem has joined #openstack-cinder14:17
rlrossitbut you made your own properties for those right?14:18
dulekrlrossit: If I recall correctly I've seen that obj_extra_fields bu default save to _obj_foo.14:18
dulekrlrossit: That's right.14:18
*** shausy has quit IRC14:18
rlrossitman I hope this isn't just something really stupid I'm overlooking14:19
rlrossitDictCompat == the devil14:19
*** anshul has joined #openstack-cinder14:20
dulekrlrossit: Ah, on your object there's only @property defined. There's no setter.14:20
*** esker has quit IRC14:20
dulekrlrossit: Then why do you exactly want to use setattr?14:20
dulekrlrossit: https://github.com/openstack/nova/blob/master/nova/objects/aggregate.py#L149-L15114:21
rlrossitgood. grief14:21
rlrossitthanks dulek14:21
*** esker has joined #openstack-cinder14:21
rlrossitI must've been looking at this too late last night/too early this morning14:21
rlrossitthanks dulek!14:22
dulekrlrossit: Glad to be of help.14:22
*** dustins has joined #openstack-cinder14:23
*** cknight has quit IRC14:24
*** jgregor has joined #openstack-cinder14:25
*** martyturner has quit IRC14:25
*** ndipanov has joined #openstack-cinder14:26
*** EinstCrazy has joined #openstack-cinder14:30
openstackgerritSzymon Borkowski proposed openstack/cinder: Update terminate_connection to use volume object  https://review.openstack.org/26479714:31
openstackgerritKedar Vidvans proposed openstack/cinder: Filtering type extra-spec support to ZFSSA drivers  https://review.openstack.org/26376614:34
*** apoorvad has joined #openstack-cinder14:35
*** baumann has joined #openstack-cinder14:35
*** itzdilip has joined #openstack-cinder14:37
*** itzdilip has left #openstack-cinder14:38
*** itzdilip1 has joined #openstack-cinder14:39
*** EinstCrazy has quit IRC14:40
*** timcl has joined #openstack-cinder14:40
*** apoorvad has quit IRC14:40
*** cknight has joined #openstack-cinder14:42
*** esker has quit IRC14:48
*** esker has joined #openstack-cinder14:48
*** dslevin_ has quit IRC14:50
*** Yogi1 has joined #openstack-cinder14:51
*** knomura has joined #openstack-cinder14:53
*** liverpooler has quit IRC14:53
*** markstur has joined #openstack-cinder14:53
*** eharney has quit IRC14:54
*** anshul has quit IRC14:55
smcginnisDuncanT: ping14:57
*** markstur has left #openstack-cinder14:59
*** 5EXAAOC18 has quit IRC15:01
*** haomaiwang has joined #openstack-cinder15:01
*** isaacb has quit IRC15:01
*** zhangjn has quit IRC15:02
*** dslev has joined #openstack-cinder15:03
flip214smcginnis: please advise.15:08
flip214in https://review.openstack.org/#/c/259973/4 the first reviewer said to use """ instead of # for comments; now, some iterations later, I get the opposite wish.15:09
*** cfriesen__ has quit IRC15:09
*** mriedem is now known as mriedem_meeting15:10
*** eharney has joined #openstack-cinder15:10
*** diogogmt has joined #openstack-cinder15:10
smcginnisflip214: Looking...15:10
flip214line 774 resp. 775 in the later commit.15:11
smcginnisflip214: Looks like you got some bad feedback unfortunately. :[15:11
smcginnisflip214: The three quotes are for marking docstrings.15:11
smcginnisflip214: So if it's just a multiline comment, not a docstring, that doesn't apply.15:11
flip214ack, so I'll ignore the earlier invalid requests, and re-push with "#" again.15:12
flip214thanks.15:12
smcginnisflip214: So I think you had it correct to start with. No problem.15:12
*** mragupat has joined #openstack-cinder15:12
*** edtubill has joined #openstack-cinder15:13
openstackgerritPhilipp Marek proposed openstack/cinder: Infrastructure to use the DRBD transport for NOVA.  https://review.openstack.org/25997315:14
*** mtanino has joined #openstack-cinder15:19
*** med_` has joined #openstack-cinder15:20
*** med_ has quit IRC15:20
*** Yogi1 has quit IRC15:21
*** diogogmt has quit IRC15:22
*** crose has joined #openstack-cinder15:26
*** markstur has joined #openstack-cinder15:29
*** markstur has left #openstack-cinder15:29
*** breitz has quit IRC15:30
*** breitz has joined #openstack-cinder15:30
*** isaacb has joined #openstack-cinder15:32
mtaninosmcginnis: Hi, Are you there?15:32
*** dims_ has joined #openstack-cinder15:35
*** dims has quit IRC15:36
*** lcurtis has joined #openstack-cinder15:38
*** garthb has joined #openstack-cinder15:40
*** mriedem_meeting is now known as mriedem15:42
*** xyang has joined #openstack-cinder15:42
*** timcl has quit IRC15:44
*** knomura has quit IRC15:44
*** martyturner has joined #openstack-cinder15:45
mtaninoDuncanT: Hello,15:46
*** knomura has joined #openstack-cinder15:47
*** itzdilip1 has quit IRC15:48
*** gcb_ has quit IRC15:50
*** tongli has joined #openstack-cinder15:51
smcginnismtanino: Back now!15:57
smcginnismtanino: Saw you're earlier ping. I will work on getting a new driver etherpad going today.15:57
*** rlrossit has quit IRC15:57
mtaninosmcginnis: Hi, happy new year :) and I have one question about driver review15:57
*** apoorvad has joined #openstack-cinder15:57
mtaninosmcginnis: Ah yes that one!15:57
smcginnismtanino: :)15:58
*** rlrossit has joined #openstack-cinder15:59
*** rlrossit_ has joined #openstack-cinder15:59
*** Yogi1_ has joined #openstack-cinder15:59
mtaninosmcginnis: Hitachi posted driver refactor patch but unfortunately we couldn't get enough review, so I think driver etherpad might be needed to get attension for driver review.15:59
smcginnismtanino: I was just going to get a list of new drivers. If it's an existing driver not sure we want to include every patch for changes.16:00
smcginnismtanino: Do you have a link? I'll take a look.16:00
mtaninosmcginnis: this one: https://review.openstack.org/#/c/255239/16:00
*** haomaiwang has quit IRC16:01
smcginnismtanino: Cool, I'll take a look today.16:01
*** haomaiwa_ has joined #openstack-cinder16:01
mtaninosmcginnis: Thank you. Want to add this to the driver etherpad!16:01
*** apoorvad has quit IRC16:02
openstackgerritSzymon Wróblewski proposed openstack/cinder: Update replication to use versionedobjects  https://review.openstack.org/23890616:02
smcginnismtanino: We'll see. :)16:02
*** pots has quit IRC16:03
*** pots has joined #openstack-cinder16:03
*** rlrossit has quit IRC16:03
*** pots3 has joined #openstack-cinder16:04
*** fthiagogv has joined #openstack-cinder16:04
*** fthiagogv has left #openstack-cinder16:04
*** jgregor has quit IRC16:05
*** fthiagogv has joined #openstack-cinder16:05
*** pots2 has quit IRC16:06
*** baumann1 has joined #openstack-cinder16:07
*** baumann has quit IRC16:07
*** ntpttr has joined #openstack-cinder16:07
*** diablo_rojo1 has joined #openstack-cinder16:07
*** merooney has joined #openstack-cinder16:08
*** jgregor has joined #openstack-cinder16:08
*** salv-orl_ has joined #openstack-cinder16:08
*** diablo_rojo has quit IRC16:08
openstackgerritSzymon Wróblewski proposed openstack/cinder: Add missing glance_metadata field to Snapshot  https://review.openstack.org/26331716:10
*** geguileo_mobile has quit IRC16:10
*** salv-orlando has quit IRC16:11
*** shausy has joined #openstack-cinder16:12
openstackgerritAlex O'Rourke proposed openstack/cinder: LeftHand: Implement v2 replication (managed)  https://review.openstack.org/25500216:14
openstackgerritAlex O'Rourke proposed openstack/cinder: LeftHand: Implement v2 replication (unmanaged)  https://review.openstack.org/25554416:17
*** zhonghua has joined #openstack-cinder16:18
*** diogogmt has joined #openstack-cinder16:19
*** Yogi1_ has quit IRC16:20
*** zhonghua-lee has quit IRC16:20
*** diogogmt_ has joined #openstack-cinder16:22
*** nkrinner has quit IRC16:22
*** diogogmt has quit IRC16:24
*** diogogmt_ is now known as diogogmt16:24
*** knomura has quit IRC16:24
*** kfarr has joined #openstack-cinder16:25
*** isaacb has quit IRC16:29
*** jdurgin1 has joined #openstack-cinder16:30
*** belmoreira has quit IRC16:34
*** geguileo_phone has joined #openstack-cinder16:34
*** sheel has joined #openstack-cinder16:37
*** alonma has quit IRC16:38
*** alonma has joined #openstack-cinder16:39
*** garthb has quit IRC16:39
*** med_` is now known as med_16:39
*** med_ has quit IRC16:39
*** med_ has joined #openstack-cinder16:39
*** med_ is now known as med16:40
*** med is now known as med_16:40
*** akerr has quit IRC16:42
*** akerr has joined #openstack-cinder16:43
*** alonma has quit IRC16:43
*** alonma has joined #openstack-cinder16:45
*** dslev has quit IRC16:47
*** shyama has quit IRC16:48
*** alonma has quit IRC16:50
*** baumann has joined #openstack-cinder16:50
*** baumann1 has quit IRC16:52
*** alonma has joined #openstack-cinder16:52
openstackgerritSzymon Borkowski proposed openstack/cinder: Update terminate_connection to use volume object  https://review.openstack.org/26479716:53
*** diablo_rojo1 has quit IRC16:55
*** diablo_rojo has joined #openstack-cinder16:55
*** jgregor has quit IRC16:56
*** rlrossit_ has quit IRC16:57
*** alonma has quit IRC16:57
*** dslevin_ has joined #openstack-cinder16:59
*** jordanP has quit IRC16:59
*** diablo_rojo has quit IRC16:59
*** jgregor has joined #openstack-cinder16:59
*** dims has joined #openstack-cinder16:59
*** mragupat has quit IRC17:00
*** mragupat has joined #openstack-cinder17:00
*** rlrossit has joined #openstack-cinder17:00
*** haomaiwa_ has quit IRC17:01
*** asselin has joined #openstack-cinder17:01
*** 77CAAHQRX has joined #openstack-cinder17:01
*** timcl has joined #openstack-cinder17:01
*** mragupat has quit IRC17:02
*** mragupat has joined #openstack-cinder17:02
*** dims_ has quit IRC17:02
*** martyturner has quit IRC17:04
*** geguileo_phone has quit IRC17:04
*** merooeny_ has joined #openstack-cinder17:04
*** esker has quit IRC17:05
*** geguileo_phone has joined #openstack-cinder17:06
*** merooney has quit IRC17:08
*** shausy has quit IRC17:13
*** shausy has joined #openstack-cinder17:14
*** markus_z has quit IRC17:17
*** leeantho has joined #openstack-cinder17:18
*** apoorvad has joined #openstack-cinder17:18
*** mragupat_ has joined #openstack-cinder17:19
*** alonma has joined #openstack-cinder17:21
*** mragupat has quit IRC17:22
*** apoorvad has quit IRC17:23
*** garthb has joined #openstack-cinder17:24
*** alonma has quit IRC17:26
openstackgerritThang Pham proposed openstack/cinder: Add finish_volume_migration to volume object  https://review.openstack.org/25766717:27
*** diablo_rojo has joined #openstack-cinder17:28
*** mudassirlatif has joined #openstack-cinder17:30
*** alonma has joined #openstack-cinder17:34
smcginnisDriver reviews started here: https://etherpad.openstack.org/p/cinder-driver-reviews17:34
smcginnisPlease add any I missed.17:34
smcginnisbbl17:34
*** alonma has quit IRC17:38
*** geguileo_mobile has joined #openstack-cinder17:39
*** alonma has joined #openstack-cinder17:40
*** geguileo_phone has quit IRC17:42
*** alonma has quit IRC17:45
*** jistr has quit IRC17:45
*** alonma has joined #openstack-cinder17:46
*** jungleboyj has joined #openstack-cinder17:48
*** timcl has quit IRC17:51
*** alonma has quit IRC17:51
*** jdurgin1 has quit IRC17:51
*** alonma has joined #openstack-cinder17:53
*** shausy has quit IRC17:53
*** shausy has joined #openstack-cinder17:53
*** rebase has joined #openstack-cinder17:55
*** can8dnSix has joined #openstack-cinder17:56
*** alonma has quit IRC17:57
*** can8dnSix has quit IRC17:58
*** timcl has joined #openstack-cinder17:58
*** timcl has quit IRC17:59
*** martyturner has joined #openstack-cinder17:59
*** alonma has joined #openstack-cinder18:00
openstackgerritEric Harney proposed openstack/cinder: Tests: Strengthen assertFalse assertions  https://review.openstack.org/26340418:00
*** chhavi has quit IRC18:00
*** 77CAAHQRX has quit IRC18:01
*** timcl has joined #openstack-cinder18:01
*** haomaiwang has joined #openstack-cinder18:01
ildikovhemna: hi18:01
ildikovhemna: quick question. was any conclusion yesterday in the terminate_connection vs remove_exports discussion?18:02
*** timcl has quit IRC18:02
ildikovhemna: there're a few comments about concerns regarding to this on this patch: https://review.openstack.org/#/c/193135/6918:03
ildikovhemna: if you have any addition to that please comment on the review18:03
ildikovsmcginnis: ^^18:03
*** alonma has quit IRC18:04
*** shausy has quit IRC18:04
*** lprice has joined #openstack-cinder18:05
*** esker has joined #openstack-cinder18:06
*** lprice1 has joined #openstack-cinder18:07
*** alonma has joined #openstack-cinder18:07
hemnaildikov, so I think jgriffith was going to look at changing the lvm driver to basically create a new export on every attach18:07
hemnawhich would be a different way of fixing the lvm issue that my patch tries to address.18:07
ildikovis the only driver with "issues" on this?18:07
ildikovI got this comment on the patch I linked: "I don't see how this patch addresses detaching in case more than one isntance using the volume is on the same host"18:08
ildikovI don't have a good answer for this at the moment18:09
ildikovFollow up comment: "the fix in cinder https://review.openstack.org/#/c/255595/ takes care of preventing export volume from being called when there are more 'attached host' using the volume. In many cinder drivers the terminate_connection implementation removes the cinder volume connection and remove_export does nothing"18:10
*** lprice has quit IRC18:10
*** jordanP has joined #openstack-cinder18:10
*** thangp has joined #openstack-cinder18:10
*** alonma has quit IRC18:11
*** tongli_ has joined #openstack-cinder18:11
*** esker has quit IRC18:11
openstackgerritVasanthi Thirumalai proposed openstack/cinder: Removes the Violin 6000 FC and iSCSI drivers     The 6000 series of arrays are no longer supported. They are now replaced by the more recent 7000 series of arrays. Therefore, the 6000 drivers are removed from here to free up the much needed CI resources.   https://review.openstack.org/26490418:13
*** xyang has quit IRC18:13
*** esker has joined #openstack-cinder18:13
*** fthiagogv has quit IRC18:14
openstackgerritGerald McBrearty proposed openstack/cinder: Storwize: Add force flag on vdisk mapping call  https://review.openstack.org/24822518:14
*** dustins has quit IRC18:14
*** tongli has quit IRC18:14
*** bardia has joined #openstack-cinder18:14
*** garthb_ has joined #openstack-cinder18:14
*** eharney has quit IRC18:14
*** markvoelker has quit IRC18:14
*** garthb has quit IRC18:15
*** alonma has joined #openstack-cinder18:16
*** crose has quit IRC18:16
*** merooeny_ has quit IRC18:16
*** fthiagogv has joined #openstack-cinder18:17
*** lpetrut has quit IRC18:17
jgriffithhemna: ildikov yeah, I can certainly look at that18:18
jgriffithhemna: ildikov didn't know if one of you were already working that or not etc18:18
jgriffithhemna: ildikov also want to make sure we're all coordinated :)18:19
hemnaworks for me18:19
jgriffithhemna: "works for me"  meaning which?  :)18:19
jgriffithhemna: meaning I should work it up?  Or you're working it?18:19
hemnaI'm not actively working on it18:19
*** esp has joined #openstack-cinder18:19
hemnaand I don't think ildikov is either.18:19
jgriffithOk18:20
jgriffithI'll see what i can get coded up early next week18:20
*** alonma has quit IRC18:20
hemnajgriffith, ok awesome thanks!18:20
jgriffithhemna: for sure18:20
*** ndipanov has quit IRC18:22
*** e0ne has joined #openstack-cinder18:23
*** alonma has joined #openstack-cinder18:26
esphello folks could I get some eyes on this bug fix for cinder quota mgmt pls https://review.openstack.org/25375918:27
espthx in advanced :)18:27
*** eharney has joined #openstack-cinder18:27
*** esker has quit IRC18:28
*** crose has joined #openstack-cinder18:29
*** dustins has joined #openstack-cinder18:30
*** alonma has quit IRC18:30
*** mriedem has quit IRC18:32
flip214hemna: I did https://review.openstack.org/#/c/256292/3 as you recommended (DRBD transport protocol for Nova); can you help to get it accepted?18:32
ildikovjgriffith: hemna: sorry, I'm back18:33
ildikovthanks for the efforts18:33
flip214it's mostly a copy of the Fibrechannel file, like you told me in Tokyo.18:33
*** esker has joined #openstack-cinder18:33
*** timcl has joined #openstack-cinder18:34
ildikovjgriffith: hemna: I believe I fixed as many things in the Nova patches as possible, I wonder whether we should allow live migration for instance18:34
flip214any help would be greatly appreciated; thank you very much in advance (as it's late here already, and you might be out for lunch)!18:34
*** alonma has joined #openstack-cinder18:34
ildikovif there's anything like this that comes into your mind that can be an issue please let me know!18:34
jgriffithildikov: not sure what you mean by that?18:35
*** haomaiwang has quit IRC18:35
*** harlowja has quit IRC18:35
hemnaflip214, ok18:36
flip214hemna: thanks a lot!18:36
hemnaildikov, with jgriffith's change to the lvm driver it should just work™18:36
*** harlowja has joined #openstack-cinder18:36
jgriffithildikov: Oh, so currently you don't allow live migration if there are multiple attachments?18:37
jgriffithildikov: that would be bad... breaking existing functionality18:37
*** fellypefca has joined #openstack-cinder18:38
fellypefcaI found this warning here ( http://docs.openstack.org/developer/cinder/drivers.html ) . Does anyone have any tips that can help me to solve this?18:38
*** mriedem has joined #openstack-cinder18:38
*** alonma has quit IRC18:38
*** edmondsw has quit IRC18:39
ildikovjgriffith: I do, it's not blocked yet18:39
ildikovI was just wondering whether it will work properly or not18:39
fellypefcaIn the docs it should get information from cinder.openstack.common.notifier.no_op_notifier, but I'm not finding the notifier file, can anyone help?18:42
*** diablo_rojo has quit IRC18:42
*** crose has quit IRC18:43
*** afaranha has joined #openstack-cinder18:43
*** edmondsw has joined #openstack-cinder18:44
*** rlrossit has quit IRC18:44
*** rlrossit has joined #openstack-cinder18:45
*** rlrossit has quit IRC18:49
*** rlrossit has joined #openstack-cinder18:49
*** lpetrut has joined #openstack-cinder18:51
*** tongli has joined #openstack-cinder18:51
*** lpabon has joined #openstack-cinder18:52
*** tongli has quit IRC18:53
*** tongli_ has quit IRC18:53
*** lpetrut has quit IRC18:55
*** jordanP has quit IRC18:55
*** haomaiwang has joined #openstack-cinder18:56
sheelfellypefca: You can refer setup.cfg ->cinder.openstack.common.notifier.no_op_notifier = oslo_messaging.notify._impl_noop:NoOpDriver18:56
fellypefcasheel thanks :)18:58
sheelfellypefca: Its ok18:58
*** haomaiwang has quit IRC19:01
openstackgerritPhilipp Marek proposed openstack/cinder: Infrastructure to use the DRBD transport for NOVA.  https://review.openstack.org/25997319:01
hemnaildikov, jgriffith if the volume attached to the instance that's being migrated has it's own target exported, it should work as it does today just fine (famous last words)19:01
ildikovhemna: I just partially understand, but sounds like it a bit19:02
ildikovhemna: my patches should be fine now for testing19:02
ildikovI don't have any tempest tests yet for instance19:02
ildikovI also couldn't get there to at least manually playing with it, so if you or anyone around has an env and some time to play with it, that would be greate and pretty helpful19:03
*** markvoelker has joined #openstack-cinder19:03
*** afaranha has quit IRC19:03
*** sheel has quit IRC19:04
hemnacoolio, I'll see if I can find some time to do some basic testing19:04
*** markvoelker_ has joined #openstack-cinder19:05
*** rebase has quit IRC19:06
*** cfriesen__ has joined #openstack-cinder19:06
*** lpetrut has joined #openstack-cinder19:06
*** esker has quit IRC19:07
fellypefcasheel I found this warning ( http://docs.openstack.org/developer/cinder/drivers.html ) and i want to fix it, but in this class ( https://github.com/openstack/oslo.messaging/blob/master/oslo_messaging/notify/_impl_noop.py#L21 ) there is no implementation. Do you know what i can write there, just to take out the warning?19:07
*** e0ne has quit IRC19:08
*** markvoelker has quit IRC19:09
ildikovhemna: cool, tnx19:09
*** manous has quit IRC19:10
*** dslevin_ has quit IRC19:10
*** esker has joined #openstack-cinder19:11
*** manous has joined #openstack-cinder19:11
fellypefcaI found this warning ( http://docs.openstack.org/developer/cinder/drivers.html ) and i want to fix it, but in this class ( https://github.com/openstack/oslo.messaging/blob/master/oslo_messaging/notify/_impl_noop.py#L21 ) there is no implementation. Does anyone know what i can write there, just to take out the warning?19:12
*** markvoelker has joined #openstack-cinder19:14
*** markvoelker_ has quit IRC19:17
openstackgerritMudassir Latif proposed openstack/cinder: Replication V2 for Pure Storage  https://review.openstack.org/21482519:17
*** cfriesen__ has quit IRC19:22
*** cfriesen__ has joined #openstack-cinder19:28
*** dslevin_ has joined #openstack-cinder19:28
*** esker has quit IRC19:28
*** vgridnev_ has joined #openstack-cinder19:29
*** garthb__ has joined #openstack-cinder19:31
*** lpetrut has quit IRC19:31
*** mriedem1 has joined #openstack-cinder19:33
*** dims_ has joined #openstack-cinder19:33
*** dims has quit IRC19:34
*** alonma has joined #openstack-cinder19:35
*** ameade_ has joined #openstack-cinder19:35
*** ociuhandu has quit IRC19:36
*** lpetrut has joined #openstack-cinder19:37
*** stevemar_znc has joined #openstack-cinder19:37
*** mriedem has quit IRC19:38
*** garthb_ has quit IRC19:38
*** vgridnev has quit IRC19:38
*** porrua has quit IRC19:38
*** adrianofr has quit IRC19:38
*** chris_morrell has quit IRC19:38
*** kbyrne has quit IRC19:38
*** ameade has quit IRC19:38
*** briancurtin has quit IRC19:38
*** sileht has quit IRC19:38
*** wilson-liu has quit IRC19:38
*** stevemar has quit IRC19:38
*** theanalyst has quit IRC19:38
*** mriedem1 is now known as mriedem19:38
*** edtubill has quit IRC19:38
*** alonma has quit IRC19:39
*** adrianofr has joined #openstack-cinder19:40
*** adrianofr has quit IRC19:40
*** adrianofr has joined #openstack-cinder19:40
*** stevemar_znc is now known as stevemar19:40
*** ChubYann has joined #openstack-cinder19:40
*** ameade_ is now known as ameade19:40
*** theanalyst has joined #openstack-cinder19:41
*** lpetrut has quit IRC19:41
*** lpetrut has joined #openstack-cinder19:41
*** alonma has joined #openstack-cinder19:41
*** sileht has joined #openstack-cinder19:41
*** manous has quit IRC19:42
*** kbyrne has joined #openstack-cinder19:42
*** gabriel has joined #openstack-cinder19:44
*** alonma has quit IRC19:45
*** briancurtin has joined #openstack-cinder19:46
*** Yogi1 has joined #openstack-cinder19:52
*** esker has joined #openstack-cinder19:57
*** sgotliv has quit IRC19:59
*** cfriesen__ has quit IRC19:59
*** dims_ has quit IRC20:00
*** esker has quit IRC20:01
*** Yogi1 has quit IRC20:02
*** esker has joined #openstack-cinder20:02
*** esker has quit IRC20:03
*** esker has joined #openstack-cinder20:04
*** fellypefca has quit IRC20:07
*** esker has quit IRC20:08
*** angela-s has joined #openstack-cinder20:08
*** esker has joined #openstack-cinder20:09
*** haomaiwang has joined #openstack-cinder20:10
*** esker has quit IRC20:11
*** dhellmann has quit IRC20:11
*** dims has joined #openstack-cinder20:12
*** changbl has quit IRC20:15
*** abhi has quit IRC20:17
*** haomaiwang has quit IRC20:17
*** dims has quit IRC20:19
*** esker has joined #openstack-cinder20:20
*** esker has quit IRC20:21
*** lpetrut has quit IRC20:21
*** dims has joined #openstack-cinder20:22
*** lpetrut has joined #openstack-cinder20:23
*** diablo_rojo has joined #openstack-cinder20:23
*** openstackgerrit has quit IRC20:23
*** dims has quit IRC20:23
*** fernnest_ has quit IRC20:24
*** fernnest_ has joined #openstack-cinder20:25
*** openstackgerrit has joined #openstack-cinder20:25
*** dims has joined #openstack-cinder20:39
*** alonma has joined #openstack-cinder20:41
*** crose has joined #openstack-cinder20:44
*** cfriesen__ has joined #openstack-cinder20:44
*** alonma has quit IRC20:45
*** alonma has joined #openstack-cinder20:47
*** JoseMello has quit IRC20:48
*** esker has joined #openstack-cinder20:48
*** baumann has quit IRC20:49
*** baumann has joined #openstack-cinder20:49
*** alonma has quit IRC20:52
*** esker has quit IRC20:56
*** ociuhandu has joined #openstack-cinder21:00
*** changbl has joined #openstack-cinder21:01
*** raildo is now known as raildo-afk21:04
*** changbl has quit IRC21:06
*** timcl has quit IRC21:06
*** akerr has quit IRC21:07
*** mudassirlatif_ has joined #openstack-cinder21:10
*** martyturner has quit IRC21:11
*** mudassirlatif has quit IRC21:12
*** mudassirlatif_ is now known as mudassirlatif21:12
*** martyturner1 has joined #openstack-cinder21:12
*** lpabon has quit IRC21:12
*** lpetrut has quit IRC21:13
*** haomaiwang has joined #openstack-cinder21:14
scottdasmcginnis: Thanks for the recheck on the api-microversions patch, but I understand what the jenkins failures are about. I put something in the commit message for it:21:14
scottdaJenkins issues: Grenade fails due to changes in /etc/cinder/api-paste.ini Fixing Grenade will require a Grendade patch and PTL approval. gate-cinder-pylint fails due to cinder/api/versions.py:112 (noqa does not solve this)21:14
smcginnisscottda: Oh, thanks. I thought that was fixed up, but I must have been thinking of something else.21:15
scottdaThe grenade failures will require a patch to grenade which you will have to approve...21:15
scottdaI hadn't moved forward on that until the code gets more reviews, but I reckon it could be done at any time we are pretty sure this is going in.21:16
smcginnisscottda: Is there a patch in the works for that?21:16
*** gcb has joined #openstack-cinder21:16
scottdano, I'll get one up. But it should maybe wait for merge to make sure everyone is OK with the microversions patch.21:17
*** alonma has joined #openstack-cinder21:17
smcginnisscottda: I'm afraid it sitting out there with a -1 from Jenkins is going to prevent folks from reviewing it.21:17
scottdaAnd the pylint thing doesn't like 2 methods with the same name. But that's what microversions lets you do, different methods for different versions. So We'll have to figure how to deal with that.21:18
smcginnisMaybe we should bring that up in the meeting next week.21:18
scottdaok, good idea. And I can start nagging as well...21:18
*** rajinir has joined #openstack-cinder21:18
smcginnisscottda: Heh, everyone ignores pylint anyway.21:18
smcginnis;)21:18
scottdahaha...that's right. non-voting so ignore :)21:19
*** haomaiwang has quit IRC21:19
ildikovhemna: smcginnis: asked by the Nova folks I captured the limitations of multi-attach by amending the approved spec: https://review.openstack.org/#/c/264965/21:21
*** alonma has quit IRC21:21
smcginnisildikov: Just more detail needed?21:22
ildikovsmcginnis: more detail?21:22
*** chris_morrell has joined #openstack-cinder21:22
smcginnisildikov: They just want more detail captured in the spec so it clearly states what the limitations are, I mean. Right?21:23
ildikovsmcginnis: ah, yes, that's correct21:23
ildikovI captured what we discussed already, I don't know what else will come up if anything21:24
ildikovwhat's written there is already implemented in the patches, or at least that was the intention, hopefully reviews will prove... :)21:24
smcginnisildikov: Reading through it now.21:24
smcginnisildikov: Though we'll definitely want hemna's input on things.21:25
ildikovsmcginnis: sure, this is why I informed you as it would be good having both sides there to discuss21:26
ildikovsmcginnis: I'm trying my best, but let's face it, I pretty much lack of domain knowledge on both sides...21:26
*** jungleboyj has quit IRC21:28
smcginnisildikov: You're doing great! :)21:32
ildikovsmcginnis: thanks! :)21:33
*** timcl has joined #openstack-cinder21:33
hemnaildikov, ping21:36
*** esker has joined #openstack-cinder21:36
ildikovhemna: pong21:36
hemnahey21:36
hemnajust reading the patch21:36
hemnathe boot from volume21:37
ildikovyeap, shoot21:37
hemnado we also cover booting 2 vms from the same volume21:37
*** akerr has joined #openstack-cinder21:37
* hemna wonders if that would ever work, even in the case of a clustered fs ?21:37
ildikovno, that is what's disabled21:37
hemnaok21:37
hemna:)21:37
smcginnishemna: I think that would be very bad. :)21:38
ildikovI hoped it will be clear from the text, but I might re-read tomorrow with fresh brain21:38
ildikovand the reword a bit :)21:38
*** mudassirlatif has quit IRC21:39
ildikovso when 'boot_index' is 0 in a BDM for a volume that specified for boot that means the instance will be booted from that device21:39
*** mudassirlatif has joined #openstack-cinder21:39
ildikovthis case is filtered and if the volume supports mutli-attach it will fail with an error21:39
ildikovI know we could check whether the volume has attachments or not, but I read the wod racey so many times this week that it felt safer this way21:40
ildikovs/wod/word/21:40
hemnasorry, got pulled into #openstack-meeting.....21:40
hemnabrb21:40
anteayahemna: he is gus on irc21:42
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Implements Replication V2  https://review.openstack.org/24486721:43
hemnaanteaya, thank you :)21:43
anteayawelcome21:43
anteayanice impromptu update21:43
anteaya:)21:43
hemnaheh yah pulled that one out of the air21:44
hemnaoh, they want an update on that...uhh.....21:44
ildikovhemna: I call it a day now21:46
ildikovif you have further questions/comments for the patch just add a review comment or post here and I'll ping you tomorrow21:47
hemnaok sounds good21:47
hemnanight!21:47
ildikovtnx, have a nice day21:47
smcginnisildikov: Have a good one!21:47
ildikovsmcginnis: tnx :)21:47
ildikovlaters21:48
*** Yogi11 has quit IRC21:48
*** gouthamr has quit IRC21:48
*** martyturner1 has quit IRC21:50
*** jgregor has quit IRC21:50
*** boris-42 has quit IRC21:50
*** obutenko has quit IRC21:50
*** ircuser-1 has quit IRC21:50
*** guitarzan has quit IRC21:50
*** gsilvis has quit IRC21:50
*** adrianofr has quit IRC21:50
*** martyturner1 has joined #openstack-cinder21:51
*** jgregor has joined #openstack-cinder21:51
*** boris-42 has joined #openstack-cinder21:51
*** obutenko has joined #openstack-cinder21:51
*** ircuser-1 has joined #openstack-cinder21:51
*** guitarzan has joined #openstack-cinder21:51
*** gsilvis has joined #openstack-cinder21:51
*** ircuser-1 has quit IRC21:51
*** Lee1092 has quit IRC21:52
*** ircuser-1 has joined #openstack-cinder21:53
openstackgerritEric Harney proposed openstack/cinder: Delete volumes with snapshots  https://review.openstack.org/26498221:53
openstackgerritEric Harney proposed openstack/python-cinderclient: Add --cascade to volume delete  https://review.openstack.org/26498321:54
*** mudassirlatif has quit IRC21:55
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Implements Replication V2  https://review.openstack.org/24486721:55
*** thangp has quit IRC21:55
*** mudassirlatif has joined #openstack-cinder21:56
Swansonxyang1, I made the change you requested.  Plus fixed the other LOG message in the same function.  FYI  Thanks!21:56
xyang1Swanson: no problem. I'll take a look21:57
*** mudassirlatif has quit IRC21:58
*** mudassirlatif has joined #openstack-cinder22:00
*** cfriesen__ has quit IRC22:00
*** baumann has left #openstack-cinder22:01
*** xyang1 has quit IRC22:02
*** mudassirlatif has quit IRC22:05
*** mudassirlatif has joined #openstack-cinder22:05
*** vgridnev_ has quit IRC22:07
*** salv-orlando has joined #openstack-cinder22:09
*** gsilvis has quit IRC22:09
*** gsilvis has joined #openstack-cinder22:10
*** salv-orl_ has quit IRC22:11
*** dustins has quit IRC22:13
*** jungleboyj has joined #openstack-cinder22:13
*** gcb has quit IRC22:16
*** haomaiwa_ has joined #openstack-cinder22:16
*** gouthamr has joined #openstack-cinder22:19
openstackgerritVasanthi Thirumalai proposed openstack/cinder: Removes the Violin 6000 FC and iSCSI drivers  https://review.openstack.org/26490422:20
*** haomaiwa_ has quit IRC22:22
*** alonma has joined #openstack-cinder22:26
*** esker has quit IRC22:27
*** esker has joined #openstack-cinder22:28
*** isaacb has joined #openstack-cinder22:29
*** alonma has quit IRC22:31
*** mudassirlatif has quit IRC22:32
*** diablo_rojo has quit IRC22:36
*** alonma has joined #openstack-cinder22:36
*** mudassirlatif has joined #openstack-cinder22:37
scottdasmcginnis: Here's the grenade patch for api-microversions: https://review.openstack.org/#/c/264995/22:38
scottdaI tried to add you as a reviewer, but gerrit couldn't seem to find you.22:38
*** martyturner1 has quit IRC22:39
openstackgerritMikhail Khodos proposed openstack/cinder: Re-add Nexenta drivers  https://review.openstack.org/18887722:39
*** eharney has quit IRC22:39
*** esker has quit IRC22:40
*** alonma has quit IRC22:40
mtaninoscottda: I did it :)22:41
openstackgerritMerged openstack/cinder: Fix delete_snapshot error case in Pure driver  https://review.openstack.org/21794022:42
*** leeantho has quit IRC22:43
scottdamtanino: Thanks. What name did you use? Probably something tricky, like smcginnis or something....22:43
mtaninoscottda: by using "Sean McGinnis"22:44
mtaninoyeah little bit tricky...22:44
mtaninoneed full name I guess22:44
openstackgerritScott DAngelo proposed openstack/cinder: cinder-api-microversions code  https://review.openstack.org/22491022:44
scottdaahh..ok. thanks22:44
scottdamtanino: Since you are such a good problem solver, do you know how to add a dependency with the new gerrit?22:45
*** isaacb has quit IRC22:46
*** jgregor has quit IRC22:47
mtaninoscottda: oops, I haven't tried to add dependency using gerrit.22:47
mtaninoUsually I add dependency at local git tree and then commit it to gerrit.22:48
*** mriedem has quit IRC22:54
*** mudassirlatif has quit IRC22:55
*** mudassirlatif has joined #openstack-cinder22:57
*** mudassirlatif has quit IRC22:58
*** dims_ has joined #openstack-cinder23:00
*** dims has quit IRC23:01
*** mudassirlatif has joined #openstack-cinder23:01
scottdaThat's probably how it shows up. Thanks23:01
mtanino:)23:01
*** erlon has quit IRC23:06
*** rlrossit has quit IRC23:07
openstackgerritChris Morrell proposed openstack/cinder: Enable trim/discard in SolidFire's driver  https://review.openstack.org/26500423:09
*** daneyon has joined #openstack-cinder23:13
*** jungleboyj has quit IRC23:13
*** cknight has quit IRC23:16
*** daneyon_ has quit IRC23:17
*** mragupat_ has quit IRC23:20
*** chris_morrell has quit IRC23:21
openstackgerritThelo Gaultier proposed openstack/cinder: ITRI DISCO cinder driver  https://review.openstack.org/25335623:28
hemnascottda, ping23:31
*** frakt has quit IRC23:31
*** frakt has joined #openstack-cinder23:31
*** kfarr has quit IRC23:33
*** alonma has joined #openstack-cinder23:37
*** leeantho has joined #openstack-cinder23:38
*** dims_ has quit IRC23:39
*** alonma has quit IRC23:41
*** mriedem has joined #openstack-cinder23:44
*** alonma has joined #openstack-cinder23:45
*** dims_ has joined #openstack-cinder23:45
*** MIDENN_ has quit IRC23:48
*** dims_ has quit IRC23:50
*** alonma has quit IRC23:50
openstackgerritMerged openstack/os-brick: Updated from global requirements  https://review.openstack.org/26291123:51
*** [1]Thelo has joined #openstack-cinder23:54
*** pratap has joined #openstack-cinder23:56
*** Thelo has quit IRC23:56
*** [1]Thelo is now known as Thelo23:56
*** garthb__ has quit IRC23:57
*** garthb__ has joined #openstack-cinder23:57
*** mdenny has joined #openstack-cinder23:58
*** pratap_ has joined #openstack-cinder23:58

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