| *** dviroel is now known as dviroel|out | 00:59 | |
| opendevreview | Merged openstack/os-brick master: Lightbits LightOS driver https://review.opendev.org/c/openstack/os-brick/+/821603 | 02:03 |
|---|---|---|
| opendevreview | Merged openstack/cinder master: Lightbits LightOS driver https://review.opendev.org/c/openstack/cinder/+/821602 | 04:01 |
| *** _pewp__ is now known as _pewp_ | 07:37 | |
| *** hemna8 is now known as hemna | 07:38 | |
| opendevreview | Muli Ben-Yehuda proposed openstack/os-brick master: releasenotes: add Lightbits LightOS connector release note https://review.opendev.org/c/openstack/os-brick/+/827423 | 08:30 |
| opendevreview | Muli Ben-Yehuda proposed openstack/os-brick master: releasenotes: add Lightbits LightOS connector release note https://review.opendev.org/c/openstack/os-brick/+/827423 | 08:45 |
| opendevreview | Yan Tseitlin proposed openstack/cinder master: NO merge, test lightbits third party ci https://review.opendev.org/c/openstack/cinder/+/824031 | 09:53 |
| *** geguileo is now known as Guest1498 | 10:05 | |
| *** Guest1498 is now known as geguileo | 10:05 | |
| opendevreview | Lior Friedman proposed openstack/os-brick master: NVMeOF connector support multipath-enabled kernels https://review.opendev.org/c/openstack/os-brick/+/806687 | 10:41 |
| opendevreview | Merged openstack/cinder stable/wallaby: Fix detach notification https://review.opendev.org/c/openstack/cinder/+/818883 | 11:02 |
| *** dviroel|out is now known as dviroel | 11:15 | |
| opendevreview | Lior Friedman proposed openstack/os-brick master: nvmeof connector utilize replica_count https://review.opendev.org/c/openstack/os-brick/+/811717 | 11:17 |
| opendevreview | Lior Friedman proposed openstack/os-brick master: nvmeof connector check controller already connected https://review.opendev.org/c/openstack/os-brick/+/811718 | 11:21 |
| opendevreview | Merged openstack/cinder stable/xena: [Pure Storage] Add check for NVMe-FC capable array https://review.opendev.org/c/openstack/cinder/+/817090 | 11:58 |
| opendevreview | Merged openstack/cinder stable/xena: Doc: IBM flashsystem driver update https://review.opendev.org/c/openstack/cinder/+/826558 | 11:58 |
| opendevreview | Merged openstack/cinder stable/xena: Doc: Typos in Spectrum Virtulize family user guide https://review.opendev.org/c/openstack/cinder/+/826559 | 11:59 |
| yuval | Hey there is a cinder meeting today? | 12:01 |
| agarwchh | Do we have API ratelimiter for cinder ?? | 12:12 |
| rosmaita | yuval: yes, 1400 UTC: https://meetings.opendev.org/#Cinder_Team_Meeting | 13:11 |
| rosmaita | agarwchh: https://bugs.launchpad.net/cinder/+bug/1662637 | 13:13 |
| agarwchh | rosmaita: thanks, I saw this since the changeset to abandon ratelimit is abandon, the plan is to continue with ratelimiter, and enable it for v3 | 13:16 |
| rosmaita | agarwchh: well, it should already be available in v3, the issue had to do with moving the code that handles it out of the v2 directory | 13:17 |
| agarwchh | got it | 13:18 |
| rosmaita | agarwchh: here is a documentation patch you can review: https://review.opendev.org/c/openstack/cinder/+/782518 | 13:18 |
| agarwchh | thanks | 13:19 |
| opendevreview | yuval proposed openstack/os-brick master: Lightos connector - refactor disconnect volume https://review.opendev.org/c/openstack/os-brick/+/827496 | 13:27 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: docs: Add docs for 'RateLimitingMiddleware' https://review.opendev.org/c/openstack/cinder/+/782518 | 13:32 |
| rosmaita | Cinder meeting in #openstack-meeting-alt at 1400 UTC | 14:00 |
| rosmaita | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval ^^ | 14:00 |
| fabiooliveira | thanks | 14:00 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: policy.py https://review.opendev.org/c/openstack/cinder/+/775008 | 14:21 |
| opendevreview | Eric Harney proposed openstack/cinder master: Tests: Skip sleeping during netapp cmode unit tests https://review.opendev.org/c/openstack/cinder/+/827511 | 14:46 |
| rosmaita | people interested in the bug squad: while enriquetaso is away, we are declaring cinder to be bug-free | 15:02 |
| rosmaita | but, if you have filed a critical bug, might be a good idea to mention it here | 15:03 |
| hemna | can we please get reviews on this: https://review.opendev.org/c/openstack/cinder/+/784477 | 15:13 |
| hemna | my async backup rework. This is been around for ages now and I'm ready to give up on it. We have been using it in production for almost a year now | 15:13 |
| rosmaita | hemna: i will take a look ... in the meantime, could you look into the CI situation for rbd-iscsi-client? https://review.opendev.org/c/openstack/cinder/+/667108 | 15:18 |
| hemna | sure | 15:18 |
| rosmaita | e0ne: hemna's backup patch would be a good one for you to look at too: https://review.opendev.org/c/openstack/cinder/+/784477 | 15:18 |
| e0ne | rosmaita: ok, will take a look on it a bit later today | 15:19 |
| rosmaita | e0ne: thanks! | 15:19 |
| jungleboyj | hemna: I did a look through. Looks ok to me and if you have been running it ... | 15:22 |
| hemna | thank you | 15:22 |
| hemna | rosmaita looks like for some reason using ceph as a backup driver along w/ ceph-iscsi as a volume driver isn't happy | 15:34 |
| hemna | ceph-iscsi => ceph rbd | 15:34 |
| rosmaita | that's kind of weird | 15:34 |
| rosmaita | i guess since we want to test ceph-iscsi there, maybe use swift for the backup driver in these tests? | 15:36 |
| rosmaita | and then i guess file a bug to look into the other issue (about not being able to use ceph backup driver when using ceph-iscsi driver) | 15:36 |
| hemna | I'm not sure what the problem is just yet | 15:37 |
| rosmaita | hopefully just some kind of configuration issue | 15:37 |
| hemna | https://github.com/openstack/cinder/blob/master/cinder/backup/drivers/ceph.py#L994-L996 | 15:37 |
| hemna | that looks like the failure | 15:37 |
| rosmaita | i'm not clear on the reason for the raise at line 996 given lines 997-998 | 15:46 |
| hemna | looks like the ceph backup driver simply doesn't support non rbd incremental backups. | 16:03 |
| *** dviroel is now known as dviroel|lunch | 16:09 | |
| *** jbernard_ is now known as jbernard | 16:12 | |
| opendevreview | Merged openstack/os-brick master: mypy: utils.py https://review.opendev.org/c/openstack/os-brick/+/786731 | 16:42 |
| *** dviroel|lunch is now known as dviroel | 17:12 | |
| opendevreview | Walt proposed openstack/cinder master: Ensure driver is initialized before attach https://review.opendev.org/c/openstack/cinder/+/794001 | 18:42 |
| opendevreview | Merged openstack/cinder stable/wallaby: Fixed copy-on-write mode in GPFS NFS driver https://review.opendev.org/c/openstack/cinder/+/817833 | 19:48 |
| opendevreview | Merged openstack/cinder stable/wallaby: Delete attachment on remove_export failure https://review.opendev.org/c/openstack/cinder/+/818884 | 19:48 |
| opendevreview | Walt proposed openstack/cinder master: Add ceph-iscsi CI job https://review.opendev.org/c/openstack/cinder/+/667108 | 20:26 |
| hemna | rosmaita also looks like the rbd_iscsi driver fails during multiattach tests | 20:38 |
| hemna | I'm not sure that the way the ceph-iscsi is done that it can do multiattach | 20:38 |
| opendevreview | Walt proposed openstack/cinder master: [Ceph-iscsi] Disable multiattach https://review.opendev.org/c/openstack/cinder/+/827554 | 20:44 |
| opendevreview | Eric Harney proposed openstack/os-brick master: mypy: initiator https://review.opendev.org/c/openstack/os-brick/+/786732 | 20:55 |
| *** dviroel is now known as dviroel|out | 21:22 | |
| *** geguileo is now known as Guest1564 | 22:43 | |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!