| opendevreview | Moritz Tanner proposed openstack/os-brick master: Refuse encryptor attach if device_path is no symlink https://review.opendev.org/c/openstack/os-brick/+/1008416 | 06:50 |
|---|---|---|
| opendevreview | Biser Milanov proposed openstack/cinder master: StorPool: Do not use the option 'storpool_replication' when creating or retyping volumes https://review.opendev.org/c/openstack/cinder/+/973592 | 07:50 |
| opendevreview | Biser Milanov proposed openstack/cinder master: StorPool: Use only baseOn when cloning a volume https://review.opendev.org/c/openstack/cinder/+/973593 | 07:50 |
| opendevreview | Biser Milanov proposed openstack/cinder master: StorPool: Add the storpool_qos option and deprecate the storpool_template one https://review.opendev.org/c/openstack/cinder/+/973594 | 07:50 |
| opendevreview | Biser Milanov proposed openstack/cinder master: StorPool: Remove the storpool_template option https://review.opendev.org/c/openstack/cinder/+/973595 | 07:50 |
| opendevreview | Biser Milanov proposed openstack/cinder master: Add iSCSI export support to the StorPool driver https://review.opendev.org/c/openstack/cinder/+/847536 | 07:50 |
| noonedeadpunk | rosmaita: hey! have a question about of reviews festival. Is there a place where I can add review request for my patches? As I guess etherpad in email is populated during the meeting? | 08:09 |
| noonedeadpunk | Or I can just add a "October 2" and links ? | 08:09 |
| noonedeadpunk | or they are pulled from some other list? | 08:09 |
| noonedeadpunk | (as I can recall some other etherpad existing with actually list of patches) | 08:10 |
| noonedeadpunk | I think I have 3 of them :) | 08:18 |
| opendevreview | Biser Milanov proposed openstack/cinder master: StorPool: Do not use the option 'storpool_replication' when creating or retyping volumes https://review.opendev.org/c/openstack/cinder/+/973592 | 08:28 |
| opendevreview | Biser Milanov proposed openstack/cinder master: StorPool: Use only baseOn when cloning a volume https://review.opendev.org/c/openstack/cinder/+/973593 | 08:28 |
| opendevreview | Biser Milanov proposed openstack/cinder master: StorPool: Add the storpool_qos option and deprecate the storpool_template one https://review.opendev.org/c/openstack/cinder/+/973594 | 08:28 |
| opendevreview | Biser Milanov proposed openstack/cinder master: StorPool: Remove the storpool_template option https://review.opendev.org/c/openstack/cinder/+/973595 | 08:28 |
| opendevreview | Biser Milanov proposed openstack/cinder master: Add iSCSI export support to the StorPool driver https://review.opendev.org/c/openstack/cinder/+/847536 | 08:28 |
| sp-bmilanov | hi jayaanand re: https://review.opendev.org/c/openstack/cinder/+/973593/7..8 is that ok for a release note? | 08:33 |
| sfv880 | Hello, when you have a chance, I’d appreciate a review of the 940236: Infinidat: Add support for snapshot promotion change: https://review.opendev.org/c/openstack/cinder/+/940236 All previous review comments have been addressed, and the latest patch set has passed Zuul check tests and Infinidat CI/CD. It has also received one Code-Review +2. If another core reviewer has time to take a look, that would be very helpful. Any additional | 08:55 |
| sfv880 | feedback from the community is welcome too. Thank you very much! | 08:55 |
| opendevreview | minwoo seo proposed openstack/cinder master: Add cinder-manage image-cache warm/list subcommand https://review.opendev.org/c/openstack/cinder/+/986740 | 08:56 |
| opendevreview | Biser Milanov proposed openstack/cinder master: StorPool: Do not use the option 'storpool_replication' when creating or retyping volumes https://review.opendev.org/c/openstack/cinder/+/973592 | 12:03 |
| opendevreview | Biser Milanov proposed openstack/cinder master: StorPool: Use only baseOn when cloning a volume https://review.opendev.org/c/openstack/cinder/+/973593 | 12:03 |
| opendevreview | Biser Milanov proposed openstack/cinder master: StorPool: Add the storpool_qos option and deprecate the storpool_template one https://review.opendev.org/c/openstack/cinder/+/973594 | 12:03 |
| opendevreview | Biser Milanov proposed openstack/cinder master: StorPool: Remove the storpool_template option https://review.opendev.org/c/openstack/cinder/+/973595 | 12:03 |
| opendevreview | Biser Milanov proposed openstack/cinder master: Add iSCSI export support to the StorPool driver https://review.opendev.org/c/openstack/cinder/+/847536 | 12:03 |
| opendevreview | Biser Milanov proposed openstack/cinder master: StorPool: Fix a leftover dict unpacking https://review.opendev.org/c/openstack/cinder/+/1008446 | 12:03 |
| opendevreview | Biser Milanov proposed openstack/cinder master: StorPool: Fix a leftover dict unpacking https://review.opendev.org/c/openstack/cinder/+/1008446 | 12:37 |
| opendevreview | Biser Milanov proposed openstack/cinder master: StorPool: Do not use the option 'storpool_replication' when creating or retyping volumes https://review.opendev.org/c/openstack/cinder/+/973592 | 12:37 |
| opendevreview | Biser Milanov proposed openstack/cinder master: StorPool: Use only baseOn when cloning a volume https://review.opendev.org/c/openstack/cinder/+/973593 | 12:37 |
| opendevreview | Biser Milanov proposed openstack/cinder master: StorPool: Add the storpool_qos option and deprecate the storpool_template one https://review.opendev.org/c/openstack/cinder/+/973594 | 12:37 |
| opendevreview | Biser Milanov proposed openstack/cinder master: StorPool: Remove the storpool_template option https://review.opendev.org/c/openstack/cinder/+/973595 | 12:37 |
| opendevreview | Biser Milanov proposed openstack/cinder master: Add iSCSI export support to the StorPool driver https://review.opendev.org/c/openstack/cinder/+/847536 | 12:37 |
| opendevreview | Biser Milanov proposed openstack/cinder master: StorPool: Fix a leftover dict unpacking https://review.opendev.org/c/openstack/cinder/+/1008446 | 12:41 |
| opendevreview | Biser Milanov proposed openstack/cinder master: StorPool: Do not use the option 'storpool_replication' when creating or retyping volumes https://review.opendev.org/c/openstack/cinder/+/973592 | 12:41 |
| opendevreview | Biser Milanov proposed openstack/cinder master: StorPool: Use only baseOn when cloning a volume https://review.opendev.org/c/openstack/cinder/+/973593 | 12:41 |
| opendevreview | Biser Milanov proposed openstack/cinder master: StorPool: Add the storpool_qos option and deprecate the storpool_template one https://review.opendev.org/c/openstack/cinder/+/973594 | 12:41 |
| opendevreview | Biser Milanov proposed openstack/cinder master: StorPool: Remove the storpool_template option https://review.opendev.org/c/openstack/cinder/+/973595 | 12:41 |
| opendevreview | Biser Milanov proposed openstack/cinder master: Add iSCSI export support to the StorPool driver https://review.opendev.org/c/openstack/cinder/+/847536 | 12:41 |
| rosmaita | noonedeadpunk: i put 3 of your patches on https://etherpad.opendev.org/p/cinder-festival-of-reviews , you've got 5 open, but 2 have -1s | 13:12 |
| rosmaita | i think https://review.opendev.org/c/openstack/cinder/+/949717 is probably OK, but requires a closer look than we want for a Festival patch | 13:12 |
| rosmaita | I personally don't like https://review.opendev.org/c/openstack/cinder/+/962360 ; see my comment on the patch | 13:12 |
| noonedeadpunk | Yeah, I was going to check on 949717 one more time, 962360 I will probably even abandon | 13:14 |
| noonedeadpunk | 962909 tomorrow has 1y anniversary btw. I should prepare some biirthday cake 自 for it :) | 13:19 |
| rosmaita | reminder: Cinder Festival of Reviews today, 1400-1600 UTC | 14:00 |
| rosmaita | join us in Google meet: https://meet.google.com/ijz-ivqg-bam | 14:00 |
| rosmaita | or go directly to the etherpad: https://etherpad.opendev.org/p/cinder-festival-of-reviews | 14:00 |
| rosmaita | jungleboyj rosmaita tosky whoami-rajat m5z e0ne geguileo eharney jbernard sfernand hemna fabiooliveira happystacker sfv880 jayaanan bryanneumann agalica erlon lutimura sp-bmilanov kaisers Anoop_Shukla agalica | 14:00 |
| opendevreview | Gireesh Awasthi proposed openstack/cinder master: NetApp - ASAr2 Driver Snapshot and Cache Acceleration Support https://review.opendev.org/c/openstack/cinder/+/971174 | 14:20 |
| opendevreview | Merged openstack/cinder master: Everpure: report the backend as an explicit single pool https://review.opendev.org/c/openstack/cinder/+/1008073 | 15:45 |
| opendevreview | Merged openstack/cinder master: NetApp ONTAP REST: fix dropped SSC extra spec https://review.opendev.org/c/openstack/cinder/+/1004875 | 16:13 |
| opendevreview | Merged openstack/python-cinderclient master: Fix typo https://review.opendev.org/c/openstack/python-cinderclient/+/986728 | 16:41 |
| opendevreview | Gireesh Awasthi proposed openstack/cinder master: NetApp - ASAr2 Driver Snapshot and Cache Acceleration Support https://review.opendev.org/c/openstack/cinder/+/971174 | 16:52 |
| JayF | JFYI I set PTG etherpad using the IRC bot to https://etherpad.opendev.org/p/cinder-indri-ptg-planning | 16:56 |
| JayF | it appears to be where the actual etherpad is and we had to go find it for the ironic planning | 16:56 |
| opendevreview | Stephen Finucane proposed openstack/os-brick master: Remove *args from CacheManager https://review.opendev.org/c/openstack/os-brick/+/1007715 | 17:06 |
| opendevreview | Stephen Finucane proposed openstack/os-brick master: privsep: Convert 'os_brick.caches.opencas' https://review.opendev.org/c/openstack/os-brick/+/791275 | 17:06 |
| opendevreview | Stephen Finucane proposed openstack/os-brick master: privsep: Harden opencas utils https://review.opendev.org/c/openstack/os-brick/+/1007282 | 17:06 |
| opendevreview | Merged openstack/cinder master: Infinidat: Add support for snapshot promotion https://review.opendev.org/c/openstack/cinder/+/940236 | 17:20 |
| opendevreview | Merged openstack/cinder master: doc: Remove deprecated queue_if_no_path from Nimble multipath example https://review.opendev.org/c/openstack/cinder/+/992818 | 17:21 |
| opendevreview | Merged openstack/cinder master: Cleanup unused only_quotas path in quota_destroy_all_by_project https://review.opendev.org/c/openstack/cinder/+/1005579 | 17:21 |
| TheJulia | Regarding a ironic/cinder cross-project session, I'm proposing Tuesday at 17:15 UTC for 45 minutes. | 19:14 |
| TheJulia | And have penciled in that time on ironic's chedule. | 19:14 |
| opendevreview | Merged openstack/cinder master: Modify the spelling mistakes https://review.opendev.org/c/openstack/cinder/+/992879 | 19:31 |
| opendevreview | Merged openstack/cinder master: Nexenta: Fix re.split Deprecation warning https://review.opendev.org/c/openstack/cinder/+/984429 | 19:32 |
| opendevreview | Merged openstack/cinder master: Modify the spelling mistakes https://review.opendev.org/c/openstack/cinder/+/992878 | 19:32 |
| opendevreview | Merged openstack/cinder master: Modify the spelling mistakes https://review.opendev.org/c/openstack/cinder/+/992863 | 19:32 |
| opendevreview | Pratik Shah proposed openstack/cinder master: [Hitachi] Add volume group replication https://review.opendev.org/c/openstack/cinder/+/1008534 | 20:40 |
| opendevreview | Pratik Shah proposed openstack/cinder master: [Hitachi] Add volume group replication https://review.opendev.org/c/openstack/cinder/+/1008534 | 20:41 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!