| *** mhen_ is now known as mhen | 02:42 | |
| opendevreview | zhou zhong proposed openstack/cinder master: Make sure dstpath exists when exec BlkioCgroup throttling https://review.opendev.org/c/openstack/cinder/+/934198 | 03:02 |
|---|---|---|
| opendevreview | zhou zhong proposed openstack/cinder master: Make sure dstpath exists when exec BlkioCgroup throttling https://review.opendev.org/c/openstack/cinder/+/934198 | 03:11 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: [Pure Storage] Add volume group support https://review.opendev.org/c/openstack/cinder/+/933675 | 07:02 |
| opendevreview | Yian Zong proposed openstack/cinder master: Dell PowerStore: support Metro volumes Added metro volume support in Dell PowerStore driver. https://review.opendev.org/c/openstack/cinder/+/933628 | 07:29 |
| opendevreview | jayaanand borra proposed openstack/cinder master: NVMe namespace mapping fails during VM migration with "Namespace is already mapped to subsystem" https://review.opendev.org/c/openstack/cinder/+/928486 | 08:02 |
| opendevreview | Yian Zong proposed openstack/cinder master: Dell PowerStore: support Metro volumes https://review.opendev.org/c/openstack/cinder/+/933628 | 08:36 |
| opendevreview | Biser Milanov proposed openstack/os-brick master: StorPool: Do not use packages `storpool` and `storpool.spopenstack` https://review.opendev.org/c/openstack/os-brick/+/932176 | 09:49 |
| opendevreview | Biser Milanov proposed openstack/cinder master: StorPool: Do not use packages `storpool` and `storpool.spopenstack` https://review.opendev.org/c/openstack/cinder/+/933078 | 09:52 |
| *** geguileo is now known as Guest8608 | 09:59 | |
| opendevreview | Si Snow proposed openstack/cinder master: Fix concurrent creation of encrypted volumes by multiple users occasionally fails https://review.opendev.org/c/openstack/cinder/+/934223 | 10:40 |
| opendevreview | Si Snow proposed openstack/cinder master: Fix concurrent creation of encrypted volumes by multiple users occasionally fails https://review.opendev.org/c/openstack/cinder/+/934223 | 11:08 |
| zigo | eharney: Hi there! Is there anything holding https://review.opendev.org/c/openstack/cinder/+/930682 from being merged? | 13:23 |
| zigo | It fixes a real very annoying bug ... | 13:23 |
| zigo | lanson ^ | 13:24 |
| zigo | rosmaita: ^ | 13:24 |
| rosmaita | zigo: thanks for the pointer, will take a look | 13:26 |
| zigo | Looks like I've just hit it when running Tempest on Antelope (I'm currently testing upgrades ...). | 13:27 |
| zigo | So this patch is a good target for backporting up to Antelope, it seems. | 13:27 |
| zigo | I'll let you know when I have tested that. | 13:27 |
| lanson | that's weird, I think the bug was introduced in bobcat. Looking at the antelope code, the memoryview cast had not been made yet ? | 13:29 |
| zigo | lanson: Maybe on the latest point release of Antelope? | 13:29 |
| zigo | Building with your patch in Anotelope, I have a unit test failure: | 13:30 |
| zigo | File "/<<PKGBUILDDIR>>/cinder/backup/chunkeddriver.py", line 78, in _write_nonzero | 13:30 |
| zigo | volume_file.write(chunk.tobytes()) | 13:30 |
| zigo | ^^^^^^^^^^^^^ | 13:30 |
| zigo | AttributeError: 'bytes' object has no attribute 'tobytes' | 13:30 |
| zigo | So probably that's something else? | 13:30 |
| lanson | oh no that's side-effect of the patch being backported I believe | 13:30 |
| zigo | Can you help me backporting it? | 13:30 |
| lanson | running tobytes on a non memoryview casted var | 13:30 |
| lanson | yessir | 13:31 |
| zigo | What hinted me that we still have the issue in Antelope is the below Tempest failures: | 13:31 |
| zigo | {4} tempest.api.volume.admin.test_volumes_backup.VolumesBackupsAdminTest.test_volume_backup_export_import [1523.929807s] ... FAILED | 13:31 |
| zigo | {3} tempest.api.volume.test_volumes_backup.VolumesBackupsTest.test_volume_backup_create_get_detailed_list_restore_delete [630.199587s] ... FAILED | 13:31 |
| zigo | {5} cinder_tempest_plugin.api.volume.test_volume_backup.VolumesBackupsTest.test_backup_create_and_restore_to_an_existing_volume [624.427931s] ... FAILED | 13:31 |
| zigo | {5} cinder_tempest_plugin.api.volume.test_volume_backup.VolumesBackupsTest.test_incr_backup_create_and_restore_to_an_existing_volume [934.137877s] ... FAILED | 13:31 |
| zigo | Ah, shit, also failing in Bobcat. | 13:32 |
| lanson | in bobcat/antelope, the restored chunk is not casted to memoryview, so it's not necessary to apply tobytes to read it, it already passes a bytestring to the rbd thingy | 13:34 |
| zigo | Ok, so that's another bug then. :/ | 13:35 |
| zigo | I get some "backup <UUID> failed to reach available status (current restoring) within the required time (300 s)." | 13:35 |
| lanson | File "/<<PKGBUILDDIR>>/cinder/backup/chunkeddriver.py", line 78, in _write_nonzero | 13:36 |
| lanson | volume_file.write(chunk.tobytes()) | 13:36 |
| lanson | this looks exactly like what my patch does. is it possible that this patch was backported to antelope/bobcat debian packages, even tho it didnt need to ? | 13:36 |
| zigo | Nope. I tried to do it, and then got a unit test failure. | 13:36 |
| zigo | So it's my bad, and it's probably unrelated. | 13:36 |
| zigo | Maybe just our Infomaniak CI env being too slow. | 13:37 |
| zigo | I'll try increasing the timeout and run it alone. | 13:37 |
| whoami-rajat | jbernard, rosmaita do we have cinder meeting today or did i miss something ... | 14:04 |
| jungleboyj | Wondering if the time change has caused issues. :-) | 14:04 |
| whoami-rajat | ah right | 14:04 |
| rosmaita | i suspect that's the issue | 14:04 |
| jungleboyj | lol | 14:04 |
| jungleboyj | I have been off all week. | 14:05 |
| sp-bmilanov | hi all, so that means we are having the meet in ~55minutes? | 14:06 |
| rosmaita | jungleboyj: can you hit https://review.opendev.org/c/openstack/cinder/+/932636 again? looks like a rebase wiped out your +2 | 14:08 |
| rosmaita | sp-bmilanov: probably ... hang on | 14:08 |
| jungleboyj | rosmaita: Done. | 14:09 |
| rosmaita | ty | 14:09 |
| jungleboyj | Anytime. | 14:09 |
| rosmaita | whoami-rajat: jungleboyj: sp-bmilanov: are you all OK with having the meeting at 1500 utc today (i.e., in 49 minutes from now)? | 14:11 |
| jungleboyj | Sure. | 14:11 |
| rosmaita | ok, i'll put a note on the agenda | 14:13 |
| sp-bmilanov | rosmaita: yep | 14:19 |
| rosmaita | sp-bmilanov: great, see you at 1500 | 14:20 |
| jbernard | yes, sorry, time change | 14:26 |
| jbernard | 9am, whenever we decide it is, is not possible for me | 14:26 |
| mmalchuk | folks, is there any chance to get this https://review.opendev.org/c/openstack/cinder/+/909245 merged? #openstack-unmaintained silently ignores messages | 14:32 |
| jbernard | mmalchuk: you'll need to ask an unmaintained core for that one, it's a different set of reviewers (!= cinder core) | 14:33 |
| mmalchuk | understand, and ping almost every week last month | 14:34 |
| jbernard | mmalchuk: i believe ironic recnetly made a change to include cores in unmaintained core, maybe we should consider the same, could be a topic for the meeting in ~20 minutes | 14:41 |
| mmalchuk | sure | 14:42 |
| manudinesh | #help | 14:55 |
| jbernard | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney jbernard hemna fabiooliveira yuval tobias-urdin adiare happystacker dosaboy hillpd msaravan sp-bmilanov Luzi: meeting in #openstack-meeting-alt | 15:00 |
| manudinesh | Hi jbernard, I had requested for some comments on bug : https://bugs.launchpad.net/cinder/+bug/2084117 | 15:01 |
| manudinesh | Is there anyway it can be discussed in this meeting, Thanks | 15:02 |
| jbernard | manudinesh: sure, you can either add a topic to agenda | 15:02 |
| jbernard | https://etherpad.opendev.org/p/cinder-epoxy-meetings | 15:02 |
| jbernard | or raise it in the open dicussion segment | 15:02 |
| jbernard | i cannot promise you'll here what you're hoping for, but it's the best place to raise an issue to the group :) | 15:03 |
| manudinesh | Thank you, I had added it in etherpad mentioned for cinder_bs agenda ...Let me compare if the link you shared is same where i added it earlier.. | 15:04 |
| jbernard | manudinesh: join the #openstack-meeting-alt channel | 15:05 |
| jbernard | manudinesh: we're about to start this week's meeting | 15:05 |
| manudinesh | sure, Thanks .. | 15:05 |
| manudinesh | I have added my discussion point here earlier : https://etherpad.opendev.org/p/cinder-bug-squad-meeting | 15:05 |
| opendevreview | Takashi Natsume proposed openstack/cinder master: Replace deprecated datetime.utcfromtimestamp() https://review.opendev.org/c/openstack/cinder/+/931005 | 15:26 |
| ganso | hi folks, could you please review this 1-liner patch when you have a minute to spare? thanks in advance! https://review.opendev.org/c/openstack/cinder/+/933124 | 15:32 |
| ganso | jbernard, eharney, whoami-rajat, rosmaita ^ | 15:33 |
| rosmaita | looks like requirements-check job workaround has just merged, so probably OK to just recheck: https://review.opendev.org/c/openstack/requirements/+/934249 | 15:56 |
| opendevreview | Eric Harney proposed openstack/cinder master: WIP: Use keyword-only parameters for image_utils.upload_volume https://review.opendev.org/c/openstack/cinder/+/934257 | 16:41 |
| opendevreview | Dan Smith proposed openstack/cinder master: WIP: Handle glance disk_format=gpt images https://review.opendev.org/c/openstack/cinder/+/934261 | 17:42 |
| eharney | rosmaita: fyi that change hasn't actually merged yet (the failure showed up again) | 17:54 |
| opendevreview | Merged openstack/cinder master: Always upload image volumes in a native thread https://review.opendev.org/c/openstack/cinder/+/934086 | 19:38 |
| opendevreview | Ghanshyam proposed openstack/cinder-tempest-plugin master: Remove stable/2023.1 job from master gate https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/934275 | 19:50 |
| opendevreview | Rodrigo Barbieri proposed openstack/cinder master: Fix "signature_verified" metadata propagation to images https://review.opendev.org/c/openstack/cinder/+/933124 | 20:12 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!