Thursday, 2015-07-02

*** jtriley has quit IRC00:06
*** thetrav has quit IRC00:08
openstackgerritMartin André proposed stackforge/kolla: Add validation script for broken symlinks  https://review.openstack.org/19746500:10
openstackgerritMartin André proposed stackforge/kolla: Add link validation to pep8 gate  https://review.openstack.org/19746600:10
openstackgerritMartin André proposed stackforge/kolla: Add min env vars doc validation to pep8 gate  https://review.openstack.org/19747400:33
openstackgerritMartin André proposed stackforge/kolla: Add validation script for min env vars documentation  https://review.openstack.org/19747300:33
openstackgerritMartin André proposed stackforge/kolla: Sort variables and remove duplicates for min env doc  https://review.openstack.org/19758600:33
*** vinkman has left #kolla00:37
*** thetrav has joined #kolla00:46
*** jtriley has joined #kolla00:57
*** dims_ has quit IRC01:19
*** jasonsb has quit IRC01:31
*** alisonh has joined #kolla01:31
openstackgerritMartin André proposed stackforge/kolla: Add min env vars doc validation to pep8 gate  https://review.openstack.org/19747401:38
openstackgerritMartin André proposed stackforge/kolla: Sort variables and remove duplicates for min env doc  https://review.openstack.org/19758601:38
openstackgerritMartin André proposed stackforge/kolla: Make tools/min-env.sh portable  https://review.openstack.org/19780601:38
*** dims has joined #kolla01:39
*** erkules_ has joined #kolla01:51
*** erkules has quit IRC01:52
*** vinkman1 has joined #kolla02:04
*** vinkman1 has quit IRC02:04
*** jtriley has quit IRC02:24
*** jtriley has joined #kolla02:55
*** walters has quit IRC02:57
*** walters has joined #kolla03:02
*** dims has quit IRC03:22
*** jtriley has quit IRC03:24
*** coolsvap|away is now known as coolsvap03:45
*** alisonh has quit IRC03:51
*** jasonsb has joined #kolla04:00
*** bmace has joined #kolla04:00
*** alisonh has joined #kolla04:18
*** sdake has joined #kolla04:30
*** absubram has quit IRC04:35
*** alisonh has quit IRC04:44
*** alisonh has joined #kolla04:55
*** coolsvap is now known as coolsvap|away05:37
*** coolsvap|away is now known as coolsvap05:38
*** nihilifer has joined #kolla05:40
nihilifergood morning05:40
mandrehi nihilifer05:41
*** dasm|afk is now known as dasm05:52
*** coolsvap is now known as coolsvap|away05:54
vincent_vdkhi05:55
*** coolsvap|away is now known as coolsvap06:02
vincent_vdkso how's the work on cinder going?06:05
sdakerhallisey was afk most of the day so unclear06:07
sdakehey mandre nih*06:07
*** coolsvap is now known as coolsvap|away06:16
SamYapleyo06:23
SamYaplesdake: you around06:23
sdakewound me06:23
SamYaplei fell asleep06:24
sdakethat is good06:25
sdakesleep is necessary evil :)06:25
vincent_vdkso are people using kolla for production environments already?06:35
SamYaplevincent_vdk: i dont think06:35
sdakewho knows such things06:35
sdakeonly the oracle06:35
harmwSamYaple: can you take a look at https://review.openstack.org/#/c/197761/?06:35
SamYapleharmw: im reviewing it now06:37
harmwok, cool06:37
SamYapleim going to be a bit nitpicky so we can keep them the same across all projects, we can do back and forth on this and then use it as a template06:38
harmwI'm expecting you to do just that06:38
harmwthis'll be the template for other containers indeed06:38
vincent_vdkjust asking because we are testing several openstack "flavors"06:44
vincent_vdkto me, Kolla looks really interesting06:45
vincent_vdkso i'll be probably testing it in the near future06:45
harmwvincent_vdk: which other flavors are you trying out?06:45
vincent_vdkHelion which uses TripleO for deployment06:45
vincent_vdkI also briefly touched osad06:46
harmwah ok06:46
vincent_vdkanother colleague is testing Ubuntu currently06:46
harmwfor true container experience, stick with kolla06:46
vincent_vdki tend to follow you on that06:47
vincent_vdkalso the over/undercloud architecture is something i don't really like06:47
vincent_vdkthere is to much going on and to take into account when something fails06:48
*** shardy has quit IRC06:48
vincent_vdkbut that is imho :)06:48
harmwah yes, well, with kolla your laptop is your undercloudthingy :p running ansible and talking to the docker 'hypervisor'06:48
*** shardy has joined #kolla06:48
harmwtriplo is basically just 2 clouds, right?06:48
vincent_vdkyes06:48
thetravundercloud ftw06:48
harmwSamYaple: don't tell me I'm way off on Glance :p06:49
thetravI thought compose was currently the kolla undercloud?06:49
harmwyea thetrav06:49
* thetrav hasn't started using it yet06:49
harmwwe're working on ansible though06:49
thetravI've been putting together a setup using ansible for my lab06:49
vincent_vdkcompose just orchestrates your container deployment?06:49
harmwvincent_vdk: yes06:49
vincent_vdkthetrav: is that available somewhere06:50
SamYapleharmw: nope you are write on target functionally06:50
thetravunfortunately not vincent06:50
thetravit's also a bit of a hodge podge06:50
vincent_vdkjust to help testing or eventually help fixing things06:50
vincent_vdksure06:50
thetravI've been trying to re-create: http://behindtheracks.com/2014/04/redundant-load-balancers-haproxy-and-keepalived/ using kolla containers where possible06:50
harmwvincent_vdk: thetrav , if you're interested we have a /ansible in our repo that will get a lot of love these next couple of weeks06:51
vincent_vdkI once started a deplyment framework to deploy docker containers with Ansible06:51
vincent_vdkah, nice06:51
thetravso while I've got you talking harmw it looks a bit like compose is handling the ansible like part... is there anything to handle the fleet/swarm/kubernetes part?06:52
vincent_vdkthe fact you can deploy on thing like atomic is really interesting06:52
thetravlike... does compose actually deal with the multiple host thing?06:52
vincent_vdki don't think so06:53
sdakevincent_vdk over last 3 weeks our project has made a conscience and unanimous decision to make our work deploy with ansible as our #1 priority06:53
vincent_vdksdake: makes sense06:53
sdakeour #2 priority is ha, #3 priority is from source installs06:53
vincent_vdkcompose is nice for local dev imho06:54
thetravhave you hit the super annoying bug with ansible's docker module?06:54
sdakeit took alot of work to get people on board with that, but now we are all in on that model06:54
thetravneed to have expose as well as port06:54
sdakeso if you want to help, help implemenet ansible or test it :)06:54
thetravtook me about a week to figure out what was going on there06:54
vincent_vdkthetrav: which bug, it's been like 6 months ago i used Ansible+Docker06:54
sdakewe use --pid=host rathe rthen expose and port06:54
vincent_vdksdake: sure06:55
thetravnet: host in ansible06:55
thetravI found I had to do that for some containers but not all06:55
thetravit's one of those "avoid when possible" things06:55
SamYaplethetrav: in github.com/SamYaple/yaodu I do keepalived+haproxy and I will be copying that implmentation here06:55
SamYapleI even think I have a way we can rehup and load new configs without restarting haproxy06:56
vincent_vdkSamYaple: multi datacenter?06:56
vincent_vdkwe plan on deploying OS spanning 2 datacenters06:56
thetravI haven't managed to stick haproxy in a container06:56
SamYaplevincent_vdk: neither yaodu nor kolla supports regions or cells at the moment06:56
sdakevincent_vdk we plan to deploy openstack in one data center at present ;)06:57
vincent_vdkok06:57
sdakeif you want multi-datacenter not sure there is a container docker option for you06:57
SamYaplethetrav: sdake there will be06:57
SamYaplethetrav: sorry ignore06:57
thetravsdake: what's the multi-datacenter option?06:57
*** jmccarthy has joined #kolla06:57
sdakewill be and are are two different things06:57
sdakethere is none I  am aware of06:58
SamYaplethetrav: https://github.com/SamYaple/yaodu/tree/master/ansible/roles/docker_build/templates/ubuntu/haproxy that whole project shows my (working) implementation of haproxy in a container if you want some pointers06:58
thetravahh, I missed the "not sure" :P06:58
SamYaplesdake: I agree. but it will be easier to do than I initially though thanks to ansible variables06:59
thetravthanks SamYaple that will be super useful06:59
vincent_vdkthing is we have a 10G link between both DC's06:59
sdakethetrav you ought to join us in kolla - if you want to implement that you can do so :)06:59
vincent_vdkbut what would happen if it goes down06:59
vincent_vdknot sure keepalived handles this06:59
thetravsdake, I very much want to do that06:59
vincent_vdkbut we are not there yet06:59
SamYaplevincent_vdk: if the latency isn't to high, you can probably make that "one" cluster with different nova and availability zones07:00
thetravmight be able to swing it as an after hours activity07:00
thetravhaven't been able to convince my employers so far to contribute07:00
vincent_vdkSamYaple: indeed07:00
SamYaplevincent_vdk: you will have a much higher risk of splitbrain07:00
vincent_vdkSamYaple: that's what i'm afraid of07:00
SamYapleyou would need to use pacemaker or similiar to build a quorum on top of that to ensure no splitbrain07:01
SamYaplewe dont currently have that logic07:01
vincent_vdkuhu07:01
sdakethetrav roger, the model should be to use kolla as an upstream for your downstream products ;-)07:01
sdakeafter hours activity is good, sevearl of our best devs operate that way07:01
thetravsdake that's the angle of attack I'm taking07:02
thetravnow that you guys have your own channel, and have given a bit of a preso at a summit I can probably make a stronger case07:03
thetrav"we wanted this thing to have this feature, we didn't want to build and maintain it ourselves, our vendors didn't want to do it, so we did it, pushed it upstream, now everyone supports it for us and our vendors are all fine with it"07:04
thetravactually...07:05
thetravdid you guys do a preso?07:05
thetravor was that a planned thing07:05
thetravI never actually saw any video / write ups07:05
vincent_vdkthere is one from the summit07:05
vincent_vdkwhich grabbed my attention for this project07:06
sdakethere is a video on our wiki page07:06
sdakehttps://wiki.openstack.org/wiki/Kolla07:07
sdakealong with our mission07:07
vincent_vdkyou guys should submit something for FOSDEM07:07
sdake-ETOOMUCHTRAVEL07:09
sdakefeel free to submit something and speak on our behalf :)07:10
thetravfound: https://www.openstack.org/summit/vancouver-2015/summit-videos/presentation/thanks-docker-the-pro-and-039s-and-con-and-039s-of-containerizing-your-openstack-services07:10
SamYaplesdake: what were your plans for labs/talks? I wanted to do a Kolla+Ansible lab07:10
sdakeat summit?07:10
sdakenot doing a lab07:10
sdakefeel free ;)07:10
SamYaplecool07:10
sdakethat would actually be pretty sweet07:10
SamYapleill do an all-in-one grow to 3 node cluster07:11
sdakei think labs are 90 minute sessions07:11
SamYapleeasy peasy07:11
SamYapleill just have a local-private repo for download speed reasons07:11
harmwSamYaple: config-external just always does 1 thing, right? copy * files from A to B07:16
SamYapleit won't be * files07:16
SamYapleand the files wont all be going to the same place07:16
harmwah07:16
SamYaplebut it does copy from A-B yes07:16
harmwyou got examples of where they don't go to the same place?07:17
SamYapleneutron07:17
SamYaplewhen we do support ceph, ceph has conf files in /etc/ceph07:17
harmwok, with dnsmasq and neutron configs07:17
SamYaplethere are a few scenarios07:17
sdakesamyaple I dont know how the labs work - I've never done one07:17
harmwno no, what I mean is there a $source and a $destination and config-external should copy -all- files from $source to $destination07:18
SamYaplenope07:18
harmwthere can me multiple $destination vars here?07:18
harmweg. some files go in /etc/neutron, others to /etc/sysconfig?07:19
SamYapleyes07:19
SamYaplesysconfig is probably a bad example, but yea07:19
sdakediga you about07:19
harmwI know I know07:19
sdakeccolsvap|afk ^^07:19
harmwI'd like to have just one config-external.sh though, instead of a c/p in everycontainer07:20
*** coolsvap|away has quit IRC07:20
SamYapleharmw: no thats never goign to work07:20
SamYapleand we are movign away from symlink speghetti07:20
harmwwe could use a directory structure to point out where on / it should go?07:20
SamYapleyea thats really just over complicating it though07:20
harmwyou think?07:21
SamYaplea straight forward uniquie bash script to handle unqie names and unique priviliges and permissiosn07:21
harmwI just hate seeing the same code in multiple places07:21
harmwpowered by env vars07:21
SamYapleWhile I agree, this doesnt count as a DRY violation07:21
SamYapleno07:21
SamYapleENV vars dont play a role here07:21
harmw(set through DEPLOYMENTTOOL)07:21
SamYaplenope07:21
SamYaplethe container is the one place we have static control over things07:22
SamYaplewe control the content of the container07:22
SamYaplegive me a valid reason the deployer would need to adust the location of a file in a container07:22
harmwuhm, he shouldn't07:22
harmwnever ever07:22
SamYapleI wont go _that_ far, but that would be the rare case he would need to, so no ENV vars needed here07:23
*** coolsvap|away has joined #kolla07:23
harmwI was merely thinking of using env vars to give power the copy-external script, setting a username that way to chown or setting the path wich holds the configs and settings a target path07:24
harmwthat way the script is clean, and reusable07:24
harmwand ansible or whatever can set those vars07:24
harmwinstead of a pure c/p of config-external to all containers07:24
SamYapleYea I know what your getting at, but it seems like over complication to me07:25
SamYaplethe only thing that _would_ work is a for loop on the outside of config-external called by start.sh so the copy and permission update code is only used once07:25
harmwlets note this down under future improvements07:26
SamYaplebut that changes the logic from CONFIG_EXTERNAL_COPY_ONCE and CONFIG_EXTERNAL_COPY_ALWAYS07:26
harmwand go c/p for now, even though I hate it07:26
SamYapleharmw: agreed. it can be improved, lets just get it down for now07:26
SamYaplei know :/07:26
harmwit's probably easier to figure out a good solution once everything is in place anyway, when we're all on-par with your ansible skills :P07:27
SamYapleI dont think ansible will come into play here07:30
SamYaplebut once its all on paper we can see what we like and what we dont07:30
*** mickt has joined #kolla07:33
harmwSamYaple: rather then going through the BP I'll just ask you here (if I may), what should I do with KOLLA_BOOTSTRAP07:34
SamYapleYou did it right in the glance review07:34
SamYaplewell, except it shouldnt be on the glance-api at all07:34
harmwwhy not, thats what I want to understand :)07:35
SamYaplethere is nothing to bootstrap07:35
SamYapleon the DB has to be bootstrapped and the glance-registry would be incharge of that07:35
harmwbut where is glance-registry started?07:35
SamYaple(technically glance service container can do it)07:36
SamYapleansible would start it07:36
harmwand why wouldn't it start glance-api?07:36
SamYapleIt would. It just doesn't need to bootstrap glance-api07:36
harmwok.. but start.sh will also start glance-api07:37
SamYapleglance-api and glance-registry are two different services07:37
harmwI know07:37
SamYaplethen im confused07:37
harmwso am I07:37
harmw:)07:37
SamYapleglance-registry talks to and bootstraps the database07:37
harmwyep07:38
SamYaplethen glance-api and glance-registry are started as new containers07:38
harmwah!07:38
harmwmissed that07:38
SamYapleexternal tool can use procedures and do things serially07:39
SamYaplemuch better :)07:39
harmwyou init containers (bootstrap), have them restart and then they know they are not bootstrapping and can just start07:39
SamYapleah yes hold ill show07:40
SamYaplehttps://github.com/stackforge/kolla/blob/master/ansible/roles/keystone/tasks/bootstrap.yml07:40
SamYaplethose vars are plugged into https://github.com/stackforge/kolla/blob/master/ansible/roles/bootstrap.yml#L2307:41
SamYaplethen that whole container is cleaned up after the database is up to date https://github.com/stackforge/kolla/blob/master/ansible/roles/bootstrap.yml#L2807:41
SamYapleafter all of that _new_ containers are started normally for both services07:41
harmwwicked07:42
*** thetrav has quit IRC07:45
openstackgerritHarm Weites proposed stackforge/kolla: Add config-internal/config-external to Glance containers  https://review.openstack.org/19776107:48
*** loth has quit IRC07:49
sdakebed time07:53
sdakenight07:53
SamYaplenight07:53
sdakesamyaple use mailig list if ou have questions07:53
SamYaplefor what?07:53
*** loth has joined #kolla07:53
sdakefoks thre was a general qeury to use the mailing list07:53
sdakewe idd't get to "for whta"just usjg irc better07:54
harmwSamYaple: could you please take another look on that review? :)07:54
SamYaplelooking now07:54
harmwI wonder where that first bashate error comes from btw07:54
SamYapleharmw: i want the same comments inline from the keystone one07:55
SamYaplecomments inline are not cludge and in the case of set_configs it was a super important comment07:55
harmwthe same? I'm not allowed to expand on the topic? give a little more insights on the how/when/where/why?07:55
SamYapleyou can expand, but you didnt copy any over at all07:56
harmwwas there even a comment about set_configs?07:57
harmwlet me check on that07:57
harmwI've skipped # loading common functions07:57
harmwfor redundancy07:57
SamYapleif you aren't going to look at what i asked in my review why bother reviewing your change?07:57
SamYapledont skip the comments just add it. there is no need to remove it07:58
SamYaplehaving a comment about what the source does, it just loads common function, rather than having it be code that is executed off to the side is important07:59
SamYaplesorry. youre right the comment is not in the keystone start.sh08:00
SamYaplehttps://review.openstack.org/#/c/196428/6/docker/common/galera/start.sh08:00
harmwSamYaple: aha!08:01
harmwofcourse we need descriptive comments, I'm just saying we don't need the comment from L8 on that review you just pasted :)08:03
harmwIll happily add the one on L11 though08:04
SamYapleI disagree08:04
SamYaplewe do need it08:04
SamYaple`source` executes code08:04
SamYaplethat comment says it only loads functions08:04
harmwbecause there just so happen to be only functions in kolla-common.sh08:04
SamYapleits not "just so happens" it is by design08:05
SamYaplehence the comment08:05
harmwindeed, why we don't need that comment08:05
harmw*wouldn't08:05
harmwbut Ill add it back in, np08:05
SamYaplewrong. there is no way for the reader to know that is by design08:05
*** erkules_ is now known as erkules08:06
*** erkules has quit IRC08:06
*** erkules has joined #kolla08:06
SamYapleit could also say '# changing permissions and setting up users' if it was doing something, it is not doing something in this case. just loading functions08:06
SamYaplewith the comments in place you can follow the entire flow of the program without leaving this file08:06
SamYaplewithout the comments you have to read through each sourced file08:06
*** athomas has joined #kolla08:06
*** shardy has quit IRC08:09
harmwok, so, I've added the comments again08:09
openstackgerritHarm Weites proposed stackforge/kolla: Add config-internal/config-external to Glance containers  https://review.openstack.org/19776108:10
harmwwell, typed 'em in08:10
micktQuick Q regarding murano dashboard.  Should it be part of horizon docker image or separate?  I presume the latter.08:10
SamYaplemickt: is Murano a plugin to the horizon dash, or a standable web?08:15
micktplugin08:18
SamYapleit would need to be in the Horizon container then08:18
SamYapleis it triggered to be active by something, or is it always active mickt ?08:19
harmw[ ! -f $cfg ] && continue, that shouldn't error out because there is not set -oerrorbla08:19
SamYapleharmw: it will because we are sourcing that file, not exec08:19
harmwok08:20
SamYapledidn't we just agree to do that simple copy and come back around to this issue?08:20
harmwI thought this was simple08:20
SamYaple:/08:20
micktSY - mmmm, just started looking into it; I'll get back to you on that. I would presume the latter.08:21
SamYaplemickt: if it is always active then well need some wayto toggle it on/off08:21
micktsorry former08:21
SamYapleoh if its triggered active then it should be fine to include08:21
micktas I say, just started to look into it so I'll have a better idea later08:21
SamYaplesure thing!08:22
SamYaplemickt: did I talk to you about the Murano dashboard in #openstack a few weeks back?08:22
micktno08:23
SamYaplek08:23
SamYaplecarry on08:23
*** gfidente has joined #kolla08:29
harmwSamYaple: looks like your comments have evolved between keystone and galera as well, which is why I'm missing some bits08:33
SamYapleyea it was that damn rebasing 20 times08:33
openstackgerritHarm Weites proposed stackforge/kolla: Add config-internal/config-external to Glance containers  https://review.openstack.org/19776108:36
SamYapleharmw: "When bootstrapping is not required, start the service." that isn't correct since starting the service is not in the if block08:37
vincent_vdkkolla can be deployed on atomic hosts right?08:40
harmwshould just work, I know CoreOS works vincent_vdk08:40
vincent_vdkreally?08:41
vincent_vdkso how does it work with ovs? that lives on the host afaik08:41
SamYaplevincent_vdk: OVS doesn't work with the current deploymethods at this time08:41
SamYaplesupport still needs to be written for it08:42
SamYapleOVS + containers work _in_ _general_ because OVS is a kernel module, so the processes talk to the kernel module08:42
openstackgerritHarm Weites proposed stackforge/kolla: Add config-internal/config-external to Glance containers  https://review.openstack.org/19776108:44
vincent_vdkright, currently linux bridge is supported08:44
harmwyep08:44
SamYapleindeed08:44
harmwSamYaple: thanks for +2, finally! :) Ill use it as template tonight for other containers08:50
SamYaplesounds good. i need that glance patch to unblock ansible work08:52
SamYapleI need my other patches merged too08:52
SamYaplehttps://etherpad.openstack.org/p/ansibalising-containers08:52
*** shardy has joined #kolla08:59
*** shardy_ has joined #kolla09:03
SamYapleharmw: cluster that was running from ealier today, i nuked 1 of the three nodes completely09:05
SamYaplei then reran the playbooks and BAM its back09:05
harmwyea yea, you're godlike, we know09:06
harmw:p09:06
harmwnice man09:06
SamYapleYaodu never worked this good, but i took a bunch of lessons from Kolla and Yaodu when building this09:06
*** shardy_ has quit IRC09:13
*** openstackgerrit has quit IRC09:19
*** openstackgerrit has joined #kolla09:20
*** mstachow has joined #kolla09:20
*** dims has joined #kolla09:24
*** pbourke has quit IRC09:25
*** pbourke has joined #kolla09:25
*** bradjones has quit IRC09:33
*** bradjones has joined #kolla09:35
*** bradjones has joined #kolla09:35
*** mickt has quit IRC09:36
harmwSamYaple: I know what *.*:USAGE does, just wondering why there is a need for that user in the first place since there is no database :)09:39
harmwa link where I can read it myself would suffice, np with that09:40
openstackgerritPaul Bourke proposed stackforge/kolla: Update centos-source-base from symlink to a real layer  https://review.openstack.org/19789709:46
*** akwasnie has joined #kolla09:50
*** jmccarthy has quit IRC09:53
*** jmccarthy has joined #kolla09:54
openstackgerritMichal Rostecki proposed stackforge/kolla: Add --no-install-recommends option to Ubuntu image  https://review.openstack.org/19748109:54
*** gfidente has quit IRC09:57
*** gfidente has joined #kolla09:58
mstachowmorning all10:09
akwasniehi Michal :)10:10
SamYapleharmw: its for the future haproxy health check on the database. the user creation task is also being used to block until the database is active10:17
harmwok10:28
openstackgerritPaul Bourke proposed stackforge/kolla: Update centos-source-base from symlink to a real layer  https://review.openstack.org/19789710:46
openstackgerritPaul Bourke proposed stackforge/kolla: Add new script to fetch tarballs for source installation  https://review.openstack.org/19791910:54
openstackgerritPaul Bourke proposed stackforge/kolla: Update centos-source-base from symlink to a real layer  https://review.openstack.org/19789710:55
openstackgerritPaul Bourke proposed stackforge/kolla: Move declaration of TMPDIR to top of build-docker-image  https://review.openstack.org/19702710:55
openstackgerritPaul Bourke proposed stackforge/kolla: Keystone installation from source  https://review.openstack.org/19316610:55
openstackgerritPaul Bourke proposed stackforge/kolla: Keystone installation from source  https://review.openstack.org/19316610:56
openstackgerritPaul Bourke proposed stackforge/kolla: Keystone installation from source  https://review.openstack.org/19316610:56
openstackgerritPaul Bourke proposed stackforge/kolla: Keystone installation from source  https://review.openstack.org/19316610:57
*** rhallisey has joined #kolla11:27
*** ogelbukh1 has quit IRC11:35
*** ogelbukh1 has joined #kolla11:35
SamYapleso harmw are you happy with the wording for comments in glance start.sh? If so i will copy those for keystone since I have to update keystone11:48
harmwgo ahead11:49
SamYapleroger roger11:50
rhalliseymorning11:53
rhalliseysorry I missed the meeting yesterday11:53
SamYaplerhallisey: good morning!11:53
mstachowo/ rhallisey11:55
*** jtriley has joined #kolla11:58
*** jtriley has quit IRC12:02
SamYaplerhallisey: i poked at cinder. have to ideas im playing with to improve things. what are the biggest pain points do you see right now?12:05
rhalliseySamYaple, that is use to work :p12:05
SamYaple:)12:05
rhalliseyif you do a nova volume-attach12:05
rhalliseyyou will see in cinder-volumes log and iscsi error12:06
rhalliseyfailed to create or something12:06
SamYapleyup12:06
SamYaplei think i know how to fix that12:06
rhalliseythen in the volume-attach you will see another iscsi error : session not found12:06
SamYapleyea12:06
rhalliseywhen /dev was mounted this went away12:07
rhalliseywith docker 1.712:07
rhalliseyso I'm not sure what it is this time12:07
SamYapleright so i had to revert the thin neutron agents because docker reverted the patch in 1.7 that made them work, the same patch you were using12:07
SamYaplethats probably why it broke12:07
rhalliseythat's what I was thinking too12:07
SamYaplehttps://github.com/docker/docker/commit/ff770d33cdee5ab4dc0bdeb06dcfea49aa72ab8e12:08
SamYaplebetween 1.7.0 rc2 and rc3 this commit happened12:08
rhalliseyya I saw you email to dwalsh12:08
rhalliseyok12:08
SamYapleif you test on rc2 and it works, then thats almost certainly whats happening12:08
SamYaplewe will probably have to wait for 1.8 to get this fixed12:09
SamYaplethats 2 months out12:09
rhalliseyI', using rc1 though12:09
rhalliseythat's why I'm sure if it's docker12:10
SamYapleits broken with rc1?12:10
rhalliseyit worked for a time12:10
rhalliseyso I think it's something in cinder, but I'm not sure what since I solved this by moving to 1.712:10
SamYapleso we are not setting the keystone bind host or port anymore12:45
SamYapleit is binding to the default port on all ip addresses12:45
*** jtriley has joined #kolla12:50
openstackgerritHarm Weites proposed stackforge/kolla: Add config-internal/config-external to Nova-controller containers  https://review.openstack.org/19795413:00
*** alisonh has left #kolla13:00
openstackgerritPaul Bourke proposed stackforge/kolla: Update centos-source-base from symlink to a real layer  https://review.openstack.org/19789713:04
*** shardy has quit IRC13:05
openstackgerritPaul Bourke proposed stackforge/kolla: Add new script to fetch tarballs for source installation  https://review.openstack.org/19791913:06
*** shardy has joined #kolla13:06
openstackgerritSam Yaple proposed stackforge/kolla: Updated keystone for config-external  https://review.openstack.org/19796013:11
*** prad has joined #kolla13:13
*** dasm is now known as dasm|afk13:15
SamYapleharmw: recheck the glance and nova bootstrap sections13:25
SamYaplei just noticed the login shell in /etc/password wont allow executing commands so we need to override it13:25
*** jtriley has quit IRC13:25
openstackgerritPaul Bourke proposed stackforge/kolla: Update centos-source-base from symlink to a real layer  https://review.openstack.org/19789713:26
openstackgerritPaul Bourke proposed stackforge/kolla: Move declaration of TMPDIR to top of build-docker-image  https://review.openstack.org/19702713:26
*** thumpba has joined #kolla13:27
pbourkesorry for the barage of patches folks13:29
*** jtriley has joined #kolla13:29
pbourkerebase hell13:29
openstackgerritPaul Bourke proposed stackforge/kolla: Keystone installation from source  https://review.openstack.org/19316613:29
*** shardy has quit IRC13:38
*** shardy has joined #kolla13:44
*** nihilifer has quit IRC13:45
SamYaplepbourke: cloning master is what we should be doing14:05
SamYaplestable/kilo is not the branch you are commiting to14:05
SamYapleyou are commiting to master14:05
pbourkeim not committing to anything :)14:06
pbourkei just thought people would want to build from kilo by default14:06
pbourkemaybe im missing the point here14:06
SamYaplekolla master == stable/liberty14:06
pbourkeah14:06
SamYaplethere is no stable/liberyty at the moment14:06
pbourkedid you see my follow up comment though on that maybe we shouldnt do shallow clone14:07
SamYapleso shallow clones are what we need14:07
SamYaplethis is why no buioldconfs in teh repo14:07
SamYaplethats a user config file14:07
*** athomas has quit IRC14:08
pbourkehow will it work if someone wants a custom ref14:11
*** akwasnie has quit IRC14:11
*** vbel has quit IRC14:12
pbourkeSamYaple: ^14:13
SamYaplei responded in the review14:14
pbourkeah, thanks14:14
pbourkewhy worried about the tarball size though? we remove it once it's expanded14:15
*** athomas has joined #kolla14:15
*** shardy has quit IRC14:18
SamYapleyou tar the whole git folder which means you add ALL of that into the container affecting final size14:18
SamYaplei dont care about the builders system at all14:18
pbourkeah, I see, even if I rm it within the container it gets added via the build context right?14:19
*** dims has quit IRC14:19
SamYapleugh you switched back to COPY14:20
SamYaplewe can use ADD14:21
SamYapleyou just need to do i a bit different14:21
SamYaplealso yes the tarball and the xtracted content will have layers and add to the size14:21
pbourkewrt copy - the tarballs from tarballs.openstack.org have a top level dir of keystone_2015.0.1/14:22
pbourkeif I use ADD I'll have to make sure to tar up the git repo with the same name14:22
pbourkewe're bound to that naming for any other tarball method14:23
SamYapleln -s keystone* /keystone14:23
SamYaple^14:23
pbourkekk14:24
SamYaplewhile in the real world that *might* be "bad practice" but in containers in a controlled environment it is safe and allows us to save that layer14:26
openstackgerritPaul Bourke proposed stackforge/kolla: Add new script to fetch tarballs for source installation  https://review.openstack.org/19791914:37
openstackgerritSam Yaple proposed stackforge/kolla: Updated keystone for wsgi in config-external  https://review.openstack.org/19796014:37
*** dims has joined #kolla14:38
*** shardy has joined #kolla14:43
*** dims has quit IRC14:44
*** dims has joined #kolla14:49
*** mstachow_ has joined #kolla14:56
*** bradjones has quit IRC14:57
*** bradjones has joined #kolla14:58
*** bradjones has joined #kolla14:58
*** absubram has joined #kolla15:02
openstackgerritPaul Bourke proposed stackforge/kolla: Update centos-source-base from symlink to a real layer  https://review.openstack.org/19789715:05
*** dims has quit IRC15:06
*** Guest42148 has joined #kolla15:07
openstackgerritPaul Bourke proposed stackforge/kolla: Keystone installation from source  https://review.openstack.org/19316615:08
*** Guest42148 is now known as dims_15:09
*** jtriley has quit IRC15:16
*** shardy_ has joined #kolla15:18
*** shardy has quit IRC15:21
*** jasonsb has quit IRC15:29
*** jtriley has joined #kolla15:30
*** jasonsb has joined #kolla15:30
*** jasonsb has quit IRC15:34
*** jtriley_ has joined #kolla15:41
*** jtriley has quit IRC15:44
*** jtriley_ has quit IRC15:46
openstackgerritPaul Bourke proposed stackforge/kolla: Keystone installation from source  https://review.openstack.org/19316615:48
*** nihilifer has joined #kolla15:55
*** nihilifer has quit IRC16:05
openstackgerritPaul Bourke proposed stackforge/kolla: Keystone installation from source  https://review.openstack.org/19316616:07
openstackgerritPaul Bourke proposed stackforge/kolla: Keystone installation from source  https://review.openstack.org/19316616:10
SamYaplesdake: https://etherpad.openstack.org/p/ansibalising-containers16:12
SamYaplethe two in "needs code review" should be good to go16:13
SamYapleonce those merge you can test them out, follow the docs patch here https://review.openstack.org/#/c/197581/16:13
pbourkeSamYaple: while you're still awake - did you have anything in mind for how to default to a git url / tarball without specifying in buildconf?16:16
digasdake: Hi16:22
digaI was travelling since morning today, so couldn't see your ping16:22
digasdake: will be working over the weekend to finish up the things16:23
*** jasonsb has joined #kolla16:26
*** mstachow has quit IRC16:29
*** mstachow_ has quit IRC16:30
pbourkewould anyone be opposed to using python for some of the config scripts rather than bash?16:31
pbourkespecific case is where I need to specify ring info for swift and its going to be a bit ugly to parse a list of triplets in bash (ip,port,device_name,weight)16:32
*** shardy_ has quit IRC16:37
*** shardy has joined #kolla16:42
*** vinkman has joined #kolla17:05
sdakehey cats17:06
sdakepbourke python is fine imo17:06
sdakethe shell scripts started at 10 lines17:06
sdakenow they are hundreds17:06
sdakerhallisey cinder goes into reboot loops17:08
sdakerhallisey that should be an easy fix17:08
rhalliseyroger that17:09
sdakethere is a bug filed for that as well17:20
*** absubram has quit IRC17:36
openstackgerritHarm Weites proposed stackforge/kolla: Add config-internal/config-external to Glance containers  https://review.openstack.org/19776117:39
openstackgerritHarm Weites proposed stackforge/kolla: Add config-internal/config-external to Nova-controller containers  https://review.openstack.org/19795417:39
*** absubram has joined #kolla17:40
openstackgerritHarm Weites proposed stackforge/kolla: Add config-internal/config-external to Nova-compute containers  https://review.openstack.org/19807517:51
*** absubram has quit IRC17:55
*** thumpba has quit IRC17:58
* sdake hates expense reporting day in our household18:07
*** jasonsb has quit IRC18:11
*** absubram has joined #kolla18:15
sdakeharmw where are you with the config internal exteranl18:21
sdakethose two reviews gtg?18:21
harmwfirst 2 yea, nova-compute is new but probably good18:22
harmwworking on neutron atm18:22
harmwif I continue at this rate it'll all be done this sunday18:22
*** bmace has quit IRC18:29
sdakeharmw i have an update for your review18:31
sdakeharmw try to save some work for diga ;)18:31
sdakeharmw i'd like him to learn how the code works too18:31
sdakeand coolsvap|afk if he contribute sin this area as well18:31
*** bmace has joined #kolla18:32
harmwhehe, I'm not taking over containers from other assignees sdake :) not without consulting them first, at the very least18:32
sdakethanks18:38
sdakethey are actually volunteers, they weren't assigned iirc ;)18:39
harmwthey assigned themselves, judging by the etherpad ;)18:39
sdakei assinged you some tho18:39
harmwyep18:40
sdakebecause i wanted them done fast and I now you are further upt o speed on the code base18:40
sdakenow/now18:40
sdakeknow18:40
sdakeif your attending midcycle in person pleae book hotel rooms asap - silicon valley rates are going up quickly!18:43
openstackgerritHarm Weites proposed stackforge/kolla: Add config-internal/config-external to Neutron containers  https://review.openstack.org/19809118:54
*** athomas has quit IRC18:55
harmwcoolsvap|away: could you ping me when back? you opted to do Cinder so I'm keen on knowing if you've started on that or need help or anything :) re: https://etherpad.openstack.org/p/kolla-standard-start18:56
*** sdake_ has joined #kolla18:58
harmwsdake: I took the wrong spec? jeez18:58
harmwcan I just change that on my end in the commitmsg, or is more work needed?19:00
*** athomas has joined #kolla19:00
*** athomas has quit IRC19:00
*** sdake has quit IRC19:00
*** sdake has joined #kolla19:01
*** absubram has quit IRC19:01
openstackgerritHarm Weites proposed stackforge/kolla: Add config-internal/config-external to Nova-compute containers  https://review.openstack.org/19807519:03
openstackgerritHarm Weites proposed stackforge/kolla: Add config-internal/config-external to Glance containers  https://review.openstack.org/19776119:03
openstackgerritHarm Weites proposed stackforge/kolla: Add config-internal/config-external to Nova-controller containers  https://review.openstack.org/19795419:03
openstackgerritHarm Weites proposed stackforge/kolla: Add config-internal/config-external to Neutron containers  https://review.openstack.org/19809119:03
*** sdake_ has quit IRC19:04
*** shardy has quit IRC19:16
*** shardy has joined #kolla19:17
openstackgerritHarm Weites proposed stackforge/kolla: Add config-internal/config-external to Heat containers  https://review.openstack.org/19809819:18
harmwnow for swift19:19
*** jasonsb has joined #kolla20:24
*** jasonsb has quit IRC20:24
*** jasonsb has joined #kolla20:25
sdakei bought one of these last night http://www.theabsolutesound.com/articles/lehmannaudio-decade-phonostage/20:30
*** prad has quit IRC21:24
openstackgerritMerged stackforge/kolla: Remove duplicate PUBLIC_IP entry from tools/genenv  https://review.openstack.org/19750921:27
*** gfidente has quit IRC21:33
*** shardy has quit IRC21:34
openstackgerritHarm Weites proposed stackforge/kolla: Add config-internal/config-external to Swift containers  https://review.openstack.org/19814021:39
*** erkules_ has joined #kolla21:46
*** erkules has quit IRC21:47
sdakeharmw I assume your testing these changes with config internal config external in the config internal mode as you go?22:01
*** sdake has quit IRC22:22
*** sdake has joined #kolla22:22
sdakeharmw I assume your testing these config internal external changes as you go22:23
sdakeif fols dont' start filing techdebt bugs when they submit patches and I +2 them, I am going to start -1 patches until the techdebt bug is ifled or the bug is fixed ;)22:32
sdakejust a FYI :)22:32
*** sdake has quit IRC22:43
*** sdake has joined #kolla22:44
*** rhallisey has quit IRC22:45
*** sdake has quit IRC22:46
*** sdake has joined #kolla22:50
*** sdake has quit IRC22:51
*** sdake has joined #kolla22:52
*** sdake has quit IRC22:58
*** jasonsb has quit IRC23:07
*** jasonsb has joined #kolla23:08
*** jasonsb has quit IRC23:12
*** jasonsb has joined #kolla23:13
*** sdake has joined #kolla23:37
sdakerhallisey you around23:40
sdakecan you add me as an operator plz23:40
sdakeapparently not around ;(23:40
*** vinkman1 has joined #kolla23:50
*** vinkman has quit IRC23:51

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