Wednesday, 2015-08-12

*** jasonsb has quit IRC00:00
-openstackstatus- NOTICE: Zuul was restarted due to an error; events (such as approvals or new patchsets) since 23:01 UTC have been lost and affected changes will need to be rechecked00:04
*** diogogmt has quit IRC00:12
sdakeanyone know a way to print 19 spaces in python without pressing the space key 19 times00:16
sdakeand without using '%-19s'% ''00:17
bmaceyou could do it in a for loop? :)00:21
*** jruano has quit IRC00:23
*** bmace has quit IRC00:32
*** achanda_ has joined #kolla00:35
*** zhiwei has joined #kolla00:36
*** ccrouch has quit IRC00:37
*** achanda has quit IRC00:38
*** achanda_ has quit IRC00:39
*** thumpba has quit IRC00:42
*** bmace has joined #kolla00:52
*** thumpba has joined #kolla00:56
*** thumpba has quit IRC00:59
*** thumpba has joined #kolla01:08
*** thumpba_ has joined #kolla01:13
*** thumpba has quit IRC01:14
*** thumpba_ has quit IRC01:15
*** thumpba has joined #kolla01:15
sdakebeen working on diversity script a bit:01:20
sdakehttp://paste.openstack.org/show/412638/01:20
*** weiyu has joined #kolla01:20
sdakeif you want to play with it you can pull itfrom here https://review.openstack.org/21179901:21
*** wshao has joined #kolla01:23
*** diogogmt has joined #kolla01:28
*** britthouser has quit IRC01:30
*** erkules has joined #kolla01:32
*** jasonsb has joined #kolla01:34
*** erkules_ has quit IRC01:35
*** tobe_ has joined #kolla01:39
*** britthouser has joined #kolla01:39
*** jruano has joined #kolla01:40
*** erkules has quit IRC01:52
*** erkules has joined #kolla01:54
SamYapleheyo01:59
*** Haomeng|2 has joined #kolla02:00
*** Haomeng has quit IRC02:03
*** thumpba has quit IRC02:04
*** thumpba has joined #kolla02:05
*** thumpba has quit IRC02:05
*** Haomeng has joined #kolla02:15
*** Haomeng|2 has quit IRC02:15
*** unicell has quit IRC02:20
SamYaplesdake: https://review.openstack.org/#/c/202216/02:23
SamYaplepleez02:23
sdakesamyaple looking02:24
openstackgerritMerged stackforge/kolla: Improve logging and prepare for gate  https://review.openstack.org/20221602:29
*** jruano has quit IRC02:34
openstackgerritSam Yaple proposed stackforge/kolla: Add Dockerfile template for rabbitmq  https://review.openstack.org/21181702:51
SamYaplesdake: much cleaner, this dockerfile is02:51
*** dolpher has joined #kolla02:52
*** britthouser has quit IRC02:52
sdakelet me light up a kill stick and i'll have a look02:54
sdakesam can you do me a solid and put how to build the containers in the etherpad03:00
SamYaple./build.py --template ?03:00
sdakebecause I'm not really sure how to build templated containers03:00
sdakeand i'm sure everyone will ask03:00
SamYapleits literally in the help file03:00
sdakethat is it?03:00
SamYaple./build.py -h03:00
SamYapleyea03:00
*** dolpher1 has joined #kolla03:00
sdake-t rdo or -t source?03:01
SamYaplethose all work, but we havent been implementing the rdo tag03:01
SamYaplebinary and source are all we implemented03:01
sdakethat is good03:01
sdakeso -t binary?03:01
SamYaplewhich is default, but yes03:01
sdakehow do you select the distro base?03:02
SamYapleplease run ./build.py -h03:02
sdakei'll just looka t the source03:02
sdakei have to disconnec t to do that03:02
SamYaple:/03:02
*** ccrouch has joined #kolla03:02
SamYaplehelpfile is there for a reason03:02
sdakeya this vpn drives me crazy03:02
*** jruano has joined #kolla03:03
sdakeholy shit jpeleer grew facial hair03:03
SamYapleim getting google fiber in san antonio03:03
SamYaplefunny i am clean shaven at the moment03:03
sdakei've got fibre from cox coming end of month03:04
*** dolpher has quit IRC03:04
*** banix has quit IRC03:08
sdakedo we hae a working debian base image?03:12
sdakesamyaple ^^03:13
*** dolpher has joined #kolla03:13
SamYapletechincially it should work, yes03:14
SamYaplei dont think its been tested no03:14
*** tobe_ has quit IRC03:15
*** dolpher1 has quit IRC03:16
*** tobe_ has joined #kolla03:16
sdakesamyaple to speed up the process have a look at my instructions https://etherpad.openstack.org/p/kolla-dockerfile-template03:21
*** weiyu has quit IRC03:24
SamYapleid remove debian03:25
SamYaplealso the source deploy for ubuntu isnt reall large03:26
SamYaplereally only source+centos is needed03:26
SamYapleubuntu only has 2 or 3 services with "source"03:26
SamYaplealso fedora03:26
SamYaplei dont think fedora needs to be built and tested riht now03:27
*** achanda has joined #kolla03:27
SamYaplewe know its broke (hence the bug yo uahve in your name)03:27
*** weiyu has joined #kolla03:29
*** diogogmt has quit IRC03:29
sdakei'll fix fedora tonight03:31
sdakeas far as from source i'd like to make sure the builds work for ubuntu else it will create big rework later03:32
sdakebut ack on the lack of services03:32
SamYaplefunny thing is some of the ubuntu builds dont work right now03:32
SamYapleso im not so worried03:32
SamYapleI am about to go through all teh ubuntu container since i need them now03:33
SamYaplebut it can wait fro the templating to be dont03:33
SamYapledone03:33
sdakewhat is done03:34
sdakereconnecting bbiaf03:34
*** sdake_ has joined #kolla03:35
sdake_-T=1 is a cool feature03:38
sdake_oraclelinux03:39
*** sdake has quit IRC03:39
sdake_INFO:__main__:base:No package epel-release available.03:39
openstackgerritSwapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add ansible support for Cinder  https://review.openstack.org/20886803:41
sdake_ubuntu builds03:42
coolsvapsdake_, i might take a dig at ubuntu builds once i am finished with current work in hand03:43
coolsvapregarding the the updated images on dockerhub, i think it might be worth a try to assign individuals for specific builds03:44
sdake_debian fails to build03:44
sdake_coolsvap i am not suggesting pushing03:44
coolsvapmaybe have a bug or something at each milestone to track status03:44
sdake_just building03:44
sdake_i can make one script that pushes all the changes per milestone03:45
sdake_it will probably take 10 hours to run or more03:45
sdake_i should have caught that epel thing in review03:46
* sdake_ ughs03:46
coolsvapyes, i would still suggest to break it down since it will help us track status and a point person who can provide updates03:46
*** sdake_ is now known as sdake03:46
sdakei am not concerned about docker registry builds atm03:47
coolsvapbecause no one probably would track the build/push status for 10 hours03:47
sdakei have a rax account where i run the build03:47
sdakei dont want to disribute that job - its easy to put in one script and have one person care for it03:48
sdakeubuntu builds from source and from binary for rabbitmq03:49
sdakedebian is busted03:49
sdakecentos and ubuntu build03:56
sdakeINFO:__main__:base:Yum command has been deprecated, redirecting to '/usr/bin/dnf install -y https://rdo.fedorapeople.org/rdo-release.rpm'.03:58
* sdake shakes fists at fedora03:59
sdakesamyaple do you have a tidy solution for that problem?03:59
sdakedoen't hae to be now04:00
sdakejust before its removed04:00
*** jruano has quit IRC04:03
sdakeINFO:__main__:base:/usr/share/man/man5/cert8.db.5.gz: No such file or directory04:05
sdakecannot reconstruct rpm from disk files04:05
sdakeI don't even know what to make of that04:05
sdakeso here is the score, oraclelinux, fedora, and debian are broken04:06
sdakeonly ubuntu and centos work04:06
*** unicell has joined #kolla04:06
SamYaplesdake: yea, the solution is an alias04:09
SamYaplebut once the tools really start to diverge the options will start to change04:09
sdakewe use install and update04:10
sdakei guess they could change that04:10
sdakebut unlikely04:10
sdakeand clean all04:10
sdakethat could possibly change04:10
* sdake groans04:10
sdakethose idiots said they were going to keep the cli intact04:11
SamYaplethen why change the name ;)04:12
sdakeoraclelinux from source fails to build04:14
sdakeINFO:__main__:rabbitmq:/bin/sh: /usr/lib/rabbitmq/bin/rabbitmq-plugins: No such file or directory04:14
SamYaplei dont think oraclelinux has any from source stuff04:15
sdaketheir entire stuff is from source04:16
SamYaplehttps://github.com/stackforge/kolla/tree/master/docker/ol04:16
sdakeya i know04:16
SamYapletheres nothing there dude04:16
sdakei just saw04:16
sdakei wonder if they are only building centos or something04:17
openstackgerritSam Yaple proposed stackforge/kolla: Add Dockerfile template for rabbitmq  https://review.openstack.org/21183804:17
sdakeneed to ask pbourke04:17
openstackgerritSam Yaple proposed stackforge/kolla: Add Dockerfile template for keystone  https://review.openstack.org/21183804:17
SamYaplewrong name04:17
SamYapleonce more04:17
openstackgerritSam Yaple proposed stackforge/kolla: Add Dockerfile template for keystone  https://review.openstack.org/21183804:17
sdakebase file and keystone are inconsistent04:23
sdakeone is if distro if type04:23
sdakeother is if type if distro04:23
*** weiyu has quit IRC04:25
sdakeinfra is if distro04:25
sdakeis there a good reason for openstack services using if type if distro?04:25
SamYapleThe other way requires alot of duplicate?04:26
SamYaplethe source part is the same across distros04:26
sdakeok makes sense04:26
sdakewould have to have same if source block twice04:27
SamYapleyea04:27
sdakethanks for answering my q ;)04:27
SamYaplethe case is opposite for the base image since the depluicate is within the same distro rather than without04:27
sdakeya well i didn't actually implement the stuff so thats why i asked04:28
SamYaplenot a problem im just busy04:28
openstackgerritSam Yaple proposed stackforge/kolla: Add Dockerfile template for keystone  https://review.openstack.org/21183804:32
sdakesamyaple looks like you c aught my review comments04:33
sdakeover ESP04:33
SamYapleand you complain when you dont understand me04:33
sdakewhere does keystone.tar come from?04:43
sdakesamyaple ^^ trying to build yoru containers fails for me with keystone04:44
SamYapleyea the source installs dont work with the build.py script yet04:45
sdakeI can see why you recommended no testing earlier ;-)04:46
SamYapleyep04:47
sdakethat was sarcasm ;)04:47
sdakewhat needs to change for build.py to build the tarballs?04:47
sdakerun a script or something?04:47
SamYaplethe way the current build scripts do it is run the code in .buildconf04:48
SamYaplethats not teh way it should be done for the python code04:48
SamYaplebut no one has written the python code yet04:48
SamYapleor added what it should look like04:49
sdakeyou mean .buildinfo?04:49
SamYapleyea04:49
SamYaple.build* something04:49
sdakewhy should it not be done that way?04:49
SamYapleto get away from bash code and hidden files (the purpose f the new build script)04:49
SamYaplea conf file or the like was suggested04:50
sdakeexpand please04:50
sdakelike keystone.build.conf ?04:50
SamYapleif python reads the conf it would have all the info on how and where to clone/pull/cp the tarball and then the methods for the python code would need ot be written04:51
SamYapleno like a global one04:51
SamYaplewe are trying ot get away from each directory having config stuff04:51
sdakeok, so global one containing info for every package?04:51
SamYapleevery openstack service, yes04:52
*** tobe_ has quit IRC04:53
sdakeini format or yaml?04:53
SamYaplei dont mind yaml, however that does place a yaml requirement on the build server04:54
sdakeis ini built into python?04:54
SamYapleyea04:54
SamYapleconfigparser04:54
coolsvapSamYaple, sdake a question regarding templates, can we skip the ubuntu/debian related stuff in template if its not there?04:57
coolsvapwith a note to add it later04:57
SamYaplecoolsvap: yea you dont have to implement anything that doesnt already exist04:57
SamYaplejust keep the structre there04:57
sdakenothing exists for ubuntu right?04:57
* sdake can't keep up with the state of change of the repo :)04:57
SamYaplethere is stuff for ubuntu04:58
sdakelet me look second04:58
SamYapleactually coolsvap can you put 'RUN /bin/fasle' in there if it doesnt exist?04:58
SamYapleto force the container ot not build04:58
sdakeonly rabbitmq nova-base and keystone04:58
sdakeoh wrongdir04:58
sdakeno correct dir04:59
sdakethere are no services implemented from source for ubuntu04:59
sdakeand little from binary04:59
SamYaplekeystone04:59
coolsvapsdake, its on the list but not prioritized :-/04:59
SamYapleoh i guess its not04:59
sdakewhat is on the list coolsvap05:00
coolsvapadd build from source for ubuntu and test05:00
sdakeis there a blueprint for that?05:00
coolsvapno not yet, need to create one05:01
sdakei thought there was for some reason05:01
sdakeubuntu falls into the same category as rhel05:01
SamYaplehow so?05:01
SamYapleno licensing issues05:01
coolsvapthere was for ubuntu but its no longer active05:01
coolsvapi meant implemented05:02
sdakeit clearly wasn't implemented ;)05:02
coolsvapsdake, yes05:02
sdakegot a link05:02
sdakesamyaple meaning we do it after we do centos05:02
SamYaplesdake: im about to do ubuntu full time so that wil lchange in the next week or so05:02
SamYapleubuntu from source even05:03
sdakewfm05:03
SamYaplesince ubuntu has no 'delorean' type repo05:03
sdakecoolsvap plz link the blueprint and i'll pull it into l305:03
coolsvaphttps://blueprints.launchpad.net/kolla/+spec/install-from-ubuntu05:03
sdakeya policy i set unilaterally :) is if there is no binary link to from source05:03
SamYaplewhat?05:03
*** tobe_ has joined #kolla05:04
sdakewe will discuss at tomorrow meeting05:05
sdakewell that really bums me out i thought we had ubuntu sorted out05:07
SamYapleme too05:07
SamYaplei just hadnt looked05:07
sdakereally all we have sorted out is centos05:07
sdakecoolsvap do as sam suggested keep structure but run bin false05:09
* SamYaple crasks the whip05:09
coolsvapyes will do05:09
coolsvapsdake, i might take a dig at ubuntu once i am done with ceilometer-ansible and templating05:10
sdakeya dont overwork yourself :)05:10
sdakeso guys lets brainstorm ini format05:10
sdakewe need to get this fixed asap05:10
sdakebecause i want from source and binary at minimum tested on centos05:11
sdakeneed to keep parity with what we have05:11
sdakehttp://ur1.ca/neza0 -> http://paste.fedoraproject.org/254187/3562891405:11
sdakethere is the buildinfo05:11
*** weiyu has joined #kolla05:11
sdakehow is COMPONENT used there?05:12
sdakehow about05:14
coolsvapits mostly used for temp variable05:14
sdake[containername]05:14
sdakegit_master=05:14
sdakegit_stable=05:14
sdaketarball+master=05:14
SamYaplewhat?05:14
sdaketarball_stable=05:14
SamYaplewhy all those things?05:14
SamYaplewe need three things. tarball, git clone, raw copy from filesystem05:15
SamYaplewhere to download/clone/copy should be configurable05:15
SamYapleso pull_method and pull_source should be enough05:15
sdakebeen up since 5am05:15
SamYapleno problem05:15
SamYaplei have a few more templates to hammer out then ill handle that sdake05:16
*** Haomeng has quit IRC05:16
sdakenice05:16
sdakethis is a blocker for our work05:16
SamYaplewell its a blocker to remove the other builds stuff05:16
sdakeleave some of the easy templates for peopel in the community to learn on sam05:16
SamYaplebut as long as its there when the template work is done its fine05:16
SamYapleyea sdake its just ones i already started on05:16
SamYapleglance and mariadb05:16
SamYaplebefore your email05:17
sdakeoh ya thats fine05:17
SamYaplei just did keystone and rabbitmq cause yo uasked05:17
sdakeya mariadb would have been good in place of rabbitmq ;)05:17
sdakeI just randomly pickd that05:17
SamYapleeh its done05:18
SamYaplelots of RAX work at the moment05:18
sdakeyar05:19
coolsvapsdake, i have another approach to reduce the number of lines in ini and make it hierarchical05:20
sdakewhy do we need hiearcy05:20
sdakeonly the base image or 1 image will ever need a tarball I think05:21
coolsvapsdake, yes05:21
sdakeif there is a use case f or hiearcy lets hear it now because we can't use ini in that case05:21
coolsvapi think the git-clone urls are mostly same05:22
SamYaplecoolsvap: right but mostly the same != the same05:22
SamYapleim ok with a bit of duplication there05:22
sdakei dont want a complex format for config05:22
SamYapleyea05:22
SamYaplea static "here is where you get this file" is fine with me05:23
SamYapleany changes across the file is a quick sed command away05:23
coolsvapSamYaple, yes its not same always, i was just trying to put templatize the ini05:23
sdakecoolsvap interesting idea but lets not do that to our users :)05:24
sdakeits hard enough on devs to deal with all that05:24
sdakebut i expect devs to be able to handle it05:24
coolsvapsdake, it would be easy than the sed, just put explicity what you need, rest is done by default05:25
sdakethe prototype template files for dockerfiles are simpel enough for devs to understand but I dont know if a non dev would ge tit05:25
sdakeya overrides inheritence05:25
sdakei hear what your saying05:25
sdakedont' like :)05:25
coolsvaphmm05:26
coolsvapnp :)05:26
*** weiyu has quit IRC05:27
sdakesamyaple how do we select between git and uri?05:28
sdakeyet another cli option?05:29
sdakei think better might be a default section05:29
sdakewith a type field or something similar05:30
coolsvapsdake, https://github.com/stackforge/kolla/blob/master/tools/gen-source-tar.sh#L505:30
sdakelets try not to add more stuff to the cli if at all possible05:30
coolsvaphttps://github.com/stackforge/kolla/blob/master/tools/gen-source-tar.sh#L505:30
coolsvapsorry https://github.com/stackforge/kolla/blob/master/tools/gen-source-tar.sh#L3205:30
sdakeso its hard coded05:30
coolsvapyes05:34
sdakesamyaple for git we also need a branch I think05:38
sdakeor tag05:38
sdaketag05:38
sdakemaybe we can get that out of the uri05:38
sdakeok time to pass the foxtrot out05:39
sdakenight05:39
sdakebest wishes on that build..py work sam ;)05:39
sdakenight coolsvap05:39
*** akwasnie has joined #kolla05:47
*** weiyu has joined #kolla05:47
*** diga_ has joined #kolla05:50
coolsvapnight sdake05:50
coolsvapSamYaple, still around?06:04
coolsvaphave a look at http://paste.openstack.org/show/412651 just trying to sort out the template for ceilometer-base06:06
SamYaplecoolsvap: hey06:13
SamYaplein the source section you should have any if blocks06:13
SamYaplethe CMD shouldnt even be in a base image like this06:14
coolsvaphttp://paste.openstack.org/show/412652/06:17
coolsvapSamYaple, ^^06:18
SamYapleim not sure what youre asking me here06:18
coolsvapjust look once if this is correct06:19
SamYapleno06:19
SamYaplethere is no binary stuff06:20
SamYaplethe source stuff should work for everything06:20
SamYapleyou dont need to have bit/ture like that06:20
SamYaple/bin/true06:20
SamYaplewe can do this in a review06:20
coolsvapin ceilometer base there is nothing06:20
coolsvaphttps://github.com/stackforge/kolla/blob/master/docker/centos/binary/ceilometer/ceilometer-base/Dockerfile06:20
coolsvapotherwise i wouldnt have asked06:21
SamYapleright but doesnt that mean we just need to have one if source section?06:21
SamYapleman our images really need updating06:22
coolsvapyes, thats what i wanted to confirm, but then we will not have the ceilometer-base for binary06:22
coolsvapisnt it?06:23
*** echoingumesh has joined #kolla06:23
SamYapleno it needs to be there. but we _should_ be installing something like ceilometer-common in the base but dont worry about it for right now06:24
SamYaplehttp://paste.openstack.org/show/412653/06:25
SamYaplecoolsvap: ^06:25
*** unicell1 has joined #kolla06:28
*** unicell has quit IRC06:28
coolsvapSamYaple, alrighty06:29
*** echoingumesh has quit IRC06:30
sdakethe reaosn we have a ceilometer-base is to install ceilomter-config06:33
sdakebut that went away06:33
sdakecoolsvap06:33
SamYapleyea sdake but we should be installing the ceilometer-common package06:33
SamYapleotherwise we have really large images06:33
sdakemay possibly make sense06:34
SamYaplei really need to do the image tuning i did for yaodu on kolla06:34
SamYaplethose images were super tiny06:34
sdakejust explaining why it is seemingy empty :)06:34
SamYaplelike 1.2gb size across the whoe things06:34
openstackgerritSam Yaple proposed stackforge/kolla: Add Dockerfile template for mariadb  https://review.openstack.org/21186406:34
*** sdake_ has joined #kolla06:35
*** sdake has quit IRC06:39
openstackgerritSam Yaple proposed stackforge/kolla: Add Dockerfile template for glance  https://review.openstack.org/21187106:51
SamYaplejpeeler: do yo ureally live in SC?06:57
*** fangfenghua has joined #kolla07:00
*** inc0 has joined #kolla07:05
inc0good morning07:06
SamYaplemorning inc007:06
akwasniehi:)07:07
SamYaplehello akwasnie07:07
akwasnieSamYaple which service can I take for templates? Are u working on sth more than glance, mariadb, keystone and rabbit?07:08
SamYapleakwasnie: no those are all done now07:08
SamYaplethere is an etherpad wit ha list07:09
SamYapleakwasnie: https://etherpad.openstack.org/p/kolla-dockerfile-template07:09
SamYapleany one not claimed just throw your name beside it!07:09
*** diga_ has quit IRC07:09
akwasnieok07:10
openstackgerritSwapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add Dockerfile template for glance  https://review.openstack.org/21187807:29
SamYaplecoolsvap: glance? that looks like ceilometer to me!07:30
* coolsvap bad07:31
SamYaplecoolsvap: unfortunately i know there is no way you tested that07:31
SamYaplerabbitmq has the code needed to template teh FROM section07:31
*** achanda has quit IRC07:31
coolsvapSamYaple, yes the build.py update07:32
openstackgerritSam Yaple proposed stackforge/kolla: Add Dockerfile template for glance  https://review.openstack.org/21187107:35
*** achanda has joined #kolla07:36
*** jmccarthy has joined #kolla07:37
*** tobe_ has quit IRC07:44
*** jmccarthy has quit IRC07:46
*** achanda has quit IRC07:46
openstackgerritSwapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add Dockerfile template for Ceilometer  https://review.openstack.org/21187807:47
*** tobe_ has joined #kolla07:50
*** jmccarthy has joined #kolla08:00
*** tobe_ has quit IRC08:00
*** sdake_ has quit IRC08:04
*** tobe_ has joined #kolla08:05
*** diga_ has joined #kolla08:14
*** Haomeng has joined #kolla08:14
*** gfidente has joined #kolla08:33
*** tfukushima has joined #kolla08:34
openstackgerritSwapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add Dockerfile template for Ceilometer  https://review.openstack.org/21187808:40
*** jmccarthy has quit IRC09:02
*** vbel has quit IRC09:08
*** vbel has joined #kolla09:09
openstackgerritSwapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add Dockerfile template for Cinder  https://review.openstack.org/21191909:16
*** jmccarthy has joined #kolla09:21
openstackgerritMerged stackforge/kolla: Glance bootstrap container mounts wrong config folder  https://review.openstack.org/21158709:22
openstackgerritMerged stackforge/kolla: Add Dockerfile template for rabbitmq  https://review.openstack.org/21181709:26
openstackgerritMerged stackforge/kolla: Add Dockerfile template for mariadb  https://review.openstack.org/21186409:26
SamYaplepbourke: ping09:26
*** jmccarthy has quit IRC09:26
pbourkeSamYaple: morning09:26
SamYaplehows it go09:26
pbourkegood :)09:26
SamYaplegot some questions for oy09:27
pbourkewasnt me09:27
SamYapleim implementing the source config stuff in python build script09:27
SamYaplelies!09:27
SamYapleso because we are trying to remove all config-in-each-folder type stuff that leads to only one real option, a global config09:27
pbourkeok09:29
SamYaplethis is what i have: http://paste.openstack.org/show/412672/09:29
SamYaplecan you forsee any issues here?09:29
SamYaplefile or local should also be a type09:29
pbourkeoo09:30
pbourkenice09:30
pbourkeno that looks pretty nifty09:30
SamYapleok. This leaves the build.py to do the implementation09:30
SamYapleive got the tarball part done09:30
SamYaplebut the git part :/09:30
SamYaplemay just end up wrapping `git`09:31
pbourkeyou may not need dest_filename09:32
pbourkedepends on the implementation09:32
pbourkewhats the issue with git?09:32
SamYaplegit support with python09:32
SamYaplewithout external libs09:33
SamYaplethe dest_filename is needed because i use it to name the file in the final location09:33
SamYaplei really want to pull that information dynamically though09:34
SamYaplecan't quite figure it out yet09:34
pbourkecan you not use the section name?09:35
pbourkealso I would just subprocess.call the git command09:36
SamYapleah another caveat09:36
SamYaplethe section name will need to be the name of the image where i install it09:36
SamYaplelike cinder-base glance-base09:36
SamYaplesince i use the image name to fetch the configs09:36
pbourkecoolsvap: question on https://review.openstack.org/#/c/211919/1/docker_templates/cinder-app/cinder-scheduler/Dockerfile.j2 if you have time09:38
coolsvappbourke, sure09:39
pbourkeSamYaple: not following that last part09:39
coolsvappbourke, we are skipping building the images which are not available currently09:39
SamYaplepbourke: the /bin/false is becase the ubuntu stuff has no install09:39
*** fangfenghua has quit IRC09:41
pbourkewould something like this be better?09:43
pbourkeRUN echo '{{ base_distro }} not yet supported' && /bin/false09:43
SamYapleyea09:43
SamYapleas long as it breaks the image i dont care09:43
SamYapleor rather, i dont have an opinion09:44
SamYapleI was considering just catching /bin/false in the build.py script and having the failure type be 'unsupported'09:44
SamYaplesince the logs are eh09:44
openstackgerritAlicja Kwasniewska proposed stackforge/kolla: Add Dockerfile template for horizon  https://review.openstack.org/21193409:46
*** jmccarthy has joined #kolla09:46
*** dolpher has quit IRC09:50
*** erkules has quit IRC09:54
*** erkules has joined #kolla09:54
openstackgerritMerged stackforge/kolla: Add Dockerfile template for keystone  https://review.openstack.org/21183809:59
inc0SamYaple, neutron binary still not working right? we need to use source with your patch?09:59
*** zhiwei has quit IRC10:04
SamYapleinc0: it works for me now10:07
SamYaplewhats wrong with it?10:07
inc0I'll try when I'll build it10:07
inc0guys, do we want to make release a variable for source?10:17
*** gfidente has quit IRC10:18
inc0like downloading nova-{{ release }}.tar?10:18
SamYapleinc0: not with this new build.py script10:18
SamYaplethe conf file is simple10:18
SamYaplebut its all modifiable in one location10:19
SamYapleso a single thread10:19
SamYaplesing sed*10:19
SamYaplewhatever10:19
*** diga_ has quit IRC10:19
inc0so first sed all .buildconfs and replace master to whatever?10:19
SamYapleno10:20
SamYapleall thats going away10:20
inc0how do I deploy kilo for example?10:20
SamYaplewell from master that wont be supported10:20
inc0wait, I thought we agree it will10:21
inc0on midcycle10:21
SamYapleno we agreed people will do that10:21
SamYaplewe wont block it10:21
SamYaplebut its not an option in the kolla repo10:21
inc0I kinda have issues with that. especially if it's this simple10:21
SamYapleits not though10:22
openstackgerritSwapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add Dockerfile template for Ceilometer  https://review.openstack.org/21187810:22
*** fangfenghua has joined #kolla10:22
inc0building docker images is10:22
inc0deploying with ansible, not so, but that's just deployment10:22
inc0and not-so only with ansible generating configs10:23
inc0if you have working config, that's doable10:23
SamYapleright but nothing stops you from building images for kilo right now by tweaking hte buildinfo things10:24
SamYapleon the from source side10:24
openstackgerritSam Yaple proposed stackforge/kolla: Implement source fetching for build.py  https://review.openstack.org/21195410:31
SamYaplepbourke: https://review.openstack.org/21195410:31
SamYaplefeedback needed10:31
openstackgerritSwapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add Dockerfile template for Ceilometer  https://review.openstack.org/21187810:33
openstackgerritAlicja Kwasniewska proposed stackforge/kolla: Add Dockerfile template for horizon  https://review.openstack.org/21193410:35
*** jmccarthy has quit IRC10:35
openstackgerritSwapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add Dockerfile template for Cinder  https://review.openstack.org/21191910:38
pbourkelooking10:38
*** jmccarthy has joined #kolla10:59
*** tobe_ has quit IRC11:03
*** tobe_ has joined #kolla11:04
*** inc0_ has joined #kolla11:04
*** inc0 has quit IRC11:04
*** tobe_ has quit IRC11:09
*** Haomeng|2 has joined #kolla11:11
*** juggler has quit IRC11:13
pbourkeSamYaple: it works anyway11:15
*** Haomeng has quit IRC11:15
pbourkeand implementation looks clean11:15
SamYaplepbourke: yea but suggestions?11:17
*** inc0_ has quit IRC11:17
*** rhallisey has joined #kolla11:17
pbourkei dont think I have anything for you other than the usual like a little more error checking, maybe the ability to specify a custom build.ini, etc11:18
*** weiyu has quit IRC11:20
openstackgerritRyan Hallisey proposed stackforge/kolla: Glance-registry is missing a messaging config  https://review.openstack.org/21161611:20
*** fangfenghua has quit IRC11:20
SamYaplepbourke: yea i was going to do a cli option for an override config11:23
*** inc0 has joined #kolla11:29
*** tobe_ has joined #kolla11:33
*** inc0 has quit IRC11:35
*** tobe_ has quit IRC11:39
*** inc0 has joined #kolla11:43
*** inc0_ has joined #kolla11:44
*** gfidente has joined #kolla11:46
*** inc0 has quit IRC11:47
*** britthouser has joined #kolla11:53
*** britthou_ has joined #kolla11:54
openstackgerritSam Yaple proposed stackforge/kolla: Add Dockerfile template for glance  https://review.openstack.org/21187111:55
*** britthouser has quit IRC11:57
openstackgerritSwapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add Dockerfile template for Ceilometer  https://review.openstack.org/21187812:04
*** jmccarthy has quit IRC12:07
*** jmccarthy has joined #kolla12:07
openstackgerritSwapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add Dockerfile template for Cinder  https://review.openstack.org/21191912:08
openstackgerritSwapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add Dockerfile template for Ceilometer  https://review.openstack.org/21187812:10
*** diogogmt has joined #kolla12:11
*** diogogmt has quit IRC12:24
openstackgerritSwapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add Dockerfile template for Cinder  https://review.openstack.org/21191912:24
*** 7GHAAXY5L has joined #kolla12:32
*** dwalsh has joined #kolla12:36
*** jruano has joined #kolla12:38
*** inc0__ has joined #kolla12:46
*** inc0_ has quit IRC12:49
*** 7GHAAXY5L has quit IRC12:50
*** inc0__ has quit IRC12:53
openstackgerritAlicja Kwasniewska proposed stackforge/kolla: Add Dockerfile template for horizon  https://review.openstack.org/21193413:03
*** weiyu_ has joined #kolla13:07
*** britthou_ has quit IRC13:14
rhalliseyanyone have an idea why cinder can't connect to the db13:32
rhalliseyeven though I can manually connect to it and it gets created..13:33
rhalliseyhttps://bugs.launchpad.net/kolla/+bug/148192813:34
openstackLaunchpad bug 1481928 in kolla "Cinder is unable to connect to the database" [Critical,Triaged] - Assigned to Ryan Hallisey (rthall14)13:34
*** blahRus has joined #kolla13:39
*** jtriley has joined #kolla13:43
jpeelerSamYaple: yeah, I do live in SC. why?13:45
rhalliseyok I know..13:46
*** rhallisey has quit IRC13:47
*** dwalsh has quit IRC13:50
openstackgerritVladislav Belogrudov proposed stackforge/kolla: Glance cannot store images due to incomplete configuration  https://review.openstack.org/21203313:50
vbelhello, can we check ^^ so that glance finally works out of box? :)13:51
*** weiyu_ has quit IRC13:51
vbelcinder is my next service to torture :)13:52
*** diogogmt has joined #kolla13:55
*** diogogmt has quit IRC13:55
*** rhallisey has joined #kolla13:56
vbelI could finally store and download cirros images from glance without loss :)13:57
*** sdake_ has joined #kolla14:00
sdake_morning14:01
sdake_such as it is14:01
vbelmorning sdake14:01
openstackgerritJeff Peeler proposed stackforge/kolla: Add new test script for parallel builds  https://review.openstack.org/21204014:01
*** absubram has joined #kolla14:04
*** jmccarthy has quit IRC14:07
openstackgerritJeff Peeler proposed stackforge/kolla: Add new test script for parallel builds  https://review.openstack.org/21204014:09
*** thumpba has joined #kolla14:15
*** dwalsh has joined #kolla14:16
*** jmccarthy has joined #kolla14:17
*** jruano has quit IRC14:17
*** banix has joined #kolla14:17
*** britthouser has joined #kolla14:21
*** rhallisey_ has joined #kolla14:23
*** dwalsh has quit IRC14:23
*** rhallisey has quit IRC14:27
sdake_jpeeler can you get a gat job going in openstack-infra to run that build script14:31
vbelsdake: https://review.openstack.org/212033 ? To get things going :) It is a basic config to use glance out of box. If users want another store they can re-configure it14:34
*** rhallisey_ is now known as rhallisey14:35
sdake_vbel how does oracle buidl ol images14:37
sdake_or only ben focusing on the centos images?14:37
vbelwe have ol source based images similar to centos-source-14:38
sdake_how doy ou build them?14:38
sdake_what is your process, is what i'm after14:38
vbeldocker 1.8 with build-env patch14:38
sdake_do you build the ol/base image?14:39
vbelyes14:39
*** banix has quit IRC14:39
sdake_ok well we are doing this new ninja2 templating thing14:39
sdake_and i'd like the oracle images to work14:39
sdake_one question i had is i notice there is no other package besides base, do you just hack the build script to rename the centos images to oraclelinx o rsomething?14:40
vbelwe have mirror in house, should be able to push upstream some day14:40
sdake_mean mirror of the git repo?14:41
vbelsome stuff is very easy - like linking ol-source-heat to centos-source-heat14:41
vbelyes14:41
sdake_probably not a mirror :)14:41
sdake_but thanks that answers my q14:41
vbelwell, it is enhanced mirror :)14:41
vbelsome day I will start adding stuff to docker/ol/source/14:42
sdake_what other divergence is there besides linking ol-source-heat and all the other projects14:42
sdake_vbel we are deprecating ht docker dir14:42
sdake_going to docker_templates14:42
vbelso we will fetch/merge them14:43
sdake_regarding glance i dont get why that changeis necessary, glance wfm when i tested it, but I don't have objection to it so approving14:43
vbelor rather adopt since there is not much to merge14:43
vbelit does not work when trying to push image into it because config section is missing14:44
sdake_that did work for me14:44
sdake_at end of l214:44
sdake_perhaps new change in glance14:44
sdake_i manually tested the tag before pushing the tag14:45
vbelI added simple file store as in openstack manuals. Could be14:45
vbelI thought by default it should be local file store but suddenly it wanted the config explicitly14:45
vbelis it 2015.1.1 or 2015.1.0 that was tested?14:46
sdake_master14:47
vbelok14:47
vbelI tested with  2015.1.1 (stable kilo bugfix)14:47
sdake_thats the difference then14:47
vbelkolla uses 2015.1.0 by default14:47
vbelwrong...14:48
sdake_rdo uses master only14:48
vbelyes, correct14:48
jpeelersdake_: once the review gets approved, i'll add it as an experimental job14:48
openstackgerritMerged stackforge/kolla: Glance cannot store images due to incomplete configuration  https://review.openstack.org/21203314:48
sdake_pbourke can you approve peeler's patch or rhallisey14:48
pbourkesdake_: which one14:49
sdake_according to the governance repos official tool for teamstats, our team size is 14 committers and 14 reviewers :)14:49
jpeelerpbourke: https://review.openstack.org/#/c/212040/ , thanks in advance!14:49
sdake_jpeeler i see you grew facial hair :)14:50
jpeelerHa, yeah the other picture was approaching 10 years old14:50
sdake_akwasnie my bad for not inviting you to the kolla-drivers team14:50
sdake_its a mostly open team - (kolla devs and whatnot are all welcome)14:51
sdake_I'll add you now14:51
sdake_you actually look like not a boy in that  picture ;)14:51
sdake_i hate to use the word man because your still a young buck :)14:51
jpeelerlike i said, long overdue14:52
sdake_its a pretty solid picture14:52
sdake_i see your hair is doing the same shit mine is doing, thinning :)14:52
jpeeleryes :/ but it's who i am!14:52
sdake_i was losing my hair badly at 25 or so, so i took propecia for about 1 year14:52
sdake_and that stopped it14:53
sdake_and then neverhad to do anything else14:53
jpeelerreally?14:53
* rhallisey goes to photoshop picture with beard14:53
sdake_yup propecia seems permanent14:53
sdake_even though people take it forever14:53
sdake_my hair isn't like that dude from ancient alien astronauths but its not terrible ;)14:53
sdake_https://en.wikipedia.org/wiki/Giorgio_A._Tsoukalos14:55
sdake_that cat has the best hair in the business :)14:55
sdake_ya I highy recommend 1 year propecia treatment for folks that want  to keep their hair at the 25-28 age rang when it starts to fall out :)14:57
*** diogogmt has joined #kolla14:57
sdake_doctors will want to keep you on it forever but once it does its thing it seems permanent - several people have told me this as well after the fact14:57
jpeelerhow expensive is this drug14:57
sdake_its cheap14:58
sdake_its a pill rather then a topical too which is nice14:58
jpeelerinteresting14:58
sdake_iirc it does require blood tests14:58
sdake_every 3 months or so14:58
sdake_thats the part that sucke14:59
sdake_d14:59
sdake_i used to pass out on blood draws but now pretty resilient to the blood loss :)14:59
jpeelerjust need one more review sdake15:01
*** diogogmt has quit IRC15:03
*** banix has joined #kolla15:03
*** diogogmt has joined #kolla15:04
jpeelerdocker 1.8.0 came out, i can't keep up15:04
sdake_nice15:04
sdake_that is why i wanted to use the curl thing ;)15:04
sdake_jpeeler just fyi docker is on a 2 month release schedule15:05
jpeelersince when?15:05
sdake_since forever15:05
jpeelergood to know15:05
sdake_since i got involved in docker 1.515:05
sdake_probably before then15:05
sdake_they are nt super strict on the schedule but try to keep to it15:06
* coolsvap remember looking at a review for cinder-app -> cinder or was i dreaming of having that review15:17
rhalliseylooks like only cinder-api gets cinder.conf... great15:27
*** thumpba has quit IRC15:31
*** thumpba has joined #kolla15:33
*** banix has quit IRC15:48
sdake_coolsvap I wrote we need to rename it, but never did the rename because people were busy in the space15:48
sdake_feel free to submit a rename - cinder-app is out of place15:48
sdake_bybusy in the space I mean busy in the cinder dir during config-internal deletion15:49
coolsvapsdake_, hmm sometime later15:49
sdake_are there any cinder commits out?  If not I'll do right now15:49
sdake_both coolsvap and rhallisey would know :)15:50
coolsvapsdake_, later15:50
rhalliseystill trying fix the ansible piece15:50
sdake_this wont affect that unless your changing docker/cinder-app15:50
sdake_docker/*/*/cinder_app15:51
coolsvapsdake_, there are multiple changes, i think we can do it at once15:51
rhalliseyya should be fine15:51
coolsvapeither way i am not much worried about it now15:51
coolsvapi was just digging sometime earlier i could not find the change set or bu15:52
coolsvapso just asked15:52
sdake_i am already in action15:52
sdake_;)15:52
*** thumpba has quit IRC15:57
*** thumpba has joined #kolla15:57
*** thumpba has quit IRC15:58
*** thumpba has joined #kolla15:58
openstackgerritSteven Dake proposed stackforge/kolla: Rename cinder-app to cinder to be more consistent  https://review.openstack.org/21208816:05
openstackgerritSteven Dake proposed stackforge/kolla: Rename cinder-app to cinder to be more consistent  https://review.openstack.org/21208816:07
sdake_i coudln't tst that because i deved it on a mac16:08
*** pbourke has quit IRC16:08
sdake_but lets get it merged and if it breaks the build we can get it fixed16:09
sdake_rhallisey can you review16:09
sdake_i'll add a bug id now sec16:09
*** pbourke has joined #kolla16:09
pbourkejust realised I need ansible memcache as a precursor to swift16:09
pbourkedoes memcache need to run in some kind of cluster or can it be one instance per server16:09
openstackgerritSteven Dake proposed stackforge/kolla: Rename cinder-app to cinder to be more consistent  https://review.openstack.org/21208816:10
sdake_pbourke pbourke rhallisey please review that before we need further cinder changes16:10
sdake_I dont want to have to rebase it :)16:10
pbourkesdake_: been wanting that change for ages16:10
sdake_pbourke I have no idea how memcache works16:11
pbourkewhy are there so many deletes?16:11
pbourkedid you use git mv?16:11
sdake_rhallisey actually review this first: https://review.openstack.org/#/c/211221/16:12
sdake_yes I used git mv16:12
sdake_but I had to relink a whole bunch of shit16:12
pbourkeah16:12
sdake_because the links were cinder-app not cinder16:12
pbourkeprobably just gerrit not being pretty16:12
sdake_please lets get that other cinder review merged first16:12
* sdake_ doens't want to erbase ;-)16:13
*** sdake_ is now known as sdake16:13
sdakerhallisey or jpeeler this could use a rubber stamp https://review.openstack.org/#/c/210823/16:13
rhalliseymy stamp is made of gold16:14
rhalliseylol jks16:14
sdakeso besides diversity the other thing that is used to measure how stable a project is is the team size16:15
sdakea larger team naturally is more resilient16:15
sdakewe have 14 reviewers and 14 commiters according to the teamstats.py tool in the governance repo (renamed from diversity.py)16:16
sdakewe have about 30 poeple that contributed code in the last 3 months16:16
sdakeMIN_REVIEWS = 3016:16
sdakeMIN_COMMITS = 616:16
sdakethese are the counts to hit an increment in either reviewers or committers16:17
sdakejpeeler https://review.openstack.org/#/c/212040/ needs some minor edits - can you crank it out so we canget er merged16:22
sdakethe faster we have the gate running the better16:22
*** unicell1 has quit IRC16:25
*** jmccarthy has quit IRC16:28
openstackgerritMerged stackforge/kolla: Remove tox -egenconfig since its not needed for cinder  https://review.openstack.org/21122116:29
openstackgerritMerged stackforge/kolla: Remove mariadb container  https://review.openstack.org/21082316:31
*** akwasnie has quit IRC16:33
sdakepbourke or rhallisey can one of you review https://review.openstack.org/#/c/211934/16:33
sdakeneed m0ar reviewers!16:34
sdakecan't wait until mandre is back16:34
sdakerhalllisey rubber stamping on https://review.openstack.org/#/c/212088/ appreciated ;)16:35
sdakethen if you can build it and see if it still works once merged that would be extra+ credit16:36
*** diga_ has joined #kolla16:40
openstackgerritMerged stackforge/kolla: Add Dockerfile template for horizon  https://review.openstack.org/21193416:43
*** diga_ has quit IRC16:46
*** gfidente has quit IRC16:48
*** unicell has joined #kolla16:48
*** mfalatic has quit IRC16:58
*** mfalatic has joined #kolla16:59
harmwsdake: ill make sure I can do some reviews this friday evening17:00
harmwthat worked out in the past quite well17:00
harmwfor me atleast :p17:00
*** mfalatic has quit IRC17:02
*** mfalatic has joined #kolla17:03
*** achanda has joined #kolla17:13
*** britthou_ has joined #kolla17:27
*** britthouser has quit IRC17:30
*** banix has joined #kolla17:34
*** Haomeng has joined #kolla17:36
*** Haomeng|2 has quit IRC17:38
coolsvapsdake, SamYaple can please review https://review.openstack.org/211442 https://review.openstack.org/211445 https://review.openstack.org/211878 https://review.openstack.org/21191917:42
rhalliseycinder ansible -> https://review.openstack.org/#/c/208868/17:43
rhalliseysdake, just need another +217:43
*** bmace has quit IRC17:43
*** absubram has quit IRC17:55
*** absubram has joined #kolla17:57
*** banix has quit IRC18:01
openstackgerritMerged stackforge/kolla: Add ansible support for Cinder  https://review.openstack.org/20886818:06
openstackgerritMerged stackforge/kolla: Glance-registry is missing a messaging config  https://review.openstack.org/21161618:07
openstackgerritMerged stackforge/kolla: Update neutron confs and ports  https://review.openstack.org/21096218:07
openstackgerritRyan Hallisey proposed stackforge/kolla: Ansible Cinder needs to create the v2 endpoint  https://review.openstack.org/21214018:21
rhalliseythat's a little closer...18:21
*** banix has joined #kolla18:22
*** tfukushima has quit IRC18:27
*** tfukushima has joined #kolla18:28
*** tfukushima has quit IRC18:28
openstackgerritJeff Peeler proposed stackforge/kolla: Add new test script for parallel builds  https://review.openstack.org/21204018:30
*** absubram has quit IRC18:35
*** absubram has joined #kolla18:38
*** absubram has quit IRC18:48
*** absubram has joined #kolla18:54
sdakeback18:56
openstackgerritSteven Dake proposed stackforge/kolla: Rename cinder-app to cinder to be more consistent  https://review.openstack.org/21208819:05
sdakepbourke you around?19:05
sdakerhallisey can you re-rubber stamp that - it had a link error19:05
sdakeharmw if you could rubber stamp the above - it was alredy reviewed twice ^^19:07
sdakejust had an error in one of the links19:08
sdakeor jpeeler ^^19:08
* jpeeler got it19:09
sdakejpeeler thanks19:09
sdakejpeeler suggestion for new gate naming19:09
sdakeas you make new gates for each of the distro types19:09
sdakeor atleast the ones you are interested in19:09
sdakeactually I was sort of doing that work19:09
sdakeanyway19:09
jpeeleri actually already have it ready for review19:09
jpeelerbut haven't posted it19:09
sdakei think what makes sense is gate-build-images-centos-src19:09
sdakegate-build-images-centos-bin19:10
sdakegate-build-images-oraclelinux-src19:10
sdakeetc19:10
jpeeleri currently have kolla-parallel-build-f21...19:10
sdakeya lets rename it19:11
sdakewe are doing gates for every distro + src/bin combo19:11
sdakemake sense?19:12
jpeelerit does19:12
jpeeleronce i get this in the gate, i probably won't work on getting all the other distro combinations in. but they should be super easy to add19:13
jpeelersdake: i guess i'll use "rdo" since that was used for the tox target19:17
sdakewfm19:17
sdakeplease link so i can add centos from src19:18
sdakeor alternatively follow up the first patch with a ssecond19:18
sdakei'd like to get those two gates working since we are doing this tempalting work now19:18
jpeelerthe project-config review? i haven't submitted it yet since the tox review hasn't merged19:18
sdakecan you link and ask rhallisey to review it plz19:19
rhalliseywow.. just figured out the bug..19:19
sdakeneeds to happen soon its already the 12th :)19:19
jpeelerhttps://review.openstack.org/#/c/212040/19:19
rhalliseythat sucked19:19
jpeelersdake: i had all this staged before i left last month, but reviews didn't get done :/19:19
sdakerhallisey can you look that over19:19
sdakeits a little complicated19:20
sdakeits all good jpeeler19:20
rhalliseynobody every spell something wrong in config-external again please19:20
sdakewe need more core revierers - we produce more code then reviews typically :)19:20
rhalliseyya looking19:20
jpeeleri'm bad about reviewing if i'm not actively submitting code19:20
rhalliseygetting over bug hangover here19:21
jpeeleri guess i need to learn ansible a lot more too19:24
sdakeya jpeeler you ought to take on a service - its esay to leanr took me about 4 hours to do heat19:24
sdakealthough I ran into trobule and the work is stalled atm19:25
sdakei wouldn't necessarily start with ironic either, that may b edifficult to do ansible with or may not not sure19:25
jpeeleryeah i will once my back log gets cleared up19:25
sdakeif you just do api and conductor it hsould be easy enough19:25
jpeeler(this gate stuff part of that)19:25
openstackgerritRyan Hallisey proposed stackforge/kolla: All the Cinder services need to reference the correct config  https://review.openstack.org/21215719:28
sdakerhallisey can you hold off on that one19:29
sdakeand wait until cinder-app finishes19:30
sdakeit will be easy for you to rebase19:30
rhalliseyya19:30
sdakeit will be a pita for me to rebase19:30
rhalliseyI have one more patch too but I will hold the merge19:30
sdakefor you to rebase git pull --rebase should figure out the renames19:30
rhalliseythere was a spelling error in config-external19:30
sdakebut git doens't work in revrse, if I have a rename in my repo it doesn't rebase your work into it19:31
rhalliseyso cinder-backups was never configured O.o19:31
rhalliseythat drove me crazy19:31
rhalliseyit was just one letter :(19:31
rhalliseysdake, no worries i'll -1 until yours merges19:31
*** jasonsb has quit IRC19:33
*** jasonsb has joined #kolla19:34
openstackgerritRyan Hallisey proposed stackforge/kolla: Cinder-backup is not being configured because config-external spelling error  https://review.openstack.org/21216019:35
rhalliseywill -1 that also19:35
sdakenot sure how we miss these things :(19:36
sdakewe have a plan to remove the config-external.sh script entirely19:37
sdakeand specify the config files to load up via ansible and a common json file19:37
rhalliseythat bug sucked19:37
rhalliseybecause it was undetectable19:37
rhalliseyoh well19:37
*** jasonsb has quit IRC19:38
*** echoingumesh has joined #kolla19:39
*** echoingumesh has quit IRC19:41
*** echoingumesh has joined #kolla19:42
*** echoingumesh has quit IRC19:46
*** bmace has joined #kolla19:52
openstackgerritMerged stackforge/kolla: Rename cinder-app to cinder to be more consistent  https://review.openstack.org/21208819:56
sdakeok rhallisey a quick rebae and i'll look at your patches19:59
openstackgerritRyan Hallisey proposed stackforge/kolla: Cinder-backup is not being configured because config-external spelling error  https://review.openstack.org/21216019:59
rhalliseythat's the only one that needed a rebase20:00
rhalliseynow I think cinder bootstrap needs to be fixed..20:01
rhalliseygettin there20:01
openstackgerritMerged stackforge/kolla: Removed config-internal check.sh for glance  https://review.openstack.org/21087220:07
openstackgerritMerged stackforge/kolla: Glance source based containers cannot write to /var/log/glance  https://review.openstack.org/21158020:09
*** achanda has quit IRC20:15
openstackgerritMerged stackforge/kolla: Update glance confs and ports  https://review.openstack.org/21096920:22
openstackgerritMerged stackforge/kolla: Update rabbitmq confs and ports  https://review.openstack.org/21097120:22
openstackgerritMerged stackforge/kolla: Update nova.conf and ports  https://review.openstack.org/21096020:22
*** echoingumesh has joined #kolla20:23
*** jruano has joined #kolla20:28
*** jasonsb has joined #kolla20:36
openstackgerritMerged stackforge/kolla: Add new test script for parallel builds  https://review.openstack.org/21204020:36
*** thumpba has quit IRC20:40
openstackgerritRyan Hallisey proposed stackforge/kolla: Ansible Cinder needs to create the v2 endpoint  https://review.openstack.org/21214020:42
*** rhallisey has quit IRC20:53
jpeelerinfra changed check/gate config naming/convenctions20:57
sdakerhallisey i hae an emergency20:57
sdakeif i am not back by meeting start time please get it started20:57
sdakeok?20:57
sdakei expect to be back20:58
sdakei didn't update the agenda20:58
sdakeso ymmv there :)20:58
sdakebmace ^^20:58
*** jruano has quit IRC21:00
*** bmace has quit IRC21:01
*** achanda has joined #kolla21:01
*** echoingumesh has quit IRC21:02
*** echoingumesh has joined #kolla21:03
*** echoingumesh has quit IRC21:04
*** echoingumesh has joined #kolla21:04
*** bmace has joined #kolla21:19
*** jtriley has quit IRC21:34
jpeelersdake: hope everything's okay21:38
jpeelerhttps://review.openstack.org/212194 - build.py in gate review21:39
sdakebackin time21:53
sdakeya just car problem21:53
sdakenothing major21:53
*** rhallisey has joined #kolla21:56
sdakemeeting now in openstack-meeting-422:01
jpeelersdake: do whatever you have to do for that gate review above when you get a chance22:06
sdakecool i am going to append to it a new review22:07
sdakeopefully it doens't damage yours22:07
sdakei think gerrit will leave yours intact22:07
jpeeleras long as you make it dependent on it, it won't damage it22:07
sdakei make it dependent by pulling it and then applying mine as a new patch22:07
jpeelerright22:07
sdakerhallisey asked for the teamstats output, so here it is for everyone to look at:22:19
sdakehttp://paste.openstack.org/raw/412795/22:19
sdakenote kolla is one of 15 projects with the team:diversity flag22:19
sdake;)22:19
sdakethe cutoffs are 30 reviews or 6 commits22:20
sdakein the script22:20
*** diogogmt has quit IRC22:22
sdakealready 3pm and I dont think I've done a single thing today22:32
* sdake wtfawks22:32
sdakejpeeler is the build with tox -e images-centos-rdo parallel or serial?22:40
jpeelerparallel22:40
sdakenice22:40
sdakewhy did you decide to add -T=1 (which is a fantastic improvement btw) in that change then?22:40
jpeelerreally curious to see how fast it goes22:40
*** achanda has quit IRC22:40
jpeelersdake: i had a stalled build when testing and was like, this sucks waiting to see what's happening22:41
sdakeroger22:42
*** vinkman has joined #kolla22:50
sdakewhat -T level did you use22:52
sdakeonce the gate commit goes live we ought to play with that paramemter to see what produces best results22:53
sdakei think the machines are 4 cpus avaialble but it may jsut be single cpu22:53
jpeelerthe default is 8 i believe22:54
sdake8gb ram22:54
jpeelerit's easy to play with once the gate config merges22:54
openstackgerritSteven Dake proposed stackforge/kolla: Test commit - do not merge  https://review.openstack.org/21221422:56
*** weiyu has joined #kolla22:56
jpeelerha22:57
*** bmace has quit IRC22:58
sdakejpeeler whre is that gate job running for what I just submitted22:59
sdakeit prints out /proc/cpuinfo22:59
jpeelersdake: if i recall correctly it doesn't enter the pipeline until a vote is submitted22:59
jpeeleri was typing that before you pinged me heh22:59
jpeelerjust add +1 to it22:59
sdakeit gets gated automaticaly without votes22:59
sdakerather checked23:00
sdakethe check jobs run on commit not vote23:00
jpeelerwell i bet it speeds it up then23:00
jpeeleri don't know where the job is then...23:00
jpeelerthere it is23:01
jpeeler8 cores23:02
jpeeleractually i'm not sure how to read it23:03
jpeelerdoes 8 processors mean it necessarily has 8 cores?23:03
*** weiyu has quit IRC23:04
jpeelerthe reason i'm confused is because i wasn't sure how hyperthreading affects things (if at all)23:04
sdakeits actually virtualized23:09
sdakeso its not like you get a 1:1 mapping23:09
sdakeopenstack allows overcommit23:09
sdakeso the raw cpus get shared23:09
sdakeits a nova config option23:09
sdakebut it looks like 8 cores to the os23:10
sdakewe could probably run with -T 1000 then23:10
sdakeI have 16 cores and rn with that23:10
*** weiyu has joined #kolla23:11
*** vinkman has left #kolla23:13
sdakejepeeler can yu paste the link23:15
*** blahRus has quit IRC23:16
*** achanda has joined #kolla23:16
sdakejpeeler xeon 8 core cpu without hyperthreading23:19
sdakesurprised ht isn't turned on23:19
sdakejpeeler trying to figure out how to template-ize your review23:20
sdakefor my change23:20
sdakejpeeler do you know how tox is templatized?  We have an example of using parameters with tox in our repo23:20
jpeelertox? i don't off the top of my head. headed to dinner now, but if you just want to test the different thread numbers, you can change the args in test_build.py23:26
*** banix has quit IRC23:36
*** devlaps has joined #kolla23:57
*** achanda has quit IRC23:59

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