16:00:16 <inc0> #startmeeting kolla
16:00:17 <openstack> Meeting started Wed Nov 22 16:00:16 2017 UTC and is due to finish in 60 minutes.  The chair is inc0. Information about MeetBot at http://wiki.debian.org/MeetBot.
16:00:18 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
16:00:21 <openstack> The meeting name has been set to 'kolla'
16:00:30 <inc0> #topic woot!
16:00:32 <inc0> good morning
16:00:48 <duonghq> wo0t
16:00:50 <spsurya__> morning  w00t
16:00:56 <egonzalez90> Woot
16:00:57 * inc0 is sleepy :(
16:01:05 <Jeffrey4l> woot
16:02:07 <sdake> o/
16:02:12 <vhosakot> o/
16:02:24 <duonghq> \o o/ \o/
16:03:20 <inc0> ok let's gett this going
16:03:24 <inc0> #topic announcements
16:03:36 <inc0> happy thanksgiving to our US folk
16:03:46 <gema> happy thanksgiving everyone!
16:03:52 <inc0> anyone wants to add any announcement?:)
16:04:01 <vhosakot> turkey day all!
16:04:42 <vhosakot> https://1funny.com/wp-content/uploads/2010/11/thanksgiving_funny_picture_07.jpg
16:05:08 <inc0> :)
16:05:16 <inc0> soo we don't have agenda
16:05:24 <duonghq> again
16:05:27 <inc0> yeah
16:05:29 <vhosakot> inc0: the Sydney sessions went well.  Sorry I missed last week's meeting.
16:05:34 <inc0> it's slow last couple of weeks
16:05:39 <inc0> no worries
16:05:45 <inc0> I'm glad, I watched videos
16:05:50 <vhosakot> cool
16:06:02 <inc0> let me do one thing before we go to open discussion
16:06:07 <inc0> #topic gate updates
16:06:29 <inc0> soo some of you might've missed, but we're not using tarballs.o.o any more
16:06:40 <inc0> I mean we're in process of getting rid of them
16:06:57 <inc0> now kolla gates builds images, publish to dockerhub itself
16:07:03 <inc0> and kolla gates uses dockerhub
16:07:10 <inc0> + caching proxy in nodepools
16:07:15 <gema> inc0: where do images end?
16:07:48 <inc0> https://hub.docker.com/u/kolla/
16:07:53 <gema> cool, thanks
16:08:00 <egonzalez90> Im not seeing all images pushed, only source and not all of them
16:08:07 <inc0> right
16:08:13 <inc0> we use only source in gates
16:08:22 <inc0> and we don't use all images in ansible gates:)
16:08:40 <Jeffrey4l> inc0, 1. should we push all images?  2. should we push binary images?
16:08:56 <inc0> so reason I did what I did (limit images)
16:09:08 <inc0> is because 1. we want to conserve bandwidth
16:09:09 <Jeffrey4l> yes. i am un-limit this ;(
16:09:30 <inc0> 2. some images are often broken
16:09:51 <inc0> and that will affect arguably more important images
16:10:15 <inc0> but if you want to push all, I'm fine with it
16:10:51 <Jeffrey4l> i propose to push them. especially for stable branch. they are more stable ;D
16:11:18 <inc0> well both stable and master should push same set of images tbh
16:11:51 <Jeffrey4l> yes. so let us push them until we really meet issue ( like network or stability )
16:11:52 <hrw> o/
16:12:08 <inc0> ok
16:12:20 <chason> Sorry for late o/
16:12:28 <gema> inc0: you push on every commit?
16:12:37 <inc0> gema: no, it's once a day
16:12:41 <gema> inc0: ok, makes sense
16:12:50 <gema> then I am with Jeffrey4l , push them all
16:13:06 <Jeffrey4l> inc0, btw, i am trying to push the commit once a day + every commit.
16:13:23 <inc0> I think that's a bit overkill
16:13:25 <Jeffrey4l> push the image*
16:13:29 <gema> Jeffrey4l: that's too much
16:13:41 <Jeffrey4l> kolla repo is not changed too much
16:13:53 <gema> how many commits a day?
16:14:35 * Jeffrey4l is calculating
16:14:40 <gema> (ballpark figure, average)
16:15:14 <Jeffrey4l> i think less 1 per day recently.
16:15:26 <inc0> yeah, it's not changing that much
16:15:29 <gema> then once a day like inc0 is saing makes sense, no?
16:16:11 <inc0> it might be issue when we find out bug and our gates gets unblocked only after one day
16:16:29 <gema> inc0: cannot you make a manual push if needed?
16:16:33 <duonghq> I think we will have more and more commit in some rush case (like in the end of cycle when we fix bug)
16:16:36 <Jeffrey4l> inc0, yes.
16:16:49 <inc0> right duonghq is right
16:16:54 <gema> duonghq: yeah, that's why once a day makes sense
16:16:59 <inc0> and that will impact merge speed negatively
16:17:07 <gema> we don't want to clog infra during release rush
16:17:17 <Jeffrey4l> duonghq, if it is really two much. we can disable this. not it is just in-project jobs. very easy to control by ours.
16:17:25 <inc0> if we need to merge few things, very long publish gates, more prone to error too, will make it harder
16:17:50 <inc0> frankly? I'd keep it in periodic
16:17:58 <inc0> we also need to be careful about shared infra
16:18:03 <duonghq> I agreed with gema on we should have some mechanism for force push beside once a day policy
16:18:13 <inc0> it's not just our project that will get impacted
16:18:44 <gema> inc0: +1
16:19:03 <inc0> so let's keep it daily for now please
16:19:27 <inc0> if we need to push manually for whatever reason, we can quickly merge gate change
16:19:31 <inc0> as Jeffrey said
16:19:45 <inc0> and rollback if needed later
16:20:16 <Jeffrey4l> so daily push, without every commit push, right?
16:20:24 <inc0> yes
16:20:30 <inc0> I'd keep it this way
16:20:42 <inc0> that being said, we still need push for pike and ocata
16:20:46 <Jeffrey4l> and push all type of image?
16:21:15 <inc0> I'd be ok with that, I don't think it's needed, but sure, why not
16:22:18 <Jeffrey4l> btw, i am think we can implement that when commmit mesage container "push-the-image" string, then it push the image for that commit.
16:22:44 <inc0> that's be cool
16:22:56 <inc0> like new commit msg tag ForceImagePush
16:23:03 <Jeffrey4l> yeah. i will try this later.
16:23:19 <chason> Jeffrey4l good idea :p
16:23:41 <inc0> anything else on that topic?
16:24:01 <Jeffrey4l> no from me.
16:24:18 <inc0> #topic open discussion
16:24:20 <inc0> anyone
16:24:22 <inc0> ?
16:25:16 <inc0> guess not:)
16:25:20 <duonghq> ah
16:25:22 <duonghq> I have one
16:25:24 <inc0> shoot
16:25:26 <duonghq> spsurya__, ping
16:25:35 <spsurya__> inc0:  <spsurya> can we mark https://blueprints.launchpad.net/kolla-ansible/+spec/ks-rolling-upgrade-role     implemented
16:25:40 <spsurya__> duonghq: yeah
16:25:48 <inc0> if it's implemented:)
16:25:50 <inc0> sure
16:25:52 <inc0> great job!
16:26:16 <spsurya__> small downtime after containers restart
16:26:27 <spsurya__> so i added in description tab
16:26:31 <spsurya__> about that
16:26:42 <spsurya__> and will mark as implented
16:27:02 <spsurya__> duonghq: ^^
16:27:16 <duonghq> I agreed with spsurya__
16:27:23 <duonghq> but it's your bp, inc0
16:27:31 <duonghq> how about you?
16:27:46 <spsurya__> yeah that is why need inc0 view
16:27:59 <inc0> haha, I trust you guys completely
16:28:21 <inc0> also sub-second downtime on keystone won't even be visible
16:29:18 <duonghq> iirc, I conducted small benchmark and I can see a little dark here, I posted the result in one passed meeting
16:29:33 <hrw> or add 'forcepush' like we have with 'recheck'?
16:29:38 <inc0> I marked it completed
16:29:51 <inc0> hrw: well commit message not commit comment
16:30:04 <inc0> you can't push to hub from not-merged change
16:30:09 <inc0> secrets won't work
16:30:24 <hrw> inc0: kind of 'after merge comment' for core revs
16:30:42 <inc0> no, zuul doesn't have this mechanism
16:30:51 <inc0> but we might be able to parse commit message in gates
16:30:55 <hrw> anyway it goes through review anyway so it can be 'please add forcepush into commit msg' comment ;D
16:31:12 <inc0> right
16:31:39 <Jeffrey4l> btw here is the patch related to push images to dockerhub, please review https://review.openstack.org/519969
16:33:31 <hrw> gave +1 as looks ok for me but no idea much about infra
16:34:35 <Jeffrey4l> thanks.
16:34:44 <inc0> I'll review it later
16:35:04 <Jeffrey4l> okay
16:35:15 <inc0> anything else anyone?
16:35:54 <hrw> can someone look at my patches?
16:36:16 <hrw> 4 kolla, 1 kolla-ansible. 3 master, 2 backports...
16:36:38 <hrw> ;D
16:36:49 <spsurya__> hrw: sure
16:37:10 <hrw> or rather :8 (trying to imitate Cat from Shrek movie)
16:37:39 <inc0> ok everyone, let's wrap it up :)
16:37:47 <inc0> and give everyone 20min back
16:37:58 <gema> thank you guys! o/
16:38:02 <inc0> again, happy thanksgiving and thank you all for coming!
16:38:06 <inc0> #endmeeting kolla