| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: Fix to have compatibility of GAD volumes https://review.opendev.org/c/openstack/cinder/+/901318 | 01:37 |
|---|---|---|
| opendevreview | OpenStack Proposal Bot proposed openstack/cinder master: Imported Translations from Zanata https://review.opendev.org/c/openstack/cinder/+/918708 | 04:11 |
| opendevreview | Pham Le Gia Dai proposed openstack/cinder master: Fixed reading 2 or more GiB of a backup metadata file at a time https://review.opendev.org/c/openstack/cinder/+/911411 | 07:00 |
| opendevreview | Atsushi Kawai proposed openstack/cinder master: Hitachi: Support QoS https://review.opendev.org/c/openstack/cinder/+/912042 | 07:28 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: Add functions to srbac base file Reason: There are multiple patches that contains the same file and code. Now, we will remove all the code from the other patches and rebase on this patch. https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/920168 | 09:26 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: [srbac] Testing group snapshots https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/896979 | 10:21 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: [srbac] Testing group snapshots https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/896979 | 10:21 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: [srbac] Testing QOS specs https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/897003 | 10:29 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: [srbac] Testing QOS specs https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/897003 | 10:30 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: [srbac] Testing group types https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/896605 | 10:34 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: [srbac] Testing group types https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/896605 | 10:34 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: [srbac] Testing groups https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/896977 | 10:36 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: [srbac] Testing groups https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/896977 | 10:36 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: [srbac] Testing volume attachments https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/891393 | 10:39 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: [srbac] Testing volume attachments https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/891393 | 10:39 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: [srbac] Testing Volume https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/889852 | 10:45 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: [srbac] Testing Volume https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/889852 | 10:45 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: [srbac] Testing Volume Actions https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/889379 | 10:49 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: [srbac] Testing Volume Actions https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/889379 | 10:49 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: [srbac] Testing snaphost metadata https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/890926 | 10:53 |
| opendevreview | Yosi Ben Shimon proposed openstack/cinder-tempest-plugin master: [srbac] Testing snaphost metadata https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/890926 | 10:54 |
| opendevreview | Eric Harney proposed openstack/cinder master: mypy: Cover message code https://review.opendev.org/c/openstack/cinder/+/877967 | 12:27 |
| opendevreview | Brian Rosmaita proposed openstack/os-brick stable/2023.2: Silence warning when running in a container with overlayfs https://review.opendev.org/c/openstack/os-brick/+/920130 | 13:07 |
| mmalchuk | hi cores, any change to get this merged? https://review.opendev.org/c/openstack/cinder/+/909243 | 13:09 |
| mmalchuk | chance* | 13:09 |
| opendevreview | Josephine Seifert proposed openstack/cinder-specs master: Spec for the Cinder part of the Image Encryption https://review.opendev.org/c/openstack/cinder-specs/+/919499 | 13:11 |
| opendevreview | Andre Aranha proposed openstack/cinder stable/2023.1: Add fips check jobs https://review.opendev.org/c/openstack/cinder/+/885478 | 14:00 |
| 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 | 14:01 |
| jbernard | it's meeting time! | 14:01 |
| geguileo | whoami-rajat: simondodsley there is a quick fix for the problem | 14:36 |
| whoami-rajat | geguileo, great | 14:37 |
| geguileo | filters = {'id': image_volume_ids} | 14:38 |
| geguileo | if volume.cluster_name: | 14:38 |
| geguileo | filters['cluster_name'] = volume.cluster_name | 14:38 |
| geguileo | image_volumes = self.db.volume_get_all_by_host( | 14:38 |
| geguileo | context, volume['host'], filters=filters) | 14:38 |
| geguileo | whoami-rajat: simondodsley ^ | 14:38 |
| geguileo | Although, now that I think about it, that won't work, because the host part will prevent it from getting it :-( | 14:38 |
| geguileo | It needs a bit more logic in the different layers, so let me actually propose draft | 14:39 |
| simondodsley | thanks geguileo - appreciate the support | 14:39 |
| whoami-rajat | geguileo, i was thinking along the same lines, checking the cluster_name property, don't we have a method that gets all volumes by cluster name? | 14:39 |
| * whoami-rajat checking | 14:39 | |
| whoami-rajat | i don't see any method like that | 14:41 |
| whoami-rajat | adding it should be able to fix it | 14:41 |
| geguileo | whoami-rajat: I think we can use the volume_get_all method | 14:46 |
| geguileo | filters={'id': image_volume_ids} | 14:46 |
| geguileo | if volume.cluster_name: | 14:46 |
| geguileo | filters['cluster_name'] = volume.cluster_name | 14:46 |
| geguileo | else: | 14:46 |
| geguileo | filters['host'] = volume.host | 14:46 |
| geguileo | image_volumes = self.db.volume_get_all(context, filters=filters) | 14:46 |
| geguileo | I'm just not loving how there is a bunch of code being executed related to the pagination, ordering, and such | 14:47 |
| geguileo | with that solution | 14:47 |
| whoami-rajat | geguileo, that could be the reason why we separated out volume_get_all_by_host from the generic method volume_get_all https://github.com/openstack/cinder/commit/eb486867d3d39d3fc3478d7d50371683fbddd9c1 | 14:59 |
| geguileo | whoami-rajat: could be, but I really dislike it... We should have a single query for everything and make sure it does nothing when no pagination is requested... | 15:02 |
| opendevreview | Bryan Neumann proposed openstack/cinder master: Dell PowerMax: RDF consistency exempt https://review.opendev.org/c/openstack/cinder/+/915935 | 16:38 |
| opendevreview | Brian Rosmaita proposed openstack/cinder-specs master: Spec for the Cinder part of the Image Encryption https://review.opendev.org/c/openstack/cinder-specs/+/919499 | 16:58 |
| -opendevstatus- NOTICE: There will be a short Gerrit outage while we update to the latest 3.8 release in preparation for next weeks 3.9 upgrade. | 17:00 | |
| opendevreview | Sang Tran proposed openstack/cinder master: Fail to migrate, resize, evacuate VM contains volume state backing-up https://review.opendev.org/c/openstack/cinder/+/900037 | 17:02 |
| opendevreview | Merged openstack/cinder stable/2024.1: Ceph: Catch more failure conditions on volume backup https://review.opendev.org/c/openstack/cinder/+/916764 | 17:08 |
| opendevreview | Merged openstack/cinder stable/2023.1: 3PAR: Error out if vol cannot be converted to base https://review.opendev.org/c/openstack/cinder/+/882782 | 17:08 |
| opendevreview | Merged openstack/cinder stable/2023.1: Do not ignore availability_zone in backup creation https://review.opendev.org/c/openstack/cinder/+/892352 | 17:22 |
| opendevreview | Merged openstack/cinder stable/2023.2: RBD: Use "RBD" capitalization in user-facing text https://review.opendev.org/c/openstack/cinder/+/908689 | 17:22 |
| opendevreview | Merged openstack/cinder stable/2024.1: Fix broken backup_swift_service_auth=True https://review.opendev.org/c/openstack/cinder/+/919867 | 17:22 |
| opendevreview | Merged openstack/cinder stable/2023.2: Tests: Quiet Fungible invalid UUID warnings https://review.opendev.org/c/openstack/cinder/+/905540 | 17:22 |
| opendevreview | Merged openstack/cinder stable/2023.1: Fix 'cinder-backup' service when Swift with TLS enabled https://review.opendev.org/c/openstack/cinder/+/909243 | 17:59 |
| opendevreview | Merged openstack/cinder stable/2023.1: Stable-only: Remove skip-level upgrade job https://review.opendev.org/c/openstack/cinder/+/901037 | 18:20 |
| opendevreview | Merged openstack/cinder master: Tests: Fix invalid assert calls https://review.opendev.org/c/openstack/cinder/+/903503 | 18:20 |
| opendevreview | Ashley Rodriguez proposed openstack/devstack-plugin-ceph stable/2023.2: Add ingress deamon https://review.opendev.org/c/openstack/devstack-plugin-ceph/+/908940 | 19:19 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!