Monday, 2015-08-10

SamYaplesdake: on it00:00
openstackgerritSam Yaple proposed stackforge/kolla: Remove config-internal for Horizon  https://review.openstack.org/21057700:02
*** thumpba has quit IRC00:03
SamYaplesdake: youre stuff could be merged if you didnt have 10 patch long dependacy chains00:07
sdakemy stuff was already mreged samyaple00:19
sdakethose are new patches00:20
*** thumpba has joined #kolla00:20
SamYapleyou had those changes in originally for those patches though00:21
SamYaplethey got removed right?00:21
sdakethese are new bug fixes to master for left over stuff00:22
sdakeI dont know what happened with ceilometer00:22
sdakeI think it was left undone00:22
SamYapleno https://review.openstack.org/#/c/210520/4/docker/centos/source/ceilometer/ceilometer-alarm/Dockerfile00:22
SamYaplei think one of your rebases screwed things up00:22
SamYapleor someone elses00:22
SamYapleyou clearly did it00:23
sdakeprobably was just broken to begin with00:23
SamYapleeh anyway00:23
SamYaplethis seems to break builds...00:23
sdakewith my patch set almost all containers build again ;-)00:24
sdakehttp://ur1.ca/nedq9 -> http://paste.fedoraproject.org/253283/1439166300:25
openstackgerritSam Yaple proposed stackforge/kolla: Removed config-internal check.sh for glance  https://review.openstack.org/21087200:26
sdakeits config-glance.sh that is the problem00:27
sdakeit was deleted out of common/base00:27
openstackgerritSam Yaple proposed stackforge/kolla: Removed config-internal check.sh for cinder  https://review.openstack.org/21087300:29
SamYapledo we use it at all?00:29
openstackgerritSteven Dake proposed stackforge/kolla: Remove config-internal for glance  https://review.openstack.org/21087400:30
sdakecheck.sh?  Nah we had planned to but I dont know how we would actually use it00:30
sdakewith config-external, we for the most part don't need config-service.sh bash scripts00:31
sdakewe should remove all that are still left00:31
sdakeaccording to my build that was pretty much just glance00:31
openstackgerritSam Yaple proposed stackforge/kolla: Removed config-internal check.sh for neutron  https://review.openstack.org/21087500:31
SamYaplewe do need it00:32
SamYaplemaybe not glance00:32
SamYaplebut for rabbitmq and mariadb00:32
sdakeok00:32
SamYaplei specifically used some functions from them00:32
sdakeno openstack service should be using it I think00:32
SamYaplewe can reintegrate them back into the start.sh if you want, but dont just delete them00:33
SamYaplei beleive thats right00:33
sdakei didn't delete anything from rabbitmq or mariadb00:33
sdakeyou did those containers00:33
sdakewe dont need config-glance.sh that was only used by config-internal00:33
SamYapleI want to propose we rename CONFIG_EXTERNAL_COPY_ONCE to COPY_ONCE00:33
SamYaplei didn't delete them from rabbitmq or mariadb. just letting you know so you didnt ;)00:34
sdakeroger00:34
sdakethat neutron set-configs should be some special version of set_configs that takes a parameter imo ;-)00:35
SamYaplesdake: nah we can just wait for thin containers00:36
SamYapleno big thing00:36
sdakeif we rename copy_once we have to rename copy_always00:37
sdakeif we are going to do that, lets do it before liberty-3 please00:37
sdakei dont see the point but i dont care strongly00:37
*** thumpba has quit IRC00:38
*** jpeeler has joined #kolla00:38
sdakeone area for gold plating is removing config-external.sh, inject a yaml file with the files source/target locations to copy, and make set-configs process it in a standard location00:39
sdakethis is sort of based on an idea you may ahve had previously mentioned, not certain if it was you or someone else00:40
SamYaplei suggested json since we do a similiar thing with swift00:41
SamYapleadditionally it will need information about the owner and permissions00:41
sdakei find json difficult to use typically00:41
SamYaplei would prefer it not to be yaml00:41
SamYaplewell no one will be using it directly00:41
SamYapleit will be generated00:42
SamYapleyaml is a pita to geenrate and we already generate json00:42
sdakegenerate from what?00:42
sdakeyaml is easy to generate last I tried ;-)00:42
SamYapleAnsible will generate the file00:42
SamYaplethe container will read the file and do the actions00:42
sdakeoh i see00:43
SamYaplewe do the simliar thing with swift and we use json00:43
sdakethats a pretty good approach00:43
SamYaplei want to keep it consistent00:43
sdakegold plating but unfortunately affects our abi00:43
sdakei'd like our abi to be stable after l300:43
SamYapleif you convince pbourke to switch swift to yaml, i wouldn't care00:43
sdakei didn't gt the autogneration parth with ansible at runtime00:44
sdakei thought we woudl inject at container create00:44
sdakebut runtime injection is much better00:44
SamYapleyea buildtime is the least flexible00:44
sdakeit does put a burden on those that integrate with the containers to comprehend the format00:44
sdakeatm there is no such burden00:45
SamYaplethere is with swift00:45
SamYapleand json or yaml is still going to be a format00:45
sdakewhat i mean is config-external atm is hard  coded00:45
SamYapleyea i get that00:45
sdakei dont know about swift00:45
SamYapleim not for or against yaml _except_ json is builtin for python and yaml is not, and we already use json00:46
SamYaplewith that in mind we should just generate json00:46
SamYaplehmmm00:46
SamYapleif we do generated yaml that puts yaml deps on the destination hosts00:46
sdakethe nice thing bout that is we ould get rid of all config-external.sh files00:47
SamYapleyep00:47
sdakeyaml in python is built in i think00:47
SamYaplewhy does it require libyaml00:48
sdakei dont feel strongly about json vs yaml if its autogenerated00:48
sdakeif its human generated i have a very strong preference for yaml00:48
SamYapleagreed00:49
sdakei spent a year plus generating templates in json for heat, and it was super painful00:49
sdakeand then steve baker added yaml support and i was olike "wow wtf why didn't we start with that. :)00:49
sdakesamyaple can you work out a prototype keystone container with the yaml copy stuff so we can see what it looks like00:51
sdakei want to make sure the tripleoguys are onboard with that00:51
sdakeif its autogenerated, where are the original file data stored, in yaml in ansible?00:53
*** Slower has quit IRC00:53
SamYapleyou mean the json?00:56
sdakecontainers containing less and less code - I ike that00:56
SamYaplesince im not doing yaml00:56
SamYapleswift uses json00:56
sdakeso we have to put the data in manually in some how right?00:56
SamYaplenot neccesarily00:56
sdakei would like to see the implementation top to bottom for keysstone for eg00:57
*** zhiwei has joined #kolla00:57
SamYapleyea got alot of stuff going on atm. itll be a while00:57
sdakeok00:57
SamYapleyaml is not builtin for python00:58
SamYapleit is a hard -2 since it will add deps to teh destination nodes00:58
sdakeyes but ansible uses yaml on the deployment host00:58
SamYaplewhich is not where the code runs00:58
SamYaplewe went ofver this00:58
sdakei know i got it00:59
sdakewhat i'm saying is the human generated part can be in yaml00:59
sdakeif there is any to be00:59
SamYapleit cannot00:59
SamYapleit would still be parsed on the destination node00:59
sdakeif its an argument to merge_configs ?01:00
sdake(not yaml but just straight arguments)01:00
sdakejust trying to think through how you get the uid and permissions in01:00
sdakeor if we care to specify those at all01:00
SamYaplewell those can be assumed to be keystone (or service) and 0644 i think01:00
SamYaplethe reason we set it at all is the container <--> host pid mapping issues01:01
sdakecool so we need not specify them01:01
*** Slower has joined #kolla01:01
SamYapleit shoudl be an option. im sure well need it01:01
SamYaplebut thats not a problem01:01
SamYaplesdake: im sorry i missed your point01:03
SamYapleyea as an option from yaml into a module it should work01:03
SamYaplethere is an issue with that in that you would need to specify lots of things and then merge the json01:04
SamYaplemultiple files means multiple runs01:04
sdakemultiple runs of what01:05
SamYapleneutron.conf ml2_conf.ini l3_agent.ini etc01:06
SamYaplethat would spit out multiple jsons that would need to merge01:06
SamYapleah ive got it01:07
sdakejust copy the raw ini files to /var/lib/kolla an dcopy those in01:07
sdakeall you need is the metadata01:07
sdakein json01:07
SamYapleyou dont understand what im saying01:07
SamYaplebut thats ok01:07
SamYaplewe can do something different01:07
SamYaplewe can have a dict we build in yaml (the defaults/main.yml for each service) and use an ansible filter to convert that to json01:08
SamYapleitll be simple01:08
SamYaplei think that makes everyone happy. good01:09
*** dolpher has joined #kolla01:09
sdakeas long as the containers don't take a rocket scientiest to figure how to configure i'm happy ;)01:09
sdakemy 845 tubes arrive tomorrow - hopefullly this time they aren'tbusted01:10
sdakesamyaple can you ack this missed comment change - there ws actually a separate complaint bug on this problem https://review.openstack.org/#/c/210850/01:15
*** daneyon_ has quit IRC01:24
*** erkules has joined #kolla01:35
*** Slower has quit IRC01:35
*** erkules_ has quit IRC01:37
*** Slower has joined #kolla01:44
*** Slower has quit IRC01:49
*** erkules has quit IRC01:52
*** erkules has joined #kolla01:54
*** thumpba has joined #kolla02:08
*** banix has joined #kolla02:09
*** thumpba has quit IRC02:15
*** Slower has joined #kolla02:18
openstackgerritSteven Dake proposed stackforge/kolla: Remove check.sh from linuxbridge-agent  https://review.openstack.org/21095102:20
*** diogogmt has quit IRC02:30
*** diogogmt has joined #kolla02:42
SamYaplesdake: how are upgrades coming along?03:10
SamYapleanything new to share ideas wise?03:10
*** thumpba has joined #kolla03:12
*** thumpba has quit IRC03:17
*** unicell1 has joined #kolla03:19
*** unicell has quit IRC03:21
openstackgerritSam Yaple proposed stackforge/kolla: Update nova.conf and ports  https://review.openstack.org/21096003:35
openstackgerritSam Yaple proposed stackforge/kolla: Update nova.conf and ports  https://review.openstack.org/21096003:37
openstackgerritSam Yaple proposed stackforge/kolla: Update nova.conf and ports  https://review.openstack.org/21096003:47
openstackgerritSam Yaple proposed stackforge/kolla: Update neutron confs and ports  https://review.openstack.org/21096204:01
*** jpeeler has quit IRC04:10
*** unicell has joined #kolla04:21
*** unicell1 has quit IRC04:21
*** banix has quit IRC04:34
*** klint has joined #kolla04:40
*** banix has joined #kolla04:41
openstackgerritSam Yaple proposed stackforge/kolla: Update glance confs and ports  https://review.openstack.org/21096904:44
*** achanda has joined #kolla04:53
coolsvapSamYaple, around?04:55
*** banix has quit IRC04:55
SamYaplecoolsvap: always04:58
coolsvapSamYaple, are we moving the rabbitmq related settings to global with the update config bp?04:59
SamYaplewhat do you mean?04:59
coolsvaphttps://review.openstack.org/#/c/210969/1/ansible/roles/glance/templates/glance-api.conf.j204:59
SamYapleno04:59
coolsvapi see the rabbit mq configuraiton is removed05:00
SamYaplein the case of that its just removing them05:00
SamYaplelook at the commit message05:00
*** sdake has quit IRC05:01
coolsvapoh okay, sorry missed it, was looking at the change set so there will be one more update for the if block05:01
SamYapleglance is the only time well be removing rabbit in the configs fyi05:01
SamYapleeverything else uses it pretty heavily05:01
SamYaplekeystone and glance only use it when ceilometer is involved05:01
openstackgerritSam Yaple proposed stackforge/kolla: Update rabbitmq confs and ports  https://review.openstack.org/21097105:04
openstackgerritSwapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add ansible support for Cinder  https://review.openstack.org/20886805:06
*** thumpba has joined #kolla05:14
*** thumpba has quit IRC05:18
openstackgerritSam Yaple proposed stackforge/kolla: Update mariadb confs and ports  https://review.openstack.org/21098005:58
openstackgerritSam Yaple proposed stackforge/kolla: Update mariadb confs and ports  https://review.openstack.org/21098005:59
*** thumpba has joined #kolla06:15
*** thumpba has quit IRC06:19
*** bmace has joined #kolla06:30
openstackgerritSam Yaple proposed stackforge/kolla: Update mariadb confs and ports  https://review.openstack.org/21098006:36
SamYaplepbourke: you around?06:37
bmaceits still a bit early for him, maybe in another hour or two.06:40
coolsvapyes06:41
coolsvapor maybe not06:42
SamYapleslacker06:42
SamYaplesleep is for the weak!06:42
*** dolpher has quit IRC06:47
*** dolpher has joined #kolla06:48
*** dolpher1 has joined #kolla06:57
*** dolpher has quit IRC06:59
*** diga has quit IRC07:02
*** diga has joined #kolla07:03
*** thumpba has joined #kolla07:15
*** achanda has quit IRC07:16
*** achanda has joined #kolla07:18
*** thumpba has quit IRC07:20
*** mstachow has joined #kolla07:24
*** achanda has quit IRC07:28
*** achanda has joined #kolla07:31
*** jmccarthy has quit IRC07:40
*** jmccarthy has joined #kolla07:41
*** achanda has quit IRC07:46
*** gfidente has joined #kolla07:52
*** athomas has joined #kolla08:04
pbourkemorning08:31
*** tfukushima has joined #kolla08:47
pbourkeSamYaple: you remember we removed the json in swift in the end08:50
pbourkeSamYaple: its plain env variables now08:50
*** inc0 has joined #kolla08:51
inc0hello08:51
pbourkelooks like you guys had a busy weekend08:54
inc0yeah, still going08:55
inc0trying hard to setup relatively working openstack08:55
*** vbel has quit IRC08:59
*** vbel has joined #kolla09:00
pbourkewoohoo no more min-env09:06
coolsvappbourke, :)09:16
*** thumpba has joined #kolla09:17
*** thumpba has quit IRC09:22
*** dolpher1 has quit IRC09:22
*** weiyu_ has joined #kolla09:52
openstackgerritMerged stackforge/kolla: Remove config-internal for Horizon  https://review.openstack.org/21057709:58
*** zhiwei has quit IRC10:01
openstackgerritMerged stackforge/kolla: Remove Ansible abstraction for Glance  https://review.openstack.org/21069310:06
*** mstachow has quit IRC10:32
*** shardy has joined #kolla10:41
inc0SamYaple, did you try latest greatest ansible 2.0 with docker?10:51
*** pbourke has quit IRC10:54
SamYapleinc0: the kolla-ansible container is running 2.010:54
vbelwe have ansible 2.0 in kolla-ansible container10:54
SamYapleits required to make it work10:54
*** pbourke has joined #kolla10:54
vbelThanks, SamYaple :)10:54
SamYaple:D10:55
inc0meh, I'm getting all sorts of strange errors10:55
SamYapleinc0: from the contaienr?10:55
SamYaplei wouldn't recommend running it on the deploy host10:55
inc0yeah I guess10:55
vbelwill roles/start.yml be eliminated soon?10:55
SamYaplei mean im saying dont. i know its broke there10:55
SamYaplevbel: im still working on the patchsets10:55
SamYapleonly keystone and glance has merged10:55
inc0I'm out for few days and...kolla-ansible container?:O10:56
vbelI see, just wonder general trend10:56
SamYapleinc0: yea all the dependacies are containerized :D10:56
*** akwasnie has joined #kolla11:03
*** weiyu_ has quit IRC11:08
*** tfukushima has quit IRC11:08
*** rhallisey has joined #kolla11:10
*** sdake has joined #kolla11:28
sdakemorning11:29
inc0morning sdake11:29
rhalliseymorning11:30
*** sdake_ has joined #kolla11:32
*** sdake has quit IRC11:36
sdake_pbourke can you ahve a look at https://review.openstack.org/#/c/210848/ again plz11:37
sdake_rhallisey can you have a look at https://review.openstack.org/#/c/210874/11:41
openstackgerritMerged stackforge/kolla: Removed config-internal check.sh for neutron  https://review.openstack.org/21087511:42
sdake_and approve that review i asked pbourke to look at ? :-)11:42
sdake_our builds are broken without these patches11:45
sdake_jpeeler you around?11:46
*** sdake has joined #kolla11:49
*** sdake_ has quit IRC11:52
sdakerhallisey ping re compose11:59
rhalliseysdake, ya what it?12:04
sdakedo you plan to use compose for tripleo12:04
sdakewith config-external12:04
rhalliseypossibly12:05
sdakemake a decision i either want to remove it from the repo or rework it to support config-external12:05
rhalliseyat this point it won't look much different that just a docke rurn12:05
sdakeas is, it is broken and confusing12:05
sdaketo do the rework is a couple hours - so rather not burn time on it if not necessary12:05
sdakethis is my removal patch https://review.openstack.org/#/c/210694/12:06
sdakethat took a couple hours to make in the first place12:06
rhalliseyI think we will.  At least right now we are12:06
sdakeok rework it is then12:07
openstackgerritVladislav Belogrudov proposed stackforge/kolla: Ansible should wait bootstrap container on first host  https://review.openstack.org/21112012:07
*** tfukushima has joined #kolla12:09
vbelcan anyone from ansible fans check ^^^ :)12:11
*** tfukushima has quit IRC12:14
openstackgerritSteven Dake proposed stackforge/kolla: Use docker-engine in setup_gate  https://review.openstack.org/21112312:16
openstackgerritSteven Dake proposed stackforge/kolla: Use docker-engine in setup_gate  https://review.openstack.org/21112312:18
*** tfukushima has joined #kolla12:37
openstackgerritSteven Dake proposed stackforge/kolla: Use docker-engine in setup_gate  https://review.openstack.org/21112312:37
openstackgerritSteven Dake proposed stackforge/kolla: Use docker-engine in setup_gate  https://review.openstack.org/21112312:39
openstackgerritSteven Dake proposed stackforge/kolla: Add centos-src-build tox gate  https://review.openstack.org/21112812:43
openstackgerritMerged stackforge/kolla: Remove config-internal from galera  https://review.openstack.org/21084812:43
openstackgerritMerged stackforge/kolla: Remove config-internal from ceilometer  https://review.openstack.org/21084912:44
openstackgerritMerged stackforge/kolla: Clean up comment in rabbitmq  https://review.openstack.org/21085012:44
openstackgerritMerged stackforge/kolla: Remove config-internal from cinder-backup  https://review.openstack.org/21085112:44
openstackgerritMerged stackforge/kolla: Remove config-internal from neutron agent start files  https://review.openstack.org/21085212:44
openstackgerritMerged stackforge/kolla: Remove most kolla-common.sh code  https://review.openstack.org/21083812:44
openstackgerritMerged stackforge/kolla: Remove config-internal for glance  https://review.openstack.org/21087412:44
openstackgerritVladislav Belogrudov proposed stackforge/kolla: Ansible should wait bootstrap container on first host  https://review.openstack.org/21112013:01
SamYaplethanks vbel13:01
*** dwalsh has joined #kolla13:02
vbelSamYaple: you are welcome :) I have another bug in queue, checking/testing it. I did not specify run_once for clean-up since we could fail to do so in previous runs. So clean-up runs on all hosts13:03
*** klint has quit IRC13:03
*** thumpba has joined #kolla13:19
*** diogogmt has quit IRC13:33
openstackgerritVladislav Belogrudov proposed stackforge/kolla: Keystone bootstrap fails if keystone service is running  https://review.openstack.org/21115813:35
*** jtriley has joined #kolla13:37
*** diogogmt has joined #kolla13:38
openstackgerritVladislav Belogrudov proposed stackforge/kolla: Keystone bootstrap fails if keystone service is running  https://review.openstack.org/21115813:41
*** banix_ has joined #kolla13:43
*** jtriley has quit IRC13:45
pbourkevbel: will the new block execute on a fresh system?13:45
*** banix_ is now known as banix13:46
pbourkevbel: i.e. will it say 'cleaning up from previous run' even if there is no previous run13:46
vbelif no runs - no problem. We just tell ansible no keystone should run13:46
openstackgerritAlicja Kwasniewska proposed stackforge/kolla: Added template for base image, added new method generating Dockerfiles from templates  https://review.openstack.org/21028813:47
vbelI know left over keystone is somewhat unreal but it happened for me because I wanted to re-run bootstrapping13:48
vbelunfortunately we don't have status from bootstrapping container. I will close the bug if we decide user must take care him/herself13:49
sdakevbel I think you need a different solution for https://bugs.launchpad.net/kolla/+bug/148325913:50
openstackLaunchpad bug 1483259 in kolla "kestone bootstrap fails if previous keystone running" [Medium,In progress] - Assigned to Vladislav Belogrudov (vlad-belogrudov)13:50
sdakeI left it as valid and marked it for l3, but the proper approach would be a new integration point rather then jamming into the current playbook runs13:50
sdakevbel my theory on software dev is if it happens in development it will happen in deployment13:51
sdakeso we should strive to find solutions for these sorts of problems13:51
vbelok, I have abandoned the change but left bug open13:52
vbelThanks for reviewing!13:52
sdakea better approach would be to keep the review open, and ust modify the commit for the change id that way the comments are not lost13:53
sdakebut ymmv ;-)13:53
sdakeit would also help if you put in the bug how you got  to this state13:54
vbelwill update the bug, thanks13:55
openstackgerritAlicja Kwasniewska proposed stackforge/kolla: Added template for base image, added new method generating Dockerfiles from templates  https://review.openstack.org/21028813:55
sdakeso someone else can confirm, as per our bug triage process13:55
*** jtriley has joined #kolla13:56
sdakejtriley around?13:57
*** jpeeler has joined #kolla14:08
sdakejpeeler14:11
sdakeany chnace you can sort out build.py in the gate14:11
jpeelerthere a chance yes heh14:12
jpeeleri haven't forgotten about it though14:12
sdakei am working on source builds in the gate14:12
sdakeand they need parallel to build in time14:12
*** diogogmt has quit IRC14:14
sdakeakwasnie your template looks correct but seems too complex imo14:15
sdakefrom source build takes real86m17.636s14:16
sdakeand 8 containers didn't build14:16
openstackgerritMerged stackforge/kolla: Ansible should wait bootstrap container on first host  https://review.openstack.org/21112014:17
sdakeakwasnie strive for simplicity in the templates not optimization ;-)14:17
*** unicell1 has joined #kolla14:18
*** unicell has quit IRC14:20
sdakesamyaple does osad take 90 minutes to deploy?14:22
*** sdake_ has joined #kolla14:23
*** banix has quit IRC14:26
*** sdake has quit IRC14:27
*** jtriley has quit IRC14:29
sdake_i can see why the docker cats don't put conditionals in docker building14:31
vbelThey will end up with Makefiles :)14:33
*** sdake has joined #kolla14:34
*** sdake_ has quit IRC14:37
sdakejpeeler which toool did you use to determine we take up #4 spot in image sizes in the docker hub?14:38
jpeeleri didn't do any calculations, it was reported on the blog14:38
jpeelerand it was number of repos, just FYI14:39
sdakewhich blog14:39
jpeelerhttps://labs.ctl.io/docker-hub-top-10/14:39
sdakelooks like scratch wuld make a good data container source ;-)14:43
*** banix has joined #kolla14:48
openstackgerritPaul Bourke proposed stackforge/kolla: Implement Murano source images  https://review.openstack.org/20990514:49
pbourkehmm I wonder if we've been a bit hasty in removing config-internal for all services?14:53
pbourkesome don't have ansible roles yet14:53
sdakethe solution to that problem is ansible roles14:53
pbourkei know, but my point is some containers are now unusable14:54
pbourkeuntil ansible is implemented14:54
sdakeit is more important to have a stable abi for liberty 3 then full ansible support for supplemental services14:54
sdakeagree we need to get cracking on the ansible implementation14:55
pbourkeone step back, 2 forward I suppose :)14:55
sdakebut the thing is, if you here using config-internal prior, it wasn't compatible with ansible14:55
sdakeso you could test the container but not deployment14:55
openstackgerritPaul Bourke proposed stackforge/kolla: Fix missing config in horizon source image  https://review.openstack.org/20941514:55
openstackgerritPaul Bourke proposed stackforge/kolla: Fix missing config in horizon source image  https://review.openstack.org/20941514:56
sdakerhallisey pretty much need horizon asap for ansible ;-)14:56
sdakeyour assigned, you plan to tackle that soon?14:56
openstackgerritPaul Bourke proposed stackforge/kolla: Fix missing config in horizon source image  https://review.openstack.org/20941514:57
*** blahRus has joined #kolla14:57
rhalliseyya I was doing ceph atm14:57
openstackgerritSteven Dake proposed stackforge/kolla: Remove tox -egenconfig since its not needed for cinder  https://review.openstack.org/21122115:02
sdakepbourke which containers are you concerned with?15:04
sdakehorizon ?15:04
*** Verilium has quit IRC15:04
pbourkesdake: I dont mind too much, fully support the config-external method15:04
*** daneyon has joined #kolla15:05
pbourkesdake: just seeing some tweaks getting left behind when throwing out config-internal that might be useful to reference when doing the ansible role15:05
*** daneyon has quit IRC15:05
*** daneyon has joined #kolla15:06
sdakeagree, we can reference the l2 tag for that however15:06
sdakebut if there are ansible roles you ned first lets prioritize those15:07
sdakesince your cracking on murano which would go in supplemental, you may want horizon15:07
*** diogogmt has joined #kolla15:07
sdakehttps://blueprints.launchpad.net/kolla/+spec/restructure-template15:08
sdakepbourke ^^15:10
pbourkewha's supp15:11
pbourke:p15:11
sdakesupplemental is what it stands for15:11
sdakestuff not required for a basic operating openstack cluster15:12
pbourkeI know just being stupid15:12
pbourkeim assuming we want to avoid the word core?15:12
pbourkeah yeah I see SamYaple's comment15:13
sdakethe top description says "basic"15:13
sdakei just didn't fix the cut and paste below15:13
sdakethere fixed15:14
*** jtriley has joined #kolla15:16
*** diogogmt has quit IRC15:18
*** diogogmt has joined #kolla15:19
*** shakamunyi has joined #kolla15:27
*** jruano has joined #kolla15:29
*** thumpba has quit IRC15:30
*** daneyon has quit IRC15:32
*** daneyon has joined #kolla15:32
*** thumpba has joined #kolla15:32
sdakejtriley around?15:33
*** SourabhP has joined #kolla15:35
*** thumpba has quit IRC15:37
*** thumpba has joined #kolla15:37
*** thumpba has quit IRC15:38
*** thumpba has joined #kolla15:38
*** thumpba has quit IRC15:40
*** thumpba has joined #kolla15:41
*** thumpba has quit IRC15:44
*** thumpba has joined #kolla15:45
*** jtriley has quit IRC15:48
*** tfukushima has quit IRC15:57
*** tfukushima has joined #kolla15:58
*** tfukushima has quit IRC15:58
*** tfukushima has joined #kolla15:58
*** unicell1 has quit IRC16:03
*** athomas_ has joined #kolla16:04
*** jtriley has joined #kolla16:06
*** SourabhP has quit IRC16:06
*** athomas has quit IRC16:07
*** pbourke has quit IRC16:08
*** pbourke has joined #kolla16:08
pbourkecan anyone tell me, what's the purpose of this directory? https://github.com/stackforge/kolla/tree/master/etc/kolla/config16:12
bmacei think you can add stuff to those and they get merged into the resulting config that gets pulled into the container?16:13
pbourkeah yeah, they're the augment files16:15
sdakeright16:16
pbourkemaybe we should put some comments in those or something16:17
pbourkelooks a little strange having a dir full of empty files16:17
*** jasonsb has quit IRC16:20
*** rhallisey has quit IRC16:21
*** shardy has quit IRC16:25
*** tfukushima has quit IRC16:28
pbourkethis is a bit weird :/16:29
pbourkewhy are there like 5 config sources for each of the glance services16:29
pbourkehttps://github.com/stackforge/kolla/blob/master/ansible/roles/glance/tasks/config.yml#L6-L1016:30
*** athomas_ has quit IRC16:32
vbelmerge one into another?16:34
pbourkestarting to see it now but it seems overkill16:34
pbourkeone template + one augment per service would be enough16:34
*** vbel has quit IRC16:36
*** unicell has joined #kolla16:36
*** tfukushima has joined #kolla16:37
*** tfukushima has quit IRC16:37
*** jruano has quit IRC16:38
sdakepbourke database eg - you specify one database config and that merges to all services16:38
sdakepbourke everything gets merged into the j2 template on line 616:39
sdakeiirc we shoudl have a messaging config file per service as well that loads globally16:39
sdakethen glance.conf is the default set of config files16:39
sdakeand glance/glance.conf.aug is an augment16:39
sdakenot sur why samyaple has two there16:39
sdakeglobal.conf iirc contains port ids and stuff16:40
pbourkesdake: thanks yeah I see it now. just took me longer than I expected!16:40
pbourkethe glance.conf threw me off also16:40
sdakeregarding comments, samyaple did say he wanted to add comments16:41
sdakebut hasn't got to it16:41
sdakeso glance.conf is for all glance services (a global config augment)16:42
sdakeglance registry.conf is for registry only16:42
sdakecan someone approve this rhallisey jpeeler https://review.openstack.org/#/c/211123/16:43
*** rhallisey has joined #kolla16:49
*** britthou_ has joined #kolla17:04
*** sdake_ has joined #kolla17:04
jpeelersdake: is there really a problem with the fedora build being in the gate?17:04
*** jruano has joined #kolla17:05
sdake_jpeeler i am not sure but i am not keen on running red hat's hacked version of the gate for source builds17:05
sdake_rather gate/docker17:06
sdake_it definately wont work for push17:06
sdake_because it asks you to confirm a push operation with [Y/n]17:06
sdake_the main issue is our docs recommend using the docker inc version17:07
sdake_which ends upw ith docker group vs dockerroot17:07
sdake_and tox -e doen't work with docker inc's vresion17:07
*** shardy has joined #kolla17:07
*** britthouser has quit IRC17:07
sdake_because we use sudo -g dockerrot vs sudo -g docker17:07
*** sdake has quit IRC17:07
sdake_i dont get why rht has variance in docker17:08
sdake_its a standard implementation ffs17:08
jpeelerfor what it's worth, i think --confirm-def-push=false turns off the confirmation. i don't get the deviation17:08
jpeelerok well, can we just the yum provided builds here instead of curling it? http://yum.dockerproject.org/repo/main/17:09
*** SourabhP has joined #kolla17:09
*** britthou_ has quit IRC17:10
*** jasonsb has joined #kolla17:10
*** shardy has quit IRC17:12
*** jasonsb has quit IRC17:14
*** jasonsb has joined #kolla17:14
*** sdake_ is now known as sdake17:14
*** achanda has joined #kolla17:14
*** banix has quit IRC17:23
sdakeinteresting i was wondering how to turn tht annying feature off17:23
sdakejpeeler yum provided builds work for me although the nice thing about adding the repository to the gate is yum figures ou thte version # automatically17:24
sdakeI'm unclear of our version of fedora can change or not17:24
sdakeof/if17:25
*** inc0 has quit IRC17:29
*** bmace has quit IRC17:31
*** dwalsh has quit IRC17:31
*** sdake_ has joined #kolla17:32
*** akwasnie has quit IRC17:33
openstackgerritSteven Dake proposed stackforge/kolla: Add centos-src-build tox gate  https://review.openstack.org/21112817:34
openstackgerritSteven Dake proposed stackforge/kolla: Use docker-engine in setup_gate  https://review.openstack.org/21112317:34
*** sdake has quit IRC17:35
*** gfidente is now known as gfidente|afk17:35
*** jruano has quit IRC17:39
*** jruano has joined #kolla17:43
*** britthouser has joined #kolla17:45
*** dwalsh has joined #kolla17:47
*** britthouser has quit IRC17:50
*** britthouser has joined #kolla17:50
*** bmace has joined #kolla17:50
*** SourabhP has quit IRC17:57
*** dwalsh has quit IRC17:58
jpeelerhrm, i didn't realize that the docker.io bash script would set up the fedora rpm17:58
jpeeleri mean repo17:58
jpeelerwill comment on the review17:59
*** dwalsh has joined #kolla18:01
*** inc0 has joined #kolla18:04
sdake_its all good i did what you wanted ;)18:05
sdake_if you want me to do something else just le tme know18:05
sdake_rhallisey do you mind if I take horizon blueprint?18:05
sdake_rhallisey you have 12 bueprints, and no offense, but your not superman, I don't have high confidence youc an get em all done ;)18:05
rhalliseysdake_, ya sure if you have work for it18:05
rhalliseyI'm like 1/2 way maybe18:05
sdake_can you post your review18:06
sdake_and i'll take it over18:06
rhalliseykk18:06
sdake_i just want to get cracking on stuff18:06
rhalliseyI'm not super man18:06
sdake_dont want us single threading18:06
sdake_i am looking for work18:06
sdake_i am in the process of pushing stuff to mitaka as well18:06
sdake_we can discuss at the team meeting - also expect an email on the ml about it and priorities18:06
sdake_we barely got 20 blueprints finished last time18:07
sdake_and our community hasn't grown since last cycle by more then 1 or 2 people18:07
sdake_and we have more bug testing/filing going on ;)18:08
*** jtriley has quit IRC18:08
sdake_jpeeler are you working on ansibilizing ironic18:08
sdake_or just making a container for it18:09
jpeeleri thought you wanted me to do both18:09
jpeelerbut i don't know ansible, so if you want to do that that's fine18:09
sdake_if you think you can tackle it wfm18:09
*** banix has joined #kolla18:09
sdake_i dont want to do th eansible part no18:09
*** britthou_ has joined #kolla18:09
sdake_we have 3 weeks, just looking to unload work to mitaka18:09
*** inc0 has quit IRC18:10
openstackgerritRyan Hallisey proposed stackforge/kolla: Add ansible support for Horizon  https://review.openstack.org/21128618:10
sdake_wonder what ever happened to fang18:10
sdake_hope he is ok18:10
rhalliseysdake_, ^ go ahead18:10
rhalliseywas dealing with httpd config18:10
sdake_thanks rhallisey18:11
sdake_you actually ahve a bunch of work done ;)18:11
sdake_nice job18:11
rhalliseyya I put down getting ceph cluster going18:11
rhalliseynow I can go back to that!18:11
*** britthou_ has quit IRC18:12
*** jtriley has joined #kolla18:12
*** britthouser has quit IRC18:12
sdake_oh sorry18:14
sdake_if you want to finish the job on horizon that wfm18:14
sdake_i am just tyring to unload your plate :)18:14
sdake_i'd like folks to hve no  more then 6 blueprints if possible18:15
*** britthouser has joined #kolla18:16
harmwi'd just like more time in a day, think you can get me that sdake_ ?18:20
rhalliseysdake_, I don't care18:20
sdake_harmw cant work that magic - you figure out how to make the moon on a greater orbit, i'll do it ;)18:20
harmwgot some other projects that need my attention, hence my absence18:20
sdake_harmw ya very few kolla devs are f ull time18:20
harmwyea but even then18:21
sdake_but we appreciate your part time help ;)18:21
harmw:)18:21
*** britthouser has quit IRC18:21
sdake_especially reviews becuase our core reviewer team is too small18:21
harmwdid I mention I realy hate building websites?18:21
sdake_we need atleaset 1 more cat imo18:21
sdake_is that what you do for a living?18:21
harmwwell I do try to keep reviewing :)18:21
sdake_harmw to be honest I dont see why you dont get an openstack job dude18:21
harmwno, god no18:21
harmwuhm, websites building is spare time18:22
sdake_your an openstack developer and have good karma in the commnity ;)18:22
sdake_did I mention I hate computers ;-)18:22
harmwopenstackjob... maybe someday :)18:22
harmwlol18:22
sdake_when does mandre reappear :)18:22
* sdake_ wtb's mandre18:22
rhalliseyend of August18:22
rhalliseylike 1 week18:22
harmwyou guys heard of softwarecircus.io btw?18:23
sdake_rhallisey I cranked up ceph to essential, I really want that done for liberty18:23
rhalliseysdake_, kk I'll switch my attention to that unless you want me to take horizon back18:24
sdake_nah i'll take horizon18:24
rhalliseyk18:24
sdake_i kicked magnum to mitaka18:24
sdake_that is what i was going to work on next18:24
sdake_while also working on source gating ;)18:24
sdake_down to a manageble 24 blueprints ;)18:27
rhalliseyI need to fix ansible cinder to get this working since config internal is gone18:30
rhalliseycoolsvap, you cool if I send you patches for cinder-ansible?18:31
rhalliseyoh nvm looks like you made some changes18:32
*** SourabhP has joined #kolla18:32
*** sdake_ is now known as sdake18:35
sdakerhallisey if you https://blueprints.launchpad.net/kolla/+spec/ceph-container have started on that mark it started plz18:39
rhalliseysdake, yup, all set18:40
sdakerhallisey you will be doing ansible for ceph as well?18:40
rhalliseyya I'll handle it18:40
rhalliseyI'm going to try and help finish off cinder ansible then move to that18:41
sdakerhallisey do you need the data containers for tripleo?18:43
rhalliseyno, I just thought we needed them in general18:44
*** gfidente|afk is now known as gfidente18:44
*** jruano has quit IRC18:44
openstackgerritJeff Peeler proposed stackforge/kolla: Improve logging and prepare for gate  https://review.openstack.org/20221618:47
jpeelersdake: prerequisite to getting build.py in gate ^18:47
sdakejpeeler let me look at the last review and make sure the issues have been solved18:48
sdakeit will be after i finish the job on this launchpad pounding18:48
sdakeso give me 1 hour or so18:49
jpeelerno rush18:52
*** jruano has joined #kolla18:52
rhalliseycoolsvap, around?18:52
*** bmace has quit IRC18:55
sdakerhallisey if you dont need them for tirpleo, i am goin to remove them18:56
rhalliseyall the data containers?18:56
sdakeand you can use the api container with a different name for the data container18:56
rhalliseyare we going to have one?18:56
sdakethats the plan18:56
sdakeI want to lock down the container ABI for liberty18:56
sdakeafter liberty I'd like minimal change unless we fubared something18:57
sdakethat way tripleo has a stable base for liberty/mitaka to get the job done18:57
rhalliseyk sounds good18:59
*** britthouser has joined #kolla19:02
sdakeif we fubared something we will change it19:03
sdakebut its getting down to very basic container content now19:03
sdakeoru containers copy a file in and install packages19:04
sdakecan't get much more simple then that19:04
sdakerhallisey note I changed https://blueprints.launchpad.net/kolla/+spec/ceph-container19:05
rhalliseynoted19:05
*** britthouser has quit IRC19:26
*** britthouser has joined #kolla19:26
*** sdake_ has joined #kolla19:29
*** bmace has joined #kolla19:31
*** sdake has quit IRC19:33
*** sdake has joined #kolla19:35
openstackgerritSteven Dake proposed stackforge/kolla: Add centos-src-build tox gate  https://review.openstack.org/21112819:36
openstackgerritSteven Dake proposed stackforge/kolla: Use docker-engine in setup_gate  https://review.openstack.org/21112319:36
*** britthouser has quit IRC19:36
sdakeok jpeeler looking at your compelx review now ;-)19:37
*** sdake_ has quit IRC19:38
bmacewhats up sdake? :)19:39
*** loth has left #kolla19:39
*** britthouser has joined #kolla19:40
*** echoingumesh has joined #kolla19:43
*** echoingu_ has joined #kolla19:44
sdakejpeeler can you expalin the threads thing plz19:45
sdakei dont get why you special case threads=119:45
*** echoingumesh has quit IRC19:47
jpeelersdake: if you specify just one thread, that allows realtime output from the docker building19:49
sdakeoh didn't know that19:49
sdakeyou mean each docker operation prints out?19:50
sdakeor just the completed docker op19:50
jpeelerdue to the thread handling, that's not possible with multiple builds occurring. so for each build, when multiple threads are in use the logs are not outputted until completely finished19:50
sdakeI would expect line 92 to be in a loop for tha t to work?19:50
sdakefor each item in the log to be printed19:51
sdakerather each utput of each docker operation19:51
sdakei see its in a for loop19:52
sdakegot it19:52
jpeeleryeah19:52
sdakeI just assume that works - have no idea how they actually implemented that in python ;-)19:52
jpeelerlast time i tested it it did19:53
sdakecool well i  -1'ed your review19:53
sdakebut not for that reason19:53
sdakethat is a good improvement19:53
sdakeI recommend a comment19:53
sdakeand adding something to indicate running with threads=1 will provide realtime output19:53
sdakethat could come in handy19:53
sdake(to the hep of -T)19:53
sdakehelp of that is19:54
*** jruano has quit IRC19:55
*** echoingu_ has quit IRC19:57
*** echoingumesh has joined #kolla19:59
*** dwalsh has quit IRC20:41
*** echoingumesh has quit IRC20:46
*** jasonsb has quit IRC20:46
*** jasonsb has joined #kolla21:01
openstackgerritJeff Peeler proposed stackforge/kolla: Improve logging and prepare for gate  https://review.openstack.org/20221621:02
* jpeeler is slow today21:02
openstackgerritRyan Hallisey proposed stackforge/kolla: Add ansible support for Cinder  https://review.openstack.org/20886821:05
*** achanda has quit IRC21:07
*** mfalatic has joined #kolla21:09
*** thumpba has quit IRC21:12
rhalliseyjpeeler, it's better than being Slower21:15
rhallisey:P21:15
*** bmace has quit IRC21:16
*** rhallisey has quit IRC21:18
*** bmace has joined #kolla21:33
*** achanda has joined #kolla21:40
*** bmace has quit IRC21:47
*** tfukushima has joined #kolla21:47
*** echoingumesh has joined #kolla21:54
*** echoingumesh has quit IRC21:59
*** bmace has joined #kolla21:59
*** sdake has quit IRC22:01
*** echoingumesh has joined #kolla22:02
*** jtriley_ has joined #kolla22:07
*** jtriley has quit IRC22:10
*** jtriley_ has quit IRC22:11
*** echoingumesh has quit IRC22:28
*** banix has quit IRC22:50
*** diogogmt has quit IRC22:56
*** jtriley has joined #kolla22:59
*** jtriley has quit IRC23:11
*** echoingumesh has joined #kolla23:19
*** jasonsb has quit IRC23:30
*** jasonsb has joined #kolla23:31
*** sdake has joined #kolla23:31
*** jtriley has joined #kolla23:31
*** tfukushima has quit IRC23:32
sdakeyo23:34
bmaceyo indeed23:34
*** jasonsb has quit IRC23:35
SamYaplehello23:42
*** blahRus has quit IRC23:42
SamYaplesdake: i have a ping from you but cant find it in scrollback23:42
sdakesamyaple no idea bro23:43
sdakeprobably no tworth tlakingabout ;)23:43
sdakenothing i can recall I wanted to discuss23:44
*** daneyon has quit IRC23:44
sdakeso have physical therapy 3 days a week groan23:44
*** daneyon has joined #kolla23:44
sdakeso i'll be out mwf likely for a couple hours23:45
SamYaplelol ok23:47
*** tfukushima has joined #kolla23:52
*** sdake has quit IRC23:53
openstackgerritSam Yaple proposed stackforge/kolla: Update nova.conf and ports  https://review.openstack.org/21096023:55
*** sdake has joined #kolla23:56
*** tfukushima has quit IRC23:57

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