15:01:34 <egonzalez> #startmeeting kolla
15:01:35 <openstack> Meeting started Wed Mar  6 15:01:34 2019 UTC and is due to finish in 60 minutes.  The chair is egonzalez. Information about MeetBot at http://wiki.debian.org/MeetBot.
15:01:36 <openstack> Useful Commands: #action #agreed #help #info #idea #link #topic #startvote.
15:01:38 <openstack> The meeting name has been set to 'kolla'
15:01:44 <egonzalez> #topic rollcall
15:01:49 <hrw> o/
15:01:49 <chason> o/
15:01:49 <mnasiadka> o/
15:04:19 <egonzalez> #topic Announcements
15:04:23 <egonzalez> Hi
15:04:37 <egonzalez> Until 12th is self nomination for PTL elections
15:05:02 <egonzalez> Ensure to propose yourself if some of you wish to become PTL
15:05:25 <egonzalez> This time i´ve decided to not run for PTL, unless no other runs for it
15:05:55 <hrw> I do not plan to run
15:06:03 <egonzalez> let me know if need some help or guidance
15:07:06 <egonzalez> any other announcement?
15:10:46 <egonzalez> #topic placement-api split https://review.openstack.org/#/q/topic:split-placement+(status:open+OR+status:merged)
15:11:04 <egonzalez> patch is ready to review
15:12:06 <egonzalez> please add to your proritity reviews list as is a release blocker
15:14:03 <hrw> +2
15:14:26 <hrw> https://review.openstack.org/#/c/625261/ needs rebase
15:16:48 <egonzalez> any question to the placement api split?
15:17:38 <hrw> I see it as 'upstream nova did, we need to comply' change
15:18:11 <hrw> looked at k-a patch, looks like nova-placement-api things moved to placement-api + variables change
15:18:25 <egonzalez> yep, if we dont do it, we're broken for stein
15:18:33 <egonzalez> yep, mostly that
15:18:43 <egonzalez> except the database migration
15:20:29 <hrw> ok
15:22:25 <egonzalez> #topic open discussion
15:22:51 <egonzalez> something else wanna discuss today?
15:23:05 <hrw> Python 3
15:23:22 <hrw> I mailed openstack-discuss about it on Thursday
15:24:28 <egonzalez> yep, go ahead hrw
15:25:00 <hrw> I think that it is now rather Train than Stein material as changes may break stuff
15:26:03 <hrw> we failed to do that in Stein and it is too late to change now. the good part is that kolla itself is py2/py3 safe itself
15:26:33 <hrw> with Debian/Ubuntu moved to py3 in binary packages (mostly) we can do same for source images
15:26:34 <egonzalez> yep, except ubuntu binary which already running some things on py3
15:26:51 <hrw> egonzalez: ubuntu/binary is still rocky in stein iirc
15:27:00 <egonzalez> yep, thats the issue
15:27:12 <egonzalez> if we qwant to use stein need to change some packages to py3
15:27:51 <egonzalez> thats what this change does https://review.openstack.org/#/c/625298/
15:28:07 <hrw> yep
15:29:18 <egonzalez> but requires some changes in ansible to not break deployment https://review.openstack.org/#/c/627954/
15:30:39 <hrw> how many things will explode if we merge both now?
15:30:48 <egonzalez> likely everything
15:31:00 <hrw> and freeze is in ~2 weeks
15:31:04 <egonzalez> but hopefully not the core
15:31:34 <mnasiadka> so maybe that's the best time to explode :)
15:31:34 <egonzalez> chason is having problems to build the images in order to test ansible deployment because the china firewall
15:31:59 <egonzalez> do we merge now, and tomorrow images will be built and published to dockerhub?
15:32:20 <egonzalez> i prefer to do this today, than the day before feature freeze tbh
15:32:28 <hrw> sound good enough for me
15:32:29 <chason> egonzalez agree ^^^
15:33:06 <chason> I can pull these images from dockerhub.
15:33:19 <egonzalez> already gave my +2
15:33:46 <hrw> +2+w
15:33:54 <hrw> let hell gates open
15:34:03 <egonzalez> be aware that ubuntu binary ansible gates will be broken until the ansible patch is merged
15:34:16 <hrw> will then push change to macros.j2 to get rid of pip_version argument in install_pip macro
15:34:33 <hrw> it is not needed change which just complicated macro
15:34:33 <egonzalez> ok, sounds good
15:34:55 <hrw> one image used it
15:36:03 <hrw> have to check which images (other than kolla-toolbox/helm-repository) install pip at all when built as binary
15:37:08 <egonzalez> chason hopefully tomorrow gate builds works and you can pull the images
15:38:37 <chason> :D
15:38:55 <egonzalez> anything else missing for this topic?
15:39:02 <hrw> binary images installing pip from repos (helm-repository, openstack-base, telegraf) may rquire check for pip/pip3 thing
15:39:40 <egonzalez> hrm, helm is probably not used anywhere, we may need to ask for deprecation
15:39:42 <hrw> gnocchi-base installs python3-pip
15:39:53 <PrinzElvis> hi all, just want to "HI", I'm in Berlin at OPS https://etherpad.openstack.org/p/BER-ops-meetup
15:40:26 <hrw> and it is the only user of pip_version argument
15:40:29 <hrw> so let it stay
15:41:06 <hrw> we may need to check how to get cradox from repos instead
15:41:45 <egonzalez> i recall there was some issues when moving to ceph luminous, but dont recall exacly what was
15:41:57 <egonzalez> ^^ regarding cradox
15:42:17 <hrw> egonzalez: gcc <7 in Debian was blocker for mimic
15:42:37 <egonzalez> PrinzElvis have fun in there, give us feedback about kolla if something is discussed
15:42:38 <hrw> both Debian and Ubuntu lack cradox in repos
15:43:23 <hrw> so pip_version stays in install_pip macro. just for one image ;D
15:43:59 <PrinzElvis> egonzalez: yep 😊
15:46:06 <egonzalez> will likely change in train release ;)
15:46:16 <hrw> kolla-toolbox image requires rewriting to be more package based for binary images but that's other story
15:46:33 <hrw> egonzalez: hope not.
15:46:50 <hrw> egonzalez: I hope for 'pip' being always pip not 'pip or pip3'
15:47:11 <hrw> as name does not change if python under change from 2 to 3
15:48:21 <egonzalez> yes, agree on that
15:49:09 <hrw> so only lack of cradox forces us to have pip<>pip3 handling
15:50:25 * hrw done
15:50:33 <egonzalez> hrw thanks
15:50:41 <egonzalez> anything else missing to discuss today?
15:51:12 <hrw> egonzalez: can you reply to my comment on https://review.openstack.org/557659 (arm64 ci)?
15:52:00 <hrw> as I do not quite understand that CI configs yet
15:52:19 <egonzalez> i dont understand your comment
15:52:43 <egonzalez> you mean to left as it is now and later move to code or move it to code now?
15:52:55 <egonzalez> by code i mean https://github.com/openstack/kolla/blob/master/kolla/image/build.py#L137
15:54:01 <hrw> egonzalez: we may reuse 'debian+source' profile (L177) for now maybe?
15:54:30 <hrw> egonzalez: would add broken/missing images there in separate patch
15:54:40 <egonzalez> ok
15:54:44 <hrw> thx
15:54:54 <egonzalez> remove the profile later and move broken images there
15:56:18 <hrw> so we merge as it is and then expand build.py#L177 debian+source, right?
15:56:24 <hrw> sorry to ask again
15:56:33 <egonzalez> is in merge conflict
15:56:47 <hrw> will rebase
15:57:32 <egonzalez> ok, thanks
15:57:41 <egonzalez> anything else?
15:57:59 * hrw done
15:58:35 <egonzalez> lets end the meeting, thanks you for coming today
15:58:42 <egonzalez> #endmeeting