| *** tsmith2 has quit IRC | 00:33 | |
| *** Liang__ has joined #openstack-meeting-alt | 01:23 | |
| *** tetsuro has joined #openstack-meeting-alt | 01:23 | |
| *** gyee has quit IRC | 01:25 | |
| *** tetsuro has quit IRC | 01:25 | |
| *** ricolin_ has joined #openstack-meeting-alt | 01:31 | |
| *** wxy has joined #openstack-meeting-alt | 01:31 | |
| *** Liang__ is now known as LiangFang | 02:08 | |
| *** enriquetaso has quit IRC | 02:27 | |
| *** rcernin has quit IRC | 02:29 | |
| *** zzzeek has quit IRC | 03:18 | |
| *** zzzeek has joined #openstack-meeting-alt | 03:19 | |
| *** rcernin has joined #openstack-meeting-alt | 03:29 | |
| *** zzzeek has quit IRC | 03:55 | |
| *** zzzeek has joined #openstack-meeting-alt | 03:58 | |
| *** belmoreira has joined #openstack-meeting-alt | 04:23 | |
| *** carthaca has quit IRC | 04:32 | |
| *** zzzeek has quit IRC | 04:45 | |
| *** zzzeek has joined #openstack-meeting-alt | 04:48 | |
| *** zzzeek has quit IRC | 05:17 | |
| *** zzzeek has joined #openstack-meeting-alt | 05:18 | |
| *** vishalmanchanda has joined #openstack-meeting-alt | 06:14 | |
| *** LiangFang has quit IRC | 06:17 | |
| *** Liang__ has joined #openstack-meeting-alt | 06:18 | |
| *** ralonsoh has joined #openstack-meeting-alt | 06:34 | |
| *** e0ne has joined #openstack-meeting-alt | 06:35 | |
| *** e0ne has quit IRC | 06:39 | |
| *** slaweq has joined #openstack-meeting-alt | 06:51 | |
| *** jtomasek has joined #openstack-meeting-alt | 07:01 | |
| *** rdopiera has joined #openstack-meeting-alt | 07:27 | |
| *** rdopiera has quit IRC | 07:31 | |
| *** rdopiera has joined #openstack-meeting-alt | 07:32 | |
| *** tosky has joined #openstack-meeting-alt | 07:57 | |
| *** Liang__ is now known as LiangFang | 08:00 | |
| *** Roamer` has joined #openstack-meeting-alt | 08:07 | |
| *** slaweq has quit IRC | 08:20 | |
| *** rcernin has quit IRC | 08:20 | |
| *** apetrich has joined #openstack-meeting-alt | 08:26 | |
| *** slaweq has joined #openstack-meeting-alt | 08:29 | |
| *** derekh has joined #openstack-meeting-alt | 08:32 | |
| *** apetrich has quit IRC | 08:39 | |
| *** carthaca has joined #openstack-meeting-alt | 08:39 | |
| *** ianychoi__ has quit IRC | 08:40 | |
| *** apetrich has joined #openstack-meeting-alt | 08:44 | |
| *** slaweq has quit IRC | 08:49 | |
| *** LiangFang has quit IRC | 09:29 | |
| *** e0ne has joined #openstack-meeting-alt | 09:32 | |
| *** ricolin_ has quit IRC | 09:38 | |
| *** ralonsoh has quit IRC | 10:01 | |
| *** ralonsoh has joined #openstack-meeting-alt | 10:02 | |
| *** priteau has joined #openstack-meeting-alt | 10:43 | |
| *** rcernin has joined #openstack-meeting-alt | 10:53 | |
| *** rcernin has quit IRC | 11:28 | |
| *** raildo has joined #openstack-meeting-alt | 11:30 | |
| *** lkoranda has joined #openstack-meeting-alt | 11:36 | |
| *** rcernin has joined #openstack-meeting-alt | 11:45 | |
| *** lkoranda has quit IRC | 11:52 | |
| *** lkoranda has joined #openstack-meeting-alt | 11:53 | |
| *** rfolco has joined #openstack-meeting-alt | 12:02 | |
| *** jhesketh has quit IRC | 12:17 | |
| *** jhesketh has joined #openstack-meeting-alt | 12:17 | |
| *** derekh has quit IRC | 12:31 | |
| *** andrebeltrami has joined #openstack-meeting-alt | 12:36 | |
| *** rcernin has quit IRC | 12:41 | |
| *** enriquetaso has joined #openstack-meeting-alt | 12:56 | |
| *** derekh has joined #openstack-meeting-alt | 13:06 | |
| *** tmazur has joined #openstack-meeting-alt | 13:14 | |
| *** rcernin has joined #openstack-meeting-alt | 13:15 | |
| *** kaisers_ has joined #openstack-meeting-alt | 13:22 | |
| *** jhesketh has quit IRC | 13:31 | |
| *** rcernin has quit IRC | 13:41 | |
| *** sfernand has joined #openstack-meeting-alt | 13:41 | |
| *** vishakha has joined #openstack-meeting-alt | 13:47 | |
| *** thgcorrea has joined #openstack-meeting-alt | 13:54 | |
| *** Liang__ has joined #openstack-meeting-alt | 13:55 | |
| *** Liang__ is now known as LiangFang | 13:55 | |
| *** rosmaita has joined #openstack-meeting-alt | 13:59 | |
| *** whoami-rajat__ has joined #openstack-meeting-alt | 13:59 | |
| rosmaita | #startmeeting cinder | 14:00 |
|---|---|---|
| openstack | Meeting started Wed Sep 9 14:00:31 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 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 14:00 |
| *** openstack changes topic to " (Meeting topic: cinder)" | 14:00 | |
| openstack | The meeting name has been set to 'cinder' | 14:00 |
| rosmaita | #topic roll call | 14:00 |
| jungleboyj | o/ Kind of ... Double booked. | 14:00 |
| *** openstack changes topic to "roll call (Meeting topic: cinder)" | 14:00 | |
| tosky | o/ | 14:00 |
| walshh_ | hi | 14:00 |
| whoami-rajat__ | Hi | 14:00 |
| kaisers_ | hi | 14:00 |
| LiangFang | hi | 14:00 |
| rosmaita | jungleboyj: that's fine ... you are back, which is the key thing | 14:00 |
| e0ne | hi | 14:01 |
| sfernand | hi | 14:01 |
| jungleboyj | :-) Yeah, still trying to catch up on e-mail. :-) | 14:01 |
| rosmaita | good turnout | 14:01 |
| enriquetaso | hi | 14:01 |
| rosmaita | #link https://etherpad.opendev.org/p/cinder-victoria-meetings | 14:01 |
| lseki | o/ | 14:02 |
| dviroel | hi | 14:02 |
| rosmaita | ok, let's get started | 14:02 |
| rosmaita | #topic updates - check job failures update | 14:03 |
| *** openstack changes topic to "updates - check job failures update (Meeting topic: cinder)" | 14:03 | |
| *** slaweq has joined #openstack-meeting-alt | 14:03 | |
| rosmaita | there was a code change to tempest on 4 september that broke us | 14:03 |
| rosmaita | particularly cinder-tempest-plugin-lvm-lio-barbican and os-brick-src-tempest-lvm-lio-barbican | 14:03 |
| rosmaita | you may have seen 3 VolumesBackupsTest failures with "Failed validating 'type' in schema['properties']['backup']['properties']['name']" | 14:04 |
| rosmaita | the fix was merged late yesterday | 14:04 |
| rosmaita | https://review.opendev.org/#/c/750360/ | 14:04 |
| rajinir | o/ | 14:04 |
| LiangFang | this blocked us a lot, thanks rosmaita | 14:04 |
| rosmaita | took it 11 hours to get checked, gated, and merged | 14:04 |
| rosmaita | so go ahead and recheck if you're seeing that problem, should be ok now | 14:05 |
| rosmaita | do it fast before something else breaks | 14:05 |
| rosmaita | #topic updates - python-cinderclient | 14:05 |
| *** openstack changes topic to "updates - python-cinderclient (Meeting topic: cinder)" | 14:05 | |
| rosmaita | victoria release of cinderclient must happen tomorrow | 14:05 |
| rosmaita | one un-merged approved patch: https://review.opendev.org/#/c/739223/ | 14:05 |
| rosmaita | depends on this un-merged cinder change: | 14:06 |
| rosmaita | #link https://review.opendev.org/#/c/739223/ | 14:06 |
| rosmaita | whoami-rajat__: have you had a chance to look at the unit test failures? | 14:06 |
| whoami-rajat__ | rosmaita: yeah, still trying to figure out if it's a legit error or not | 14:06 |
| rosmaita | ok | 14:06 |
| whoami-rajat__ | nothing is changed along the lines of failure | 14:07 |
| smcginnis | #link https://review.opendev.org/#/c/737707 | 14:07 |
| smcginnis | I think that's the one you meant for the last one. | 14:07 |
| whoami-rajat__ | yeah ^^ | 14:07 |
| rosmaita | smcginnis: ty | 14:07 |
| rosmaita | sean is correct, the cinder patch is ^^ | 14:07 |
| rosmaita | my only concern is around the policy stuff, but i think that could be addressed in a follow-up | 14:08 |
| rosmaita | as far as the stuff that would impact the cinderclient, i think we are ok | 14:08 |
| whoami-rajat__ | yep working on that as well | 14:08 |
| *** lkoranda has quit IRC | 14:08 | |
| rosmaita | so as long as the check agrees, we should be good | 14:09 |
| *** LiangFang has quit IRC | 14:09 | |
| rosmaita | #topic updates - python-brick-cinderclient-ext | 14:09 |
| *** openstack changes topic to "updates - python-brick-cinderclient-ext (Meeting topic: cinder)" | 14:09 | |
| rosmaita | this also gets its victoria release tomorrow | 14:09 |
| rosmaita | not much activity on it recently | 14:09 |
| rosmaita | and no open reviews | 14:09 |
| rosmaita | so i just wanted to flag that in case anyone knows of some important pending issue? | 14:10 |
| rosmaita | let me know, otherwise we will cut the stable/victoria branch at the current HEAD of master and release | 14:11 |
| *** ricolin_ has joined #openstack-meeting-alt | 14:11 | |
| rosmaita | #updates - bugfix protocol reminder | 14:11 |
| rosmaita | #topic updates - bugfix protocol reminder | 14:11 |
| *** openstack changes topic to "updates - bugfix protocol reminder (Meeting topic: cinder)" | 14:11 | |
| rosmaita | reminder that any bugfixes for brick or clients go to master first, then must be backported to stable/victoria | 14:12 |
| rosmaita | we already have some for brick, will discuss in a minute | 14:12 |
| rosmaita | #topic updates - feature freeze | 14:12 |
| *** openstack changes topic to "updates - feature freeze (Meeting topic: cinder)" | 14:12 | |
| rosmaita | M-3 release day is FEATURE FREEZE | 14:12 |
| rosmaita | this is the current features list: | 14:13 |
| rosmaita | https://blueprints.launchpad.net/cinder/victoria | 14:13 |
| *** ricolin_ has quit IRC | 14:13 | |
| *** Liang__ has joined #openstack-meeting-alt | 14:13 | |
| rosmaita | some are sitting with one +2, so cores, please take a look | 14:13 |
| rosmaita | #topic updates - soft string freeze | 14:13 |
| *** openstack changes topic to "updates - soft string freeze (Meeting topic: cinder)" | 14:13 | |
| rosmaita | M-3 release day is Soft String Freeze | 14:14 |
| rosmaita | what this is: | 14:14 |
| rosmaita | "You are no longer allowed to accept proposed changes containing modifications in user-facing strings. Such changes should be rejected by the review team and postponed until the next series development opens (which should happen when RC1 is published)." | 14:14 |
| rosmaita | so reviewers, please keep that in mind | 14:14 |
| rosmaita | the idea is to give the translations team time to get stuff translated | 14:14 |
| rosmaita | ok, so that's all the updates | 14:15 |
| rosmaita | #topic M-3 release? | 14:15 |
| *** openstack changes topic to "M-3 release? (Meeting topic: cinder)" | 14:15 | |
| *** Liang__ is now known as LiangFang | 14:15 | |
| rosmaita | we are not required to do an M-3 release | 14:15 |
| rosmaita | so i am not sure whether there's any advantage in doing it | 14:16 |
| rosmaita | other than it gives us a hard timestamp for M-3 | 14:16 |
| rosmaita | i think that all the features with blueprints are underway to the point were everything could merge by Friday | 14:17 |
| rosmaita | (given active reviewing and revising activity) | 14:17 |
| rosmaita | smcginnis: jungleboyj: what do you think? | 14:18 |
| smcginnis | If we don't have anyone downstream asking for a beta release, I think we should skip it. | 14:18 |
| rosmaita | i think if we release M-3 it has to be on thursday, wereas if we don't we can declare M-3 to be EOD eastern time on Friday | 14:19 |
| jungleboyj | If we aren't required I don't think we have to. I defer to smcginnis . | 14:19 |
| rosmaita | ok, that is my feeling ... we are getting packaging feedback from the RDO team | 14:19 |
| rosmaita | that's really the only reason i can think of for having a beta | 14:19 |
| rosmaita | give packagers a head start | 14:19 |
| rosmaita | but i think most wait for RC-1 to see that we are serious about what will be in the release | 14:19 |
| smcginnis | Did the RDO team ask for an actual release to help with that? | 14:20 |
| e0ne | rosmaita: +1 | 14:20 |
| rosmaita | smcginnis: no, they are continually doing it themselves | 14:20 |
| smcginnis | OK, good. Then I think we can just skip it and get ready for the RC1 release coming up. | 14:20 |
| rosmaita | what i meant was, they don't need a beta, and they give us the feedback i think a beta would get us | 14:20 |
| jungleboyj | smcginnis: ++ | 14:21 |
| rosmaita | ok, cool | 14:21 |
| jungleboyj | Sounds good. | 14:21 |
| rosmaita | so to be clear: we will declare M-3 to be 5 pm New York time on Friday | 14:21 |
| jungleboyj | ++ | 14:21 |
| rosmaita | any features not merged by then will require a Feature Freeze Exception | 14:22 |
| smcginnis | ++ | 14:22 |
| rosmaita | but let's push and get this done, and no FFEs will be necessary and we can concentrate on bug fixes for RC-time | 14:22 |
| rosmaita | #topic os-brick situation | 14:22 |
| *** openstack changes topic to "os-brick situation (Meeting topic: cinder)" | 14:22 | |
| rosmaita | os-brick victoria release, 4.0.0 was released last week | 14:23 |
| rosmaita | RDO project was looking at packaging it and noticed that we added a big honking bindep | 14:23 |
| smcginnis | Glad they caught that. We didn't in our code reviews. | 14:24 |
| rosmaita | which was not good, so with some discussion in openstack-cinder channel yesterday (i think it was), we reverted it | 14:24 |
| rosmaita | #link https://review.opendev.org/#/c/750444/1 | 14:24 |
| rosmaita | and we replaced it with a lightweight fix that is in the gate now: | 14:24 |
| rosmaita | #link https://review.opendev.org/#/c/750452/ | 14:24 |
| rosmaita | so apologies to smcginnis and hemna, who has actually suggested the lightweight fix | 14:25 |
| e0ne | rosmaita: can we release 4.0.1 for victoria with the fix? | 14:25 |
| whoami-rajat__ | I'm still thinking if the new patch is tested properly and we might end up facing another issue with it | 14:25 |
| rosmaita | e0ne: that tis the plan | 14:25 |
| rosmaita | whoami-rajat__: can you articulate your concerns a bit? | 14:26 |
| rosmaita | so before we can release 4.0.1, we need to backport the changes to stable/victoria | 14:26 |
| rosmaita | the changes are: | 14:26 |
| whoami-rajat__ | i need to collect some links, can i share after the meeting? | 14:26 |
| rosmaita | #link https://review.opendev.org/#/c/750649/ | 14:26 |
| rosmaita | #link https://review.opendev.org/750655 | 14:26 |
| rosmaita | whoami-rajat__: that is fine, we can talk in openstack cinder | 14:27 |
| rosmaita | ok, so my opinion is: | 14:27 |
| whoami-rajat__ | ok | 14:27 |
| rosmaita | we should definitely revert the change and release 4.0.1 | 14:27 |
| e0ne | rosmaita: +1 | 14:27 |
| rosmaita | whether 4.0.1 includes the new fix will depend on whoami-rajat__ and others' discussion in the cinder channel today | 14:28 |
| rosmaita | sound good? | 14:28 |
| rosmaita | that will at least remove the bad bindep | 14:28 |
| rosmaita | but i agree with whoami-rajat__ that we don't want to do another release with more problems | 14:28 |
| rosmaita | ok, so once we have 4.0.1 i will file an FFE with the requirements team | 14:29 |
| rosmaita | so that we can change the victoria os-brick upper-constraint to 4.0.1 before release | 14:29 |
| smcginnis | At least we have a very good reason for seeking an exception. | 14:30 |
| jungleboyj | :-) | 14:30 |
| rosmaita | ok, so let's have a disucssion in #openstack-cinder after this meeting and sort out what will be in 4.0.1 | 14:30 |
| rosmaita | thanks! | 14:31 |
| rosmaita | #topic volume-local-cache | 14:31 |
| *** openstack changes topic to "volume-local-cache (Meeting topic: cinder)" | 14:31 | |
| rosmaita | LiangFang: you have been dropping in and out, so i guess a bad connection | 14:31 |
| LiangFang | yes, now in | 14:31 |
| rosmaita | so the current status is | 14:31 |
| LiangFang | :) | 14:31 |
| rosmaita | os-brick support has been merged into victoria | 14:31 |
| rosmaita | the cinder extra-specs patch looks good | 14:32 |
| rosmaita | could you give us a quick idea of where the nova changes and CI stuff stand? | 14:32 |
| LiangFang | the UT of nova patch previously blocked by os-brick, but now it should work | 14:33 |
| LiangFang | but i need to change os-brick from 3.1.0 to 4.0.0 | 14:33 |
| LiangFang | in nova | 14:33 |
| rosmaita | what are the odds of the nova side stuff merging in victoria? | 14:34 |
| LiangFang | I have not talked too much with them recently, long time ago, they said want to see storage patch go first. | 14:35 |
| rosmaita | right, that makes sense | 14:35 |
| rosmaita | smcginnis: jungleboyj: i will need your feedback on what i am about to say | 14:36 |
| rosmaita | i think it makes sense not to hold up the extra-specs support | 14:36 |
| rosmaita | but also we will not make a big deal about it | 14:36 |
| rosmaita | because it doesn't actually do anything useful yet | 14:37 |
| jungleboyj | I think I agree with what you have said. :-) | 14:38 |
| rosmaita | ok, and i think hemna agrees too (he just +2d the patch) | 14:38 |
| jungleboyj | :-) | 14:39 |
| rosmaita | ok, and LiangFang we will expect documentation for this in wallaby | 14:39 |
| rosmaita | and then we can announce it as a real feature once the nova side is complete in wallaby | 14:39 |
| jungleboyj | ++ | 14:39 |
| smcginnis | 👍 | 14:40 |
| LiangFang | I have finished doc and push the patch this week | 14:40 |
| jungleboyj | Cool. | 14:40 |
| LiangFang | because customer also need the doc | 14:40 |
| rosmaita | LiangFang: excellent | 14:40 |
| rosmaita | ok, so that is also useful info, that you actually have all this working somewhere | 14:41 |
| rosmaita | makes me feel better :) | 14:41 |
| rosmaita | #topic selection of forum submissions | 14:41 |
| *** openstack changes topic to "selection of forum submissions (Meeting topic: cinder)" | 14:41 | |
| LiangFang | customer want write-back mode | 14:41 |
| rosmaita | so we have gathered our brainstorming on this etherpad: | 14:41 |
| rosmaita | #link https://etherpad.opendev.org/p/2020-Wallaby-cinder-brainstorming | 14:41 |
| rosmaita | so it is now our difficult task to narrow that stuff down to 2 topics | 14:42 |
| rosmaita | mission accomplished! | 14:42 |
| lseki | lol | 14:42 |
| jungleboyj | :-) Done! | 14:42 |
| rosmaita | unless anyone absolutely hates one of those, i will submit both of those before the deatline | 14:42 |
| rosmaita | *deadline | 14:42 |
| jungleboyj | I think those are good topics. | 14:43 |
| rosmaita | works for me! | 14:43 |
| rosmaita | that was easy | 14:43 |
| rosmaita | #topic Should we be able to create a volume from a source with a different volume type? | 14:43 |
| *** openstack changes topic to "Should we be able to create a volume from a source with a different volume type? (Meeting topic: cinder)" | 14:43 | |
| rosmaita | #link https://bugs.launchpad.net/cinder/+bug/1874076 | 14:43 |
| openstack | Launchpad bug 1874076 in Cinder "create volume from source volume fails if different volume type" [Undecided,New] | 14:43 |
| rosmaita | walshh_: that's you | 14:43 |
| walshh_ | thanks | 14:44 |
| walshh_ | Just wondering if this functionality should be working | 14:44 |
| walshh_ | it is blocked in the scheduler for us | 14:44 |
| walshh_ | has anyone here information on this? | 14:44 |
| smcginnis | Does seem like it should be an implicit retype. Though that introduces a lot of tricky corner cases. | 14:44 |
| jungleboyj | I think that is working as designed. | 14:44 |
| smcginnis | Yeah, I think that is intentional. | 14:45 |
| walshh_ | so you should not be able to create a volume from a source using a different volume type? | 14:45 |
| smcginnis | Correct. | 14:45 |
| walshh_ | ok. that was easy :-) | 14:45 |
| jungleboyj | Right, in case the volume is of that type for a reason. | 14:45 |
| smcginnis | Workaround would be to create it as the same type, then retype. Or at least attempt to retype. | 14:45 |
| jungleboyj | ++ | 14:46 |
| walshh_ | thanks, i just wanted to check it wasn't just us | 14:46 |
| jungleboyj | Not just you. :-) | 14:46 |
| walshh_ | it is possible to choose a different volume type in the UI | 14:46 |
| walshh_ | should this be blocked? | 14:46 |
| enriquetaso | should we close bug 1874076 then? | 14:47 |
| openstack | bug 1874076 in Cinder "create volume from source volume fails if different volume type" [Undecided,New] https://launchpad.net/bugs/1874076 | 14:47 |
| jungleboyj | It seems that the UI should not allow selecting the destination type. | 14:48 |
| smcginnis | Yeah, that might be a horizon bug. | 14:49 |
| walshh_ | thanks. I will open a bug against horizon. | 14:49 |
| walshh_ | thanks for the clarification | 14:50 |
| jungleboyj | ++ | 14:50 |
| jungleboyj | Thanks walshh_ | 14:50 |
| rosmaita | i think the problem is that they are just making a general volume-create call, and don't have built-in logic to sort out what parameters can be successfully combined in the call | 14:50 |
| rosmaita | our api doesn't help much beyond rejecting your request | 14:51 |
| walshh_ | it doesn't get beyond the scheduler | 14:51 |
| rosmaita | anything else about this topic? | 14:52 |
| jungleboyj | So, I think the original bug can be closed as Working as Designed. | 14:53 |
| *** rfolco is now known as rfolco|ruck | 14:53 | |
| rosmaita | #topic open discussion | 14:54 |
| *** openstack changes topic to "open discussion (Meeting topic: cinder)" | 14:54 | |
| *** jadonn has joined #openstack-meeting-alt | 14:54 | |
| rosmaita | ok, don't forget, after this meeting, discussion in #openstack-cinder about whether the "new" fix for ceph octopus should go into 4.0.1 | 14:55 |
| rosmaita | (for os-brick) | 14:55 |
| tosky | as part of the "move to focal" goal, a few base nodesets are going to switch soon; testing has been done, but still, keep an eye on the jobs in the next soon | 14:55 |
| tosky | as part of the "native zuulv3" goal, there is just one job pending, which is the trickiest to adapt, but I will come out with a solution before rc | 14:56 |
| tosky | EOL | 14:56 |
| rosmaita | tosky: ++ | 14:56 |
| tosky | uhm, rather EOF | 14:56 |
| rosmaita | thanks for all your work on the zuulv3 nativity | 14:56 |
| rosmaita | 1.5 minutes | 14:58 |
| rosmaita | ok, thanks everyone ... please review anything that needs it in https://blueprints.launchpad.net/cinder/victoria | 14:59 |
| rosmaita | that is the highest priority for reviewing | 14:59 |
| rosmaita | #endmeeting | 14:59 |
| *** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/" | 14:59 | |
| openstack | Meeting ended Wed Sep 9 14:59:50 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 14:59 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/cinder/2020/cinder.2020-09-09-14.00.html | 14:59 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/cinder/2020/cinder.2020-09-09-14.00.txt | 14:59 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/cinder/2020/cinder.2020-09-09-14.00.log.html | 14:59 |
| e0ne | #startmeeting horizon | 15:01 |
| openstack | Meeting started Wed Sep 9 15:01:14 2020 UTC and is due to finish in 60 minutes. The chair is e0ne. Information about MeetBot at http://wiki.debian.org/MeetBot. | 15:01 |
| openstack | Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. | 15:01 |
| *** openstack changes topic to " (Meeting topic: horizon)" | 15:01 | |
| openstack | The meeting name has been set to 'horizon' | 15:01 |
| vishalmanchanda | hi | 15:01 |
| tmazur | o/ | 15:01 |
| rdopiera | o/ | 15:01 |
| jadonn | 0/ | 15:01 |
| *** lkoranda has joined #openstack-meeting-alt | 15:03 | |
| e0ne | ok, let's start | 15:04 |
| e0ne | I hope amotoki will join us later | 15:04 |
| *** slaweq has quit IRC | 15:04 | |
| e0ne | #topic Notices | 15:04 |
| *** openstack changes topic to "Notices (Meeting topic: horizon)" | 15:04 | |
| amotoki | hi, just working on a document :) | 15:04 |
| e0ne | :) | 15:04 |
| e0ne | so, we're reaching victoria-3 milestone this week | 15:05 |
| e0ne | #link https://releases.openstack.org/victoria/schedule.html | 15:05 |
| e0ne | it's also Soft StringFreeze | 15:05 |
| e0ne | "You are no longer allowed to accept proposed changes containing modifications in user-facing strings. Such changes should be rejected by the review team and postponed until the next series development opens (which should happen when RC1 is published)." | 15:05 |
| e0ne | please, be careful with approval any changes with user-facing strings | 15:06 |
| e0ne | some usual reminders about PTG and Summit | 15:08 |
| e0ne | Horizon Virtual PTG planning etherpad | 15:08 |
| e0ne | #link https://etherpad.opendev.org/p/horizon-w-ptg | 15:08 |
| e0ne | feel free to add topics you would like to discuss | 15:09 |
| e0ne | general PTG details and registration information could be found here: | 15:09 |
| e0ne | #link https://www.openstack.org/ptg | 15:09 |
| e0ne | Virtual Open Infrastructure summit details are available: | 15:10 |
| e0ne | #link https://www.openstack.org/summit | 15:10 |
| e0ne | please, don't forget to register for PTG and Summit if you're going to attend them | 15:10 |
| e0ne | that's all updates from me | 15:11 |
| *** LiangFang has quit IRC | 15:11 | |
| *** tsmith2 has joined #openstack-meeting-alt | 15:11 | |
| e0ne | #topic victoria-3 milestone (feature freeze) | 15:11 |
| *** openstack changes topic to "victoria-3 milestone (feature freeze) (Meeting topic: horizon)" | 15:11 | |
| e0ne | amotoki: thanks for work on policy-in-code | 15:12 |
| e0ne | #link https://review.opendev.org/#/c/750134/ | 15:12 |
| *** lpetrut has joined #openstack-meeting-alt | 15:12 | |
| e0ne | amotoki: I agree with you that it's better to have it in early in Wallaby release | 15:12 |
| amotoki | related to this, I checked if the existing policy rules in horizon match back-end services. | 15:13 |
| amotoki | https://etherpad.opendev.org/p/horizon-victoria-policy-check | 15:13 |
| e0ne | I'm feeling OK to merge such changes just before FF except cases when we really need such patches | 15:13 |
| amotoki | the check is based on policy-in-code patch, so I don't know the check is based on the current horizon codebase. | 15:13 |
| amotoki | the current horizon codebase == the current policy.json files in horizon | 15:14 |
| e0ne | technically, the patch contains pretty small code changes but it could break anything | 15:16 |
| amotoki | e0ne: yeah, that's true | 15:17 |
| e0ne | amotoki: did you check that it doesn't break horizon upgrades? | 15:17 |
| * e0ne just notices that there is no release note | 15:17 | |
| amotoki | e0ne: I haven't checked the upgrade. | 15:17 |
| amotoki | at least I confirmed it works even when the current policy.json files exist | 15:18 |
| e0ne | I propose to test this patch before putting +1/+2 on it and merge it once Wallaby cycle starts | 15:18 |
| amotoki | agree. sounds fair and steady | 15:19 |
| -amotoki- is aware of missing release note | 15:19 | |
| e0ne | :) | 15:19 |
| e0ne | #agreed to move policy-in-code to the next cycle | 15:20 |
| e0ne | #topic victoria-3 milestone (feature freeze): Error messages refactoring | 15:20 |
| *** aning has quit IRC | 15:20 | |
| *** openstack changes topic to "victoria-3 milestone (feature freeze): Error messages refactoring (Meeting topic: horizon)" | 15:20 | |
| *** tsmith2 has quit IRC | 15:20 | |
| e0ne | #link https://review.opendev.org/#/q/topic:bp/refactor-error-messages+(status:open+OR+status:merged) | 15:21 |
| e0ne | there is only one patch left for horizon: | 15:21 |
| e0ne | #link https://review.opendev.org/#/c/749739/ | 15:21 |
| e0ne | and some clean up patches to plugins | 15:21 |
| *** aning has joined #openstack-meeting-alt | 15:22 | |
| e0ne | amotoki, rdopiera: please, review this patch ^^ | 15:22 |
| *** tsmith2 has joined #openstack-meeting-alt | 15:22 | |
| e0ne | I forgot to mention that Feature Freeze is tomorrow | 15:22 |
| *** jhesketh has joined #openstack-meeting-alt | 15:23 | |
| e0ne | vishalmanchanda: you did a great job with this feature during this release | 15:23 |
| amotoki | e0ne: I thought this follow-up is a bug fix while it is nice to land it earilier. | 15:23 |
| vishalmanchanda | e0ne: thanks for helping me with JS Hack:) | 15:24 |
| e0ne | amotoki: technically, you're right | 15:24 |
| *** aning_ has joined #openstack-meeting-alt | 15:25 | |
| *** tsmith_ has joined #openstack-meeting-alt | 15:25 | |
| e0ne | amotoki: but since it contains "Partially-Implements blueprint" text in the commit message I would like to treat it as a minor feature and force to merge it before FF | 15:25 |
| e0ne | amotoki: of course, you can disagree with me :) | 15:26 |
| amotoki | e0ne: one question on the patch is whether .....fadeIn(100) is assigned to "this_alert" and you call .find() method on it. | 15:26 |
| *** aning has quit IRC | 15:26 | |
| *** tsmith2 has quit IRC | 15:27 | |
| *** tsmith_ is now known as tsmith2 | 15:27 | |
| amotoki | e0ne: I will add my comment after the meeting. I haven't checked the detail code behind this but you may know more. | 15:27 |
| e0ne | amotoki: sorry, could you please point me into the code with 'fadeIn(100)' call | 15:28 |
| e0ne | ? | 15:28 |
| amotoki | https://review.opendev.org/#/c/749739/3/horizon/static/horizon/js/horizon.messages.js | 15:28 |
| amotoki | L.50 and L.56 | 15:28 |
| amotoki | I will add my comment anyway | 15:28 |
| e0ne | oh.. I've got what do you mean | 15:29 |
| e0ne | fadeIn returns jQuery instance | 15:30 |
| e0ne | amotoki: https://api.jquery.com/fadeIn/ | 15:30 |
| amotoki | e0ne: thanks | 15:30 |
| e0ne | in our case, it will be some html element | 15:31 |
| e0ne | amotoki: np | 15:31 |
| *** zzzeek has quit IRC | 15:31 | |
| *** zzzeek has joined #openstack-meeting-alt | 15:32 | |
| e0ne | let's move on | 15:32 |
| e0ne | #topic victoria-3 milestone (feature freeze): Cinder user messages | 15:32 |
| *** openstack changes topic to "victoria-3 milestone (feature freeze): Cinder user messages (Meeting topic: horizon)" | 15:32 | |
| e0ne | #link https://review.opendev.org/#/c/734161/ | 15:32 |
| vishalmanchanda | I have to add some U.T test for this patch. | 15:33 |
| amotoki | I made several clean up comments yesterday and most of them were minor and fixed. | 15:34 |
| amotoki | the remaining thing is the new code has no test coverage. | 15:34 |
| e0ne | I didn't look on the lates patch-set. will do it after the meeting | 15:34 |
| vishalmanchanda | I will try to push it by tomorrow. Let's see if we also get this feature in this cycle. | 15:34 |
| amotoki | I am okay with either. UT in this patch or in a follow-up patch | 15:35 |
| e0ne | vishalmanchanda: is the only reason to not merge it is test coverage, you can ask for FFE tomorrow | 15:35 |
| vishalmanchanda | amotoki: It have some test to check "message-list" at least. | 15:35 |
| vishalmanchanda | amotoki: ok I will add U.T in follow-up patch. | 15:36 |
| vishalmanchanda | will update the patch after the meeting. | 15:36 |
| amotoki | vishalmanchanda: precisely speaking, the API code has coverage. no coverage for panel implementation. | 15:36 |
| vishalmanchanda | amotoki: yeah as there is no url I have added in this change so I didn't able to do much thing with res. | 15:36 |
| vishalmanchanda | amotoki: I tried to do something like this https://review.opendev.org/#/c/734161/10/openstack_dashboard/dashboards/project/volumes/tests.py@1506 but it didn't work out | 15:37 |
| *** zzzeek has quit IRC | 15:37 | |
| *** zzzeek has joined #openstack-meeting-alt | 15:38 | |
| amotoki | vishalmanchanda: I don't have a good answer right now in the meeting. | 15:40 |
| *** belmoreira has quit IRC | 15:40 | |
| vishalmanchanda | amotoki: e0ne : If it's fine for both of you I can add these U.T in a follow-up patch. | 15:41 |
| e0ne | I'm OK with it | 15:42 |
| *** zzzeek has quit IRC | 15:42 | |
| amotoki | as mentioned above, I am okay | 15:42 |
| vishalmanchanda | e0ne: amotoki thanks. then I will update it after the meeting. | 15:43 |
| e0ne | #topic victoria-3 milestone (feature freeze): Add flatten_volume_from_snapshot config option | 15:44 |
| *** openstack changes topic to "victoria-3 milestone (feature freeze): Add flatten_volume_from_snapshot config option (Meeting topic: horizon)" | 15:44 | |
| e0ne | #link https://review.opendev.org/#/c/741990/5 | 15:44 |
| e0ne | vishalmanchanda, amotoki: thanks for your reviews | 15:45 |
| e0ne | I'll ask Oleksiy to update this patch tomorrow | 15:45 |
| amotoki | e0ne: is "flatten" a common word in cinder? | 15:45 |
| *** zzzeek has joined #openstack-meeting-alt | 15:45 | |
| e0ne | he used cinder-specific tems which are not obvious for everybody | 15:45 |
| e0ne | amotoki: yes, in case of RBD and maybe few other drivers | 15:46 |
| vishalmanchanda | e0ne: yeah first I am also confused then I refer this document https://docs.openstack.org/cinder/ussuri/drivers.html#rbddriver | 15:46 |
| e0ne | we'll try to find a better name for this | 15:47 |
| amotoki | I see. I am okay to use "flatten" if we have a good description. | 15:47 |
| amotoki | the other thing from me is about the test name. | 15:47 |
| e0ne | my main question is: shoud we treat this patch as a feature or as a test fix? | 15:47 |
| e0ne | it adds a new config option but only for integration tests config | 15:47 |
| amotoki | I think it is a fix or a workaround in tests | 15:47 |
| amotoki | not a feature | 15:48 |
| e0ne | amotoki: ok, so we can merge it after FF and before RC | 15:48 |
| amotoki | we don't touch any feature in our main code. it touches only the test codes, so I think it is not a target of FF. | 15:48 |
| amotoki | e0ne: exactly. that's my thought. | 15:48 |
| e0ne | amotoki: got it. it was not clear to my, so I added it for today's discussion | 15:49 |
| amotoki | e0ne: thanks | 15:49 |
| e0ne | amotoki: np | 15:49 |
| e0ne | that's all I've found in gerrit as a horizon feature proposal | 15:49 |
| *** zzzeek has quit IRC | 15:50 | |
| *** zzzeek has joined #openstack-meeting-alt | 15:51 | |
| e0ne | if you know about any other feature in progress, just let us know | 15:51 |
| e0ne | #topic victoria-3 milestone: horizon beta-1 release | 15:51 |
| *** openstack changes topic to "victoria-3 milestone: horizon beta-1 release (Meeting topic: horizon)" | 15:51 | |
| amotoki | e0ne: I have a topic around https://review.opendev.org/#/c/743600/ in on-demand agenda. | 15:52 |
| amotoki | go ahead this topic first | 15:52 |
| e0ne | it's not mandatory anymore but | 15:52 |
| e0ne | I prefer to get this release to make sure that plugins are not broken | 15:53 |
| e0ne | but I'm OK if we decide to skip beta release | 15:53 |
| amotoki | it is a good idea to cut a release. | 15:54 |
| amotoki | our last release was shipped about three months ago now | 15:54 |
| amotoki | precisely speaking, it is not a beta release though. we adopt the intermediary release model. | 15:54 |
| e0ne | ok, let's do it tomorrow once vishalmanchanda's patches will be merged | 15:54 |
| e0ne | amotoki: you're right | 15:55 |
| e0ne | my point is: it's not a release candidate and not a final release for victoria | 15:55 |
| amotoki | e0ne: yeah, i know it too | 15:56 |
| *** zzzeek has quit IRC | 15:56 | |
| e0ne | amotoki: ok, let's move to your topic | 15:56 |
| e0ne | #topic Open Discussion | 15:56 |
| *** openstack changes topic to "Open Discussion (Meeting topic: horizon)" | 15:56 | |
| amotoki | https://review.opendev.org/#/c/743600/ proposed a revert ot OPENSTACK_NOVA_EXTENSIONS_BLACKLIST deprecation | 15:57 |
| e0ne | we've got 5 minutes left but we can continue in #openstack-horizon channel | 15:57 |
| amotoki | the discussion turns out what we want is just SimpleTenantUsage in the setting. | 15:57 |
| amotoki | so I think it is better to introduce a new setting | 15:57 |
| amotoki | I just pushed a draft of the document to clarify it https://review.opendev.org/#/c/750693/1/doc/source/configuration/settings.rst | 15:58 |
| amotoki | we have two ways now | 15:58 |
| amotoki | the one is to clarify OPENSTACK_NOVA_EXTENSIONS_BLACKLIST will not be dropped until a new setting is implemented | 15:58 |
| amotoki | the other is to add a new setting in this release | 15:58 |
| amotoki | both would be simple | 15:59 |
| amotoki | I would like to know your opinions | 15:59 |
| *** lpetrut has quit IRC | 15:59 | |
| amotoki | we run out of time, so please give me feedback in #-horizon channel. | 15:59 |
| *** zzzeek has joined #openstack-meeting-alt | 15:59 | |
| e0ne | let's continue in #openstack-horizon | 15:59 |
| e0ne | #endmeeting | 16:00 |
| *** openstack changes topic to "OpenStack Meetings || https://wiki.openstack.org/wiki/Meetings/" | 16:00 | |
| openstack | Meeting ended Wed Sep 9 16:00:04 2020 UTC. Information about MeetBot at http://wiki.debian.org/MeetBot . (v 0.1.4) | 16:00 |
| openstack | Minutes: http://eavesdrop.openstack.org/meetings/horizon/2020/horizon.2020-09-09-15.01.html | 16:00 |
| openstack | Minutes (text): http://eavesdrop.openstack.org/meetings/horizon/2020/horizon.2020-09-09-15.01.txt | 16:00 |
| openstack | Log: http://eavesdrop.openstack.org/meetings/horizon/2020/horizon.2020-09-09-15.01.log.html | 16:00 |
| *** zzzeek has quit IRC | 16:04 | |
| *** zzzeek has joined #openstack-meeting-alt | 16:05 | |
| *** rosmaita has left #openstack-meeting-alt | 16:05 | |
| *** rdopiera has quit IRC | 16:13 | |
| *** dosaboy has quit IRC | 16:51 | |
| *** slaweq has joined #openstack-meeting-alt | 16:57 | |
| *** derekh has quit IRC | 17:00 | |
| *** andrebeltrami has quit IRC | 17:06 | |
| *** slaweq has quit IRC | 17:12 | |
| *** gyee has joined #openstack-meeting-alt | 17:33 | |
| *** belmoreira has joined #openstack-meeting-alt | 17:35 | |
| *** e0ne has quit IRC | 17:37 | |
| *** ralonsoh has quit IRC | 18:05 | |
| *** zzzeek has quit IRC | 18:33 | |
| *** zzzeek has joined #openstack-meeting-alt | 18:34 | |
| *** zzzeek has quit IRC | 18:41 | |
| *** zzzeek has joined #openstack-meeting-alt | 18:42 | |
| *** zzzeek has quit IRC | 18:50 | |
| *** zzzeek has joined #openstack-meeting-alt | 18:50 | |
| *** zzzeek has quit IRC | 18:55 | |
| *** zzzeek has joined #openstack-meeting-alt | 18:56 | |
| *** priteau has quit IRC | 18:57 | |
| *** zzzeek has quit IRC | 19:03 | |
| *** zzzeek has joined #openstack-meeting-alt | 19:05 | |
| *** apetrich has quit IRC | 19:11 | |
| *** slaweq has joined #openstack-meeting-alt | 19:17 | |
| *** zzzeek has quit IRC | 19:21 | |
| *** zzzeek has joined #openstack-meeting-alt | 19:23 | |
| *** kaisers_ has quit IRC | 19:27 | |
| *** tosky has quit IRC | 19:37 | |
| *** vishalmanchanda has quit IRC | 19:45 | |
| *** slaweq has quit IRC | 20:51 | |
| *** whoami-rajat__ has quit IRC | 20:52 | |
| *** slaweq has joined #openstack-meeting-alt | 20:55 | |
| *** slaweq has quit IRC | 20:59 | |
| *** rfolco|ruck has quit IRC | 21:03 | |
| *** slaweq has joined #openstack-meeting-alt | 21:26 | |
| *** slaweq has quit IRC | 21:31 | |
| *** jtomasek has quit IRC | 21:56 | |
| *** raildo has quit IRC | 22:22 | |
| *** vishakha has quit IRC | 22:26 | |
| *** tmazur has quit IRC | 22:29 | |
| *** rcernin has joined #openstack-meeting-alt | 22:55 | |
| *** rcernin has quit IRC | 22:56 | |
| *** rcernin has joined #openstack-meeting-alt | 22:56 | |
| *** enriquetaso has quit IRC | 23:13 | |
| *** zzzeek has quit IRC | 23:17 | |
| *** zzzeek has joined #openstack-meeting-alt | 23:19 | |
| *** dosaboy has joined #openstack-meeting-alt | 23:28 | |
| *** zzzeek has quit IRC | 23:38 | |
| *** zzzeek has joined #openstack-meeting-alt | 23:40 | |
| *** zzzeek has quit IRC | 23:59 | |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!