Wednesday, 2020-09-02

*** baojg has quit IRC00:09
*** baojg has joined #openstack-cinder00:10
*** manoj_kumar_kata has quit IRC00:15
*** manoj_kumar_kata has joined #openstack-cinder00:15
*** devfaz has quit IRC00:35
*** devfaz has joined #openstack-cinder00:35
*** zhanglong has joined #openstack-cinder00:36
*** rcernin_ has joined #openstack-cinder00:45
*** rcernin has quit IRC00:47
*** zhanglong has quit IRC01:31
*** zhanglong has joined #openstack-cinder01:46
*** ajitha has joined #openstack-cinder02:07
*** baojg has quit IRC02:32
*** baojg has joined #openstack-cinder02:33
*** josephillips has quit IRC02:37
*** zenkuro has quit IRC03:10
*** rcernin_ has quit IRC03:19
*** dunweifu has joined #openstack-cinder03:28
*** rcernin_ has joined #openstack-cinder03:34
*** dunweifu has quit IRC03:35
*** psachin has joined #openstack-cinder03:35
*** dunweifu has joined #openstack-cinder03:37
*** dunweifu has quit IRC03:47
*** whoami-rajat__ has joined #openstack-cinder03:55
*** whoami-rajat has joined #openstack-cinder03:56
*** zhanglong has quit IRC03:58
*** udesale has joined #openstack-cinder04:16
*** m75abrams has joined #openstack-cinder04:44
*** ajitha has quit IRC04:47
openstackgerritVenkata krishna Thumu proposed openstack/cinder master: [Storwize] Option to retain the auxiliary volume  https://review.opendev.org/74468304:49
*** vishalmanchanda has joined #openstack-cinder04:55
*** baojg has quit IRC05:12
*** baojg has joined #openstack-cinder05:13
openstackgerritAmar proposed openstack/cinder master: [Storwize] Support to primary volume Migration  https://review.opendev.org/74742505:17
*** bhagyashris|away is now known as bhagyashris05:27
*** zhanglong has joined #openstack-cinder05:34
*** kevinz has joined #openstack-cinder05:34
*** dunweifu has joined #openstack-cinder05:55
*** jopdorp has quit IRC06:01
*** arne_wiebalck has quit IRC06:01
*** simondodsley_ has joined #openstack-cinder06:02
*** arne_wiebalck has joined #openstack-cinder06:02
*** jopdorp has joined #openstack-cinder06:02
*** simondodsley has quit IRC06:02
*** simondodsley_ is now known as simondodsley06:02
*** baojg has quit IRC06:02
*** rajinir__ has joined #openstack-cinder06:02
*** rajinir has quit IRC06:03
*** rajinir__ is now known as rajinir06:03
*** baojg has joined #openstack-cinder06:03
*** dunweifu has quit IRC06:08
*** benfelin has joined #openstack-cinder06:27
*** rcernin_ has quit IRC06:51
*** rcernin_ has joined #openstack-cinder06:54
*** e0ne has joined #openstack-cinder07:26
*** rcernin_ has quit IRC07:35
*** brinzhang0 has joined #openstack-cinder07:35
*** tosky has joined #openstack-cinder07:37
*** brinzhang_ has quit IRC07:38
*** baojg has quit IRC07:53
*** baojg has joined #openstack-cinder07:53
*** JaySee has quit IRC07:57
*** baojg has quit IRC07:59
*** martinkennelly has joined #openstack-cinder08:07
*** rcernin_ has joined #openstack-cinder08:39
*** JaySee has joined #openstack-cinder08:44
*** rcernin_ has quit IRC08:46
openstackgerritSharath Kacham proposed openstack/cinder master: [storwize]:changes in create_group_from_src in SVC  https://review.opendev.org/74529308:50
openstackgerritSharath Kacham proposed openstack/cinder master: [storwize]:changes in create_group_from_src in SVC  https://review.opendev.org/74529309:02
*** ajitha has joined #openstack-cinder09:11
*** baojg has joined #openstack-cinder09:28
*** zzzeek has quit IRC09:32
*** zzzeek has joined #openstack-cinder09:33
*** zzzeek has quit IRC09:48
*** zzzeek has joined #openstack-cinder09:49
*** baojg has quit IRC09:54
*** zhanglong has quit IRC10:04
*** zzzeek has quit IRC10:08
*** zzzeek has joined #openstack-cinder10:10
*** zenkuro has joined #openstack-cinder10:18
zenkurohello, can anybody give a look at https://review.opendev.org/#/c/726906/10:20
*** sapd1_x has joined #openstack-cinder10:21
e0nezenkuro: hi. I'll take a look on it but this patch missed a deadline https://releases.openstack.org/victoria/schedule.html#cinder-new-driver-merge-deadline, so it could be merged only in next release10:24
zenkuroe0ne: sad. But something is better then nothing.10:28
whoami-rajat__zenkuro: looking at the gate failures, you should run pep8 locally before updating new patchset10:39
*** rcernin_ has joined #openstack-cinder10:42
*** lpetrut has joined #openstack-cinder10:47
*** rcernin_ has quit IRC10:47
*** brinzhang_ has joined #openstack-cinder10:48
*** brinzhang0 has quit IRC10:51
*** brinzhang has joined #openstack-cinder10:51
*** brinzhang_ has quit IRC10:54
*** zzzeek has quit IRC10:57
*** zzzeek has joined #openstack-cinder11:00
*** manoj_kumar_kata has quit IRC11:11
*** manoj_kumar_kata has joined #openstack-cinder11:11
*** elod has quit IRC11:15
*** elod has joined #openstack-cinder11:16
openstackgerritJiri Suchomel proposed openstack/devstack-plugin-ceph master: zuul: Add devstack-plugin-ceph-compute-qcow2 job  https://review.opendev.org/74821211:42
*** Lucas_Gray has joined #openstack-cinder11:47
*** Lucas_Gray has quit IRC11:57
zenkurowhoami-rajat__: there is a failure? it is somewhere on https://review.opendev.org/#/c/726906/12:02
*** Lucas_Gray has joined #openstack-cinder12:05
*** dave-mccowan has joined #openstack-cinder12:18
*** udesale_ has joined #openstack-cinder12:18
zenkurowhoami-rajat: there is onlyt one fail cinder-plugin-ceph-tempest-mn-aa, and it looks like it is an issue with running devstack with ceph12:20
*** udesale has quit IRC12:20
whoami-rajatzenkuro, i was just giving a general suggestion as i saw a lot of previous gate fails due to pep812:23
zenkurowhoami-rajat: actually this is interesting question, why sometimes it fails. is it normal? I run pep8 pylint and flake8 to check12:24
whoami-rajatdid you run pep8 with tox ? ``tox -epep8``12:25
*** pcaruana has quit IRC12:30
*** Lucas_Gray has quit IRC12:30
rosmaitaeharney: geguileo: smcginnis: hemna: whoami-rajat: need some eyes on https://review.opendev.org/#/c/730376 (final os-brick patch for victoria)12:30
hemna*click*12:30
rosmaitae0ne can't do it because he is co-author12:30
e0ne:(12:31
rosmaitae0ne: you should give yourself a :D for your work on that patch12:31
smcginnisrosmaita: We don't have testing for octopus in gate, do we? I wonder if we need periodic jobs that can test against different versions.12:31
smcginnisGlad to see that passing though!12:31
hemnaew12:31
hemnacustom code for a specific version of ceph :(12:32
* hemna has a sad12:32
e0nerosmaita: fair enough12:32
e0nesmcginnis: we don't have ceph backups in gates12:32
smcginnisIs this only for the backup driver?12:32
rosmaitasmcginnis: we haven't been able to get it running, eharney had a patch up, but there's a problem because devstack installs logging from a package, and neutron needs a more recent version, and the stack fails12:32
eharneyi put up a test patch for octopus... it's kind of a mess so far12:32
eharneyright12:33
e0nesmcginnis: honestly, I didn't check12:33
eharneyhttps://review.opendev.org/#/c/748245/12:33
e0nesmcginnis: I reproduced this issue with backups on a devstack + ubuntu 20.04 + ceph12:33
smcginnisCool. I'm good with it as is. Just seems like we should have some sort of test coverage if there is a significant difference between versions.12:33
hemnado things fail if we always put [global] in the ini file? even with previous versions of ceph?12:35
*** pcaruana has joined #openstack-cinder12:35
hemnait would be nice if that worked, then we could avoid all this version checking and running ceph --version, etc12:35
rosmaitathe example configs in the ceph docs from mimic onward have a [global] section, couldn't find anything prior to that12:37
eharneyeven the hammer docs reference [global]12:37
eharneymight be a good follow-up effort to test that and see if we can just use the same conf everywhere, not sure if there are subtle differences12:38
e0neeharney: +112:38
rosmaitathe way the gate has been, i'd prefer to do it as a follow-up12:38
eharneyyeah12:39
e0neI can test with nautilus now12:39
rosmaitabut you are right hemna, it would be good if we could simplify this12:39
*** Lucas_Gray has joined #openstack-cinder12:39
eharneyisn't the gate testing it w/ nautilus?12:40
e0neeharney: it does, but without backups12:40
zenkurowhoami-rajat: just checked --- 0 fails, all good.12:40
e0nedo we have a list of supported ceph versions?12:40
whoami-rajatcool12:41
eharneyi think not, but the driver intends to work with quite a lot of versions12:43
eharneylooking at the ceph release schedules, i think the answer is that we support all of them that are actively maintained12:46
e0netaking into account, tomorrow is a deadline to get os-brick released for victoria, I would like to get this patch merged "as is"12:47
e0neI mean with different config formats support12:47
rosmaitae0ne: ++12:47
e0neand we can fix everything else as a follow up patches12:48
hemnahrmm12:48
hemnaas a bug fix later?12:48
e0nehemna: it looks like more like a code clean up12:48
hemnahow much time do we need to test always having [global] there?12:49
*** Reepicheep has quit IRC12:49
e0neI'm testing with nautilus version now12:49
e0neI don't have other envs by the hand12:49
hemnais that our minimum ceph version we support?12:49
smcginnisWe should be able to just add [default] and have the gate test it.12:50
rosmaitai don't think we have a published min version, we should probably also look into that12:50
eharneynautilus is not the minimum that we support12:50
hemnayah12:50
smcginnisIf that works fine, I think it would be better to just skip this patch and do a simple update that always adds the default section header.12:50
hemnasmcginnis +112:50
smcginnisWe're getting close to the deadline for os-brick, but it's not going to take that much time. And more risk in adding all this conditional logic at the last minute anyway.12:51
hemnayah I figure if we have a day, test it and then we can merge it if we have to tomorrow, but with the idea of a cleaner patch would be best.12:52
*** zhanglong has joined #openstack-cinder12:53
rosmaitai think we've got pretty good tests around the conditional logic ... my concern is that we're not going to be able to test with all relevant ceph versions12:55
rosmaitathe devstack-ceph-plugin only updated to nautilus very recently12:55
rosmaitawhich makes me think there are a lot of luminous and mimic installations out there12:56
hemnathe devstack-ceph-plugin hasn't gotten a lot of attention in ages though12:56
eharney?12:57
hemnain terms of upgrading the version of ceph being installed12:57
e0neconfirmed: [global] work for nautilus too12:58
hemnasweet12:58
rosmaitaso just h, i, j, k, l, m left to check12:59
e0ne:)13:00
e0neit will be a much easier, if devstack-ceph-plugin  can setup a needed version13:00
rosmaitaagreed13:00
hemnayes13:00
hemnaI wanted to redo the plugin and use ceph-ansible13:01
hemnain my 'spare time'13:01
hemnaheh13:01
e0nehemna: it's a good time to get a vacation for it ;)13:01
*** Reepicheep has joined #openstack-cinder13:02
rosmaitawell, my opinion is that this issue has only been reported for octopus, so i don't want to make a change that could break earlier versions without more testing13:03
rosmaitaso i think we go with this patch and follow up once we know it's safe to simplify the code13:04
hemnaI'm 50/50 on it13:04
hemnaI think as a result of this we need to specify a minimum ceph version we support and test for that in the driver at some point.13:05
rosmaitahemna: ++13:07
*** johanssone has quit IRC13:11
*** m75abrams has quit IRC13:13
*** zhanglong has quit IRC13:15
*** m75abrams has joined #openstack-cinder13:15
*** zhanglong has joined #openstack-cinder13:16
eharneyhemna: i think for anything we've added to the ceph driver, we ensure it doesn't break compat with older versions of ceph -- so like i said, i think the aim is to support any versions of ceph that ceph supports13:37
hemnaI suppose that's fine, I just don't care for the hacky nature of that patch that tests for versions and does special stuff13:38
*** zhanglong has quit IRC13:45
*** enriquetaso has joined #openstack-cinder13:47
*** kaisers has joined #openstack-cinder13:50
rosmaitaCourtesy reminder: Cinder meeting in #openstack-meeting-alt at 1400 UTC13:58
rosmaitajungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard lseki sfernand rajinir enriquetaso hemna ^^13:58
*** Liang__ has joined #openstack-cinder13:58
*** Liang__ is now known as LiangFang13:59
*** m75abrams has quit IRC14:01
*** sfernand has joined #openstack-cinder14:03
*** Lucas_Gray has quit IRC14:05
*** LiangFang has quit IRC14:12
openstackgerritMerged openstack/os-brick master: Fix for rbd connector to work with ceph octopus  https://review.opendev.org/73037614:14
*** Liang__ has joined #openstack-cinder14:14
*** zenkuro has quit IRC14:30
*** Lucas_Gray has joined #openstack-cinder14:32
*** johanssone has joined #openstack-cinder14:36
*** rcernin_ has joined #openstack-cinder14:44
*** rcernin_ has quit IRC14:49
rosmaita\o/14:51
*** udesale_ has quit IRC14:59
*** Liang__ has quit IRC15:29
*** tosky has quit IRC15:31
openstackgerritEric Harney proposed openstack/cinder master: Add mypy tox env  https://review.opendev.org/73362015:37
openstackgerritEric Harney proposed openstack/cinder master: mypy: annotate volume manager  https://review.opendev.org/73362115:37
openstackgerritEric Harney proposed openstack/cinder master: mypy: annotate api.py  https://review.opendev.org/73362215:37
*** zenkuro22 has joined #openstack-cinder15:43
zenkuro22e0ne: or you are talking that this patch will not be merged at all in the next several month?15:44
e0nezenkuro22: new drivers could be merged not sooner than Wallaby cycle begins15:45
zenkuro22but can they be merged in code tree?15:45
zenkuro22e0ne: but can they be merged in code tree?15:45
rosmaitazenkuro22: only after the stable/victoria branch is cut15:45
e0nezenkuro22: it's a week of Sep 21-Sep 2515:47
zenkuro22so the branches is not separated yet, I know it is against the rules, but since code is ok and it is just a driver with basic features, like there is nothing that affects cinder itself been introduced15:48
zenkuro22may be it is possible to squise it in?15:48
zenkuro22I promise to support it on daily bases if ANY issue happen till 25 of sept15:49
openstackgerritMerged openstack/python-cinderclient master: Add support for Cinder API mv3.61  https://review.opendev.org/74299415:50
zenkuro22+ this code was previously cheked by Lucio Seki, Eric Harney, Sean McGinnis and Ive fixed coding style issues pointed out by them.15:53
*** lpetrut has quit IRC15:53
rosmaitasmcginnis: when you have some time, can you look at the release patch for brick? not sure i have the branch point correct: https://review.opendev.org/74954415:54
smcginniszenkuro22: It should be good to go once wallaby opens up. It is too late for victoria.15:55
smcginnisrosmaita: Will do. Do you use new-release for those? There's a --stable-branch switch for that.15:55
smcginnisrosmaita: It does look correct through.15:56
smcginnis*though15:56
rosmaitasmcginnis: i will have to investigate new-release ... i just do it by hand the old fashioned way15:56
openstackgerritzhoulinhui proposed openstack/python-cinderclient master: Use importlib to take place of imp module  https://review.opendev.org/74885715:59
*** dsariel has quit IRC16:28
*** enriquetaso has quit IRC16:40
openstackgerritOpenStack Release Bot proposed openstack/os-brick stable/victoria: Update .gitreview for stable/victoria  https://review.opendev.org/74955516:43
openstackgerritOpenStack Release Bot proposed openstack/os-brick stable/victoria: Update TOX_CONSTRAINTS_FILE for stable/victoria  https://review.opendev.org/74955616:43
openstackgerritOpenStack Release Bot proposed openstack/os-brick master: Update master for stable/victoria  https://review.opendev.org/74955716:43
openstackgerritOpenStack Release Bot proposed openstack/os-brick master: Add Python3 wallaby unit tests  https://review.opendev.org/74955816:43
*** Lucas_Gray has quit IRC16:50
*** enriquetaso has joined #openstack-cinder16:56
*** rcernin_ has joined #openstack-cinder17:04
*** rcernin_ has quit IRC17:08
*** psachin has quit IRC17:09
*** manoj_kumar_kata has quit IRC17:12
*** manoj_kumar_kata has joined #openstack-cinder17:12
*** manoj_kumar_kata has quit IRC17:14
*** manoj_kumar_kata has joined #openstack-cinder17:14
*** josephillips has joined #openstack-cinder17:15
openstackgerritMerged openstack/os-brick master: Update master for stable/victoria  https://review.opendev.org/74955717:17
openstackgerritMerged openstack/os-brick stable/victoria: Update .gitreview for stable/victoria  https://review.opendev.org/74955517:17
openstackgerritMerged openstack/os-brick stable/victoria: Update TOX_CONSTRAINTS_FILE for stable/victoria  https://review.opendev.org/74955617:28
*** manoj_kumar_kata has quit IRC17:31
*** manoj_kumar_kata has joined #openstack-cinder17:32
openstackgerritVictoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: DNM Testing Manila gates  https://review.opendev.org/74715417:55
*** e0ne has quit IRC18:29
*** vishalmanchanda has quit IRC18:43
openstackgerritFernando Ferraz proposed openstack/cinder master: NetApp SolidFire: Add storage assisted migration support  https://review.opendev.org/74694118:58
openstackgerritFernando Ferraz proposed openstack/cinder master: NetApp SolidFire: Add storage assisted migration support  https://review.opendev.org/74694119:01
*** zzzeek has quit IRC19:17
*** zzzeek has joined #openstack-cinder19:18
*** manoj_kumar_kata has quit IRC19:21
*** zzzeek has quit IRC19:25
*** zzzeek has joined #openstack-cinder19:27
*** e0ne has joined #openstack-cinder19:51
*** zzzeek has quit IRC19:53
*** zzzeek has joined #openstack-cinder19:56
*** martinkennelly has quit IRC20:32
*** Lucas_Gray has joined #openstack-cinder20:42
*** e0ne has quit IRC20:44
*** zzzeek has quit IRC20:50
*** zzzeek has joined #openstack-cinder20:51
*** sapd1_x has quit IRC21:04
*** rcernin_ has joined #openstack-cinder21:05
*** rcernin_ has quit IRC21:12
*** sapd1_x has joined #openstack-cinder21:17
openstackgerritMerged openstack/os-brick stable/train: Leverage the iSCSI mpath to get the WWN  https://review.opendev.org/74933421:41
openstackgerritMerged openstack/cinder master: RBD: remove rbd_keyring_conf option  https://review.opendev.org/74749421:43
*** ajitha has quit IRC21:50
zenkuro22smcginnis: hi, is there specific format for re running ci?22:02
zenkuro22smcginnis: open-e triggers on run open-e-joviandss-ci22:03
*** rcernin_ has joined #openstack-cinder22:15
smcginniszenkuro22: That should be fixed then.22:35
zenkuro22smcginnis: many thanks for review! I will fix it.22:35
smcginniszenkuro22: https://wiki.openstack.org/wiki/Cinder/tested-3rdParty-drivers#How_do_I_trigger_my_CI_to_rerun_on_gerrit_comments.3F22:35
smcginniszenkuro22: Looks pretty good overall. So I think once these little issues are addressed, things should be ready to go for Wallaby.22:35
zenkuro22smcginnis: thanks again \>_</22:37
*** enriquetaso has quit IRC22:58
*** zenkuro22 has quit IRC23:00
*** rcernin_ has quit IRC23:04
*** rcernin has joined #openstack-cinder23:04
*** Lucas_Gray has quit IRC23:07
*** lifeless has quit IRC23:29
*** irclogbot_2 has quit IRC23:29
*** irclogbot_3 has joined #openstack-cinder23:32
*** whoami-rajat has quit IRC23:35

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