opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Update volume delete rejection message https://review.opendev.org/c/openstack/cinder/+/826608 | 03:48 |
---|---|---|
opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Deleting a volume in 'downloading' state https://review.opendev.org/c/openstack/cinder/+/826607 | 04:14 |
opendevreview | Harsh Ailani proposed openstack/cinder master: [SVf] Fixed Detach for multi-attach volumes https://review.opendev.org/c/openstack/cinder/+/809518 | 07:22 |
*** _pewp__ is now known as _pewp_ | 07:37 | |
*** hemna6 is now known as hemna | 07:37 | |
haailani | Hi reviewers, | 15:37 |
haailani | Requesting you to please review the following patches: | 15:37 |
haailani | https://review.opendev.org/c/openstack/cinder/+/811877 | 15:37 |
haailani | https://review.opendev.org/c/openstack/cinder/+/809518 | 15:37 |
haailani | Thank you | 15:37 |
*** dviroel is now known as dviroel|lunch | 15:39 | |
opendevreview | Gorka Eguileor proposed openstack/cinder master: Remove attach and detach volume driver methods https://review.opendev.org/c/openstack/cinder/+/814898 | 15:59 |
opendevreview | Gorka Eguileor proposed openstack/cinder master: Join do_setup and check_for_setup_error https://review.opendev.org/c/openstack/cinder/+/815659 | 15:59 |
opendevreview | Gorka Eguileor proposed openstack/cinder master: Docs: Document clone_image driver method https://review.opendev.org/c/openstack/cinder/+/815660 | 15:59 |
geguileo | rosmaita: jungleboyj should we merge the Lightbits driver? It has our 3 +2s | 16:31 |
jungleboyj | geguileo: I am good with that. | 16:34 |
*** dviroel|lunch is now known as dviroel | 16:38 | |
rosmaita | geguileo: we could, but i believe it depends-on the os-brick connector | 16:44 |
* geguileo facepalms | 16:44 | |
* geguileo blames Monday | 16:45 | |
jungleboyj | rosmaita: who did we have set up to do the os-brick connector approval? | 17:03 |
jungleboyj | hemna: ? | 17:04 |
rosmaita | i think it was geguileo and hemna | 17:04 |
jungleboyj | ++ | 17:04 |
geguileo | jungleboyj: I reviewed it, but not the latest changes | 17:06 |
TusharTgite | rosmaita: could you take a look at this patch please https://review.opendev.org/c/openstack/cinder/+/804035/25 i need a one +2 on this for zuul test we need to merge this one https://review.opendev.org/c/openstack/tempest/+/825702/4 | 17:15 |
rosmaita | TusharTgite: ack | 17:20 |
opendevreview | Stephen Finucane proposed openstack/cinder master: db: Add missing foreign keys, indexes to models https://review.opendev.org/c/openstack/cinder/+/813221 | 18:17 |
opendevreview | Stephen Finucane proposed openstack/cinder master: db: Resolve additional migration-model mismatches https://review.opendev.org/c/openstack/cinder/+/813222 | 18:17 |
opendevreview | Stephen Finucane proposed openstack/cinder master: db: Add tests to ensure we keep migrations in sync https://review.opendev.org/c/openstack/cinder/+/813223 | 18:17 |
opendevreview | Stephen Finucane proposed openstack/cinder master: docs: Add whereto for testing redirect rules https://review.opendev.org/c/openstack/cinder/+/813224 | 18:17 |
opendevreview | Stephen Finucane proposed openstack/cinder master: docs: Rework and move upgrades guide https://review.opendev.org/c/openstack/cinder/+/813225 | 18:17 |
opendevreview | Stephen Finucane proposed openstack/cinder master: db: Enable auto-generation of database migrations https://review.opendev.org/c/openstack/cinder/+/813226 | 18:17 |
opendevreview | Stephen Finucane proposed openstack/cinder master: docs: Remove unnecessary 'blockstorage-' prefix https://review.opendev.org/c/openstack/cinder/+/813227 | 18:17 |
opendevreview | Stephen Finucane proposed openstack/cinder master: db: Move utility methods to top of file https://review.opendev.org/c/openstack/cinder/+/813228 | 18:17 |
opendevreview | Stephen Finucane proposed openstack/cinder master: WIP: db: Remove unnecessary sqlalchemy abstraction https://review.opendev.org/c/openstack/cinder/+/813229 | 18:17 |
*** tkajinam is now known as Guest1307 | 18:43 | |
opendevreview | Merged openstack/cinder master: Run database migration tests https://review.opendev.org/c/openstack/cinder/+/819905 | 20:43 |
*** dviroel is now known as dviroel|brb | 20:48 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!