| opendevreview | Amit Uniyal proposed openstack/cinder master: Add CI jobs to run unit tests with native threading (no-eventlet) https://review.opendev.org/c/openstack/cinder/+/988746 | 03:52 |
|---|---|---|
| opendevreview | Amit Uniyal proposed openstack/cinder master: Add CI jobs to run functional tests with native threading (no-eventlet) https://review.opendev.org/c/openstack/cinder/+/988747 | 03:52 |
| opendevreview | Amit Uniyal proposed openstack/cinder master: Adds threading mode tests for semaphore funcs https://review.opendev.org/c/openstack/cinder/+/990681 | 03:52 |
| opendevreview | Amit Uniyal proposed openstack/cinder master: Add CI job to run cinder-tempest-plugin tests with native threading (no-eventlet) https://review.opendev.org/c/openstack/cinder/+/991065 | 03:52 |
| opendevreview | Sailaja Mada proposed openstack/cinder master: Fix for async failover failure https://review.opendev.org/c/openstack/cinder/+/991534 | 05:40 |
| opendevreview | Jyotsna Lothe proposed openstack/cinder master: New cinder driver for HPE Alletra MP https://review.opendev.org/c/openstack/cinder/+/969373 | 06:28 |
| opendevreview | Rutuja Pawar proposed openstack/cinder master: IBM-SVF: Fix force detach volume bug https://review.opendev.org/c/openstack/cinder/+/984538 | 09:44 |
| *** Rutuja is now known as RutujaPawar | 10:30 | |
| opendevreview | Sailaja Mada proposed openstack/cinder master: Fix for async failover failure https://review.opendev.org/c/openstack/cinder/+/991534 | 10:42 |
| opendevreview | soheil proposed openstack/cinder master: Fix: avoid automatic checksum headers breaking S3-compatible backups https://review.opendev.org/c/openstack/cinder/+/977761 | 11:26 |
| opendevreview | Rutuja Pawar proposed openstack/cinder master: IBM-SVF: Fix force detach volume bug https://review.opendev.org/c/openstack/cinder/+/984538 | 12:20 |
| rosmaita | gmaan: left a question for you on https://review.opendev.org/c/openstack/cinder/+/991434 when you have a few minutes, no rush | 12:32 |
| opendevreview | Merged openstack/cinder stable/2025.1: Fix tempest-integrated-storage-ubuntu-jammy https://review.opendev.org/c/openstack/cinder/+/991434 | 14:26 |
| opendevreview | Eric Harney proposed openstack/cinder master: Remove eventlet import from backup API https://review.opendev.org/c/openstack/cinder/+/991674 | 14:59 |
| stephenfin | jbernard, eharney: rosmaita asked me to rebase these old patches a few weeks back. Would either of you be able to take a look and (hopefully) add the second +2/+W before they go back into merge conflict (like the last one already has)? https://review.opendev.org/c/openstack/cinder/+/837543 | 15:08 |
| jbernard | stephenfin: looking | 15:08 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: objects: Remove aliases of objects https://review.opendev.org/c/openstack/cinder/+/988246 | 15:18 |
| opendevreview | Stephen Finucane proposed openstack/cinder master: typing: "Import" objects https://review.opendev.org/c/openstack/cinder/+/985090 | 15:18 |
| stephenfin | jbernard++ ty! | 15:41 |
| gmaan | rosmaita: replied. I will say if nobody is fixing it then remove it. it is not mandatory to test the upgrade from the unmaintained releases. but I left elodilles to comment on that as he is the one making effort to keep them green | 15:53 |
| rosmaita | gmaan: thanks! | 15:53 |
| opendevreview | Jon Bernard proposed openstack/cinder master: Add a non-voting job to test eventlet removal https://review.opendev.org/c/openstack/cinder/+/982144 | 16:33 |
| opendevreview | Merged openstack/cinder master: db: Move helpers to cinder.db.sqlalchemy.api https://review.opendev.org/c/openstack/cinder/+/837543 | 18:20 |
| opendevreview | Merged openstack/cinder master: db: Use cinder.db.api directly https://review.opendev.org/c/openstack/cinder/+/846173 | 18:20 |
| opendevreview | Rajat Dhasmana proposed openstack/devstack-plugin-ceph master: Test multi ceph + replication configuration https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/952107 | 18:50 |
| opendevreview | Rajat Dhasmana proposed openstack/devstack-plugin-ceph master: Test multi ceph configuration https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/952107 | 18:51 |
| opendevreview | Brian Rosmaita proposed openstack/cinder-tempest-plugin master: Drop python 3.10 support https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/991733 | 20:11 |
| *** elodilles is now known as elodilles_OoO | 20:17 | |
| jbernard | rosmaita: have you ever seen a TypeError on catching a class that doesn't inherit from BaseException? | 20:41 |
| eharney | jbernard: i remember this one - https://review.opendev.org/c/openstack/nova/+/852795 | 20:49 |
| eharney | catching an instance instead of a class | 20:49 |
| jbernard | eharney: im seeing one on line 348 of ceph.py (self.rados.Error) | 20:51 |
| eharney | when running unit tests or actually running the code? | 20:52 |
| jbernard | the except statement looks well formed, and rados.Error inherits from Exception… | 20:52 |
| jbernard | running the tests | 20:52 |
| eharney | because mocking may be a little weird w/ rados since it isn't a pip installable library | 20:52 |
| jbernard | hmm | 20:52 |
| eharney | i would guess it's related to that | 20:52 |
| jbernard | i was thinking something like that | 20:52 |
| jbernard | ill make a note to look at that later | 20:53 |
| jbernard | the test would fail anyway, but it's not supposed to fail like that | 20:54 |
| jbernard | i might be wrong about that | 20:54 |
| opendevreview | Eric Harney proposed openstack/devstack-plugin-ceph master: Enable scheduled backend trash purging https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/919539 | 21:01 |
| opendevreview | Merged openstack/cinder-tempest-plugin master: Remove 2024.2 job https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/987032 | 21:46 |
| opendevreview | Merged openstack/devstack-plugin-ceph master: Add support to deploy two ceph clusters https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/951810 | 22:22 |
| opendevreview | Merged openstack/devstack-plugin-ceph master: Add support for ceph replication https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/953042 | 22:44 |
Generated by irclog2html.py 4.1.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!