| opendevreview | Merged openstack/cinder stable/ussuri: [stable-em only] move grenade jobs to experimental https://review.opendev.org/c/openstack/cinder/+/871792 | 02:40 |
|---|---|---|
| opendevreview | Konrad Gube proposed openstack/cinder master: Add the os-extend_volume_completion volume action https://review.opendev.org/c/openstack/cinder/+/873557 | 09:51 |
| opendevreview | Konrad Gube proposed openstack/cinder master: NFS: Add online extend support https://review.opendev.org/c/openstack/cinder/+/873686 | 09:51 |
| opendevreview | Konrad Gube proposed openstack/cinder master: Netapp NFS: Add online extend support https://review.opendev.org/c/openstack/cinder/+/873889 | 09:51 |
| opendevreview | Merged openstack/cinder master: Doc: Branding change for Spectrum Virtualize family https://review.opendev.org/c/openstack/cinder/+/873083 | 11:00 |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Delete volume and cloned-volume concurrently https://review.opendev.org/c/openstack/cinder/+/843309 | 11:21 |
| opendevreview | Jean Pierre Roquesalane proposed openstack/cinder master: PowerMax Driver - Fix for force flag https://review.opendev.org/c/openstack/cinder/+/797970 | 12:20 |
| whoami-rajat | jbernard, rosmaita hey, can you take a look at this release patch? https://review.opendev.org/c/openstack/releases/+/873520 | 12:26 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: Replace plain Mock() call with spec https://review.opendev.org/c/openstack/cinder/+/873905 | 12:58 |
| opendevreview | Sofia Enriquez proposed openstack/cinder master: Replace plain Mock() call with spec https://review.opendev.org/c/openstack/cinder/+/873905 | 13:06 |
| jbernard | whoami-rajat: done | 13:53 |
| whoami-rajat | jbernard, thanks | 13:53 |
| whoami-rajat | Cinder meeting in #openstack-meeting-alt at 1400 UTC | 13:59 |
| whoami-rajat | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard sfernand enriquetaso hemna fabiooliveira yuval tobias-urdin adiare happystacker dosaboy happystacker | 13:59 |
| opendevreview | Sathyanarayana R proposed openstack/cinder master: [SVf] As part of Flashcopy 2.0 adding config parameter to support volumegroup https://review.opendev.org/c/openstack/cinder/+/869302 | 15:00 |
| hemna- | anyone know off the top of their heads how the service list state is tracked ? (re cinder service-list) | 15:03 |
| hemna- | I have a cinder-scheduler that has bailed out due to an exception and yet the service is still marked as up | 15:03 |
| enriquetaso | #startmeeting cinder_bs | 15:03 |
| opendevmeet | Meeting started Wed Feb 15 15:03:59 2023 UTC and is due to finish in 60 minutes. The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:03 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:03 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 15:03 |
| enriquetaso | hello, welcome to the bug meeting | 15:04 |
| enriquetaso | #topic Failure to mount volumes with LUN ID > 255 | 15:04 |
| enriquetaso | #link https://bugs.launchpad.net/os-brick/+bug/2006960 | 15:04 |
| enriquetaso | Pure Storage by default uses peripheral addressing for LUN ID < 256 while it uses flat addressing for LUN >= 256 . | 15:05 |
| enriquetaso | At some point, we stop discovering new volumes. It looks like /sys/class/scsi_host/host13/device/session7/target13:0:0/ targets goes from 13:0:0:1 to 13:0:0:255 and after that, no new luns can be mapped to the compute server. | 15:05 |
| enriquetaso | The reason of this is explain in https://bugzilla.redhat.com/show_bug.cgi?id=1975368#c5 | 15:06 |
| enriquetaso | i think there's people working on this but not fix proposed to master yet | 15:06 |
| whoami-rajat | isn't this a OS limitation that only allows LUN IDs from 0-255 ? | 15:07 |
| enriquetaso | oh, sorry, i lost track of it | 15:09 |
| enriquetaso | yes.. okay, then i'm not sure what is the state of the bug | 15:09 |
| enriquetaso | i think it's out of cinder scope | 15:09 |
| whoami-rajat | I'm going through it, the bug also says RHEL is able to detect those LUNs but not os-brick | 15:09 |
| hemna- | https://access.redhat.com/solutions/5120951 | 15:09 |
| whoami-rajat | needs further investigation | 15:09 |
| enriquetaso | hemna-++ | 15:10 |
| enriquetaso | okay, i'll point the articule and let's see what the reporter says | 15:12 |
| opendevreview | Lukas Piwowarski proposed openstack/cinder-tempest-plugin master: Enable enforce_scope in -protection-functional job https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/873939 | 15:12 |
| whoami-rajat | geguileor, has a proposed solution which will require changes in the pure driver and os-brick code | 15:13 |
| whoami-rajat | The solution would be to add a parameter to the connection information that tells the addressing mode when a conversion is needed (defaults to no conversion), and add the code to handle different values in os-brick. | 15:13 |
| whoami-rajat | we can surely discuss it during PTG | 15:14 |
| geguileor | whoami-rajat: I think we first need to know if someone would be working on this... | 15:14 |
| geguileor | It's easy work, but special hardware may be required | 15:15 |
| whoami-rajat | ok, would be good to bring it up during PTG and see it there are any takers | 15:15 |
| enriquetaso | maybe the reporter would be interested in working on it, i'll leave a question asking happystacker | 15:15 |
| rosmaita | seems like Pure would be interested in this | 15:15 |
| enriquetaso | okay | 15:16 |
| enriquetaso | sounds good to me | 15:16 |
| opendevreview | Brian Rosmaita proposed openstack/cinder-tempest-plugin master: Get ready for tox 4 https://review.opendev.org/c/openstack/cinder-tempest-plugin/+/868319 | 15:16 |
| enriquetaso | moving on | 15:17 |
| happystacker | enriquetaso: I'm not the reporter or did I miss anything? | 15:17 |
| enriquetaso | oh, sorry about it happystacker | 15:17 |
| enriquetaso | lol | 15:18 |
| happystacker | enriquetaso: np | 15:18 |
| enriquetaso | #topic Error created new rsc_name in linstor driver | 15:18 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/2007275 | 15:18 |
| enriquetaso | Linstor driver ran into a problem with creating new volumes using the linstor driver. In the linstor logs, we see that cinder is trying to create a resource with an empty name. | 15:18 |
| enriquetaso | too much `linstor` wording in that sentence | 15:19 |
| enriquetaso | No fix proposed to master yet | 15:19 |
| happystacker | name should be different from id | 15:19 |
| happystacker | like volume-<id> | 15:20 |
| happystacker | or something similar | 15:20 |
| opendevreview | Vladislav Belogrudov proposed openstack/cinder master: Fix temp volume project id for Tatlin storage https://review.opendev.org/c/openstack/cinder/+/858293 | 15:21 |
| opendevreview | Vladislav Belogrudov proposed openstack/cinder master: Tatlin Unified driver - simplify code https://review.opendev.org/c/openstack/cinder/+/862491 | 15:21 |
| opendevreview | Vladislav Belogrudov proposed openstack/cinder master: Tatlin Unified driver - remove unused code https://review.opendev.org/c/openstack/cinder/+/862489 | 15:21 |
| enriquetaso | i've left a comment happystacker, thanks | 15:21 |
| opendevreview | Vladislav Belogrudov proposed openstack/cinder master: Tatlin unified driver - rename tatlin_api object https://review.opendev.org/c/openstack/cinder/+/862490 | 15:21 |
| whoami-rajat | they're passing volume['id'] to the _is_clean_volume_name function | 15:21 |
| whoami-rajat | i guess they should pass volume.name | 15:21 |
| opendevreview | Vladislav Belogrudov proposed openstack/cinder master: Tatlin unified driver - rename tatlin_api object https://review.opendev.org/c/openstack/cinder/+/862490 | 15:21 |
| enriquetaso | excellent | 15:24 |
| enriquetaso | i've update with this comments | 15:24 |
| enriquetaso | thanks | 15:24 |
| enriquetaso | moving on | 15:24 |
| enriquetaso | #topic NFS Encryption Clone operation is broken | 15:24 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/2007139 | 15:24 |
| enriquetaso | This bug report aims to track the NFS encryption work. Is the last bug as far as I can tell and it has a fix proposed to master already. | 15:24 |
| enriquetaso | The work for the NFS can be track using | 15:24 |
| enriquetaso | #link https://review.opendev.org/q/topic:bp%252Fnfs-volume-encryption | 15:24 |
| enriquetaso | Initial work is ready for reviews, please review after the priorities for this week: | 15:25 |
| enriquetaso | #link https://review.opendev.org/c/openstack/cinder/+/749155 | 15:25 |
| enriquetaso | OK, moving on ( I have 5 minutes left) | 15:25 |
| enriquetaso | #topic Service token does not work if user auth is token based | 15:26 |
| enriquetaso | #link https://bugs.launchpad.net/keystone/+bug/2006631 | 15:26 |
| enriquetaso | Looks like a Keystone bug so I've removed Cinder from it. | 15:26 |
| enriquetaso | rosmaita, ^ | 15:26 |
| rosmaita | i think that's correct | 15:26 |
| rosmaita | we can always get added back if keystone disagrees | 15:26 |
| happystacker | it's keystone right? | 15:26 |
| rosmaita | but it looks like maybe a configuration problems | 15:26 |
| happystacker | don't know what we can do about this one | 15:26 |
| rosmaita | happystacker: well, keystoneauth1 specifically | 15:27 |
| happystacker | correct | 15:27 |
| rosmaita | i'm not sure what the constraints are on how you configure the service users in the various config files | 15:27 |
| rosmaita | so hopefully keystone has some docs or something | 15:27 |
| enriquetaso | I'll keep an eye on it then | 15:28 |
| enriquetaso | last bug: | 15:28 |
| enriquetaso | #topic Inaccurate docstring test_boot_cloned_encrypted_volume | 15:28 |
| enriquetaso | #link https://bugs.launchpad.net/cinder-tempest-plugin/+bug/2006594 | 15:28 |
| enriquetaso | It's an error on a docstring. It's a good low-hanging-fruit for Outreachy potential intern | 15:29 |
| enriquetaso | Thank you for being so active today and joining the bug meeting! | 15:29 |
| happystacker | than you | 15:30 |
| enriquetaso | #endmeeting | 15:30 |
| opendevmeet | Meeting ended Wed Feb 15 15:30:13 2023 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 15:30 |
| opendevmeet | Minutes: https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-02-15-15.03.html | 15:30 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-02-15-15.03.txt | 15:30 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2023/cinder_bs.2023-02-15-15.03.log.html | 15:30 |
| happystacker | thank you | 15:30 |
| enriquetaso | happystacker++ | 15:30 |
| opendevreview | Jean Pierre Roquesalane proposed openstack/cinder master: WIP: PowerMax driver - documentation update https://review.opendev.org/c/openstack/cinder/+/873945 | 15:57 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: [Pure Storage] Add new array status for replication capability https://review.opendev.org/c/openstack/cinder/+/871261 | 17:33 |
| opendevreview | Merged openstack/os-brick master: Update hacking to 5.0 https://review.opendev.org/c/openstack/os-brick/+/873085 | 17:36 |
| Sathya | Hi Reviwers,can you review the patch https://review.opendev.org/c/openstack/cinder/+/869302 | 17:46 |
| Sathya | comments are addressed, Hoping to get this merged before code freeze | 17:46 |
| opendevreview | Merged openstack/cinder master: Update hacking to 5.0 series https://review.opendev.org/c/openstack/cinder/+/873406 | 17:46 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: [Pure Storage] Add new array status for replication capability https://review.opendev.org/c/openstack/cinder/+/871261 | 18:37 |
Generated by irclog2html.py 2.17.3 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!