Friday, 2014-10-03

*** rainya has quit IRC00:06
*** akuznetsov has joined #tripleo00:13
openstackgerritLars Kellogg-Stedman proposed a change to stackforge/kolla: fix keystone dockerfile  https://review.openstack.org/12582300:16
*** akuznetsov has quit IRC00:18
*** sdake_ has joined #tripleo00:23
openstackgerritLars Kellogg-Stedman proposed a change to stackforge/kolla: fix mariadb Dockerfile  https://review.openstack.org/12582700:25
*** spzala has quit IRC00:30
*** ccrouch has quit IRC00:33
*** eghobo has joined #tripleo00:38
*** eghobo has quit IRC00:56
openstackgerritjames labocki proposed a change to stackforge/kolla: Changes to README.md and starting heat-engine-api dockerfile work  https://review.openstack.org/12584301:03
*** panda has quit IRC01:08
*** panda has joined #tripleo01:08
openstackgerritSteven Dake proposed a change to stackforge/kolla: Keystone: Do a service and endpoint create  https://review.openstack.org/12584401:12
sdake_larsks do you know if the docker image is stored in memory for an active docker container, or does it use mmap for cow?01:20
larskssdake_: errr...neither.  It uses devicemapper devices mounted under /var/lib/docker.01:21
sdake_ya that is like mmap01:21
sdake_thanks01:21
sdake_so cow model01:21
sdake_mmap would have been tidier ;)01:22
larsksBut I suspect not suitable.01:22
sdake_ /var/lib/docker with a bunch of docker mmaped files = better then lvm01:22
sdake_boy I'm beat01:23
sdake_and I only wrote 24 lines of code01:23
sdake_boy tha t go compiler sure is fast01:23
sdake_does walter update his repos on a schedule?01:24
*** xuhaiwei_ has joined #tripleo01:27
larsksI'm not sure how often walter updates the repo...01:28
sdake_know his irc nick?01:29
larsksHe is "walters" on internal irc.  I don't know his freenode nick.01:29
openstackgerritSteven Dake proposed a change to stackforge/kolla: mariadb: Set container port in pod to 3306  https://review.openstack.org/12584801:36
sdake_time fo rpizza01:36
*** dsneddon has quit IRC01:36
*** yuanying_ has joined #tripleo01:36
*** yuanying has quit IRC01:37
*** dsneddon has joined #tripleo01:38
*** sdake_ has quit IRC01:41
*** weshay has quit IRC01:49
*** sdake_ has joined #tripleo01:50
openstackgerritSteven Dake proposed a change to stackforge/kolla: Remove extra commas in keystone JSON  https://review.openstack.org/12585302:12
*** akuznetsov has joined #tripleo02:15
*** cody-somerville has quit IRC02:15
*** andreaf has quit IRC02:16
*** andreaf has joined #tripleo02:16
*** akuznetsov has quit IRC02:20
openstackgerritSteven Dake proposed a change to stackforge/kolla: glance: remove JSON trailing commas that don't parse  https://review.openstack.org/12585702:20
openstackgerritSteven Dake proposed a change to stackforge/kolla: rabbitmq: remove trailing JSON commas that don't parse  https://review.openstack.org/12585802:20
openstackgerritSteven Dake proposed a change to stackforge/kolla: Heat: Remove JSON trailing commas that don't parse  https://review.openstack.org/12585902:20
openstackgerritSteven Dake proposed a change to stackforge/kolla: Swift: Remove trailing JSON commas that don't parse  https://review.openstack.org/12586002:20
*** cody-somerville has joined #tripleo02:30
*** cody-somerville has joined #tripleo02:30
*** jsidhu has joined #tripleo02:33
*** rhallisey has quit IRC02:39
*** killer_prince is now known as lazy_prince02:41
*** jsidhu has quit IRC02:47
*** noslzzp has quit IRC02:47
*** untriaged-bot has joined #tripleo03:00
untriaged-botUntriaged bugs so far:03:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/137541403:00
uvirtbotLaunchpad bug 1375414 in tripleo "mysql/rabbit overcloud issues when the bootstrap node is removed" [Undecided,New]03:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/137620003:00
uvirtbotLaunchpad bug 1376200 in tripleo "Stale connections when VIP is moved" [High,New]03:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/137542203:00
uvirtbotLaunchpad bug 1375422 in tripleo "Vlan not recreated on overcloud controller reboot" [Undecided,In progress]03:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/136616403:00
uvirtbotLaunchpad bug 1366164 in tripleo "Add domain roles for admin user" [Undecided,New]03:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/136774203:00
uvirtbotLaunchpad bug 1367742 in tripleo "keepalived brings the VIP up in multiple places on a controller reboot" [Undecided,New]03:00
*** untriaged-bot has quit IRC03:00
*** jsidhu has joined #tripleo03:03
*** akuznetsov has joined #tripleo03:15
*** akuznetsov has quit IRC03:20
*** Yanyanhu has joined #tripleo03:21
*** Yanyan has joined #tripleo03:22
*** Yanyanhu has quit IRC03:25
*** Yanyan has quit IRC03:28
*** yamahata has joined #tripleo03:28
*** spzala has joined #tripleo03:29
*** nosnos has quit IRC03:32
*** dsneddon has quit IRC03:32
*** nosnos has joined #tripleo03:33
*** dsneddon has joined #tripleo03:34
*** nosnos has quit IRC03:37
*** Yanyanhu has joined #tripleo03:37
*** spzala has quit IRC03:39
*** Yanyanhu has quit IRC03:43
*** dsneddon has quit IRC03:53
*** dsneddon has joined #tripleo03:53
*** cwolferh has quit IRC04:00
*** cody-somerville has quit IRC04:04
*** lazy_prince is now known as killer_prince04:13
*** akuznetsov has joined #tripleo04:16
*** cody-somerville has joined #tripleo04:19
*** cody-somerville has joined #tripleo04:19
*** jsidhu has quit IRC04:19
*** jsidhu has joined #tripleo04:21
*** akuznetsov has quit IRC04:21
*** yuanying has joined #tripleo04:22
*** yuanying_ has quit IRC04:22
*** pcrews has quit IRC04:27
*** jsidhu has quit IRC04:28
*** ramishra has joined #tripleo04:29
*** sdake_ has quit IRC04:29
*** nosnos has joined #tripleo04:35
xuhaiwei_StevenK: hi, are you around?04:38
StevenKxuhaiwei_: Hai04:39
*** takadayuiko has joined #tripleo04:39
*** ramishra has quit IRC04:40
xuhaiwei_I noticed you summited some patches for baremetal multi-flavor, will you do the whole job for https://review.openstack.org/#/c/106996?04:40
*** ramishra has joined #tripleo04:41
StevenKxuhaiwei_: In effect, yes. The code changes in tripleo-incubator to use the new setup-flavors script will allow you to specify the flavors used for things04:42
xuhaiwei_StevenK: will you make a patch for that? or it is already done?04:44
StevenKxuhaiwei_: I have a number of patches locally for it that I'm currently working on04:44
xuhaiwei_ok, I see, hope it will be landed soon04:45
StevenKxuhaiwei_: My hopes are somewhat less at the moment. I hope it will *work* soon :-)04:45
StevenKCopying things to a USB key while devtest is running may end up with a multi-minute sync :-/04:46
xuhaiwei_StevenK: I noticed this from the spec recentlly, but the spec is not merged, and Matthew Gilliard has abandoned his patches, so I thought this is stopped04:47
*** tzumainn has quit IRC04:47
*** ramishra has quit IRC04:54
*** jml has quit IRC04:58
*** kviiri has quit IRC05:00
*** ramishra has joined #tripleo05:00
*** jsidhu has joined #tripleo05:01
*** jml has joined #tripleo05:02
*** jml has joined #tripleo05:02
*** nosnos has quit IRC05:07
*** nosnos has joined #tripleo05:08
*** dshulyak_ has joined #tripleo05:09
*** akuznetsov has joined #tripleo05:17
*** akuznetsov has quit IRC05:21
*** morazi has quit IRC05:27
*** andreaf has quit IRC05:32
*** andreaf has joined #tripleo05:33
openstackgerritSteven Dake proposed a change to stackforge/kolla: rabbitmq: remove trailing commas in tabs so JSON parses  https://review.openstack.org/12585805:34
openstackgerritSteven Dake proposed a change to stackforge/kolla: Heat: Remove JSON trailing commas that don't parse  https://review.openstack.org/12585905:34
openstackgerritSteven Dake proposed a change to stackforge/kolla: glance: remove JSON trailing commas that don't parse  https://review.openstack.org/12585705:34
openstackgerritSteven Dake proposed a change to stackforge/kolla: Swift: Remove trailing JSON commas that don't parse  https://review.openstack.org/12586005:34
openstackgerritSteven Dake proposed a change to stackforge/kolla: Keystone: remove extra commas and tabs in JSON  https://review.openstack.org/12585305:34
openstackgerritSteven Dake proposed a change to stackforge/kolla: Keystone: Do a service and endpoint create  https://review.openstack.org/12584405:42
*** jprovazn has joined #tripleo05:54
*** morazi has joined #tripleo05:56
*** jml has quit IRC06:08
*** lsmola has joined #tripleo06:09
*** jml has joined #tripleo06:10
*** sdake_ has joined #tripleo06:11
*** jsidhu has quit IRC06:11
*** ramishra has quit IRC06:13
*** dshulyak_ has quit IRC06:15
*** pensu has joined #tripleo06:17
*** akuznets_ has joined #tripleo06:18
*** ccrouch has joined #tripleo06:35
*** jang1 has joined #tripleo06:39
openstackgerritSteven Dake proposed a change to stackforge/kolla: Initial libvirt and nova-compute structure  https://review.openstack.org/12589606:40
*** ccrouch has quit IRC06:40
*** ifarkas has joined #tripleo06:42
*** killer_prince has quit IRC06:42
*** killer_prince has joined #tripleo06:44
*** killer_prince is now known as lazy_prince06:44
*** dshulyak_ has joined #tripleo06:46
*** lazy_prince has quit IRC06:55
*** dshulyak_ has quit IRC06:55
*** lazy_prince has joined #tripleo06:55
*** lazy_prince is now known as killer_prince06:55
*** killer_prince has quit IRC06:59
*** lazy_prince has joined #tripleo06:59
*** regebro has joined #tripleo07:02
*** lazy_prince is now known as killer_prince07:09
jprovazn2chk07:25
*** rdopieralski has joined #tripleo07:31
*** akuznets_ has quit IRC07:32
*** jcoufal has joined #tripleo07:34
*** nosnos has quit IRC07:35
*** nosnos has joined #tripleo07:36
*** grauwulf has quit IRC07:46
*** Daviey has quit IRC07:47
*** grauwulf has joined #tripleo07:48
*** andreaf has quit IRC07:49
*** andreaf has joined #tripleo07:49
*** dshulyak_ has joined #tripleo07:54
*** jml has quit IRC07:55
*** jml has joined #tripleo07:57
*** mkerrin has quit IRC07:58
*** jistr has joined #tripleo08:00
*** athomas has joined #tripleo08:00
openstackgerritA change was merged to openstack/os-cloud-config: Make keystone_pki less keystone specific  https://review.openstack.org/10363708:04
*** yuanying has quit IRC08:05
*** yuanying has joined #tripleo08:05
*** mkerrin has joined #tripleo08:07
*** andreaf has quit IRC08:07
*** andreaf has joined #tripleo08:08
*** yuanying has quit IRC08:09
openstackgerritA change was merged to openstack/tripleo-image-elements: Install static element files  https://review.openstack.org/11036608:12
*** sdake_ has quit IRC08:14
*** sdake_ has joined #tripleo08:15
*** derekh has joined #tripleo08:23
*** ramishra has joined #tripleo08:27
*** lucasagomes has joined #tripleo08:28
openstackgerritSteven Dake proposed a change to stackforge/kolla: Make a k8s top level directory for k8s files  https://review.openstack.org/12590508:29
*** ramishra has quit IRC08:31
*** sdake_ has quit IRC08:33
*** yuanying has joined #tripleo08:36
openstackgerritA change was merged to openstack/tripleo-image-elements: Keystone LDAP domain configuration  https://review.openstack.org/10811708:38
*** jang1 has quit IRC08:39
*** yuanying has quit IRC08:42
*** ramishra has joined #tripleo08:51
*** jp_at_hp has joined #tripleo08:56
*** gfidente has joined #tripleo08:58
*** untriaged-bot has joined #tripleo09:00
untriaged-botUntriaged bugs so far:09:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/137541409:00
uvirtbotLaunchpad bug 1375414 in tripleo "mysql/rabbit overcloud issues when the bootstrap node is removed" [Undecided,New]09:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/137620009:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/137542209:00
uvirtbotLaunchpad bug 1376200 in tripleo "Stale connections when VIP is moved" [High,New]09:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/136616409:00
uvirtbotLaunchpad bug 1375422 in tripleo "Vlan not recreated on overcloud controller reboot" [Undecided,In progress]09:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/136774209:00
uvirtbotLaunchpad bug 1366164 in tripleo "Add domain roles for admin user" [Undecided,New]09:00
uvirtbotLaunchpad bug 1367742 in tripleo "keepalived brings the VIP up in multiple places on a controller reboot" [Undecided,New]09:00
*** untriaged-bot has quit IRC09:00
*** ramishra has quit IRC09:05
*** andreaf has quit IRC09:15
*** andreaf has joined #tripleo09:16
*** killer_prince is now known as lazy_prince09:19
*** yamahata__ has quit IRC09:23
*** hewbrocca has joined #tripleo09:25
openstackgerrittom-howley proposed a change to openstack/tripleo-image-elements: haproxy global maxconn is less than default maxconn  https://review.openstack.org/12567509:26
openstackgerritChmouel Boudjnah proposed a change to stackforge/kolla: Add validate-json tox target  https://review.openstack.org/12591309:29
*** yamahata has quit IRC09:30
*** yuanying has joined #tripleo09:37
*** rlandy has joined #tripleo09:38
*** xuhaiwei_ has quit IRC09:40
*** yuanying has quit IRC09:43
*** lazy_prince has quit IRC09:46
*** lazy_prince has joined #tripleo09:47
*** takadayuiko has quit IRC09:48
*** andreaf has quit IRC09:49
*** andreaf has joined #tripleo09:50
*** weshay has joined #tripleo09:55
openstackgerritChmouel Boudjnah proposed a change to stackforge/kolla: Add validate-json tox target  https://review.openstack.org/12591309:59
*** dtantsur|afk is now known as dtantsur10:12
*** jprovazn has quit IRC10:13
*** pelix has joined #tripleo10:14
*** nosnos has quit IRC10:22
*** nosnos has joined #tripleo10:22
*** andreaf has quit IRC10:24
*** andreaf has joined #tripleo10:26
*** yamahata has joined #tripleo10:26
*** nosnos has quit IRC10:27
*** jml has quit IRC10:34
*** jml has joined #tripleo10:38
*** yuanying has joined #tripleo10:39
*** andreaf has quit IRC10:42
*** andreaf has joined #tripleo10:43
*** yuanying has quit IRC10:44
*** ramishra has joined #tripleo11:06
*** lucasagomes is now known as lucas-hungry11:07
*** yamahata has quit IRC11:10
*** ramishra has quit IRC11:11
*** andreaf has quit IRC11:17
*** andreaf has joined #tripleo11:17
*** morazi has quit IRC11:18
*** jprovazn has joined #tripleo11:22
*** akuznetsov has joined #tripleo11:23
*** akuznetsov has quit IRC11:24
*** dsneddon has quit IRC11:25
*** ramishra has joined #tripleo11:27
*** dsneddon has joined #tripleo11:27
openstackgerritTomas Sedovic proposed a change to openstack/tuskar: Use resource registry for seed role matching  https://review.openstack.org/12568611:31
*** ramishra has quit IRC11:31
*** athomas has quit IRC11:32
*** dprince has joined #tripleo11:35
openstackgerritLadislav Smola proposed a change to openstack/tuskar-ui: Adding tests for generating parameters  https://review.openstack.org/12593911:38
*** yuanying has joined #tripleo11:40
*** yuanying has quit IRC11:45
*** dshulyak_ has quit IRC11:51
*** ramishra has joined #tripleo11:52
*** ccrouch has joined #tripleo11:54
openstackgerritTomas Sedovic proposed a change to openstack/tuskar: Use resource registry for seed role matching  https://review.openstack.org/12568612:00
*** ramishra has quit IRC12:05
*** dsneddon has quit IRC12:07
*** dsneddon has joined #tripleo12:07
*** morazi has joined #tripleo12:09
*** tzumainn has joined #tripleo12:13
*** rhallisey has joined #tripleo12:16
*** dshulyak_ has joined #tripleo12:17
*** athomas has joined #tripleo12:21
*** lucas-hungry is now known as lucasagomes12:27
*** dshulyak_ has quit IRC12:31
*** jdob has joined #tripleo12:31
shadowerjdob: thanks for your comments. I've added the tests in the morning12:33
openstackgerritEmilien Macchi proposed a change to openstack/tripleo-puppet-elements: do not merge - qa check  https://review.openstack.org/12595712:39
*** andreaf has quit IRC12:39
*** andreaf has joined #tripleo12:39
*** yuanying has joined #tripleo12:41
openstackgerritDougal Matthews proposed a change to openstack/tuskar: Add MySQL and PostgreSQL test cases  https://review.openstack.org/12595812:42
*** dshulyak_ has joined #tripleo12:43
openstackgerritA change was merged to openstack/diskimage-builder: Install openstack-selinux on RHEL  https://review.openstack.org/12386512:44
*** yuanying has quit IRC12:46
*** athomas has quit IRC12:48
d0ugaljdob: Since you like mixins, you might dig this. https://review.openstack.org/#/c/125958/12:50
jdobshadower: cool12:51
jdobshadower: it looks good though, I'm glad it wasnt too complex to pull off12:51
shadowerjdob: yea me too12:51
shadowerjdob: do we have a way of passing an error message from tuskar to the cli? The exception I raise just result in a 500 and an uninteligable traceback client-side12:54
rbradyderekh: ping12:54
shadowerjdob: we do write stuff to the log, but still12:54
jdobI think if you raise one of the exceptions in our exceptions module, the framework will use its error message12:54
shadowerit didn't for the UnknownName ones12:55
*** jprovazn has quit IRC12:55
sdakemorning12:55
shadowerhey sdake12:55
openstackgerritLadislav Smola proposed a change to openstack/tuskar-ui: Adding tests for generating parameters  https://review.openstack.org/12593912:59
shadowerjdob: yeah just checked the client -- there's no HTTP exception handling in there at all13:01
*** noslzzp has joined #tripleo13:03
*** rushiagr_away is now known as rushiagr13:06
*** radez_g0n3 is now known as radez13:25
*** jcoufal has quit IRC13:25
*** CaptTofu has joined #tripleo13:26
radezsdake: happy to test the new container, gotta take a quick look at staypuft this morning and I'll give it a whirl13:26
*** rushiagr is now known as rushiagr_away13:26
sdakehey shadower13:27
sdakeradez cool actually I fixed my kube setup13:27
sdakeradez I upgraded apiserver to master and things work grand now13:27
radezoh nice13:27
*** dsneddon has quit IRC13:27
sdakeand go compiles in half a second 90k lines of code13:27
*** dsneddon has joined #tripleo13:28
*** bvivek has joined #tripleo13:29
derekhrbrady: ack13:37
rbradyderekh: was curious about the status of the check-tripleo-ironic-overcloud-precise-ha job.13:38
rbradyderekh: is there a specific reason it's non-voting for now?13:38
openstackgerritSteven Dake proposed a change to stackforge/kolla: Keystone: Do a service and endpoint create  https://review.openstack.org/12584413:38
*** rlandy has quit IRC13:38
derekhrbrady: It was non voting because it didn't pass enough, although now that the VM's have more RAM it seems to be passing more, not as high as the other jobs,13:39
derekhrbrady: befor putting it voting I'd love to know how many of the failures are false http://goodsquishy.com/downloads/tripleo-jobs.html13:40
openstackgerritGiulio Fidente proposed a change to openstack/tripleo-image-elements: Improve nfs-server with arbitrary shares number/name/options  https://review.openstack.org/12596813:40
*** bnemec is now known as beekneemech13:41
*** yuanying has joined #tripleo13:43
openstackgerritGiulio Fidente proposed a change to openstack/tripleo-image-elements: Improve nfs-server with arbitrary shares number/name/options  https://review.openstack.org/12596813:46
sdakelarsks that service thing doesn't appear to work13:47
sdakeI thought it should as well13:47
larsksWhich service thing?13:47
*** yuanying has quit IRC13:47
sdakeusing KEYSTONE_MASTER in start.sh13:47
*** dsneddon has quit IRC13:48
larsksOkay. Apparently the changes that will make that authoritatively not work are merging today anyway, so we need another plan.13:48
larsksAnd I have one! :)13:48
sdakewhy are they doing t hat btw13:48
sdakesecurity nonsense?13:48
sdakesee discussion in review13:49
sdakeI think we need pod to pod communication13:49
larsksPossibly? I'm not sure, exactly, but the problem with what I was suggesting is that it suddenly makes things depend on a specific host being up...so if you lost a minion, your services would all be dead.13:49
*** dsneddon has joined #tripleo13:49
larsksTHey are probably trying to avoid that situation.13:49
larsksSo here is my proposal:13:49
sdakeno the proxy will route all ports to the same ip13:49
sdakewherever the pod is13:49
sdakeall ports on all nodes end up t othe wsame proxy ip addr13:49
larsksIf you register the proxy address that the keystone container sees, it will be point *at the local minion*.13:50
larsksSo if you put that in the service catalog, that would be used by pods on other minions.13:50
larsksIf that first minions dies, the proxy they were pointing at would no longer be available.13:50
larsksThat is the problem.13:50
sdakethat isn't how it works tho13:50
sdakeeg13:50
larsksIt totally is.13:50
sdake10.0.0.3 330613:50
sdake10.0.0.4 330613:50
sdakemariadb running on 10.0.0.4 330613:50
larsksRight, but we are talking about the keystone service catalog.13:50
sdaketelnet 10.0.0.3 330613:50
*** rhallisey has quit IRC13:51
larsksYou're missing the point.13:51
sdakegoes to 10.0.0.4 330613:51
sdakeI guess :) fill me in :)13:51
larsksYOu can only register a single address in the service catalog.  If your keystone container is running on minion 10.0.0.4, it will see 10.0.0.4 as the proxy address.13:51
sdakelisting an ip doesn't matter as long as it is a minion13:51
larsksIf you register this in the keystone catalog, then pods on other minions will contact keystone using 10.0.0.413:51
sdakeand if you telnet 10.0.0.3 5000 you will get keystone13:51
larsksIf that minions dies, there is no longer anything listening at 10.0.0.413:51
sdake10.0.0.3 is listening - once it is restarted it will be repaired13:52
larsksYou want pods to see the *local* proxy address, rather than the proxy address that the keystone pod saw.13:52
larsksBut that address is not in your service catalog.13:52
sdakewhat is the local proxy address, you mean 10.251.y.z?13:52
larsksNo.13:53
larsksTHe local proxy address is the host address of the minion.13:53
d0ugalhttp://civs.cs.cornell.edu/cgi-bin/results.pl?id=E_d8d7335f4de67d9413:53
sdakeok I get its not ideal if the proxy ip doesn't match, but all hosts that use a service port go to the correct pod running the service13:53
sdakethe proxy takes care of that13:54
sdakeyou only have to access any ip in the kube minion space13:54
larsksIf you register your publicurl in keystone as http://10.0.0.4:8774/v2/..., then if 10.0.0.4 goes down your are out luck.13:54
sdakeyou mean the minion goes down?13:54
larsksYes.13:54
sdakebit nit picky :)13:55
radezI think we need to depent on load balancers for this13:55
larsksWell, I have an idea.13:55
larskssdake: I don't think it's nit picky.13:55
radezI would assume that a production env would have a HW lB or something13:55
sdakejust joking larsks13:55
larsksI think it's a fundamental problem for which there is not yet a good kubernetes solution.13:55
larsksReally, we would want openstacks services to discover endpoints through kubernetes rather than keystone :)13:56
larsksBut that's not my idea...13:56
larsksIf we run an haproxy instance in each pod that sets up proxies for all the available services discovered via environment variables, then we can always use 127.0.0.1 as the service address, at least internally.13:56
larsksThe haproxy instance will discover services via kubernetes env vars, so it will always point at the local kube-proxy.13:57
radezlarsks: why would we not depend on a LB here?13:57
larsksWell, we're depending on kube-proxy.13:57
radezbut the kube proxys are a single point of failure13:57
radezif you round robin them then you get best of both right?13:58
larsksIf you can provide a good load balancer model that works with kubernetes, I am all ears! :)13:58
larsksThis seemed reasonably simple to implement right now.13:58
radezI'd say for dev just throw a lbaas ip on the minions13:58
sdakefor dev we can ignore the problem short term imo13:59
sdakebut it does need addressing13:59
sdake(the minion dieing problem)13:59
sdakethis is what launchpad tracker is for - keeping track of issues13:59
sdakeeach patch doesn't have to be perfect going in at such an early stage13:59
larskssdake: well, we also need to fix the address to use for endpoints, and you were saying earlier that using the service env vars was not working.13:59
sdakeit just has to work for some definition of work13:59
larsksSo, we need something that works.14:00
sdakelarsks it may appear so14:00
sdakeagree larsks14:00
larsksUsing a local haproxy should work now, and should continue to work after the kube folks change the way proxies are implemenented.14:00
sdake${KEYSTONEMASTER_PORT_5000_TCP_ADDR}14:00
larsksBut let me take a look at what you were trying today.14:00
sdakeshouldn't tyhat work?14:00
larsksI think that should work now, yeah.14:00
larsksLet me take a look.14:00
sdakeI'm all for fixing it right down the road14:01
sdake;)14:01
sdakewe have sleeps in the code base = UGH14:01
radezsdake: the ENV vars arn't working?14:02
sdakeradez I don't have an env in the json14:02
sdakemaybe that is the problem14:02
larsksThose aren't things you set in the JSON.14:02
*** bvivek has quit IRC14:03
larsksThey should be set automaticaly by k8s if the service is running.14:03
radezmy thought would be to throw a LB in front of the proxies and then set an ENV var for the pods with that IP to configure the endpoints14:03
larsksYou do need to start the services before the pods, though.14:03
sdakerightr14:03
chmouelis there any reason why we use json for kolla instead of yaml?14:03
*** bvivek has joined #tripleo14:03
sdakei have been told  the services are always to be started before pods14:03
larsksbecause all the examples are in json?14:03
larskssdake: yes, that's good practice.14:03
sdakechmouel I mean to convert to yaml just have not had a chance14:03
radezdoes k8s support yaml?14:03
sdakeradez yup14:04
larsksradez: possibly it does recently.  When I looked at it during the hackathon I think it did not.14:04
sdakeKEYSTONEMASTER-5000_PORT I think it should be14:04
sdakelet me try that14:04
chmouelin the DESIGN.md doc they are mentionnig just yaml not json http://is.gd/WnF4xD (but yeah all examples are in json)14:05
larskssdake: KEYSTONEMASTER_5000_PORT_5000_TCP_ADDR14:05
radezsdake: so why would we not just pass in an external IP via ENV for the pods to configure the endpoints with?14:05
larsksWhat external ip? How do you get that?  Which minion is hosting that ip? How does it fail over?14:05
sdakeradez an external ip with a lb makes sense for that yes14:05
sdakethat is built in14:05
sdakeI am not sure how it works tho14:06
larsksNote that k8s doesn't know about "external ips" right now.  There is an issue open on that:14:06
*** rhallisey has joined #tripleo14:06
radezlarsks: it would be external to the kube something that has to be provided14:06
larskshttps://github.com/GoogleCloudPlatform/kubernetes/issues/116114:06
larsksSo, in addition to setting up kubernetes, someone would also need to setup an ha load balancer?14:06
larsksHow would our services register with the lb?14:06
larsksTHat is, how would the lb locate a keystone service launched by k8s14:07
larsks?14:07
radezthe lb would balance the proxies and the proxies would handle that14:07
radezlb is dumb in a sense...14:07
sdakeI am pretty sure that is built in14:07
larskssdake: not the external stuff right now, no. Hence that github issue...14:07
sdakebut maybe not at the level necessary14:07
radezuntil an external IP becomes a first class citizen like that bug is talking about14:07
sdakeradez right14:08
larsksradez: okay.  So how would the lb know which services to load balance? E.g., what if someone is only running a subset of services...is the lb managed manually?14:09
*** dsneddon has quit IRC14:09
*** dshulyak_ has quit IRC14:09
openstackgerritjames labocki proposed a change to stackforge/kolla: starting heat-engine-api dockerfile work  https://review.openstack.org/12584314:09
larsksI wonder if we can run the lb *in* kubernetes somehow.14:09
sdakean easy solution would be for the lb to lb all minions on external addresses14:09
radezlarsks: lb passes everything through to the proxies14:09
radezproxies operate on ports to forward traffic14:09
openstackgerritDougal Matthews proposed a change to openstack/tuskar: Add MySQL and PostgreSQL test cases  https://review.openstack.org/12595814:09
sdakeand just pass the external address14:10
radezso the lb is transparent, just making sure that we alway contact a running proxy14:10
larsksradez: right, but *on which ports*?14:10
radezthe lb? all!14:10
larsksOr do we configure to listen to all potential openstack service ports all the time?14:10
larsksOkay.14:10
sdakeya all the ports14:10
sdakethe only probelm is that is harder to set up14:10
radezlb == all the ports and let the proxies respond to porst that arn't listening14:10
larsksSo instead of a "connection failed" you get "502 gateway failure'.14:10
*** dsneddon has joined #tripleo14:10
radezlarsks: I don't think you get that in a HW LB14:11
openstackgerritAlexis Lee proposed a change to openstack/tripleo-image-elements: Revert "Install static element files"  https://review.openstack.org/12597714:11
sdakeso the k8s cats say why they want to change the proxy model - its a flat port space14:11
sdakewhich I guess they don't like14:11
sdakeI find it somewhat appealing as is14:11
larsksradez: ...a *hardware* load balancer? What about people runing in PoC environment with no dedicated lb?14:12
larskssdake: If kubernetes is aiming for multi-tenancy I can see why they want the change.14:12
radezthey get a gateway error...14:12
larsksHey, that's *my* keystone port.14:12
radezbut wouldn't a gateway error be acceptable in a PoC?14:12
larsksradez: I dunno.  If we can integrate this into things so that it's all nice and automatic and launched via kubernetes I'm all for it.14:13
openstackgerritA change was merged to openstack/tripleo-image-elements: Revert "Install static element files"  https://review.openstack.org/12597714:13
larsksIf we're introducing an unmanaged external dependency, I'm a little suspicious.14:13
sdakeagree on external dep14:13
sdakemagic better manua lworse14:13
radezagreed it's a bigger problem to solve but i don't think it's something that has no intermediate option14:14
*** sdake_ has joined #tripleo14:16
*** sdake_ has quit IRC14:16
*** sdake_ has joined #tripleo14:16
sdake_I hae an idea that would work14:16
sdake_we could register all the minion ips as endpoints14:16
*** noslzzp has quit IRC14:17
sdake_and rely on k8s to load balance the minions14:17
sdake_failures would be retried by the end user14:17
sdake_in which case, they would get a new servie ip from the service catalog14:17
sdake_might need some magic in keystone to make the service catalog load balanced14:17
sdake_dradez larsks thoughts on that?14:18
radezI think that adds a job to keystone that it shouldn't do14:19
larsksThe 'need some magic in keystone' part worries me.  Registering the minion addresses as publicurl endpoints might make sense.14:19
larsksGetting those addresses, though, isn't an automatic process right now, which is annoying.14:19
larsksBut manageable.14:19
radezlarsks: are you in westford for that pow wow next week?14:20
larsksradez: Yup.14:20
sdake_which one14:20
radezsdake_: openstack best practices documentaion14:21
sdake_sounds painful :)14:21
sdake_"start up the devstack"14:21
sdake_"hope neutron doens't implode"14:21
sdake_rather "Hope the neutron doesn't implode"14:21
sdake_if keystone has the same service registered twice in the catalog what does it do?14:22
sdake_surely it must be able to handle multiple endpoints for the same service14:22
radezI don't know that I've ever seen it done14:23
morganfainbergsdake_, there is nothing that stops you from having multiple endpoints for the same service14:23
*** dshulyak_ has joined #tripleo14:23
morganfainbergafaik14:23
sdake_morgan are they round robin accessed?14:23
morganfainbergsdake_, i belive they all end up in the catalog14:24
radezmorganfainberg: yea what happends when you ask for an endpoing from a service?14:24
radezthat has more than one14:24
radezoh it just returns all of them?14:24
morganfainbergradez, belive so.14:24
radezah, so it would be up to the client to handle it properly14:24
morganfainbergnow services (not endpoints) have different unique constraints14:24
larskssdake_: I was just thinking about that this morning.  I don't know if the clients do the right thing or not.14:24
larsksOn my list of things to check :)14:25
morganfainbergradez, sdake_, easy to test this stuff :)14:25
sdake_whatwould be logical is for the client to pull the full list and try in order until it finds a connection that works14:25
sdake_or possibly random selection14:25
openstackgerritA change was merged to openstack/tripleo-image-elements: Fix heat purge deleted cron job  https://review.openstack.org/12174414:25
openstackgerritLars Kellogg-Stedman proposed a change to stackforge/kolla: starting heat-engine-api dockerfile work  https://review.openstack.org/12584314:26
sdake_morgan agree sorry for beeing lazy :)14:26
sdake_too old to experiment i guess ;)14:27
sdake_on the plus side my bitcoin miner made 24k in the lsat year in profit14:28
sdake_although I am getting ready to unplug it14:28
sdake_when i got it was making 450/day14:29
sdake_now it makes 6$/day14:29
*** dsneddon has quit IRC14:30
*** pcrews has joined #tripleo14:30
*** dsneddon has joined #tripleo14:31
sdake_radez larsks think you can work thorug hthe review qwueue a bit ?14:42
sdake_i'm going to start having conflicts14:42
radezsure14:43
*** noslzzp has joined #tripleo14:43
sdake_tia14:44
larskssdake_: I am in fact helping jlabocki with git right now re: one of those reviews...14:44
*** yuanying has joined #tripleo14:44
*** rdopieralski has quit IRC14:45
sdake_cool14:45
larskssdake_: can you review https://review.openstack.org/#/c/125823/14:46
openstackgerritA change was merged to openstack/tripleo-image-elements: Fix DISTRO_NAME usage for centos7 element  https://review.openstack.org/12553914:46
openstackgerritA change was merged to stackforge/kolla: mariadb: Set container port in pod to 3306  https://review.openstack.org/12584814:47
openstackgerritjames labocki proposed a change to stackforge/kolla: Using new fedora base image from kollaglue  https://review.openstack.org/12584314:47
openstackgerritA change was merged to stackforge/kolla: fix keystone dockerfile  https://review.openstack.org/12582314:48
*** ifarkas has quit IRC14:48
*** yuanying has quit IRC14:50
*** yamahata has joined #tripleo14:50
sdakelarsks that keystone patch is working as expected now14:51
*** dsneddon has quit IRC14:51
larsksnice.14:52
sdakeonly takes 45 mins to test groan14:52
openstackgerritLars Kellogg-Stedman proposed a change to stackforge/kolla: Keystone: remove extra commas and tabs in JSON  https://review.openstack.org/12585314:52
*** sdake_ has quit IRC14:53
*** dsneddon has joined #tripleo14:53
openstackgerritA change was merged to stackforge/kolla: Using new fedora base image from kollaglue  https://review.openstack.org/12584314:54
openstackgerritSteven Dake proposed a change to stackforge/kolla: rabbitmq: remove trailing commas in tabs so JSON parses  https://review.openstack.org/12585814:56
openstackgerritSteven Dake proposed a change to stackforge/kolla: Heat: Remove JSON trailing commas that don't parse  https://review.openstack.org/12585914:56
openstackgerritSteven Dake proposed a change to stackforge/kolla: glance: remove JSON trailing commas that don't parse  https://review.openstack.org/12585714:56
openstackgerritSteven Dake proposed a change to stackforge/kolla: Swift: Remove trailing JSON commas that don't parse  https://review.openstack.org/12586014:56
openstackgerritSteven Dake proposed a change to stackforge/kolla: Keystone: remove extra commas and tabs in JSON  https://review.openstack.org/12585314:56
*** lazy_prince is now known as killer_prince14:58
*** pensu has quit IRC15:00
*** untriaged-bot has joined #tripleo15:00
untriaged-botUntriaged bugs so far:15:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/137541415:00
uvirtbotLaunchpad bug 1375414 in tripleo "mysql/rabbit overcloud issues when the bootstrap node is removed" [Undecided,New]15:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/137620015:00
uvirtbotLaunchpad bug 1376200 in tripleo "Stale connections when VIP is moved" [High,New]15:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/137542215:00
uvirtbotLaunchpad bug 1375422 in tripleo "Vlan not recreated on overcloud controller reboot" [Undecided,In progress]15:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/136616415:00
uvirtbotLaunchpad bug 1366164 in tripleo "Add domain roles for admin user" [Undecided,New]15:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/136774215:00
uvirtbotLaunchpad bug 1367742 in tripleo "keepalived brings the VIP up in multiple places on a controller reboot" [Undecided,New]15:00
*** untriaged-bot has quit IRC15:00
*** dtantsur is now known as dtantsur|afk15:00
openstackgerritA change was merged to stackforge/kolla: fix mariadb Dockerfile  https://review.openstack.org/12582715:01
*** marun has joined #tripleo15:01
*** jp_at_hp has quit IRC15:07
radezsdake: larsh: https://review.openstack.org/#/c/125913/ if jenkins passed then the patch is good right?15:08
sdakeradez we aren't gating atm15:11
sdakeradez I think I can add that pretty easily tho15:12
*** rhallisey has quit IRC15:12
radezah, I kinda ignored the noops :)15:12
sdakeradez i think we can go ahead and merge that tho15:12
radezack15:12
sdakeI am in process of adding a bashate gate15:13
sdakewe can add this json validate gate next15:13
*** sdake_ has joined #tripleo15:15
*** pensu has joined #tripleo15:16
*** spzala has joined #tripleo15:17
*** sdake_ has quit IRC15:21
*** hewbrocca has quit IRC15:21
*** rainya has joined #tripleo15:25
*** rhallisey has joined #tripleo15:25
*** rushiagr_away is now known as rushiagr15:26
*** sdake_ has joined #tripleo15:31
openstackgerritDougal Matthews proposed a change to openstack/tuskar: Add MySQL and PostgreSQL test cases  https://review.openstack.org/12595815:34
*** rushiagr is now known as rushiagr_away15:38
*** sdake_ has quit IRC15:44
*** yuanying has joined #tripleo15:46
*** rhallisey has left #tripleo15:49
*** rhallisey has joined #tripleo15:49
*** rainya has quit IRC15:57
*** rushiagr_away is now known as rushiagr15:59
*** cwolferh has joined #tripleo16:04
*** rainya has joined #tripleo16:06
*** rushiagr is now known as rushiagr_away16:09
openstackgerritA change was merged to stackforge/kolla: Initial libvirt and nova-compute structure  https://review.openstack.org/12589616:17
*** bvivek has quit IRC16:18
*** dshulyak_ has quit IRC16:21
*** jistr has quit IRC16:21
openstackgerrittherese-mchale proposed a change to openstack/tripleo-image-elements: Change initial state and failback policy for VIP  https://review.openstack.org/12600916:27
*** bvivek has joined #tripleo16:29
sdakelarsks do you think you can login to rhallisey's macine and get his neutron rolling16:31
sdakeI would, but I'm not entirely sure what you did ;-)16:31
rhalliseyI'm restart the vm trying again16:31
larsksrhallisey: if you can get me a login, I am happy to take a look.16:31
rhalliseydestroyed it*16:31
larskskaboom.16:31
sdakeare you trying to run heat in a vm?16:32
rhalliseyya16:32
sdakeymm, why notinstall rdo on your workstation16:32
sdakethat will actually work ;)16:32
sdakea vm will be too slow and likely not work I think16:32
rhalliseyI mean I could do it locally16:33
sdakei can get you a lab machine if you need16:33
rhalliseydo you know of one?16:33
sdakebut locally is probably better16:33
sdakeI can find things ;)16:34
larsksrhallisey: I am using a beaker machine for testing.16:35
larsksNice because it has more memory than my laptop.16:35
larsks(but getting reservations from beaker can sometimes be a crapshoot when things are busy)16:36
sdakeya beaker is good rhallisey16:37
sdakeI have an extr aworkstation in my house dedicated to dev16:37
sdakeso I just use that16:37
sdakeonly 16gig ram tho16:37
*** noslzzp has quit IRC16:39
*** bvivek has quit IRC16:41
*** pelix has quit IRC16:45
openstackgerritA change was merged to stackforge/kolla: Keystone: remove extra commas and tabs in JSON  https://review.openstack.org/12585316:47
openstackgerritA change was merged to stackforge/kolla: glance: remove JSON trailing commas that don't parse  https://review.openstack.org/12585716:47
openstackgerritA change was merged to stackforge/kolla: rabbitmq: remove trailing commas in tabs so JSON parses  https://review.openstack.org/12585816:47
openstackgerritA change was merged to stackforge/kolla: Heat: Remove JSON trailing commas that don't parse  https://review.openstack.org/12585916:47
openstackgerritA change was merged to stackforge/kolla: Swift: Remove trailing JSON commas that don't parse  https://review.openstack.org/12586016:47
openstackgerritA change was merged to stackforge/kolla: Keystone: Do a service and endpoint create  https://review.openstack.org/12584416:48
sdake\o/16:49
openstackgerritChmouel Boudjnah proposed a change to stackforge/kolla: Only push with the build script with -p  https://review.openstack.org/12601716:56
openstackgerritSteven Dake proposed a change to stackforge/kolla: Make a k8s top level directory for k8s files  https://review.openstack.org/12590516:57
openstackgerritSteven Dake proposed a change to stackforge/kolla: Remove whitespace errors in mariadb JSON  https://review.openstack.org/12601816:57
sdakechmouel have you seen https://blueprints.launchpad.net/kolla/+spec/image-dev-workflow17:01
chmouelsdake: no i didn't but i guess that goes with the comment in the review17:01
sdakeright17:02
sdakeI forgot about the blueprint until I pushed the review button17:02
chmoueli guess my patch is just a workaround and that would be the proper solution17:02
sdakeright17:02
larskschmouel: sdake: I have some patches to push this afternoon for folks to look at re: the workflow stuff.17:02
sdakelarsks sweet !17:02
chmouellarsks: cool, looking forward to it, can hold on on my patch then for now17:02
*** jlabocki has joined #tripleo17:04
sdakelarsks do you know if the rabbitmq container works?17:04
larskssdake: havne't checked recently.17:04
larsksjlabocki: welcome! :)17:05
jlabockihola17:05
sdakelarsks was it ever working?17:05
larsksI think so? It's a pretty simple service.17:05
larsksIf not, it should be an easy target.17:05
jlabockiso sdake larks - with rhallisey working on nova and radez working on glance17:06
jlabockiI'm not sure what I should focus on making work17:06
larsksjlabocki: look at the rabbitmq container and make sure it works.17:07
jlabockiok17:07
larsksyou will need a working glance or something to test it, I guess.17:07
jlabockiyes17:09
jlabockiwhat other service is working?17:09
*** rainya has quit IRC17:09
jlabockibuilding latest rabbitmq image now … thinking about ways to test it without installing glance package based on my system17:11
*** jang1 has joined #tripleo17:11
*** andreaf has quit IRC17:12
*** regebro has quit IRC17:12
*** andreaf has joined #tripleo17:12
*** rainya has joined #tripleo17:13
chmoueli have added some comments on the blueprint for swift https://blueprints.launchpad.net/kolla/+spec/kube-swift-container17:14
chmouellet me know what do you think17:14
sdakemonday we can set an arbitratry deadline for a milestone-117:24
sdakebut here is what I put together based upon what I hear folks are working on17:25
sdakehttps://blueprints.launchpad.net/kolla/milestone-117:25
sdakechmouel swift-ring-master looks pretty good17:26
chmouelsdake: but does that need to be packaged or can it be run directly from a git checkout?17:27
sdakeI am ok with git checkout17:27
*** TheJulia has quit IRC17:27
sdakealthough ideally packaged would be best17:27
sdakeI can tackle that , atleast on fedora-rdo17:27
sdakeother distros, not so much17:27
sdakeI don't see the license tho17:27
chmoueli can ping him for license17:27
sdakeyes that is mandatory17:27
sdakeASL2.0 would be ideal17:28
chmouelok17:28
sdakebut ya git checkout for now works17:28
sdakethe nice thing about packaging is it makes sure the licenses are all good to go17:28
sdakeif we use a git cvheckout of this, we are unsure of the licensing, it creates a problem for us if we distribute the image s permenatly17:28
sdakeand then hes like "the license is proprietary"17:28
sdakenot saying he would, but if that happened, it would create a charlie foxtrott17:29
jlabockilarks: there is no start.sh for rabbitmq17:30
jlabockiwith regards to the variables17:30
jlabockiin rabbitmq-env.conf17:30
sdakeinit-rabbmitmq.sh ?17:30
jlabockido you just expect those to be environment variables?17:30
jlabockiah17:30
chmouelsdake: yeah! i just sent him that https://twitter.com/chmouel/status/51809057311137382417:30
jlabockithe Dockerfile17:30
jlabockiADD start.sh /start.sh17:31
jlabockiCMD ["/start.sh"]17:31
jlabockineeds Add init-rabbitmq.sh17:31
jlabockiinstead of start.sh17:31
jlabockiyeah?17:31
sdakechmouel thanks17:31
sdakeI think init-rabbitmq is bust17:31
jlabockialso, when I run ./build the image is never tagged17:31
sdakeprobably best to start fresh with a start.sh17:31
jlabockieven though it says docker build --tag kollaglue/fedora-rdo-rabbitmq .17:32
jlabockiin build17:32
jlabockihmmm17:32
sdakeshould definat4ely tag17:32
chmouelsdake: cool, he just add it https://github.com/pandemicsyn/swift-ring-master/blob/master/LICENSE17:32
chmouelsdake: let's see if i can plug this together17:32
sdakecool I'll work out a package now then17:33
sdakewhere is the download git tag button on github chhmouel :)17:39
sdakeperhaps there isn't one17:39
sdakeoh I got it17:39
*** lucasagomes is now known as lucas-afk17:41
openstackgerritA change was merged to openstack/tuskar: Use resource registry for seed role matching  https://review.openstack.org/12568617:41
*** jprovazn has joined #tripleo17:43
*** andreaf has quit IRC17:47
*** andreaf has joined #tripleo17:47
*** derekh has quit IRC17:50
*** dsneddon has quit IRC17:52
*** dsneddon has joined #tripleo17:53
*** noslzzp has joined #tripleo18:03
*** lsmola has quit IRC18:03
*** spzala has quit IRC18:04
*** cwolferh_ has joined #tripleo18:06
*** cwolferh has quit IRC18:07
*** rainya has quit IRC18:14
*** andreaf has quit IRC18:27
*** andreaf has joined #tripleo18:27
*** spzala has joined #tripleo18:28
*** julim has quit IRC18:34
*** julim has joined #tripleo18:38
sdakechmouel here is a spec file if you want to build a specin the dockerfile18:43
openstackgerritA change was merged to stackforge/kolla: Only push with the build script with -p  https://review.openstack.org/12601718:43
sdakehttp://paste.fedoraproject.org/138977/1412361818:43
sdakeI'll probably hve it merged into fedora by monday18:43
sdakef20 may take more time because we have manual check policy18:43
chmouelah nice, i guess it can go to copr as well if we need it ?18:44
openstackgerritA change was merged to stackforge/kolla: Add validate-json tox target  https://review.openstack.org/12591318:44
chmouelbeen a long time since i have read specfiles18:44
openstackgerritjames labocki proposed a change to stackforge/kolla: Updated base image in Dockerfile and changed CMD to reference correct file  https://review.openstack.org/12603718:46
slaglejdob: (or any tuskar folks) can you look at http://paste.openstack.org/show/118217/ ?18:50
jdobslagle: shadower changed it so you have to pass in the environment file as well as overcloud.yaml18:51
chmouelsdake: https://review.openstack.org/#/c/126038/18:52
slaglejdob: do you know if he plan on submitting the corresponding change to the tuskar element?18:52
jdobnot sure; I'd assume so. to be honest, he might be surprised it landed so quickly18:53
openstackgerritLars Kellogg-Stedman proposed a change to stackforge/kolla: replace jsonlint with jq  https://review.openstack.org/12603918:54
larskschmouel: ^^^^ that patch uses "jq", which is packaged for fedora, etc, rather than jsonlint, which does not seem to be.18:54
*** ccrouch has quit IRC18:54
slaglejdob: if you know, can you tell me what i need to add?18:55
slaglejdob: or you could type those same characters into a patch for the tuskar element :)18:55
chmouellarsks: yeah but that would mean that would work (maybe) only on fedora and would need a custom job for infra?18:55
*** ccrouch has joined #tripleo18:55
larskschmouel: Oh, I don't, does infra have jsonlint but not jq?18:56
larsksNever mind then.18:56
larsksABANDON ALL THE THINGS!18:56
chmouellarsks: jsonlint is installed via tox and pulled up from pypi18:56
chmouellarsks: heh18:56
chmouellarsks: you can maybe test if jq exist and use that instead for manual testing on a fedora box that has it18:57
chmouelbut either way tox -e validate-json should work18:57
larsksWe could also just use 'python -mjson.tool'18:57
chmouellarsks: that was my first iteration of the patch18:58
larsksWhich will give a useful exit code, but no details.18:58
larsks:)18:58
chmouellarsks: but it doesn't give you details indeed18:58
jdobslagle: hahaha, lemme finish something up and I'll take a look18:58
*** noslzzp has quit IRC19:00
slaglejdob: i think i got it actually, just need to add --resource-registry /opt/stack/tripleo-heat-templates/overcloud-resource-registry.yaml to tuskar-load-roles19:00
*** ccrouch has quit IRC19:00
jdobya, that's really all it is19:00
jdobthe rest should fall into place19:00
openstackgerritJames Slagle proposed a change to openstack/tripleo-image-elements: Use --resource-registry parameter  https://review.openstack.org/12604119:03
*** marun has quit IRC19:04
*** ccrouch has joined #tripleo19:04
*** prad has joined #tripleo19:10
openstackgerritChmouel Boudjnah proposed a change to stackforge/kolla: Rename validate-json target as pep8 and improve it  https://review.openstack.org/12604219:14
chmouellarsks: ^ that should improve things19:14
larskschmouel: you need to set the return value when using the python linter.19:15
larsks(ret=1)19:15
larsksotherwise, seems fine.19:15
openstackgerritChmouel Boudjnah proposed a change to stackforge/kolla: Rename validate-json target as pep8 and improve it  https://review.openstack.org/12604219:16
chmouellarsks: you are absolutely right :)19:16
larskschmouel: so now I am going to ask :)...why not just drop the "continue" in that clause?  There is no code following the if statement...19:16
chmouellarsks: i am not sure i have added that in case we had new suff after (i.e: other linters)19:17
chmouelbut nevermind not necessary ince it's the last else19:17
larsksso why not for the previous caluse then?19:17
larsksE.g., the error behavior should be the same for jsonlint vs. python...19:18
openstackgerritChmouel Boudjnah proposed a change to stackforge/kolla: Rename validate-json target as pep8 and improve it  https://review.openstack.org/12604219:18
chmouellarsks: true true just changed it ^19:19
larsksAll right, I will stop doing my columbo impression now ("...and one more thing...")19:19
chmouelhahah, been a long time since i have watched that19:21
*** andreaf has quit IRC19:21
*** andreaf has joined #tripleo19:22
sdakechmouel wait for my merge to go in for gating before finishing up 12603819:24
*** marun has joined #tripleo19:25
chmouelsdake: ah cool okay,19:25
sdakehttps://review.openstack.org/#/c/125893/19:27
sdakeexample of how to add gating19:27
chmouelok19:28
sdakeI am not sure on tox gate19:28
sdakeprobably very similiar19:28
sdakeI am going to make the bashate job gating once we fix the bashate errors19:28
*** marun has quit IRC19:30
*** sdake_ has joined #tripleo19:30
*** pensu has quit IRC19:33
*** sdake_ has quit IRC19:35
*** ccrouch has quit IRC19:42
*** marun has joined #tripleo19:47
openstackgerritA change was merged to openstack/os-cloud-config: fix formatting issues in README.rst  https://review.openstack.org/12577519:56
openstackgerritA change was merged to stackforge/kolla: Rename validate-json target as pep8 and improve it  https://review.openstack.org/12604219:57
*** jprovazn has quit IRC20:03
*** sdake_ has joined #tripleo20:10
openstackgerritDan Radez proposed a change to stackforge/kolla: Updates to glance start scripts, db and user/role creation - adding db sync and db creation as utf8 - fixing user and role creation  https://review.openstack.org/12579120:11
sdakeradez wher edoes DB_ROOT_PASSWORD come from?20:19
radezenv20:19
sdakeis that a global environment variable between all services?20:19
radezlook at glance.json20:19
*** dprince has quit IRC20:20
radezsdake:  is this the error you were getting before you ran off master?20:21
radez Failed to get the key: registry/services/endpoints/mariadbmaster 100: Key not found (/registry/services/endpoints)20:21
sdakeack20:21
sdakeneed new apiserver20:21
openstackgerritLars Kellogg-Stedman proposed a change to stackforge/kolla: introduce a new build mechanism  https://review.openstack.org/12605720:21
sdakejust pull master compile systemd stop the service20:21
sdakereplace binary systemd start the service20:21
radezkk, thx20:21
sdakeand your good to go :)20:21
larskschmouel: that patch ^^^ represents my thoughts on image building. Let me know what you think.20:22
chmouellarsks: will do, prob not now as it's kind of late here,20:23
larsksWhat, the whole world isn't on my time zone? ;)20:23
sdakeyay for the US20:24
sdakeare you in europe chmouel?20:24
chmouelsdake: ya in paris20:24
sdakecool20:24
sdakeI've been to paris 3 times20:24
chmouelnice, i guess you will have a fourth time soon :)20:24
sdakestruggle feeding myself20:24
sdakeI don't do cheese20:24
sdakeI wont be at summit this year20:24
chmouelwine is cheep tho20:24
sdakeI do wine :)20:24
chmouelheh20:25
sdakeI could always feed myself with liquor on the road, that was easy20:25
sdakebut my manager would always lecture me on return20:25
chmouelhahah20:25
sdakestartups ftw20:25
sdakerht is so cheap, we get 40$ /day for food20:25
sdakeand it gets cut if breakfast/lunch/dinner ar eprovided by the venues20:25
chmoueli have just learnt that20:25
sdakebasically I just pay for my own food when I travel at rht20:26
sdakeeven though the meals are usually3-4x what I'd pay at home20:26
sdakebut I dont travel much \o/20:26
chmouelwhere are you  based?20:26
sdakescottsdale, Arizona20:26
sdakegmt-720:26
sdakegetting nice here now - summers are 115F20:27
sdakebut summer is almost bust20:27
sdakeI have to use the heater outside in the mornings when I smoke a cigar20:27
chmouelwooh  i don't think it ever reach that here20:27
sdakeI was at paris in summer and it was humid iird20:27
chmouelyou smoke cigars in the morning?20:27
sdakeiirc20:27
sdakeya little ones20:28
chmouelthat's boss20:28
sdakeusually try to limit to 3 a day20:28
sdakebasically I smoke pimp cigars :)20:28
chmouelheheh20:28
sdakehttp://www.famous-smoke.com/arturo+fuente+cubanitos+10~10+cigars/item+1422320:28
sdakethats what I smoke typically20:29
*** andreaf has quit IRC20:29
sdakewith some bigger cigars mixed in occiosunally20:29
sdakebut I don't have time for a real cigar except on the weekends20:29
*** andreaf has joined #tripleo20:29
radezsdake: I did a yum downgrade kubernetes just for fun to see if the last version was ok, now I can connect to mysql without any trouble20:29
chmouelsdake: nice, i never got into big cigars, it took too much time to smoke (used to be a heavy ciggies smoker)20:33
sdakeI've never smoked a cigarette if you can believe that20:34
sdakeonly cigars20:34
sdakebut I dont like to go without cigars - so definately still addictive20:34
sdakebut I can not smoke a ccigar for 3-4 days and not freak20:34
sdakewhereas cigarrate smokers freak after a few hours :)20:34
chmoueltrue that, it takes a while to get the nicotine out of your brain, and even now when i have a few drinks i think about smoking a cig20:36
chmouelcigars is much more chill20:36
chmouelok well night all20:38
sdaketake care chmouel20:44
sdakeradez was that g lance jason correct out of the gate?20:45
*** jdob has quit IRC20:45
sdakeI just kind of hacked it together20:45
sdakejason/json20:45
radezalmost, the registry port was wrong20:45
radezit was 9191 not 929220:45
radezthere's a service on both of those ports20:45
radezand there was no db creation20:45
radezthat's most of what I fixed up20:45
radezendpoints and services tenant were also needed...20:46
sdakeall of t he incomplete services have hacked together json20:46
sdakejustsearch replace guess etc20:46
radezI noticed :)20:46
radezit's a good starting point20:46
sdakewe need to read the json into a python program20:46
sdakeand modify it with a global ini config file20:47
sdakeand contact the rest server directly20:47
sdakeso we don't have kubecfg involved20:47
sdakework for later :)20:47
openstackgerritLars Kellogg-Stedman proposed a change to stackforge/kolla: use "crux" for creating users/endpoints  https://review.openstack.org/12606720:48
radezsdake: I think the keystone endpoint ip didn't get passed to my giddy up...20:48
* radez checks20:48
larsksradez: ^^^ is my proposal for handling keystone user/endpoint/service stuff.20:48
larsksmarked as WIP right now.20:48
openstackgerritA change was merged to stackforge/kolla: Updates to glance start scripts, db and user/role creation - adding db sync and db creation as utf8 - fixing user and role creation  https://review.openstack.org/12579120:49
*** lucas-afk has quit IRC20:49
openstackgerritDan Radez proposed a change to stackforge/kolla: adding ADDR env vars while the start script depends on them  https://review.openstack.org/12607020:55
radezsdake: ^ this make sure that the json files have the env you added to the start script20:56
radezbah, I forgot commas....20:57
openstackgerritDan Radez proposed a change to stackforge/kolla: adding ADDR env vars while the start script depends on them  https://review.openstack.org/12607020:58
*** untriaged-bot has joined #tripleo21:00
untriaged-botUntriaged bugs so far:21:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/137541421:00
uvirtbotLaunchpad bug 1375414 in tripleo "mysql/rabbit overcloud issues when the bootstrap node is removed" [Undecided,New]21:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/137620021:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/137542221:00
uvirtbotLaunchpad bug 1376200 in tripleo "Stale connections when VIP is moved" [High,New]21:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/136616421:00
uvirtbotLaunchpad bug 1375422 in tripleo "Vlan not recreated on overcloud controller reboot" [Undecided,In progress]21:00
uvirtbotLaunchpad bug 1366164 in tripleo "Add domain roles for admin user" [Undecided,New]21:00
untriaged-bothttps://bugs.launchpad.net/tripleo/+bug/136774221:00
uvirtbotLaunchpad bug 1367742 in tripleo "keepalived brings the VIP up in multiple places on a controller reboot" [Undecided,New]21:00
*** untriaged-bot has quit IRC21:00
*** marun is now known as maroon21:01
sdakeradez I dont think ou need that21:06
radezsdake: oh where does it come from?21:07
sdakeradez you need to start the services prior to the pods21:07
radezoh... but then doesn it get the internal and not the external addrs?21:07
sdakeonce the service is registered, global env vars are set for each container in each pod21:07
sdakeyes internal addresses21:07
radezkk, i'll update the patch then just to show that services need to be started first in the README21:08
sdakeI am not sure if that will fix the problem you see, because I'm not sure what problem you face :)21:08
sdaketime for a smoke bbiaf :)21:08
larsksradez: and if you have any free time this evening to take a look at https://review.openstack.org/#/c/126067/ and let me know if that looks sane, that would be awesome.21:10
openstackgerritLars Kellogg-Stedman proposed a change to stackforge/kolla: use "crux" for creating users/endpoints  https://review.openstack.org/12606721:11
openstackgerritDan Radez proposed a change to stackforge/kolla: Updating the README to show starting services before pods  https://review.openstack.org/12607021:12
radezlarsks: yea it's on my list to take a look at... probably be later tonight21:12
jlabockiso larsks sdake - what I get from rabbitmq image21:12
jlabockihttp://pastebin.test.redhat.com/23807321:12
larsksNo worries.21:13
jlabockiI think I need to update my erlang cookie21:13
jlabockibut the documentation on rabbitmq is difficult for me to understand21:13
jlabockiI think rabbitmq creates a cookie has when it starts21:13
sdake_jlabocki if you dont get the work finished tonight can you git review and set workflow to -121:13
sdake_so I can take a look over the weekend21:13
jlabockisure, how do I do that? (set workflow to -1)21:14
sdake_just click review21:14
sdake_-1 means work in progress21:14
jlabockisorry, I'm very slow, where do I click review?21:14
sdake_should be on the page somewhere21:14
sdake_you can review your own patches21:14
sdake_maybe its "comment"21:15
larsksjlabocki: Yeah, "add comment"21:15
larsksAt the bottom of the review page.21:15
radezsdake_: I've got keystone happy and launching glance now. I think I can have something working well for monday21:15
*** andreaf has quit IRC21:15
*** andreaf has joined #tripleo21:15
sdake_sweet :)21:16
sdake_jlabocki dont sweat being slow to learn this gerrit stuff - it literally took me 3 months of continuous dev to learn the openstack workflow to a high degree of skill21:17
radezsdake_: do you want to show it all coming up or do you want to have everything running already?21:17
sdake_you have an advantage - trainers :)21:17
sdake_going to show launching I think21:17
radezI would imagine it would be more efficent to have everythign up and walk through what's already working...21:17
radezok... maybe prelaunch to get all the images in place?21:18
sdake_yup of course21:18
radezack21:18
jlabockisdake: done - https://review.openstack.org/#/c/126037/21:18
jlabockilet me know if I screwed anything up21:18
radezjlabocki: you spelled your name wrong....21:18
radezjlabocki: j/k21:18
radez:)21:18
jlabockiha21:19
jlabockinot the first time21:19
*** prad has quit IRC21:19
sdake_jlabocki nice work on reporting  the state of the wip21:20
jlabockihey hey! I did something right!21:21
jlabocki;)21:21
jlabockihopefully next week when my calendar isn't riddled with meetings I can make more progress during updates21:22
sdake_joys of engineering - no meetings :)21:22
sdake_i'm going to try to get libvirt and nova done this weekend21:23
sdake_I a mgoing out tonight with friends but rest of time is free21:23
sdake_nova may not be so easy21:24
sdake_fun fact installing libguestfs installs 250 packages21:24
radezsdake_: the keystone pod still needs to create the services tenant... I'll put in a patch later but I need to run home21:24
sdake_radez cool maybe I'll do that now21:25
radezthat'd be great. thx21:25
sdake_although I have this rpm i'm looking at fixing21:25
sdake_so many fun things to work on :)21:25
radezhehe c ya later21:25
*** radez is now known as radez_g0n321:25
sdake_ya radez your probablygoing to have to tackle that21:25
sdake_I am not sure i can get to it21:25
sdake_I need to get my car washed21:26
rhalliseysdake_, I'll be around all weekend too21:37
sdake_cool21:37
*** andreaf has quit IRC21:40
*** andreaf has joined #tripleo21:40
sdake_rhallisey did you get that beaker machine rolling?21:40
*** Dafna has quit IRC21:41
rhalliseyI actually grabbed a lab machine21:41
rhalliseyIll be all set by tonight21:41
sdake_cool21:41
*** rhallisey has quit IRC21:42
*** rhallisey has joined #tripleo21:42
*** sdake_ has quit IRC21:57
*** rhallisey has quit IRC21:59
*** andreaf has quit IRC22:05
openstackgerritTzu-Mainn Chen proposed a change to openstack/tuskar-ui: Changed service configuration table to multiple tabs  https://review.openstack.org/12608722:07
*** chuckC_ has quit IRC22:08
*** maroon has quit IRC22:15
*** ChuckC has quit IRC22:16
*** andreaf has joined #tripleo22:23
*** morazi has quit IRC22:24
*** rwsu has quit IRC22:24
*** uberj has joined #tripleo22:32
*** chuckC has joined #tripleo22:33
*** jang1 has quit IRC22:49
*** ChuckC_ has joined #tripleo22:54
*** chuckC has quit IRC22:57
*** ChuckC_ is now known as ChuckC22:58
*** chuckC_ has joined #tripleo22:58
openstackgerritBen Nemec proposed a change to openstack/diskimage-builder: Save extended attributes when creating tar  https://review.openstack.org/12542822:59
*** chuckC_ has quit IRC23:00
*** chuckC_ has joined #tripleo23:01
*** rhallisey has joined #tripleo23:21
*** sdake_ has joined #tripleo23:28

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