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