| opendevreview | Xuan Yandong proposed openstack/cinder-specs master: Support revert any snapshot to the volume https://review.opendev.org/c/openstack/cinder-specs/+/736111 | 03:21 |
|---|---|---|
| *** abhishekk is now known as akekane|home | 05:26 | |
| *** akekane|home is now known as abhishekk | 05:26 | |
| opendevreview | Tushar Trambak Gite proposed openstack/cinder master: Dropping explicit unicode literal https://review.opendev.org/c/openstack/cinder/+/799631 | 07:39 |
| *** bhagyashris_ is now known as bhagyashris|ruck | 08:37 | |
| *** whoami-rajat is now known as Guest89 | 09:05 | |
| *** whoami-rajat__ is now known as whoami-rajat | 09:05 | |
| opendevreview | Helen Walsh proposed openstack/cinder master: PowerMax Driver - Improve error handling around deletes https://review.opendev.org/c/openstack/cinder/+/796286 | 11:18 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add user messages for backup operations https://review.opendev.org/c/openstack/cinder/+/786627 | 12:02 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add user message for revert to snapshot https://review.opendev.org/c/openstack/cinder/+/792209 | 12:02 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add user messages for volume operations https://review.opendev.org/c/openstack/cinder/+/793515 | 12:02 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add user messages for backup operations https://review.opendev.org/c/openstack/cinder/+/786627 | 12:07 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add user message for revert to snapshot https://review.opendev.org/c/openstack/cinder/+/792209 | 12:08 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: Add user messages for volume operations https://review.opendev.org/c/openstack/cinder/+/793515 | 12:08 |
| opendevreview | Rajat Dhasmana proposed openstack/cinder master: WIP: Add user messages for volume group https://review.opendev.org/c/openstack/cinder/+/799799 | 12:25 |
| opendevreview | Walt proposed openstack/cinder master: Fix message confusion during backup restore https://review.opendev.org/c/openstack/cinder/+/799821 | 13:16 |
| hemna | I'm seeing a case where I'm having initialize_connection getting called when the volume is in creating status. (this is during restore from backup) | 13:18 |
| hemna | I can't seem to find where that exists in code | 13:18 |
| hemna | also fwiw I think I found a minor issue with the cinder create --backup-id <id> flow | 13:34 |
| hemna | the task flow manager tries to use backend assisted restore from backup, which almost no drivers implement, and then falls back to creating a raw volume, then calls backup to restore. | 13:35 |
| hemna | the volume is left in 'creating' status during the rpc call to the backup manager | 13:35 |
| hemna | I think that should be set to restoring backup | 13:35 |
| hemna | since the volume has been created on the backend correctly, and is now going to be restored | 13:36 |
| hemna | right here: https://github.com/openstack/cinder/blob/master/cinder/volume/flows/manager/create_volume.py#L1059 | 13:36 |
| rosmaita | Courtesy reminder: Cinder meeting in #openstack-meeting-alt at 1400 UTC | 13:59 |
| rosmaita | jungleboyj rosmaita smcginnis tosky whoami-rajat m5z e0ne geguileo eharney walshh_ jbernard lseki sfernand rajinir enriquetaso hemna ^^ | 13:59 |
| *** bhagyashris is now known as bhagyashris|ruck | 13:59 | |
| whoami-rajat | thanks rosmaita | 14:00 |
| geguileo | rosmaita: thanks | 14:00 |
| enriquetaso | thanks | 14:00 |
| rosmaita | happy to be of service! | 14:00 |
| *** bhagyashris_ is now known as bhagyashris|ruck | 14:24 | |
| opendevreview | Gorka Eguileor proposed openstack/cinder master: NFS: Update connection info on online snap create https://review.opendev.org/c/openstack/cinder/+/799843 | 14:27 |
| geguileo | lyarwood: Thanks for your help with the NFS case!! I have proposed the fix to cinder https://review.opendev.org/c/openstack/cinder/+/799843 | 14:50 |
| lyarwood | geguileo: ack np thanks, I'll review after my next call | 14:58 |
| geguileo | lyarwood: thanks :-) | 14:58 |
| geguileo | eharney: I was wondering if we should add a user message | 15:02 |
| eharney | geguileo: https://review.opendev.org/c/openstack/cinder/+/789602 causes volume creation to throw sensible errors instead of nasty errors at the volume service when an encryption spec was not created in a correct format | 15:02 |
| eharney | geguileo: don't need one, because the second patch will reject invalid encryption specs at the API | 15:02 |
| enriquetaso | are you going to keep the discussion or should I start the bug meeting? | 15:03 |
| enriquetaso | oh, i'll wait :) | 15:03 |
| geguileo | eharney: great | 15:03 |
| geguileo | enriquetaso: we just finished, sorry | 15:03 |
| enriquetaso | don't worry, it was a complex topic | 15:05 |
| enriquetaso | OK, i'll start: | 15:05 |
| enriquetaso | #startmeeting cinder_bs | 15:06 |
| opendevmeet | Meeting started Wed Jul 7 15:06:02 2021 UTC and is due to finish in 60 minutes. The chair is enriquetaso. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:06 |
| opendevmeet | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:06 |
| opendevmeet | The meeting name has been set to 'cinder_bs' | 15:06 |
| enriquetaso | hello! | 15:06 |
| rosmaita | o/ | 15:06 |
| enriquetaso | Hey Brian, we have 5 new bugs reported this week. | 15:06 |
| enriquetaso | #topic LVM calls - CI failure status | 15:06 |
| enriquetaso | As Brian wrote early in the cinder meeting, Wallaby backport https://review.opendev.org/c/openstack/cinder/+/798671 is waiting for https://review.opendev.org/c/openstack/cinder/+/798696/ to be merged. | 15:06 |
| enriquetaso | Please core reviews if you have some time please check those 2 ^ | 15:06 |
| enriquetaso | :) | 15:06 |
| enriquetaso | #topic bug_1: Cinder matrix - clarification of snapshot attachment feature | 15:07 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1934168 | 15:07 |
| enriquetaso | Thanks hemna for reporting this. | 15:07 |
| enriquetaso | The question is if this should be pitched as an optimization rather than missing functionality in the driver and/or backend. | 15:07 |
| enriquetaso | Summary: In most arrays, snapshots are not attachable entities. The default behavior is that a volume is created from the snapshot and that volume is then attached. | 15:08 |
| enriquetaso | Option 1: remove this feature from the matrix. | 15:09 |
| enriquetaso | Option 2. Call out in 'Notes' that this is an optimization in backup support for drivers that can attach snapshots and is not a gap in functionality for those who don't. | 15:09 |
| rosmaita | i seem to remember discussing this -- was it at last week's meeting? | 15:09 |
| rosmaita | cinder meeting, not bug squad, i mean | 15:09 |
| enriquetaso | oh, sorry about that | 15:09 |
| walshh_ | Yes, I brought up it | 15:09 |
| geguileo | yes, I think we agreed on the note, right? | 15:10 |
| enriquetaso | great, sorry! I'll update the launchpad but all this then | 15:10 |
| walshh_ | I am happy to make that change if there are no objections. Thank you | 15:10 |
| enriquetaso | :) cool | 15:11 |
| rosmaita | even better! | 15:11 |
| geguileo | walshh_: thanks! | 15:11 |
| rosmaita | yes, please put up a patch and we can discuss there | 15:11 |
| walshh_ | will do | 15:11 |
| enriquetaso | thanks! | 15:11 |
| enriquetaso | #topic bug_2: Cinder can not use system scope due to endpoint | 15:11 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1934480 | 15:11 |
| enriquetaso | Summary: Cinder cannot use system scope since its endpoint has project_id and will get a empty url in catalog in system scope. | 15:12 |
| enriquetaso | I'm not completely sure if this is a feature or a bug. | 15:12 |
| enriquetaso | Does cinder use the other scopes already (project-scope, domain-scope)? | 15:12 |
| enriquetaso | I've marked it as Medium importance but I should move it to the wishlist if it's a feature. | 15:12 |
| eharney | as written, this seems to be a bug about how keystone endpoints were deployed and not a cinder bug | 15:12 |
| rosmaita | i think also it is a duplicate | 15:12 |
| geguileo | didn't jokke report that one too? | 15:12 |
| rosmaita | pretty sure lbragstad filed a bug around this | 15:12 |
| rosmaita | i think jokke's was a bit different | 15:13 |
| geguileo | ah, ok | 15:13 |
| enriquetaso | so, looks like a keystone bug then | 15:13 |
| rosmaita | well, it's also a cinder problem | 15:14 |
| eharney | no | 15:14 |
| rosmaita | a system-scoped token is not related to any project | 15:14 |
| enriquetaso | make sense | 15:14 |
| rosmaita | but we assume that every user , including an admin, is in a project | 15:14 |
| rosmaita | and that project-id must be in the URL | 15:14 |
| rosmaita | (and we check somewhere that the URL project and token project match) | 15:15 |
| eharney | so do we want this to be an RFE for "support system scope" in general? | 15:15 |
| rosmaita | something like that | 15:15 |
| rosmaita | there's no good status in launchpad for that | 15:16 |
| enriquetaso | but, shouldn't mark it as duplicate of lbragstad bug ? and continue the work on that one? | 15:16 |
| rosmaita | yes, if there is a bug | 15:16 |
| rosmaita | not sure if there is though, i may be thinking of a patch | 15:16 |
| rosmaita | maybe this one: https://bugs.launchpad.net/cinder/+bug/1926336 | 15:17 |
| opendevreview | jinyuanliu proposed openstack/cinder master: Clean up extra spaces https://review.opendev.org/c/openstack/cinder/+/799848 | 15:18 |
| hemna | rosmaita fwiw, I haven't had time to look at the ceph-iscsi CI job failures yet | 15:18 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1926336 | 15:18 |
| enriquetaso | oh, well, then I can make 1934480 and RFE for "support system scope" as Eric mentioned? | 15:20 |
| rosmaita | hemna: that's ok, i did get the github mirroring working though | 15:21 |
| hemna | ok perfect thanks | 15:21 |
| enriquetaso | #action (enriquetaso): make 1934480 RFE for "support system scope" :) | 15:23 |
| enriquetaso | OK, last one | 15:23 |
| rosmaita | enriquetaso: sorry, i got pulled away for a minute | 15:23 |
| enriquetaso | sure, don't worry, i think it was an obvious question doh :P | 15:23 |
| enriquetaso | #bug_3: Solidfire with OSprofiller and ElementOs | 15:24 |
| enriquetaso | There are two bugs that sort of confused me. Maybe the reporter already discussed it in the cinder meeting and I forgot. :s | 15:24 |
| enriquetaso | On the one hand, we have (1)https://bugs.launchpad.net/cinder/+bug/1934459 "Enabling OSprofiler in a Openstack cluster with Element OS , runs into an infinite loop recursion". | 15:24 |
| enriquetaso | On the other hand (2) https://bugs.launchpad.net/cinder/+bug/1934435 I left a question asking if it's a feature or a bug report and asking for more info. | 15:24 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1934459 | 15:24 |
| enriquetaso | #link https://bugs.launchpad.net/cinder/+bug/1934435 | 15:24 |
| enriquetaso | The first one looks like solidfire config isn't correct | 15:25 |
| eharney | bug 1934435 looks like the backend is throwing errors, someone from solidfire will need to look at that one | 15:25 |
| eharney | for 1934459, the solidfire driver should probably just stop overriding __getattr__, dunno why it does that | 15:26 |
| eharney | unsurprising that it causes weird issues | 15:26 |
| enriquetaso | eharney++ | 15:28 |
| enriquetaso | thanks Eric | 15:28 |
| enriquetaso | OK, that's all I have for today's meeting | 15:28 |
| enriquetaso | #topic open discussion | 15:28 |
| enriquetaso | do we have any bug to discuss in this 2 min left | 15:29 |
| enriquetaso | OK! Thanks everyone! | 15:30 |
| rosmaita | thanks, sofia! | 15:30 |
| enriquetaso | sorry for the misunderstanding in the first bugs | 15:30 |
| enriquetaso | my head was a little clueless | 15:30 |
| enriquetaso | #endmeeting | 15:30 |
| opendevmeet | Meeting ended Wed Jul 7 15:30:40 2021 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/2021/cinder_bs.2021-07-07-15.06.html | 15:30 |
| opendevmeet | Minutes (text): https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-07-07-15.06.txt | 15:30 |
| opendevmeet | Log: https://meetings.opendev.org/meetings/cinder_bs/2021/cinder_bs.2021-07-07-15.06.log.html | 15:30 |
| rosmaita | enriquetaso: not at all, anything involving multiple openstack projects is complicated | 15:31 |
| enriquetaso | :D | 15:32 |
| jungleboyj | rosmaita: Did you seen diablo_rojo's mailing list item on Grace Hopper Open Source Days? | 15:55 |
| rosmaita | jungleboyj: no, looking now | 16:02 |
| rosmaita | cinder people: if anyone is interested in mentoring on July 15 for Grace Hopper Open Source Day, here is some info: http://lists.openstack.org/pipermail/openstack-discuss/2021-July/023465.html | 16:09 |
| jungleboyj | rosmaita: Right. Wanted to make sure you had seen that. Not sure if we have any work that is appropriate, but thought I would share. | 16:16 |
| rosmaita | ok, thanks | 16:17 |
| jungleboyj | NP. | 16:17 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: [WIP] Add Pure Storage NVMe-RoCE driver https://review.opendev.org/c/openstack/cinder/+/799871 | 17:00 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: [WIP] Add Pure Storage NVMe-RoCE driver https://review.opendev.org/c/openstack/cinder/+/799871 | 17:22 |
| opendevreview | Simon Dodsley proposed openstack/cinder master: [WIP] Add Pure Storage NVMe-RoCE driver https://review.opendev.org/c/openstack/cinder/+/799871 | 17:47 |
| opendevreview | Douglas Mendizábal proposed openstack/cinder master: WIP/DNM: Add fips check jobs https://review.opendev.org/c/openstack/cinder/+/790535 | 20:05 |
| opendevreview | Brian Rosmaita proposed openstack/cinder master: Drop lower-constraints jobs https://review.opendev.org/c/openstack/cinder/+/789077 | 20:30 |
| opendevreview | Brian Rosmaita proposed openstack/os-brick master: Drop lower-constraints jobs https://review.opendev.org/c/openstack/os-brick/+/789079 | 20:35 |
| opendevreview | Brian Rosmaita proposed openstack/python-cinderclient master: Dropping lower constraints testing https://review.opendev.org/c/openstack/python-cinderclient/+/789065 | 20:41 |
| opendevreview | Brian Rosmaita proposed openstack/cinderlib master: Drop lower-constraints file https://review.opendev.org/c/openstack/cinderlib/+/799913 | 20:50 |
| opendevreview | Brian Rosmaita proposed openstack/rbd-iscsi-client master: Drop lower constraints testing https://review.opendev.org/c/openstack/rbd-iscsi-client/+/799914 | 20:57 |
| opendevreview | Brian Rosmaita proposed openstack/python-cinderclient stable/wallaby: Remove skip_missing_interpreters https://review.opendev.org/c/openstack/python-cinderclient/+/799882 | 21:13 |
| opendevreview | Merged openstack/cinder stable/wallaby: LVM: Use --readonly for lvdisplay in lv_has_snapshot https://review.opendev.org/c/openstack/cinder/+/798696 | 22:42 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!