15:01:05 #startmeeting horizon 15:01:06 Meeting started Wed May 22 15:01:05 2019 UTC and is due to finish in 60 minutes. The chair is e0ne. Information about MeetBot at http://wiki.debian.org/MeetBot. 15:01:07 Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 15:01:09 The meeting name has been set to 'horizon' 15:01:35 hi 15:01:45 is anybody around for horizon meeting today? 15:02:18 hi 15:02:28 hi 15:02:54 I'm from 99cloud Inc,my name is Duochao Wu 15:03:22 let's wait few minutes more if people join us 15:03:28 the first part of the auto refresh patch came 15:03:42 hi all 15:03:58 #topic Notices 15:04:15 #link https://releases.openstack.org/train/schedule.html 15:04:29 we're two weeks before Train-1 milestone 15:04:50 nothing special for horizon at this point 15:05:34 but I'm going to finally finish mox removing patch until that date and release horizon for train to check if it affects any plugin 15:05:53 that's all updates from me 15:05:56 e0ne: great 15:07:15 I would like to clarify the milestones on ini-based-configuration along with train stations (i.e. milestones) but it can be discussed later. 15:07:19 #topic Open Discussion 15:07:42 amotoki: I just opened your patches before the meeting 15:08:26 #link https://review.opendev.org/#/q/status:open+project:openstack/horizon+branch:master+topic:bp/ini-based-configuration 15:08:51 I think we can start with the planning document https://review.opendev.org/#/c/656181/ 15:09:03 amotoki: I'll read once more time https://review.opendev.org/#/c/656181/ right after the meeting and +2 on it 15:09:07 amotoki: e0ne : any opinion on this bug https://bugs.launchpad.net/horizon/+bug/1829461 15:09:08 Launchpad bug 1829461 in OpenStack Dashboard (Horizon) "Volume Group Under project panel not working " [Undecided,Confirmed] 15:09:25 let's discuss topics one by one 15:09:30 amotoki: +1 15:10:13 my current proposed patches are all related to step 1 in the planning document. 15:10:45 I will resolve merge conflicts once parent patches land. 15:11:10 I think it would be nice if one or two patches are ready for review. 15:11:14 amotoki: I'm ok with the proposal. just want to confirm that I didn't miss anything before I +2 on it 15:11:40 hopefully vishalmanchanda or other cores can sponsor this effort. 15:11:54 at now, e0ne is the only sponsor for it. 15:12:05 amotoki: sure. 15:12:42 I guess this effort will have more than 20 patches :) and :p 15:12:43 ini-based (oslo-based config) is one of the most asked thing by operators 15:13:50 amotoki: we'd got a great experience in the past with django2 support patches :) 15:13:53 they might be expecting more from it than is feasible 15:15:58 rdopiera: who knows... 15:18:44 rdopiera: do you have more concrete examples? I haven't got your point fully. 15:19:20 I know they expect a lot by this word :) 15:19:35 I was just thinking that people might expect this to solve all their problems related to configuration 15:19:59 hehe :p 15:20:27 the only way to solve configuration issues is to remove config:) 15:20:30 amotoki: looking at your patches, I wonder if we want a helper function for definiting the settings and their defaults instead 15:20:54 amotoki: something like "setting("NAME", "Description", default)" 15:21:05 this way we can easily swap it later 15:21:36 under the hood it can just set the variables like the current approach does, for now 15:22:29 rdopiera: yeah, good point. I thought this can be done as part of oslo.config migration but I am not sure it is better to be a separate step. 15:23:24 we need to merge "description" of settings and our document on settings at some time. 15:24:35 a separate step is fine for me 15:24:47 having the defaults in one place is already a big help 15:24:57 and we can do the rest automaticlaly, I think 15:25:01 rdopiera: +1 15:25:04 rdopiera: +1 15:25:14 amotoki: I left few comments in your patch 15:25:44 e0ne: thanks. 15:26:35 nothing critical, so I put +1 15:26:39 I agree my plan has several (or many) discussion points and I hope my document helps clarify our understandng. 15:27:16 I think that many of those details don't matter, as long as we are consistent 15:28:28 but having it all captured in a document surely helps, and also makes it easier to cooperate 15:29:04 amotoki: if you have a step that could be done in parallel, I'm happy to help 15:29:49 rdopiera: thanks. I would like to focus on the first step to define defaults in one place first. As of now, any reviews and comments would be appreciated. 15:31:02 Advices and ideas on how to handle settings on django and its ecosystem are welcome too :) 15:31:13 (it can be tackled as second step) 15:34:06 amotoki: thanks for this effort 15:34:36 vishalmanchanda: what do you want to discuss about https://bugs.launchpad.net/horizon/+bug/1829461? 15:34:37 Launchpad bug 1829461 in OpenStack Dashboard (Horizon) "Volume Group Under project panel not working " [Undecided,Confirmed] 15:36:30 e0ne: just wanted to confirm this bug, which i think you already confirmed. 15:36:58 I reproduced it but didn't look into the details what went wrong 15:37:20 e0ne: same here. 15:38:05 regarding bugs, I think we need to discuss how we can address important bugs **as a team**. 15:38:24 (it is also important to discuss specific bugs though) 15:39:14 we have 103 medium or higher bugs 15:39:46 and most of them have not been visit for long. 15:39:47 I think we can use this https://etherpad.openstack.org/p/horizon-release-priorities etherpad to track important bugs 15:40:31 e0ne: does it mean we have two places to track bugs? 15:40:37 no 15:41:11 launchpad should be the main source of truth for bugs 15:42:08 but if we need to have some place to have a list of most critical bugs to discuss on meeting - we can use etherpad or separate tag on launchpad 15:43:40 perhaps High priority bugs are such ones if the priority works. 15:43:52 makes sense for me 15:44:40 we've got 4 open high bugs and 3 more in progress 15:45:26 actually, 2 in progress and 1 has fix commited status 15:45:52 but 3 "in progress" bugs have no progress for a long time :( 15:46:39 :( 15:46:44 The status of 2 in progress bugs should be revisit as the proposed fixes have been abandoned. 15:47:36 I moved the to the confirmed status again 15:47:54 i will try to proposed a fix for this bug https://bugs.launchpad.net/horizon/+bug/1746184 asap. 15:47:55 Launchpad bug 1746184 in OpenStack Dashboard (Horizon) "Table of networks or routers does't support pagination" [High,Confirmed] 15:47:58 we need to take care on bugfixing too 15:48:17 I tried to visit them but failed by myself alone... 15:48:40 I mean medium or higher bugs by "them". 15:48:49 we need to work together on this 15:49:28 we've got 10 minutes more 15:49:48 we can continue this discussion after the meeting on our channel or on the next meeting 15:50:17 Hanxiang and duochao: thanks for joining us. do you have anything to discuss? 15:50:36 the first part of the auto refresh patch came 15:50:46 Are you interested? 15:51:00 https://review.opendev.org/#/c/660612/ 15:51:14 #link https://review.opendev.org/#/c/660612/ 15:51:40 Implemented the common part of auto refresh, and added auto-refresh to the details page 15:51:52 I committed a partial functional implementation. 15:52:15 this says "part 1". Could you share the whole picture/plan on this? 15:53:33 amotoki: see this, https://blueprints.launchpad.net/horizon/+spec/auto-refresh :) 15:54:48 Hanxiang_: thanks. If so, the blueprint needs to be linked from your patch(es) as recommended in the OpenStack contributor guide. this makes our reviewer life easier. 15:56:13 amotoki:ok,thanks for your suggestion 15:57:11 https://docs.openstack.org/infra/manual/developers.html#development-workflow would help you 15:57:22 especially ""Starting a Change" section 15:58:30 we're almost out of time 15:58:50 thanks everybody for your contributions! see you next week 15:59:13 e0ne: although it is still in debug, you can review it first and make good suggestions. 15:59:48 Modify patch and resubmit will come late. 16:00:24 let's move to #openstack-horizon if we've got anything more to discuss today 16:00:26 #endmeeting