| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: support extending volume with snapshot https://review.opendev.org/c/openstack/cinder/+/850830 | 01:23 |
|---|---|---|
| opendevreview | XuQi proposed openstack/cinder master: Fujitsu Driver: Add QoS support https://review.opendev.org/c/openstack/cinder/+/847730 | 02:44 |
| *** akekane is now known as abhishekk | 05:36 | |
| *** amoralej|off is now known as amoralej | 08:22 | |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: add extra spec "hbsd:remote_target_ports" for GAD https://review.opendev.org/c/openstack/cinder/+/867340 | 08:39 |
| opendevreview | OpenStack Release Bot proposed openstack/cinderlib stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/cinderlib/+/867952 | 08:50 |
| opendevreview | OpenStack Release Bot proposed openstack/cinderlib master: Update master for stable/zed https://review.opendev.org/c/openstack/cinderlib/+/867953 | 08:50 |
| *** bhagyashris is now known as bhagyashris|sick | 09:26 | |
| enriquetaso | Cinderlib Gate is working again in Zed \o/ | 09:56 |
| 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 | 10:25 |
| *** dviroel|rover|afk is now known as dviroel|rover | 11:16 | |
| *** amoralej is now known as amoralej|off | 11:34 | |
| kgube | whoami-rajat: would it be possible to get an extension for https://review.opendev.org/c/openstack/cinder-specs/+/866718 ? | 11:44 |
| kgube | whoami-rajat: eharney hasn't reviewed it yet, but there is strong support from Nova now | 11:44 |
| whoami-rajat | kgube, yes, i think we can provide exception for all the specs | 11:45 |
| whoami-rajat | they didn't get much review attention this week | 11:45 |
| kgube | thank you! | 11:46 |
| opendevreview | Merged openstack/cinderlib master: Update master for stable/zed https://review.opendev.org/c/openstack/cinderlib/+/867953 | 12:15 |
| *** dviroel|rover is now known as dviroel | 12:20 | |
| opendevreview | Merged openstack/cinderlib stable/zed: Update .gitreview for stable/zed https://review.opendev.org/c/openstack/cinderlib/+/867952 | 13:25 |
| harsh | Hi #zaitcev #geguileo , i went through the code changes again and you guys were correct. There was a logical redundancy in the loops. So i have modified the code and now each node_id will get its on IP_address in the corresponding portset. Here is the patch: https://review.opendev.org/c/openstack/cinder/+/853126 | 13:52 |
| harsh | Requesting the core reviewers to please review this patch as it was on hold for a few months due to third party CI issues. It important for the Red Hat customers to have this fix. | 13:55 |
| rosmaita | Cinder Festival of Reviews today, 1400-1600 UTC | 14:00 |
| rosmaita | there have been some improvements in meetpad, so let's try it again: https://meetpad.opendev.org/cinder-festival-of-reviews | 14:00 |
| rosmaita | and here is the direct link to the etherpad: https://etherpad.opendev.org/p/cinder-festival-of-reviews | 14:00 |
| rosmaita | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira happystacker ^^ | 14:01 |
| enriquetaso | hey | 14:02 |
| enriquetaso | Simon, i think you are unmute | 14:07 |
| *** akekane is now known as abhishekk | 14:11 | |
| rosmaita | harsh: left a comment for you on https://review.opendev.org/c/openstack/cinder/+/853126/ | 14:43 |
| opendevreview | Eric Harney proposed openstack/cinder master: Tests: Fix double mock of Popen (test_backup_ceph) https://review.opendev.org/c/openstack/cinder/+/867824 | 14:50 |
| enriquetaso | harsh, are you planing to include cinder-tempest-plugin on the IBM CI? | 14:54 |
| harsh | Thanks #rosmiata, i will add the UT for the code. | 14:58 |
| harsh | #enriquetaso, no we do not include the cinder-tempest-plugin in the IBM CI. storwize-tempest-full-iscsi and storwize-tempest-full-fc are only two testcase sets which verify our driver patches. | 15:01 |
| *** dviroel is now known as dviroel|lunch | 15:05 | |
| opendevreview | Brian Rosmaita proposed openstack/cinderlib master: Open cinderlib for 2023.1 (Antelope) development https://review.opendev.org/c/openstack/cinderlib/+/867904 | 15:06 |
| opendevreview | Brian Rosmaita proposed openstack/cinderlib master: Remove unnecessary #egg= URL fragment https://review.opendev.org/c/openstack/cinderlib/+/867905 | 15:09 |
| enriquetaso | harsh, just would like to doble check, have you check that https://review.opendev.org/c/openstack/cinder/+/855451 doesn't break stable/yoga IBM Storage CI ? | 15:21 |
| harsh | i haven't checked in the IBM-CI in stable/yoga but i have tested this code in stable/yoga and it doesn't break any functionality. | 15:29 |
| andrewbogott_ | If folks are reviewing can I put in a plug for merging https://review.opendev.org/c/openstack/cinder/+/861566 and https://review.opendev.org/c/openstack/cinder/+/864585? They've been in +1 limbo for quite a while. | 15:32 |
| harsh | #enriquetaso, the original review was tested on stable/yoga IBM - CI : https://review.opendev.org/c/openstack/cinder/+/844572 | 15:34 |
| *** dviroel|lunch is now known as dviroel | 15:58 | |
| rosmaita | the Festival of XS Reviews is over ... thanks for attending! | 16:03 |
| happystacker | sorry guys, I just realized that I've missed the XS review meeting, was wrapping up a few things before going on PTO until next year | 16:05 |
| rosmaita | happystacker: if you have some time, you can always grab a few reviews: http://tiny.cc/cinder-reviewfest | 16:09 |
| happystacker | sure | 16:16 |
| happystacker | and btw, I'd appreciate if you give your +2 or not to https://review.opendev.org/c/openstack/cinder/+/844563 | 16:29 |
| opendevreview | Merged openstack/cinder stable/ussuri: [PURE] support IPv6 / add parameter pure_iscsi_cidr_list https://review.opendev.org/c/openstack/cinder/+/867294 | 17:02 |
| opendevreview | Alan Bishop proposed openstack/cinder stable/yoga: Fix and unify capacity calculations https://review.opendev.org/c/openstack/cinder/+/867994 | 19:26 |
| opendevreview | Brian Rosmaita proposed openstack/cinderlib master: Get ready for tox 4 https://review.opendev.org/c/openstack/cinderlib/+/866936 | 19:42 |
| opendevreview | Merged openstack/cinder master: Remove unused session parameter https://review.opendev.org/c/openstack/cinder/+/850605 | 20:00 |
| opendevreview | Merged openstack/cinder stable/yoga: Fix Infinidat driver to return all iSCSI portals https://review.opendev.org/c/openstack/cinder/+/858344 | 20:04 |
| opendevreview | Merged openstack/cinder stable/yoga: Tests: Fix test runner never finishing https://review.opendev.org/c/openstack/cinder/+/858819 | 20:04 |
| opendevreview | Merged openstack/cinder master: nit: correct comment in backup manager https://review.opendev.org/c/openstack/cinder/+/866788 | 20:18 |
| *** dviroel is now known as dviroel|out | 20:30 | |
| *** osmanlicilegi is now known as Guest0 | 22:33 | |
| *** ChanServ changes topic to "The Block Storage Project | https://wiki.openstack.org/wiki/Cinder | https://tiny.cc/CinderPriorities" | 22:39 | |
| opendevreview | Merged openstack/cinder master: Fix Infinidat driver generic volume migration https://review.opendev.org/c/openstack/cinder/+/850320 | 23:28 |
| opendevreview | Merged openstack/cinder master: Fix Infinidat driver multi-attach feature https://review.opendev.org/c/openstack/cinder/+/850321 | 23:34 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!