| opendevreview | Tony Saad proposed openstack/cinder master: Fixed volume caching in PowerStore driver. Volume caching stops working after 32nd volume created from the same image due to snapshot limit per volume in PowerStore. The driver needs to throw exception SnapshotLimitReached to invalidate cache and start over with new volume snapshots. https://review.opendev.org/c/openstack/cinder/+/867753 | 01:53 |
|---|---|---|
| opendevreview | XuQi proposed openstack/cinder master: Fujitsu Driver: Add QoS support https://review.opendev.org/c/openstack/cinder/+/847730 | 03:35 |
| *** amoralej|off is now known as amoralej | 07:21 | |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Deleting a volume in 'downloading' state https://review.opendev.org/c/openstack/cinder/+/826607 | 07:29 |
| opendevreview | Moritz Wanzenböck proposed openstack/cinder master: Add install docs for LINSTOR driver dependencies https://review.opendev.org/c/openstack/cinder/+/773870 | 08:40 |
| opendevreview | XuQi proposed openstack/cinder master: Fujitsu Driver: Modify extend volume https://review.opendev.org/c/openstack/cinder/+/852007 | 08:54 |
| opendevreview | Konrad Gube proposed openstack/cinder-specs master: Add extend volume completion action https://review.opendev.org/c/openstack/cinder-specs/+/866718 | 11:01 |
| *** dviroel|out is now known as dviroel|rover | 11:12 | |
| opendevreview | Harsh Ailani proposed openstack/cinder master: [SVf]: mkhost failure when volume and node are on different iogrp https://review.opendev.org/c/openstack/cinder/+/853126 | 12:43 |
| *** amoralej is now known as amoralej|lunch | 13:00 | |
| *** akekane is now known as abhishekk | 13:45 | |
| *** amoralej|lunch is now known as amoralej | 14:08 | |
| harsh | Hello reviewers, for a while due to IBM - CI unavailability many SVf patches were on hold. But we just managed to fix the third party IBM-CI. | 15:06 |
| zaitcev | Coo! | 15:07 |
| harsh | Requesting you to please review the patch: https://review.opendev.org/c/openstack/cinder/+/853126 | 15:07 |
| harsh | Customer have already confirmed that the code change has resolved their issue. | 15:07 |
| harsh | I have many more patches on hold which i will share in a timely order. | 15:08 |
| zaitcev | Sorry, I'm going to be a jerk and punt the StoreVise onto Alan and Gorka | 15:08 |
| harsh | But this patch is on high priority as it is a fix for a customer production environment. | 15:08 |
| harsh | Appreciate your quick response #zaitcev :) | 15:10 |
| zaitcev | harsh: BTW it does not look right. The code maintains a list, which the "for" loop pads. Okay. In the old code, the collected list was then stored once it's done. But the new code continuously stores that list on every iteration... while it's being updated...? That just looks bogus to me. I don't have the time to really understand, but the it looks wrong. | 15:16 |
| zaitcev | Maybe you want some finally: there or maybe else: | 15:17 |
| harsh | thanks for the input. let me see how i can resolve this | 15:19 |
| zaitcev | Keep in mind maybe I'm just an idiot. | 15:19 |
| zaitcev | So I'm not putting -1 to it. | 15:19 |
| harsh | no worries. i will cross check this again and see if any changes are required. | 15:20 |
| opendevreview | Eric Harney proposed openstack/cinder master: Tests: Fix double mock of Popen (test_backup_ceph) https://review.opendev.org/c/openstack/cinder/+/867824 | 15:52 |
| *** amoralej is now known as amoralej|off | 16:36 | |
| opendevreview | Merged openstack/cinderlib master: Fix cinderlib tests https://review.opendev.org/c/openstack/cinderlib/+/867132 | 17:45 |
| opendevreview | Merged openstack/cinderlib master: Open cinderlib for zed development https://review.opendev.org/c/openstack/cinderlib/+/848846 | 17:45 |
| opendevreview | Merged openstack/cinderlib master: Fix py310 unit tests https://review.opendev.org/c/openstack/cinderlib/+/867583 | 17:45 |
| opendevreview | Merged openstack/cinderlib master: Don't limit use of importlib-metadata to Python < 3.8 https://review.opendev.org/c/openstack/cinderlib/+/844981 | 17:45 |
| opendevreview | Merged openstack/cinderlib master: Remove six https://review.opendev.org/c/openstack/cinderlib/+/840568 | 17:45 |
| opendevreview | Merged openstack/cinderlib master: Support driver's setup method https://review.opendev.org/c/openstack/cinderlib/+/815783 | 17:45 |
| opendevreview | Merged openstack/cinderlib master: Add Zed release note https://review.opendev.org/c/openstack/cinderlib/+/867652 | 17:45 |
| *** dviroel|rover is now known as dviroel|rover|afk | 21:16 | |
| opendevreview | Brian Rosmaita proposed openstack/cinderlib master: Open cinderlib for 2023.1 (Antelope) development https://review.opendev.org/c/openstack/cinderlib/+/867904 | 21:40 |
| *** zaitcev_ is now known as zaitcev | 22:29 | |
| *** felipe_rodrigues_ is now known as felipe_rodrigues | 22:29 | |
| opendevreview | Brian Rosmaita proposed openstack/cinderlib master: Open cinderlib for 2023.1 (Antelope) development https://review.opendev.org/c/openstack/cinderlib/+/867904 | 22:29 |
| opendevreview | Tony Saad proposed openstack/cinder master: Fixed volume caching in PowerStore driver. Volume caching stops working after 32nd volume created from the same image due to snapshot limit per volume in PowerStore. The driver needs to throw exception SnapshotLimitReached to invalidate cache and start over with new volume snapshots. https://review.opendev.org/c/openstack/cinder/+/867753 | 22:36 |
| *** sfinucan is now known as stephenfin | 22:55 | |
| opendevreview | Brian Rosmaita proposed openstack/cinderlib master: Remove unnecessary #egg= URL fragment https://review.opendev.org/c/openstack/cinderlib/+/867905 | 23:58 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!