Tuesday, 2016-09-06

*** alonma has joined #openstack-cinder00:04
*** alonma has quit IRC00:08
*** Julien-zte has joined #openstack-cinder00:20
*** mriedem has joined #openstack-cinder00:26
*** mriedem has left #openstack-cinder00:26
*** felix23ma has quit IRC00:29
*** felix23ma has joined #openstack-cinder00:40
*** laughterwym has joined #openstack-cinder00:45
*** alonma has joined #openstack-cinder00:45
openstackgerritwanghao proposed openstack/cinder: Refactor volume status in managing vol  https://review.openstack.org/33316400:46
*** laughterwym has joined #openstack-cinder00:46
*** yangyape_ has quit IRC00:48
*** alonma has quit IRC00:50
*** kaisers1 has joined #openstack-cinder00:51
*** kaisers has quit IRC00:51
*** kaisers has joined #openstack-cinder00:52
*** kaisers2 has quit IRC00:52
*** markvoelker has joined #openstack-cinder00:53
*** lixiaoy1 has joined #openstack-cinder01:04
openstackgerritYaoZheng proposed openstack/cinder: support volume driver set readonly  https://review.openstack.org/33758001:23
openstackgerritCao ShuFeng proposed openstack/python-cinderclient: Parse filter item "name" correctly for snapshot-list  https://review.openstack.org/35759401:23
*** cknight has joined #openstack-cinder01:25
*** alonma has joined #openstack-cinder01:27
*** Lee1092 has joined #openstack-cinder01:27
*** alonma has quit IRC01:31
*** akshai has joined #openstack-cinder01:56
*** EinstCrazy has joined #openstack-cinder02:01
*** alonma has joined #openstack-cinder02:09
*** itzdilip has joined #openstack-cinder02:10
*** alonma has quit IRC02:13
*** akshai_ has joined #openstack-cinder02:29
*** salv-orl_ has joined #openstack-cinder02:29
*** akshai has quit IRC02:32
*** salv-orlando has quit IRC02:32
*** bkopilov has quit IRC02:50
*** alonma has joined #openstack-cinder02:50
*** zhangguoqing has joined #openstack-cinder02:52
*** akshai_ has quit IRC02:52
*** markvoelker has quit IRC02:53
*** alonma has quit IRC02:55
*** houming has joined #openstack-cinder03:04
*** yangyapeng has joined #openstack-cinder03:12
*** zhugaoxiao has quit IRC03:14
*** zhugaoxiao has joined #openstack-cinder03:14
*** fredli has joined #openstack-cinder03:21
*** alonma has joined #openstack-cinder03:22
*** alonma has quit IRC03:26
*** adisky has joined #openstack-cinder03:47
*** vladani__ has quit IRC03:49
*** vladani__ has joined #openstack-cinder03:49
*** coolsvap_ has joined #openstack-cinder03:52
*** alonma has joined #openstack-cinder03:53
*** alonma has quit IRC03:58
*** cknight1 has joined #openstack-cinder04:02
*** cknight has quit IRC04:03
*** sdake has joined #openstack-cinder04:05
*** watanabe_isao has joined #openstack-cinder04:06
*** cknight1 has quit IRC04:06
*** lixiaoy1 has quit IRC04:12
*** pgadiya has joined #openstack-cinder04:17
*** lprice has quit IRC04:20
*** bkopilov has joined #openstack-cinder04:24
*** alonma has joined #openstack-cinder04:25
*** sdake has quit IRC04:25
*** alonma has quit IRC04:29
*** links has joined #openstack-cinder04:39
*** pdeore has joined #openstack-cinder04:40
*** harlowja_at_home has joined #openstack-cinder04:42
*** harlowja_at_home has quit IRC04:47
*** lixiaoy1 has joined #openstack-cinder04:47
*** sdake has joined #openstack-cinder04:50
*** lkuchlan has joined #openstack-cinder04:53
*** alonma has joined #openstack-cinder04:56
*** laughterwym has quit IRC04:58
*** laughterwym has joined #openstack-cinder04:58
*** laughter_ has joined #openstack-cinder04:59
*** alonma has quit IRC05:00
*** sdake has quit IRC05:01
*** laughterwym has quit IRC05:03
*** sdake has joined #openstack-cinder05:10
*** laughter_ has quit IRC05:13
*** laughterwym has joined #openstack-cinder05:14
*** laughterwym has quit IRC05:15
*** laughterwym has joined #openstack-cinder05:15
*** sdake has quit IRC05:17
*** diogogmt has quit IRC05:19
*** laughterwym has quit IRC05:20
*** laughterwym has joined #openstack-cinder05:21
*** laughterwym has quit IRC05:26
*** laughterwym has joined #openstack-cinder05:27
*** alonma has joined #openstack-cinder05:27
*** avishay has joined #openstack-cinder05:28
*** zzzeek has quit IRC05:30
*** nkrinner_afk is now known as nkrinner05:32
*** alonma has quit IRC05:32
*** laughterwym has quit IRC05:33
*** laughterwym has joined #openstack-cinder05:33
*** ChubYann has quit IRC05:36
*** laughterwym has quit IRC05:38
*** laughterwym has joined #openstack-cinder05:40
*** EinstCrazy has quit IRC05:40
*** laughterwym has quit IRC05:41
*** laughterwym has joined #openstack-cinder05:41
*** sandanar has joined #openstack-cinder05:41
*** diogogmt has joined #openstack-cinder05:41
*** liverpooler has quit IRC05:42
openstackgerritVijay Ladani proposed openstack/cinder: Correction in formatting syntax  https://review.openstack.org/36483705:43
*** snarwade has quit IRC05:44
*** snarwade has joined #openstack-cinder05:44
*** lixiaoy1 has quit IRC05:45
*** lixiaoy11 has joined #openstack-cinder05:45
*** e0ne has joined #openstack-cinder05:46
*** e0ne has quit IRC05:47
*** hoonetorg has quit IRC05:48
*** avishay has quit IRC05:50
*** fredli has quit IRC05:50
*** mjura has joined #openstack-cinder05:51
*** arecknag has joined #openstack-cinder05:52
*** EinstCrazy has joined #openstack-cinder05:52
*** mjura has quit IRC05:58
*** mjura has joined #openstack-cinder05:59
*** alonma has joined #openstack-cinder05:59
*** rcarrillocruz has quit IRC06:02
*** alonma has quit IRC06:04
*** avishay has joined #openstack-cinder06:04
*** hoonetorg has joined #openstack-cinder06:12
*** raunak has joined #openstack-cinder06:14
*** bardia has joined #openstack-cinder06:19
*** laughterwym has quit IRC06:21
*** EinstCrazy has quit IRC06:21
*** EinstCrazy has joined #openstack-cinder06:23
*** laughterwym has joined #openstack-cinder06:23
*** laughter_ has joined #openstack-cinder06:25
*** laughterwym has quit IRC06:25
*** laughter_ has quit IRC06:25
*** avishay has quit IRC06:26
*** alonma has joined #openstack-cinder06:30
*** raunak_ has joined #openstack-cinder06:34
*** raunak has quit IRC06:34
*** raunak_ is now known as raunak06:34
*** alonma has quit IRC06:35
*** nikeshm has joined #openstack-cinder06:36
nikeshmhi06:36
nikeshmDuncanT06:37
nikeshmhi06:37
DuncanTHi06:38
*** avishay has joined #openstack-cinder06:38
nikeshmcan we still target small bugs for newton for volume driver06:38
nikeshm1 line change06:39
*** laughterwym has joined #openstack-cinder06:39
*** felix23ma has quit IRC06:42
*** felix23ma has joined #openstack-cinder06:43
DuncanTIf it has an associated bug, and is clearly a bug not a feature in disguise, then yesm definitely06:45
*** pcaruana has joined #openstack-cinder06:48
*** bardia has quit IRC06:51
*** felix23ma has quit IRC06:54
*** markvoelker has joined #openstack-cinder06:54
*** laughterwym has joined #openstack-cinder06:57
*** laughterwym has joined #openstack-cinder06:57
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Imported Translations from Zanata  https://review.openstack.org/36589106:57
nikeshmDuncanT: thanks, https://github.com/openstack/cinder/blob/master/cinder/volume/drivers/kaminario/kaminario_common.py#L593-L604   here i am using cinder volume manager _connect_device to attach volume and view but terminate_connection for view and volume shows http://paste.openstack.org/show/567084/06:58
nikeshmany idea06:58
*** markvoelker has quit IRC06:59
nikeshmfrom array side it is unmapping06:59
nikeshmfrom cinder side where i am attaching temporary view and volume, it is not cleaning06:59
*** tesseract- has joined #openstack-cinder07:00
*** EinstCra_ has joined #openstack-cinder07:00
*** EinstCrazy has quit IRC07:00
*** alonma has joined #openstack-cinder07:02
*** felix23ma has joined #openstack-cinder07:02
*** Julien-zte has quit IRC07:04
*** alonma has quit IRC07:06
*** EinstCra_ has quit IRC07:09
*** liverpooler has joined #openstack-cinder07:10
jschwarzgeguileo, ping07:11
*** lixiaoy11 has quit IRC07:14
*** lixiaoy1 has joined #openstack-cinder07:16
*** EinstCrazy has joined #openstack-cinder07:17
*** raunak has quit IRC07:18
*** chrichip has quit IRC07:25
*** laughterwym has quit IRC07:29
*** laughterwym has joined #openstack-cinder07:29
*** alonma has joined #openstack-cinder07:32
*** felix23ma has quit IRC07:32
*** laughterwym has quit IRC07:34
*** laughterwym has joined #openstack-cinder07:38
*** laughterwym has quit IRC07:39
*** felix23ma has joined #openstack-cinder07:41
*** laughterwym has joined #openstack-cinder07:42
*** vladani__ has quit IRC07:48
*** vladani has joined #openstack-cinder07:49
*** Julien-zte has joined #openstack-cinder07:56
openstackgerritxiaoqin proposed openstack/cinder: Storwize: do the correct update after failover  https://review.openstack.org/29257007:57
*** sborkows has joined #openstack-cinder07:58
*** lpetrut has joined #openstack-cinder08:02
*** alonma_ has joined #openstack-cinder08:06
*** lpetrut has quit IRC08:07
*** jistr is now known as jistr|dentist08:07
*** alonma has quit IRC08:09
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376608:10
*** EinstCrazy has quit IRC08:11
*** sborkows has quit IRC08:11
*** sborkows has joined #openstack-cinder08:12
*** gaurangt has joined #openstack-cinder08:14
*** markvoelker has joined #openstack-cinder08:19
*** mvk has quit IRC08:22
*** vladani has quit IRC08:23
*** vladani_ has joined #openstack-cinder08:23
*** markvoelker has quit IRC08:24
*** EinstCrazy has joined #openstack-cinder08:27
*** alonma_ has quit IRC08:28
*** EinstCrazy has quit IRC08:28
*** EinstCrazy has joined #openstack-cinder08:28
*** salv-orlando has joined #openstack-cinder08:29
*** alonma has joined #openstack-cinder08:30
*** salv-orl_ has quit IRC08:32
*** watanabe_isao has quit IRC08:37
*** laughterwym has quit IRC08:43
*** laughterwym has joined #openstack-cinder08:44
*** laughterwym has quit IRC08:48
*** mvk has joined #openstack-cinder08:51
*** lixiaoy1 has quit IRC08:57
*** EinstCrazy has quit IRC08:59
*** EinstCrazy has joined #openstack-cinder09:01
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/36420909:05
*** shausy has joined #openstack-cinder09:09
*** liuyulong has joined #openstack-cinder09:16
liuyulongHi cinder guys, please help and took a look at this: https://bugs.launchpad.net/cinder/+bug/1430641/comments/209:18
openstackLaunchpad bug 1430641 in Cinder "'ResourceClosedError: This result object does not return rows. It has been closed automatically.' even with MySQL-python (1.2.5) " [Undecided,New]09:18
*** jistr|dentist is now known as jistr09:23
*** lpetrut has joined #openstack-cinder09:30
*** lpetrut has quit IRC09:30
*** jungleboyj has quit IRC09:30
*** diablo_rojo has quit IRC09:30
*** lpetrut has joined #openstack-cinder09:31
*** diablo_rojo has joined #openstack-cinder09:31
openstackgerritHa Van Tu proposed openstack/cinder: Remove unused parameters  https://review.openstack.org/36601109:32
openstackgerritMichal Dulko proposed openstack/cinder: Remove dead code - replication RPC APIs  https://review.openstack.org/36562009:36
*** mvk has quit IRC09:38
*** JayConroy has joined #openstack-cinder09:39
*** gaurangt1 has joined #openstack-cinder09:39
*** gaurangt has quit IRC09:40
openstackgerritMerged openstack/cinder: Fixed indentation  https://review.openstack.org/36576409:42
*** alonma has quit IRC09:44
openstackgerritMichal Dulko proposed openstack/cinder: Remove dead code - replication RPC APIs  https://review.openstack.org/36562009:45
*** sdake has joined #openstack-cinder09:45
*** jungleboyj has joined #openstack-cinder09:46
openstackgerritHa Van Tu proposed openstack/cinder: Remove unused parameters  https://review.openstack.org/36601509:48
*** laughterwym has joined #openstack-cinder09:56
*** mvk has joined #openstack-cinder09:58
*** TommyLikeHu has joined #openstack-cinder09:58
*** laughterwym has quit IRC10:00
*** pgadiya has quit IRC10:02
*** alonma has joined #openstack-cinder10:05
*** alonma has quit IRC10:05
*** alonma has joined #openstack-cinder10:06
openstackgerritDavanum Srinivas (dims) proposed openstack/cinder: [WIP] Testing latest u-c  https://review.openstack.org/30376610:10
*** pgadiya has joined #openstack-cinder10:11
*** Julien-zte has quit IRC10:13
*** pgadiya has quit IRC10:15
*** e0ne has joined #openstack-cinder10:15
*** liuyulong is now known as liuyulong|away10:15
*** pgadiya has joined #openstack-cinder10:17
*** Julien-zte has joined #openstack-cinder10:23
*** kaisers2 has joined #openstack-cinder10:29
*** kaisers1 has quit IRC10:29
*** kaisers1 has joined #openstack-cinder10:30
*** kaisers has quit IRC10:30
*** ntpttr has quit IRC10:30
*** ntpttr has joined #openstack-cinder10:31
*** sdake has quit IRC10:33
*** pgadiya has quit IRC10:35
*** thrawn01 has quit IRC10:37
*** alonma has quit IRC10:41
*** alonma has joined #openstack-cinder10:41
*** HenryG_ is now known as HenryG10:46
*** gaurangt has joined #openstack-cinder10:46
*** gaurangt1 has quit IRC10:48
*** zzzeek has joined #openstack-cinder10:51
*** pgadiya has joined #openstack-cinder10:51
openstackgerritMerged openstack/cinder: zfssaiscsi driver log messges need cleaned up  https://review.openstack.org/36321910:53
*** yangyapeng has quit IRC10:54
*** EinstCrazy has quit IRC10:54
*** zzzeek has quit IRC10:56
*** zzzeek has joined #openstack-cinder10:57
*** kaisers2 has quit IRC10:58
*** kaisers1 has quit IRC10:59
*** kaisers has joined #openstack-cinder10:59
*** kaisers1 has joined #openstack-cinder10:59
*** vladani__ has joined #openstack-cinder11:01
*** vladani_ has quit IRC11:01
*** cdelatte has joined #openstack-cinder11:19
*** alyson_ has joined #openstack-cinder11:20
*** alonma has quit IRC11:22
*** e0ne has quit IRC11:25
*** bkopilov has quit IRC11:27
*** gaurangt has quit IRC11:34
openstackgerritLiang Chen proposed openstack/os-brick: Wait a moment for mdev to come up  https://review.openstack.org/36606511:35
*** sdague has joined #openstack-cinder11:39
*** kaisers has quit IRC11:41
*** kaisers has joined #openstack-cinder11:42
*** kaisers1 has quit IRC11:43
*** kaisers1 has joined #openstack-cinder11:43
*** alonma has joined #openstack-cinder11:43
*** kaisers2 has joined #openstack-cinder11:46
*** kaisers3 has joined #openstack-cinder11:47
*** alonma has quit IRC11:48
*** kaisers1 has quit IRC11:48
*** kaisers has quit IRC11:49
*** salv-orl_ has joined #openstack-cinder11:52
*** salv-orlando has quit IRC11:52
*** kaisers3 has quit IRC11:55
*** kaisers has joined #openstack-cinder11:55
*** sborkows has quit IRC12:01
*** gcb has quit IRC12:04
*** laughterwym has joined #openstack-cinder12:07
*** pgadiya has quit IRC12:10
dulekUS holidays mixed with feature freeze are so cool. I have the emptiest Gerrit mailbox since at least a year.12:12
*** edmondsw has joined #openstack-cinder12:13
*** Lee1092 has quit IRC12:14
*** sborkows has joined #openstack-cinder12:16
*** alonma has joined #openstack-cinder12:17
*** pdeore has quit IRC12:18
*** rlrossit has joined #openstack-cinder12:19
*** liverpooler has quit IRC12:21
*** yangyapeng has joined #openstack-cinder12:21
smcginnisdulek: :D12:25
*** sandanar_ has joined #openstack-cinder12:26
*** clenimar has quit IRC12:26
*** pgadiya has joined #openstack-cinder12:28
*** ccesario has quit IRC12:29
*** sandanar has quit IRC12:29
*** Julien-zte has quit IRC12:30
*** clenimar has joined #openstack-cinder12:31
*** sandanar_ has quit IRC12:31
smcginnisAnyone already working on the CinderUnicodeTest problem CIs are hitting?12:35
*** Lee1092 has joined #openstack-cinder12:37
openstackgerritMerged openstack/cinder: Remove doc for promote/reenable replica  https://review.openstack.org/36468012:38
*** alonma has quit IRC12:40
*** tongli has joined #openstack-cinder12:40
*** EinstCrazy has joined #openstack-cinder12:43
openstackgerritDuncan Thomas proposed openstack/cinder-specs: Support multiple nova consuming one cinder  https://review.openstack.org/36263712:44
*** gaurangt has joined #openstack-cinder12:49
*** crose has joined #openstack-cinder12:50
*** pcaruana has quit IRC12:50
*** ccesario has joined #openstack-cinder12:52
openstackgerritAleksey Ruban proposed openstack/cinder: Nexenta: Added use of sessions for NexentaEdge drivers  https://review.openstack.org/35847412:56
*** alonma has joined #openstack-cinder12:56
*** baojg has joined #openstack-cinder12:58
*** lkuchlan has quit IRC13:00
*** houming has quit IRC13:01
*** bkopilov has joined #openstack-cinder13:03
openstackgerritSean McGinnis proposed openstack/cinder: Provide volume size for tempest volume creation  https://review.openstack.org/36611413:07
*** cknight has joined #openstack-cinder13:12
*** pgadiya has quit IRC13:13
*** alkhodos has joined #openstack-cinder13:13
*** Guest36215 has quit IRC13:15
openstackgerritAleksey Ruban proposed openstack/cinder: Nexenta: Added use of sessions for NexentaEdge drivers  https://review.openstack.org/35847413:15
*** e0ne has joined #openstack-cinder13:17
*** nikeshm has quit IRC13:17
*** crose has quit IRC13:17
*** akerr has joined #openstack-cinder13:18
*** akshai has joined #openstack-cinder13:18
*** avishay has quit IRC13:18
openstackgerritSean McGinnis proposed openstack/cinder: Provide volume size for tempest volume creation  https://review.openstack.org/36611413:22
*** rooneym has joined #openstack-cinder13:24
*** eharney has joined #openstack-cinder13:25
*** Yogi1 has joined #openstack-cinder13:26
openstackgerritShilpa Jagannath proposed openstack/cinder: Fix return values of consistencygroup actions in Pure storage driver.  https://review.openstack.org/36613513:32
openstackgerritSean McGinnis proposed openstack/cinder: Provide volume size for tempest volume creation  https://review.openstack.org/36611413:33
*** mriedem has joined #openstack-cinder13:34
*** markvoelker has joined #openstack-cinder13:36
openstackgerritShilpa Jagannath proposed openstack/cinder: Fix return values of consistencygroup actions in Pure storage driver.  https://review.openstack.org/36613513:37
*** andymaier has joined #openstack-cinder13:37
flip214Our stack of DRBD patches, starting at https://review.openstack.org/#/c/347268/, could use a few reviews... some of the later ones are already +2+2+1, but the early ones are not13:39
duleksmcginnis: We're enabling our in-repo tempest tests only in third party CIs?13:39
smcginnisdulek: At least so far.13:40
duleksmcginnis: Okay. Maybe it's a good idea - if we would, our whole CI would be broken with such tempest change.13:41
openstackgerritMerged openstack/cinder: Imported Translations from Zanata  https://review.openstack.org/36589113:41
*** merooney has joined #openstack-cinder13:41
smcginnisdulek: Yeah. My thought was the in-tree tests would be good for the tests we want to run against vendor CIs that we can't really run against LVM.13:41
smcginnisMaybe not, but it at least gives us a way to run some extra tests.13:42
*** gaurangt has left #openstack-cinder13:45
*** shausy has quit IRC13:47
*** alonma_ has joined #openstack-cinder13:51
*** alonma has quit IRC13:54
*** rlrossit has quit IRC13:56
*** alonma_ has quit IRC13:56
*** alonma has joined #openstack-cinder13:57
*** alonma has quit IRC13:58
*** alonma has joined #openstack-cinder13:58
*** krtaylor has joined #openstack-cinder14:02
*** Julien-zte has joined #openstack-cinder14:03
*** arecknag has quit IRC14:03
*** ametts has joined #openstack-cinder14:06
*** baumann has joined #openstack-cinder14:06
*** avishay has joined #openstack-cinder14:07
*** avishay_ has joined #openstack-cinder14:07
*** avishay_ has quit IRC14:07
*** nikeshm has joined #openstack-cinder14:11
*** ducttape_ has joined #openstack-cinder14:12
nikeshmsmcginnis: hi, just to cross-verify, we have one bug in kaminario driver, can we address it in newton release14:12
duleknikeshm: I think that until RC-1 (next week), all the bugfixes can be accepted.14:13
*** ducttape_ has quit IRC14:15
erloneharney: ping14:16
eharneyerlon: hi14:16
erloneharney: hey, have a few minutes?14:16
eharneyerlon: sure14:16
smcginnisnikeshm: Yes, we now are in the stage where it is bug fix only.14:17
erloneharney: Im trying to figure out what to do with the security_permissions in the NFS driver14:17
*** ducttape_ has joined #openstack-cinder14:18
erloneharney: I think that won't work with snapshots14:18
eharneyerlon: i suspect you are right, at least in its current state14:18
erloneharney: libvirt does some things that I think its a bug14:19
nikeshmsmcginnis: thanks, as pointed by dulek, is any cutoff date?14:19
erloneharney: like, changing the permissions and not putting it back14:19
erloneharney: some operation it changes from ubuntu -> libvirt-qemu -> root14:19
*** avishay has quit IRC14:19
eharneyerlon: it should reset the permissions when the volume is detached from the instance14:20
erlonand the file remains as root forever14:20
smcginnisnikeshm: The RC-1 deadline. Then only extrememly critical bugfixes after that.14:20
*** takedakn has joined #openstack-cinder14:20
erloneharney: reset you mean, return to what was before?14:20
eharneyerlon: yes14:20
*** diogogmt has quit IRC14:20
smcginnisnikeshm: So the week of September 12.14:20
*** links has quit IRC14:21
erloneharney: hmm, then its broken14:21
nikeshmsmcginnis: does it mean any day in that week14:21
nikeshmwill be RC-114:21
smcginnisnikeshm: It will most likely be the Wednesday of that week, but better to try to get things done well before then.14:21
nikeshmthanks14:22
smcginnisnp14:22
erloneharney: not even the actual NFS driver I could make work with security enabled14:22
eharneyerlon: yes, i think there are some issues still to be sorted out in that security feature14:22
erloneharney: I configured the NFS server to rootsquash, and when nova try to attach it fails14:22
eharneyerlon: you have to leave root squash off afaik14:23
eharneyi've never seen it work, anyway14:23
erloneharney: I don't see the point of using security if we can't use it with rootsquash14:24
eharneyerlon: i was not a fan of the security patch when it was introduced :)14:24
*** ametts has quit IRC14:24
*** zigo has quit IRC14:25
erloneharney: hmm, ok, so its useless as I thought :)14:25
openstackgerritChuck Fouts proposed openstack/cinder: NetApp: Fix issue with busy snapshot deletion.  https://review.openstack.org/35878914:26
erloneharney: one solution I could tough was to have groups configured in the controllers and in computes, so the IDs of the volumes matches14:27
*** akshai has quit IRC14:27
erloneharney: but haven't tried14:28
erloneharney: maybe put everybody involved in a common secondary group14:28
*** jordanP has joined #openstack-cinder14:28
*** baumann has quit IRC14:29
*** ametts has joined #openstack-cinder14:29
*** baumann has joined #openstack-cinder14:29
*** laurent\ has quit IRC14:29
*** salv-orlando has joined #openstack-cinder14:29
*** felix23ma has quit IRC14:30
*** laurent\ has joined #openstack-cinder14:31
*** avishay has joined #openstack-cinder14:31
*** takedakn has quit IRC14:32
*** zigo has joined #openstack-cinder14:32
*** yangyapeng has quit IRC14:32
*** salv-orl_ has quit IRC14:32
*** zigo is now known as Guest3914714:33
*** yangyapeng has joined #openstack-cinder14:33
*** akshai has joined #openstack-cinder14:37
*** jdurgin1 has joined #openstack-cinder14:38
*** timcl has joined #openstack-cinder14:38
*** Guest39147 has quit IRC14:39
*** zigo_ has joined #openstack-cinder14:42
*** krtaylor has quit IRC14:43
*** sdake has joined #openstack-cinder14:44
*** baumann has quit IRC14:46
*** zigo_ has quit IRC14:47
*** baumann has joined #openstack-cinder14:47
*** baumann has quit IRC14:47
*** baumann has joined #openstack-cinder14:48
*** baumann has quit IRC14:48
*** baumann has joined #openstack-cinder14:48
*** zigo_ has joined #openstack-cinder14:48
*** sborkows has quit IRC14:50
openstackgerritwanghao proposed openstack/cinder: Refactor volume status in managing vol  https://review.openstack.org/33316414:51
openstackgerritwanghao proposed openstack/cinder: Refactor volume status in managing vol  https://review.openstack.org/33316414:56
*** _ducttape_ has joined #openstack-cinder14:59
*** lprice has joined #openstack-cinder14:59
*** mjura has quit IRC15:01
*** ducttape_ has quit IRC15:02
*** apuimedo has left #openstack-cinder15:05
*** gouthamr has joined #openstack-cinder15:10
*** diogogmt has joined #openstack-cinder15:11
*** gouthamr_ has joined #openstack-cinder15:12
*** sayalilunkad has quit IRC15:13
*** sayalilunkad has joined #openstack-cinder15:13
*** hemnafk has quit IRC15:13
*** alonma has quit IRC15:14
*** gouthamr has quit IRC15:15
*** xinli has joined #openstack-cinder15:17
*** nkrinner is now known as nkrinner_afk15:18
*** hemna has joined #openstack-cinder15:18
*** baumann_ has joined #openstack-cinder15:20
*** baumann_ has quit IRC15:20
*** zigo_ is now known as zigo15:20
*** baumann_ has joined #openstack-cinder15:20
*** baumann has quit IRC15:23
*** alonma has joined #openstack-cinder15:23
*** alonma has quit IRC15:23
*** alonma has joined #openstack-cinder15:24
*** EinstCrazy has quit IRC15:26
*** enriquetaso has joined #openstack-cinder15:27
*** mriedem1 has joined #openstack-cinder15:27
*** Julien-zte has quit IRC15:28
jgriffithsmcginnis: ping15:30
*** mriedem has quit IRC15:30
smcginnisjgriffith: Hey!15:30
jgriffithsmcginnis: howdy15:30
jgriffithsmcginnis: hey... I'm fixing the rootwrap on that env patch for LVM...15:30
jgriffithsmcginnis: I was wondering about privsep and brick?15:31
smcginnisjgriffith: Oh good!15:31
smcginnisjgriffith: Sure, what's up?15:31
jgriffithsmcginnis: it looks like it need/reads in from Cinder's rootwrap?15:31
smcginnisjgriffith: Hmm, shouldn't be.15:31
smcginnisjgriffith: Looking now. But one of the main reasons for privsep was to get rid of the need to have a rootwrap filter file.15:32
jgriffithsmcginnis: yeah :)15:32
jgriffithsmcginnis: http://logs.openstack.org/75/365175/1/check/gate-os-brick-python27-ubuntu-xenial/4794390/console.html15:32
jgriffithsmcginnis: so that statement is probably wrong.. but I can not figure out how to add the privsep stuff15:33
jgriffithsmcginnis: so I added it to the test I thought and that didn't work, then started wondering how this will work with Cinder in the future15:33
jgriffithsmcginnis: http://logs.openstack.org/75/365175/1/check/gate-os-brick-python27-ubuntu-xenial/4794390/console.html15:34
jgriffitherrr... sorry15:34
*** faiz89 has joined #openstack-cinder15:34
jgriffithsmcginnis: http://logs.openstack.org/75/365175/1/check/gate-os-brick-python27-ubuntu-xenial/4794390/console.html#_2016-09-03_00_10_13_72291915:34
smcginnisjgriffith: Is that just a unit test/mocking issue?15:35
jgriffithsmcginnis: probably :)15:35
hemnamep15:35
smcginnishemna: We're talking one of your favorite subjects. :)15:36
avishayhemna, jgriffith: any idea about this bug? it's blocking me... https://bugs.launchpad.net/cinder/+bug/161994115:36
openstackLaunchpad bug 1619941 in Cinder "os-brick returns wrong iscsi device path" [Undecided,New]15:36
jgriffithsmcginnis: but when looking at it, it raised some questions for me regarding how the whole privsep thing is going to work and I couldn't find any answers15:36
jgriffithavishay: looking15:36
hemnais it "Fun with privsep" day ?15:36
jgriffithavishay: haha... my favorite!!!15:36
smcginnishemna: ;)15:36
avishayjgriffith, you know this issue?15:36
jgriffithavishay: Not this one inparticular but I get all sorts of random failures in brick around the targets15:37
hemna:(15:37
hemnalaunchpad isn't coming up for me15:37
jgriffithavishay: I'll look and see if there's anything I can figure out if/when I get past this silly test thing15:37
* hemna waits15:37
jgriffithavishay: oh15:38
smcginnisjgriffith: So that failure does look like it's just a unit test that needs to be updated.15:38
hemnahuh15:38
jgriffithavishay: hmm... ok, I think I know how to fix that15:38
hemnaso, that's a udev rule that creates those paths fyi15:38
smcginnisjgriffith: But the bigger question around privsep, I remember an overview somewhere. I'll see if I can find that.15:38
jgriffithsmcginnis: yeah.. but the problem is if you look I have the _lvm_env added to fake_execute in the test15:38
jgriffithsmcginnis: haha.. unless we're calling a different fake_execute :)15:38
smcginnisjgriffith: Hah, wouldn't be the first time something like that has happened. ;)15:39
avishayhemna: yep, i looked into it but the udev rule seems to just create the path according to what it gets from the kernel, which gets it from god knows where15:39
jgriffithavishay: I think the way to address that is that I just add the port number to the model explicitly all the time.  IIRC that should work, but I'll have to verify15:39
*** andymaier has quit IRC15:39
jgriffithavishay: unless brick does something goofy like filter things out and do it's "own" thing :(15:39
avishayjgriffith: what do you mean by model?15:40
jgriffithavishay: the provider info in the model15:40
jgriffithavishay: provider_location15:41
jgriffithiscsi_portal iqn 015:41
hemnaavishay, so, I can add another set of search paths that also include the port #15:41
avishayjgriffith: os-brick just creates the path like i wrote in the bug - "/dev/disk/by-path/ip-%s-iscsi-%s-lun-%s"15:41
avishayhemna: yea that's what i was thinking15:41
*** zhangguoqing has quit IRC15:41
avishayjgriffith: hemna: i'm just wondering why i'm the first person to hit this bug15:41
hemnait's pretty easy to do15:41
avishaythat's my real concern15:42
hemnafwiw, I've never seen one w/ a port #15:42
*** pcaruana has joined #openstack-cinder15:42
hemnabut each distro has different udev rules for the creation of those paths15:42
avishayhemna: we've tested iSCSI against LVM, EMC, NetApp, Dell, and some others.  Solidfire was the first this happened with.15:42
hemnaI'd debug the udev discovery15:43
jgriffithhemna:  avishay if it's not setting the "ip-xxxxxx" based off of the provider_info string exactly then it's wrong.  Regardless of distro15:43
hemnait's probably adding that15:43
jgriffithand regardless of udev15:43
avishayjgriffith: i didn't see where it gets it from, will check...15:43
jgriffithanyway, I'd have to look at it and I don't know why it's just now showing up for avishay that's no good15:44
jgriffithavishay: are you running off of stable-M or master?15:44
*** sdake has quit IRC15:44
avishayjgriffith: liberty15:44
jgriffithavishay: ok, give me a big and I'll try it out15:45
avishayjgriffith: much appreciated15:45
jgriffithavishay: certainly15:45
smcginnisjgriffith: Might be interesting: https://www.youtube.com/watch?v=0FOQB4PZVe015:47
*** xinli has quit IRC15:47
*** xinli has joined #openstack-cinder15:47
smcginnisjgriffith: Privsep stuff starts around 17:20, but the background on issues with rootwrap are interesting too.15:47
jgriffithsmcginnis: cool, thanks!15:48
jgriffithsmcginnis: full house :)15:48
hemnaI think that was his original proposal15:48
smcginnishehe15:48
hemnait's a good watch.  I've seen it15:48
hemnait doesn't necessarily match to what exists today.15:48
smcginnishemna: Was looking for something from the session in Austin, but couldn't find it.15:48
jgriffithhemna: swell :)15:49
smcginnisDon't think that was recorded since it was in the design summit.15:49
*** baumann_ has quit IRC15:49
*** alonma has quit IRC15:50
*** bardia has joined #openstack-cinder15:52
*** mriedem1 is now known as mriedem15:53
*** timcl has quit IRC15:54
*** baumann has joined #openstack-cinder15:55
*** baumann has quit IRC15:55
*** baumann has joined #openstack-cinder15:56
*** _ducttape_ has quit IRC15:58
*** ducttape_ has joined #openstack-cinder15:59
jgriffithavishay: do you have sf_svip set in your cinder.conf file?16:00
avishayjgriffith: yes16:00
jgriffithavishay: and you don't have :3260 on there right?16:00
jgriffithavishay: you just have the IP?16:00
avishayjgriffith: just the IP, no port16:00
jgriffithavishay: add the port :)16:01
jgriffithavishay: let me find the bug...16:01
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Error attaching after LV-AFO  https://review.openstack.org/36145316:01
jgriffithavishay: hmm... maybe I didn't fix it upstream yet :(16:02
avishayjgriffith: bah... but if i put the port it should work?16:03
jgriffithavishay: I just stumbled across this for a customer a couple weeks ago when using VLANs16:03
*** laughterwym has quit IRC16:03
jgriffithavishay: yeah, it *should*16:03
*** leeantho has joined #openstack-cinder16:03
jgriffithavishay: there's a bug in there that be default I append the port, and if you use the svip I'm *supposed to* but don't :(16:04
*** bardia has quit IRC16:04
jgriffithavishay: most use auto-detect on the SVIP so that's why nobody has hit it16:04
avishayjgriffith: OK will try to test tomorrow16:04
jgriffithavishay: in the off case that somebody uses VLAN's or something where they have to specify that's where this gets exposed16:04
*** david-lyle_ has joined #openstack-cinder16:05
*** david-lyle_ has quit IRC16:05
avishayjgriffith: gotcha, that explains why it's not something like "solidfire never works with centos and avishay is the first person it hit" :)16:06
jgriffithavishay: yes, exactly16:06
jgriffithavishay: I have a patch that updates the conf help, and also does a check/add in the driver of the port all the time16:06
jgriffithavishay: yeah.. :)16:06
jgriffithsorry about that16:06
jgriffithbut I'm glad because it reminds me that I haven't put the bug/patch up yet :)16:07
avishayjgriffith: no worries, i'll add the port and cross my fingers16:07
jgriffithavishay: Ohhhh haha16:07
jgriffithI just updated the openstack docs :)16:07
avishayjgriffith: gracias!16:07
avishayjgriffith: gotta run, thank you!  i'll close the bug i opened once verified16:08
jgriffithavishay: np, just leave it, I'll do some things in the driver to make it more robust16:08
jgriffithavishay: I pushed the docs update a couple weeks ago and called it good with that :(16:08
jgriffithbut I can make it better16:09
avishayjgriffith: OK cool, assigned it to you16:09
jgriffithavishay: thank you sir... have a good night16:09
avishayjgriffith: good day!16:10
*** avishay has quit IRC16:10
*** lpetrut has quit IRC16:15
SwansonStacking stacking stacking...16:15
*** lpetrut has joined #openstack-cinder16:15
*** rlrossit has joined #openstack-cinder16:16
*** timcl has joined #openstack-cinder16:16
*** arch-nemesis has joined #openstack-cinder16:18
*** baumann_ has joined #openstack-cinder16:19
Swansonhacking hacking hacking preventing me from stacking stacking stacking stacking again!16:19
*** lpetrut has quit IRC16:20
*** e0ne has quit IRC16:20
*** gouthamr_ has quit IRC16:20
openstackgerritHelen Walsh proposed openstack/cinder: VMAX driver - Won't delete pre-existing initiator groups.  https://review.openstack.org/36570516:21
*** baumann has quit IRC16:21
*** cknight has quit IRC16:24
*** raunak has joined #openstack-cinder16:24
*** rlrossit has quit IRC16:25
*** raunak has quit IRC16:25
*** akshai has quit IRC16:26
*** akshai has joined #openstack-cinder16:29
*** Apoorva has joined #openstack-cinder16:29
*** cknight has joined #openstack-cinder16:30
*** stevemar has quit IRC16:31
*** gouthamr has joined #openstack-cinder16:33
*** jordanP has quit IRC16:35
*** raunak has joined #openstack-cinder16:35
lpriceHey, I have a bug that needs to be targeted to RC1. We're working on a fix for it right now. Can someone from the bug team please update the target release for it? -> https://bugs.launchpad.net/cinder/+bug/161901016:41
openstackLaunchpad bug 1619010 in Cinder "NetApp E-Series: Startup issue with certain platforms" [Undecided,New]16:41
*** xyang1 has joined #openstack-cinder16:42
openstackgerritJohn Griffith proposed openstack/cinder: Ignore lvm descriptor leak warning on vgs calls  https://review.openstack.org/36516716:43
*** ChubYann has joined #openstack-cinder16:43
*** timcl has quit IRC16:49
*** rlrossit has joined #openstack-cinder16:49
smcginniseharney: Got a moment? https://review.openstack.org/#/c/366114/16:50
*** alonma has joined #openstack-cinder16:50
*** lpetrut has joined #openstack-cinder16:52
eharneysmcginnis: sure, looks good.  Though I see looking in some c-vol tempest logs, we have another UnicodeDecodeError....16:54
*** alonma has quit IRC16:55
smcginniseharney: Oh great.16:55
eharneysmcginnis: do any of the infra jobs run these tests?16:56
smcginniseharney: No, the in-tree tempest are just the CIs that have switched to tox -e all_plugin (which should be all of them but is not)> :)16:56
smcginniseharney: At least this will get a few of our CIs passing again.16:56
*** bardia has joined #openstack-cinder16:57
smcginniseharney: Mine has failed every run since Sunday when the upstream tempest change landed.16:57
*** sdake has joined #openstack-cinder16:57
*** hogepodge has quit IRC16:58
*** alonma has joined #openstack-cinder16:58
*** lpetrut has quit IRC16:59
*** faiz89 has quit IRC17:00
*** jdross_ has joined #openstack-cinder17:02
*** alonma has quit IRC17:03
*** hogepodge has joined #openstack-cinder17:03
openstackgerritOpenStack Proposal Bot proposed openstack/cinder: Updated from global requirements  https://review.openstack.org/36420917:04
*** vladani__ has quit IRC17:04
*** vladani__ has joined #openstack-cinder17:04
openstackgerritShilpa Jagannath proposed openstack/cinder: Fix return values of consistencygroup actions in Pure storage driver.  https://review.openstack.org/36613517:05
*** Swanson has quit IRC17:05
*** jdross has quit IRC17:06
*** bradjones has quit IRC17:06
*** Swanson has joined #openstack-cinder17:07
*** alonma has joined #openstack-cinder17:08
*** lpetrut has joined #openstack-cinder17:10
*** bradjones has joined #openstack-cinder17:11
*** bradjones has quit IRC17:11
*** bradjones has joined #openstack-cinder17:11
*** sdake_ has joined #openstack-cinder17:12
*** alonma has quit IRC17:12
*** Suyi_ has joined #openstack-cinder17:14
*** alonma has joined #openstack-cinder17:15
*** sdake has quit IRC17:15
*** esker has joined #openstack-cinder17:17
*** alonma has quit IRC17:20
*** asselin_ has joined #openstack-cinder17:20
*** stevemar has joined #openstack-cinder17:26
*** mriedem1 has joined #openstack-cinder17:26
*** mriedem has quit IRC17:28
*** e0ne has joined #openstack-cinder17:28
*** sdake_ has quit IRC17:28
*** sdake has joined #openstack-cinder17:29
*** jdurgin1 has quit IRC17:30
*** akshai has quit IRC17:30
*** bardia has quit IRC17:31
*** aspiers has quit IRC17:31
*** bardia has joined #openstack-cinder17:31
*** baojg has quit IRC17:32
*** xinli has quit IRC17:34
*** cknight has quit IRC17:36
*** e0ne has quit IRC17:36
*** harlowja has joined #openstack-cinder17:38
*** timcl has joined #openstack-cinder17:38
*** thiagolib has joined #openstack-cinder17:40
openstackgerritMerged openstack/cinder: XtremIO: fix bug deleting cgsnapshot  https://review.openstack.org/36427217:49
openstackgerritMerged openstack/cinder: Provide volume size for tempest volume creation  https://review.openstack.org/36611417:49
*** sdake has quit IRC17:49
*** mriedem1 is now known as mriedem17:53
*** sdake has joined #openstack-cinder17:56
*** bardia has quit IRC17:57
*** akshai has joined #openstack-cinder17:57
*** e0ne has joined #openstack-cinder17:58
cFoutssmcginnis: thanks for getting the tempest fix merged so quickly17:59
*** bardia has joined #openstack-cinder17:59
*** thiagolib has quit IRC18:00
*** bardia has quit IRC18:00
*** bardia has joined #openstack-cinder18:01
*** thiagolib has joined #openstack-cinder18:01
smcginniscFouts: Glad we could get it reviewed and merged so quickly. ;)18:01
*** rlrossit_ has joined #openstack-cinder18:07
erlonall: is there a way to downgrade the DB version?18:07
e0neharlowja: hi Joshua18:08
harlowjayo18:08
e0neharlowja: regarding to your comment for https://review.openstack.org/36488718:09
harlowjaright18:09
harlowjaformatter is what u want18:09
e0neharlowja: what do you think about extending DynamicLoggingListener API?18:09
harlowjawfm18:09
e0neharlowja: ok, I'll propose a blueprint with patch for Ocata18:10
e0neharlowja: thanks for the help18:10
openstackgerritMerged openstack/cinder: RBD: Error when managing a nonexistent volume  https://review.openstack.org/35428918:10
*** rlrossit has quit IRC18:10
*** mdenny has joined #openstack-cinder18:11
harlowjae0ne cool18:12
e0neharlowja: according to my patch for cinder: IMO, it doesn't metter what will be done in Newton, it will be refactored in Ocata anyway18:14
harlowjakk18:14
e0neharlowja: more or less hacky - any solution will be dropped when we change DynamicLoggingListener in oslo18:15
scottdaerlon: We used to support downgrade of DB, but decided at some point that we would no longer do so. I'm not sure if we removed support , or just haven't tested, or what the details are at this point.18:15
harlowjae0ne ya, feel free to propose a tiny patch for that, shouldn't be much to pass in a formatter18:15
e0neharlowja: sure. I'll do it tomorrow18:16
harlowjak18:16
*** Lee1092 has quit IRC18:34
erlonscottda: hmm, yep, it returns an error18:47
*** raunak has quit IRC18:48
erlonscottda: I had a easier solution, just rebased the code18:48
*** raunak has joined #openstack-cinder18:59
*** raunak has quit IRC19:02
*** baumann_ has quit IRC19:06
*** xyang has joined #openstack-cinder19:07
*** e0ne has quit IRC19:08
*** akshai has quit IRC19:09
openstackgerritMerged openstack/os-brick: Update reno for stable/newton  https://review.openstack.org/36242519:10
*** xinli has joined #openstack-cinder19:10
*** pcaruana has quit IRC19:11
*** Guest78236 is now known as melwitt19:14
*** alkhodos has quit IRC19:16
*** mriedem has quit IRC19:16
erlonsmcginnis: ping19:17
jgriffithsmcginnis: sigh.. so we don't update volume.filters on grenade upgrades now?19:17
*** merooney has quit IRC19:18
erloneharney: got security working \o/19:19
eharneyerlon: great19:20
erloneharney: you just need to configure /etc/libvirt/qemu.conf with user,group=<same_as_controller>19:21
eharneyerlon: hrmm... that sounds kind of messy19:22
*** merooney has joined #openstack-cinder19:22
jgriffitheharney: how did you get rootwrap changes to go through without breaking grenade in the past?19:23
*** jungleboyj has quit IRC19:23
jgriffitheharney: although I guess your change wouldn't have broke things19:23
*** alonma has joined #openstack-cinder19:23
erloneharney: yeahh, more or less, generally speaking, you need to do a lot of server + storage confs to make cinder work19:23
* jgriffith guesses he needs both the old and the new entry :(19:23
jgriffithahh dulek had the similar change19:24
jgriffithhttps://github.com/openstack/cinder/commit/81645a9ca68ad7ec4a5986925b835d28df078b4c#diff-9fc3890d4fca7fb0f7562f47da0e4d5a19:24
erloneharney: it is a trade of between the 'messy' and run everything as root19:24
eharneyjgriffith: yeah, i don't think i ever had one that required a big problem19:24
*** tesseract- has quit IRC19:25
eharneyerlon: what user and group are you setting those values to?19:25
*** lpetrut has quit IRC19:25
*** akshai has joined #openstack-cinder19:25
erloneharney: the same as the cinder-volume process, in my case ubuntu:ubuntu19:26
erloneharney: it will work provided compute and controllers have the same id mapping19:27
*** alonma has quit IRC19:27
erloneharney: which I don't think is a problem19:28
sbezverkjgriffith: how is it going? I was wondering if you saw email I sent you with one question..19:28
jgriffithsbezverk: not since yesterday... lemme look19:29
*** diogogmt has quit IRC19:29
eharneyerlon: it would be good to understand why that's required exactly19:29
sbezverkjgriffith: thank you, I just want to close this point before the review..19:29
jgriffithsbezverk: oh... you had a  follow up question :)19:29
sbezverkyes19:30
jgriffithsbezverk: i'm kinda stuck on that review BTW19:30
jgriffithsbezverk: http://lists.openstack.org/pipermail/openstack-dev/2016-June/098294.html19:30
jgriffithsbezverk: we've decided that things like rootwrap shouldn't change19:30
jgriffithsbezverk: or at least change such that compatability breaks19:30
*** alonma has joined #openstack-cinder19:30
*** diogogmt has joined #openstack-cinder19:31
eharneyjgriffith: this might help: https://review.openstack.org/#/c/151408/  (note it was abandoned)19:31
*** alwaysatthenoc has joined #openstack-cinder19:31
jgriffitheharney: oh.. thanks!19:31
sbezverkjgriffith: hm, bad news for us then :-(19:31
eharneyjgriffith: but something like that should work for cinder if needed19:31
*** coolsvap_ has quit IRC19:32
erloneharney: that's required because by default, libvirt spawns the qemu processes as libvirt:kvm. Then you have a problem because cinder is running as cinder:cinder or other user, and is unable to access the volume file19:32
jgriffithsbezverk: so in your follow up question... I don't think you want that19:32
jgriffithsbezverk: I'll figure that out19:32
jgriffithsbezverk: back to your other question about conf entries19:32
eharneyerlon: do you have dynamic_ownership=1 turned on?  because it should handle that19:32
erloneharney: when you set the user/group in qemu.conf, the qemu process has the same permissions as Cinder19:32
sbezverkjgriffith: right please clarify19:32
jgriffithsbezverk: that syntax you have is for multi-backend configs; ie multiple c-vol services running on a single node19:33
jgriffithsbezverk: but in the case of containers you don't want to do that19:33
sbezverkjgriffith: nope19:33
*** alwaysatthenoc has left #openstack-cinder19:33
erloneharney: where should this conf be set?19:33
sbezverksince ip addresses are different it is multinode19:33
jgriffithsbezverk: you want an individual container for each c-vol and each of those needs it's own unique cinder.conf file19:33
eharneyerlon: same place, libvirt/qemu.conf19:33
erloneharney: ok19:33
eharneyerlon: when that's on (which i thought it was by default), libvirt will chown/chmod the files as needed19:34
jgriffithsbezverk: if you do like you have there it will try and run all of those in a single container on every container :(19:34
erloneharney: ill check19:34
*** Apoorva has quit IRC19:34
erloneharney: the problem is that roosquash does not allow chown19:34
erloneharney: Im running rootsquash19:34
eharneyerlon: i think you have to have root squash off at this point19:34
sbezverkjgriffith: got it, so it is preferable to have only relevant config19:34
jgriffithsbezverk: yes for sure19:35
jgriffithsbezverk: multi-backend is a special case that's kinda become a default :)19:35
jgriffithsbezverk: ignore it19:35
sbezverkjgriffith: it just complicates the logic but should be doable19:35
jgriffithsbezverk: you're better off and more easily scale without it19:35
*** alonma has quit IRC19:35
sbezverkjgriffith: thank you for clarification..19:36
jgriffithsbezverk: sure19:36
erloneharney: yes, it defaults to 119:36
jgriffitheharney: interesting... so if we fully recognize that we're doing this on purpose and it's REALLY REALLY what we want  it *might* be ok :)19:37
jgriffitheharney: that aligns with the ML thread I posted19:37
jgriffitheharney: thanks!19:37
eharneyjgriffith: maybe so, it's been a long while since i looked at it, so i'm not sure what the current thinking is19:37
jgriffitheharney: I'll let ya know after I get flame sprayed :)19:37
eharneyhaha19:38
erloneharney: but if that is set, we fall in the same permission problem, and the snapshots will only work with security_mode=False19:39
jgriffitheharney: looks like that's the right approach https://github.com/openstack-dev/grenade/tree/master/projects/60_nova/from-mitaka19:40
jgriffitheharney: thanks once again19:40
*** rcernin has joined #openstack-cinder19:41
scottdaeharney: If I want to add thin_provisioning_tools to devstack to fix https://bugs.launchpad.net/cinder/+bug/1615134 , does the bug stay as a "cinder" bug? Or does it need to change to "devstack"?19:41
openstackLaunchpad bug 1615134 in Cinder "volume create fails in /usr/sbin/thin_check" [Undecided,New]19:41
eharneyerlon: my opinion is that the secure mode stuff seems to have a few other problems anyway, so that's a fine starting point19:42
scottdaeharney: I'm thinking it should be Devstack bug...19:43
eharneyscottda: IMO we shouldn't change Cinder to do anything there, so it should just move to the devstack component19:43
scottdaeharney: Thanks.19:43
eharneyscottda: Ubuntu's packages seem kinda broken there, but i dunno how easily we can change that19:43
erloneharney: what other problems?19:43
eharneyerlon: we've had to advise a customer or two to turn it off due to seeing some other failures -- i don't recall what they were at the moment19:44
erloneharney: I think that adding the option to allow secure mode with some configuration in compute nodes its a better starting point than not having it at all19:45
eharneyerlon: sure, i think it's a great idea.  I just think it needs some further work to actually work right.19:45
erloneharney: and it won't change the behavior of the no_secure configuration19:45
*** sdake has quit IRC19:46
erloneharney: ok, ill push the changes I made so you can have a look, but basically I removed the places were you call run_as_root=True, and changed it to self._execute_as_root19:47
*** xinli has quit IRC19:47
eharneyerlon: i'm pretty sure that some of those were done that way because in cases where qemu-img needs to read from the qcow2 headers and the files are attached/owned by libvirt/qemu, it will fail if not run as root19:48
eharneyerlon: there's a comment in nfs.py:424 to that effect19:48
*** rcernin has quit IRC19:48
scottdaeharney: and others, fix for thin_provisioning on (Ubuntu) devstack: https://review.openstack.org/#/c/366309/19:50
erloneharney: exactly, but it just does not make sense to have some commands running as root while others are as user. If there's any command running as root, the setup can't be called secure, as the admin could be lead to believe19:51
erloneharney: so, the idea is either run insecure with norootsquash, or run secure, with the aditional configuration on computes and rootsquashing19:52
eharneyerlon: i dunno about that argument.  But the fundamental error that the nas_secure work made was that it mixed up the two questions of "who owns these files?" and "should these commands run with sudo?" in a way that needs to be fixed now19:52
erloneharney: hmm19:54
erloneharney: what would be the ideal behavior of the nas_secure?19:54
*** crose has joined #openstack-cinder19:54
*** diablo_rojo has quit IRC19:55
eharneyerlon: support root-squash and only run the minimum number of commands that are required as root, i suppose19:55
*** diablo_rojo has joined #openstack-cinder19:55
eharneyand, well, support NFS snapshots19:56
eharneybrb19:56
*** eharney has quit IRC19:56
erloneharney: ok19:56
*** porrua has quit IRC19:59
*** mvk has quit IRC20:01
*** kfarr has joined #openstack-cinder20:02
*** raunak has joined #openstack-cinder20:03
*** sdake has joined #openstack-cinder20:03
smcginnisjgriffith: Part of the reason to go to privsep was to get away from rootwrap and the need to change the filter file on upgrade.20:07
smcginnisjgriffith: That was a big blocker for awhile since the grenade "rules" state that you can't change config files.20:07
*** eharney has joined #openstack-cinder20:08
openstackgerritJohn Griffith proposed openstack/cinder: Ignore lvm descriptor leak warning on vgs calls  https://review.openstack.org/36516720:08
smcginnisjgriffith: You probably don't care anymore, but I just saw your earlier ping. :)20:08
jgriffithsmcginnis: :)20:08
smcginniserlon: Still looking for me?20:08
erlonsmcginnis: hey Sean, yes20:08
erlonsmcginnis: I'm thinking about the documention for the NFS driver20:08
*** raunak has quit IRC20:09
smcginniserlon: Or lack thereof? :)20:09
erlonsmcginnis: is there a good place to put it inside cinder repo or only in openstack manuals20:09
erlonsmcginnis: yeaah20:09
smcginniserlon: Right now there's only the devref in the cinder repo. Which as its name implies is a reference for development.20:09
smcginniserlon: But we are starting to get more and more that I'd like generally published like the sample.conf, etc.20:10
smcginniserlon: So I think right now openstack-manuals is the only logical place, but maybe that will change.20:10
openstackgerritMerged openstack/os-brick: Remove self.__dict__ for formatting strings  https://review.openstack.org/36404420:10
smcginniserlon: You could add info to the devref, even though it's not a great fit. But at least somewhere else to capture details for now if it makes sense.20:10
erlonsmcginnis: hmm, ok, I hopefully the RST formatting won't be the PITA that was when the docs where in XML20:11
erlonsmcginnis: mhm20:11
smcginniserlon: Hmm, yeah. A little reformatting would have to be done.20:11
*** arch-nemesis has quit IRC20:12
patrickeastthe config reference is much nicer to deal with now20:12
patrickeasta million times better than the xml stuff20:13
smcginnispatrickeast: Oh, haven't looked lately.20:13
patrickeasterlon: smcginnis: https://raw.githubusercontent.com/openstack/openstack-manuals/master/doc/config-reference/source/block-storage/drivers/nfs-volume-driver.rst20:13
*** edmondsw has quit IRC20:13
smcginnispatrickeast: Nice!20:14
erlonpatrickeast: ow, that is beautiful!20:14
smcginniserlon: So probably best to just add to that.20:14
erlonsmcginnis: yep, Ill20:14
*** akerr has quit IRC20:15
*** raunak has joined #openstack-cinder20:18
*** Yogi1 has quit IRC20:20
*** crose has quit IRC20:23
*** salv-orl_ has joined #openstack-cinder20:30
*** salv-orlando has quit IRC20:33
*** cdelatte has quit IRC20:44
openstackgerritErlon R. Cruz proposed openstack/cinder: NFS snapshots  https://review.openstack.org/14718620:54
*** rlrossit_ has quit IRC20:54
*** chris_morrell has quit IRC20:54
openstackgerritSean McGinnis proposed openstack/cinder: Dell SC: Error attaching after LV-AFO  https://review.openstack.org/36145320:56
*** esker is now known as esker[away]21:00
*** timcl has quit IRC21:00
*** mriedem has joined #openstack-cinder21:01
*** sdake has quit IRC21:02
*** esker[away] is now known as esker21:04
*** raunak has quit IRC21:06
*** pauloewerton has quit IRC21:10
*** raunak has joined #openstack-cinder21:11
*** raunak has quit IRC21:12
*** mvk has joined #openstack-cinder21:13
*** raunak has joined #openstack-cinder21:18
*** xyang has quit IRC21:20
*** Apoorva has joined #openstack-cinder21:21
openstackgerritMerged openstack/cinder: EMC VMAX - multi-attach failure to VMAX3 when SLO is omitted  https://review.openstack.org/30895221:25
openstackgerritMerged openstack/cinder: VMAX Driver - Live Migration for VMAX3  https://review.openstack.org/33077821:28
*** xyang has joined #openstack-cinder21:29
*** ametts has quit IRC21:30
*** esker is now known as esker[away]21:30
*** diablo_rojo has quit IRC21:36
*** alonma has joined #openstack-cinder21:39
openstackgerritJohn Griffith proposed openstack/cinder: Ignore lvm descriptor leak warning on vgs calls  https://review.openstack.org/36516721:43
*** alonma has quit IRC21:43
*** diogogmt has quit IRC21:45
*** alonma has joined #openstack-cinder21:46
*** esker[away] is now known as esker21:50
*** alonma has quit IRC21:51
*** stevemar has quit IRC21:52
*** alonma has joined #openstack-cinder21:53
*** lprice has quit IRC21:54
*** alonma has quit IRC21:58
*** alonma has joined #openstack-cinder22:03
*** xyang has quit IRC22:07
*** alonma has quit IRC22:08
*** alyson_ has quit IRC22:09
openstackgerritTom Swanson proposed openstack/cinder: Dell SC: Error attaching after LV-AFO  https://review.openstack.org/36145322:10
*** jungleboyj has joined #openstack-cinder22:13
*** kfarr has quit IRC22:13
scottdaGreat. Devstack patch fails on unittests because it cannot delete cowsay.22:17
smcginnisscottda: lol22:17
*** xyang1 has quit IRC22:17
*** eharney has quit IRC22:18
smcginnis _________________22:18
smcginnis< That's annoying >22:18
smcginnis -----------------22:18
smcginnis        \   ^__^22:18
smcginnis         \  (oo)\_______22:18
smcginnis            (__)\       )\/\22:18
smcginnis                ||----w |22:18
smcginnis                ||     ||22:18
openstackgerritWalter A. Boring IV (hemna) proposed openstack/cinder: 3PAR use OSLO versioned Object for volume  https://review.openstack.org/33914722:18
hemnacowsay!22:19
scottda Looks like a generic problem with xenial. At least I get to file a bug against devstack for the 'cowsay' test not passing.22:24
*** stevemar has joined #openstack-cinder22:24
scottdacrap, known issue: https://bugs.launchpad.net/bugs/161418522:25
openstackLaunchpad bug 1614185 in devstack "devstack run_tests.sh fails on Ubuntu xenial" [Undecided,In progress] - Assigned to Masanori Itoh (thatsdone)22:25
scottdaThat's my signal to go cook dinner.22:25
*** stevemar has quit IRC22:26
*** _ducttape_ has joined #openstack-cinder22:26
*** _ducttape_ has quit IRC22:26
*** stevemar has joined #openstack-cinder22:26
Swansonhmmm.... can't pull the last patch I pushed.22:27
SwansonThat's irritating.22:27
*** merooney has quit IRC22:28
smcginnisSwanson: What happens when you try to?22:29
*** ducttape_ has quit IRC22:29
Swansonsmcginnis, from a devstack machine it gives me couldn't find remote ref. Although it finds the previous ref just fine.22:30
smcginnisSwanson: Are you sure you have all the numbers right?22:30
Swansonsmcginnis, from my dev (read with git setup for github) machine it works just fine.22:30
*** stevemar has quit IRC22:31
Swansonsmcginnis, yes22:31
*** krtaylor has joined #openstack-cinder22:33
Swansonsmcginnis, Patch is above ^^^ using CINDER_BRANCH=refs/changes/53/361453/722:35
Swansonsmcginnis, CINDER_BRANCH=refs/changes/53/361453/6 works.22:35
Swansonsmcginnis, this is where you tell me the stupid thing I'm doing before I bug infra.22:35
smcginnisSwanson: Nope, looks right to me.22:36
*** rajinir has joined #openstack-cinder22:36
smcginnisSwanson: Maybe something isn't getting synced right away? I'd try one more time, then bug infra.22:36
*** akshai has quit IRC22:53
*** thiagolib has quit IRC22:58
openstackgerritDrew Balfour proposed openstack/cinder: zfssaiscsi driver initiator group option processing  https://review.openstack.org/35141322:59
*** alonma has joined #openstack-cinder23:01
openstackgerritMerged openstack/cinder: Huawei: Modify array LUNs naming rules  https://review.openstack.org/30564223:03
*** laurent\ has quit IRC23:04
*** alonma has quit IRC23:06
*** krtaylor has quit IRC23:06
*** stevemar has joined #openstack-cinder23:07
*** sdake has joined #openstack-cinder23:10
*** alonma has joined #openstack-cinder23:13
jgriffithsbezverk: so I added this  https://review.openstack.org/#/c/366320/23:17
jgriffithsbezverk: and updated https://review.openstack.org/#/c/365167/ with a "Depends-On"23:18
jgriffithsbezverk: but it doesn't seem to work :(23:18
jgriffithI'll have to ask around and see what I'm doing wrong here and how to get it fixed up23:18
*** alonma has quit IRC23:18
jgriffithsbezverk: or if there's another alternative here... like maybe I just make it a config option ?23:18
jgriffithsbezverk: and keep both sets of filters in place23:19
*** markvoelker has quit IRC23:19
sbezverkjgriffith: not sure, I mean we do not control anyhitng23:19
sbezverkjgriffith: if you can introduce cinder.conf variable which would disable thie warning, then it will work23:20
*** alonma has joined #openstack-cinder23:20
sbezverkotherthan that I do not know :-(23:20
jgriffithsbezverk: yeah... I'll have to ping some folks that are more knowledgeable on this process than I am and go from there23:21
jgriffithsbezverk: either that or see what smcginnis and other cores think of the conf idea23:22
*** akshai has joined #openstack-cinder23:22
jgriffithsbezverk: I kinda hate to do that but might be worth considering to avoid compatibility issues23:23
*** alonma has quit IRC23:25
*** mriedem has quit IRC23:30
*** laurent\ has joined #openstack-cinder23:30
*** mriedem has joined #openstack-cinder23:32
*** akshai has quit IRC23:33
*** bapalm has quit IRC23:35
*** esker is now known as esker[away]23:37
openstackgerritAnthony Lee proposed openstack/cinder: Disabling online volume copy in HPE 3PAR drivers  https://review.openstack.org/34806023:40
*** bapalm has joined #openstack-cinder23:41
*** raunak has quit IRC23:41
*** merooney has joined #openstack-cinder23:46
*** markvoelker has joined #openstack-cinder23:57
*** bardia has quit IRC23:59

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