| opendevreview | Brian Rosmaita proposed openstack/cinder master: WIP/DNM: Add fips check jobs https://review.opendev.org/c/openstack/cinder/+/790535 | 00:57 |
|---|---|---|
| opendevreview | Rajat Dhasmana proposed openstack/cinder stable/wallaby: Fix: Schema validation for attachment create API https://review.opendev.org/c/openstack/cinder/+/802781 | 03:56 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Test glance with cinder backend optimization https://review.opendev.org/c/openstack/cinder/+/757936 | 04:08 |
| gksk | To the core reviewers , Please review the below patch related to update replication properties fix for empty values | 05:27 |
| gksk | https://review.opendev.org/c/openstack/cinder/+/801617 | 05:27 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: NetApp ONTAP: Add storage assisted migration support https://review.opendev.org/c/openstack/cinder/+/799668 | 06:33 |
| opendevreview | Fernando Ferraz proposed openstack/cinder master: NetApp ONTAP: Add storage assisted migration support https://review.opendev.org/c/openstack/cinder/+/799668 | 06:37 |
| opendevreview | katari manoj kumar proposed openstack/cinder master: [SVF]:Fix add volumes to GMCV group https://review.opendev.org/c/openstack/cinder/+/784092 | 06:57 |
| * lyarwood < https://matrix.org/_matrix/media/r0/download/matrix.org/QePrOjsgAjfliTiJWgvYveQm/message.txt > | 07:54 | |
| lyarwood | https://zuul.opendev.org/t/openstack/build/fb2574989dcd41069df44b37fd74db01 - anyone working on this? | 07:55 |
| lyarwood | Odd, I can't seem to reproduce it, hopefully just a weird one off. | 09:28 |
| venkatakrishnat | Hi eharney, Could you please review https://review.opendev.org/c/openstack/cinder/+/788782 | 10:49 |
| *** dviroel|ruck|out is now known as dviroel|ruck | 11:26 | |
| *** bhagyashris_ is now known as bhagyashris | 11:31 | |
| opendevreview | Merged openstack/cinder stable/wallaby: PowerMax Driver - Fix for create snapshot https://review.opendev.org/c/openstack/cinder/+/805060 | 12:34 |
| rosmaita | lyarwood: had not seen that, but the job looks OK now ... hate it when these weird one-offs start happening just before feature freeze | 13:11 |
| lyarwood | yeah I've seen things like this before and it was just an issue with our pypy caches in CI, hopefully that's all it is this time. I'll let you know if I see it again. | 13:19 |
| lyarwood | oh looks like others have seen this in #openstack-qa | 13:20 |
| lyarwood | and #opendev | 13:21 |
| rosmaita | reminder: festival of XS reviews at 1400 UTC | 13:59 |
| rosmaita | join us in meetpad: https://meetpad.opendev.org/cinder-festival-of-reviews | 13:59 |
| rosmaita | direct link to the etherpad: https://etherpad.opendev.org/p/cinder-festival-of-reviews | 13:59 |
| rosmaita | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna ^^ | 13:59 |
| * enriquetaso connecting | 14:03 | |
| rosmaita | #startmeeting cinder_reviewfest | 14:05 |
| opendevmeet | Meeting started Fri Aug 20 14:05:24 2021 UTC and is due to finish in 60 minutes. The chair is rosmaita. Information about MeetBot at http://wiki.debian.org/MeetBot. | 14:05 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:05 |
| opendevmeet | The meeting name has been set to 'cinder_reviewfest' | 14:05 |
| rosmaita | #link https://etherpad.opendev.org/p/cinder-festival-of-reviews | 14:05 |
| rosmaita | #topic roll call | 14:06 |
| rosmaita | o/ | 14:06 |
| smcginnis | o/ (not really here, but didn't want rosmaita to feel too lonely) | 14:36 |
| rosmaita | :) | 14:36 |
| rosmaita | actually, tosky walshh_ enriquetaso and Stefan are all here too | 14:37 |
| opendevreview | Helen Walsh proposed openstack/cinder stable/victoria: PowerMax Driver - Fix for create snapshot https://review.opendev.org/c/openstack/cinder/+/805398 | 14:47 |
| enriquetaso | https://review.opendev.org/c/openstack/cinder/+/805072 | 14:58 |
| opendevreview | Helen Walsh proposed openstack/cinder stable/victoria: PowerMax Driver - Fix for create snapshot https://review.opendev.org/c/openstack/cinder/+/805398 | 15:09 |
| opendevreview | Helen Walsh proposed openstack/cinder stable/victoria: PowerMax Driver - Fix for create snapshot https://review.opendev.org/c/openstack/cinder/+/805398 | 15:10 |
| manoj_katari | @core team: please review this change https://review.opendev.org/c/openstack/cinder/+/784092, addressed all comments and CI is green. | 15:56 |
| rosmaita | thanks to everyone who participated in the review festival! | 16:04 |
| rosmaita | #endmeeting | 16:04 |
| opendevmeet | Meeting ended Fri Aug 20 16:04:42 2021 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:04 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_reviewfest/2021/cinder_reviewfest.2021-08-20-14.05.html | 16:04 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_reviewfest/2021/cinder_reviewfest.2021-08-20-14.05.txt | 16:04 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_reviewfest/2021/cinder_reviewfest.2021-08-20-14.05.log.html | 16:04 |
| venkatakrishnat | Hi geguileo, hemna, jungleboyj, rosmaita: Could you please review https://review.opendev.org/c/openstack/cinder/+/786524 which is to manage Global mirror change volumes on separate storage pools. | 16:29 |
| opendevreview | Merged openstack/cinder master: mypy: continued manager, scheduler, rpcapi https://review.opendev.org/c/openstack/cinder/+/784453 | 17:17 |
| opendevreview | Merged openstack/cinder master: Doc: Remove backup_id from backup import url and parameter https://review.opendev.org/c/openstack/cinder/+/803896 | 17:18 |
| opendevreview | Merged openstack/cinder master: Doc: api-ref doc update https://review.opendev.org/c/openstack/cinder/+/793386 | 17:18 |
| opendevreview | Merged openstack/cinder master: Replace deprecated tenacity.Retrying.call https://review.opendev.org/c/openstack/cinder/+/801916 | 17:18 |
| opendevreview | Merged openstack/cinder master: Allow removing NFS snapshots in error status https://review.opendev.org/c/openstack/cinder/+/679138 | 17:18 |
| opendevreview | Merged openstack/cinder stable/wallaby: Add libcgroup related packages in bindep.txt https://review.opendev.org/c/openstack/cinder/+/801236 | 17:18 |
| opendevreview | Sofia Enriquez proposed openstack/cinder stable/wallaby: Allow removing NFS snapshots in error status https://review.opendev.org/c/openstack/cinder/+/805283 | 18:21 |
| opendevreview | Merged openstack/cinder master: conditional_update(): disable false pylint error message https://review.opendev.org/c/openstack/cinder/+/804118 | 18:58 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Prototype base policy file https://review.opendev.org/c/openstack/cinder/+/803748 | 20:42 |
| *** dviroel|ruck is now known as dviroel|out | 22:04 | |
| opendevreview | Merged openstack/cinder stable/victoria: PowerMax Driver - Fix for create snapshot https://review.opendev.org/c/openstack/cinder/+/805398 | 23:19 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!