Monday, 2015-08-17

openstackgerritSteven Dake proposed stackforge/kolla: Remove compose from documenation  https://review.openstack.org/21340100:03
*** sdake_ has joined #kolla00:04
*** sdake has quit IRC00:07
*** ccrouch has quit IRC00:08
*** sdake has joined #kolla00:20
*** sdake_ has quit IRC00:23
*** achanda has quit IRC00:26
openstackgerritSteven Dake proposed stackforge/kolla: Add Ansible support for Heat  https://review.openstack.org/20552800:59
*** weiyu_ has joined #kolla01:00
openstackgerritSteven Dake proposed stackforge/kolla: WIP: Add Ansible support for Heat  https://review.openstack.org/20552801:12
*** erkules_ has joined #kolla01:26
*** erkules has quit IRC01:29
*** unicell1 has quit IRC01:52
*** unicell has joined #kolla01:52
*** banix has quit IRC02:09
*** unicell1 has joined #kolla02:12
*** unicell has quit IRC02:13
sdakesamyaple are you alive02:20
*** weiyu_ has quit IRC02:22
*** weiyu_ has joined #kolla02:22
*** klint has joined #kolla02:50
SamYaplesdake: yea03:18
sdakehey sam03:19
sdakemind taking over that bootstrap removal patches for nova and neutron03:19
sdakeI've got a demo coming up03:19
sdakelooks bit sloppy with that in there03:19
SamYapleyep03:19
SamYaplei was waiting to hear back form coolsvap|away but ill do it03:19
sdakecool thanks i doubt he would mind03:20
sdakei've almost got heat ported over to the new structure03:21
SamYapleoh03:21
SamYaple i have code for that...03:21
SamYapleit doesnt matter03:21
sdakeis it complete?03:21
SamYapleits being held up by the shade modules anyway03:21
SamYapleyea except for the shade modules in the project03:22
sdakei've got the gate working for templating for binary and source03:22
sdakehammered on it all day03:24
sdakepainful03:24
SamYapleyea i see alot of patches from you03:24
sdakelots of easy reviews in the commit queue03:24
sdakedid zaqar and murano03:24
sdakewe have 5 contianers left03:24
sdakeand a slew of reviews outstanding03:25
SamYaplemandre: all these distros aren't source-based distros. There is no way to install these other applications from source and then all the deps etc etc. You would be doing a LFS build. I don't know what there is to discuss on that point to be honest03:26
SamYaplethe openstack stuff can be installed via python/pip which is the only 'from-source' anyone cares about when refering to openstack03:26
sdakesamyaple i have a few technical quesitons about how the ansible code currently operates, have a few minutes?03:37
SamYaplesure03:39
SamYaplebut first03:39
SamYaplehttp://gliderlabs.viewdocs.io/docker-alpine/usage/03:39
SamYaplelook at the virtual packages section!03:39
SamYapleme and a coworker are building kolla images around alpine03:39
sdakeya alpine is the hot new hotness apparently03:40
sdakereal2m58.870s03:41
sdaketime spent deploying heat03:41
sdakeso read that article03:41
SamYapleok03:41
sdakeok questions03:42
sdakeif a new container is avaialble in the registry and i run a deploy operation does the new image get started?03:42
SamYapleyes03:42
SamYaplethe 'reloaded' flag for the ansible module/docker-py03:42
SamYaplethat means if anything is different (image hash/volumes/env) reload the image03:43
SamYaplewhich means new container03:43
sdakeis there a way to query the current hash for every container in the deployment if we wanted that feature?03:43
SamYapleno ansible module for it but I could write one no problem03:43
SamYapleansible isnt good at returning info though03:43
sdakei think that will be important03:43
sdakemaybe not immediately03:44
sdakebut down the road03:44
SamYaplei dont think you really want ansible for that though03:44
SamYapleansible does things, it doesnt return things03:44
SamYaplelol i have a branch called 'make_sdake_happy'03:45
sdakeis there a way to mirror an external registry with an internal one03:45
SamYaplesdake: what do you mean?03:46
sdakei mean take docker registry and load it into my internal registry03:46
sdakelike docker pull and push in one :)03:46
SamYapledocker pull/push?03:46
SamYapleah i dont think so03:46
SamYaplei have seen some talk about it but its not a priority03:46
SamYapleits talk related to fixing v2 registry03:47
sdakewould docker pull followed by push do the job?03:47
SamYapleno03:47
SamYapledocker pull/docker tag/docker push03:47
sdakenever used docker tag03:48
SamYapleits how you tag images03:48
sdakei gathered that by the name03:48
SamYaplejust making sure03:48
sdakebut if you pull, you pull latest03:48
SamYapleno03:48
sdakewhy does it need to be tagged lateset agin?03:48
SamYapleif you git pull with no tag it will try to pull the 'latest' tag03:48
SamYaplewhich we dont have at all03:48
SamYapleif you docker pull with a tag, it will pull that tag03:49
SamYapleyou can even docker pull a hash03:49
sdakecan you add a compute node to the inventory, run deploy, and that compute node will be deployed and operational?03:50
SamYaplesince you have asked that question at least twice in the past 3 days, can you write down the answer?03:50
SamYaple'yes'03:50
sdakejust verifying :003:51
sdake;) i mean03:51
SamYaple=O03:51
sdakeif i kill the main ha proxy, how long does it  take to recover to a redundant haproxy?03:51
SamYaplethats controlled by the keepalived script/values03:51
SamYaplehold03:52
sdakethnks forchecking03:52
SamYaplehttps://github.com/stackforge/kolla/blob/master/ansible/roles/haproxy/templates/keepalived.conf.j2#L3-503:52
SamYapleso it takes 2 intervals of 2 seconds of being down to switch03:52
SamYapleso 4 seconds03:52
sdakenice03:53
sdakeahve you tested  that?03:53
SamYapleit takes 20 seconds of successful checks to be 'up'03:53
SamYapleyup03:53
sdakehow wuld you feel about putting the ports in a separate config file03:54
SamYaplecant hot reload them that way03:54
sdakewhy not?03:54
SamYaplethe hot reload only works with a single file according to inc003:54
sdakei see03:55
sdakethats annying :(03:55
SamYapleI haven't verified that, but it is haproxy, so i wouldn't doubt it03:55
sdakeis the keepaived interval tuff configurable?03:55
SamYaplenope03:55
SamYaplethe only things configurable are the ini-style configs03:55
SamYaplethe best I can offer is a complete override file03:56
SamYaplenot configurable, but entire file is overridable03:56
sdakedont like that much03:56
sdakesomeone likely to fubar their system03:57
sdakedown the road there is no other way to do  the job of confuginrg just those two things?03:57
SamYaplewe can't stop people from breaking thier system03:58
SamYapleif they have need to change these things, they can break thier system03:58
SamYapleno matter what we do they can break thier system03:58
SamYaplei wont make decisions around trying ot prevent people from breaking the system03:59
SamYapleany merge type setup on the other files will be way more likely to break the stuff than a full override file03:59
sdakea perfect system cannot be broken ;)03:59
SamYaplethere is no such thing as a perfect system. i wont entertain you on this04:00
sdakewhat does an ovveride file look like for keepalived?04:01
SamYaplea complete override. like the template but with your options04:02
sdakethe override template is parsed by ansible to fill in values?04:03
SamYapleyea its all run through ansible templates04:03
sdakeone option is to write out the override file with just the two things we care about04:04
*** banix has joined #kolla04:04
sdakewhere is this template - i dont see i nsource base04:04
SamYaple03:52 < SamYaple> https://github.com/stackforge/kolla/blob/master/ansible/roles/haproxy/templates/keepalived.conf.j2#L3-504:05
sdakeyoucoudln't do interval {{ openstack_keepalive_interval }} there?04:06
SamYaplesure. but i dont want that. youll never be able to template all the options. If we go down the path of templating them, we will be adding new things forever.04:07
sdakeor olla_keepalive_interval?04:07
SamYapleThis is the OSAD paradox04:07
SamYaplewhat about an option that isnt even o nhere?04:07
SamYaplewhat about unicast vs multicast?04:07
SamYaplehaving a full override solves everything04:07
SamYapletemplating causes overhead04:07
sdakeif my 10 years of experience with ha tells me anything people will want to tune those values04:08
sdakewithout having to be a rocket surgeon04:08
SamYapleright they will04:09
SamYapleand they will want to tune options we dont have04:09
sdakewhat other shit did osad end up templating in?04:09
SamYaplethats my point04:09
sdakei want to keep config options to an absolute minimum04:09
SamYapleyou cant have it both ways04:10
sdakethatis why i asked about osad paradox04:10
SamYapleyou know all about it04:10
sdakeall the shit they hve tunable is because customers asekd for it04:10
SamYaplea dev-cycle to template an option04:10
*** britthouser has joined #kolla04:11
SamYaplefull override == no dev-cycle04:11
SamYaplefull operators control04:11
SamYapleexactly what it needs to be04:11
SamYapletemplating shit will get us nowhere fast and no one will be happy about it04:11
sdakeinteresting04:11
sdakeone last technical question04:11
sdakewhat is the echnical difference between external address and internal address04:11
sdakeatm i set them to the same thing04:12
SamYapleas answered before, the external is the address that sets the 'public_endpoint' for openstack04:12
*** britthou_ has joined #kolla04:12
SamYaplein a public cloud this would be public addresses04:12
sdakesorry i mut have missed yur answer04:12
sdakethat maks sense04:12
sdakei dc alot to switch to email network04:12
SamYaplebut you have responded with 'that makes sense' before04:13
sdakewhat serves it?  haproxy?04:13
SamYaplenothing serves the external address at all04:13
SamYaplethat would be on the deployer to setup04:13
SamYaple1:1 nat maybe04:13
SamYaplethats what I would do04:13
sdakei recall you saying that would be onthe deployer before04:14
sdakehow does a 1;1 natwork04:14
SamYapleIt _should_ be a dns address04:14
SamYaplethat would be the proper way to do it04:14
SamYapleso its not even an ip04:14
SamYaplebut it _can_ be an ip04:15
sdakedns gets converted to ip by resolver lib04:15
*** britthouser has quit IRC04:15
SamYapleon the client04:15
SamYaplenotihng to do with the deployment04:15
sdakehere is where i get stuck - nothing forwards that traffic to the internal network04:16
sdakethat would be done by a 1:1 nat?04:16
*** openstack has joined #kolla04:19
sdakedesign makes sense04:20
sdakei think we need to think more about implementing a 1:1 nat or documenting how one could be implemented04:20
sdakeotherwise people a re going to go wtf when they have their privat enetwork isolated from their endpoint network04:20
SamYaplei have no problem with whatever is in docs04:20
SamYaplewe cannot implement it in kolla though...04:21
sdakecool I'll start a faq document04:23
sdakethese are questions i get sked over an over04:23
sdakewhich is why im asking you04:23
*** weiyu_ has quit IRC04:25
sdakei'll consult daneyon on docs for a 1:1 nat solution04:26
sdakehe should probably have an idea04:26
sdakehe is a networking genius04:26
SamYapleim not sure what you really want to document there04:26
sdakehow to isolate your internal network from your endpoint network essentially04:27
sdakethat is the whole purpose of external_network04:27
SamYaplethats realy not whats happening04:27
SamYapleat all04:27
SamYapleyou seem to have external network and public endpoint confused04:28
sdakepossible expand pls04:28
SamYaplei dont know what you want me to expand on.04:28
SamYaplei dont know what you dont know04:29
sdakelets take my network for example04:29
sdakemy public endpoints are 192.168.1.14904:29
sdakemy prviate network is 192.168.1.z04:29
sdakesomeone could connect to 192.168.1.100 3306 and DOS my database04:29
sdake100 is my machine where kolla is running04:30
SamYapleon your private network, yes04:30
SamYaplebut your public endpoint is also on your private, thats your choice04:30
SamYaplenothing to document there04:30
*** diogogmt has quit IRC04:30
SamYapleif you want it available make it a public and route it04:30
SamYaplethis is basic stuff not a kolla issue04:30
sdakeso a router handles that?04:31
SamYapleor any number of other methods04:31
sdakecool never played with a router04:31
sdakehave no idea how they work ;-)04:31
sdakedoes the router forward traffic to the VIF blocking ports that are not pubilc endpoints?04:33
SamYaplei have no idea what you set your router up to do04:33
sdakei mean in a real deployment04:33
SamYapledoesnt change the answer04:34
sdakei dont havea router in my home setup - just cable modem and wireless "router"04:34
sdakeit does have a nat built in and port forwarding04:34
sdakei guess i could assign openstack ports to my vif04:35
sdakeand set my openstack external to my public ip04:35
sdakeif that is the model, then i got it04:35
sdakestandard networking04:36
sdakeis that the model?04:36
SamYaplethere is no model. you can do that yes04:37
SamYapleits a deployers decision04:37
sdakecool thanks for bearing with me :)04:38
sdakeyour right not an impelemtnation issue for kolla to sort out04:41
sdakedefinately a documentation issue04:41
sdakebasically the only answer I can see is nat04:42
sdakeboy networking is hard04:44
sdakebut as my wife says, "your company didn't build its empire because networking is easy."04:44
sdakesamyaple here is the templating gate: https://review.openstack.org/#/c/213538/04:48
sdake16 and 13 min runtimes04:48
sdakedo you know why a knife must have a sheath?04:57
sdakevictory got heat stack-list working04:59
openstackgerritSam Yaple proposed stackforge/kolla: Remove start.yml abstraction for kolla-ansible  https://review.openstack.org/21360005:04
*** daneyon_ has quit IRC05:13
*** daneyon has joined #kolla05:14
openstackgerritSam Yaple proposed stackforge/kolla: Remove Ansible abstraction for Nova  https://review.openstack.org/21144505:15
sdakesay samyaple should the registry have a data container associated with it05:18
sdakeglance-registry05:18
*** weiyu has joined #kolla05:18
SamYaplefor what?05:18
sdakei dont know05:18
sdakei have no idea what it does05:18
sdakemandre mentioned it05:19
SamYapleonly if it has persistent data05:19
SamYaplewhere is the persistent data stored, api or registry?05:20
sdakeno idea05:21
sdakehttp://docs.openstack.org/juno/config-reference/content/section_glance-registry.conf.html05:22
*** banix has quit IRC05:23
sdakelooks lik eit uses edb for storing images05:24
sdakesamyaple can you review this please05:30
sdakehttps://review.openstack.org/#/c/213412/05:30
*** achanda has joined #kolla05:30
*** weiyu has quit IRC05:30
sdakeand https://review.openstack.org/#/c/213401/05:30
sdakeTIA05:30
*** coolsvap|away is now known as coolsvap05:32
sdakeyo coolsvap05:36
coolsvapsdake, hi05:36
coolsvapsorry was away traveling05:36
coolsvapjust landed in seattle this evening05:36
sdakenice05:36
sdakei like seattle05:36
coolsvapwas at the containerday05:36
coolsvapmy first time05:36
sdakehow was it05:37
coolsvapit was nice, do you know dave nielsen?05:37
sdakedoesn't ring a bell05:37
coolsvapgood lightning talks05:37
sdakewhat was his about05:37
coolsvapand unconferences05:37
sdakeanything on openstack deployment?05:37
coolsvaphe organized it, i think he's kinda doing the container days at different cities05:38
coolsvapvery litte05:38
coolsvapthe docker volume-driver implementation -> flocker05:38
coolsvapfrom clusterhq05:38
coolsvapbut their openstack integration is at very basic state05:39
coolsvapthey are still playing around vagrant05:39
coolsvapbut good concept for stateful applications on docker05:39
coolsvaphttps://github.com/ClusterHQ/flocker05:40
coolsvapapart from that there were basic talks on docker05:42
sdakefamiliar with it05:42
coolsvaphmm05:43
sdakesamyaple you ever tried out bluebox'es thing?05:46
sdakeor looked at it05:46
* coolsvap sleeping now or i will not get over the jet lag05:51
coolsvapand will sleep in the conf tomorrow :)05:52
sdakeenjoy05:54
*** coolsvap is now known as coolsvap|away05:55
*** weiyu has joined #kolla06:09
sdakeworking heat -> http://ur1.ca/ngjg5 -> http://paste.fedoraproject.org/255729/4397920006:13
sdakegreat stack delete doesn't do anything06:16
* sdake shakes fists06:16
openstackgerritSteven Dake proposed stackforge/kolla: WIP: Add Ansible support for Heat  https://review.openstack.org/20552806:26
sdakesamyaple your patch appears busted http://ur1.ca/ngjlc -> http://paste.fedoraproject.org/255733/9792993106:30
*** akscram has quit IRC06:32
*** akscram has joined #kolla06:33
sdakebedtime06:43
sdakelater06:43
*** sdake_ has joined #kolla06:44
*** sdake has quit IRC06:47
*** inc0 has joined #kolla06:56
inc0good morning kolla06:56
SamYaplemorning inc006:58
sdake_hey inc006:58
sdake_inc0 5 containers need jinja-2ing :)06:58
inc0hehe, allright, I have 3 hrs but should be enough for them;)06:59
sdake_atleat a few ;)06:59
inc0can I have link to etherpad please?06:59
sdake_takes aout 10-15 mins per i think06:59
sdake_yup sec06:59
sdake_https://etherpad.openstack.org/p/kolla-dockerfile-template07:00
inc0last week I was fighting with internal stuff, few commits would be good for my mental health07:00
sdake_make that 607:00
*** ccrouch has joined #kolla07:01
sdake_goal with these is they match existing implementation, not necessarily work07:01
sdake_mongodb being the exception07:02
sdake_since that is a dependency for ceilometer07:02
inc0lol, I'd feel bad with commiting whatever not working;)07:02
sdake_well you can't treally test them07:02
sdake_since config-internal is long gone07:02
sdake_but you can et that they build ;)07:02
inc0hmm, but these 6 have people assigned07:03
inc0bigger question tho07:04
inc0did anyone actually setup kolla-kilo openstack?07:05
inc0sdake_, you've mentioned that people needs that07:05
sdake_you mean besides me?07:05
-openstackstatus- NOTICE: Gerrit is currently under very high load and may be unresponsive. infra are looking into the issue.07:05
inc0well I didn't know you did that, but good to know07:05
*** ccrouch has quit IRC07:05
inc0if I've tried source build with different tarball, I ran into problems07:06
sdake_daneyon gave live demo at ods07:06
sdake_the images in docker registry work07:06
sdake_with the kilo banch ;-)07:06
inc0I need these built locally tho07:06
inc0did you use source build or change repos for binary?07:07
SamYapleinc0: he is talking about the kilo branch07:07
sdake_kilo only had binary building07:07
SamYaplenot master07:07
SamYaplewe dont do kilo in master07:07
sdake_bedtime07:08
sdake_later07:08
inc0I see, you mean https://github.com/stackforge/kolla/tree/stable/kilo ?07:08
*** sdake_ has quit IRC07:08
SamYapleyea07:08
SamYapleno one is deploying kilo from kilo branch07:08
openstackgerritSam Yaple proposed stackforge/kolla: Remove start.yml abstraction for kolla-ansible  https://review.openstack.org/21360007:11
SamYaplejeez i did that like 30 minutes ago07:11
openstackgerritSam Yaple proposed stackforge/kolla: Add heat to docker_templates  https://review.openstack.org/21362007:32
openstackgerritSam Yaple proposed stackforge/kolla: Add memcached to docker_templates  https://review.openstack.org/21362007:33
SamYaplejeez this stuff is taking for ever to appear07:39
openstackgerritSam Yaple proposed stackforge/kolla: Add mongodb to docker_templates  https://review.openstack.org/21362307:40
inc0meh07:49
inc0I'll wait with reviews till they solve this07:49
SamYapleye aits unusable at the moment07:49
*** shardy has joined #kolla08:09
*** achanda has quit IRC08:10
*** achanda has joined #kolla08:20
*** jmccarthy has quit IRC08:28
*** shardy_ has joined #kolla08:33
*** shardy has quit IRC08:34
*** achanda has quit IRC08:35
*** shardy_ has quit IRC08:38
*** shardy has joined #kolla08:39
*** jmccarthy has joined #kolla08:46
*** gfidente has joined #kolla08:48
*** ccrouch has joined #kolla08:50
*** ccrouch has quit IRC08:54
*** tfukushima has joined #kolla08:58
*** inc0 has quit IRC09:00
*** tfukushima has quit IRC09:02
*** inc0 has joined #kolla09:02
*** tfukushima has joined #kolla09:04
*** athomas has joined #kolla09:23
*** inc0 has quit IRC09:59
pbourkeSamYaple: hey got a minute?10:07
*** tfukushima has quit IRC10:11
*** tfukushima has joined #kolla10:12
SamYaplepbourke: yea whats up10:16
pbourkeSamYaple: just wondering if I could bounce some of the keystone config Im trying for swift off you. Its driving me demented, have spent almost two days now trying to debug it10:16
*** dims has joined #kolla10:17
SamYapleok10:17
pbourkeSamYaple: far as I can see its the same as what I was using for conifg-internal, but the proxy can't seem to authenticate itself10:17
pbourkeSamYaple: I'll put some pastebins together of the various parts10:17
-openstackstatus- NOTICE: review.openstack.org (aka gerrit) is going down for an emergency restart10:19
*** ChanServ changes topic to "review.openstack.org (aka gerrit) is going down for an emergency restart"10:19
pbourkeSamYaple: register.yml - http://paste.openstack.org/show/418992/10:23
pbourkeSamYaple: defaults.yml - http://paste.openstack.org/show/419000/10:23
pbourkeSamYaple: proxy-server.j2 - http://paste.openstack.org/show/419001/10:24
pbourkeSamYaple: the conf template is the piece I find the most confusing, the combinations in examples online seem endless and no matter what I put in there I get messages about it been deprecated10:24
pbourkeSamYaple: If I put "admin" in for all the user/pass items it seems to work, but that's not what we want10:25
openstackgerritwanghao proposed stackforge/kolla: Add image tag specification in dev-quickstart.md  https://review.openstack.org/21365210:26
SamYapletry swiftoperator rather than username swift10:26
pbourkeeven though I'm registering with swift?10:26
SamYaplethis isn't about what youre registering10:27
SamYapleits about other conf options you arent setting properly10:27
SamYapleif admin works, swiftoperator will10:27
SamYapleill show you why10:27
SamYaple***10:28
SamYapleno user sorry, role10:28
SamYapleI do question why oyu have an issue with admin though10:29
openstackgerritwanghao proposed stackforge/kolla: Add image tag specification in dev-quickstart.md  https://review.openstack.org/21365210:29
pbourkedon't we register the swift user for the services to validate tokens with keystone?10:30
*** dims has quit IRC10:30
pbourkehmm I seem to have found a working combination10:30
*** dims has joined #kolla10:31
pbourkehttp://paste.openstack.org/show/419003/10:31
pbourkewas sure I tried that before10:31
SamYaplemake sure to drop the v2.0 on that auth_uri line10:32
SamYapleoh yea10:32
SamYapleadmin isn't the tenant10:32
openstackgerritMerged stackforge/kolla: Update readme to match current state  https://review.openstack.org/21341210:32
SamYapleits service10:32
pbourkeyeah10:33
pbourkeok thanks, sometimes just going through what you have with someone seems to help :)10:33
SamYapleyea10:34
SamYapleyoull want to use the new plugin too10:35
SamYaplenot that deprecated old one10:35
pbourkeI really do10:35
pbourkejust need to find up to date docs10:35
*** dims_ has joined #kolla10:36
SamYaplehere ya go: http://paste.openstack.org/show/419018/10:36
SamYapleyou wont find docs to help you10:36
*** akwasnie has joined #kolla10:37
*** dims has quit IRC10:38
pbourkethanks!10:38
SamYapleits what i dooooo10:39
pbourkeby the time its documented it will be deprecated again :)10:39
SamYaplelurn2source10:40
SamYaplebut seriously I have given up on google and openstack10:40
SamYaplesource is way quicker10:40
pbourkegood advice10:42
SamYaplefyi this is what it looks like10:43
SamYaplehttps://github.com/openstack/cinder/blob/master/cinder/common/config.py10:43
SamYaplemost are common like that10:43
SamYaplealso list deprecated options10:43
*** ChanServ changes topic to "Kolla IRC meetings on Wednesday - Agenda @ https://wiki.openstack.org/wiki/Meetings/Kolla - IRC channel is *LOGGED* @ http://eavesdrop.openstack.org/irclogs/%23kolla/"10:46
-openstackstatus- NOTICE: Gerrit restart has resolved the issue and systems are back up and functioning10:46
*** dims_ has quit IRC10:50
*** dims has joined #kolla10:51
*** inc0 has joined #kolla10:57
*** dims has quit IRC10:59
*** daneyon has quit IRC10:59
*** daneyon has joined #kolla11:00
*** dims has joined #kolla11:00
*** dims has quit IRC11:06
*** dims has joined #kolla11:08
*** weiyu has quit IRC11:10
*** tfukushima has quit IRC11:10
*** dims_ has joined #kolla11:12
*** dims has quit IRC11:12
openstackgerritAlicja Kwasniewska proposed stackforge/kolla: Add Dockerfile template for swift  https://review.openstack.org/21366911:16
*** dims_ has quit IRC11:17
*** dims has joined #kolla11:22
*** dims has quit IRC11:27
*** dims has joined #kolla11:29
*** dims_ has joined #kolla11:34
*** dims has quit IRC11:35
*** dims_ has quit IRC11:39
*** dims has joined #kolla11:40
*** shardy_ has joined #kolla11:43
*** shardy has quit IRC11:45
*** shardy_ has quit IRC11:49
*** dims has quit IRC11:49
*** shardy has joined #kolla11:49
*** rhallisey has joined #kolla11:53
*** dims has joined #kolla11:53
*** dims_ has joined #kolla11:56
*** dims has quit IRC11:58
*** dims_ has quit IRC12:04
*** dims has joined #kolla12:04
*** dims_ has joined #kolla12:08
*** dims has quit IRC12:10
*** tfukushima has joined #kolla12:11
*** dims has joined #kolla12:12
*** dims_ has quit IRC12:14
*** tfukushima has quit IRC12:16
*** dims_ has joined #kolla12:18
*** dims has quit IRC12:18
*** dims has joined #kolla12:19
*** dwalsh has joined #kolla12:20
openstackgerritSam Yaple proposed stackforge/kolla: Remove Ansible abstraction for Nova  https://review.openstack.org/21144512:22
*** dims_ has quit IRC12:23
*** dims_ has joined #kolla12:26
*** dims has quit IRC12:26
*** shardy_ has joined #kolla12:28
*** shardy has quit IRC12:29
*** dims has joined #kolla12:30
*** dims_ has quit IRC12:31
*** shardy_ has quit IRC12:33
*** shardy has joined #kolla12:34
*** dims has quit IRC12:35
*** dims has joined #kolla12:35
*** dims_ has joined #kolla12:38
*** dims has quit IRC12:40
*** cfarquhar has quit IRC12:42
*** dims has joined #kolla12:45
*** dims_ has quit IRC12:45
*** dims has quit IRC12:55
*** dwalsh has quit IRC12:55
*** klint has quit IRC12:55
*** tfukushima has joined #kolla12:56
*** dims has joined #kolla12:58
*** sdake has joined #kolla13:00
sdakemorning13:00
akwasniehi:)13:04
sdakehey akwasnie!13:04
*** britthouser has joined #kolla13:05
*** dims has quit IRC13:06
*** dims has joined #kolla13:09
*** dims has quit IRC13:14
*** dims has joined #kolla13:14
*** sdake_ has joined #kolla13:15
openstackgerritAlicja Kwasniewska proposed stackforge/kolla: Add Dockerfile template for designate  https://review.openstack.org/21370313:18
*** sdake has quit IRC13:19
*** sdake_ has quit IRC13:19
*** dims has quit IRC13:21
pbourkemorning13:23
*** dims has joined #kolla13:27
*** absubram has quit IRC13:29
*** britthou_ has joined #kolla13:36
*** dims has quit IRC13:36
*** stvnoyes has joined #kolla13:36
*** stvnoyes has left #kolla13:37
*** britthouser has quit IRC13:39
*** banix has joined #kolla13:41
*** diogogmt has joined #kolla13:41
*** dims has joined #kolla13:42
*** dims has quit IRC13:48
*** sdake has joined #kolla13:49
*** sdake has quit IRC13:49
*** sdake has joined #kolla13:49
*** dims has joined #kolla13:54
*** inc0 has quit IRC13:54
*** diogogmt has quit IRC13:54
*** dims has quit IRC13:59
*** dims has joined #kolla14:04
*** dwalsh has joined #kolla14:05
*** sdake has quit IRC14:10
*** dims has quit IRC14:11
*** dims has joined #kolla14:16
*** sdake has joined #kolla14:17
openstackgerritSteven Dake proposed stackforge/kolla: Remove compose from documenation  https://review.openstack.org/21340114:21
sdakejpeeler/rhallisey can you guys review that plz14:21
sdakeor pbourke if your around14:21
sdakethe review queue needs some love in general14:21
pbourkesdake: looking. I skipped reviews this morning as gerrit was acting up14:21
*** dwalsh has quit IRC14:22
*** dims has quit IRC14:22
sdakeif we can get through the gate-from-source blueprint reviews that would help14:22
sdakethey are mostly 1 liners14:22
sdakegerrit seems to be workingwell now14:24
sdakethey probably restarted it14:24
sdakerhallisey what other kinds of cinder drivers can there be besdies lvm?14:29
sdakeonly ceph?14:29
rhalliseyI figure those would be the 2 we'd support14:30
rhalliseythere's more though14:30
rhalliseynot sure what else14:30
rhalliseyec214:30
*** bmace has joined #kolla14:38
*** britthou_ has quit IRC14:39
*** britthouser has joined #kolla14:39
openstackgerritMerged stackforge/kolla: Add Dockerfile template for swift  https://review.openstack.org/21366914:47
*** diogogmt has joined #kolla14:48
*** dims has joined #kolla14:48
*** vbel has joined #kolla14:57
vbelgood morning/evening14:58
bmacegood morning vbel :)14:58
*** shardy_ has joined #kolla14:59
*** achanda has joined #kolla14:59
openstackgerritMerged stackforge/kolla: Fix openvswitch template  https://review.openstack.org/21348815:00
*** shardy has quit IRC15:01
*** shardy_ has quit IRC15:04
*** achanda has quit IRC15:04
*** shardy has joined #kolla15:05
*** blahRus has joined #kolla15:10
*** jmccarthy has quit IRC15:15
*** jmccarthy has joined #kolla15:20
vbelanyone could get centos-source-nova-compute running recently?15:25
rhalliseybuilding from source is breaking for me atm15:26
vbellooks like nova-compute is missing libvirt-python package.15:27
vbelin binary distros it is requirement of openstack-nova-compute package15:27
*** stevem has quit IRC15:35
*** smeisner has joined #kolla15:35
*** diogogmt_ has joined #kolla15:35
*** diogogmt has quit IRC15:37
*** diogogmt_ is now known as diogogmt15:37
*** vbel has quit IRC15:51
*** pbourke has quit IRC16:10
*** pbourke has joined #kolla16:11
pbourkebmace: ping16:11
*** unicell1 has quit IRC16:15
*** akwasnie has quit IRC16:23
sdakehey guys just got back from pt whats hanging16:24
sdakevbel i neve rrun from source16:26
sdakevbel i am likely thinking it doesn't work well16:26
sdakerhallisey re cinder driver16:26
sdakei'd propose we remove the cinder specifc lvm thing16:27
sdakeand if enabled_ceph is set, then cinder would just use that16:27
sdakei don't want config options unless absolutely mandatory to run the system16:27
sdakecinder specific stroage driver option16:27
sdakeand if there are only two storage options lvm or ceph, ifcep his not set we know its lvm16:28
sdakei dont want to template anything in that isn't mandatory16:28
sdakeceph will be a mandatory templating16:28
*** daneyon has quit IRC16:30
rhalliseywe should have local storage support though16:30
*** daneyon has joined #kolla16:30
sdakeyou just said lvm is the only option?16:31
sdakeisn't that local storage or d oyou mean something else16:31
rhalliseyya what we have16:32
rhalliseyjust not iscsi16:32
sdakeok, so lets start over :)16:32
sdakethere are going to be only 2 storage modes we will ever implement, lvm and ceph?16:33
rhalliseyyea16:33
rhalliseyI think just those 2 is good16:33
sdakeif enabled_ceph==yes confiigure condier for ceph else configure for lvm16:33
rhalliseycoorect16:33
sdakethat allows us to remove the cinder_driver option16:33
sdakeif someone wants a custom cinder driver they are on thei own with regards to config16:34
sdakeok so would you agree to a course of action of me filing a bug to remove that config option cinder_driver or whatever it is16:34
sdake(and you or coolsvap fixing it)16:35
sdakei hsould have caught it in review but idn't think of it until yesterday16:35
*** tfukushima has quit IRC16:40
*** tfukushima has joined #kolla16:41
*** unicell has joined #kolla16:45
rhalliseysdake, are you referring to -> {% if cinder_volume_driver == "lvm" %}16:48
rhalliseyin the config16:49
sdakeYES16:49
sdakejust default to lvm16:50
*** unicell has quit IRC16:50
sdakeand when ceph is implemented make a conditional around ceph_enabled16:50
rhallisey{% elif cinder_volume_driver == "ceph" %}16:51
rhalliseyit's already there16:51
*** mfalatic has joined #kolla16:51
*** mfalatic_ has joined #kolla16:52
sdakei know I want to remove it16:52
sdakeI want to remove the config option cinder_volume_driver16:53
sdakethe les config options our software has the beetter it will be for operators16:53
sdakeoh sorry didn't dsee the ceph in teh conditional16:53
*** unicell has joined #kolla16:53
sdakecan you just make that ceph_enables == yes?16:54
rhalliseyso is it ok? if lvm else ceph16:54
sdakeceph_enabled == yes?16:54
sdakeif ceph_enabled == yes use ceph else use lvm16:54
sdakeis the logic i'd like16:54
rhalliseyya I can abstract out to ansible16:54
rhalliseyfine with that.  Easier to track16:55
sdakejust add a ceph_enabled=n to the config options list16:55
rhalliseyk16:55
*** mfalatic has quit IRC16:56
*** bmace has quit IRC17:01
sdakerhallisey if you want to build from source you have to pull my patchset17:03
rhalliseysdake, which one?17:03
sdakeor atleast cherrypick this patch https://review.openstack.org/#/c/213472/1817:03
rhalliseykk17:03
openstackgerritSteven Dake proposed stackforge/kolla: Make gnocchi-base build from source  https://review.openstack.org/21352217:06
openstackgerritSteven Dake proposed stackforge/kolla: Make heat build from source in the templates  https://review.openstack.org/21352317:06
openstackgerritSteven Dake proposed stackforge/kolla: Neutron base does not need python-neutron-client package  https://review.openstack.org/21353917:06
openstackgerritSteven Dake proposed stackforge/kolla: Just a test patch to test building with templates  https://review.openstack.org/21353817:06
openstackgerritSteven Dake proposed stackforge/kolla: Make cinder build from source  https://review.openstack.org/21352417:06
openstackgerritSteven Dake proposed stackforge/kolla: Make openvswitch build from source  https://review.openstack.org/21352517:06
openstackgerritSteven Dake proposed stackforge/kolla: Fix gate source building  https://review.openstack.org/21347217:06
openstackgerritXiaBing Yao proposed stackforge/kolla: update config-external.sh in galera,keystone,rabbitmq,swift  https://review.openstack.org/21378717:09
sdakepbourke around?17:18
rhalliseycool that fixed it17:19
sdakewell give it a +2 then ! :)17:19
sdakei've got like 20 patches outstanding17:19
sdakereally hard to keep a tidy workspace with that much work outstanding17:19
sdakethe rest in that chain are all 1 liners as ewll17:20
sdakebrb relogging17:20
*** sdake_ has joined #kolla17:21
*** athomas has quit IRC17:21
*** jasonsb has quit IRC17:23
*** jasonsb has joined #kolla17:24
*** sdake has quit IRC17:24
*** achanda has joined #kolla17:27
*** jasonsb has quit IRC17:28
*** sdake has joined #kolla17:31
*** sdake_ has quit IRC17:35
sdakerhallisey https://bugs.launchpad.net/kolla/+bug/148570417:39
openstackLaunchpad bug 1485704 in kolla "remove cinder_volume_driver config option from globals.yml" [Critical,Confirmed] - Assigned to Ryan Hallisey (rthall14)17:39
sdakeanytime before we release is fine17:39
rhalliseyroger that17:39
sdakeits marked critical because it must be fixed17:40
sdakeconfig file is in lockdown17:40
sdakesamyaple you around atm?17:40
openstackgerritSteven Dake proposed stackforge/kolla: Move enable_cinder to the correct location  https://review.openstack.org/21379817:45
rhalliseysdake, I suggest the same thing on the review  for cinder ^17:46
rhalliseysam did not agree17:46
sdakebecause?17:47
sdakei dont want defaults.yml changing for every new service17:47
sdakethat is rediculous17:47
sdakelets have another go at the discussion17:48
sdakeor whatever the main config file is17:48
rhalliseyhttps://review.openstack.org/#/c/208868/7/etc/kolla/globals.yml17:48
rhalliseywait no that's not it17:49
rhalliseyoh nvm ya it it is17:49
rhalliseythat's the suggestion and response17:49
sdakei think you saying it should be in defaults.yml was not in queestion17:49
sdakeits the enabled by default in sam's comment17:49
*** inc0 has joined #kolla17:50
sdakehey inc017:50
inc0afternoon17:50
rhalliseyiirc I think there was more discussion on irc..17:51
rhalliseywill have to bring it up again just to clarify since I agree with the patch17:51
sdakewell +2 it then :)17:52
* sdake wtbs lots of reviews from our core reviewers today17:52
sdakemandre if your around pls vote on coolsvap's core proposal17:53
sdakeor let me know your abstaining so i can pull the trigger :)17:53
*** dims_ has joined #kolla17:53
rhalliseysdake, he starts next week17:55
inc0jpeeler, around?17:56
sdakenice what is he going to be working on?17:56
*** dims has quit IRC17:57
inc0sdake, do you know status of ironic container?17:57
sdakenot sure if its started or not17:58
sdakeit says started in launchpad17:58
sdakeit is just a conductor and api so should be 30 minute job to make the container17:58
inc0yeah, would be cool to setup functional undercloud with ansible tho17:59
inc0might be good use case17:59
inc0for tripleo17:59
inc0instead of all this devtest bullshit17:59
inc0anyway, I just came to say hello18:00
inc0now I'm going to have evening glass of scotch and chill18:01
inc0cya all tomorrow18:01
*** inc0 has quit IRC18:01
* rhallisey struggles with devtest18:02
sdakerhallisey is https://bugs.launchpad.net/kolla/+bug/1460129 finished?18:02
openstackLaunchpad bug 1460129 in kolla "Cinder needs to be broken out into thin containers" [Medium,Confirmed] - Assigned to Ryan Hallisey (rthall14)18:02
rhalliseynot yet since tgt is still running along side cinder-volumes18:03
sdakebut as is, cinder definately needs docker 1.7?18:04
sdakeif looking at the deployment today18:04
sdakeif its enabled18:04
sdakeneutron are not in thin containers so that rgument is moot18:04
*** tfukushima has quit IRC18:07
rhalliseysdake, oh I thought they've been broken out18:08
rhalliseycinder will need 1.718:08
rhalliseyat least for local storage18:08
rhalliseynot sure about ceph yet18:08
*** jasonsb has joined #kolla18:11
openstackgerritMerged stackforge/kolla: Add Dockerfile template for designate  https://review.openstack.org/21370318:14
openstackgerritMerged stackforge/kolla: Fix heat docker templates  https://review.openstack.org/21354718:16
openstackgerritXiaBing Yao proposed stackforge/kolla: update config-external.sh in galera,keystone,rabbitmq,swift  https://review.openstack.org/21378718:18
mandresdake: i'm abstaining on coolsvap's core proposal, i feel i don't have the required background to vote18:20
mandrebeen away a lot lately18:20
mandrebtw, I'm back to work on 25th18:21
mandrei'm totally trusting other cores' judgment, I'm sure coolsvap will do excellent job as core18:25
sdakemandre our process requires an email ack18:26
sdakeis that not possible?18:26
sdakeif not, I can ack in your stead linking to irc chat18:26
mandrenah, I can do it18:27
mandregive me a few minutes18:27
sdakerhallisey cinder will need, or does need?18:30
sdakerhallisey difference being will need = in the future, does need = needs today18:31
sdakethis needs an ack pls https://review.openstack.org/#/c/213472/18:31
sdakemandre if you can review quickly ;-)18:32
openstackgerritSteven Dake proposed stackforge/kolla: Remove compose from documenation  https://review.openstack.org/21340118:34
mandresdake: sure thing, once I put the kids to bed18:35
sdakemandre if you could spend 20 mins on the queue that would be fantastic :)18:35
sdakeour reviews are really lagging with you mia :(18:35
sdakeonce the kids are asleep ;)18:35
sdakerhallisey this is an easy one https://review.openstack.org/#/c/213524/ ;-)18:37
rhalliseyya that must've been an accidental addition18:38
*** britthou_ has joined #kolla18:43
sdakerhallisey which must've been accidental?18:43
rhalliseythe line you deleted18:44
*** britthouser has quit IRC18:45
sdakeit wasn't accidental i deleted it18:49
sdakethe container won't build with it18:49
sdakei htink it was just never tested18:49
sdakei am fixing that now18:49
sdakesoon we will have 8 build gates18:49
sdakethis needs merging or our documentation wont work18:50
sdakehttps://review.openstack.org/#/c/213398/18:50
Slowersdake: http://git.openstack.org/cgit/openstack/puppet-neutron/tree/manifests/params.pp18:51
SlowerSamYaple: ^18:51
Slowerso this is the craziness we were talking about with multiple config files18:51
openstackgerritSteven Dake proposed stackforge/kolla: Templatize Murano container  https://review.openstack.org/21355818:52
rhalliseyya a more flexible config support would make dealing with that easier18:53
rhalliseys/config/externalconfig18:53
sdakeslower we are aware of the problem and have a solution involving a json file18:53
Slowersdake: ah interesting18:53
rhalliseysdake, there's a bp for that right?18:53
sdakethere is not18:53
sdakethere should probably be, but it would be low priority18:54
sdakeif you want to file one18:54
sdakeit requires big changes all ove the code base and its a minor improvement18:54
Slowersdake: so basically an env var with json defining the configs?18:54
sdakei'm satisified with adding to config-external for the short term18:54
sdakeconfig-external.sh would copy a json file18:54
Slowerah ok18:54
sdakeor set_configs would read the json and do the copying18:55
sdakeand we could remove config-external entirely18:55
sdakerhallisey can you make a blueprint pls and link to channel18:55
sdakei would highy prefer yaml but it introduces a dependency on the hot18:55
sdakePyYAML18:55
sdakeas a result, we are going with json, as sucky as it is18:56
SlowerI think json is fine18:57
sdakewe dont use env variables for passing data any longer18:57
sdakeexcept for special conditoins like bootstrapping18:57
Sloweryeah that's probably a good call18:57
sdakeit just makes two config systems, which is maddening to deal with :)18:57
sdakeI dont know if that work will land for liberty or not18:57
sdakeit definately wont land for liberty-318:58
sdakeour launchpad tracker is overflowing with work for liberty18:58
sdakeslower we need critical/high to land for liberty-3 https://launchpad.net/kolla/+milestone/liberty-318:59
rhalliseysdake, re cinder discussion above.  I accidentally added the line you correctly deleted it19:05
sdakeroger19:05
sdakewhat about th docker 1.7 thing19:05
rhalliseycinder will need 1.719:06
sdakeis it needed *NOW* to run cinder as is in teh source base19:06
sdakeyou say will - meaning future tense19:06
sdakedo you mean does?19:06
rhalliseyya right now19:06
sdakeok cinder does need 1.7 then ;)19:06
rhalliseyfor local storage you need /dev properly mounter19:06
rhalliseymounted19:06
rhalliseywhich is only supported 1.7+19:06
rhalliseysorry for lack of clarity19:06
rhalliseytoo many conversations at once19:07
sdakeya i jump around alot19:07
sdakethats how I think too ;)19:07
rhalliseyuncontrolled add today19:07
rhalliseysdake, so I'm not quite able to visualize this json file19:09
rhalliseywhat would be in it? List of configs?19:09
sdakerhallisey can you look at https://review.openstack.org/#/c/213401/19:10
sdakeso i dont know json format19:10
sdakebut i'll put it in yaml so you can get an idea19:11
sdakefiles:19:11
sdake  project_name:19:11
sdake let me start over19:12
sdakefiles:19:12
sdake    keystone:19:12
sdakelet me start over19:12
sdakefiles:19:12
sdake   - source: /var/lib/kolla/configs/keystone/keystone.conf19:13
sdake    - dest: /etc/keystone/keystone.conf19:13
sdake    - owner: keystone19:13
sdake    - perm: 06419:14
sdakethat needs to be the entry in the list19:14
sdakeand thee needs to be a bunch of entries19:14
sdakemy yaml foo isn't great either - i usually edit other people's yamls :)19:14
rhalliseysame19:14
sdakebasically 4 parameters over and over in a dictionary19:15
rhalliseyok so we just add based on whats needed19:15
rhalliseygotcha19:15
sdakeyou add to tripleo19:15
sdakewe add to ansible19:15
rhalliseyset_config will interpret19:15
sdakethatsthe idea19:15
rhalliseyheat will read for tripleo19:16
rhalliseyok will outline this19:16
openstackgerritMerged stackforge/kolla: Add Dockerfile template for keepalived  https://review.openstack.org/21347919:16
sdakeit has to be in json because of ansible19:17
sdakeand because of the dependency problem19:17
sdakethe dependency problem is on the genreation side19:17
sdakenot on the decoding side19:17
sdakei dont care if PyYaml is in a container image19:18
sdakebut i do care if its required on the host19:18
rhalliseyright19:18
rhalliseyno worries19:18
sdakethis would make hosts like coreos or aotmic not work since they area read only filesystems and proably dont include pymal19:18
rhalliseyya json is fine19:20
sdakethis needs a review to finish up the build from source gating bluleprint19:23
sdakehttps://review.openstack.org/#/c/213525/19:23
rhalliseyhttps://blueprints.launchpad.net/kolla/+spec/replace-config-external19:24
sdakeall I have to say is centos binary and source gating work with this patch stream with our automated tests : https://review.openstack.org/#/c/213539/19:24
rhalliseysdake, add further comments to that if needed19:25
openstackgerritJeff Peeler proposed stackforge/kolla: Change default install type to binary  https://review.openstack.org/21383119:35
*** tfukushima has joined #kolla19:43
openstackgerritMerged stackforge/kolla: Fix gate source building  https://review.openstack.org/21347219:44
openstackgerritMerged stackforge/kolla: Make gnocchi-base build from source  https://review.openstack.org/21352219:44
openstackgerritMerged stackforge/kolla: Make heat build from source in the templates  https://review.openstack.org/21352319:45
sdakei've made an rc1 milestone in launchpad19:46
sdakehttps://launchpad.net/kolla/+milestone/liberty-rc119:46
sdakeit includes all the things not yet started19:46
openstackgerritMerged stackforge/kolla: Make cinder build from source  https://review.openstack.org/21352419:46
sdakethe one problem is "mongodb"19:46
*** bmace has joined #kolla19:46
openstackgerritRyan Hallisey proposed stackforge/kolla: Change cinder_volume_driver config option to enable_ceph  https://review.openstack.org/21384019:46
sdakewhich is a dependency of ceilometer in liberty-319:47
sdakedeadline for rc1 is sept 25th19:47
rhalliseyI'm still testing ceph..19:47
rhalliseysince I have no idea what I'm doing taking some time19:47
sdakerhallisey i think you have too much work to do between now and 31st (deadling  for liberty-3) to do mongodb19:48
sdakedo you mind if i remove you from the assignee?19:48
rhalliseyfor ceph?19:48
*** dims_ has quit IRC19:48
sdakeno for mongodb19:48
sdakeyour #1 job is ceph :)19:48
rhalliseyjust push to rc119:48
sdakepushceilometer to rc1?19:48
rhalliseyunless someone else wants it19:48
rhalliseyno mongo19:48
rhalliseyisn't that what you did?19:48
sdakeceilometer depends on mongo19:48
*** dims has joined #kolla19:48
rhalliseyohh19:48
sdakei want to pull it back into liberty-3 and possibly get coolsvap to take it oon19:49
rhalliseysorry I missed your earlier comments :/ jumped in at the wrong time19:49
rhalliseyya just set it loose19:50
rhalliseyI grabbed cause no one wanted it19:50
sdakei think nobody still wants it ;-)19:52
rhalliseyjust leave it for me then I'll hopefully get to it19:54
rhalliseyI'm still stuck on the cinder db issue19:54
rhalliseyI have a running cinder elsewhere that I'm trying to attach to a ceph node19:55
rhalliseyI think it's working19:55
rhalliseyI don't know19:55
rhalliseyI'm just not quite satisfied yet19:55
rhalliseyI need to understand it more to make a doc at the very least19:56
openstackgerritRyan Hallisey proposed stackforge/kolla: Cinder bootstrap container is not initializing the DB  https://review.openstack.org/21384519:58
rhalliseyissues I've found so far ^19:58
*** bizarrochristy has joined #kolla19:58
*** mfalatic_ has quit IRC19:58
*** mfalatic has joined #kolla19:58
openstackgerritRyan Hallisey proposed stackforge/kolla: Change cinder_volume_driver config option to enable_ceph  https://review.openstack.org/21384020:02
sdakerhallisey i left two comments - one in partchset 1 and patchset 220:05
rhalliseywfm20:06
sdakerhallisey there is a comment for you in https://review.openstack.org/#/c/213623/20:07
* sdake works peopole like dogs!20:07
rhalliseydidn't even notice the line #'s20:08
sdakeyup thats why i expalined it :)20:08
rhalliseywell I did. But didn't notice that the last line wasn't given a new line #20:09
sdakejpeeler or mandre can we get an ack on https://review.openstack.org/#/c/213401/20:09
openstackgerritRyan Hallisey proposed stackforge/kolla: Change cinder_volume_driver config option to enable_ceph  https://review.openstack.org/21384020:10
sdakerhallisey you missed my comment in patch set #120:10
rhalliseyadding20:12
sdakenobrainer review: https://review.openstack.org/#/c/213620/2/docker_templates/memcached/Dockerfile.j220:12
openstackgerritRyan Hallisey proposed stackforge/kolla: Change cinder_volume_driver config option to enable_ceph  https://review.openstack.org/21384020:13
sdakeok folks, mandre rhallisey need revies on this one, its been through the ringer- i've fixed all the problems and tested the deploy several times with it in various different patch sets20:14
sdakehttps://review.openstack.org/#/c/212804/20:14
*** daneyon has quit IRC20:14
sdakealso rhallisey no brainer review here too https://review.openstack.org/#/c/213831/20:15
sdakeand then  you can get bck to ceph work :)20:15
sdakeTIA :)20:15
openstackgerritMerged stackforge/kolla: Add mongodb to docker_templates  https://review.openstack.org/21362320:18
openstackgerritMerged stackforge/kolla: Remove compose from documenation  https://review.openstack.org/21340120:19
sdakeyay the docs patch merged finally ;)20:19
openstackgerritMerged stackforge/kolla: Change default install type to binary  https://review.openstack.org/21383120:26
*** tfukushima has quit IRC20:27
sdakejpeeler when you have a moment to read https://review.openstack.org/#/c/213489/1/tools/build.py20:30
sdakewhat are your thoughts on somehow running build.py multiple times if it fails and collating the data into one data set?20:30
openstackgerritMerged stackforge/kolla: Add memcached to docker_templates  https://review.openstack.org/21362020:30
sdakein test_build.py rather then adding a flag20:31
jpeeleri'd think people would be using build.py directly locally too, so although it may not be as easy a retry flag would be nice20:32
sdakecan you add a comment in the review pleae20:32
* jpeeler copy pastes20:32
jpeelerit's sad that the mirrors are the reason we need this20:33
sdakeit is terribly sad agree20:33
*** jasonsb has quit IRC20:35
sdakejpeeler if you could ack https://review.openstack.org/#/c/213525/ its holding up build from source gating working properly20:35
*** jasonsb_ has joined #kolla20:35
sdakequeue isn't 3 pages long anymore ;)20:36
*** tfukushima has joined #kolla20:40
sdakerhallisey jpeeler mandre this needs reviewing love - would really like to finish the job on this change20:44
sdakehttps://review.openstack.org/#/c/212804/20:44
mandreon it :)20:45
*** shardy has quit IRC20:50
sdakelook ma, the containers are all passing the gate: https://review.openstack.org/#/c/213845/20:52
openstackgerritMerged stackforge/kolla: Make openvswitch build from source  https://review.openstack.org/21352520:53
openstackgerritMerged stackforge/kolla: Neutron base does not need python-neutron-client package  https://review.openstack.org/21353920:53
openstackgerritSteven Dake proposed stackforge/kolla: Use one data container for all data volumes  https://review.openstack.org/21280421:01
sdakemandre for your viewing pleasure ;)21:02
sdakemandre re the newline, I did try copying over the old file and just making the changes i needed, same result21:05
mandresdake, bad copy and paste, you need to remove the RUN instruction :)21:05
mandresdake: have you tried with git reset -p?21:06
mandrenot a big deal, though21:06
sdakewhich file has he bad copy and paste?21:07
mandrethe ones you modified21:08
mandre:P21:08
openstackgerritSteven Dake proposed stackforge/kolla: Use one data container for all data volumes  https://review.openstack.org/21280421:08
mandre1 sec, I'll add comments to the review21:09
*** banix has quit IRC21:12
*** britthou_ has quit IRC21:14
sdakeare you talking about the whitespace?21:14
sdakeI just tested it and it no worky21:14
sdakeso rebuilding imges, then if that works ill remove the workflow -121:14
sdakebut comments still appreciated21:15
* sdake wtb delorean mirrors21:16
mandrenah, I don't care about the whitespace, i care about valid dockerfile21:21
mandrehave you seen my comments about the extra RUN?21:21
mandreRUN is not a valid bash command, you'll get an error21:22
mandregit reset for the witespace, yeah... but not important, really21:25
*** achanda has quit IRC21:35
*** unicell1 has joined #kolla21:38
*** unicell has quit IRC21:39
*** gfidente has quit IRC21:41
*** bizarrochristy has quit IRC21:43
*** bmace has quit IRC21:46
sdakeoh i see mandre21:50
sdakeok i'll fix that up21:50
sdakemandre sorry was away farrying children from school21:50
*** banix has joined #kolla21:51
openstackgerritMerged stackforge/kolla: Fix volumes_from for the ansible playbooks  https://review.openstack.org/21349221:52
openstackgerritSteven Dake proposed stackforge/kolla: Use one data container for all data volumes  https://review.openstack.org/21280421:54
openstackgerritSwapnil Kulkarni (coolsvap) proposed stackforge/kolla: Update Fedora magnum images to use fedora repo  https://review.openstack.org/21297222:02
sdakemandre that patch just tested as working22:03
sdakewith a rebuild of images22:04
sdakemandre git reset -p prints out "No changes."22:05
*** sdake_ has joined #kolla22:10
*** sdake has quit IRC22:13
*** achanda has joined #kolla22:16
*** achanda has quit IRC22:16
*** achanda has joined #kolla22:16
*** dims_ has joined #kolla22:25
*** dims has quit IRC22:28
*** banix has quit IRC22:38
*** banix has joined #kolla22:40
sdake_yo coolsvap|away22:45
sdake_welcome to the core reviewer team :)22:45
sdake_mandre https://review.openstack.org/#/c/212804/ could use a final review ;)22:46
*** dims_ has quit IRC22:47
*** dims has joined #kolla22:47
sdake_coolsvap|away if you want to cut your chops on a hard review have a look at thatone ;)22:48
*** dims has quit IRC22:51
*** sdake_ is now known as sdake23:01
sdakesamyaple is shade adding domain support in some way?23:11
*** rhallisey has quit IRC23:11
*** sdake_ has joined #kolla23:23
*** sdake has quit IRC23:26
*** blahRus has quit IRC23:31
sdake_samyaple that method with my wireless worked!23:41
*** britthouser has joined #kolla23:57
*** britthou_ has joined #kolla23:58

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