Thursday, 2020-08-13

*** rambo_li has joined #openstack-cinder00:27
*** hamalq has quit IRC00:36
*** rambo_li has quit IRC00:41
*** spatel has joined #openstack-cinder00:56
*** zhanglong has joined #openstack-cinder01:01
*** rcernin has quit IRC01:11
*** rcernin has joined #openstack-cinder01:11
*** zhanglong has quit IRC01:13
*** zhanglong has joined #openstack-cinder01:14
*** Liang__ has joined #openstack-cinder01:23
*** whoami-rajat__ has quit IRC01:48
*** rambo_li has joined #openstack-cinder01:50
*** spatel has quit IRC02:09
*** spatel has joined #openstack-cinder02:14
*** spatel has quit IRC02:16
*** Liang__ has quit IRC03:01
*** Liang__ has joined #openstack-cinder03:06
*** rcernin has quit IRC03:06
*** Liang__ has quit IRC03:10
*** Liang__ has joined #openstack-cinder03:14
*** psachin has joined #openstack-cinder03:31
*** rcernin has joined #openstack-cinder03:52
*** lseki has quit IRC04:01
*** Liang__ has quit IRC04:02
*** eharney has quit IRC04:05
*** Liang__ has joined #openstack-cinder04:05
*** zhanglong has quit IRC04:06
*** eharney has joined #openstack-cinder04:24
*** CeeMac has quit IRC04:26
*** vishalmanchanda has joined #openstack-cinder04:32
*** Liang__ has quit IRC04:34
*** zhanglong has joined #openstack-cinder06:02
*** CeeMac has joined #openstack-cinder06:04
*** brinzhang has quit IRC06:06
*** zhanglong has quit IRC06:11
*** rambo_li has quit IRC06:12
*** dsariel has joined #openstack-cinder06:17
*** e0ne has joined #openstack-cinder06:19
*** e0ne has quit IRC06:20
*** zhanglong has joined #openstack-cinder06:21
*** markvoelker has joined #openstack-cinder06:45
*** Liang__ has joined #openstack-cinder06:46
*** zhanglong has quit IRC06:48
*** markvoelker has quit IRC06:49
*** zhanglong has joined #openstack-cinder06:51
*** abdysn has joined #openstack-cinder07:18
*** Liang__ has quit IRC07:23
*** Liang__ has joined #openstack-cinder07:24
*** geguileo has quit IRC07:27
*** geguileo has joined #openstack-cinder07:27
*** e0ne has joined #openstack-cinder07:35
*** markvoelker has joined #openstack-cinder07:35
*** vishakha has joined #openstack-cinder07:39
*** markvoelker has quit IRC07:39
*** tosky has joined #openstack-cinder07:42
*** zhanglong has quit IRC07:46
*** markvoelker has joined #openstack-cinder07:50
*** TusharTgite has joined #openstack-cinder07:51
*** markvoelker has quit IRC07:51
*** zhanglong has joined #openstack-cinder07:54
*** Lucas_Gray has joined #openstack-cinder08:01
*** pcaruana has quit IRC08:07
*** Wryhder has joined #openstack-cinder08:14
*** Lucas_Gray has quit IRC08:15
*** Wryhder is now known as Lucas_Gray08:15
*** pcaruana has joined #openstack-cinder08:22
*** openstackgerrit has joined #openstack-cinder08:43
openstackgerritfengjiankui proposed openstack/cinder master: Add backup_swift_storage_policy cfg  https://review.opendev.org/74602508:43
*** brinzhang has joined #openstack-cinder08:43
*** rcernin has quit IRC09:05
*** pcaruana has quit IRC09:09
*** pcaruana has joined #openstack-cinder09:21
*** Lucas_Gray has quit IRC09:22
*** Liang__ has quit IRC09:24
*** Lucas_Gray has joined #openstack-cinder09:30
*** Lucas_Gray has quit IRC09:48
*** Wryhder has joined #openstack-cinder09:48
*** Wryhder is now known as Lucas_Gray09:49
*** rcernin has joined #openstack-cinder09:57
*** brinzhang_ has joined #openstack-cinder09:58
*** brinzhang has quit IRC10:01
openstackgerritRajat Dhasmana proposed openstack/cinder stable/stein: RBD: Cleanup temporary file during exception  https://review.opendev.org/74604110:07
*** e0ne has quit IRC10:14
*** e0ne has joined #openstack-cinder10:15
*** brinzhang0 has joined #openstack-cinder10:33
*** ajitha has joined #openstack-cinder10:34
*** brinzhang_ has quit IRC10:36
*** brinzhang has joined #openstack-cinder10:37
ajithaHi all, I committed a patch yesterday and zuul check is failed in cinder-tempest-plugin-lvm-lio-barbican and cinder-grenade-mn-sub-volbak. Are they both due to upstream changes?10:39
*** brinzhang0 has quit IRC10:39
*** rcernin has quit IRC10:40
*** rcernin has joined #openstack-cinder10:46
*** rcernin has quit IRC11:00
rosmaitaajitha: yes, we hope to get them fixed today11:05
*** carloss has joined #openstack-cinder11:08
ajitharosmaita: ok Thank you11:11
openstackgerritGorka Eguileor proposed openstack/os-brick master: ScaleIO: Connection info backward compatibility  https://review.opendev.org/74610911:18
*** TusharTgite has quit IRC11:26
rosmaitageguileo: good catch ^^11:29
geguileorosmaita: thanks :-)11:29
rosmaitahow do you think 3rd party CI would feel about running grenade tests?11:31
*** brinzhang_ has joined #openstack-cinder11:32
openstackgerritSimon O'Donovan proposed openstack/cinder master: PowerMax Driver - Failover group vol update fix  https://review.opendev.org/74286111:32
*** brinzhang has quit IRC11:36
*** geguileo has quit IRC11:37
*** geguileo has joined #openstack-cinder11:37
openstackgerritRajat Dhasmana proposed openstack/cinder master: Default type overrides  https://review.opendev.org/73770711:39
*** whoami-rajat___ has joined #openstack-cinder11:54
toskyajitha (and rosmaita : the cinder-grenade-mn-sub-volbak issue is already fixed; the other one has a proposed fix (which I tried to improve today, if it doesn't work I will rollback to the old one)11:59
rosmaitatosky: thanks for the news11:59
rosmaitathe for lvm-lio-barbican, the old fix was working?12:00
rosmaitai guess i can just check my DNM test patch12:00
rosmaitaand i see that it did12:01
toskyrosmaita: it was, but I tried to implement a more general solution (we briefly mentioned the need for a follow-up yesterday)12:01
rosmaitaexcellent, everything except tempest-slow-py3 has passed on the test run with your updated patch, so i guess now it's just a matter of bugging gmann and company to review & merge it12:04
*** whoami-rajat___ is now known as whoami-rajat__12:06
rosmaitagmann: ^^ please look at https://review.opendev.org/#/c/745838/12:06
rosmaitatosky: i have a quick change for you to make on https://review.opendev.org/#/c/745855/ and then that one will be ready to go12:08
toskyoh, sure12:08
toskythat's less urgent luckily12:08
rosmaitayes, so no rush12:08
*** zhanglong has quit IRC12:10
openstackgerritLuigi Toscano proposed openstack/os-brick master: bindep: remind people to sync the dependencies with devstack  https://review.opendev.org/74585512:11
toskyI didn't add the link to the review but I think it should be clear enough12:11
rosmaitai guess that's ok, but the review eliminates any ambiguity about which files or where exactly they are located12:13
*** brinzhang0 has joined #openstack-cinder12:13
*** brinzhang_ has quit IRC12:16
smcginnisrosmaita: Third party CI running grenade would be great. But I think we have a hard enough time (and they have a hard enough time) running current branch tempest.12:20
rosmaitasmcginnis: i was just dreaming about what a nice world it would be12:20
rosmaita:)12:20
smcginnisHaha, yep. ;)12:20
*** brinzhang0 has quit IRC12:26
*** josephillips has quit IRC12:27
*** josephillips has joined #openstack-cinder12:30
*** enriquetaso has joined #openstack-cinder12:54
*** ajitha has quit IRC13:03
*** ociuhandu has quit IRC13:09
*** lseki has joined #openstack-cinder13:11
*** ociuhandu has joined #openstack-cinder13:24
gmannrosmaita: ack13:26
rosmaitagmann: good morning! thanks13:26
*** ociuhandu has quit IRC13:29
openstackgerritSofia Enriquez proposed openstack/cinder-tempest-plugin master: Extend while attached volume  https://review.opendev.org/74294313:30
*** tkajinam has quit IRC13:32
gmannrosmaita: tosky so job which does not have os-brick in required_project (means pip install) are only broken right?13:48
gmannbecause install from source was handled by smcginnis fix13:49
toskygmann: yes, that's the problem; devstack can handle requirements from bindep.txt but that doesn't work for pip installed libraries13:53
gmanntosky: do you have any testing patch with fix?13:54
rosmaitahttps://review.opendev.org/74541513:54
rosmaitagmann: ^^ that one depends on tosky's fix13:54
rosmaitaand it's finally passing!!!13:55
gmannthanks13:57
*** dave-mccowan has quit IRC13:59
gmannrosmaita: tosky +2, marked that as gate blocker. let me get 2nd core to merge that14:00
toskythanks!14:00
rosmaitagmann: ty!14:00
*** dave-mccowan has joined #openstack-cinder14:01
*** ociuhandu has joined #openstack-cinder14:02
*** abdysn has quit IRC14:06
*** ajitha has joined #openstack-cinder14:10
*** ociuhandu_ has joined #openstack-cinder14:19
*** dave-mccowan has quit IRC14:21
*** ociuhandu has quit IRC14:22
*** andrebeltrami has joined #openstack-cinder14:24
*** ajitha has quit IRC14:40
*** benfelin has joined #openstack-cinder15:15
*** vishalmanchanda has quit IRC15:32
smcginnisdevstack os-brick patch merged \o/15:43
openstackgerritRajat Dhasmana proposed openstack/cinder stable/stein: RBD: Cleanup temporary file during exception  https://review.opendev.org/74604115:44
*** e0ne_ has joined #openstack-cinder15:45
*** priteau has joined #openstack-cinder15:46
*** e0ne has quit IRC15:47
hemnaanyone else seeing this again in devstack?  "E: Package 'python-urwid' has no installation candidate"15:54
e0ne_hemna: I set up devstack+ceph+ubuntu18.04 about one hour ago and it works for me15:55
hemnathis is on focal15:55
e0ne_I did't  try that for w while15:57
hemnaI have to use focal for ceph-iscsi16:00
*** psachin has quit IRC16:25
openstackgerritWalter A. Boring IV (hemna) proposed openstack/devstack-plugin-ceph master: Update override-defaults to support ceph_iscsi  https://review.opendev.org/66866716:28
*** vishakha has quit IRC16:34
*** Lucas_Gray has quit IRC16:36
*** ociuhandu_ has quit IRC16:43
*** tosky has quit IRC16:43
*** ociuhandu has joined #openstack-cinder16:44
*** ociuhandu has quit IRC16:50
*** ociuhandu has joined #openstack-cinder16:57
*** ociuhandu has quit IRC17:04
whoami-rajat__rosmaita: https://review.opendev.org/#/c/745838/ --- this patch fixes the gate right?17:14
whoami-rajat__lvm-lio-barbican job to be specific17:14
rosmaitathat's the theory17:14
rosmaitabut there are some other flaky aspects of that job17:14
rosmaitawhat are you seeing?17:14
whoami-rajat__ok, since that's merged i think we can recheck patches now?17:15
whoami-rajat__yeah, that can be fixed by rechecks i guess17:15
rosmaitaoh yes, please do17:15
whoami-rajat__great, thanks for all the work guys!17:15
rosmaitait was all tosky and smcginnis ^^17:16
whoami-rajat__yep, thanks everyone involved17:18
whoami-rajat__rosmaita: it would be great to get this one in now -- https://review.opendev.org/#/c/741498/1317:18
rosmaitawhoami-rajat__: i agree ... i've been waiting for a zuul +1 to give the patch a +217:19
whoami-rajat__ack, will give you a ping when zuul passes17:22
rosmaitaok, i will keep an eye on it too17:22
whoami-rajat__great, thanks17:23
*** priteau has quit IRC17:36
*** priteau has joined #openstack-cinder17:45
*** e0ne_ has quit IRC17:50
*** priteau has quit IRC17:53
*** andrebeltrami has quit IRC18:03
*** spatel has joined #openstack-cinder18:24
*** ajitha has joined #openstack-cinder18:45
*** e0ne has joined #openstack-cinder19:01
openstackgerritMerged openstack/os-brick master: Improve WWN detection  https://review.opendev.org/73240719:01
*** openstackgerrit has quit IRC19:02
*** e0ne has quit IRC19:15
*** ajitha has quit IRC19:38
*** andrebeltrami has joined #openstack-cinder19:40
whoami-rajat__rosmaita:  the default type patch got +1 from zuul ... https://review.opendev.org/#/c/741498/1319:44
rosmaita\o/19:44
whoami-rajat__rosmaita: also is there any intent of backporting this one? i think not since we are mandating a CONF field which was optional before19:46
rosmaitai think we should backport it to Train19:47
rosmaitathe config option already existed, and it has a default that behaves the same as the old hard-coded default19:47
whoami-rajat__yeah, functionality remains same, i don't see any strong reason not to but at the same time i worry about breaking any existing deployment with this19:50
whoami-rajat__i think you're right, it seems safe19:50
rosmaitawe can think about it some more, but i think it's safe and will mean consistent behavior across releases, so easier on operators19:51
whoami-rajat__the main thing in my mind is we introduced 3 new exceptions failing at different steps that operators aren't aware about but yeah lets discuss it later19:54
*** e0ne has joined #openstack-cinder19:59
*** e0ne has quit IRC20:02
*** e0ne has joined #openstack-cinder20:07
*** e0ne has quit IRC20:07
*** tosky has joined #openstack-cinder20:10
*** dave-mccowan has joined #openstack-cinder20:22
*** ajitha has joined #openstack-cinder21:20
*** ajitha has quit IRC21:35
*** jmlowe has quit IRC21:42
*** jmlowe has joined #openstack-cinder21:43
*** rcernin has joined #openstack-cinder22:00
*** spatel has quit IRC22:20
*** andrebeltrami has quit IRC22:23
*** openstackgerrit has joined #openstack-cinder22:45
openstackgerritLuigi Toscano proposed openstack/python-cinderclient master: zuul functional job: drop the custom playbooks  https://review.opendev.org/74623722:45
openstackgerritLuigi Toscano proposed openstack/os-brick master: bindep: remind people to sync the dependencies with devstack  https://review.opendev.org/74585522:50
*** tkajinam has joined #openstack-cinder22:58
*** ociuhandu has joined #openstack-cinder23:00
*** ociuhandu has quit IRC23:05
openstackgerritMerged openstack/os-brick master: iSCSI detect multipath DM with no WWN  https://review.opendev.org/73242323:08
*** tosky has quit IRC23:09
*** enriquetaso has quit IRC23:26
*** ircuser-1 has quit IRC23:29

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