| opendevreview | Hao Wang proposed openstack/cinder master: Time comparison filter didn't support time zone offset https://review.opendev.org/c/openstack/cinder/+/734415 | 02:11 |
|---|---|---|
| opendevreview | Hao Wang proposed openstack/cinder master: Support multiple filters together for volume time comparison filter https://review.opendev.org/c/openstack/cinder/+/740146 | 02:31 |
| opendevreview | Hao Wang proposed openstack/cinder master: Support to query snapshot filter by updated_at/created_at https://review.opendev.org/c/openstack/cinder/+/733062 | 02:51 |
| opendevreview | Hao Wang proposed openstack/cinder master: Time comparison filter didn't support time zone offset https://review.opendev.org/c/openstack/cinder/+/734415 | 03:24 |
| opendevreview | Desire Barine proposed openstack/cinder master: HPE: Fix keyerror seen during volume migration https://review.opendev.org/c/openstack/cinder/+/877583 | 06:56 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: Test srbac on user_messages https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/876730 | 08:56 |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: [coordination] backend_url should be secret https://review.opendev.org/c/openstack/cinder/+/877934 | 09:32 |
| opendevreview | Takashi Kajinami proposed openstack/cinder master: [coordination] backend_url should be secret https://review.opendev.org/c/openstack/cinder/+/877934 | 09:34 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Include volume type constraints in internal API https://review.opendev.org/c/openstack/cinder/+/846146 | 11:29 |
| opendevreview | Raghavendra Tilay proposed openstack/cinder master: DNM: Create for temporary test - Please ignore https://review.opendev.org/c/openstack/cinder/+/875643 | 12:24 |
| akhodos | Hi everyone! I am developing a new cinder driver, but our backend does not support snapshots at the moment. Is it possible to get merged without snapshot feature implemented or is that a hard requirement? | 14:38 |
| jbernard | akhodos: creation and deletion of snapshots is considered required functionality for acceptance | 14:48 |
| jbernard | akhodos: see https://docs.openstack.org/cinder/latest/reference/support-matrix.html#driver-support-matrix | 14:48 |
| jbernard | akhodos: if snapshot ability is in progress you could certainly develop your driver in parallel and submit it once things have landed | 14:50 |
| akhodos | it is still uncertain when snapshots will be introduced, so I was wondering if getting merged without snapshots is a possibility at all | 14:52 |
| jbernard | akhodos: the way I read the document, it seems like a requirment - but others may chime in, | 14:54 |
| jbernard | whoami-rajat: ^ curious | 14:54 |
| eharney | akhodos: can you implement snapshot support by using volume clones on the backend? | 14:56 |
| akhodos | eharney: no, that is also no possible | 14:57 |
| eharney | akhodos: does it support volume clones at all? | 14:57 |
| akhodos | no | 14:58 |
| opendevreview | Eric Harney proposed openstack/cinder master: Message API: Handle empty message_action from context https://review.opendev.org/c/openstack/cinder/+/877966 | 15:06 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: Cover message code https://review.opendev.org/c/openstack/cinder/+/877967 | 15:06 |
| eharney | rosmaita: i think i sorted out that message bug you guys were asking about last week ^ | 15:08 |
| opendevreview | Toheeb Oyekola proposed openstack/cinder-tempest-plugin master: Fix test_boot_cloned_encrypted_volume docstring https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/876905 | 15:13 |
| opendevreview | Toheeb Oyekola proposed openstack/cinder-tempest-plugin master: Fix test_boot_cloned_encrypted_volume docstring https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/876905 | 15:14 |
| rosmaita | eharney: i thought you were going to go a different direction with that, not allow the action to be an optional, and then figure out who's not setting it (on the theory that it never makes sense to not specify the action when creating a user message) | 15:15 |
| eharney | rosmaita: the issue is, since it's read from the context, i don't think we are going to guarantee that the context always had an associated action in it when we hit these errors and want to generate messages | 15:15 |
| eharney | (since the action is read from the context) | 15:16 |
| rosmaita | eharney: yeah, though the context support was added fairly recently, i wish we could figure out a way to make sure that if you're using the context to pass user message info, that an action has been specified | 15:21 |
| rosmaita | or maybe i'm misunderstanding what's going on here | 15:21 |
| opendevreview | Toheeb Oyekola proposed openstack/cinder-tempest-plugin master: Fix test_boot_cloned_encrypted_volume docstring https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/876905 | 15:22 |
| opendevreview | Toheeb Oyekola proposed openstack/cinder-tempest-plugin master: Fix test_boot_cloned_encrypted_volume docstring https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/876905 | 15:22 |
| eharney | rosmaita: i think the point of confusion (for me now at least) is whether it's valid to send messages when there is no action specified. i.e. when we send messages, are we supposed to assume that an action was already set? | 15:24 |
| rosmaita | eharney: i thought so, but maybe i'm confusing it with another field | 15:25 |
| rosmaita | maybe we can look at this for the PTG | 15:26 |
| eharney | even if we are supposed to do that, i doubt our code actually does | 15:26 |
| eharney | yeah | 15:26 |
| opendevreview | Peter Penchev proposed openstack/cinder master: StorPool: create_cloned_volume() improvements https://review.opendev.org/c/openstack/cinder/+/843277 | 16:06 |
| opendevreview | Peter Penchev proposed openstack/cinder master: Add iSCSI export support to the StorPool driver https://review.opendev.org/c/openstack/cinder/+/847536 | 16:06 |
| eharney | has anyone been looking at cinder.tests.unit.db.test_migrations.TestModelsSyncSQLite.test_models_sync failures? | 16:10 |
| opendevreview | Eric Harney proposed openstack/cinder master: Bump mypy to 1.1.1 https://review.opendev.org/c/openstack/cinder/+/877981 | 16:18 |
| opendevreview | Merged openstack/cinder master: Yadro tatlin_client: Fix bad message formatting https://review.opendev.org/c/openstack/cinder/+/871518 | 17:11 |
| *** User_ is now known as Toheeb | 17:21 | |
| opendevreview | Merged openstack/cinder master: RemoteFS: Fix messy string formatting https://review.opendev.org/c/openstack/cinder/+/876987 | 20:54 |
| opendevreview | Damian Pietras proposed openstack/cinder master: LVM-nvmet: persist configuration after changing it https://review.opendev.org/c/openstack/cinder/+/878031 | 21:44 |
| opendevreview | Toheeb Oyekola proposed openstack/cinder-tempest-plugin master: Fix test_boot_cloned_encrypted_volume docstring https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/876905 | 22:28 |
| opendevreview | Peter Penchev proposed openstack/cinder master: Add iSCSI export support to the StorPool driver https://review.opendev.org/c/openstack/cinder/+/847536 | 22:50 |
| Roamer` | ^^ so now the StorPool iSCSI export implementation has more lines in unit tests than it does in the actual driver change... as it should have been from the start :) | 22:59 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: [Pure Storage] Add NVMe-TCP transport type to NVMe driver https://review.opendev.org/c/openstack/cinder/+/877918 | 23:59 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!