Wednesday, 2021-01-06

*** carloss_ has quit IRC00:09
*** tosky has quit IRC00:23
*** spatel has joined #openstack-cinder00:46
*** spatel has quit IRC00:52
*** lifeless has quit IRC00:53
*** lifeless has joined #openstack-cinder00:55
*** benfelin has quit IRC00:57
*** sapd1 has joined #openstack-cinder01:30
*** martinkennelly has quit IRC01:43
*** zenkuro has joined #openstack-cinder02:04
*** spatel has joined #openstack-cinder02:26
*** spatel has quit IRC02:28
caiqilongI proposed a change: "https://review.opendev.org/c/openstack/cinder/+/767290" to gerrit. But there are "Merge conflicts" appears at the top right corner of the review page. Will it affect the code merging?02:36
*** psachin has joined #openstack-cinder03:30
*** sapd1_x has joined #openstack-cinder04:39
*** caiqilong has quit IRC04:40
*** sapd1 has quit IRC04:43
*** udesale has joined #openstack-cinder05:00
*** ociuhandu has joined #openstack-cinder05:08
*** ociuhandu has quit IRC05:13
*** dave-mccowan has quit IRC05:19
*** m75abrams has joined #openstack-cinder05:27
*** sapd1_x has quit IRC05:38
*** whoami-rajat__ has joined #openstack-cinder05:41
*** zzzeek has quit IRC05:42
*** zzzeek has joined #openstack-cinder05:43
*** caiqilong has joined #openstack-cinder05:46
*** zzzeek has quit IRC05:50
*** zzzeek has joined #openstack-cinder05:52
*** sfernand has quit IRC05:53
*** sapd1_x has joined #openstack-cinder05:57
*** zzzeek has quit IRC05:59
*** zzzeek has joined #openstack-cinder06:00
*** zzzeek has quit IRC06:05
*** zzzeek has joined #openstack-cinder06:11
*** zzzeek has quit IRC06:18
*** zzzeek has joined #openstack-cinder06:19
*** zzzeek has quit IRC06:46
*** zzzeek has joined #openstack-cinder06:49
*** sapd1_x has quit IRC06:51
*** abdysn has joined #openstack-cinder07:03
*** sapd1_x has joined #openstack-cinder07:06
*** zzzeek has quit IRC07:09
*** zzzeek has joined #openstack-cinder07:12
*** zzzeek has quit IRC07:21
*** zzzeek has joined #openstack-cinder07:22
*** zzzeek has quit IRC07:50
*** zzzeek has joined #openstack-cinder07:54
*** sapd1_x has quit IRC07:59
*** rpittau|afk is now known as rpittau08:01
*** Abdallahyas has joined #openstack-cinder08:09
*** abdysn has quit IRC08:13
*** zzzeek has quit IRC08:16
*** zzzeek has joined #openstack-cinder08:19
crohmanneharney: May I kindly ask again, if you could get a chance to look into my patch at https://review.opendev.org/c/openstack/cinder/+/766856/ ? It's rather tiny and I would love to contribute more in the future, but need some guidance and practice on how things work with all the CI and review processes that are in place.08:31
*** elod_pto is now known as elod08:48
openstackgerritSuman proposed openstack/os-brick master: Failed to discover the volumes on ppc64le - Part 2  https://review.opendev.org/c/openstack/os-brick/+/76671708:52
*** CeeMac has joined #openstack-cinder09:01
*** zzzeek has quit IRC09:03
*** zzzeek has joined #openstack-cinder09:04
*** zenkuro has quit IRC09:26
*** zenkuro has joined #openstack-cinder09:27
*** wanghongtao has joined #openstack-cinder09:33
*** ociuhandu has joined #openstack-cinder09:34
wanghongtaohttps://review.opendev.org/c/openstack/cinder/+/768592  please review this path about cinder quota can not update,thank you very much.09:35
*** zzzeek has quit IRC09:42
*** savihou has joined #openstack-cinder09:43
*** zzzeek has joined #openstack-cinder09:43
*** wanghongtao2 has joined #openstack-cinder09:47
*** zzzeek has quit IRC09:51
*** martinkennelly has joined #openstack-cinder09:52
*** zzzeek has joined #openstack-cinder09:53
*** ociuhandu has quit IRC09:57
*** e0ne has joined #openstack-cinder09:58
*** zzzeek has quit IRC10:00
*** zzzeek has joined #openstack-cinder10:00
*** ociuhandu has joined #openstack-cinder10:02
*** zzzeek has quit IRC10:14
*** zzzeek has joined #openstack-cinder10:17
*** zzzeek has quit IRC10:35
*** zzzeek has joined #openstack-cinder10:38
*** wanghongtao_ has joined #openstack-cinder10:53
*** udesale_ has joined #openstack-cinder10:54
*** wanghongtao has quit IRC10:56
*** wanghongtao2 has quit IRC10:57
*** udesale has quit IRC10:57
*** Abdallahyas has quit IRC10:58
*** Abdallahyas has joined #openstack-cinder10:58
*** Abdallahyas has quit IRC11:11
*** e0ne has quit IRC11:16
*** abdysn has joined #openstack-cinder11:18
*** masterpe has quit IRC11:20
*** masterpe has joined #openstack-cinder11:28
*** wanghongtao__ has joined #openstack-cinder11:39
*** wanghongtao_ has quit IRC11:41
*** ociuhandu has quit IRC11:59
*** ociuhandu has joined #openstack-cinder12:00
*** ociuhandu has quit IRC12:05
*** wanghongtao_ has joined #openstack-cinder12:14
*** tosky has joined #openstack-cinder12:15
*** wanghongtao__ has quit IRC12:16
*** lpetrut has joined #openstack-cinder12:17
*** sapd1 has joined #openstack-cinder12:24
*** ociuhandu has joined #openstack-cinder12:26
*** ociuhandu has quit IRC12:26
*** ociuhandu has joined #openstack-cinder12:27
*** ociuhandu has quit IRC12:28
openstackgerritReason li proposed openstack/cinderlib master: remove unicode from code  https://review.opendev.org/c/openstack/cinderlib/+/76953712:28
*** ociuhandu has joined #openstack-cinder12:29
openstackgerritReason li proposed openstack/cinderlib master: remove unicode from code  https://review.opendev.org/c/openstack/cinderlib/+/76953812:29
*** ociuhandu has quit IRC12:35
*** e0ne has joined #openstack-cinder12:38
*** zzzeek has quit IRC12:46
*** enriquetaso has joined #openstack-cinder12:48
*** m75abrams has quit IRC12:48
*** zzzeek has joined #openstack-cinder12:49
*** ociuhandu has joined #openstack-cinder12:51
*** martinkennelly has quit IRC12:54
*** ociuhandu has quit IRC13:02
*** sfernand has joined #openstack-cinder13:05
*** ociuhandu has joined #openstack-cinder13:07
*** brinzhang0 has quit IRC13:12
*** ociuhandu has quit IRC13:12
*** rratnaka has joined #openstack-cinder13:12
*** sapd1 has quit IRC13:19
*** benfelin has joined #openstack-cinder13:25
*** enriquetaso has quit IRC13:26
*** enriquetaso has joined #openstack-cinder13:28
*** michael-mcaleer has joined #openstack-cinder13:30
*** senrique_ has joined #openstack-cinder13:31
*** ociuhandu has joined #openstack-cinder13:33
*** enriquetaso has quit IRC13:34
*** ociuhandu has quit IRC13:37
*** senrique__ has joined #openstack-cinder13:43
*** senrique_ has quit IRC13:46
*** enriquetaso has joined #openstack-cinder13:47
*** senrique__ has quit IRC13:48
*** enriquetaso has quit IRC13:53
*** m75abrams has joined #openstack-cinder13:54
*** ociuhandu has joined #openstack-cinder13:55
*** e0ne has quit IRC13:56
*** enriquetaso has joined #openstack-cinder13:59
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 michael-mcaleer ^^13:59
enriquetasothanks14:00
michael-mcaleerty14:00
*** felipe_rodrigues has joined #openstack-cinder14:09
*** lseki has joined #openstack-cinder14:09
*** dave-mccowan has joined #openstack-cinder14:10
openstackgerritHelen Walsh proposed openstack/cinder master: mypy - cinder driver.py  https://review.opendev.org/c/openstack/cinder/+/76581014:16
openstackgerritEric Harney proposed openstack/cinder master: Remove __unicode__() from CinderException  https://review.opendev.org/c/openstack/cinder/+/76954614:38
openstackgerritEric Harney proposed openstack/cinder master: Tests: Fix rbd unit test failure due to ceph keyring file  https://review.opendev.org/c/openstack/cinder/+/76954914:50
*** wanghongtao__ has joined #openstack-cinder14:52
*** wanghongtao_ has quit IRC14:54
*** m75abrams has quit IRC15:02
*** senrique_ has joined #openstack-cinder15:03
*** m75abrams has joined #openstack-cinder15:05
*** enriquetaso has quit IRC15:06
*** senrique_ has quit IRC15:08
*** enriquetaso has joined #openstack-cinder15:08
openstackgerritEric Harney proposed openstack/cinder master: WIP: LVM: Use --readonly for lvs  https://review.opendev.org/c/openstack/cinder/+/76955215:18
*** sapd1 has joined #openstack-cinder15:20
*** lpetrut has quit IRC15:27
*** enriquetaso has quit IRC15:28
*** enriquetaso has joined #openstack-cinder15:28
hemnamorning15:30
hemnais there a way to limit the # of volumes returned by the openstacksdk volumes() call?15:30
hemnaI think the cinderclient call allows limiting no?15:30
caiqilongI proposed a change: "https://review.opendev.org/c/openstack/cinder/+/767290" to gerrit. But there are "Merge conflicts" appears at the top right corner of the review page. Will it affect the code merging?15:31
*** abdysn has quit IRC15:33
*** ociuhandu has quit IRC15:34
eharneycaiqilong: that only shows that it could conflict with those changes if they merge first.  If your patch actually needs updating it will indicate "Merge Conflict" where it currently says "Active" in the upper left blue box15:35
eharneysee https://review.opendev.org/c/openstack/python-cinderclient/+/766769 for an example15:35
*** spotz has quit IRC15:44
hemnaok looks like openstacksdk doesn't use the cinderclient underneath15:44
hemnaso there is no limiting support15:44
crohmanneharney: did you see my question earlier? Is there any other "proper" way to find reviewers than i.e. nag you about it?15:44
caiqilongeharney: My change is a few versions behind the current master branch. Should I merge the changes of the master branch before submitting a new patch?15:46
*** ociuhandu has joined #openstack-cinder15:46
openstackgerritHelen Walsh proposed openstack/cinder master: mypy - cinder driver.py  https://review.opendev.org/c/openstack/cinder/+/76581015:49
*** ociuhandu has quit IRC15:56
openstackgerritHelen Walsh proposed openstack/cinder master: mypy - cinder driver.py  https://review.opendev.org/c/openstack/cinder/+/76581016:17
*** enriquetaso has quit IRC16:29
*** enriquetaso has joined #openstack-cinder16:30
openstackgerritRatnakaram Rajesh proposed openstack/cinder master: Support for multiattach and ipv6 for Zadara cinder driver  https://review.opendev.org/c/openstack/cinder/+/76956616:36
*** enriquetaso has quit IRC16:38
*** m75abrams has quit IRC16:40
openstackgerritMerged openstack/cinder master: Correct group:reset_group_snapshot_status policy  https://review.opendev.org/c/openstack/cinder/+/76722616:44
*** enriquetaso has joined #openstack-cinder16:47
openstackgerritMerged openstack/os-brick master: opencas: Use BrickException instead of Exception  https://review.opendev.org/c/openstack/os-brick/+/76378416:48
*** ociuhandu has joined #openstack-cinder16:51
*** ociuhandu has quit IRC16:52
*** ociuhandu has joined #openstack-cinder16:52
*** ociuhandu has quit IRC16:52
openstackgerritEric Harney proposed openstack/os-brick stable/victoria: opencas: Use BrickException instead of Exception  https://review.opendev.org/c/openstack/os-brick/+/76951516:53
openstackgerritCorey Bryant proposed openstack/cinder stable/victoria: RBD: Retry delete if VolumeIsBusy in _copy_image_to_volume  https://review.opendev.org/c/openstack/cinder/+/76957616:57
openstackgerritCorey Bryant proposed openstack/cinder stable/ussuri: RBD: Retry delete if VolumeIsBusy in _copy_image_to_volume  https://review.opendev.org/c/openstack/cinder/+/76957716:58
openstackgerritCorey Bryant proposed openstack/cinder stable/train: RBD: Retry delete if VolumeIsBusy in _copy_image_to_volume  https://review.opendev.org/c/openstack/cinder/+/76958017:00
hemnaso, it looks like the cinder api doesn't allow limit as an allowable filter while fetching the list of volumes17:01
hemnahttps://github.com/openstack/cinder/blob/master/etc/cinder/resource_filters.json#L2-L417:01
*** udesale_ has quit IRC17:04
*** wanghongtao_ has joined #openstack-cinder17:05
*** sfernand has quit IRC17:05
rosmaitai think it does -- pagination isn't considered a filter17:05
openstackgerritCorey Bryant proposed openstack/cinder stable/stein: RBD: Retry delete if VolumeIsBusy in _copy_image_to_volume  https://review.opendev.org/c/openstack/cinder/+/76958317:06
*** sapd1 has quit IRC17:07
*** wanghongtao__ has quit IRC17:08
*** michael-mcaleer has quit IRC17:10
*** enriquetaso has quit IRC17:15
*** ircuser-1 has joined #openstack-cinder17:20
*** sapd1 has joined #openstack-cinder17:21
*** psachin has quit IRC17:23
*** ociuhandu has joined #openstack-cinder17:27
*** rratnaka has quit IRC17:30
hemnalimit has to be in that list or it's ignored17:31
*** ociuhandu has quit IRC17:33
*** lseki has quit IRC17:34
*** rpittau is now known as rpittau|afk17:43
*** sapd1 has quit IRC17:47
hemnanon-admins should be able to limit the # of volumes coming back from list17:50
*** sfernand has joined #openstack-cinder17:53
*** wanghongtao__ has joined #openstack-cinder18:00
rosmaitathat sounds like a definite bug18:01
*** wanghongtao_ has quit IRC18:02
openstackgerritBrian Rosmaita proposed openstack/cinder stable/victoria: Correct group:reset_group_snapshot_status policy  https://review.opendev.org/c/openstack/cinder/+/76958718:17
*** zenkuro has quit IRC18:37
openstackgerritEric Harney proposed openstack/cinder master: WIP: LVM: Use --readonly for lvs  https://review.opendev.org/c/openstack/cinder/+/76955218:43
openstackgerritEric Harney proposed openstack/cinder master: Tests: Simplify LVM fake_execute  https://review.opendev.org/c/openstack/cinder/+/76960219:01
*** ociuhandu has joined #openstack-cinder19:13
*** ociuhandu has quit IRC19:26
openstackgerritMerged openstack/cinder master: Tests: Fix rbd unit test failure due to ceph keyring file  https://review.opendev.org/c/openstack/cinder/+/76954919:33
*** ociuhandu has joined #openstack-cinder20:17
*** ociuhandu has quit IRC20:18
*** ociuhandu has joined #openstack-cinder20:19
*** ociuhandu has quit IRC20:19
*** ociuhandu has joined #openstack-cinder20:20
*** whoami-rajat__ has quit IRC20:21
*** ociuhandu has quit IRC20:29
*** ociuhandu has joined #openstack-cinder20:34
*** ociuhandu has quit IRC20:43
*** ociuhandu has joined #openstack-cinder20:56
*** ociuhandu has quit IRC21:03
*** enriquetaso has joined #openstack-cinder21:07
*** wanghongtao_ has joined #openstack-cinder21:12
*** wanghongtao__ has quit IRC21:14
*** jmlowe has quit IRC21:33
*** ociuhandu has joined #openstack-cinder21:36
*** ociuhandu has quit IRC21:40
*** ociuhandu has joined #openstack-cinder21:44
*** ociuhandu has quit IRC21:51
*** ociuhandu has joined #openstack-cinder21:54
*** ociuhandu has quit IRC21:58
*** ociuhandu has joined #openstack-cinder22:10
*** wanghongtao__ has joined #openstack-cinder22:12
*** wanghongtao_ has quit IRC22:14
*** ociuhandu has quit IRC22:19
*** rcernin has joined #openstack-cinder22:27
*** enriquetaso has quit IRC22:51
*** zzzeek has quit IRC22:53
*** zzzeek has joined #openstack-cinder22:55
*** zzzeek has quit IRC23:08
*** eharney has quit IRC23:09
*** zzzeek has joined #openstack-cinder23:09
*** ociuhandu has joined #openstack-cinder23:25
*** ociuhandu has quit IRC23:30
*** tosky has quit IRC23:51

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