*** carloss_ has quit IRC | 00:09 | |
*** tosky has quit IRC | 00:23 | |
*** spatel has joined #openstack-cinder | 00:46 | |
*** spatel has quit IRC | 00:52 | |
*** lifeless has quit IRC | 00:53 | |
*** lifeless has joined #openstack-cinder | 00:55 | |
*** benfelin has quit IRC | 00:57 | |
*** sapd1 has joined #openstack-cinder | 01:30 | |
*** martinkennelly has quit IRC | 01:43 | |
*** zenkuro has joined #openstack-cinder | 02:04 | |
*** spatel has joined #openstack-cinder | 02:26 | |
*** spatel has quit IRC | 02:28 | |
caiqilong | I 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-cinder | 03:30 | |
*** sapd1_x has joined #openstack-cinder | 04:39 | |
*** caiqilong has quit IRC | 04:40 | |
*** sapd1 has quit IRC | 04:43 | |
*** udesale has joined #openstack-cinder | 05:00 | |
*** ociuhandu has joined #openstack-cinder | 05:08 | |
*** ociuhandu has quit IRC | 05:13 | |
*** dave-mccowan has quit IRC | 05:19 | |
*** m75abrams has joined #openstack-cinder | 05:27 | |
*** sapd1_x has quit IRC | 05:38 | |
*** whoami-rajat__ has joined #openstack-cinder | 05:41 | |
*** zzzeek has quit IRC | 05:42 | |
*** zzzeek has joined #openstack-cinder | 05:43 | |
*** caiqilong has joined #openstack-cinder | 05:46 | |
*** zzzeek has quit IRC | 05:50 | |
*** zzzeek has joined #openstack-cinder | 05:52 | |
*** sfernand has quit IRC | 05:53 | |
*** sapd1_x has joined #openstack-cinder | 05:57 | |
*** zzzeek has quit IRC | 05:59 | |
*** zzzeek has joined #openstack-cinder | 06:00 | |
*** zzzeek has quit IRC | 06:05 | |
*** zzzeek has joined #openstack-cinder | 06:11 | |
*** zzzeek has quit IRC | 06:18 | |
*** zzzeek has joined #openstack-cinder | 06:19 | |
*** zzzeek has quit IRC | 06:46 | |
*** zzzeek has joined #openstack-cinder | 06:49 | |
*** sapd1_x has quit IRC | 06:51 | |
*** abdysn has joined #openstack-cinder | 07:03 | |
*** sapd1_x has joined #openstack-cinder | 07:06 | |
*** zzzeek has quit IRC | 07:09 | |
*** zzzeek has joined #openstack-cinder | 07:12 | |
*** zzzeek has quit IRC | 07:21 | |
*** zzzeek has joined #openstack-cinder | 07:22 | |
*** zzzeek has quit IRC | 07:50 | |
*** zzzeek has joined #openstack-cinder | 07:54 | |
*** sapd1_x has quit IRC | 07:59 | |
*** rpittau|afk is now known as rpittau | 08:01 | |
*** Abdallahyas has joined #openstack-cinder | 08:09 | |
*** abdysn has quit IRC | 08:13 | |
*** zzzeek has quit IRC | 08:16 | |
*** zzzeek has joined #openstack-cinder | 08:19 | |
crohmann | eharney: 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 elod | 08:48 | |
openstackgerrit | Suman proposed openstack/os-brick master: Failed to discover the volumes on ppc64le - Part 2 https://review.opendev.org/c/openstack/os-brick/+/766717 | 08:52 |
*** CeeMac has joined #openstack-cinder | 09:01 | |
*** zzzeek has quit IRC | 09:03 | |
*** zzzeek has joined #openstack-cinder | 09:04 | |
*** zenkuro has quit IRC | 09:26 | |
*** zenkuro has joined #openstack-cinder | 09:27 | |
*** wanghongtao has joined #openstack-cinder | 09:33 | |
*** ociuhandu has joined #openstack-cinder | 09:34 | |
wanghongtao | https://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 IRC | 09:42 | |
*** savihou has joined #openstack-cinder | 09:43 | |
*** zzzeek has joined #openstack-cinder | 09:43 | |
*** wanghongtao2 has joined #openstack-cinder | 09:47 | |
*** zzzeek has quit IRC | 09:51 | |
*** martinkennelly has joined #openstack-cinder | 09:52 | |
*** zzzeek has joined #openstack-cinder | 09:53 | |
*** ociuhandu has quit IRC | 09:57 | |
*** e0ne has joined #openstack-cinder | 09:58 | |
*** zzzeek has quit IRC | 10:00 | |
*** zzzeek has joined #openstack-cinder | 10:00 | |
*** ociuhandu has joined #openstack-cinder | 10:02 | |
*** zzzeek has quit IRC | 10:14 | |
*** zzzeek has joined #openstack-cinder | 10:17 | |
*** zzzeek has quit IRC | 10:35 | |
*** zzzeek has joined #openstack-cinder | 10:38 | |
*** wanghongtao_ has joined #openstack-cinder | 10:53 | |
*** udesale_ has joined #openstack-cinder | 10:54 | |
*** wanghongtao has quit IRC | 10:56 | |
*** wanghongtao2 has quit IRC | 10:57 | |
*** udesale has quit IRC | 10:57 | |
*** Abdallahyas has quit IRC | 10:58 | |
*** Abdallahyas has joined #openstack-cinder | 10:58 | |
*** Abdallahyas has quit IRC | 11:11 | |
*** e0ne has quit IRC | 11:16 | |
*** abdysn has joined #openstack-cinder | 11:18 | |
*** masterpe has quit IRC | 11:20 | |
*** masterpe has joined #openstack-cinder | 11:28 | |
*** wanghongtao__ has joined #openstack-cinder | 11:39 | |
*** wanghongtao_ has quit IRC | 11:41 | |
*** ociuhandu has quit IRC | 11:59 | |
*** ociuhandu has joined #openstack-cinder | 12:00 | |
*** ociuhandu has quit IRC | 12:05 | |
*** wanghongtao_ has joined #openstack-cinder | 12:14 | |
*** tosky has joined #openstack-cinder | 12:15 | |
*** wanghongtao__ has quit IRC | 12:16 | |
*** lpetrut has joined #openstack-cinder | 12:17 | |
*** sapd1 has joined #openstack-cinder | 12:24 | |
*** ociuhandu has joined #openstack-cinder | 12:26 | |
*** ociuhandu has quit IRC | 12:26 | |
*** ociuhandu has joined #openstack-cinder | 12:27 | |
*** ociuhandu has quit IRC | 12:28 | |
openstackgerrit | Reason li proposed openstack/cinderlib master: remove unicode from code https://review.opendev.org/c/openstack/cinderlib/+/769537 | 12:28 |
*** ociuhandu has joined #openstack-cinder | 12:29 | |
openstackgerrit | Reason li proposed openstack/cinderlib master: remove unicode from code https://review.opendev.org/c/openstack/cinderlib/+/769538 | 12:29 |
*** ociuhandu has quit IRC | 12:35 | |
*** e0ne has joined #openstack-cinder | 12:38 | |
*** zzzeek has quit IRC | 12:46 | |
*** enriquetaso has joined #openstack-cinder | 12:48 | |
*** m75abrams has quit IRC | 12:48 | |
*** zzzeek has joined #openstack-cinder | 12:49 | |
*** ociuhandu has joined #openstack-cinder | 12:51 | |
*** martinkennelly has quit IRC | 12:54 | |
*** ociuhandu has quit IRC | 13:02 | |
*** sfernand has joined #openstack-cinder | 13:05 | |
*** ociuhandu has joined #openstack-cinder | 13:07 | |
*** brinzhang0 has quit IRC | 13:12 | |
*** ociuhandu has quit IRC | 13:12 | |
*** rratnaka has joined #openstack-cinder | 13:12 | |
*** sapd1 has quit IRC | 13:19 | |
*** benfelin has joined #openstack-cinder | 13:25 | |
*** enriquetaso has quit IRC | 13:26 | |
*** enriquetaso has joined #openstack-cinder | 13:28 | |
*** michael-mcaleer has joined #openstack-cinder | 13:30 | |
*** senrique_ has joined #openstack-cinder | 13:31 | |
*** ociuhandu has joined #openstack-cinder | 13:33 | |
*** enriquetaso has quit IRC | 13:34 | |
*** ociuhandu has quit IRC | 13:37 | |
*** senrique__ has joined #openstack-cinder | 13:43 | |
*** senrique_ has quit IRC | 13:46 | |
*** enriquetaso has joined #openstack-cinder | 13:47 | |
*** senrique__ has quit IRC | 13:48 | |
*** enriquetaso has quit IRC | 13:53 | |
*** m75abrams has joined #openstack-cinder | 13:54 | |
*** ociuhandu has joined #openstack-cinder | 13:55 | |
*** e0ne has quit IRC | 13:56 | |
*** enriquetaso has joined #openstack-cinder | 13:59 | |
rosmaita | Courtesy reminder: Cinder meeting in #openstack-meeting-alt at 1400 UTC | 13:59 |
rosmaita | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard lseki sfernand rajinir enriquetaso hemna michael-mcaleer ^^ | 13:59 |
enriquetaso | thanks | 14:00 |
michael-mcaleer | ty | 14:00 |
*** felipe_rodrigues has joined #openstack-cinder | 14:09 | |
*** lseki has joined #openstack-cinder | 14:09 | |
*** dave-mccowan has joined #openstack-cinder | 14:10 | |
openstackgerrit | Helen Walsh proposed openstack/cinder master: mypy - cinder driver.py https://review.opendev.org/c/openstack/cinder/+/765810 | 14:16 |
openstackgerrit | Eric Harney proposed openstack/cinder master: Remove __unicode__() from CinderException https://review.opendev.org/c/openstack/cinder/+/769546 | 14:38 |
openstackgerrit | Eric Harney proposed openstack/cinder master: Tests: Fix rbd unit test failure due to ceph keyring file https://review.opendev.org/c/openstack/cinder/+/769549 | 14:50 |
*** wanghongtao__ has joined #openstack-cinder | 14:52 | |
*** wanghongtao_ has quit IRC | 14:54 | |
*** m75abrams has quit IRC | 15:02 | |
*** senrique_ has joined #openstack-cinder | 15:03 | |
*** m75abrams has joined #openstack-cinder | 15:05 | |
*** enriquetaso has quit IRC | 15:06 | |
*** senrique_ has quit IRC | 15:08 | |
*** enriquetaso has joined #openstack-cinder | 15:08 | |
openstackgerrit | Eric Harney proposed openstack/cinder master: WIP: LVM: Use --readonly for lvs https://review.opendev.org/c/openstack/cinder/+/769552 | 15:18 |
*** sapd1 has joined #openstack-cinder | 15:20 | |
*** lpetrut has quit IRC | 15:27 | |
*** enriquetaso has quit IRC | 15:28 | |
*** enriquetaso has joined #openstack-cinder | 15:28 | |
hemna | morning | 15:30 |
hemna | is there a way to limit the # of volumes returned by the openstacksdk volumes() call? | 15:30 |
hemna | I think the cinderclient call allows limiting no? | 15:30 |
caiqilong | I 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 IRC | 15:33 | |
*** ociuhandu has quit IRC | 15:34 | |
eharney | caiqilong: 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 box | 15:35 |
eharney | see https://review.opendev.org/c/openstack/python-cinderclient/+/766769 for an example | 15:35 |
*** spotz has quit IRC | 15:44 | |
hemna | ok looks like openstacksdk doesn't use the cinderclient underneath | 15:44 |
hemna | so there is no limiting support | 15:44 |
crohmann | eharney: did you see my question earlier? Is there any other "proper" way to find reviewers than i.e. nag you about it? | 15:44 |
caiqilong | eharney: 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-cinder | 15:46 | |
openstackgerrit | Helen Walsh proposed openstack/cinder master: mypy - cinder driver.py https://review.opendev.org/c/openstack/cinder/+/765810 | 15:49 |
*** ociuhandu has quit IRC | 15:56 | |
openstackgerrit | Helen Walsh proposed openstack/cinder master: mypy - cinder driver.py https://review.opendev.org/c/openstack/cinder/+/765810 | 16:17 |
*** enriquetaso has quit IRC | 16:29 | |
*** enriquetaso has joined #openstack-cinder | 16:30 | |
openstackgerrit | Ratnakaram Rajesh proposed openstack/cinder master: Support for multiattach and ipv6 for Zadara cinder driver https://review.opendev.org/c/openstack/cinder/+/769566 | 16:36 |
*** enriquetaso has quit IRC | 16:38 | |
*** m75abrams has quit IRC | 16:40 | |
openstackgerrit | Merged openstack/cinder master: Correct group:reset_group_snapshot_status policy https://review.opendev.org/c/openstack/cinder/+/767226 | 16:44 |
*** enriquetaso has joined #openstack-cinder | 16:47 | |
openstackgerrit | Merged openstack/os-brick master: opencas: Use BrickException instead of Exception https://review.opendev.org/c/openstack/os-brick/+/763784 | 16:48 |
*** ociuhandu has joined #openstack-cinder | 16:51 | |
*** ociuhandu has quit IRC | 16:52 | |
*** ociuhandu has joined #openstack-cinder | 16:52 | |
*** ociuhandu has quit IRC | 16:52 | |
openstackgerrit | Eric Harney proposed openstack/os-brick stable/victoria: opencas: Use BrickException instead of Exception https://review.opendev.org/c/openstack/os-brick/+/769515 | 16:53 |
openstackgerrit | Corey Bryant proposed openstack/cinder stable/victoria: RBD: Retry delete if VolumeIsBusy in _copy_image_to_volume https://review.opendev.org/c/openstack/cinder/+/769576 | 16:57 |
openstackgerrit | Corey Bryant proposed openstack/cinder stable/ussuri: RBD: Retry delete if VolumeIsBusy in _copy_image_to_volume https://review.opendev.org/c/openstack/cinder/+/769577 | 16:58 |
openstackgerrit | Corey Bryant proposed openstack/cinder stable/train: RBD: Retry delete if VolumeIsBusy in _copy_image_to_volume https://review.opendev.org/c/openstack/cinder/+/769580 | 17:00 |
hemna | so, it looks like the cinder api doesn't allow limit as an allowable filter while fetching the list of volumes | 17:01 |
hemna | https://github.com/openstack/cinder/blob/master/etc/cinder/resource_filters.json#L2-L4 | 17:01 |
*** udesale_ has quit IRC | 17:04 | |
*** wanghongtao_ has joined #openstack-cinder | 17:05 | |
*** sfernand has quit IRC | 17:05 | |
rosmaita | i think it does -- pagination isn't considered a filter | 17:05 |
openstackgerrit | Corey Bryant proposed openstack/cinder stable/stein: RBD: Retry delete if VolumeIsBusy in _copy_image_to_volume https://review.opendev.org/c/openstack/cinder/+/769583 | 17:06 |
*** sapd1 has quit IRC | 17:07 | |
*** wanghongtao__ has quit IRC | 17:08 | |
*** michael-mcaleer has quit IRC | 17:10 | |
*** enriquetaso has quit IRC | 17:15 | |
*** ircuser-1 has joined #openstack-cinder | 17:20 | |
*** sapd1 has joined #openstack-cinder | 17:21 | |
*** psachin has quit IRC | 17:23 | |
*** ociuhandu has joined #openstack-cinder | 17:27 | |
*** rratnaka has quit IRC | 17:30 | |
hemna | limit has to be in that list or it's ignored | 17:31 |
*** ociuhandu has quit IRC | 17:33 | |
*** lseki has quit IRC | 17:34 | |
*** rpittau is now known as rpittau|afk | 17:43 | |
*** sapd1 has quit IRC | 17:47 | |
hemna | non-admins should be able to limit the # of volumes coming back from list | 17:50 |
*** sfernand has joined #openstack-cinder | 17:53 | |
*** wanghongtao__ has joined #openstack-cinder | 18:00 | |
rosmaita | that sounds like a definite bug | 18:01 |
*** wanghongtao_ has quit IRC | 18:02 | |
openstackgerrit | Brian Rosmaita proposed openstack/cinder stable/victoria: Correct group:reset_group_snapshot_status policy https://review.opendev.org/c/openstack/cinder/+/769587 | 18:17 |
*** zenkuro has quit IRC | 18:37 | |
openstackgerrit | Eric Harney proposed openstack/cinder master: WIP: LVM: Use --readonly for lvs https://review.opendev.org/c/openstack/cinder/+/769552 | 18:43 |
openstackgerrit | Eric Harney proposed openstack/cinder master: Tests: Simplify LVM fake_execute https://review.opendev.org/c/openstack/cinder/+/769602 | 19:01 |
*** ociuhandu has joined #openstack-cinder | 19:13 | |
*** ociuhandu has quit IRC | 19:26 | |
openstackgerrit | Merged openstack/cinder master: Tests: Fix rbd unit test failure due to ceph keyring file https://review.opendev.org/c/openstack/cinder/+/769549 | 19:33 |
*** ociuhandu has joined #openstack-cinder | 20:17 | |
*** ociuhandu has quit IRC | 20:18 | |
*** ociuhandu has joined #openstack-cinder | 20:19 | |
*** ociuhandu has quit IRC | 20:19 | |
*** ociuhandu has joined #openstack-cinder | 20:20 | |
*** whoami-rajat__ has quit IRC | 20:21 | |
*** ociuhandu has quit IRC | 20:29 | |
*** ociuhandu has joined #openstack-cinder | 20:34 | |
*** ociuhandu has quit IRC | 20:43 | |
*** ociuhandu has joined #openstack-cinder | 20:56 | |
*** ociuhandu has quit IRC | 21:03 | |
*** enriquetaso has joined #openstack-cinder | 21:07 | |
*** wanghongtao_ has joined #openstack-cinder | 21:12 | |
*** wanghongtao__ has quit IRC | 21:14 | |
*** jmlowe has quit IRC | 21:33 | |
*** ociuhandu has joined #openstack-cinder | 21:36 | |
*** ociuhandu has quit IRC | 21:40 | |
*** ociuhandu has joined #openstack-cinder | 21:44 | |
*** ociuhandu has quit IRC | 21:51 | |
*** ociuhandu has joined #openstack-cinder | 21:54 | |
*** ociuhandu has quit IRC | 21:58 | |
*** ociuhandu has joined #openstack-cinder | 22:10 | |
*** wanghongtao__ has joined #openstack-cinder | 22:12 | |
*** wanghongtao_ has quit IRC | 22:14 | |
*** ociuhandu has quit IRC | 22:19 | |
*** rcernin has joined #openstack-cinder | 22:27 | |
*** enriquetaso has quit IRC | 22:51 | |
*** zzzeek has quit IRC | 22:53 | |
*** zzzeek has joined #openstack-cinder | 22:55 | |
*** zzzeek has quit IRC | 23:08 | |
*** eharney has quit IRC | 23:09 | |
*** zzzeek has joined #openstack-cinder | 23:09 | |
*** ociuhandu has joined #openstack-cinder | 23:25 | |
*** ociuhandu has quit IRC | 23:30 | |
*** tosky has quit IRC | 23:51 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!