14:59:20 <kumarmn> #startmeeting trove 14:59:21 <openstack> Meeting started Wed Jan 17 14:59:20 2018 UTC and is due to finish in 60 minutes. The chair is kumarmn. Information about MeetBot at http://wiki.debian.org/MeetBot. 14:59:23 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote. 14:59:25 <openstack> The meeting name has been set to 'trove' 14:59:58 <fanzhang> Hi folks 15:00:24 <kumarmn> #topic rollcall 15:00:37 <Yao_> hi 15:00:45 <fanzhang> o/ 15:00:52 <songjian> o/ 15:00:56 <zhanggang> hi 15:03:06 <kumarmn> We have enough for a quorum. 15:03:10 <kumarmn> #topic queens-3 15:03:25 <kumarmn> I was going to tag later this week. Would like to get the currency items in. 15:03:57 <kumarmn> Redis 3.2: https://review.openstack.org/#/c/534179/ 15:04:23 <kumarmn> Postgres 9.6: https://review.openstack.org/#/c/534667/ 15:05:08 <kumarmn> Couchbase 3.0: https://review.openstack.org/#/c/407125/ 15:05:17 <kumarmn> Can we get these reviewed this week? 15:05:22 <fanzhang> About redis, is this redis rpm from official repo? 15:05:35 <songjian> Redis3.2 hopes delayed, because it involves the choice of fedora version, hoping to fully discuss the PTG 15:05:45 <fanzhang> About pg, zhanggang is working on it, and thanks for that. 15:06:15 <kumarmn> I saw zhanggang has put in a new review. Should I abandon the change that Doug Shelley originated? 15:06:27 <zhanggang> As commit message show, I haven't test 9.6 on fedora 15:06:31 <songjian> Not sure, but I'll investigate, just found from rpm find this site 15:08:18 <fanzhang> kumarmn: IMHO, I think we can focus on zhanggang's patch 15:09:10 <kumarmn> sounds good. Can we get it tested this week? Or do we defer merging? 15:09:14 <fanzhang> songjian: I suggest that all the rpm packages should be from official repos :) 15:09:33 <songjian> ok,will check 15:10:22 <zhanggang> I assume that we doesn't support pg<9.4 anymore, maybe this need discuss. 15:10:56 <kumarmn> That is fine for me, i.e. to drop older releases. 15:11:23 <fanzhang> should we drop support for older releases in all databases? 15:12:06 <zhanggang> OK I will test it this week 15:12:37 <kumarmn> Thanks zhanggang. I would also urge all cores to review these changes this week, so that we can get them merged. 15:12:47 <kumarmn> It sounds like Redis 3.2 might slip. 15:12:50 <fanzhang> Will do 15:13:49 <songjian> Do we have a cross-version migration of databases? I remember seeing similar features. If yes, we can remove older versions of support and reduce code maintenance 15:15:40 <fanzhang> I vote for not supporting pg<9.4 because it's not that popular, we may not need to support more than one version of pg 15:16:24 <songjian> On fedora, I hope it is able to determine a higher version before starting the database installation script. I have written this to erthpad 15:16:29 <fanzhang> but when it come to some much more popular databases like mysql, redis, and mongodb, it may need different strategies 15:17:02 <zhanggang> Agree, drop support for older releases make the code slim. 15:18:53 <fanzhang> kumarmn: what do you think, we can discuss that on PTG remote meetings. Again, will review these concurrency related patches. 15:19:02 <Yao_> Agree with Fan, we should keep older versions for popular databases 15:19:52 <kumarmn> Yes, sounds good. 15:20:26 <songjian> agree 15:20:51 <kumarmn> #topic experimental jobs 15:21:11 <fanzhang> Thanks, will do some investigations on that 15:21:12 <kumarmn> To get these other databases stable, we need to be running all the experimental jobs. 15:21:20 <kumarmn> They have been failing for a while. 15:21:32 <kumarmn> Can someone take a look at why they are failing? 15:21:55 <AJaeger> I suggest to merge https://review.openstack.org/#/c/534666/ first - that way the jobs are under trove's control and you can easily fix them. 15:22:09 <fanzhang> Yes, that's the one. 15:22:57 <fanzhang> Oh, sorry, I think you mean this one https://review.openstack.org/#/c/533137/ ? 15:23:17 <AJaeger> fanzhang: yes, that one. Sorry for wrong pasto 15:24:24 <AJaeger> 533137 is a 1:1 copy from the infra setup. It fails the same way as the old jobs ;) 15:24:31 <fanzhang> As I mentioned in the comments, I hope we can merge it first. As for make experimental jobs actually work, may need a while, I am afraid that it won't be tagged into Queen 15:25:08 <fanzhang> I can help fix them :) kumarmn 15:25:23 <kumarmn> That will be great fanzhang. Yes later is fine. 15:25:47 <fanzhang> cool. 15:26:36 <Yao_> 👏 15:27:48 <kumarmn> I want to get to a point say during Rocky that we start running them regularly, make it part of the gate, and eventually remove the experimental tag from the guestagents. 15:28:11 <kumarmn> But one step at a time. 15:28:22 <fanzhang> kumarmn: these jobs are non-voting right now, right? 15:28:47 <kumarmn> Right now, I believe they are not even running. 15:29:17 <fanzhang> LOL, it can be one big steop during Rocky :) 15:29:25 <fanzhang> s/steop/step 15:29:32 <AJaeger> fanzhang: they are implicit non-voting - an experimental job is only run if you ask for it - and then they do not vote at all. 15:30:05 <fanzhang> AJaeger: thanks for notice 15:30:33 <kumarmn> Those are the only topics I wanted to cover. 15:30:41 <kumarmn> Let me know if there is anything else. 15:31:46 <fanzhang> I want to discuss about some patches like these two :https://review.openstack.org/#/c/534707/ and 15:32:21 <fanzhang> https://review.openstack.org/#/c/534669/ 15:32:49 <songjian> I am watching too 15:33:05 <fanzhang> I am not saying that they should commit these patches, but what should we deal with similar patches in the furture? 15:33:08 <fanzhang> future 15:33:15 <AJaeger> fanzhang: for 534707 I would look at nova as reference - see http://git.openstack.org/cgit/openstack/nova/tree/api-ref/source/diagnostics.inc 15:33:16 <fanzhang> kumarmn: 15:33:28 <AJaeger> So, order is fine but why not add names? 15:35:16 <fanzhang> sounds great :), I never thought about it! 15:35:48 <songjian> Agree with this approach 15:35:50 <AJaeger> nova is for api-ref the example I always look at 15:36:25 <fanzhang> would you please leave some review comments for that patch? 15:36:27 <zhanggang> Ajaeger,I notice that the commit didn't add postgreql, will you add it in the future? 15:40:17 <fanzhang> zhanggang: AJaeger you mean the zuul job patch ? 15:41:43 <zhanggang> yes. 15:41:54 <zhanggang> https://review.openstack.org/#/c/533137/ 15:44:36 <fanzhang> AJaeger: hi, still around ? 15:47:10 <kumarmn> Perhaps Posgres did not have a job, even earlier. I would have dig into it a bit. 15:47:39 <fanzhang> :-) cool 15:48:48 <kumarmn> #topic more reviewers 15:49:21 <kumarmn> Yao__ and zhanggang, you have been dropping a lot of changes. Are you able to review more as well? 15:49:37 <Yao_> will do 15:49:44 <zhanggang> yes of course 15:51:27 <kumarmn> Great, thanks! 15:52:22 <fanzhang> So are they new cores right now? GREAT! 15:52:51 <fanzhang> Congrats 👍🏻 15:56:36 <AJaeger> fanzhang: back again... 15:57:00 <AJaeger> fanzhang: if something is missing, tell me in the change and I'll update. I thought it included full list.. 15:57:22 <kumarmn> They found postgre missing 15:57:37 <AJaeger> fanzhang: postgresql is in check/gate - no need for experimental 15:57:47 <AJaeger> and it's working ;) 15:58:09 <fanzhang> oh, thanks :) 15:58:57 <kumarmn> We are almost at the top of the hour. Any other topics? 15:59:31 <fanzhang> none, have a great day:) I am going to bed 15:59:53 <songjian> nothing from me 16:00:09 <Yao_> nothing from me too 16:00:16 <kumarmn> #endmeeting