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