| ricolin | whoami-rajat__: do I need to suggest meeting topic for https://review.opendev.org/c/openstack/cinder/+/839793 ? | 06:22 |
|---|---|---|
| *** abhishekk is now known as akekane|home | 08:31 | |
| *** akekane|home is now known as abhishekk | 08:31 | |
| raghavendrat | hi whoami-rajat__: are you around ? | 10:35 |
| whoami-rajat__ | ricolin, hey, yes, i think it needs more discussion | 10:36 |
| whoami-rajat__ | raghavendrat, hey | 10:37 |
| raghavendrat | in case you get time it would be great if you can look at https://review.opendev.org/c/openstack/cinder/+/824911 | 10:37 |
| raghavendrat | It has one +2. Thanks. | 10:37 |
| whoami-rajat__ | ack | 10:39 |
| *** whoami-rajat__ is now known as whoami-rajat | 10:40 | |
| stephenfin | whoami-rajat: I replied to you on the cinderclient shell deprecation change. Happy to attend the midcycle for a bit, though it'll have to be the June-1 since I'm travelling to Berlin the next day | 11:14 |
| stephenfin | Also, to stress this, I would be *really* happy to have a cinder core in OSC core. I've been doing most of the heavy lifting on closing cinder gaps (with some student/intern help) so having someone to share the burden would be a very good thing (TM) ;-) | 11:15 |
| whoami-rajat | stephenfin, ack thanks, i discussed this with other cinder cores as well and the general consensus was we are not ready for this. if you want we can also discuss it on the last cinder meeting of the month i.e. 25th May which will be video+IRC | 11:17 |
| whoami-rajat | otherwise we've the midcycle on June-1 1400-1600 UTC | 11:17 |
| whoami-rajat | stephenfin, ack, that would be good, the OSC commands seems to be lacking Cinder's perspective and a core there would fasten our reviews for the patches cinder team proposes to OSC | 11:18 |
| *** dviroel|out is now known as dviroel | 11:21 | |
| *** spotz_ is now known as spotz | 11:55 | |
| opendevreview | Harsh Ailani proposed openstack/cinder master: [IBM DS8000] Fixed Detach for multi-attach volumes https://review.opendev.org/c/openstack/cinder/+/837699 | 12:18 |
| rosmaita | need reviews for zuul job patches: | 12:40 |
| rosmaita | https://review.opendev.org/c/openstack/cinder/+/841782 | 12:40 |
| rosmaita | https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/841806 | 12:40 |
| rosmaita | https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/841815 | 12:40 |
| rosmaita | e0ne eharney geguileo hemna jungleboyj smcginnis whoami-rajat enriquetaso ^^ | 12:40 |
| opendevreview | Lucas Morais de oliveira proposed openstack/cinder master: api-ref: Add docs for clusters https://review.opendev.org/c/openstack/cinder/+/795785 | 13:24 |
| opendevreview | Merged openstack/cinder master: Increase swap size to 4GB https://review.opendev.org/c/openstack/cinder/+/841782 | 13:54 |
| rosmaita | \o/ | 13:54 |
| stephenfin | lucasmoliveira059: Just FYI, you seem to have undid renaming on https://review.opendev.org/c/openstack/cinder/+/795785/8..9 | 14:05 |
| stephenfin | lucasmoliveira059: Is it okay if I fix it again? | 14:05 |
| stephenfin | lucasmoliveira059: I can't speak for anyone else, but I usually try not to keep any changes locally. When you're submitting patches, I use 'git review -f' (i.e. delete the local branch once it's submitted). If I want to respin, I download it again with 'git review -d $CHANGE_ID' | 14:06 |
| stephenfin | That way I don't have to worry about stuff getting out of sync between my local branch and Gerrit, especially when working with someone else | 14:07 |
| opendevreview | Merged openstack/cinder-tempest-plugin master: Increase swap size to 4GB https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/841815 | 14:07 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: api-ref: Add docs for clusters https://review.opendev.org/c/openstack/cinder/+/795785 | 14:09 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: Add Python 3.10 functional jobs https://review.opendev.org/c/openstack/cinder/+/841753 | 14:11 |
| opendevreview | Andre Aranha proposed openstack/cinder master: Add fips check jobs https://review.opendev.org/c/openstack/cinder/+/790535 | 14:12 |
| lucasmoliveira059 | No problem Stephen, that's ok. Thank you for the tips =) . | 14:20 |
| opendevreview | Eric Harney proposed openstack/cinder stable/xena: NetApp ONTAP: Fix check QoS min support for SVM account https://review.opendev.org/c/openstack/cinder/+/841889 | 14:48 |
| *** dviroel is now known as dviroel|lunch | 15:25 | |
| enriquetaso | stephenfin++ | 15:33 |
| enriquetaso | didn't know about 'git review -f', cool | 15:34 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: Add CI Job For lvm+ceph With c-bak Enabled https://review.opendev.org/c/openstack/cinder/+/784815 | 15:35 |
| opendevreview | Alan Bishop proposed openstack/cinder master: Reject force deletion of volumes used by Cinder services https://review.opendev.org/c/openstack/cinder/+/830901 | 15:37 |
| opendevreview | Merged openstack/cinder-tempest-plugin master: Set volume/build_timeout for zuul jobs https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/841806 | 15:46 |
| opendevreview | Eric Harney proposed openstack/cinder master: Use modern type annotation format for collections https://review.opendev.org/c/openstack/cinder/+/839987 | 16:10 |
| stephenfin | enriquetaso: Glad I could help :) | 16:12 |
| *** dviroel|lunch is now known as dviroel | 16:24 | |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: cinder/volume/volume_types.py https://review.opendev.org/c/openstack/cinder/+/835881 | 16:57 |
| opendevreview | Sofia Enriquez proposed openstack/cinder-tempest-plugin master: Add scenario/test_volume_backup_restore.py https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/835512 | 16:57 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: api_utils https://review.opendev.org/c/openstack/cinder/+/840678 | 16:58 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: [RBD] Fix Restore Full Backups https://review.opendev.org/c/openstack/cinder/+/750782 | 16:58 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: cinder/api/common.py https://review.opendev.org/c/openstack/cinder/+/841125 | 17:00 |
| ricolin | Hey rosmaita whoami-rajat , thanks for the explain on how Cinder works with review score. If you can help pushing the discussion forward on feasible solution so we might get better contant to discuss on meeting this week. I think mnaser's reply for glance option is worth some feedback/reply on. https://review.opendev.org/c/openstack/cinder/+/839793 | 17:13 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: annotate image/glance.py https://review.opendev.org/c/openstack/cinder/+/766064 | 17:43 |
| opendevreview | Eric Harney proposed openstack/cinder master: Use modern type annotation format for collections https://review.opendev.org/c/openstack/cinder/+/839987 | 17:52 |
| opendevreview | Eric Harney proposed openstack/cinder master: db api: remove "inactive" arg from get_all_types https://review.opendev.org/c/openstack/cinder/+/841302 | 18:00 |
| rosmaita | ricolin: ack | 18:28 |
| rosmaita | enriquetaso: left a reply for you on https://review.opendev.org/c/openstack/cinder/+/839628 | 18:30 |
| rosmaita | ricolin: that was a good comment to add, should be helpful | 18:35 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: RBD: Fix _show_msg_check_clone_v2_api https://review.opendev.org/c/openstack/cinder/+/777031 | 19:00 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: RBD: Fix _show_msg_check_clone_v2_api https://review.opendev.org/c/openstack/cinder/+/777031 | 19:05 |
| enriquetaso | thanks rosmaita for the reply | 19:08 |
| opendevreview | Merged openstack/cinder master: Address pylint error https://review.opendev.org/c/openstack/cinder/+/839628 | 20:51 |
| *** dviroel is now known as dviroel|out | 20:51 | |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Remove single-use test function https://review.opendev.org/c/openstack/cinder/+/842000 | 21:32 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!