Wednesday, 2018-07-18

*** openstackgerrit has quit IRC05:10
*** aspiers has quit IRC05:16
*** portdirect has quit IRC05:16
*** aspiers has joined #openstack-loci05:35
*** portdirect has joined #openstack-loci05:35
*** ispp has joined #openstack-loci07:00
*** e0ne has joined #openstack-loci08:08
*** pbourke has quit IRC08:52
*** pbourke has joined #openstack-loci08:52
*** openstackgerrit has joined #openstack-loci10:20
openstackgerritPaul Bourke (pbourke) proposed openstack/loci master: [WIP] Add a profile for kolla nova  https://review.openstack.org/58354210:20
openstackgerritPaul Bourke (pbourke) proposed openstack/loci master: [WIP] Add a profile for kolla nova  https://review.openstack.org/58354210:26
openstackgerritPaul Bourke (pbourke) proposed openstack/loci master: [WIP] Add a profile for kolla neutron  https://review.openstack.org/58354510:32
pbourkeSamYaple: the kolla profiles are working :)11:51
pbourkeSamYaple: would you be ok with pushing these images as part of the gate?11:52
portdirectpbourke: how do the above work?11:53
pbourkeportdirect: take keystone for example. you build the loci image with the (kolla keystone) profile, which adds the packages kolla-ansible expects to be in the image for a functional deploy. Then I build one more image off of that which adds the kolla start scripts11:55
pbourkeportdirect: the code to do that is currently under https://github.com/brk3/kolla-loci11:55
hrwelo11:55
hrwbuilding 'requirements' image on aarch64 is fun11:56
hrwdropped '--no-deps' to check how many packages are arch specific. so far 3611:56
pbourkeif the start scripts were in kolla-ansible the kolla-loci piece wouldn't be required at all. But I met resistence with the former approach so this allows us to do it without touching kolla-ansible at all11:56
portdirectVery nice :)11:57
portdirectI'll check it out when I get into the office11:57
pbourkethanks!11:58
pbourkeportdirect: its a little funky to build right now as you have to first build the patched loci base + loci image, then finally the kolla-loci child. But if we can publish them as part of the gate it will be much simpler11:59
portdirectWould be good to get your shim in os-infra, would make me feel a lot more comfortable re merge&publish12:02
portdirectMaybe LOKI? (Loci Optimised Kolla images?) :D12:04
pbourkemaybe :)13:01
pbourkeI guess I just need to figure out what point I want to get it to before applying for addition13:04
openstackgerritPaul Bourke (pbourke) proposed openstack/loci master: Add gate job for rabbitmq  https://review.openstack.org/58357513:29
hrwlooks like about 10% of pip packages used by requirements image is architecture specific. so far 42/42313:35
SamYaplehrw: can you expand on what you mean?13:44
SamYapleim not quite following. are you saying that some of the packaging in upper-constraints.txt can't be built on aarch64?13:45
hrwSamYaple: *-linux_aarch64.whl ones. but they build13:45
SamYapleright, but im not getting why you need to drop `--no-deps`13:46
hrwSamYaple: sorry, was just experimenting13:46
SamYapleoh youre welcome to experiment, im just hoping i can save you some time if i understand what you are working toward13:47
hrwSamYaple: ;)13:47
SamYapleabout 10% of packages need to be compiled, the others are saved in thier native whl format (previously we pruned native wheels, but stopped for now, i can provide details of why if desired)13:48
SamYaplebut unless a specific package *wont* build on aarch64, the rest should work the same way13:48
SamYaplesame number of packages between all arches need to be compiled13:48
SamYaplewe don't pull compiled wheels from anywhere for any arch, we build them13:49
hrwSamYaple: we go Debian. there are no fresh enough binary packages for Debian/stable (even with backports)13:51
hrwops. replied to old message - had backlog moved too much13:51
hrwSamYaple: ok. so the only blocker for me now is that scikit-learn failing without numpy13:53
SamYaplecool. I would suggest, as a test, adding numpy after bindep on this line. https://github.com/openstack/loci/blob/master/scripts/requirements.sh#L2213:54
SamYaple*if* that works for you, i have some options on how to best solve it :)13:54
SamYapleso make that change and rebuild requirements and report back if it works13:54
hrwSamYaple: thanks13:56
hrwSamYaple: started build. the only changes from origin/master are 'pip install numpy' and '-P1' for clear logs13:58
hrwSamYaple: just moved it after install_packages.sh so toolchain is present ;D14:00
SamYapleheh fair enough :)14:03
SamYaplethe idea is to get numpy installed in the build image *before* we start building upper-constraints.txt14:03
SamYapleit looked liek scikit-learn was importing numpy in setup.py14:04
openstackgerritPaul Bourke (pbourke) proposed openstack/loci master: Add mariadb to infra profile  https://review.openstack.org/58358414:08
hrwSamYaple: looks like14:08
* hrw lunch14:22
isppHi, quick question, why u only create master images? Wouldn't make sense to create master images and at least the last stable?15:05
SamYaplehey ispp, you keep popping in and out before I can give you an answer :)15:13
SamYapleso the main reason for that is we dont have any testing of the deployment of these images yet15:13
SamYapleadditionally, we decided on a versioning structure that isn't tied to the stable branches15:14
SamYaplethe master code can build newton, ocata, pike, queens, and rocky right now15:14
SamYapleThe images we publish are meant for breif testing and not production use. You should always be building your own containers from trusted sources for production15:15
isppSamYaple thanks for the answer, yep, I have to configure it better to avoid this problems hehe, aham, ok I didn't know you were only using them for a testing environment, sometime is possible that master will be broken and the images too so, would it be possible if I propose a change in loci to include at least stable one or you want to continue basing the images for only testing?15:18
SamYapleYou are welcome to propose a patch for anything you think seems good15:20
SamYapleWe are pretty accepting of new things15:21
isppOh, great, thanks, I will try to find some easy fix for this :)15:27
*** ispp has quit IRC15:41
hrwSamYaple: added scipy as well. same scikit-learn16:45
SamYaple:+1:16:47
*** e0ne has quit IRC16:51
*** e0ne has joined #openstack-loci17:39
hrwok. scipy requires several dev packages.17:44
hrwhttps://pastebin.com/GcyZ6BpE17:44
*** e0ne has quit IRC17:44
*** e0ne has joined #openstack-loci17:48
*** e0ne has quit IRC17:51
SamYaplehrw: im sure this is a "loci expects X because all distros have X, but the aarch64 packages arent providing X"18:02
SamYapleweve seen similiar issues, but with distro instead of aarch18:02
hrwpossible18:05
hrwok, added libblas-dev liblapack-dev18:10
*** e0ne has joined #openstack-loci18:10
*** e0ne has quit IRC18:17
SamYaplehopefully when you get it built, youll be able to describe the many rough edges. most of us are just numb to them and dont see them anymore :)18:27
portdirecti find the splinters keep me warm18:27
hrwSamYaple: I lost track how many times I was dealing with blas/lapack ;D18:28
SamYapleportdirect: this assumes you are on fire to being with. so it checks out18:29
hrw  Running setup.py bdist_wheel for scipy: still running...18:31
hrwprogress was made ;D18:31
hrwshould I take care also of centos package names?18:31
SamYaplecan you confirm that this is an issue outside of aarch64?18:36
SamYaplei would like to track down why we are seeing different things between the arches18:36
hrwwill run build on x86-6418:43
SamYapleif there are differences in architcture packages needed, bindep supports that18:51
SamYapleotherwise if it fails the same way, throw up a patch and well merge it :)18:52
SamYaplesingle line patch to a central file is really easy to review :D18:52
hrwyep18:58
hrwhm. my fedora box is on docker 1.13ish and misbehave19:00
hrw20:59 (1s) hrw@puchatek:loci$ docker build   --build-arg PROJECT=requirements --build-arg FROM=debian:stretch  --tag requirements:master-debian  .19:00
hrwSending build context to Docker daemon   340 kB19:00
hrwStep 1/23 : ARG FROM=ubuntu:xenial19:00
hrwPlease provide a source image with `from` prior to commit19:00
hrwhave to update19:06
hrwSuccessfully tagged requirements:master-debian19:37
hrwon x86-6419:37
hrwno changes19:38
*** e0ne has joined #openstack-loci19:39
SamYaplethen i would suggest adding the deps with the aarch64 bindep profile19:43
SamYapleso as not to add deps to the x86_64 that arent needed19:44
SamYaplehrw: alternatively, `apt-get install python-numpy` should also work, if you want to *not* compile it like that19:45
SamYapleyou could include it to the requirements build for aarch64 and it *might* work?19:45
*** e0ne has quit IRC19:52
hrwSamYaple: what is a format for arch specific bindep?19:57
hrwwill check19:58
hrwbtw - libxml2 should be removed from bindep.txt imho. it is a bad taste to depend on library package instead of something which depends on that library20:00
hrwscikit-learn in progress...20:00
SamYaplehrw: libxml2 is literally needed everywhere since the xml api isn't completely gone20:00
SamYaplei would love to remove it20:00
hrwSamYaple: but you use either binaries using libxml2 or python binary modules which use libxml220:01
SamYapleportdirect: did we decide to do a seperate bindep.txt for arch specific things?20:01
SamYapleits been a while since we discussed arch support20:02
SamYaplehrw: while yes, we could specify `libxml2 [keystone nova glance cinder neutron etc etc etc]` it is much easier to do []20:02
portdirectSamYaple: i cant rememeber, we kicked around a few ideas20:03
SamYaplewhen openstack as a whole drops libxml2 we can remove it entirely20:03
portdirectbut not sure what we ended up thinking20:03
portdirectone thing for sure - we prob need to break out the bindep file in some way20:03
SamYapleok. lets give it some more thinking20:03
portdirectas its ending up a bit of a monster20:04
portdirecteg: the health-check stuff we want/need in osh and pauls kolla work, along with arch support20:04
portdirectprob mean its time20:04
hrwSuccessfully tagged requirements:master-debian20:08
hrwon aarch6420:08
SamYaplehrw: w00t!20:08
SamYaplenow build keystone!20:08
hrwnow split changes and send for review20:10
hrw--build-arg WHEELS=requirements:master-debian - right?20:17
openstackgerritMarcin Juszkiewicz proposed openstack/loci master: bindep: install blas and lapack headers to build scipy on aarch64  https://review.openstack.org/58372620:18
openstackgerritMarcin Juszkiewicz proposed openstack/loci master: requirements.sh: install numpy and scipy on AArch64  https://review.openstack.org/58372720:18
SamYaplehrw: if you are trying to do that, you have to upload the wheel somewhere20:20
SamYaplea local registry or dockerhub works20:20
hrwthanks20:20
hrwhttps://hub.docker.com/r/haerwu/loci-requirements/20:23
* hrw off20:37
hrwthanks for help guys and see you tomorrow20:37

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!