14:00:25 #startmeeting cinder 14:00:26 Meeting started Wed Feb 12 14:00:25 2020 UTC and is due to finish in 60 minutes. The chair is rosmaita. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:00:27 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:00:28 Hi 14:00:29 The meeting name has been set to 'cinder' 14:00:36 hi 14:00:37 #topic roll call 14:00:42 hi! o/ 14:00:51 hi 14:00:51 o/ 14:02:09 hello everyone 14:02:13 #link https://etherpad.openstack.org/p/cinder-ussuri-meetings 14:02:24 ok, let's get started 14:02:34 #topic announcements 14:02:58 #topic announcements - virtual midcycle -art 2 planning 14:03:25 o/ 14:03:28 Part ? 14:03:38 Ok got it 14:03:45 whoami-rajat: ty, yes part 2 14:03:53 o/ 14:03:57 o/ 14:03:59 first, I never reported on the virtual mid-cycle part 1 follow-up poll 14:04:08 hi 14:04:12 o/ 14:04:19 hello sofia 14:04:28 hi 14:04:52 sorry, i am having typing probs this morning 14:05:05 o/ 14:05:06 ok, closed the poll on 5 feb 14:05:10 had 9 responses 14:05:26 5 from people not there, which was good, i wanted to know why they didn't attend 14:05:39 2 had conflicts suddenly come up; 1 thought not relevant; 1 didn't know about it; 1 left 15 min early 14:05:54 so, maybe one extra announcement on the ML will help that 14:06:06 No suggestions for improving the format. 14:06:16 1 person mentioned that 2 hours was a good length. 14:06:28 People who attended rated BlueJeans connection "pretty good to excellent." 14:06:38 so i think we should stick with this for part 2 14:06:52 btw, i forgot to mention that jungleboyj uploaded the recording to the cinder youtube channel 14:06:57 Yeah, I think BlueJeans works well and the resulting recordings are good. 14:07:01 :-) 14:07:03 #link https://youtu.be/Dz28U1pQnqA 14:07:22 ok, so part 2 has been proposed for week of 16 March 14:07:32 (gives us 3 weeks before final ussuri release of client libraries) 14:07:40 my question today is: does anyone here know of a major conflict that week? 14:07:40 (i almost planned part 1 for the week of FOSDEM, so I wanted to check) 14:08:15 well, if you think of anything, let me know 14:08:20 I am not aware of conflicts 14:08:38 Can you share the link for the voting poll? 14:08:41 i'll put together a poll for day/time and send it out next week (probably after the meeting) 14:08:52 jungleboyj: you are 1 step ahead of me there 14:09:05 Ah, not out yet. Sorry, misinterpreted the previous message. 14:09:19 np 14:09:23 #topic releases 14:09:50 this is week R-minus-13 and milestone 2 14:10:00 #topic releases - cinder 14:10:06 we have about 260 changes since the stable/train branch 14:10:13 one major feature (glance multiple store support) 14:10:20 i'd like to do a milestone-2 release (even though we don't have to) 14:10:27 it will be our first python3-only release (would rather not wait until milestone-3 for that) 14:10:36 i took a quick look at the open patches, but didn't notice anything essential for M-2 14:10:43 #link https://review.opendev.org/#/q/project:openstack/cinder+status:open+branch:master 14:10:51 so unless someone can point something out, i'll put up a release patch today 14:11:03 (you don't have to tell me right this minute) 14:11:29 i'll put up the patch around 20:00 UTC today unless i hear otherwise 14:11:39 #topic releases - python-cinderclient 14:11:47 the release team is pushing for a milestone-2 release 14:11:58 i think it's a good idea to get the py3-only release out even though there are not a lot of changes there 14:12:05 here's what we have open right now: 14:12:13 #link https://review.opendev.org/#/q/project:openstack/python-cinderclient+status:open+branch:master 14:12:47 there are a few patches sitting with one +2 14:13:04 so if reviewers have time, please take a look 14:13:18 i'll put up the patch for that one tomorrow 14:13:39 (actually, the patch is there, i will just revise it) 14:13:42 #topic releases - os-brick 14:13:52 there are some things we should get in (like the cryptsetup password checking) 14:13:58 this one we can release next week 14:14:08 #link https://review.opendev.org/#/q/project:openstack/os-brick+status:open+branch:master 14:14:28 so, after you look over python-cinderclient, please direct your attention to os-brick 14:14:37 #topic releases - final rocky release 14:14:53 we need everything merged by 21 Feb (release must be ready for 24 Feb) 14:15:07 last i looked, there's not much 14:15:25 there may be a few pending backports iirc 14:15:48 right, so after you look at os-brick, please look at anything pending for stable/rocky 14:15:49 or at least going their way to rocky, so hurry up with backports if you also want to get them in the last rocky tarball 14:16:06 yes, if you have things on the way, please keep on top of them 14:16:07 Would be useful to do a sweep all the way from train backports to see if there is anything that needs to go all the way back. 14:16:24 any volunteers for ^^ ? 14:16:27 it's a good idea 14:16:43 Not the end of the world, but last chance to do it. 14:16:52 rocky is indeed a long way back 14:16:54 I went through backports the other day and didn't see too many outstanding patches but may need to get more backported. 14:16:59 * last chance to do it and get in an official release 14:17:36 I started working on a script to automatically find bugfixes that have not been backported. Really should find where that went. 14:17:47 :-) 14:17:50 right, the branch will go into "extended maintenance" mode -- no more releases, but we can still backport changes 14:18:24 anyway, if you have done an important bugfix recently, please assess it for backport-worthiness and get it proposed ASAP 14:18:44 ok, final announcement and then i will shut up 14:18:58 #topic volume-local-cache spec 14:19:08 cross-project meeting with nova about volume-local-cache is tomorrow, 13:30-14:00 UTC 14:19:16 details are here: 14:19:17 thanks rosmaita 14:19:19 #link http://lists.openstack.org/pipermail/openstack-discuss/2020-February/012457.html 14:19:26 LiangFang: np 14:19:52 I have replied some info in: https://etherpad.openstack.org/p/volume-local-cache 14:20:10 ok, i will definitely take a look before the meeting 14:20:11 sorry about the weird time thing on that poll, btw 14:21:07 the reason for the middle of the night in europe time choices, though, was that only new york, mumbai, and shanghai time zones were mentioned when i asked in the meeting last week who might attend 14:21:28 i was trying to be creative, but forgot that it was like 2:30 am in europe 14:21:47 Not really a good way to include US and China without that happening. 14:21:47 anyway, 13:30 UTC seems to be ok for most people 14:22:11 yes, our geographic spread makes it really challenging to schedule stuff 14:22:16 not sure if eric will join or not 14:22:36 you mean eric fried? 14:22:44 eric honey 14:22:47 Or Harney. 14:22:52 :-) 14:22:54 smcginnis: yes 14:22:56 :) 14:22:58 I Like Eric Honey 14:23:14 ... 14:23:31 well, we'll see what we can get done 14:23:50 ok, on to our regular topics, bunch of stuff on the agenda 14:24:07 #topic RSD Cinder Driver move to unsupported 14:24:15 IvensZambrano: that's you 14:24:21 thanks 14:24:25 hi everyone 14:24:34 Hey IvensZambrano 14:25:19 Due to an org change the team I belong to is moving away from some of the activites we were supporting and performing for the project 14:25:33 o/ 14:25:57 that includes the RSD cinder driver, the new proposal for the Swordfish driver and the changes proposed for os-brick 14:26:08 for NVMeoF 14:26:32 yes rajinir? 14:26:38 :-( 14:26:42 Wasn't Mellanox also participating in some of that? 14:26:47 Ok. 14:27:25 well, that's kind of a bummer 14:27:27 i'm not sure smcginnis, we were not directly interacting with them on those three items 14:27:27 sorry to hear it 14:27:37 So, IvensZambrano brought this up to me last week. 14:27:41 it is, sorry to come with such announcements 14:27:43 Yeah, that's really too bad. 14:28:03 Lenovo had been leveraging some of this work but when I raised it internally didn't get any concern. 14:28:15 Thought we should bring it up here given that it is pretty new technology. 14:28:26 Does this impact anyone else? 14:28:33 Any concern from the distros? 14:28:38 Anyone at companies using this, now would be a good time to reach out internally and make sure they are aware of the impact of this. 14:28:51 smcginnis: ++ 14:29:21 i guess a ML announcement would be in order, too 14:29:31 rosmaita: Probably a good idea. 14:29:36 yes please, it will be good to know if any impacts to current usages of it will come 14:29:45 I am really sorry to see the RedFish work being stopped. 14:29:47 as we are hosting the RSD Driver CI as well 14:29:54 and Swordfish 14:29:58 and it will go off soon 14:30:07 jungleboyj: +1 14:30:14 ok, thanks for keeping us posted 14:30:31 well, there is always the option of handing over to another organisation the evolution of the Swordfish driver 14:30:42 I shared the process for unsupporting. Guess whether it gets removed in the next release depends on the outcome of those other discussions. 14:30:50 that is a nice project and it is aligned to open standards 14:30:59 IvensZambrano: ++ 14:31:52 So, don't see anyone super concerned here. 14:31:54 i guess we have until M-3 to get it sorted out 14:32:10 Are there storage devices supporting Swordfish yet? 14:32:11 maybe everyone is shocked? 14:32:17 :-) 14:33:11 we were using an intermediate software layer for our development but the end goal was to extend the BMCs at some point to natively support SF 14:33:44 IvensZambrano: do you want to send something to the operators list, or do you want me to? 14:34:07 We should get an e-mail out there somewhere. 14:34:45 rosmaita: what is the usual procedure? 14:34:55 not sure exactly 14:35:12 tell you what, we can collaborate on an etherpad and i can send something out 14:35:24 rosmaita: ++ That sounds like a good plan. 14:35:32 rosmaita: thanks! that will be great 14:35:35 i want to make sure i get the detals right, especially about who to contact if they are interested in picking up the SF work 14:35:50 IvensZambrano: Thanks for being proactive and communicating with us. 14:36:16 yes, thanks 14:36:17 now I'm more sad to leave the project with this great support 14:36:24 IvensZambrano: https://etherpad.openstack.org/p/swordfish-and-stuff 14:36:27 jungleboyj: ^^ 14:36:30 rosmaita: It is working ;-) 14:36:54 rosmaita: thanks! 14:37:04 IvensZambrano: Sad to see you go. Enjoyed the bit we got to work together. 14:37:14 jungleboyj: thanks as well for the collaboration the past few days! 14:37:32 davidsha: anything to add? 14:38:18 you were actively collaborating with the project from our team 14:38:56 It's been a pleasure working with you all, and hope we get to collaborate again in the future! 14:39:07 davidsha: likewise! 14:39:23 thank you all! 14:39:33 ok, thanks for that sad announcement 14:39:38 #topic Backport Policy: Should we backport the Incremental Backup patch? 14:39:51 #link https://review.opendev.org/#/c/705553/ 14:39:58 enriquetaso: this one is you 14:40:02 hi 14:40:08 hello 14:40:14 I think there is some misunderstood between the incremental patch. The patch is changing object version and because of that It may be seen as a new feature. I would like to discuss if backporting the patch is a good idea for us : https://review.opendev.org/#/c/705553/ 14:40:25 I would say that's a bug since the end user could specify --incremental and have no idea it didn't do what they expected. 14:40:41 smcginnis: ++ 14:40:43 ++ 14:40:49 On the other hand, rosmaita showed me this reported bug https://bugs.launchpad.net/cinder/+bug/1862635 related to the incremental patch. I didn't tought about it at the time and having CI with c-bak disable didn't help.. So, i'm working on a fix. 14:40:50 Launchpad bug 1862635 in Cinder "Cinder backup export broken" [Undecided,New] - Assigned to Sofia Enriquez (lsofia-enriquez) 14:40:50 smcginnis: +1 14:41:03 is't somewhere between a feature and a bug 14:41:10 smcginnis: ++ 14:41:34 That does raise the point again - in most tempest runs, cinder backup has been disabled because it was constantly failing. 14:41:34 yeah, my point about the bug is that we'll want to backport that too if we decide to backport the original change 14:41:39 We should really fix that (tm). 14:41:39 from users' perspective it's a bug fix without any new CLI flags, config options or API changes 14:42:02 Should we add that to our Logo? 14:42:07 ;) 14:42:09 :) 14:42:26 :P 14:42:28 That's what the horse is looking over its shoulder at! 14:42:39 Yep. That was what I was thinking. 14:42:54 rosmaita++ sounds like a good plan 14:42:56 i think e0ne clearly stated the reason why its backportable 14:43:05 rosmaita: thanks :) 14:43:16 e0ne++ 14:43:24 ok, good ... wanted to discuss this because of an objection on enriquetaso's backport patch 14:43:35 but it sounds like we are in agreement here 14:43:36 One nit-ish thing - best not to propose backports to older branches until the patch is merged to the more recent branch. 14:43:57 smcginnis, yes, sorry 14:44:04 I know folks want to make sure it's everywhere, but that puts the burden on reviewers to make sure things don't get messed up for stable policy. 14:44:08 smcginnis: ++ 14:44:15 Not a huge deal, just wanted to point that out. 14:44:18 Otherwise it sounds fine to backport. 14:44:35 Hey, Red Hat made a logo for cinderlib. :-) 14:44:40 smcginnis: thank you for noticing that 14:45:02 where is the cinderlib logo ? :O 14:45:47 In a presentation I am watching. 14:46:06 ok, so the consensus is: is backportable along with the related bugfix (after the bug is fixed); needs to go in the regular backport order T->S->R 14:46:15 jungleboyj: is multitasking! 14:46:17 ++ 14:46:23 ++ 14:46:40 ok, enriquetaso -- all your questions answered? 14:46:54 yes 14:46:56 thanks 14:47:13 great, this is exciting, we can have open discussion! 14:47:18 #topic open discussion 14:47:27 actually, I have something 14:47:34 i forgot to mention before, i've a patch that i want it to go to rocky release https://review.opendev.org/#/c/707343/ 14:48:37 whoami-rajat: did you figure out the problem with lower-constraints ? 14:48:47 yep, unrelated 14:48:54 its currently passing on zuul jobs 14:49:05 ok, cool -- i dind't notice the recheck 14:49:05 Ok. LGTM when it passes the check. 14:49:19 thanks!! 14:49:40 i remember looking at the patch to master, i think this should be fine 14:49:46 rosmaita: Don't need any discussion on the Datera driver? 14:49:59 i haven't heard anything back yet 14:50:05 Ok. 14:50:26 but not to keep people in suspense 14:50:33 :-) 14:50:50 Datera has got their CI running again, and are proposing that their driver be marked supported in U 14:51:03 there is some discussion on that patch 14:51:35 which i can't find the link to atm 14:52:09 anyway, things are kind of up in the air, but will hopefully be resolved ok 14:52:18 https://review.opendev.org/#/c/704153/ 14:52:24 ty 14:52:36 part of the problem is python2 compatibility 14:53:24 I really would like to see that broken out into multiple patches, but I think I've said enough on that patch. :| 14:53:48 yes, and i did mention that to them 14:54:08 :-) 14:54:45 i think part of the problem is that they'd like to fix the driver in U and then be able to have people use it with N through T also 14:54:53 not sure how viable a plan that is 14:55:09 Another scary part of all that. 14:55:18 :-( That is scary. 14:55:18 and i think that's part of the big monolithic patch 14:55:30 rosmaita: +1 14:55:44 because i think they're thinking they need to backport all that stuff everywhere 14:55:57 Ouch. That won't work. 14:55:59 instead of having multiple small patches and only backporting the appropriate stuff 14:56:05 rosmaita: ++ 14:56:21 Maybe communicating that better to them will help. 14:56:28 but, of course, it's easy to say after they've done a bunch of work: break it into smaller pieces! 14:56:39 it's pretty hard to hear that when you've done all the work! 14:56:45 True. 14:56:47 so we'll see what happens 14:57:10 rosmaita: Thanks for following up. 14:57:15 One other topic. 14:57:19 Well, all of those changes built up over time in their GitHub fork, so theoretically they could port over commits from there piecemeal. 14:57:32 ok, one thing from me for people running Train or who know such people 14:57:40 smcginnis: ++ 14:57:44 this patch has been rejected for nova (i'm working on a more robust fix) 14:57:44 but the release note is worth looking at: 14:57:44 #link https://review.opendev.org/#/c/706298/1 14:57:55 it's about a problem that arises in a really weird situation with our automatic encryption key handling (it was actually a change in glance, but really for us) 14:57:55 anyway, it can be addressed immediately by a nova config change 14:57:55 i will be sending out something to the operators list 14:57:55 but if you are running Train, or know people who are, you might want to bring it to the appropriate person's attention 14:58:15 that's it from me 14:58:20 previously i mentioned there's a white paper of volume local cache. Now it is approved to publish. But it is in chinese, co-authored by China Unicom. Mainly shows the performance gain with Optane SSD. Will upload the writepaper to some website later and let team know. Does openstack.org has some place to containing write paper? 14:58:23 1.75 minutes left 14:58:27 One other topic as well if I may, I filed a bug a souple of days ago. I intend to write a patch for it but would at some point apreciate some discussions on the best way to architect the fix. - https://bugs.launchpad.net/cinder/+bug/1862636 14:58:28 Launchpad bug 1862636 in Cinder "cinder retype corrupts volume in some scenarios" [Undecided,New] 14:58:53 that doesn't sound good 14:59:00 trident: thanks for reporting it 14:59:14 and glad to hear you want to fix it, too! 14:59:47 rosmaita: Have you polled the team to see who is planning to be in Vancouver? 15:00:07 Think we need to let OSF know if we are going to be there soon. 15:00:13 i have not, and the table planning is due feb 21 i think 15:00:28 show of hands: who is probably going to be in vancouver? 15:00:36 o/ 15:00:36 o/ Planning to be. 15:01:04 i don't know 15:01:18 actually, i don't know either 15:01:22 Probably. No official word yet. 15:01:24 u.u 15:01:27 it's kind of early 15:01:27 :P 15:01:41 Don't need to know for sure. Just need to know how many want to attend. 15:01:43 i think the same size table we had in shanghai will be good 15:01:49 rosmaita: ++ 15:01:58 which was what, 12 seats? 15:01:59 rosmaita, ++ 15:02:02 Lets do a ML post and see what kind of response we get. 15:02:13 ok, i'll get one out this afternoon 15:02:16 thanks everyone! 15:02:19 Thanks! 15:02:22 jungleboyj: thanks for the reminder 15:02:27 :-) 15:02:27 Thanks! 15:02:31 That is what I am here for. 15:02:36 :D 15:02:39 #endmeeting