| opendevreview | Merged openstack/cinder stable/ussuri: Open local image files with "rb" mode https://review.opendev.org/c/openstack/cinder/+/793169 | 02:31 |
|---|---|---|
| opendevreview | Felix Huettner proposed openstack/cinder master: Revert attachment creation on failure https://review.opendev.org/c/openstack/cinder/+/793136 | 10:01 |
| *** akekane_ is now known as abhishekk | 10:49 | |
| opendevreview | Andrei proposed openstack/cinder master: Open-E JovianDSS: Addes multi-attach support https://review.opendev.org/c/openstack/cinder/+/794962 | 12:00 |
| *** dviroel|out is now known as dviroel | 12:12 | |
| opendevreview | Helen Walsh proposed openstack/cinder stable/wallaby: PowerMax Driver - Temporary snapshot enhancements https://review.opendev.org/c/openstack/cinder/+/798088 | 13:43 |
| opendevreview | Helen Walsh proposed openstack/cinder stable/wallaby: PowerMax Driver - Fix for deleting replication group https://review.opendev.org/c/openstack/cinder/+/798091 | 13:53 |
| opendevreview | Helen Walsh proposed openstack/cinder stable/wallaby: PowerMax Driver - Fix for create group from source https://review.opendev.org/c/openstack/cinder/+/798094 | 14:05 |
| TusharTgite | hi team, eharney i've prepared a reset state sheet https://wiki.openstack.org/wiki/Reset_State_Robustification and mention all the changes which i think need to do in this spec https://review.opendev.org/c/openstack/cinder-specs/+/766716 please give a review on this, this sheet is editable so you can put comments directly in it. | 14:50 |
| hemna | anyone here today? | 15:45 |
| hemna | rosmaita, geguileo, smcginnis ping | 15:47 |
| hemna | I think I found an issue with the filter scheduler | 15:48 |
| hemna | during retype it's always requiring --migration-policy on-demand basically | 15:48 |
| hemna | even though the retype wouldn't result in a migration | 15:49 |
| hemna | I have a volume A which lives on foo@bar#poolA | 15:49 |
| hemna | I'm retyping to #poolB | 15:49 |
| hemna | it's failing because it says 'Current backend not valid for volume {} with type {}, migration not allowed | 15:50 |
| hemna | even though #poolB lives on foo@bar#poolB | 15:50 |
| hemna | wtf | 15:50 |
| hemna | I think this block is failing | 15:56 |
| hemna | https://github.com/openstack/cinder/blob/master/cinder/scheduler/filter_scheduler.py#L154-L157 | 15:56 |
| hemna | the backend_state.backend_id == backend matches the entire string (foo@bar#poolB) | 15:56 |
| smcginnis | Wouldn't that still be considered a migration? Depending on the backend, it would need to copy over the volume from one pool to the other. | 15:56 |
| hemna | I think it depends on the backend | 15:57 |
| geguileo | afaik that is the expected behavior | 15:57 |
| geguileo | it may depend on the backend, and even on the pool where the volume is living | 15:57 |
| geguileo | so that's why the behavior is to be safe and require the parameter | 15:58 |
| hemna | ok I guess that's ok then | 15:58 |
| geguileo | because in theory the volume needs to be moved | 15:58 |
| geguileo | even if it's efficiently done | 15:58 |
| geguileo | in some cases | 15:58 |
| hemna | ok makes sense. thanks | 16:26 |
| fungi | in my usual patrolling of wiki edits by new users, i came across these which seem to be cinder-related but i can't determine if they're legitimate or spam: https://wiki.openstack.org/wiki/Reset_state https://wiki.openstack.org/wiki/Reset_State_Robustification | 17:03 |
| fungi | anybody know if this user is a cinder contributor maybe? https://wiki.openstack.org/wiki/Special:Contributions/Tushargite96 | 17:04 |
| eharney | fungi: he is | 17:06 |
| fungi | thanks, i'll add that account to the auto-patrol group in that case so i can ignore further edits by them | 17:07 |
| fungi | can someone maybe add a bit more context to those pages, and or get rid of/redirect one if they're duplicates? | 17:07 |
| fungi | a wiki page which only has a google docs link isn't terribly helpful | 17:08 |
| fungi | i certainly very nearly blocked the account, since almost 100% of the spam we get on the wiki is new users adding pages consisting of only a hyperlink to some outside site | 17:10 |
| fungi | on a whim i loaded the google doc and saw cinder commands in it, which is the only reason i didn't | 17:10 |
| *** dviroel is now known as dviroel|afk | 17:26 | |
| *** dviroel|afk is now known as dviroel | 18:15 | |
| opendevreview | Felipe Rodrigues proposed openstack/cinder master: DNM: testing gerrit CI https://review.opendev.org/c/openstack/cinder/+/798147 | 18:38 |
| opendevreview | Felipe Rodrigues proposed openstack/cinder master: DNM: testing gerrti CI 2 https://review.opendev.org/c/openstack/cinder/+/798149 | 18:51 |
| opendevreview | Merged openstack/cinder-specs master: NVMe-oF connection agent https://review.opendev.org/c/openstack/cinder-specs/+/796365 | 20:58 |
| *** abishop_ is now known as abishop | 21:18 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!