Wednesday, 2020-09-09

*** hamalq has quit IRC00:31
*** Liang__ has joined #openstack-cinder01:23
*** amar7ibm has joined #openstack-cinder02:02
*** Liang__ is now known as LiangFang02:08
*** enriquetaso has quit IRC02:27
*** rcernin has quit IRC02:29
*** josephillips has quit IRC02:43
*** udesale has joined #openstack-cinder03:07
*** zzzeek has quit IRC03:18
*** zzzeek has joined #openstack-cinder03:19
*** sapd1_x has joined #openstack-cinder03:27
*** rcernin has joined #openstack-cinder03:29
*** rcernin has quit IRC03:29
*** rcernin has joined #openstack-cinder03:29
*** zzzeek has quit IRC03:55
*** abishop_ has joined #openstack-cinder03:55
*** udesale_ has joined #openstack-cinder03:55
*** zzzeek has joined #openstack-cinder03:58
*** udesale has quit IRC03:58
*** abishop has quit IRC03:58
*** sapd1_x has quit IRC04:03
*** sapd1_x has joined #openstack-cinder04:04
openstackgerritTushar Trambak Gite proposed openstack/cinder master: Stop configuring install_command in tox.  https://review.opendev.org/75030804:29
*** manoj_kumar_kata has joined #openstack-cinder04:29
openstackgerritTushar Trambak Gite proposed openstack/os-brick master: Stop configuring install_command in tox.  https://review.opendev.org/75030904:30
openstackgerritTushar Trambak Gite proposed openstack/cinderlib master: Stop configuring install_command in tox.  https://review.opendev.org/75031004:30
openstackgerritTushar Trambak Gite proposed openstack/python-cinderclient master: Stop configuring install_command in tox.  https://review.opendev.org/75031104:31
*** sapd1_x has quit IRC04:35
*** dsariel has joined #openstack-cinder04:41
*** zzzeek has quit IRC04:45
*** zzzeek has joined #openstack-cinder04:48
openstackgerritLiang Fang proposed openstack/cinder master: Add support volume local cache  https://review.opendev.org/70079905:02
*** zzzeek has quit IRC05:17
*** zzzeek has joined #openstack-cinder05:18
*** manoj_kumar_kata has quit IRC05:36
*** Reepicheep has quit IRC05:42
*** Reepicheep has joined #openstack-cinder05:42
*** pcaruana has joined #openstack-cinder06:08
*** vishalmanchanda has joined #openstack-cinder06:14
*** m75abrams has joined #openstack-cinder06:16
*** LiangFang has quit IRC06:17
*** Liang__ has joined #openstack-cinder06:18
*** e0ne has joined #openstack-cinder06:35
*** benfelin has joined #openstack-cinder06:37
*** brinzhang_ has joined #openstack-cinder06:39
*** e0ne has quit IRC06:39
*** brinzhang0 has quit IRC06:42
*** Reepicheep has quit IRC06:43
*** abdysn has joined #openstack-cinder06:47
*** Reepicheep has joined #openstack-cinder06:47
*** manoj_kumar_kata has joined #openstack-cinder07:20
*** dsariel has quit IRC07:41
*** dsariel has joined #openstack-cinder07:47
*** dsariel has quit IRC07:52
*** sapd1_x has joined #openstack-cinder07:52
*** pcaruana has quit IRC07:55
*** tosky has joined #openstack-cinder07:57
*** Liang__ is now known as LiangFang08:00
openstackgerritYong Huang proposed openstack/cinder master: [Unity]: Fix failback issue of multiple volumes  https://review.opendev.org/75059008:01
*** manoj_kumar_kata has quit IRC08:04
*** manoj_kumar_kata has joined #openstack-cinder08:04
*** pcaruana has joined #openstack-cinder08:04
*** Roamer` has joined #openstack-cinder08:05
*** brinzhang0 has joined #openstack-cinder08:09
*** brinzhang_ has quit IRC08:12
*** rcernin has quit IRC08:20
*** Lucas_Gray has joined #openstack-cinder08:30
*** Reepicheep has quit IRC08:32
*** Reepicheep has joined #openstack-cinder08:33
openstackgerritMerged openstack/os-brick master: Revert "Fix for rbd connector to work with ceph octopus"  https://review.opendev.org/75044408:34
*** ianychoi__ has quit IRC08:40
*** martinkennelly has joined #openstack-cinder08:43
*** brinzhang_ has joined #openstack-cinder08:53
*** brinzhang0 has quit IRC08:56
*** Reepicheep has quit IRC09:15
*** sapd1_x has quit IRC09:15
*** Reepicheep has joined #openstack-cinder09:24
*** Lucas_Gray has quit IRC09:25
*** lxkong has joined #openstack-cinder09:27
*** LiangFang has quit IRC09:29
*** e0ne has joined #openstack-cinder09:32
*** lxkong has quit IRC09:34
*** Lucas_Gray has joined #openstack-cinder09:34
*** Abdallahyas has joined #openstack-cinder09:44
*** abdysn has quit IRC09:45
*** Reepicheep has quit IRC10:33
*** Reepicheep has joined #openstack-cinder10:38
*** udesale__ has joined #openstack-cinder10:41
*** priteau has joined #openstack-cinder10:42
*** udesale_ has quit IRC10:43
*** manoj_kumar_kata has quit IRC10:45
*** manoj_kumar_kata has joined #openstack-cinder10:46
*** manoj_kumar_kata has quit IRC10:47
*** manoj_kumar_kata has joined #openstack-cinder10:47
*** rcernin has joined #openstack-cinder10:52
*** manoj_kumar_kata has quit IRC10:55
*** manoj_kumar_kata has joined #openstack-cinder10:55
*** sapd1_x has joined #openstack-cinder11:17
*** Abdallahyas has quit IRC11:19
*** abdysn has joined #openstack-cinder11:23
*** rcernin has quit IRC11:28
*** rcernin has joined #openstack-cinder11:44
*** sapd1_x has quit IRC11:56
*** Abdallahyas has joined #openstack-cinder12:27
*** abdysn has quit IRC12:30
*** andrebeltrami has joined #openstack-cinder12:36
*** rcernin has quit IRC12:41
*** ajitha has joined #openstack-cinder12:43
*** jgriffith has quit IRC12:44
*** enriquetaso has joined #openstack-cinder12:56
*** manoj_kumar_kata has quit IRC13:05
*** manoj_kumar_kata has joined #openstack-cinder13:05
*** Reepicheep has quit IRC13:06
openstackgerritBrian Rosmaita proposed openstack/os-brick stable/victoria: Revert "Fix for rbd connector to work with ceph octopus"  https://review.opendev.org/75064913:08
*** Reepicheep has joined #openstack-cinder13:08
*** rcernin has joined #openstack-cinder13:14
*** udesale__ has quit IRC13:17
*** udesale has joined #openstack-cinder13:20
*** kaisers_ has joined #openstack-cinder13:22
*** brinzhang0 has joined #openstack-cinder13:26
*** sapd1_x has joined #openstack-cinder13:27
*** brinzhang_ has quit IRC13:29
*** udesale has quit IRC13:32
openstackgerritBrian Rosmaita proposed openstack/os-brick stable/victoria: New fix for rbd connector to work with ceph octopus  https://review.opendev.org/75065513:32
*** udesale has joined #openstack-cinder13:32
*** rcernin has quit IRC13:41
*** sfernand has joined #openstack-cinder13:41
*** udesale has quit IRC13:45
*** udesale_ has joined #openstack-cinder13:45
*** vishakha has joined #openstack-cinder13:47
amar7ibm@smcginnis @core-reviewers: please review this and address your comments accordingly, https://review.opendev.org/#/c/747425/13:51
*** thgcorrea has joined #openstack-cinder13:54
*** Liang__ has joined #openstack-cinder13:55
*** amar7ibm has quit IRC13:55
*** Liang__ is now known as LiangFang13:55
toskyrosmaita: time for the ping? Or was the meeting cancelled this week?13:57
rosmaitausually 2 minutes before13:58
toskyoh, I've got confused with other meetings' reminders then13:58
rosmaitaCourtesy reminder: Cinder meeting in #openstack-meeting-alt at 1400 UTC13:59
rosmaitajungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard lseki sfernand rajinir enriquetaso hemna ^^13:59
*** whoami-rajat__ has joined #openstack-cinder13:59
*** manoj_kumar_kata has quit IRC13:59
*** LiangFang has quit IRC14:09
*** Reepicheep has quit IRC14:11
*** Liang__ has joined #openstack-cinder14:13
*** Reepicheep has joined #openstack-cinder14:15
*** Liang__ is now known as LiangFang14:15
*** Reepicheep has quit IRC14:35
*** Reepicheep has joined #openstack-cinder14:37
*** udesale_ has quit IRC14:45
*** venkatakrishnath has joined #openstack-cinder14:56
venkatakrishnathhttps://review.opendev.org/#/c/744683/ got Zuul +1 and +1 from Tushar Trambak Gite and storage CI(IBM storage CI) also succeeded. Could someone please review.14:57
rosmaitaok, let's discuss os-brick 4.0.115:01
rosmaitaparticularly, the safety of this patch:15:01
rosmaitahttps://review.opendev.org/#/c/750655/15:01
whoami-rajat__is this tested against glance + cinder + rbd (octopus) ?15:02
whoami-rajat__glance backend is cinder15:02
openstackgerritMerged openstack/os-brick master: New fix for rbd connector to work with ceph octopus  https://review.opendev.org/75045215:02
*** Abdallahyas has quit IRC15:03
rosmaitawhoami-rajat__: no, i don't think that combination has been tested15:07
*** manoj_kumar_kata has joined #openstack-cinder15:08
rosmaitabut my impression was that the problem RDO reported was that the 'ceph' binary was missing15:08
rosmaitathis fix doesn't require it15:08
whoami-rajat__the ceph binary was missing on the glance container when glance backend was cinder (rbd)15:09
*** m75abrams has quit IRC15:09
whoami-rajat__the patch tried to execute ceph --version  on the glance container which shouldn't be run there15:10
smcginnisI'm not sure what the concern is if ceph has supported (and recommended?) this [default] section for years.15:10
rosmaitayeah, the new patch just writes the "new" style config unconditionally15:10
smcginnisSeems like what we would want to do anyway.15:10
*** bhagyashris is now known as bhagyashris|ruck15:11
rosmaitawe have verified that it works (in the sense that the options are found) for octopus, nautilus, and luminous15:11
*** bhagyashris|ruck is now known as bhagyashris|rove15:11
rosmaitaand the ceph docs say it's supposed to be there at least back to hammer15:11
*** LiangFang has quit IRC15:11
rosmaitawhich i think is the liberty openstack time frame15:11
*** lpetrut has joined #openstack-cinder15:12
jungleboyjSeems like we should do it then.15:12
*** penick has joined #openstack-cinder15:12
rosmaitayes, brick is basically doing exactly the same thing it has been doing15:13
rosmaitajust one extra line in the file15:13
rosmaitaso no new privsep stuff or anything like that15:13
smcginnisMuch safer than adding conditional logic and external calls.15:14
jungleboyj++15:14
rosmaitai think this is safe -- the problem last week was we wanted to beat the release deadline, and the current change hadn't been tested against anything other than octopus and nautilus15:14
rosmaitaso we went with the conditional logic and external calls, which worked as long as you had that binary installed15:15
*** manoj_kumar_kata has quit IRC15:21
rosmaitawhoami-rajat__: i don't want to close off debate early if you still have concerns?15:21
whoami-rajat__rosmaita: i'm setting up an env with the configuration i mentioned above, i will let you know if i find anything unusual15:23
openstackgerritMichael Arndt proposed openstack/cinder master: [NetApp] Adding support for Adaptive QoS in NetApp driver.  https://review.opendev.org/74132715:23
rosmaitawhoami-rajat__: ok, cool15:23
whoami-rajat__but AFAIC, the new patch looks kind of safe15:23
openstackgerritMichael Arndt proposed openstack/cinder master: [NetApp] Adding support for Adaptive QoS in NetApp driver.  https://review.opendev.org/74132715:25
*** Lucas_Gray has quit IRC15:29
*** penick has quit IRC15:30
*** Lucas_Gray has joined #openstack-cinder15:30
*** zzzeek has quit IRC15:31
*** zzzeek has joined #openstack-cinder15:32
openstackgerritRajat Dhasmana proposed openstack/cinder master: Default type overrides  https://review.opendev.org/73770715:36
*** Reepicheep has quit IRC15:37
*** zzzeek has quit IRC15:37
*** zzzeek has joined #openstack-cinder15:38
*** Reepicheep has joined #openstack-cinder15:39
*** zzzeek has quit IRC15:42
*** zzzeek has joined #openstack-cinder15:45
*** penick has joined #openstack-cinder15:47
*** venkatakrishnath has quit IRC15:48
*** zzzeek has quit IRC15:50
*** zzzeek has joined #openstack-cinder15:51
*** josephillips has joined #openstack-cinder15:51
*** zzzeek has quit IRC15:56
openstackgerritSimon O'Donovan proposed openstack/cinder master: PowerMax Driver - Remove deprecated config options  https://review.opendev.org/74395715:57
*** lpetrut has quit IRC15:59
*** zzzeek has joined #openstack-cinder15:59
openstackgerritSimon O'Donovan proposed openstack/cinder master: PowerMax Driver - Force array and srp configuration  https://review.opendev.org/75018116:03
*** zzzeek has quit IRC16:04
*** zzzeek has joined #openstack-cinder16:05
*** sapd1_x has quit IRC16:10
*** Lucas_Gray has quit IRC16:12
*** Lucas_Gray has joined #openstack-cinder16:20
openstackgerritMichael Arndt proposed openstack/cinder master: [NetApp] Adding support for Adaptive QoS in NetApp driver.  https://review.opendev.org/74132716:20
*** Reepicheep has quit IRC16:22
*** Reepicheep has joined #openstack-cinder16:23
*** josephillips has quit IRC16:31
*** Reepicheep has quit IRC16:35
*** Reepicheep has joined #openstack-cinder16:38
*** Lucas_Gray has quit IRC16:51
*** dosaboy has quit IRC16:51
*** manoj_kumar_kata has joined #openstack-cinder16:54
*** abishop_ is now known as abishop|afk16:55
whoami-rajat__rosmaita: so i checked and glance cinder backend is able to create images on ceph octopus16:57
*** Lucas_Gray has joined #openstack-cinder16:58
*** penick has quit IRC17:00
openstackgerritMerged openstack/cinder master: Add support volume local cache  https://review.opendev.org/70079917:01
*** Reepicheep has quit IRC17:03
*** Lucas_Gray has quit IRC17:03
*** Reepicheep has joined #openstack-cinder17:05
*** andrebeltrami has quit IRC17:06
*** Lucas_Gray has joined #openstack-cinder17:07
*** Reepicheep has quit IRC17:12
*** Reepicheep has joined #openstack-cinder17:16
manoj_kumar_kata@smcginnis and core reviewers : could you please review this, i have addressed all the comments17:19
manoj_kumar_katahttps://review.opendev.org/#/c/744652/17:19
openstackgerritMichael Arndt proposed openstack/cinder master: [NetApp] Adding support for Adaptive QoS in NetApp driver.  https://review.opendev.org/74132717:23
*** GirishChilukuri has joined #openstack-cinder17:26
*** Lucas_Gray has quit IRC17:31
whoami-rajat__rosmaita: also the unit test passed on my default type patch17:31
openstackgerritPeter Penchev proposed openstack/cinder master: Implement revert_to_snapshot() for StorPool.  https://review.opendev.org/68088917:36
*** e0ne has quit IRC17:37
Roamer`hi, so.... yeah... I know it's really close to the FF cutoff on Friday, so this time I'm not even going to beg - if you people say "you should have come sooner", fine, next release it is... But if anybody feels like reviewing a really trivial "pass the revert-to-snapshot request directly to the StorPool API" patch, I'd be grateful.17:38
Roamer`(for the record, it's already been activated in our CI and the revert tests passed in the https://spfactory.storpool.com/logs/04/749704/2/silent/cinder-storpool-tempest-exp/a4f6583/ silent run)17:41
*** manoj_kumar_kata has quit IRC17:42
hemnaany reason why this hasn't verified? https://review.opendev.org/#/c/668667/17:58
clarkbhemna: the depends on is -1'd by zuul and has no code reviews17:59
openstackgerritPeter Penchev proposed openstack/cinder master: Implement revert_to_snapshot() for StorPool.  https://review.opendev.org/68088918:03
rosmaitawhoami-rajat__: ty18:03
*** Reepicheep has quit IRC18:04
hemnachicken and egg18:05
*** Reepicheep has joined #openstack-cinder18:05
openstackgerritAndrei Perepiolkin proposed openstack/cinder master: Add Cinder driver for Open-E JovianDSS data storage  https://review.opendev.org/72690618:07
openstackgerritMerged openstack/python-cinderclient master: Use importlib to take place of imp module  https://review.opendev.org/74885718:07
smcginnishemna: That depends-on actually looks backwards. Doesn't the driver need the devstack plugin overrides?18:07
rosmaitasmcginnis: will brick 4.0.1 need a release note saying how it's different from 4.0.0 ?18:11
openstackgerritMerged openstack/os-brick stable/victoria: Revert "Fix for rbd connector to work with ceph octopus"  https://review.opendev.org/75064918:21
smcginnisrosmaita: Might be nice if we quick add one with a Fixes note calling out that the last release had an issue that this one addresses.18:22
smcginnisI think if a patch only has a release note, we should have things set up so only a few of the jobs run against it. So should be a quick thing to get through.18:23
rosmaitaok, i'll get that together right now18:23
*** suryasingh has quit IRC18:23
GirishChilukurismcginnis18:25
GirishChilukurireview this changes https://review.opendev.org/#/c/746267/18:25
*** josephillips has joined #openstack-cinder18:25
openstackgerritMerged openstack/os-brick stable/victoria: New fix for rbd connector to work with ceph octopus  https://review.opendev.org/75065518:30
openstackgerritSofia Enriquez proposed openstack/cinder master: Add option for no rbd volumes in the ceph backup driver  https://review.opendev.org/75078218:31
smcginnisPlease?18:31
*** zzzeek has quit IRC18:33
*** zzzeek has joined #openstack-cinder18:34
GirishChilukurismcginnis: please review this code changes https://review.opendev.org/#/c/746267/18:38
*** zzzeek has quit IRC18:41
*** zzzeek has joined #openstack-cinder18:42
*** GirishChilukuri has quit IRC18:46
*** Reepicheep has quit IRC18:47
*** Reepicheep has joined #openstack-cinder18:48
openstackgerritBrian Rosmaita proposed openstack/os-brick stable/victoria: Release note for os-brick 4.0.1  https://review.opendev.org/75078518:49
*** zzzeek has quit IRC18:50
rosmaitasmcginnis: please let me know what you think ^^18:50
rosmaitajungleboyj: you too ^^18:50
*** zzzeek has joined #openstack-cinder18:50
jungleboyjrosmaita:  That makes sense to me.18:52
*** zzzeek has quit IRC18:55
*** zzzeek has joined #openstack-cinder18:56
*** priteau has quit IRC18:57
*** zzzeek has quit IRC19:03
*** zzzeek has joined #openstack-cinder19:05
smcginnisFYI, if we start seeing any failures, base jobs were just switched over to focal.19:12
*** zzzeek has quit IRC19:21
*** zzzeek has joined #openstack-cinder19:23
*** kaisers_ has quit IRC19:27
*** tosky has quit IRC19:37
*** vishalmanchanda has quit IRC19:45
openstackgerritMerged openstack/os-brick stable/victoria: Release note for os-brick 4.0.1  https://review.opendev.org/75078519:47
smcginnisrosmaita: Did you post a requirements FFE?20:09
rosmaitawriting it up now20:09
rosmaitawas it supposed to happen before the new release?20:09
smcginnisIt should happen before I approve the release patch. Almost did before I realized. :)20:10
rosmaitaoh, ok.  i thought we could release first, and then petition to get the upper constraints changed20:10
smcginnisWe should get it first. Otherwise we end up in this weird state where a new version is published, but it's not actually used by the release.20:11
smcginnisIt can end up being a stable update, but I'm sure it won't be a problem to get an FFE approved.20:11
rosmaitasmcginnis: how does this look?20:13
rosmaitahttps://etherpad.opendev.org/p/KLis9NnQuVa8_1myg5fa20:13
*** openstackgerrit has quit IRC20:17
smcginnisrosmaita: Ship it. :)20:18
rosmaitacool, ty20:18
*** Lucas_Gray has joined #openstack-cinder20:21
*** zenkuro has joined #openstack-cinder20:40
*** qqmber has joined #openstack-cinder20:42
qqmberHi, if I can't attach any volume from a spescific storage node with cinder, and I telnet that server to port 3260 and connects but inmediatly closes the connection (if I do the same to other servers where Cinder works fine maintain the connection open), where should I look for the problem? I think is tgtd, but I don't see anything wrong there...20:44
*** spatel has joined #openstack-cinder20:46
smcginnisqqmber: I take it you are using the LVM driver. It could be that the volume is not exported or the initiator doesn't have access to the exported LUN.20:47
smcginnisqqmber: There should be some clues in the log files.20:47
qqmberyes, LVM20:47
qqmberbut even before that.. the telnet should answer... right?20:47
qqmberin nove compute logs: "VolumeDeviceNotFound: Volume device not found at ."20:48
qqmbernova*20:48
smcginnisNot sure on telnet. I've actually never tried that.20:49
qqmberIf I do the telnet to another server that is working it keep it open... to this one (from another server) it connects and then closes it.20:49
qqmberI'll check cinder.conf stuff that talks about the LV driver...20:50
qqmber[lvm-val-2]20:50
qqmbervolume_group = cinder-vol-val-c20:50
qqmbervolume_driver = cinder.volume.drivers.lvm.LVMVolumeDriver20:50
qqmbervolume_backend_name = lvm-val-220:50
qqmbertarget_helper = tgtadm20:50
smcginnisPossible it is a configuration thing, but the logs would be the best place to look to see what the issue is.20:50
qqmbertarget_protocol = iscsi20:50
*** openstackgerrit has joined #openstack-cinder20:51
openstackgerritVictoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Stop using shaman  https://review.opendev.org/74972620:51
qqmbersmcginnis: ok, thanks.. i'll check it out20:51
*** whoami-rajat__ has quit IRC20:52
*** spatel has quit IRC20:52
*** adrianc has quit IRC21:05
*** adrianc has joined #openstack-cinder21:05
qqmbergot the problem... my_ip in cinder.conf was pointing to another IP21:10
qqmberthanks!21:11
*** ajitha has quit IRC21:11
smcginnisqqmber: Glad it was an easy one to fix!21:17
openstackgerritVictoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Stop using shaman  https://review.opendev.org/74972621:18
*** adrianc_ has joined #openstack-cinder21:22
*** adrianc has quit IRC21:22
openstackgerritGhanshyam Mann proposed openstack/cinder master: [goal] Fixing lower constraints for Ubuntu Focal  https://review.opendev.org/74308022:07
gmannrosmaita: this is fix for failing lower constraint job  - https://review.opendev.org/#/c/743080/22:09
gmannhope it pass now.22:09
*** Lucas_Gray has quit IRC22:13
rosmaitagmann: ty22:21
rosmaitagmann: thanks for that patch, i saw the job failing, but it appeared to be intermittent22:23
*** vishakha has quit IRC22:26
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax Driver - Remove mandatory failover BID  https://review.opendev.org/74648622:33
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax Driver - Remove deprecated config options  https://review.opendev.org/74395722:34
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax Driver - Force array and srp configuration  https://review.opendev.org/75018122:39
openstackgerritHelen Walsh proposed openstack/cinder master: PowerMax Driver - Feedback for migrate exception handling  https://review.opendev.org/74724322:46
*** martinkennelly has quit IRC22:53
*** rcernin has joined #openstack-cinder22:53
*** rcernin has quit IRC22:56
*** rcernin has joined #openstack-cinder22:56
*** enriquetaso has quit IRC23:13
*** zzzeek has quit IRC23:17
*** zzzeek has joined #openstack-cinder23:19
*** dosaboy has joined #openstack-cinder23:28
*** CeeMac has quit IRC23:34
*** zzzeek has quit IRC23:38
*** zzzeek has joined #openstack-cinder23:40
*** zzzeek has quit IRC23:59

Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!