| opendevreview | Merged openstack/cinder master: Change cinder-mypy job to voting https://review.opendev.org/c/openstack/cinder/+/850939 | 00:45 |
|---|---|---|
| opendevreview | Oleg proposed openstack/cinder master: Add NVMe/TCP support to Dell EMC PowerStore driver https://review.opendev.org/c/openstack/cinder/+/819149 | 01:37 |
| opendevreview | Alan Bishop proposed openstack/cinder master: Add support for transferring encrypted volumes https://review.opendev.org/c/openstack/cinder/+/851449 | 03:27 |
| opendevreview | OpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/c/openstack/cinder/+/851279 | 04:23 |
| opendevreview | Friedrich Hiekel proposed openstack/cinder master: add netapp copyoffload provider location https://review.opendev.org/c/openstack/cinder/+/834822 | 05:13 |
| *** bhagyashris is now known as bhagyashris|ruck | 06:05 | |
| olegnest | jungleboyj sfernand: Hello, please recheck a new patchset for DEMC NVMe driver https://review.opendev.org/c/openstack/cinder/+/819149. Zuul and CI have been passed. | 10:23 |
| *** dviroel|afk is now known as dviroel | 11:15 | |
| *** dviroel is now known as dviroel|rover | 11:18 | |
| opendevreview | Oleg proposed openstack/cinder master: Add NVMe/TCP support to Dell EMC PowerStore driver https://review.opendev.org/c/openstack/cinder/+/819149 | 12:06 |
| *** dviroel|rover is now known as dviroel | 13:20 | |
| *** dviroel is now known as dviroel|afk | 13:25 | |
| opendevreview | Simon Dodsley proposed openstack/cinder master: Add Pure Storage NVMe-RoCE driver https://review.opendev.org/c/openstack/cinder/+/799871 | 13:34 |
| opendevreview | Victoria Martinez de la Cruz proposed openstack/devstack-plugin-ceph master: Remove pg_num and pgp_num config from bootstrap_ceph https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/851521 | 14:01 |
| whoami-rajat | hello cinder cores, out of 4 drivers, 1 is merged and 3 looks good for merging, please take a look https://etherpad.opendev.org/p/cinder-zed-new-drivers | 14:31 |
| whoami-rajat | and also today is the deadline | 14:32 |
| enriquetaso_ | whoami-rajat, ok! | 14:34 |
| HappyStacker | Thanks guys for your help on https://review.opendev.org/c/openstack/cinder/+/851249 | 14:46 |
| whoami-rajat | HappyStacker, no problem, but make sure to follow up on the review comments | 14:57 |
| HappyStacker | sure, We talked about it already with the developers | 14:59 |
| simondodsley | All - The Pure NVME driver had all the comments addressed and has local the Pure CI and the main zuul CI | 15:54 |
| simondodsley | s/local/passed/ | 16:13 |
| *** dviroel|afk is now known as dviroel | 16:14 | |
| simondodsley | All cores- can this be addressed please https://review.opendev.org/c/openstack/cinder/+/849690 - needed for Pure NVMe driver | 16:31 |
| *** hemna7 is now known as hemna | 16:48 | |
| opendevreview | Merged openstack/cinder master: Add Cinder NFS driver for Dell PowerStore https://review.opendev.org/c/openstack/cinder/+/851249 | 17:13 |
| enriquetaso_ | simondodsley, ill review | 17:21 |
| opendevreview | Merged openstack/cinder master: Add NVMe/TCP support to Dell EMC PowerStore driver https://review.opendev.org/c/openstack/cinder/+/819149 | 17:54 |
| whoami-rajat | jungleboyj, enriquetaso_ eharney can you take a look at the Pure driver, it's the last one left (minor nits can be addressed in a followup) https://review.opendev.org/c/openstack/cinder/+/799871 | 17:59 |
| enriquetaso_ | whoami-rajat, done! | 18:43 |
| whoami-rajat | enriquetaso_, thank you | 18:44 |
| whoami-rajat | with that we've merged all the targeted drivers, thanks everyone for the reviews! | 18:44 |
| whoami-rajat | specially enriquetaso_ ! | 18:45 |
| enriquetaso_ | \o/ | 18:48 |
| enriquetaso_ | we expect some follow up patches but we made it \o/ | 18:48 |
| HappyStacker | thanks everyone for merging all new drivers | 18:53 |
| opendevreview | Merged openstack/cinder master: Add additional transport type constants https://review.opendev.org/c/openstack/cinder/+/849690 | 18:57 |
| sfernand | So sorry team, I'm working on urgency Today and couldn't revisit to check the CI. I had checked the answers to my comments earlier and was ready to give the +1s anyway. | 19:37 |
| sfernand | Congrats ;) | 19:38 |
| whoami-rajat | sfernand, thanks for all your reviews, it helped a lot | 19:49 |
| sfernand | yw :D | 19:57 |
| opendevreview | Merged openstack/cinder master: Add Pure Storage NVMe-RoCE driver https://review.opendev.org/c/openstack/cinder/+/799871 | 20:10 |
| *** dviroel is now known as dviroel|out | 21:33 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!