Tuesday, 2018-03-20

openstackgerritMerged openstack/cinder master: Add support for unmanaging snapshot of rbd driver  https://review.openstack.org/55074400:05
openstackgerritMerged openstack/cinder master: Fix incorrect usage of assertEqual() method in RBD unit tests  https://review.openstack.org/55422100:05
*** markstur has quit IRC00:08
*** ircuser-1 has joined #openstack-cinder00:12
*** wanghao has quit IRC00:21
*** wanghao has joined #openstack-cinder00:21
*** itlinux has joined #openstack-cinder00:23
*** liverpooler has joined #openstack-cinder00:30
*** Dinesh_Bhor has joined #openstack-cinder00:37
*** elmaciej_ has quit IRC00:46
*** lifeless_ has joined #openstack-cinder00:46
*** lifeless has quit IRC00:46
*** elmaciej has joined #openstack-cinder00:46
*** itlinux has quit IRC00:51
*** Dinesh_Bhor has quit IRC00:57
*** Dinesh_Bhor has joined #openstack-cinder00:57
openstackgerritJeremy Zhang proposed openstack/cinder master: RBD snapshot needs to be protected after being managed  https://review.openstack.org/55421500:58
*** odyssey4me has quit IRC00:59
*** odyssey4me has joined #openstack-cinder00:59
*** Dinesh_Bhor has quit IRC00:59
openstackgerritzhufl proposed openstack/cinder master: Fix typos in release notes  https://review.openstack.org/55439001:01
*** Dinesh_Bhor has joined #openstack-cinder01:02
*** elmaciej has quit IRC01:07
*** yongwc has joined #openstack-cinder01:09
*** wxy has joined #openstack-cinder01:10
openstackgerritJeremy Zhang proposed openstack/cinder master: RBD: support to get manageable snapshots  https://review.openstack.org/55293601:24
*** Dinesh_Bhor has quit IRC01:49
*** Dinesh_Bhor has joined #openstack-cinder01:52
*** Dinesh_Bhor has quit IRC01:52
openstackgerritMerged openstack/cinder master: Fix policy in code docs for extended_snapshot_attributes  https://review.openstack.org/55407101:59
openstackgerritMerged openstack/cinder master: nimble: update documentation for supportability  https://review.openstack.org/55264601:59
openstackgerrityenai proposed openstack/cinder master: Fix assignment error in image_utils  https://review.openstack.org/55439402:00
*** Dinesh_Bhor has joined #openstack-cinder02:01
*** dikonoor has joined #openstack-cinder02:02
*** Dinesh_Bhor has quit IRC02:02
*** Dinesh_Bhor has joined #openstack-cinder02:08
*** dikonoor has quit IRC02:11
*** boungenannter has joined #openstack-cinder02:14
tommylikehuhey smcginnis, eharney, jungleboyj could you take a look at this patch? https://review.openstack.org/#/c/548650/ thanks!02:16
*** wes_dillingham has quit IRC02:19
*** kaisers_ has joined #openstack-cinder02:27
*** kaisers has quit IRC02:29
*** dave-mccowan has quit IRC02:31
*** hoangcx has joined #openstack-cinder02:31
*** r-daneel has joined #openstack-cinder02:32
*** r-daneel_ has joined #openstack-cinder02:34
openstackgerrityenai proposed openstack/cinder master: Fix parameter error in image_utils  https://review.openstack.org/55314002:35
*** r-daneel has quit IRC02:36
*** r-daneel_ is now known as r-daneel02:36
*** links has joined #openstack-cinder02:40
*** itlinux has joined #openstack-cinder02:42
*** salv-orl_ has joined #openstack-cinder02:50
*** salv-orlando has quit IRC02:52
*** hoangcx has quit IRC02:55
*** hoangcx has joined #openstack-cinder03:00
*** psachin has joined #openstack-cinder03:01
openstackgerritPeng Wang proposed openstack/cinder master: DS8K: correct in-use replication vol status  https://review.openstack.org/53996003:04
openstackgerritFabo Yi proposed openstack/cinder master: Group model: ensure 'volume_types' are mapped to VolumeTypes  https://review.openstack.org/55422903:04
*** bkopilov has quit IRC03:08
*** itlinux has quit IRC03:49
*** dalgaaf has quit IRC03:51
*** dalgaaf has joined #openstack-cinder03:52
*** armaan has quit IRC04:17
*** armaan has joined #openstack-cinder04:18
*** armaan has quit IRC04:25
*** armaan has joined #openstack-cinder04:26
*** vivsoni__ has joined #openstack-cinder04:29
*** andreas_s has joined #openstack-cinder04:29
*** andreas_s has quit IRC04:34
*** wanghao has quit IRC04:35
*** yongwc has quit IRC04:38
*** yongwc has joined #openstack-cinder04:39
*** bkopilov has joined #openstack-cinder04:41
*** chhagarw has joined #openstack-cinder04:49
*** vivsoni__ has quit IRC04:51
*** wanghao has joined #openstack-cinder04:51
*** vivsoni__ has joined #openstack-cinder04:53
*** pgadiya has joined #openstack-cinder04:54
*** wangyong has joined #openstack-cinder04:57
*** Dinesh__Bhor has joined #openstack-cinder04:58
*** Dinesh_Bhor has quit IRC04:58
*** yongwc has quit IRC05:00
*** lpetrut has joined #openstack-cinder05:02
*** wangyong has quit IRC05:05
*** wangyong has joined #openstack-cinder05:05
*** imacdonn has quit IRC05:14
*** imacdonn has joined #openstack-cinder05:14
*** nikeshm has joined #openstack-cinder05:18
nikeshmhi guys05:18
*** jmlowe has quit IRC05:18
nikeshmgetting an error in Kaminario CI while running devstack05:19
nikeshmhttp://54.209.116.144/28/554328/2/check/kaminario-dsvm-tempest-full-FC/1cf3902/logs/devstack-early.txt.gz05:19
nikeshm+ lib/cinder:create_volume_types:545       :   openstack --os-region-name=RegionOne volume type create --property volume_backend_name=kaminario-FC kaminario-FC Policy doesn't allow volume_extension:types_manage to be performed. (HTTP 403) (Request-ID: req-0f3c0cb7-1a19-445a-8bb4-8a95d8e9ad8d)05:19
*** Aditya_ has joined #openstack-cinder05:21
*** wangyong has quit IRC05:24
*** wangyong has joined #openstack-cinder05:24
*** vivsoni__ has quit IRC05:25
*** hoangcx has quit IRC05:26
nikeshmis any other getting this issue05:26
*** hoangcx has joined #openstack-cinder05:29
*** armaan has quit IRC05:31
*** armaan has joined #openstack-cinder05:31
*** jmlowe has joined #openstack-cinder05:37
*** jmlowe has quit IRC05:42
*** armaan has quit IRC05:53
*** armaan has joined #openstack-cinder05:54
*** lpetrut has quit IRC06:00
*** armaan has quit IRC06:02
*** dineshbhor__ has joined #openstack-cinder06:07
*** Dinesh__Bhor has quit IRC06:09
*** wanghao has quit IRC06:09
*** lpetrut has joined #openstack-cinder06:12
*** masber has quit IRC06:12
*** Aditya_ has quit IRC06:18
*** e0ne has joined #openstack-cinder06:21
*** Jack_Iv has joined #openstack-cinder06:22
*** Jack_Iv has quit IRC06:26
*** psachin has quit IRC06:27
*** masber has joined #openstack-cinder06:44
*** gaoyan has joined #openstack-cinder06:46
*** psachin has joined #openstack-cinder06:46
*** masber has quit IRC06:47
*** lkuchlan has joined #openstack-cinder06:47
*** masber has joined #openstack-cinder06:47
openstackgerritPeter Wang proposed openstack/cinder master: Unity: Enable ssl verification  https://review.openstack.org/55200506:56
*** lpetrut has quit IRC06:57
*** lhx_ has joined #openstack-cinder06:57
*** dineshbhor__ has quit IRC07:00
*** sar has joined #openstack-cinder07:08
*** abhishekk has joined #openstack-cinder07:09
*** rcernin has quit IRC07:23
*** rcernin has joined #openstack-cinder07:24
*** rcernin has quit IRC07:24
*** armaan has joined #openstack-cinder07:25
*** lhx_ has quit IRC07:26
*** alexchadin has joined #openstack-cinder07:30
*** salv-orl_ has quit IRC07:32
*** salv-orlando has joined #openstack-cinder07:33
*** salv-orlando has quit IRC07:33
*** salv-orlando has joined #openstack-cinder07:33
*** lhx_ has joined #openstack-cinder07:34
*** pcaruana has joined #openstack-cinder07:34
*** alexchadin has quit IRC07:36
*** alexchadin has joined #openstack-cinder07:37
*** erlon has quit IRC07:52
*** tesseract has joined #openstack-cinder08:00
*** namnh has joined #openstack-cinder08:01
*** armaan has quit IRC08:02
*** andreas_s has joined #openstack-cinder08:02
*** AlexeyAbashkin has joined #openstack-cinder08:04
*** lkuchlan_ has joined #openstack-cinder08:05
*** lkuchlan has quit IRC08:06
*** _pewp_ has quit IRC08:11
openstackgerrityixuan zhang proposed openstack/cinder master: Storwize: modify hyperswap host_site configuration  https://review.openstack.org/53311308:13
*** armaan has joined #openstack-cinder08:13
*** _pewp_ has joined #openstack-cinder08:15
openstackgerritTommyLike proposed openstack/cinder master: [WIP] Support image signature verification  https://review.openstack.org/55416608:16
*** gkadam has joined #openstack-cinder08:22
*** gaoyan has quit IRC08:28
*** _alastor_ has quit IRC08:30
*** _alastor_ has joined #openstack-cinder08:31
*** e0ne has quit IRC08:33
*** moshele has joined #openstack-cinder08:33
*** _pewp_ has quit IRC08:41
*** armaan has quit IRC08:43
*** armaan has joined #openstack-cinder08:43
*** _pewp_ has joined #openstack-cinder08:47
*** lpetrut has joined #openstack-cinder08:49
*** lpetrut_ has joined #openstack-cinder08:50
*** lpetrut has quit IRC08:50
openstackgerritNeha Alhat proposed openstack/cinder master: V3 jsonschema validation: types_extra_specs  https://review.openstack.org/55445808:51
openstackgerritPooja Jadhav proposed openstack/cinder master: V3 jsonschema validation: Clusters  https://review.openstack.org/52818208:55
*** alexchadin has quit IRC08:58
*** alexchadin has joined #openstack-cinder08:59
*** jiaopengju has joined #openstack-cinder09:00
*** andreas_s has quit IRC09:01
*** andreas_s has joined #openstack-cinder09:01
*** jiaopengju has quit IRC09:02
*** jiaopengju_ is now known as jiaopengju09:02
*** alexchadin has quit IRC09:09
*** andreas_s has quit IRC09:10
*** alexchadin has joined #openstack-cinder09:10
*** andreas_s has joined #openstack-cinder09:13
*** andreas_s has quit IRC09:22
*** andreas_s has joined #openstack-cinder09:22
*** _pewp_ has quit IRC09:24
*** _pewp_ has joined #openstack-cinder09:26
openstackgerritNam Nguyen Hoai proposed openstack/cinder master: [DO NOT REVIEW][DO NOT MERGE][TEST]  https://review.openstack.org/54967209:31
*** _pewp_ has quit IRC09:33
*** abhishekk has quit IRC09:35
*** andreas_s has quit IRC09:36
*** andreas_s has joined #openstack-cinder09:37
*** armaan has quit IRC09:38
*** armaan has joined #openstack-cinder09:39
*** _pewp_ has joined #openstack-cinder09:40
*** abhishekk has joined #openstack-cinder09:48
*** josecastroleon has quit IRC09:49
*** josecastroleon has joined #openstack-cinder09:50
*** kaisers_ has quit IRC09:53
*** liverpooler has quit IRC09:54
*** e0ne has joined #openstack-cinder09:55
*** andreas_s has quit IRC09:56
*** andreas_s has joined #openstack-cinder09:57
*** lkuchlan_ has quit IRC10:00
*** alexchadin has quit IRC10:04
*** andreas_s has quit IRC10:07
*** andreas_s has joined #openstack-cinder10:07
*** namnh has quit IRC10:10
*** kaisers has joined #openstack-cinder10:21
*** mvk has quit IRC10:31
*** masber has quit IRC10:43
*** armaan has quit IRC10:46
*** elmaciej has joined #openstack-cinder10:46
*** mvk has joined #openstack-cinder10:46
*** chhagarw has quit IRC11:01
*** rcernin has joined #openstack-cinder11:02
*** josecastroleon has quit IRC11:06
*** rcernin has quit IRC11:06
*** masber has joined #openstack-cinder11:08
*** masuberu has joined #openstack-cinder11:12
*** masber has quit IRC11:14
*** tpsilva has joined #openstack-cinder11:20
*** pcaruana has quit IRC11:23
*** psachin has quit IRC11:28
*** psachin has joined #openstack-cinder11:33
*** abhishekk has quit IRC11:44
*** lkuchlan_ has joined #openstack-cinder11:44
nikeshmthere is no policy.json inside /etc/cinder11:44
nikeshmin my CI11:44
nikeshmso it is failing11:45
nikeshmany guess11:45
nikeshmwhy it is not copying11:45
nikeshmonly neutron and glance policy file is present11:45
*** armaan has joined #openstack-cinder11:48
*** abishop has joined #openstack-cinder11:49
*** pchavva has joined #openstack-cinder11:52
*** pcaruana has joined #openstack-cinder11:55
*** chhagarw has joined #openstack-cinder12:07
*** josecastroleon has joined #openstack-cinder12:09
*** odyssey4me has quit IRC12:11
*** odyssey4me has joined #openstack-cinder12:11
*** wes_dillingham has joined #openstack-cinder12:14
nikeshmhttps://bugs.launchpad.net/openstack-gate/+bug/175712412:17
openstackLaunchpad bug 1757124 in OpenStack-Gate "policy issue in Kaminario CI" [Undecided,New]12:17
*** edmondsw has joined #openstack-cinder12:17
chhagarwgeguileo: need to discuss on the review comments https://review.openstack.org/#/c/552454/312:18
geguileochhagarw: sure12:18
chhagarwon x84, while running the rescan_hosts node_name is used to scan the device12:19
chhagarwfor ppc64, we don't use hba node_name rather we use target_wwn ports12:20
chhagarwso the implementation is different on intel and power12:20
geguileochhagarw: could you point me to the code where you see that difference? r:-???12:21
* geguileo must be blind :-(12:21
chhagarwhttps://github.com/openstack/os-brick/blob/master/os_brick/initiator/linuxfc.py#L9212:22
chhagarwhttps://github.com/openstack/os-brick/blob/master/os_brick/initiator/linuxfc.py#L32612:23
chhagarwsee these 2 lines,12:23
geguileochhagarw: that's logging, not how the method works12:24
geguileoyou don't use that, you just log that info12:24
chhagarwyeah i see now, checking why it was put and how target_wwn is getting used.12:25
chhagarwlet me re-check once if not used will remove it12:26
openstackgerritShay Halsband proposed openstack/cinder master: XtremIO: support multiattach  https://review.openstack.org/55452412:26
geguileochhagarw: OK, feel free to ping me again to discuss it  :-)12:26
chhagarwsure thanks12:27
*** bkopilov has quit IRC12:32
*** openstackgerrit has quit IRC12:33
*** openstackgerrit has joined #openstack-cinder12:37
openstackgerritTommyLike proposed openstack/cinder master: Support availability-zone type  https://review.openstack.org/55224312:37
*** r-daneel has quit IRC12:41
*** AlexeyAbashkin has quit IRC12:48
nikeshmtommylikehu : hi12:57
*** lhx_ has quit IRC13:02
*** abhishekk has joined #openstack-cinder13:05
*** AlexeyAbashkin has joined #openstack-cinder13:10
*** mriedem has joined #openstack-cinder13:18
*** psachin has quit IRC13:21
*** dustins has joined #openstack-cinder13:22
openstackgerritChhavi Agarwal proposed openstack/os-brick master: Incorporate the connection_properties input for PPC64  https://review.openstack.org/55245413:23
*** armaan has quit IRC13:24
chhagarwgeguileo: updated the review can u check, we no longer needs a PPC64 class now13:25
*** armaan has joined #openstack-cinder13:25
chhagarwthanks for bringing this,13:25
*** knylander has joined #openstack-cinder13:28
*** pgadiya has quit IRC13:30
*** r-daneel has joined #openstack-cinder13:32
*** Cyb3rManiak has quit IRC13:35
*** Cyb3rManiak has joined #openstack-cinder13:35
*** r-daneel has quit IRC13:36
*** Cyb3rManiak has quit IRC13:42
*** Cyb3rManiak has joined #openstack-cinder13:43
*** wangyong has quit IRC13:44
*** bkopilov has joined #openstack-cinder13:51
*** links has quit IRC13:51
*** SamYaple has quit IRC13:52
*** jmlowe has joined #openstack-cinder13:58
*** gouthamr has joined #openstack-cinder14:00
*** eharney has joined #openstack-cinder14:03
*** finucannot is now known as stephenfin14:07
*** SamYaple has joined #openstack-cinder14:11
*** pcaruana has quit IRC14:21
*** eharney_ has joined #openstack-cinder14:23
*** eharney has quit IRC14:24
*** armaan has quit IRC14:25
*** armaan has joined #openstack-cinder14:25
*** prometheanfire has left #openstack-cinder14:25
*** felipemonteiro has joined #openstack-cinder14:27
*** pcaruana has joined #openstack-cinder14:28
*** felipemonteiro_ has joined #openstack-cinder14:29
*** felipemonteiro has quit IRC14:32
*** dustins_ has joined #openstack-cinder14:33
openstackgerritSean McGinnis proposed openstack/python-cinderclient master: Correct errors in snapshot-manageable-list help text  https://review.openstack.org/55457014:33
*** eharney_ is now known as eharnye14:34
*** eharnye is now known as eharney14:34
*** gouthamr has quit IRC14:35
*** DHE has joined #openstack-cinder14:35
*** dustins has quit IRC14:35
*** hemna_ has joined #openstack-cinder14:36
*** sar has quit IRC14:40
*** liverpooler has joined #openstack-cinder14:44
*** elmaciej has quit IRC14:45
*** itlinux has joined #openstack-cinder14:50
*** eharney_ has joined #openstack-cinder14:50
*** gouthamr has joined #openstack-cinder14:50
*** eharney has quit IRC14:51
*** wxy| has joined #openstack-cinder14:54
*** r-daneel has joined #openstack-cinder14:55
*** armaan has quit IRC14:57
*** armaan has joined #openstack-cinder14:57
*** jistr is now known as jistr|mtg15:01
*** Swanson has quit IRC15:09
*** Swanson has joined #openstack-cinder15:10
*** moshele has quit IRC15:12
*** eharney_ is now known as eharney15:16
openstackgerritAlan Bishop proposed openstack/cinder master: Sync snapshot's encryption_key_id with volume's value  https://review.openstack.org/55349515:18
*** abhishekk has quit IRC15:18
*** armaan has quit IRC15:19
*** armaan has joined #openstack-cinder15:19
openstackgerritAlexey Khodos proposed openstack/cinder master: Add NexentaEdge drivers  https://review.openstack.org/51314315:26
openstackgerritGorka Eguileor proposed openstack/cinder master: Avoid second restart on offline upgrades  https://review.openstack.org/55377215:26
*** dave-mccowan has joined #openstack-cinder15:29
*** crose has joined #openstack-cinder15:30
jgriffithmtreinish: ping15:33
*** jistr|mtg is now known as jistr15:36
openstackgerritTom Swanson proposed openstack/cinder stable/queens: Dell EMC SC: ISCSI initialize_connection fixes.  https://review.openstack.org/55459215:37
*** voelzmo has joined #openstack-cinder15:39
openstackgerritMerged openstack/cinder master: Dell EMC SC: ISCSI initialize_connection fixes.  https://review.openstack.org/55293315:45
SwansonOMG!15:45
hemna_?15:45
SwansonTHE MERGE! IT IS BEAUTIFUL! HAHAHAHAHA!15:46
*** crose has quit IRC15:48
* jgriffith reverts15:48
*** felipemonteiro_ has quit IRC15:48
SwansonOh, my.15:48
*** felipemonteiro_ has joined #openstack-cinder15:49
hemna_:)15:50
e0ne :)15:56
*** salv-orlando has quit IRC15:56
*** salv-orlando has joined #openstack-cinder15:57
*** salv-orlando has quit IRC15:57
*** salv-orlando has joined #openstack-cinder15:57
nikeshmhi guys16:02
nikeshmis any one facing this in their CI16:03
nikeshmhttps://bugs.launchpad.net/openstack-gate/+bug/175712416:03
openstackLaunchpad bug 1757124 in OpenStack-Gate "policy issue in Kaminario CI" [Undecided,New]16:03
nikeshmi am facing it from past 20+ hours16:04
jgriffithnikeshm: I'm not, but I'm having issues with testr16:04
*** zhaochao has quit IRC16:05
jgriffithnikeshm: are you using admin creds?16:05
jgriffithalthough I guess you're letting tempest sort all that out, sorry don't know what's up there16:05
*** felipemonteiro_ has quit IRC16:06
nikeshmjgriffith:  i logged in inside CI VM and ran "source openrc admin admin" and tried failed command but got same policy issue , then i copied manually policy.json file inside /etc/cinder and then the error was not coming16:07
jgriffithnikeshm: well, the policy in code stuff landed a while ago, maybe something changed that affected that for you?16:10
jgriffithnikeshm: the only commit I see so far is doc changes to snapshots rules16:11
nikeshmjgriffith : my bad all failures are on driverfixes branch  :)16:14
eharneydo we log anything at debug level that would say more about why that call was rejected?16:14
openstackgerritTom Swanson proposed openstack/cinder stable/queens: Dell EMC SC: ISCSI initialize_connection fixes.  https://review.openstack.org/55459216:14
*** voelzmo has quit IRC16:17
*** masuberu has quit IRC16:18
*** wxy|_ has joined #openstack-cinder16:22
*** andreas_s has quit IRC16:22
*** wxy| has quit IRC16:22
*** lkuchlan_ has quit IRC16:23
*** andreas_s has joined #openstack-cinder16:26
*** salv-orlando has quit IRC16:28
*** salv-orlando has joined #openstack-cinder16:28
*** salv-orlando has quit IRC16:33
*** armaan has quit IRC16:35
*** andreas_s has quit IRC16:36
*** armaan has joined #openstack-cinder16:36
*** gkadam has quit IRC16:47
*** mvk has quit IRC16:49
*** moshele has joined #openstack-cinder16:50
jgriffitheharney: it's kinda odd; looking at his logs I don't see any errors/debug messages related to policy in c-api at all16:51
*** kaisers has quit IRC16:53
*** AlexeyAbashkin has quit IRC16:53
*** kaisers has joined #openstack-cinder16:53
*** felipemonteiro has joined #openstack-cinder16:55
openstackgerritHelen Walsh proposed openstack/cinder master: VMAX docs - miscellaneous clarifications  https://review.openstack.org/55462816:57
*** masuberu has joined #openstack-cinder16:59
*** mvk has joined #openstack-cinder17:03
*** felipemonteiro_ has joined #openstack-cinder17:04
nikeshmjgriffith eharney:  my bad, i closed the issue, since CI was failing on driverfixes branch17:08
*** felipemonteiro has quit IRC17:08
*** felipemonteiro has joined #openstack-cinder17:08
nikeshmwhich might be expected17:09
nikeshmwhile provides some fixes, driverfixes branch was left in my layout.yaml17:09
nikeshms/provides/providing17:10
*** felipemonteiro_ has quit IRC17:11
nikeshmand somehow many driverfixes patch were picked up by my CI simultaneously17:11
*** knylander has quit IRC17:12
*** wxy|_ has quit IRC17:12
nikeshmand it diverted my attention on knowing the failure and i did not check the branch17:12
nikeshmjust to confirm that we can not test driverfixes branch in CI right17:26
nikeshmwe have to test driverfixes manually r817:26
*** vivsoni__ has joined #openstack-cinder17:26
*** liverpooler has quit IRC17:27
*** salv-orlando has joined #openstack-cinder17:29
*** lpetrut_ has quit IRC17:30
*** masuberu has quit IRC17:33
*** salv-orlando has quit IRC17:34
*** e0ne has quit IRC17:35
*** moshele has quit IRC17:48
*** dustins has joined #openstack-cinder17:58
*** dustins_ has quit IRC17:58
*** moshele has joined #openstack-cinder18:03
smcginnisYes, please do not run CI on driverfixes. There is no guarantee that will be in a runable state by itself and we don't need the extra CI noise.18:04
chhagarwgeguileo: I replied to your comments, please have a look when u get some time.18:04
smcginnisSince pretty much all that are configured to run on those branches just fail repeatedly.18:04
*** salv-orlando has joined #openstack-cinder18:07
geguileochhagarw: I don't understand your answers18:09
geguileochhagarw: you are removing the _get_possible_volume_paths methods which doesn't seem related to that patch18:09
geguileosince it's not a method for which we have removed all existing calls, those calls didn't exist in the first place18:10
geguileoand if you are doing cleanup you can also remove __init__ and set_execute methods in that class18:10
chhagarwgeguileo: _get_possible_volume_paths was added earlier to pass 'target_wwn', with the latest change now its not required, as we are not using it anywhere.18:10
geguileochhagarw: but we were not using it anywhere before your patch18:11
chhagarwearlier its been used in the linuxfc for the loop and the log message18:11
geguileochhagarw: by earlier you mean some time ago?18:12
geguileobecause you are not removing any calls to that method18:13
chhagarwearlier means before my patch, it was there used in the LinuxFCPPC6418:13
*** lpetrut has joined #openstack-cinder18:13
*** AlexeyAbashkin has joined #openstack-cinder18:14
chhagarw if 'target_wwn' not in hba.keys()18:14
geguileochhagarw: if that's the case, could you show me where in your patch you removed the call to the method?18:14
geguileochhagarw: that has nothing to do with the _get_possible_volume_paths method you are removing18:14
*** AlexeyAbashkin has quit IRC18:14
geguileochhagarw: you are referring to the linuxfc.py change and I'm talking about the fibre_channel_ppc64.py change18:15
*** AlexeyAbashkin has joined #openstack-cinder18:15
*** armaan has quit IRC18:15
*** armaan has joined #openstack-cinder18:16
chhagarwwith the new change since now I am not using hba['target_wwn'] in linuxfc.py, I no longe need to override _get_possible_volume_paths18:16
openstackgerritTom Swanson proposed openstack/cinder stable/pike: Dell EMC SC: ISCSI initialize_connection fixes.  https://review.openstack.org/55465618:17
imacdonnchhagarw: where were you previously overriding _get_possible_volume_paths()? Don't any such change in your patchset18:19
chhagarwits in fibre_channel_ppc64.py18:21
imacdonnchhagarw: that's where the method is, but where was it being called ?18:22
geguileochhagarw: I see now how you were overwriting it, but still...18:22
*** diablo_rojo has quit IRC18:22
*** harlowja has joined #openstack-cinder18:23
chhagarwits been called by fibre_channel.py18:23
imacdonnchhagarw: oh, wait, you're falling back to the parent class' implementation18:23
geguileochhagarw: Before your patch it was looking for ['node_name'] if present and ['target_wwn'] otherwise18:23
chhagarwyes18:23
geguileochhagarw: and now we always look for ['target_wwn']18:23
chhagarwbut now its not requied any more because new changes takes care of target_wwn18:23
geguileochhagarw: ooooooooooooooooooooooooooh, I see now18:24
geguileochhagarw: that driver was doing some nasty stuff and using the HBA information to pass data18:24
geguileochhagarw: since we now pass connection_properties we no longer need to do that because we have the information there18:25
geguileoman, am I slow today or what?18:25
imacdonngeguileo: speaking of which, don't go away yet - I have another topic for you after this one ;)18:26
chhagarwsure18:26
geguileochhagarw: OK, changed my vote, sorry for the confussion18:27
Swansonhttps://review.openstack.org/#/c/554592/18:27
chhagarwno problem. thanks waiting for another +218:28
*** vivsoni__ has quit IRC18:28
smcginnisSwanson: It's best not to cherry pick to an older branch until it has merged to the newer branch.18:29
imacdonngeguileo: Need an update on https://bugs.launchpad.net/os-brick/+bug/1756206 - show-stopper for me upgrading to Queens18:29
openstackLaunchpad bug 1756206 in os-brick "iSCSI boot volume connection fails when multipath enabled" [Undecided,New]18:29
geguileoimacdonn: trying to find the time to actually test it, sorry18:30
Swansonsmcginnis, I agree. My balls do not.18:30
imacdonngeguileo: OK. I'm going to have to start digging into it too. Let me know if you get to it.18:31
geguileoimacdonn: I'll update the LP bug  you created if I figure this one out18:32
*** dave-mccowan has quit IRC18:32
imacdonngeguileo: thanks18:32
geguileoimacdonn: and sorry for the delay18:32
smcginnisSwanson: Are they in a vise. :D18:32
imacdonngeguileo: we're all busy, I'm sure ;)18:32
*** diablo_rojo has joined #openstack-cinder18:37
*** AlexeyAbashkin has quit IRC18:44
*** andreas_s has joined #openstack-cinder18:45
*** dave-mccowan has joined #openstack-cinder18:45
*** andreas_s has quit IRC18:50
*** chhagarw has quit IRC19:02
*** dave-mccowan has quit IRC19:03
*** AlexeyAbashkin has joined #openstack-cinder19:03
*** psachin has joined #openstack-cinder19:28
abishopreminder that https://review.openstack.org/554347 unblocks patches on driverfixes/ocata19:33
eharneyyeah, it would be great to merge that :)19:34
abishopthen https://review.openstack.org/554355 for driverfixes/newton19:34
*** dave-mccowan has joined #openstack-cinder19:36
*** diablo_rojo has quit IRC19:44
*** psachin has quit IRC19:44
openstackgerritLucian Petrut proposed openstack/cinder master: Fix vhd image conversion regression  https://review.openstack.org/55439419:45
*** wes_dillingham has quit IRC19:46
*** psachin has joined #openstack-cinder19:47
*** AlexeyAbashkin has quit IRC19:51
*** liverpooler has joined #openstack-cinder19:51
*** salv-orlando has quit IRC19:51
*** salv-orlando has joined #openstack-cinder19:52
*** e0ne has joined #openstack-cinder19:53
*** tesseract has quit IRC19:56
*** salv-orlando has quit IRC19:56
*** armaan has quit IRC19:57
*** sar has joined #openstack-cinder20:01
*** masber has joined #openstack-cinder20:02
*** dave-mccowan has quit IRC20:03
*** dave-mccowan has joined #openstack-cinder20:04
*** armaan has joined #openstack-cinder20:05
*** psachin has quit IRC20:08
*** psachin has joined #openstack-cinder20:09
*** psachin has quit IRC20:14
*** dave-mccowan has quit IRC20:31
*** eharney has quit IRC20:31
*** masber has quit IRC20:32
*** openstackgerrit has quit IRC20:33
*** openstackgerrit has joined #openstack-cinder20:35
openstackgerritMerged openstack/os-brick master: Incorporate the connection_properties input for PPC64  https://review.openstack.org/55245420:35
*** csaikia has joined #openstack-cinder20:36
*** csaikia has quit IRC20:41
jgriffith(┛ಠ_ಠ)┛彡┻━┻20:41
*** csaikia has joined #openstack-cinder20:42
smcginnisI'm sure that looks a lot cooler without unicode blocks. :)20:42
jgriffithsmcginnis: :)20:42
jgriffithsmcginnis: It looks 'the coolest'20:43
*** csaikia has quit IRC20:47
*** csaikia has joined #openstack-cinder20:48
*** edmondsw has quit IRC20:51
*** felipemonteiro_ has joined #openstack-cinder20:51
*** salv-orlando has joined #openstack-cinder20:52
*** felipemonteiro has quit IRC20:54
*** salv-orlando has quit IRC20:57
*** kaisers has quit IRC20:58
*** dave-mccowan has joined #openstack-cinder21:00
*** pchavva has quit IRC21:02
*** lkuchlan_ has joined #openstack-cinder21:02
*** csaikia has quit IRC21:04
*** lkuchlan_ has quit IRC21:04
*** lkuchlan_ has joined #openstack-cinder21:04
openstackgerritMerged openstack/cinder stable/queens: Dell EMC SC: ISCSI initialize_connection fixes.  https://review.openstack.org/55459221:07
*** jmlowe has quit IRC21:08
*** lkuchlan_ has quit IRC21:09
*** jmlowe has joined #openstack-cinder21:11
*** crose has joined #openstack-cinder21:13
openstackgerritTom Swanson proposed openstack/cinder stable/pike: Dell EMC SC: API 4.0 incompatibilities  https://review.openstack.org/55470121:19
*** AlexeyAbashkin has joined #openstack-cinder21:19
*** abishop has quit IRC21:20
*** diablo_rojo has joined #openstack-cinder21:21
*** wes_dillingham has joined #openstack-cinder21:22
*** AlexeyAbashkin has quit IRC21:23
openstackgerritiain MacDonnell proposed openstack/os-brick master: Accept ISCSI_ERR_NO_OBJS_FOUND from iscsiadm when getting node info  https://review.openstack.org/55470421:24
*** kaisers has joined #openstack-cinder21:27
*** itlinux has quit IRC21:31
*** kaisers has quit IRC21:32
*** moshele has quit IRC21:32
*** e0ne has quit IRC21:33
*** kaisers has joined #openstack-cinder21:39
*** lpetrut has quit IRC21:44
*** josecastroleon has quit IRC21:47
*** gouthamr has quit IRC21:49
*** salv-orlando has joined #openstack-cinder21:53
openstackgerritTomcsi proposed openstack/os-brick master: Nova is configured to use multipath and we're trying to start an instance with attached volumes.  https://review.openstack.org/55471821:55
*** sar has quit IRC22:09
*** rcernin has joined #openstack-cinder22:13
*** diablo_rojo has quit IRC22:13
*** diablo_rojo has joined #openstack-cinder22:15
*** dave-mccowan has quit IRC22:17
*** felipemonteiro_ has quit IRC22:18
*** AlexeyAbashkin has joined #openstack-cinder22:19
*** liverpooler has quit IRC22:23
*** AlexeyAbashkin has quit IRC22:23
*** dustins has quit IRC22:24
*** dave-mccowan has joined #openstack-cinder22:28
*** crose has quit IRC22:41
*** andreas_s has joined #openstack-cinder22:47
openstackgerritMerged openstack/cinder stable/queens: Fix leftovers after backup abort  https://review.openstack.org/55249522:47
*** armaan has quit IRC22:48
*** andreas_s has quit IRC22:51
*** harlowja has quit IRC22:57
*** tpsilva has quit IRC22:57
*** masber has joined #openstack-cinder23:03
*** masuberu has joined #openstack-cinder23:04
imacdonn2018-03-20 22:16:15.059783 | primary |       File "tempest/api/volume/test_volumes_extend.py", line 181, in test_extend_attached_volume23:06
imacdonn2018-03-20 22:16:15.059829 | primary |         'request_id': action['request_id']})23:06
imacdonn2018-03-20 22:16:15.059914 | primary |       File "/usr/local/lib/python2.7/dist-packages/unittest2/case.py", line 702, in assertTrue23:06
imacdonn2018-03-20 22:16:15.059957 | primary |         raise self.failureException(msg)23:06
imacdonn2018-03-20 22:16:15.060097 | primary |     AssertionError: False is not true : Unexpected compute_extend_volume result 'Error' for request req-817a0483-8a2e-4f5c-96fe-752e6f41782e.23:06
imacdonnany known issues here ?23:06
imacdonnfrom legacy-tempest-dsvm-full-lio-src-os-brick on https://review.openstack.org/#/c/554704/23:07
*** masber has quit IRC23:08
*** AlexeyAbashkin has joined #openstack-cinder23:19
*** AlexeyAbashkin has quit IRC23:23
*** Anticime1 is now known as Anticimex23:29
*** mriedem has quit IRC23:34
*** harlowja has joined #openstack-cinder23:49
*** mriedem has joined #openstack-cinder23:52

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