| opendevreview | Felipe Rodrigues proposed openstack/cinder master: NetApp NFS ONTAP: Deprecate Copy Offload Tool https://review.opendev.org/c/openstack/cinder/+/847733 | 00:31 |
|---|---|---|
| *** brinzhang0 is now known as brinzhang | 01:36 | |
| *** brinzhang0 is now known as brinzhang | 02:51 | |
| TusharTgite_ | hi can anyone confirm my assesment on this is correct ? https://bugs.launchpad.net/cinder/+bug/1908181 for closing this bug | 04:19 |
| *** brinzhang_ is now known as brinzhang | 09:12 | |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Dell EMC Unity driver in cinder https://review.opendev.org/c/openstack/cinder/+/856460 | 10:25 |
| opendevreview | Matus Brandys proposed openstack/cinder master: 3PAR driver - Fix for return value while getting updated comment. https://review.opendev.org/c/openstack/cinder/+/856463 | 10:45 |
| opendevreview | Felipe Rodrigues proposed openstack/cinder master: NetApp NFS ONTAP: Deprecate Copy Offload Tool https://review.opendev.org/c/openstack/cinder/+/847733 | 11:15 |
| SDrozdov | geguileo: Would it be possible for you to check https://review.opendev.org/c/openstack/cinder/+/820027, please? I think it is almost done. If anyone else is interested in reviewing please feel free, the topic is RBD backend QoS implementation | 11:53 |
| opendevreview | Matus Brandys proposed openstack/cinder master: 3PAR driver - Fix for return value while getting updated comment. https://review.opendev.org/c/openstack/cinder/+/856463 | 12:43 |
| opendevreview | Sergey Drozdov proposed openstack/cinder master: RBD backend QoS implementation https://review.opendev.org/c/openstack/cinder/+/820027 | 13:00 |
| sfernand | hi whoami-rajat, felipe addressed the missing reno in the deprecation patch | 13:29 |
| sfernand | is the feature freeze ocurring today our tomorrow? | 13:29 |
| whoami-rajat | sfernand, tomorrow EOD would be suitable to get things in | 13:30 |
| whoami-rajat | ack will take a look | 13:30 |
| sfernand | whoami-rajat: cool thanks | 13:31 |
| yuval | whoami-rajat can you attach again the high priority patches need to be reviewed? | 13:47 |
| whoami-rajat | yuval, the top ones are higher priority https://etherpad.opendev.org/p/cinder-zed-features | 13:48 |
| yuval | I though ONTAP is next cycle | 13:50 |
| akawai | rosmaita: The patch "Hitachi: Add port scheduler | https://review.opendev.org/c/openstack/cinder/+/828696 " had got RP+1. Does the patch need one more core-reviewing to merge? | 14:26 |
| rosmaita | yes, the RP+1 is just to get people's attention | 14:27 |
| akawai | rosmaita: thx | 14:27 |
| akawai | core reviewers: pls review "Hitachi: Add port scheduler | https://review.opendev.org/c/openstack/cinder/+/828696 ". the patch had got RP+1 on Yoga, but time was up... | 14:31 |
| whoami-rajat | akawai, will take a look but that should've been mentioned here https://etherpad.opendev.org/p/cinder-zed-features | 14:32 |
| whoami-rajat | we prioritized reviews based on that | 14:32 |
| opendevreview | Jean Pierre Roquesalane proposed openstack/cinder master: PowerMax Driver - Fix for renaming GVG https://review.opendev.org/c/openstack/cinder/+/801354 | 14:34 |
| akawai | whoami-rajat: i see. | 14:34 |
| *** bhagyashris is now known as bhagyashris|ruck | 14:37 | |
| akawai | whoami-rajat: My patch "Update docs for Hitachi driver | https://review.opendev.org/c/openstack/cinder/+/809534 " is to fix document. Is the merging deadline for the patch in the next week? | 14:46 |
| whoami-rajat | akawai, yes, doc changes can merge next week | 14:47 |
| akawai | whoami-rajat: thx! | 14:47 |
| whoami-rajat | np | 14:51 |
| whoami-rajat | jungleboyj, hey, can you take a look at this change? small change, it's deprecating a netapp config parameter https://review.opendev.org/c/openstack/cinder/+/847733 | 14:54 |
| hemna | mep | 14:56 |
| *** dviroel is now known as dviroel|lunch | 14:57 | |
| *** dviroel_ is now known as dviroel | 14:57 | |
| yuval | harsh ailani is around here? | 14:57 |
| yuval | anyway I think I see an issue here: https://review.opendev.org/c/openstack/cinder/+/852364 its the SVF patch | 14:59 |
| jungleboyj | whoami-rajat: Done. :-) | 15:03 |
| whoami-rajat | thanks Jay! | 15:04 |
| yuval | I know its not priority but would glad for some reviews on: https://review.opendev.org/c/openstack/cinder/+/856259 or https://review.opendev.org/c/openstack/cinder/+/839768 | 15:20 |
| opendevreview | yuval proposed openstack/cinder master: Lightos - add lightos new volume states https://review.opendev.org/c/openstack/cinder/+/856259 | 15:26 |
| *** dviroel is now known as dviroel_ | 15:46 | |
| *** dviroel|lunch is now known as dviroel | 15:46 | |
| hemna | mep | 15:53 |
| opendevreview | Merged openstack/cinder master: NetApp NFS ONTAP: Deprecate Copy Offload Tool https://review.opendev.org/c/openstack/cinder/+/847733 | 16:44 |
| opendevreview | Matus Brandys proposed openstack/cinder master: 3PAR driver - Fix for return value while getting updated comment. https://review.opendev.org/c/openstack/cinder/+/856463 | 18:00 |
| opendevreview | Walt proposed openstack/cinder master: Fix and unify capacity calculations https://review.opendev.org/c/openstack/cinder/+/831247 | 19:12 |
| whoami-rajat | hi jungleboyj , if you get some time, can you take a look at this change as well? this is a continuation of the last change we reviewed and completes this feature https://review.opendev.org/c/openstack/cinder/+/847732 | 19:30 |
| hemna | man I still believe that drivers should be out of the tree | 19:33 |
| hemna | we have so many driver reviews and reviews on the core of cinder are just sitting | 19:33 |
| whoami-rajat | hemna, are you referring to any particular cinder feature? | 19:38 |
| hemna | just looking through review and so many are stagnant | 19:38 |
| hemna | reviews | 19:38 |
| opendevreview | Merged openstack/cinder master: Tests: Fix test runner never finishing https://review.opendev.org/c/openstack/cinder/+/854261 | 19:39 |
| whoami-rajat | yeah, we've had quite a lot of driver features and even drivers this cycle | 19:39 |
| hemna | like the db patches and my fixes for the scheduler and tracking of allocated_capacity | 19:39 |
| hemna | etc | 19:39 |
| hemna | all the drivers should be pypi packages that can be installed by the deployer. | 19:40 |
| hemna | </rant> | 19:40 |
| hemna | heh another good reason.....https://review.opendev.org/c/openstack/cinder/+/854263 | 19:41 |
| *** dviroel is now known as dviroel|out | 21:57 | |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: NetApp ONTAP: Add volume replication functions on REST client https://review.opendev.org/c/openstack/cinder/+/852209 | 22:20 |
| opendevreview | Merged openstack/cinder master: Tests: Fix inspur unit test https://review.opendev.org/c/openstack/cinder/+/854262 | 23:16 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!