| *** vhari_ is now known as vhari | 00:47 | |
| harsh | Hi everyone. I am getting an error message while triggering Zuul on a patch : https://review.opendev.org/c/openstack/cinder/+/926844 | 09:05 |
|---|---|---|
| harsh | ERROR MESSAGE: Error 409 (Conflict): Change 926844 may not exceed 1000 updates. It may still be abandoned, submitted and you can add/remove reviewers to/from the attention-set. To continue working on this change, recreate it with a new Change-Id, then abandon this one. | 09:06 |
| harsh | I checked about this error and the solution is either get this merged or create a new review. If i create a new review all the review history is lost. | 09:06 |
| harsh | But as i have addressed all the comments and there was a +2 from Jon, could you please merge this patch ? | 09:07 |
| harsh | This is affecting other linked patches as well. | 09:07 |
| harsh | rosmaita : jbernard : if you are around, could you please suggest on the above situation ? | 10:20 |
| f0o | Any chance anyone can have a look at https://review.opendev.org/c/openstack/cinder/+/971586 ? | 11:22 |
| rosmaita | harsh: i think you will have to do what the error message suggests, that is, push patchset 94 on https://review.opendev.org/c/openstack/cinder/+/926844 as a new change | 13:18 |
| rosmaita | put the first comment on the new review as "Continuation of change Idcb96f37846d0e9e69330dc3901c69f00dabceba , which reached the maximum number of updates allowed by gerrit" | 13:18 |
| rosmaita | then abandon the old review with the comment "This review has reached the maximum number of updates allowed by gerrit, discussion continues as change <change_id of new review>" | 13:18 |
| rosmaita | harsh: ^^ | 13:19 |
| jbernard | harsh: it's not lost, it's just split between two subimssions | 13:41 |
| jbernard | harsh: have you really reached 1000 updates? that | 13:42 |
| jbernard | seems like a lot, maybe a bug in gerrit? | 13:42 |
| jbernard | either way, what brian suggests will work | 13:43 |
| raghavendrat | hi tkajinam: erlon: eharney: the review comments have been addressed on below patch. | 14:26 |
| raghavendrat | New cinder driver for HPE Alletra MP | 14:27 |
| raghavendrat | https://review.opendev.org/c/openstack/cinder/+/969373 | 14:27 |
| raghavendrat | May i request your help to please review. Thanks. | 14:27 |
| opendevreview | yuval proposed openstack/cinder master: lightos: Early exit in terminate_connection with retry logic https://review.opendev.org/c/openstack/cinder/+/975414 | 16:02 |
| harsh | jbernard: no i haven't reached 1000 updates.. :D there are many linked patches on top of this patch. so everytime anyone of them is rebased, the whole stack is rebased. | 16:04 |
| opendevreview | Florent Le Lain proposed openstack/cinder master: Fix encryption key cleanup when volume creation fails https://review.opendev.org/c/openstack/cinder/+/975304 | 16:14 |
| opendevreview | Eric Harney proposed openstack/cinder master: RemoteFS/NFS: Fix backing_file parameter type https://review.opendev.org/c/openstack/cinder/+/953239 | 16:18 |
| opendevreview | Eric Harney proposed openstack/cinder master: WIP: API: Allow newlines in snapshot descriptions https://review.opendev.org/c/openstack/cinder/+/926914 | 16:34 |
| dan_with | Two questions for the Cinder team: | 17:03 |
| dan_with | 1) If slow BFV builds are happening, can this be due to Glance being rate limited while laying down an image because of volume type QoS policies? If so, how does an OpenStack admin exempt Glance from volume type QoS policies? | 17:03 |
| dan_with | 2) Currently, I have some BF server builds failing when the VM is being built from a server (VM) snapshot. If the min_disk of the image used to create the source VM (which the server snapshot has been created from) is exactly the same size as the specified Cinder volume for the New (target) BFV VM being built, the build will fail. Basically, the snapshot needs an extra 200MiB up to 1GiB to decompress inside the target | 17:03 |
| dan_with | volume. Is this due to the decompression operation or LUKS encryption headers? I have a patch for our Cinder driver that adds up to 1 GiB, but this seems like something that should have been encountered prior and could be mitigated with a config option? What's the best way to deal with this? | 17:03 |
| dan_with | Steps to reproduce item 2: | 17:04 |
| dan_with | 1. Take a Snapshot from a VM or use a Custom Image | 17:04 |
| dan_with | 2. Try to build a new VM from the Snapshot/Image and use the min_disk value as the volume size. | 17:04 |
| dan_with | 3. It will fail. | 17:04 |
| dan_with | 4. Try again and increase the volume size by 1GB more than the min_disk | 17:04 |
| dan_with | 5. It should succeed. | 17:04 |
| dan_with | Thanks!!! | 17:04 |
| opendevreview | Raghavendra Tilay proposed openstack/cinder master: New cinder driver for HPE Alletra MP https://review.opendev.org/c/openstack/cinder/+/969373 | 17:06 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: cinder/api/v3/volume_metadata.py https://review.opendev.org/c/openstack/cinder/+/878534 | 17:15 |
| opendevreview | Eric Harney proposed openstack/cinder-tempest-plugin master: DNM: Test devstack fix 975247 https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/975429 | 17:19 |
| opendevreview | Eric Harney proposed openstack/cinder master: Bump flake8-import-order to >=0.19.2 https://review.opendev.org/c/openstack/cinder/+/975439 | 18:00 |
| opendevreview | Eric Harney proposed openstack/cinder master: Bump hacking to 8.0.0 https://review.opendev.org/c/openstack/cinder/+/975441 | 18:06 |
| opendevreview | Eric Harney proposed openstack/os-brick master: Bump hacking to 8.0.0 https://review.opendev.org/c/openstack/os-brick/+/975442 | 18:08 |
| opendevreview | Merged openstack/cinder master: Prevent database errors when quota fields overflow https://review.opendev.org/c/openstack/cinder/+/972476 | 18:18 |
| opendevreview | Merged openstack/cinder stable/2025.2: stable: Remove some non-voting jobs from stable checks https://review.opendev.org/c/openstack/cinder/+/964700 | 18:39 |
| opendevreview | Zachary Mark Raines proposed openstack/cinder stable/2025.2: Add 512e/4k disk geometry configuration https://review.opendev.org/c/openstack/cinder/+/975457 | 19:32 |
| opendevreview | Ade Lee proposed openstack/cinder master: Add region_name support for Glance and Swift backup driver https://review.opendev.org/c/openstack/cinder/+/973447 | 20:40 |
| opendevreview | Ade Lee proposed openstack/cinder master: Add region_name support for Barbican key migration https://review.opendev.org/c/openstack/cinder/+/973448 | 20:40 |
| opendevreview | Eric Harney proposed openstack/os-brick master: Doc: Update README version references https://review.opendev.org/c/openstack/os-brick/+/975470 | 20:45 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!