*** achanda has quit IRC | 00:00 | |
*** achanda_ has quit IRC | 00:01 | |
*** dims has joined #kolla | 00:03 | |
*** dims has quit IRC | 00:05 | |
*** dims has joined #kolla | 00:05 | |
*** dims has quit IRC | 00:10 | |
*** jruano has quit IRC | 00:10 | |
*** dims has joined #kolla | 00:17 | |
openstackgerrit | Steven Dake proposed stackforge/kolla: Start of removal of docker directory https://review.openstack.org/217981 | 00:20 |
---|---|---|
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove kolla-common.sh symlink https://review.openstack.org/217984 | 00:26 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move kolla-common.sh to base directory https://review.openstack.org/217985 | 00:26 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove config-external and start for ceilometer https://review.openstack.org/217987 | 00:36 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move config-external and start for ceilometer https://review.openstack.org/217988 | 00:36 |
rhallisey | sdake_, don't you want to leave docker_template alone and kill docker? | 00:38 |
sdake_ | read the first commit | 00:38 |
*** jtriley has joined #kolla | 00:38 | |
sdake_ | it explains the approach | 00:38 |
sdake_ | we have to rid ourselves of the symlinks in docker/common first | 00:38 |
rhallisey | gotcah | 00:39 |
rhallisey | gotcha | 00:39 |
*** weiyu_ has joined #kolla | 00:44 | |
*** britthou_ has joined #kolla | 00:48 | |
sdake_ | rhallisey we need naother option in the dictionary | 00:48 |
sdake_ | this tells the tool whether to generate a --config-file option for the copied file or not | 00:48 |
sdake_ | make sense? | 00:49 |
rhallisey | yup | 00:49 |
*** britthouser has quit IRC | 00:51 | |
*** britthou_ has quit IRC | 00:52 | |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove config-external and start for ceilometer https://review.openstack.org/217987 | 00:54 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move config-external and start for ceilometer https://review.openstack.org/217988 | 00:54 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove config-external and start for cinder https://review.openstack.org/217994 | 00:54 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move config-external and start for cinder https://review.openstack.org/217995 | 00:54 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks for designate https://review.openstack.org/217996 | 00:54 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move config-external and start for designate https://review.openstack.org/217997 | 00:54 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks for galera https://review.openstack.org/217998 | 00:57 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for galera https://review.openstack.org/217999 | 00:57 |
*** jtriley has quit IRC | 01:00 | |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks for glance https://review.openstack.org/218001 | 01:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for glance https://review.openstack.org/218002 | 01:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove scripts for gnocchi https://review.openstack.org/218003 | 01:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for gnocchi https://review.openstack.org/218004 | 01:04 |
rhallisey | that's going to be lik 60 patchs | 01:04 |
rhallisey | holy hell | 01:04 |
sdake_ | you want in 1 patch instead? ;-) | 01:04 |
rhallisey | no | 01:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks for haproxy scripts https://review.openstack.org/218005 | 01:06 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for haproxy https://review.openstack.org/218006 | 01:06 |
*** juggler_ has joined #kolla | 01:07 | |
sdake_ | rhallisey your work depends on this work merging first | 01:08 |
rhallisey | ya | 01:08 |
*** juggler_ is now known as juggler | 01:08 | |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks for heat scripts https://review.openstack.org/218008 | 01:10 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move heat scripts to docker_teplates https://review.openstack.org/218009 | 01:10 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove horizon symlinks to scripts https://review.openstack.org/218010 | 01:10 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move horizon scripts to docker_templates https://review.openstack.org/218011 | 01:10 |
*** zhiwei has joined #kolla | 01:14 | |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks to scripts for keepalived https://review.openstack.org/218015 | 01:16 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move Keepalived scripts to docker_tmeplates dir https://review.openstack.org/218016 | 01:16 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks to scripts for keystone https://review.openstack.org/218017 | 01:16 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move keystone scripts to docker_templates dir https://review.openstack.org/218018 | 01:16 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove kolla_ansible symlinks https://review.openstack.org/218019 | 01:18 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move kolla-ansible scripts to docker_templates https://review.openstack.org/218020 | 01:18 |
sdake_ | look at the bright side | 01:21 |
sdake_ | once this is done we won't have to maintian two sets of dockerfiles ;-) | 01:21 |
*** jasonsb has quit IRC | 01:25 | |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks in magnum docer_templates https://review.openstack.org/218025 | 01:26 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for magnum to docker_templates https://review.openstack.org/218026 | 01:26 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks to scripts in memcached https://review.openstack.org/218027 | 01:26 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for memcached to docker_templates directory https://review.openstack.org/218028 | 01:26 |
rhallisey | sdake_, ya I'm dealing with that now | 01:26 |
rhallisey | messy branch | 01:26 |
*** erkules_ has joined #kolla | 01:28 | |
*** erkules has quit IRC | 01:30 | |
*** jtriley has joined #kolla | 01:45 | |
*** jtriley has quit IRC | 01:50 | |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks for murano in docker_templates https://review.openstack.org/218039 | 01:53 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for murano to docker_templates directory https://review.openstack.org/218040 | 01:53 |
openstackgerrit | Steven Dake proposed stackforge/kolla: remove neutron symlinks in docker_templates https://review.openstack.org/218041 | 01:53 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for neutron to docker_templates https://review.openstack.org/218042 | 01:53 |
openstackgerrit | Kai Qiang Wu(Kennan) proposed stackforge/kolla: Add developer docs automatic produce support https://review.openstack.org/218046 | 02:00 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks for nova in docker_templates https://review.openstack.org/218048 | 02:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move all nvoe scripts to docker_templates https://review.openstack.org/218049 | 02:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove all symlinks for openvswitch in docker_templates https://review.openstack.org/218050 | 02:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move all scripts for openvswitch to docker_templates https://review.openstack.org/218051 | 02:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks for rabbitm in docker_templates https://review.openstack.org/218052 | 02:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move all scripts for rabbitmq to docker_templates https://review.openstack.org/218053 | 02:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks for wift in docker-templates https://review.openstack.org/218054 | 02:04 |
sdake_ | brainless work this is | 02:04 |
*** sdake_ is now known as sdake | 02:04 | |
*** dims has quit IRC | 02:06 | |
*** shardy_ has joined #kolla | 02:08 | |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for swift to docker_templates https://review.openstack.org/218055 | 02:09 |
*** shardy has quit IRC | 02:09 | |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove zaqar symlinks from docker_templates https://review.openstack.org/218056 | 02:10 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for zaqar to docker_templates dir https://review.openstack.org/218057 | 02:10 |
sdake | rhallisey did you ahve another review for me to bang on :) | 02:11 |
rhallisey | I do | 02:11 |
rhallisey | I'm just testing atm | 02:11 |
*** unicell1 has quit IRC | 02:13 | |
*** SourabhP has joined #kolla | 02:13 | |
*** shardy_ has quit IRC | 02:13 | |
*** shardy has joined #kolla | 02:14 | |
*** SourabhP_ has joined #kolla | 02:16 | |
*** SourabhP has quit IRC | 02:19 | |
*** SourabhP_ is now known as SourabhP | 02:19 | |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for zaqar to docker_templates dir https://review.openstack.org/218057 | 02:20 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove zaqar symlinks from docker_templates https://review.openstack.org/218056 | 02:20 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for swift to docker_templates https://review.openstack.org/218055 | 02:20 |
*** ccrouch has joined #kolla | 02:22 | |
*** jasonsb has joined #kolla | 02:23 | |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove docker directory https://review.openstack.org/218061 | 02:24 |
sdake | 46 patches yay :) | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks to scripts in memcached https://review.openstack.org/218027 | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for magnum to docker_templates https://review.openstack.org/218026 | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks in magnum docer_templates https://review.openstack.org/218025 | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for memcached to docker_templates directory https://review.openstack.org/218028 | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove kolla_ansible symlinks https://review.openstack.org/218019 | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move keystone scripts to docker_templates dir https://review.openstack.org/218018 | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks to scripts for keystone https://review.openstack.org/218017 | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move Keepalived scripts to docker_tmeplates dir https://review.openstack.org/218016 | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move kolla-ansible scripts to docker_templates https://review.openstack.org/218020 | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for neutron to docker_templates https://review.openstack.org/218042 | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for murano to docker_templates directory https://review.openstack.org/218040 | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: remove neutron symlinks in docker_templates https://review.openstack.org/218041 | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Start of removal of docker directory https://review.openstack.org/217981 | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks for murano in docker_templates https://review.openstack.org/218039 | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove docker directory https://review.openstack.org/218061 | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move config-external and start for cinder https://review.openstack.org/217995 | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove config-external and start for cinder https://review.openstack.org/217994 | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for zaqar to docker_templates dir https://review.openstack.org/218057 | 02:27 |
rhallisey | O.o | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks to scripts for keepalived https://review.openstack.org/218015 | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks for glance https://review.openstack.org/218001 | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for glance https://review.openstack.org/218002 | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove scripts for gnocchi https://review.openstack.org/218003 | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for gnocchi https://review.openstack.org/218004 | 02:27 |
sdake | gate was -1 on vlidate-links | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks for haproxy scripts https://review.openstack.org/218005 | 02:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for haproxy https://review.openstack.org/218006 | 02:27 |
sdake | sorry abou tthat ;) | 02:27 |
rhallisey | when mine's it will be 24 | 02:27 |
rhallisey | :/ | 02:27 |
rhallisey | :) | 02:28 |
*** achanda has joined #kolla | 02:28 | |
*** dims has joined #kolla | 02:33 | |
sdake | master of docker_templates doesnt seem to work on centos | 02:33 |
*** SourabhP has quit IRC | 02:52 | |
sdake | time to eat bbiaf | 02:53 |
*** weiyu_ has quit IRC | 02:57 | |
*** weiyu has joined #kolla | 03:03 | |
openstackgerrit | Sam Yaple proposed stackforge/kolla: DO NOT MERGE - gate haxoring https://review.openstack.org/217831 | 03:06 |
*** mfalatic has quit IRC | 03:07 | |
SamYaple | sdake: why are you doing 2 patches for each, this works fine https://review.openstack.org/#/c/217831/ | 03:07 |
SamYaple | you can just move it, gerit tracks the rename? | 03:07 |
sdake | interesting i'll try squashing them | 03:08 |
SamYaple | i told you this the other day too | 03:08 |
sdake | it used to not wor properly | 03:08 |
sdake | btw keystone is busted on centos binary templates | 03:08 |
sdake | and mariadb-server is not installed in the base image | 03:08 |
SamYaple | mysql-server is | 03:08 |
sdake | just the mariadb cli | 03:08 |
sdake | show me where | 03:09 |
sdake | if you see "mysql" its only the cli | 03:09 |
openstackgerrit | Ryan Hallisey proposed stackforge/kolla: WIP:Replace config-external with a JSON file https://review.openstack.org/216874 | 03:09 |
sdake | mysql-server is the server package | 03:09 |
SamYaple | what does mysql-devel pull in? | 03:09 |
sdake | no idea | 03:10 |
sdake | fixing keystone atm or attempting to do so | 03:10 |
SamYaple | either way, mariadb/mysql shooudlny be installed in the base like that | 03:10 |
SamYaple | and it does generated /var/lib/mysql crap | 03:10 |
sdake | i removed mariadb and for the most part things seem tow ork | 03:11 |
sdake | unless build from templates | 03:11 |
sdake | it takes me 70 minutes to do a build and push | 03:11 |
sdake | something got slower in the build script | 03:11 |
SamYaple | its still in base for the templates | 03:11 |
SamYaple | are you still running with -T1000 | 03:12 |
sdake | my unsubmitted patch address that | 03:12 |
sdake | no i dont ever use -T options | 03:12 |
sdake | not sure what the deal is because the gate is obviously fast | 03:12 |
SamYaple | the build patch made everything quick, and didnt touch pushing at all | 03:12 |
SamYaple | i blame your registry | 03:13 |
sdake | who knows what it is | 03:13 |
SamYaple | i dont see any remove mariadb from base patch in queue | 03:14 |
sdake | yes its unsubmitted | 03:15 |
SamYaple | ah | 03:15 |
*** Haomeng has joined #kolla | 03:15 | |
sdake | trying to verify it works before submitting | 03:15 |
sdake | it works for docker dir | 03:15 |
sdake | not sure on j2 | 03:15 |
sdake | I decided I am going to abandon docker dir entirely | 03:15 |
sdake | and focus work on j2 | 03:15 |
sdake | however keystone doesn't work on j2 | 03:15 |
SamYaple | anything you change please test ubuntu since that is 100% functional for all images | 03:16 |
SamYaple | and we have no ubuntu gate | 03:16 |
sdake | maria-libs must be installed in base as well | 03:16 |
sdake | i cant test ubuntu i don't have an ubuntu system | 03:16 |
SamYaple | you can build ubuntu | 03:16 |
SamYaple | and launch it | 03:16 |
SamYaple | on centos | 03:16 |
sdake | will that work? | 03:17 |
openstackgerrit | Ryan Hallisey proposed stackforge/kolla: WIP: Replace config-external with a JSON file for Galera https://review.openstack.org/217880 | 03:17 |
SamYaple | why wouldnt it | 03:17 |
SamYaple | anyway the options are let me review each patch and dont let patches merge without me reviewing them, or test it yourself | 03:17 |
SamYaple | since we dont have an ubuntu gate | 03:17 |
SamYaple | and ubuntu is the only thing everything works on | 03:17 |
*** alisonh has quit IRC | 03:18 | |
sdake | i'll see if it works | 03:18 |
openstackgerrit | Ryan Hallisey proposed stackforge/kolla: WIP:Replace config-external with a JSON file https://review.openstack.org/216874 | 03:18 |
sdake | but not atm working on keystone | 03:18 |
rhallisey | k I'm going to sleep | 03:18 |
SamYaple | night rhallisey | 03:18 |
sdake | enjoy | 03:18 |
rhallisey | night guys | 03:18 |
*** rhallisey has quit IRC | 03:19 | |
sdake | rhallisey's json wor coming along | 03:19 |
sdake | you should take a look | 03:19 |
sdake | need m0ar gates | 03:21 |
*** dims has quit IRC | 03:23 | |
*** loth has quit IRC | 03:24 | |
*** daneyon has joined #kolla | 03:27 | |
*** daneyon_ has quit IRC | 03:28 | |
*** apmelton has quit IRC | 03:29 | |
*** apmelton has joined #kolla | 03:32 | |
openstackgerrit | Steven Dake proposed stackforge/kolla: Make keystone work on centos+binary https://review.openstack.org/218075 | 03:34 |
*** alisonh has joined #kolla | 03:37 | |
*** ccrouch has quit IRC | 03:40 | |
*** achanda has quit IRC | 03:41 | |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove maridb client from base image https://review.openstack.org/218077 | 03:41 |
*** ccrouch has joined #kolla | 03:43 | |
*** sdake_ has joined #kolla | 03:44 | |
sdake_ | samyaple what do you make of this https://review.openstack.org/#/c/218075/ | 03:47 |
sdake_ | that works | 03:47 |
*** sdake has quit IRC | 03:48 | |
sdake_ | centos binary + templates works almost all the way except gents fail | 03:48 |
sdake_ | https://bugs.launchpad.net/kolla/+bug/1489664 | 03:49 |
openstack | Launchpad bug 1489664 in kolla "regression in neutron_agents in /docker" [Critical,Confirmed] | 03:49 |
sdake_ | one of your recent changes broke that ^^ | 03:49 |
sdake_ | it appears to happen in docker_templates as well | 03:49 |
*** sdake has joined #kolla | 03:57 | |
*** sdake_ has quit IRC | 04:01 | |
*** unicell has joined #kolla | 04:04 | |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks to scripts in memcached https://review.openstack.org/218027 | 04:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for magnum to docker_templates https://review.openstack.org/218026 | 04:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks in magnum docer_templates https://review.openstack.org/218025 | 04:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for memcached to docker_templates directory https://review.openstack.org/218028 | 04:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove kolla_ansible symlinks https://review.openstack.org/218019 | 04:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move keystone scripts to docker_templates dir https://review.openstack.org/218018 | 04:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks to scripts for keystone https://review.openstack.org/218017 | 04:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move Keepalived scripts to docker_tmeplates dir https://review.openstack.org/218016 | 04:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move kolla-ansible scripts to docker_templates https://review.openstack.org/218020 | 04:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for neutron to docker_templates https://review.openstack.org/218042 | 04:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for murano to docker_templates directory https://review.openstack.org/218040 | 04:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: remove neutron symlinks in docker_templates https://review.openstack.org/218041 | 04:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Start of removal of docker directory https://review.openstack.org/217981 | 04:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks for murano in docker_templates https://review.openstack.org/218039 | 04:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove docker directory https://review.openstack.org/218061 | 04:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move config-external and start for cinder https://review.openstack.org/217995 | 04:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove config-external and start for cinder https://review.openstack.org/217994 | 04:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for zaqar to docker_templates dir https://review.openstack.org/218057 | 04:04 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks to scripts for keepalived https://review.openstack.org/218015 | 04:05 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks for glance https://review.openstack.org/218001 | 04:05 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for glance https://review.openstack.org/218002 | 04:05 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove scripts for gnocchi https://review.openstack.org/218003 | 04:05 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for gnocchi https://review.openstack.org/218004 | 04:05 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove symlinks for haproxy scripts https://review.openstack.org/218005 | 04:05 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for haproxy https://review.openstack.org/218006 | 04:05 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove docker directory https://review.openstack.org/218061 | 04:09 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for zaqar to docker_templates https://review.openstack.org/218057 | 04:09 |
sdake | cool dleete and rename works | 04:10 |
SamYaple | looking now | 04:14 |
*** sdake_ has joined #kolla | 04:14 | |
SamYaple | sdake: you know that file gets overridden by config-external.sh right? | 04:15 |
SamYaple | that patch doesnt fix anything | 04:15 |
SamYaple | my guess is wwsgi-keystone.conf isn't put in the right place for centos or something | 04:16 |
SamYaple | sdake i closed your bug | 04:16 |
SamYaple | i left a note on it | 04:17 |
*** sdake has quit IRC | 04:18 | |
sdake_ | closed which bug | 04:20 |
sdake_ | samyaple keystone works with that patch does not work without it | 04:21 |
sdake_ | there are 100 bugs in the launchpad tracker could you be more specific | 04:21 |
*** sdake_ is now known as sdake | 04:22 | |
SamYaple | https://bugs.launchpad.net/kolla/+bug/1489664 | 04:23 |
openstack | Launchpad bug 1489664 in kolla "regression in neutron_agents in /docker" [Critical,Invalid] | 04:23 |
SamYaple | the only one you posted in the past little while | 04:24 |
SamYaple | oh nvm | 04:24 |
SamYaple | thats valid | 04:24 |
SamYaple | appologies | 04:24 |
SamYaple | im a bit off :) | 04:24 |
SamYaple | ill patch asap! | 04:25 |
*** sdake_ has joined #kolla | 04:25 | |
sdake_ | samyaple when I put a patch up without a bug # attached to it it is not pointless it is to get other people to have a look at it | 04:26 |
sdake_ | so I can share it | 04:26 |
sdake_ | clealry if I wanted it merged i'd put a bug # on it | 04:26 |
SamYaple | i have no idea what youre talking about | 04:26 |
sdake_ | re the config strategy | 04:27 |
sdake_ | i am not oerrinding it | 04:27 |
sdake_ | [sdake@bigiron kolla]$ grep config_strategy globals.yml | 04:29 |
sdake_ | #config_strategy: "COPY_ONCE" | 04:29 |
*** sdake has quit IRC | 04:29 | |
sdake_ | i of course copied a fresh etc dir with a new tree checkout | 04:29 |
sdake_ | and built with --no-cache | 04:29 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Fix neutron-agents regression https://review.openstack.org/218084 | 04:30 |
SamYaple | sdake_: yea i already reopened it | 04:30 |
SamYaple | i read it to quickly | 04:30 |
sdake_ | cool | 04:30 |
SamYaple | i appologized | 04:30 |
sdake_ | its ok | 04:31 |
sdake_ | dont sweat it | 04:31 |
SamYaple | im sweating bullets here man! | 04:31 |
sdake_ | which bullt | 04:31 |
sdake_ | bullets | 04:31 |
sdake_ | thats my line ;) | 04:32 |
*** weiyu has quit IRC | 04:32 | |
*** chandankumar has joined #kolla | 04:33 | |
sdake_ | say samyaple the launchpad blueprint tracker up to date? | 04:33 |
SamYaple | sdake_: i havent checked since this morning and i had a few closing blueprint patches left | 04:34 |
SamYaple | will verify | 04:34 |
sdake_ | tia | 04:34 |
SamYaple | tie? | 04:34 |
SamYaple | tia*? | 04:34 |
sdake_ | thanks in advancce | 04:34 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove docker directory https://review.openstack.org/218061 | 04:53 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for magnum to docker_templates https://review.openstack.org/218026 | 04:53 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for cinder to docker_templates https://review.openstack.org/217995 | 04:53 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for zaqar to docker_templates https://review.openstack.org/218057 | 04:53 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for designate to docker_templates https://review.openstack.org/217997 | 04:53 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for galera to docker_templates https://review.openstack.org/217999 | 04:53 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for memcached to docker_templates https://review.openstack.org/218028 | 04:53 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for rabbitmq to docker_templates https://review.openstack.org/218053 | 04:53 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move kolla-common to bsae directory https://review.openstack.org/217985 | 04:53 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for keystone to docker_templates https://review.openstack.org/218018 | 04:53 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for swift to docker_templates https://review.openstack.org/218055 | 04:53 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for keepalive to docker_templates https://review.openstack.org/218016 | 04:53 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for nova to docker_templates https://review.openstack.org/218049 | 04:53 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for ceilometer to docker_templates https://review.openstack.org/217988 | 04:53 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for openvswitch to docker_templates https://review.openstack.org/218051 | 04:53 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for kolla-ansible to docker_templates https://review.openstack.org/218020 | 04:53 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for neutron to docker_templates https://review.openstack.org/218042 | 04:53 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for heat to docker_templates https://review.openstack.org/218009 | 04:53 |
Kennan | hi sdake_ SamYaple: | 05:18 |
Kennan | did we have some changes for build image process? | 05:18 |
Kennan | from build from source | 05:18 |
Kennan | do we need other configuration ? | 05:18 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove maridb client from base image https://review.openstack.org/218077 | 05:20 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Make keystone work on centos+binary https://review.openstack.org/218075 | 05:20 |
sdake_ | no | 05:20 |
Kennan | I found build image failed for cp: cannot copy a directory, 'etc/neutron', into itself, '/etc/neutron/neutron' | 05:21 |
sdake_ | which distro are you buding | 05:21 |
Kennan | still run on ubuntu, build from source for ubuntu | 05:22 |
Kennan | tools/build.py --base ubuntu --type source --template -T 1 neutron-base | 05:22 |
SamYaple | Kennan: on a fresh rebuild i also see that | 05:22 |
SamYaple | moemnt | 05:22 |
SamYaple | ah i know the issue, moment | 05:23 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove maridb client from base image https://review.openstack.org/218077 | 05:24 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Make keystone work on centos+binary https://review.openstack.org/218075 | 05:24 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Fixes issue in neutron base after recent change https://review.openstack.org/218096 | 05:25 |
SamYaple | Kennan: try that patch, apologies | 05:26 |
SamYaple | i didnt test my last one before i went to sleep and it we approved before checking the gate output | 05:26 |
sdake_ | gate wouldn't catch that anyway | 05:27 |
sdake_ | we dont gate on templated imgaes atm | 05:27 |
Kennan | let me try | 05:27 |
SamYaple | sdake_ oh i thought we did | 05:27 |
SamYaple | but this explains alot | 05:28 |
SamYaple | i thought the centos ones were templates | 05:28 |
sdake_ | nothing gates on templates | 05:28 |
sdake_ | patche i infra queue | 05:28 |
SamYaple | yea that one is on me Kennan, i tested all teh other patches, but the neutron one i submitted and went to sleep without testing | 05:28 |
Kennan | hi, SamYaple I am not sure what's the difference between templates and wo tempaltes? | 05:30 |
openstackgerrit | Merged stackforge/kolla: Fix neutron-agents regression https://review.openstack.org/218084 | 05:31 |
sdake_ | --templates builds from docker_templates directory | 05:31 |
sdake_ | without --templates you bulid from docker directory | 05:31 |
sdake_ | docker directory going away in next few days | 05:31 |
Kennan | sdake_ from kolla readme, I got that docker would be replaced by docker_tempaltes | 05:31 |
Kennan | right? | 05:31 |
sdake_ | roger | 05:31 |
Kennan | so why we still need docker dir? | 05:32 |
sdake_ | we dont - there is 24 patchees in the patch queue to remove it | 05:32 |
sdake_ | waiting on the gate to test em all | 05:32 |
*** weiyu has joined #kolla | 05:37 | |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove docker directory https://review.openstack.org/218061 | 05:39 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for magnum to docker_templates https://review.openstack.org/218026 | 05:39 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for cinder to docker_templates https://review.openstack.org/217995 | 05:39 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for zaqar to docker_templates https://review.openstack.org/218057 | 05:39 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for designate to docker_templates https://review.openstack.org/217997 | 05:39 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for galera to docker_templates https://review.openstack.org/217999 | 05:39 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for memcached to docker_templates https://review.openstack.org/218028 | 05:39 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for rabbitmq to docker_templates https://review.openstack.org/218053 | 05:39 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move kolla-common to bsae directory https://review.openstack.org/217985 | 05:39 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for keystone to docker_templates https://review.openstack.org/218018 | 05:39 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for swift to docker_templates https://review.openstack.org/218055 | 05:39 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for keepalive to docker_templates https://review.openstack.org/218016 | 05:39 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for nova to docker_templates https://review.openstack.org/218049 | 05:39 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for ceilometer to docker_templates https://review.openstack.org/217988 | 05:39 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for openvswitch to docker_templates https://review.openstack.org/218051 | 05:39 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for kolla-ansible to docker_templates https://review.openstack.org/218020 | 05:39 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for neutron to docker_templates https://review.openstack.org/218042 | 05:39 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for heat to docker_templates https://review.openstack.org/218009 | 05:39 |
sdake_ | samyaple dont review those until i remove the -2 from the head patch | 05:39 |
SamYaple | sdake_: it was mostly to get them in my review queue ;) | 05:43 |
SamYaple | quicker than searching for my name | 05:43 |
SamYaple | plus i actually review them, looks good | 05:43 |
sdake_ | cool thanks | 05:43 |
sdake_ | i'm waiting for them all to pass the various gates | 05:43 |
*** chandankumar has quit IRC | 05:47 | |
sdake_ | well i sure hope all that rework was worth the effort | 05:52 |
sdake_ | wow what a pain in the ass all that wa | 05:52 |
SamYaple | it already has been worth it with teh templates | 05:55 |
SamYaple | much easier to follow | 05:55 |
sdake_ | well the symlink thing was not gong to work | 06:02 |
Kennan | SamYaple: sdake_ so templates would the official way. right? | 06:02 |
sdake_ | kennan yes - docker dir is going bye as soon as the patches pass the gating | 06:02 |
sdake_ | kennan - the --template option likely to be removed as well | 06:03 |
Kennan | ok sdake_ one another question, for build, I found we have lots of **.tar, like keystone.tar. nova.tar etc | 06:03 |
*** sdake_ is now known as sdake | 06:03 | |
Kennan | who packaged that tar ? I did not understand it | 06:03 |
sdake | ye that is the build from source tarballs | 06:03 |
sdake | it comes from master | 06:04 |
sdake | from tarballs.opentack.org | 06:04 |
sdake | look at build.ini for the source of the tarballs | 06:04 |
Kennan | ok. sdake: which code did that? I noticed build.ini | 06:04 |
Kennan | not sure which code did that | 06:04 |
sdake | kolla/cmd/build.py | 06:04 |
Kennan | ok. thanks guy. would check it more | 06:06 |
*** weiyu has quit IRC | 06:06 | |
sdake | can you fix your sphinx support real quick befoe heading out kennan? | 06:07 |
Kennan | sure sdake: fixing it now | 06:07 |
openstackgerrit | Kai Qiang Wu(Kennan) proposed stackforge/kolla: Add developer docs automatic produce support https://review.openstack.org/218046 | 06:11 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Use external address in openrc-example https://review.openstack.org/218112 | 06:13 |
coolsvap | sdake, hi, are you waiting for the gates to be green for the remove-docker-dir patches? | 06:13 |
*** chandankumar has joined #kolla | 06:14 | |
sdake | your free to review them now if you want | 06:14 |
sdake | but ye waiting on gates to remove my -2 from the head patch | 06:14 |
sdake | i would prefer they merge as one block | 06:15 |
sdake | note gate-functional-f21 is going to always fail here | 06:15 |
sdake | i have a patch submitted to openstack-nfra to remove it entirely | 06:16 |
SamYaple | sdake: just fyi, the gate is failing without my neutron patch | 06:16 |
openstackgerrit | Merged stackforge/kolla: Fixes issue in neutron base after recent change https://review.openstack.org/218096 | 06:16 |
SamYaple | yea that one ;) | 06:17 |
SamYaple | sdake: you may want to rebase again and rekick off the gate | 06:17 |
sdake | ya i have to make a change anyway to fix the gate | 06:17 |
Kennan | SamYaple, that fix work | 06:25 |
Kennan | I verified that in my env | 06:25 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove docker directory https://review.openstack.org/218061 | 06:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for magnum to docker_templates https://review.openstack.org/218026 | 06:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for cinder to docker_templates https://review.openstack.org/217995 | 06:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for zaqar to docker_templates https://review.openstack.org/218057 | 06:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for designate to docker_templates https://review.openstack.org/217997 | 06:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for galera to docker_templates https://review.openstack.org/217999 | 06:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for memcached to docker_templates https://review.openstack.org/218028 | 06:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for rabbitmq to docker_templates https://review.openstack.org/218053 | 06:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move kolla-common to bsae directory https://review.openstack.org/217985 | 06:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for keystone to docker_templates https://review.openstack.org/218018 | 06:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for swift to docker_templates https://review.openstack.org/218055 | 06:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for keepalive to docker_templates https://review.openstack.org/218016 | 06:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for nova to docker_templates https://review.openstack.org/218049 | 06:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for ceilometer to docker_templates https://review.openstack.org/217988 | 06:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for openvswitch to docker_templates https://review.openstack.org/218051 | 06:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for kolla-ansible to docker_templates https://review.openstack.org/218020 | 06:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for neutron to docker_templates https://review.openstack.org/218042 | 06:27 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for heat to docker_templates https://review.openstack.org/218009 | 06:27 |
sdake | sweet we move to the big tent namespace on september 11th | 06:30 |
sdake | rather openstack namespace | 06:30 |
sdake | kennan are you able to boot a vm with nova? | 06:30 |
Kennan | yes, sdake: I tried that two days ago when the first time I build images successfully | 06:31 |
Kennan | I not try it today | 06:31 |
sdake | what is your take on the deployment process with kolla? | 06:31 |
sdake | besides the fact we have bugs ;) | 06:32 |
*** weiyu_ has joined #kolla | 06:33 | |
Kennan | hi sdake: I think it is interesting, so want to try it, just try it recently | 06:35 |
sdake | we have gone through a major transition in liberty 3 | 06:35 |
sdake | ansible + docker templates | 06:35 |
sdake | hopefully the lsat of the major transitions :) | 06:35 |
Kennan | ok. would dig more about it | 06:38 |
sdake | need a global requirements sync | 06:38 |
Kennan | yes | 06:38 |
sdake | cool let us know what you think | 06:38 |
sdake | sync job i mean | 06:38 |
Kennan | do we need sync requirements now? | 06:38 |
sdake | need all our requirements in global | 06:38 |
sdake | we can't setup a requirements sync job until our requierments are in global requirements | 06:39 |
Kennan | yes sdake: not sure if we decide do it now, as i noticed kolla many changes today | 06:39 |
Kennan | I could help synch the reuqirements jobs in infra if it is the right time | 06:39 |
Kennan | I need check with clarkb or lifeless about this | 06:40 |
sdake | our code base will likely slow down shortly on the major changes | 06:40 |
sdake | i know how to set it up - just a matter of getting our requirements in global requirements | 06:40 |
Kennan | yes. so it is good to sync it | 06:40 |
Kennan | what can I do for that? I also learn some in magnum before | 06:41 |
sdake | submit changes to requirements.txt in global requirements i guess | 06:41 |
sdake | I dont know the process for it | 06:41 |
sdake | been busy tyring to get our gateing rolling | 06:42 |
Kennan | ok sdake: I could do that and get it through if you not avialable | 06:42 |
sdake | kennan if you want to contribute there are all kinds of places we need help :) | 06:43 |
Kennan | ok sdake: let me try sync job now, if nobody do it now. what do you think ? | 06:43 |
sdake | i think it wont work because not all of our requierments are in global requirements | 06:44 |
Kennan | yes, that need to submit kolla changes in infra code | 06:44 |
sdake | right | 06:44 |
sdake | getting a docs publish job is super high prioirty | 06:44 |
sdake | if you nkow how to do that that would be fantastic :) | 06:44 |
Kennan | ok sdake: first is docs publibsh, and second is synch requriemetns job | 06:45 |
Kennan | right? | 06:45 |
Kennan | I could help do it | 06:45 |
sdake | cool that would be great | 06:46 |
sdake | sync reuquirements not so super high prioirty | 06:46 |
sdake | we have alot of other important things to work on | 06:46 |
sdake | but docs job would rock | 06:46 |
Kennan | ok sdake: let me get someone in infra if need more info. will update infor here | 06:46 |
Kennan | I think we also need a jenkins job for docs. anyway, let me try do it now | 06:47 |
sdake | need a tarball and pypi publishing job too | 06:55 |
sdake | these are things i can do | 06:55 |
sdake | or if you want t take them on you can ;) | 06:55 |
sdake | if your not familiar with project-config i can tacle em | 06:56 |
sdake | samyaple given that we just removed the docker dir, i removed tall the docker related gate submissions i had out there | 06:57 |
*** weiyu_ has quit IRC | 06:57 | |
*** britthouser has joined #kolla | 07:02 | |
*** 16WAANOXH has joined #kolla | 07:02 | |
openstackgerrit | Steven Dake proposed stackforge/kolla: Functional test Ubuntu source building https://review.openstack.org/218122 | 07:02 |
sdake | coolsvap samyaple please ack that so we can get the gate rolling with ubuntu source building | 07:03 |
*** achanda has joined #kolla | 07:05 | |
SamYaple | sdake: what docker related gate submissions? | 07:06 |
SamYaple | werent those for the templates? | 07:06 |
sdake | templates are gong to be the only gates | 07:06 |
sdake | no docker gates at all | 07:06 |
sdake | i abandoned the chnge with the docker vs template optoin in openstack-infra's queue | 07:06 |
sdake | this means i need a job named as is in that tox.ini file | 07:07 |
sdake | we will have 3 jobs instead of 6 | 07:07 |
sdake | ubuntu-source, centos-binary centos-source | 07:08 |
SamYaple | yea ok | 07:08 |
SamYaple | i trust you know hat youre doign there so no complaints from me | 07:08 |
sdake | i'm on it | 07:08 |
sdake | i actuallly know project-config pretty well now after beating my head against it for 3 projects ;) | 07:09 |
sdake | i had thought those template job s would merge faster then they hvae | 07:09 |
sdake | infra taking awhile to +a em :) | 07:09 |
SamYaple | we just got the gate change on our side merged 2 days ago | 07:11 |
SamYaple | thats why i beleieve | 07:11 |
coolsvap | SamYaple, sdake i think the docker_templates patches are gtg | 07:12 |
coolsvap | https://review.openstack.org/#/c/217985/ just got Green | 07:13 |
*** inc0 has joined #kolla | 07:13 | |
sdake | nice well i'm going to bed | 07:14 |
sdake | feel free to get em merged ;-) | 07:14 |
sdake | night all | 07:14 |
coolsvap | night sdake | 07:15 |
sdake | probably going to have to rebse after that meres | 07:16 |
SamYaple | sdake: https://review.openstack.org/#/c/218055/ | 07:17 |
SamYaple | do you want me to fix that? | 07:17 |
SamYaple | all the other patches are goo | 07:17 |
SamYaple | good* | 07:17 |
SamYaple | that is the the only patch with issues | 07:17 |
sdake | i'll fix real quick | 07:17 |
SamYaple | awesome! | 07:17 |
SamYaple | when you wake it will be to a brand new gate (hopefully) | 07:17 |
sdake | wonder how that happened | 07:18 |
SamYaple | you moved .s instead of the real file it looks like | 07:20 |
SamYaple | no idea | 07:20 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove docker directory https://review.openstack.org/218061 | 07:20 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for zaqar to docker_templates https://review.openstack.org/218057 | 07:20 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for swift to docker_templates https://review.openstack.org/218055 | 07:20 |
SamYaple | coolsvap mandre can i get a review on https://review.openstack.org/#/c/217981/ ? | 07:22 |
SamYaple | its the start of sdake patchsets | 07:22 |
SamYaple | ill remove the -2 and add a +1 once all the downstream patches are good for the gate | 07:22 |
sdake | i suspect itwill collide | 07:23 |
SamYaple | sdake: how so? | 07:23 |
sdake | the patch that just got submitted for the gate | 07:23 |
sdake | edits tox.ini | 07:23 |
sdake | one of my patches - i think tha tfirst one edits tox.ini | 07:23 |
sdake | so i'd be gating agiaintt the templates | 07:23 |
SamYaple | ok | 07:24 |
SamYaple | go to bed man | 07:24 |
SamYaple | well figure it out | 07:24 |
SamYaple | our team is stellar! | 07:24 |
sdake | enjoy | 07:24 |
SamYaple | thanks sdake | 07:24 |
mandre | night sdake | 07:24 |
inc0 | good mornign | 07:25 |
SamYaple | sdake: do you might if i rebase it then? | 07:25 |
sdake | i am up for a little bit | 07:25 |
inc0 | a non-stellar part of our team just apeared, rejoyce | 07:25 |
SamYaple | sdake: so youll do it? | 07:25 |
coolsvap | all green :) | 07:25 |
sdake | i'll likely rebase as soon as that patch hits the repo | 07:25 |
mandre | SamYaple, lets have +a for all patches before unblocking the stream of patch, no? | 07:25 |
SamYaple | inc0: mandre is great! what do you mean? | 07:26 |
inc0 | I was talking about mysel | 07:26 |
inc0 | f | 07:26 |
inc0 | ;) | 07:26 |
SamYaple | mandre: yea i have a -2 on the first patch in that stream | 07:26 |
SamYaple | mandre: im waiting for the gate for the final three | 07:26 |
SamYaple | i wont unblock it until all is good | 07:26 |
SamYaple | inc0: i know :[] | 07:26 |
mandre | haha, let's get it rolling | 07:27 |
SamYaple | sdake: just to clarify, you are going to rebase on your other tox.ini patch before you go to sleep? | 07:32 |
SamYaple | this affects the heat patch im about to submit | 07:33 |
sdake | i tried that gate wont take a git review | 07:33 |
sdake | so i have to wait for it to hit the repo | 07:33 |
sdake | then git pull --rebase | 07:33 |
sdake | then git reiew | 07:33 |
sdake | gate is 30 mins in - on magnum in src building | 07:33 |
sdake | but i think it will take 1 hour for the functional gate to timeout | 07:34 |
sdake | or do its thing | 07:34 |
sdake | whatevere thta may be | 07:34 |
sdake | i'll stay up for another 30 mins | 07:34 |
sdake | wathcing tv | 07:34 |
sdake | only midnight here | 07:34 |
SamYaple | sdake: you cant rebase the whole stream on a new base? | 07:36 |
SamYaple | thats wierd | 07:36 |
sdake | ya git review rjects it | 07:36 |
sdake | says there is no chane | 07:36 |
SamYaple | put a small comment ;) | 07:36 |
SamYaple | change the commit msg | 07:36 |
SamYaple | all kinds of ways :D | 07:36 |
sdake | just tried that no beuno | 07:39 |
*** jmccarthy has quit IRC | 07:39 | |
sdake | gate is on swift atm | 07:39 |
*** jmccarthy has joined #kolla | 07:40 | |
*** 16WAANOXH has quit IRC | 07:41 | |
*** weiyu_ has joined #kolla | 07:41 | |
*** weiyu_ has quit IRC | 07:44 | |
*** mandre has quit IRC | 07:45 | |
sdake | gate is done | 07:47 |
Kennan | sdake: check this https://review.openstack.org/#/c/218137/ | 07:48 |
Kennan | I would do requirements jobs now if kolla is ready | 07:48 |
sdake | i just saw it | 07:48 |
sdake | nice job | 07:48 |
sdake | getting ready to +1 | 07:48 |
*** weiyu has joined #kolla | 07:48 | |
SamYaple | alrighty sdake patch is merging now | 07:49 |
sdake | yup | 07:49 |
sdake | jobs just finished | 07:50 |
sdake | now running gate obs i suspect | 07:50 |
openstackgerrit | Merged stackforge/kolla: Functional test Ubuntu source building https://review.openstack.org/218122 | 07:50 |
sdake | those take under 5 minutes | 07:50 |
SamYaple | i win! | 07:50 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Remove docker directory https://review.openstack.org/218061 | 07:50 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for magnum to docker_templates https://review.openstack.org/218026 | 07:50 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for cinder to docker_templates https://review.openstack.org/217995 | 07:50 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for zaqar to docker_templates https://review.openstack.org/218057 | 07:50 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for designate to docker_templates https://review.openstack.org/217997 | 07:50 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for galera to docker_templates https://review.openstack.org/217999 | 07:50 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for memcached to docker_templates https://review.openstack.org/218028 | 07:50 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for rabbitmq to docker_templates https://review.openstack.org/218053 | 07:50 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move kolla-common to bsae directory https://review.openstack.org/217985 | 07:50 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for keystone to docker_templates https://review.openstack.org/218018 | 07:50 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for swift to docker_templates https://review.openstack.org/218055 | 07:50 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for keepalive to docker_templates https://review.openstack.org/218016 | 07:50 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for nova to docker_templates https://review.openstack.org/218049 | 07:50 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for ceilometer to docker_templates https://review.openstack.org/217988 | 07:50 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for openvswitch to docker_templates https://review.openstack.org/218051 | 07:50 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for kolla-ansible to docker_templates https://review.openstack.org/218020 | 07:50 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for neutron to docker_templates https://review.openstack.org/218042 | 07:50 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Move scripts for heat to docker_templates https://review.openstack.org/218009 | 07:51 |
inc0 | more reviews! this will be a good Friday! | 07:51 |
sdake | recommend not +a'ing anything until that straem merge | 07:52 |
sdake | s | 07:52 |
sdake | as to avoid further rebases | 07:52 |
sdake | kennan if you want to sort out a tarball pbuhslihing job that would be grand too :) | 07:53 |
Kennan | I notcied tarball job in zuul,and project, but not sure if kolla ready for that? | 07:54 |
sdake | redy for what? | 07:54 |
sdake | tarball publshing? | 07:54 |
SamYaple | Kennan: i mean we are going to need that in the next 2 months | 07:54 |
sdake | yes I'd like master tarballs per comit | 07:54 |
SamYaple | and we just started versioning | 07:54 |
sdake | and tarballs on tags | 07:55 |
Kennan | OK. I can add that in another patch | 07:55 |
Kennan | in infra project | 07:55 |
*** achanda has quit IRC | 07:56 | |
sdake | i have two pc patches in the queue as well | 07:56 |
sdake | hopefully one of us doesn't end up having to rebase | 07:56 |
sdake | btw, after the 5th i'd like to lock down project config changes for kolla | 07:56 |
sdake | so the rename happens properly | 07:57 |
sdake | and reopen after the 11th | 07:57 |
sdake | every time projec tconfig changes I have to rebase the rename patch | 07:57 |
sdake | and its entirely possile i could screw it up :) | 07:57 |
inc0 | sdake, rename? | 07:58 |
inc0 | you mean stackforge -> openstack? | 07:58 |
sdake | ha to openstack/kolla | 07:58 |
sdake | ya instead of ha | 07:58 |
inc0 | cool! | 07:58 |
sdake | hey since there appears to be a bunch of reviewers about | 07:59 |
sdake | please have a look at https://review.openstack.org/#/c/216874/ | 07:59 |
sdake | i've been hammering on the reviews with rhallisey the last couple of days | 07:59 |
sdake | and I htink its finally beaten into submission | 07:59 |
sdake | this is json encoding of files to copy | 07:59 |
sdake | there is a bit of python code | 07:59 |
sdake | this is an example of how it is used https://review.openstack.org/#/c/217880/ | 08:00 |
SamYaple | sdake: heat patch is good to go! | 08:00 |
SamYaple | tested and working for ansible | 08:00 |
SamYaple | using openstack client | 08:00 |
sdake | what was the problem with stack delete? | 08:00 |
SamYaple | the good client | 08:00 |
SamYaple | oh no thats still busted | 08:00 |
sdake | oh right | 08:00 |
SamYaple | but the ansible stuff is all done right | 08:00 |
sdake | cool | 08:01 |
SamYaple | not with heat-stack-domain-create | 08:01 |
sdake | i'll look at fixing keystone and horizon properly tomorrow | 08:01 |
sdake | for centos binary | 08:01 |
inc0 | sdake, every container has python inside? | 08:01 |
sdake | inc0 yes | 08:01 |
SamYaple | we install it in the base as a common dep | 08:01 |
SamYaple | but if we have an openstack-base we wouldnt need to for every container | 08:02 |
inc0 | with this review sdake linked, we will | 08:02 |
sdake | i think writing json decoding in shell would be pretty painful :) | 08:02 |
SamYaple | that being said we would need kolla-ansible on every single node, and that has an python req | 08:02 |
SamYaple | inc0: not if that uses python3 | 08:02 |
SamYaple | thats defualt in all contaienrs | 08:02 |
SamYaple | but python2 is needed for ansible, so eh | 08:03 |
inc0 | python2 is needed for lots of things | 08:03 |
sdake | anyway please have a review sincce it drastically affects how kolla containers operate | 08:03 |
sdake | it removes kolla-common and config-extrnal | 08:03 |
sdake | which is sweet | 08:03 |
inc0 | hence my asking | 08:03 |
sdake | and keeps start.sh fairly much the same | 08:03 |
SamYaple | we shouldn't really remove kolla-common.sh | 08:04 |
SamYaple | ill bet money well need it later | 08:04 |
sdake | well we dont have to | 08:04 |
SamYaple | yea i know what you mean though | 08:04 |
sdake | i think your correct samyaple | 08:04 |
inc0 | but I think we should remove it someday | 08:04 |
inc0 | maintaining bash is painful | 08:04 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Add Ansible support for Heat https://review.openstack.org/205528 | 08:04 |
SamYaple | patch tested and working | 08:05 |
sdake | i do not want to run the main binary from a python interpreator | 08:05 |
SamYaple | (heat still has bug with delete) | 08:05 |
SamYaple | yea no | 08:05 |
inc0 | but we can do it from Dockerfile | 08:05 |
sdake | but using python in start.sh is ok from my prespective | 08:05 |
SamYaple | agreed sdake | 08:05 |
inc0 | or keep start.sh pretty much frozen and all logic will be in python | 08:06 |
sdake | ryan's first work was running the binary from python ;-) | 08:06 |
inc0 | or at least great most of it | 08:06 |
sdake | that is what ryan's patch does, it puts all the arg generation in python code | 08:06 |
sdake | and arg copying | 08:07 |
sdake | and config strategy determination | 08:07 |
SamYaple | that should help decouple the containers | 08:07 |
SamYaple | allowing them to update independant of the ansible code | 08:07 |
SamYaple | which would be great | 08:07 |
sdake | atm the json is hard coded on the ansible side | 08:07 |
sdake | i am good with that for now | 08:07 |
sdake | we will need autogeneration later - hopefully in rc1 | 08:07 |
SamYaple | will do | 08:08 |
sdake | ;) | 08:08 |
sdake | but lets make sure this reference implementation is as prefect as we can get it | 08:08 |
SamYaple | yea we will be frozen earth soon | 08:08 |
SamYaple | lets get stable | 08:08 |
*** sdake_ has joined #kolla | 08:14 | |
Kennan | also requirements job: https://review.openstack.org/#/c/218144/ it need evaluated by them if license or lib is ok | 08:16 |
*** sdake has quit IRC | 08:18 | |
sdake_ | ok ptfoing | 08:19 |
sdake_ | night all | 08:19 |
sdake_ | see you in t-8 hours | 08:19 |
*** weiyu has quit IRC | 08:22 | |
Kennan | bye | 08:22 |
*** weiyu_ has joined #kolla | 08:24 | |
*** britthouser has quit IRC | 08:32 | |
*** inc0 has quit IRC | 08:39 | |
*** inc0 has joined #kolla | 08:42 | |
inc0 | guys, I have an idea | 08:55 |
inc0 | how about create "consistency checker" container, and deploy it after each service? | 08:55 |
inc0 | and for each service, there will be requirement to provide such docker | 08:56 |
inc0 | so ansible will for example deploy keystone, and consistency check will try to list users, log in and such | 08:56 |
inc0 | after adding glance, it will add cirros and such | 08:56 |
inc0 | this way we'll have functional tests that everyone could consume | 08:57 |
inc0 | and it will also work as gate | 08:57 |
inc0 | SamYaple, ^ we talked yesterday about something like that...thought? | 08:57 |
inc0 | s | 08:57 |
vbel | I would like everyone concentrate on CI and adding tests to the whole thing. Otherwise we will get very frustrated soon :) There are more and more developers and testers around and a lot of changes | 08:58 |
inc0 | vbel, yeah, but we need functional tests | 08:59 |
inc0 | of whole openstack | 08:59 |
inc0 | that's main problem | 08:59 |
inc0 | and this approach would also allow us to test gradually | 09:00 |
inc0 | one service at time | 09:00 |
vbel | tempest is for functional tests of whole openstack. we need a sanity. | 09:00 |
inc0 | tempest doesn't really do functional tests per se | 09:00 |
vbel | like running very basic things | 09:00 |
inc0 | it tests nova, it tests glance...it doesn't test openstack | 09:00 |
inc0 | I think we speak about same thing, but call it different names | 09:00 |
vbel | I mean we need sanity - to disallow commits that break things. Yes | 09:01 |
SamYaple | inc0: hold on catching up | 09:04 |
inc0 | SamYaple, essence is to have per-service consistency test in separate container. I think this itself would be good functionality of kolla as whole | 09:09 |
inc0 | consumable itself for testing existing deployments | 09:09 |
inc0 | or sanity tests:) vbel actually that's better name for it | 09:10 |
vbel | :) | 09:11 |
vbel | self-diagnostics | 09:11 |
inc0 | yup, that's what it is really | 09:11 |
inc0 | and gating on this would be easy - just try to deploy this container and look if it fails | 09:12 |
jmccarthy | Sounds like smoke testing i.e. check it doesn't go up in smoke when you plug it in ;) | 09:15 |
inc0 | a little more than that | 09:16 |
inc0 | smoke tests is build script really | 09:16 |
inc0 | maybe running it | 09:16 |
inc0 | but not doing anything with it | 09:16 |
SamYaple | so we originally had checks from within a container, we removed those | 09:16 |
SamYaple | youre tlaking about a monitoring system | 09:17 |
SamYaple | i dont think that is right for Kolla at this time | 09:17 |
SamYaple | ideally i would like to consume someone elses monitoring | 09:17 |
inc0 | SamYaple, not really monitoring | 09:18 |
inc0 | I don't want to monitor it, constantly | 09:18 |
inc0 | I want to spool up new container, do few tasks to see if deployment is going well so far | 09:18 |
inc0 | and kill container | 09:18 |
inc0 | like bootstrap container, but for consistency test | 09:18 |
inc0 | and each step will be tested independently | 09:19 |
SamYaple | we do that though | 09:19 |
SamYaple | the register test does that | 09:19 |
SamYaple | we arent going to be uploading images to glance, thats for sure | 09:19 |
SamYaple | but we can do some additional gets against the api for checking as we go | 09:20 |
SamYaple | read-only operations only | 09:20 |
inc0 | in register we check if keystone works | 09:20 |
SamYaple | i know i said additional | 09:20 |
inc0 | I think add img -> delete img is ok (as long as it cleans after itself) but that's something to talk about | 09:21 |
inc0 | what do you think about general idea of containers like that? | 09:21 |
inc0 | aslo, fuel for example add some basic stuff after deploy, and I for one think it's cools | 09:22 |
inc0 | cool | 09:22 |
inc0 | like create ext network with floatings, int network for vms and router in between | 09:22 |
inc0 | if someone doesn't like that, there is always neutron net-delete and disable our checks | 09:23 |
inc0 | (since it'll be different taks, there is no obligation to run it) | 09:23 |
inc0 | but will give you ready to use OS at the end | 09:23 |
inc0 | just click boot | 09:24 |
inc0 | you know what? I'll write this bp, let's start discussion | 09:24 |
inc0 | https://blueprints.launchpad.net/kolla/+spec/sanity-check-container - bash it and abuse it, but think about it plz | 09:29 |
SamYaple | coolsvap: https://review.openstack.org/#/c/217981/ | 09:31 |
SamYaple | can you ack this | 09:31 |
SamYaple | thanks coolsvap its all merging | 09:33 |
SamYaple | fingers crossed | 09:33 |
coolsvap | SamYaple, np, I will also keep a watch | 09:34 |
SamYaple | inc0: sure thats no problem to talk about, but keep in mind that we should never be adding/deleting things from the environemnt | 09:34 |
SamYaple | thats the operators job | 09:34 |
SamYaple | if yo uwant a check script in the tools folderi m sure that can be aranged | 09:35 |
SamYaple | i dont think anyone will fight that | 09:35 |
SamYaple | but it wont be inline in the playbooks | 09:35 |
inc0 | it will be additional task in ansible | 09:35 |
openstackgerrit | Merged stackforge/kolla: Start of removal of docker directory https://review.openstack.org/217981 | 09:35 |
openstackgerrit | Merged stackforge/kolla: Move kolla-common to bsae directory https://review.openstack.org/217985 | 09:35 |
SamYaple | a api check (nova list, neutron net-list) are ok, it will not be a create delete task | 09:35 |
inc0 | completely optional | 09:35 |
SamYaple | that isnt going to happen | 09:35 |
openstackgerrit | Merged stackforge/kolla: Move scripts for ceilometer to docker_templates https://review.openstack.org/217988 | 09:35 |
SamYaple | we do not delete things from the environment | 09:36 |
openstackgerrit | Merged stackforge/kolla: Move scripts for cinder to docker_templates https://review.openstack.org/217995 | 09:36 |
SamYaple | thats the ops job | 09:36 |
inc0 | SamYaple, one request, keep an open mind about project plz | 09:36 |
openstackgerrit | Merged stackforge/kolla: Move scripts for designate to docker_templates https://review.openstack.org/217997 | 09:36 |
openstackgerrit | Merged stackforge/kolla: Move scripts for galera to docker_templates https://review.openstack.org/217999 | 09:36 |
openstackgerrit | Merged stackforge/kolla: Move scripts for glance to docker_templates https://review.openstack.org/218002 | 09:36 |
SamYaple | inc0: sure, open mind but the reasons for not deleting have already been discussed | 09:36 |
inc0 | killing ideas before even talking about it isn't good for project | 09:37 |
SamYaple | we can discuss them again | 09:37 |
SamYaple | we have discussed them before | 09:37 |
SamYaple | and we will in the future im sure | 09:37 |
inc0 | I don't remember it, so maybe, and I'm not saying it's necessary good idea | 09:37 |
SamYaple | inc0: this isnt an original idea, we already removed those checks from the code base, you are proposing things we have removed | 09:37 |
inc0 | thing is, if we have it in completely separate container, that will mean its 100% optional | 09:38 |
SamYaple | but it requires tasks in ansible | 09:38 |
SamYaple | so additional skipped tasks for 99% of people | 09:38 |
openstackgerrit | Merged stackforge/kolla: Move scripts for gnocchi to docker_templates https://review.openstack.org/218004 | 09:38 |
openstackgerrit | Merged stackforge/kolla: Move scripts for haproxy to docker_templates https://review.openstack.org/218006 | 09:38 |
openstackgerrit | Merged stackforge/kolla: Move scripts for heat to docker_templates https://review.openstack.org/218009 | 09:38 |
inc0 | I don't think it'll be 99% tho | 09:39 |
SamYaple | you want a seperate playbook that does those things, thats a good idea | 09:39 |
openstackgerrit | Merged stackforge/kolla: Move scripts for horizon to docker_templates https://review.openstack.org/218011 | 09:39 |
SamYaple | running them inline isnt | 09:39 |
SamYaple | that means they would have to be executed ever single playbook run, or skipped | 09:39 |
SamYaple | both are no good | 09:39 |
openstackgerrit | Merged stackforge/kolla: Move scripts for keepalive to docker_templates https://review.openstack.org/218016 | 09:39 |
inc0 | I don't want separate playbook, because I don't want to have one big ass test after everything is done | 09:39 |
openstackgerrit | Merged stackforge/kolla: Move scripts for keystone to docker_templates https://review.openstack.org/218018 | 09:39 |
openstackgerrit | Merged stackforge/kolla: Move scripts for kolla-ansible to docker_templates https://review.openstack.org/218020 | 09:39 |
inc0 | I think gradual testing each step is much better | 09:39 |
openstackgerrit | Merged stackforge/kolla: Move scripts for magnum to docker_templates https://review.openstack.org/218026 | 09:39 |
SamYaple | ok inc0 well it wont be inline | 09:39 |
openstackgerrit | Merged stackforge/kolla: Move scripts for memcached to docker_templates https://review.openstack.org/218028 | 09:40 |
SamYaple | a seperate playbook was always the plan | 09:40 |
openstackgerrit | Merged stackforge/kolla: Move scripts for murano to docker_templates https://review.openstack.org/218040 | 09:40 |
inc0 | tell me why? | 09:40 |
SamYaple | i just did | 09:40 |
openstackgerrit | Merged stackforge/kolla: Move scripts for neutron to docker_templates https://review.openstack.org/218042 | 09:40 |
SamYaple | ugh | 09:40 |
openstackgerrit | Merged stackforge/kolla: Move scripts for nova to docker_templates https://review.openstack.org/218049 | 09:40 |
SamYaple | ill be back ones these merge | 09:40 |
SamYaple | we can talk then | 09:40 |
openstackgerrit | Merged stackforge/kolla: Move scripts for openvswitch to docker_templates https://review.openstack.org/218051 | 09:40 |
openstackgerrit | Merged stackforge/kolla: Move scripts for rabbitmq to docker_templates https://review.openstack.org/218053 | 09:40 |
inc0 | I'm going to cluster of meetings, so later on | 09:40 |
openstackgerrit | Merged stackforge/kolla: Move scripts for swift to docker_templates https://review.openstack.org/218055 | 09:40 |
openstackgerrit | Merged stackforge/kolla: Move scripts for zaqar to docker_templates https://review.openstack.org/218057 | 09:40 |
openstackgerrit | Merged stackforge/kolla: Remove docker directory https://review.openstack.org/218061 | 09:40 |
inc0 | I think it's better to run these than bootstrap | 09:41 |
inc0 | bootstrap only run once, these actually makes sense to be run for each playbook run | 09:41 |
inc0 | as long as you can turn them off easily | 09:42 |
inc0 | if it's separate play, that means we won't gain advantage of reporting problems exactly when they appear | 09:42 |
inc0 | if keystone logging is busted, keystone play might actually be finished with success | 09:43 |
inc0 | and fail will be in other play | 09:43 |
inc0 | and that's bad thing | 09:43 |
inc0 | for debugging - it's horrible | 09:43 |
inc0 | when play finishes, I expect service to be working | 09:44 |
inc0 | and separate play doesn't give us that | 09:44 |
SamYaple | register runs everytime | 09:44 |
SamYaple | if we also do a get then it will check the api | 09:44 |
SamYaple | im not adding and removing each run | 09:44 |
SamYaple | thats horrible | 09:44 |
inc0 | ok, adding and removing is | 09:44 |
inc0 | but get's are not | 09:45 |
SamYaple | a seperate playbook for checks is fune | 09:45 |
SamYaple | i said i was good with inline gets immediatly | 09:45 |
SamYaple | read-only operations, lets go for it | 09:45 |
SamYaple | adding/removing now | 09:45 |
SamYaple | no* | 09:45 |
inc0 | ok, I'm ok with that | 09:45 |
SamYaple | cool i think that is a good idea | 09:45 |
inc0 | I misunderstood that you didn't want tests alltogether | 09:45 |
SamYaple | no im ok with the add/delete in a seperate playbook | 09:46 |
SamYaple | for stress-testing | 09:46 |
inc0 | can you inject tasks to deployment play? | 09:46 |
SamYaple | well | 09:46 |
SamYaple | no stress | 09:46 |
SamYaple | how do you mean? | 09:46 |
inc0 | so it will be possible to do add-delete in main playbook | 09:46 |
inc0 | after service start | 09:46 |
inc0 | if you say you want that EXPLICITLY | 09:46 |
inc0 | hmm...we can create play "deploy with testing" which will extend normal deploy | 09:47 |
inc0 | I think it's possible with ansible | 09:47 |
inc0 | anyway, that's technicality, let's start with basic GET tests for each service | 09:48 |
inc0 | and go from there | 09:48 |
inc0 | your nova boot test for gates we were talkign about - that's cool start for create-delete play | 09:48 |
SamYaple | basic get tests i can have for you fairly quickly | 09:49 |
inc0 | I know, it's qick and valuable change | 09:50 |
inc0 | maybe even squeeze it to RC1 broadly | 09:50 |
inc0 | and gives us fairly reliable gate job as well | 09:51 |
inc0 | just do ansible-aio deploy | 09:51 |
inc0 | but let's discuss it with other | 09:52 |
inc0 | s | 09:52 |
inc0 | anyway, I'm off to meetings, I hope I'll be back today | 09:53 |
inc0 | depends of level of brainwashing I'm about to experience | 09:53 |
inc0 | cyas | 09:53 |
*** inc0 has quit IRC | 09:53 | |
*** akwasnie has joined #kolla | 09:54 | |
vbel | include header / footer - shall I add footer only to final containers? Or make general header/footer for all Dockerfiles? | 09:56 |
vbel | SamYaple, inc0 & Co: ^^^ | 09:57 |
SamYaple | vbel: i would think only final docker file | 09:57 |
vbel | ok, sounds good, will post review after test | 09:57 |
SamYaple | otherwise we would have to have a way to sepcify footer header _per_ dockerfile which is complicated | 09:58 |
*** weiyu_ has quit IRC | 10:01 | |
*** weiyu_ has joined #kolla | 10:03 | |
vbel | yes, true | 10:04 |
vbel | we would need template for template :) | 10:04 |
openstackgerrit | Vladislav Belogrudov proposed stackforge/kolla: Add custom footer section to final image Dockefiles https://review.openstack.org/218190 | 10:21 |
*** shardy_ has joined #kolla | 10:22 | |
*** shardy has quit IRC | 10:23 | |
*** apmelton has quit IRC | 10:27 | |
*** daneyon has quit IRC | 10:27 | |
*** jasonsb has quit IRC | 10:27 | |
*** juggler has quit IRC | 10:27 | |
*** Kennan has quit IRC | 10:27 | |
*** vbel has quit IRC | 10:27 | |
*** anteaya has quit IRC | 10:27 | |
*** harmw has quit IRC | 10:27 | |
*** shardy_ has quit IRC | 10:28 | |
*** shardy has joined #kolla | 10:28 | |
*** anteaya has joined #kolla | 10:29 | |
*** harmw has joined #kolla | 10:29 | |
*** apmelton has joined #kolla | 10:30 | |
*** daneyon has joined #kolla | 10:30 | |
*** jasonsb has joined #kolla | 10:30 | |
*** juggler has joined #kolla | 10:30 | |
*** Kennan has joined #kolla | 10:30 | |
*** vbel has joined #kolla | 10:30 | |
*** apmelton has quit IRC | 10:30 | |
*** daneyon has quit IRC | 10:30 | |
*** jasonsb has quit IRC | 10:30 | |
*** juggler has quit IRC | 10:30 | |
*** Kennan has quit IRC | 10:30 | |
*** vbel has quit IRC | 10:30 | |
*** zhiwei has quit IRC | 10:32 | |
*** apmelton has joined #kolla | 10:33 | |
*** daneyon has joined #kolla | 10:33 | |
*** jasonsb has joined #kolla | 10:33 | |
*** juggler has joined #kolla | 10:33 | |
*** Kennan has joined #kolla | 10:33 | |
*** vbel has joined #kolla | 10:33 | |
vbel | common, don't quit all the time :( | 10:34 |
*** weiyu_ has quit IRC | 10:40 | |
vbel | Vladislav Belogrudov proposed stackforge/kolla: Add custom footer section to final image Dockefiles https://review.openstack.org/218190 | 10:46 |
SamYaple | i didnt quit :( | 10:49 |
*** dimsum__ has joined #kolla | 10:53 | |
vbel | there were two waves of quitting/rejoining | 10:55 |
vbel | gate-kolla-functional is broken since yesterday? | 10:57 |
vbel | 2015-08-28 10:23:54.740 | Completed /home/jenkins/workspace/gate-kolla-functional-f21/tests/setup_fedora.sh. | 10:59 |
vbel | 2015-08-28 10:23:54.754 | + tox -e images | 10:59 |
vbel | 2015-08-28 10:23:55.026 | ERROR: unknown environment 'images' | 10:59 |
vbel | 2015-08-28 10:23:55.116 | Build step 'Execute shell' marked build as failure | 10:59 |
SamYaple | vbel: yes | 10:59 |
SamYaple | we just removed teh dokcer directory | 10:59 |
SamYaple | only templates now | 10:59 |
vbel | I see | 11:00 |
SamYaple | we are removing the f21 gate | 11:00 |
SamYaple | hey guys, heat patch incoming https://review.openstack.org/#/c/218195/ | 11:01 |
SamYaple | itll fix heat for us | 11:01 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Add Ansible support for Heat https://review.openstack.org/205528 | 11:06 |
*** weiyu has joined #kolla | 11:11 | |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Add Ansible support for Heat https://review.openstack.org/205528 | 11:12 |
*** rhallisey has joined #kolla | 11:13 | |
*** shardy has quit IRC | 11:14 | |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Remove maridb client from base image https://review.openstack.org/218077 | 11:15 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Add Ansible support for Heat https://review.openstack.org/205528 | 11:17 |
SamYaple | vbel: does this implement your proxy blueprint? | 11:20 |
SamYaple | if so can I amrk it as so? | 11:20 |
vbel | yes, ok | 11:21 |
SamYaple | awesome | 11:21 |
openstackgerrit | Merged stackforge/kolla: Add custom footer section to final image Dockefiles https://review.openstack.org/218190 | 11:24 |
*** pbourke has quit IRC | 11:29 | |
*** pbourke has joined #kolla | 11:29 | |
*** mandre has joined #kolla | 11:40 | |
*** britthouser has joined #kolla | 11:41 | |
*** britthou_ has joined #kolla | 11:42 | |
vbel | SamYaple: thanks for reviewing! | 11:43 |
*** inc0 has joined #kolla | 11:46 | |
*** britthouser has quit IRC | 11:46 | |
inc0 | SamYaple, basic question | 11:46 |
inc0 | how to cleant ansible's run_once? | 11:47 |
inc0 | how does ansible even know that task have already been run? | 11:47 |
SamYaple | what do you mean? | 11:48 |
SamYaple | i dont follow | 11:48 |
vbel | run_once means run on one of hosts in a group (takes first one in inventory) | 11:50 |
inc0 | ah...ok | 11:50 |
SamYaple | first non-failed invetory host in the group | 11:50 |
vbel | no history :) | 11:50 |
inc0 | yeah make sense...brain freeze I'd call my current state | 11:51 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Remove all fail sections for Ubuntu https://review.openstack.org/218225 | 11:51 |
*** britthouser has joined #kolla | 11:52 | |
*** britthou_ has quit IRC | 11:54 | |
openstackgerrit | Merged stackforge/kolla: Remove maridb client from base image https://review.openstack.org/218077 | 11:57 |
inc0 | guys, did you run multinode lately? | 12:06 |
jmccarthy | Anyone have a handy shortcut to change that dark red ansible error color to a lighter one ? | 12:07 |
vbel | use solarized? | 12:11 |
vbel | https://github.com/Anthony25/gnome-terminal-colors-solarized | 12:11 |
*** Haomeng has quit IRC | 12:17 | |
*** Kennan has quit IRC | 12:18 | |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Move docker_templates to docker dir https://review.openstack.org/218238 | 12:18 |
inc0 | guys...include footer patches | 12:18 |
inc0 | patchset | 12:18 |
SamYaple | coolsvap mandre rhallisey https://review.openstack.org/218238 | 12:19 |
inc0 | sdake didn't want any new options | 12:19 |
inc0 | for build script | 12:19 |
SamYaple | thats going to bust alot of peoples patchsets, its pretty empty right now so we should do it sooner rather than later | 12:19 |
SamYaple | inc0: footer is fine. | 12:19 |
openstackgerrit | Merged stackforge/kolla: Add developer docs automatic produce support https://review.openstack.org/218046 | 12:25 |
inc0 | SamYaple, footer is not DRYful | 12:30 |
inc0 | but whatever, right now we dont have anything better | 12:31 |
SamYaple | inc0: how is it not? | 12:31 |
inc0 | because you need to add this line to EVERY container | 12:32 |
inc0 | that is potentially lasy | 12:32 |
SamYaple | not every container | 12:32 |
SamYaple | just the bottom of the tree | 12:32 |
SamYaple | there is no dry way to do it... | 12:32 |
inc0 | and if it won't be last any more, it will need remvoal | 12:32 |
inc0 | oh, there is | 12:32 |
inc0 | but you didn't like it;) | 12:32 |
SamYaple | how so? | 12:32 |
inc0 | template inheritance | 12:33 |
inc0 | you have master template with header and footer | 12:33 |
SamYaple | oh yea were not doign that nonsense | 12:33 |
SamYaple | and that wont solve this issue either | 12:33 |
inc0 | it's not nonsense, but we've been arguing about that | 12:33 |
SamYaple | different containers | 12:33 |
inc0 | it will | 12:33 |
SamYaple | header in the base | 12:33 |
*** Kennan has joined #kolla | 12:33 | |
SamYaple | footer at the bottom of the tree | 12:33 |
SamYaple | it wont | 12:33 |
inc0 | you will set up envs at beggingin and unset at end | 12:33 |
inc0 | of every container | 12:33 |
SamYaple | no | 12:33 |
inc0 | well, it will work, but you wanted only ifs, and you can't do it with ifs;) | 12:35 |
inc0 | it is perfectly doable with more advanced templating, I did that a lot | 12:35 |
SamYaple | it wont because setting them before and after each dockerfile is wasteful | 12:36 |
SamYaple | you cant do what we have with your scenario | 12:36 |
SamYaple | you know AUFS has a layer limit of like 255 and anything more than 80 hurts right? | 12:36 |
SamYaple | thats 2-8 additional layers _per_ image if we did it your way | 12:37 |
SamYaple | horrible | 12:37 |
inc0 | then it can be added in build script dynamically | 12:37 |
inc0 | because build script knows which one is last and which one is first | 12:37 |
inc0 | it's doable | 12:38 |
SamYaple | yea thats not a bad idea. it was just copied from the include header you implemented | 12:38 |
inc0 | without having to write it down in every Dockerfile | 12:38 |
inc0 | include header was in single Dockerfile, this is not | 12:38 |
inc0 | that's my point | 12:38 |
SamYaple | i get it, but it is here now. | 12:38 |
SamYaple | you want to rework it in rc1 go for it | 12:38 |
SamYaple | just loop through all images with no children and inject it at the bottom | 12:39 |
SamYaple | (filter data contaienr) | 12:39 |
inc0 | ehh...I guess I just feel bad because it happend behind my back - I did a lot of thinking about that stuff and there were reasons why I didn't do it in the first place | 12:39 |
SamYaple | well L3 is wrapping up and that blueprint needed to be closed | 12:40 |
SamYaple | there was alot of disccsion about this | 12:40 |
SamYaple | it wasnt behind someones back | 12:40 |
SamYaple | footer was suggested several times for the past two weeks | 12:40 |
SamYaple | its the only one that worked for vbel so we implemented it | 12:40 |
SamYaple | by all means rework it to make it better | 12:40 |
inc0 | anyway its not terrible | 12:40 |
inc0 | just need refACTORING | 12:41 |
inc0 | imho | 12:41 |
SamYaple | agreed | 12:41 |
SamYaple | but we also needed it now | 12:41 |
SamYaple | so it works, isnt horrible, and meets our timeframe | 12:41 |
SamYaple | so its implemented | 12:41 |
SamYaple | now we refine | 12:41 |
*** jasonamyers has joined #kolla | 12:42 | |
inc0 | anyway, I'm off. | 12:42 |
SamYaple | when you coming to US? | 12:42 |
SamYaple | see ya inc0 | 12:44 |
*** inc0 has quit IRC | 12:44 | |
*** Kennan has quit IRC | 12:46 | |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Move docker_templates to docker dir https://review.openstack.org/218238 | 12:53 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Move docker_templates to docker dir https://review.openstack.org/218238 | 12:56 |
vbel | SamYaple, inc0: think of footer as runtime conf :) there is build.conf and there is runtime.conf | 12:59 |
*** Kennan has joined #kolla | 13:00 | |
vbel | you won't like runtime conf everywhere | 13:00 |
vbel | it can be something more sophisticated than ENV | 13:01 |
vbel | inc0: we needed it VERY urgently :) | 13:03 |
SamYaple | vbel: im sure you have no problem if it is reworked while retaining the same functionality right? | 13:03 |
SamYaple | like dynamically injecting the footer? | 13:04 |
vbel | SamYaple: yes, no problem with that | 13:04 |
SamYaple | i thought not | 13:04 |
*** vbel has quit IRC | 13:05 | |
*** vbel has joined #kolla | 13:05 | |
vbel | it's hard to tell whether docker container is final for automatic injection of footer | 13:05 |
SamYaple | no its not in the build script vbel | 13:06 |
SamYaple | each image has a list of children | 13:06 |
SamYaple | if not image['children']: inject footer | 13:06 |
vbel | and if I am to hand over built image to other people to enhance it? | 13:07 |
SamYaple | im not sure i follow | 13:07 |
SamYaple | whats being proposed is equivalnt to what exists right now | 13:07 |
vbel | in the kolla context you know for sure that cinder-api is final | 13:07 |
SamYaple | just dynamically | 13:07 |
vbel | yes, I understand | 13:08 |
SamYaple | ok | 13:08 |
SamYaple | can i get another code for https://review.openstack.org/#/c/218238/ ? it has passed the gate | 13:26 |
openstackgerrit | Ryan Hallisey proposed stackforge/kolla: WIP:Replace config-external with a JSON file https://review.openstack.org/216874 | 13:33 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Add Ansible support for Heat https://review.openstack.org/205528 | 13:34 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Move docker_templates to docker dir https://review.openstack.org/218238 | 13:34 |
SamYaple | damn it pushed a new patch :/ | 13:34 |
SamYaple | oh its ok the votes stayed | 13:34 |
*** jtriley has joined #kolla | 13:41 | |
*** Kennan2 has joined #kolla | 13:44 | |
*** Kennan has quit IRC | 13:44 | |
*** dimsum__ has quit IRC | 13:49 | |
*** dimsum__ has joined #kolla | 13:50 | |
*** britthouser has quit IRC | 13:53 | |
*** weiyu has quit IRC | 13:54 | |
*** britthouser has joined #kolla | 13:56 | |
*** weiyu_ has joined #kolla | 13:59 | |
*** Kennan2 has quit IRC | 14:00 | |
*** ccrouch has quit IRC | 14:09 | |
*** ccrouch has joined #kolla | 14:09 | |
*** Kennan has joined #kolla | 14:14 | |
*** britthouser has quit IRC | 14:16 | |
*** mickt has joined #kolla | 14:33 | |
rhallisey | SamYaple, why is it that ansible won't copy a file to '/opt/kolla/' ? | 14:40 |
rhallisey | do you have to like create the dir or something | 14:40 |
rhallisey | dest: "{{ node_config_directory }}/config.json" | 14:41 |
rhallisey | ^ doesn't do anything | 14:41 |
rhallisey | dest: "{{ node_config_directory }}/mariadb/config.json" | 14:41 |
rhallisey | works | 14:41 |
*** jasonamyers has quit IRC | 14:41 | |
*** jruano has joined #kolla | 14:43 | |
*** ftcpops has joined #kolla | 14:44 | |
*** jasonamyers has joined #kolla | 14:46 | |
*** mickt has quit IRC | 14:57 | |
*** chandankumar has quit IRC | 14:57 | |
sdake_ | morning | 15:11 |
sdake_ | anything happen ov er night | 15:11 |
sdake_ | rhallisey does that set_config python code work | 15:12 |
rhallisey | sdake_, yes | 15:12 |
rhallisey | sdake_, it won't work with current galera atm. I haven't push a new patch for that | 15:13 |
sdake_ | mandre has some more comments | 15:13 |
sdake_ | and then I think its good for a+2 | 15:13 |
rhallisey | trying to figure out this ansible copy thing | 15:13 |
rhallisey | sdake_, ya I saw | 15:13 |
sdake_ | fwiw we don't run ansible in the gates | 15:13 |
sdake_ | functaionL_f21 gtate will ALWAYS fail | 15:14 |
rhallisey | I mean I'd like to get the <service> piece out | 15:14 |
sdake_ | oh right | 15:14 |
rhallisey | so config doesn't have to be in /opt/kolla/service/config.json | 15:14 |
sdake_ | os.environ is one easy way | 15:14 |
*** sdake_ has quit IRC | 15:15 | |
*** sdake has joined #kolla | 15:16 | |
sdake | did tat remove stack have to be rebased? | 15:20 |
*** jasonamyers has quit IRC | 15:26 | |
openstackgerrit | Paul Bourke proposed stackforge/kolla: Rename support to control in Vagrantfile https://review.openstack.org/218352 | 15:28 |
*** chandankumar has joined #kolla | 15:31 | |
*** chandankumar has quit IRC | 15:32 | |
*** blahRus has joined #kolla | 15:33 | |
openstackgerrit | Merged stackforge/kolla: Move docker_templates to docker dir https://review.openstack.org/218238 | 15:44 |
openstackgerrit | Merged stackforge/kolla: Use external address in openrc-example https://review.openstack.org/218112 | 15:50 |
*** sdake_ has joined #kolla | 15:55 | |
*** sdake has quit IRC | 15:59 | |
*** weiyu_ has quit IRC | 16:00 | |
*** jasonsb has quit IRC | 16:04 | |
*** jasonsb has joined #kolla | 16:05 | |
*** jruano has quit IRC | 16:07 | |
*** jasonsb has quit IRC | 16:10 | |
sdake_ | vvbel around/ | 16:14 |
sdake_ | vbel are the include footers in the correct place here; | 16:15 |
sdake_ | Change-Id: I23ad325e0898d77804a22038ddc618e1e49afd88 | 16:15 |
pbourke | vbel is gone for weekend I think | 16:17 |
pbourke | let me take a look | 16:17 |
pbourke | sdake_: are you thinking they should be above CMD ? | 16:18 |
pbourke | sdake_: because that's what im thinking now looking at it :/ | 16:18 |
sdake_ | dont know | 16:18 |
*** unicell has quit IRC | 16:18 | |
pbourke | im pretty sure he tested it | 16:19 |
sdake_ | the use cae is for env in proxies | 16:19 |
sdake_ | so if it works for tht use cae i'm good with it | 16:19 |
sdake_ | can you sync up with him and hae him ack with me that he tested it or didnt test it | 16:19 |
sdake_ | nobody has written "how to use proxies' docuemtnation so i cant test it | 16:19 |
pbourke | (10:29:39) vladislav.belogrudov: I am working on include-header-footer, yes, it works | 16:21 |
pbourke | (10:30:06) vladislav.belogrudov: I mean by manually adding those 4 lines in Dockerfiles of api/registry/... | 16:21 |
pbourke | (10:30:26) vladislav.belogrudov: I could run ndb/keystone/glance with that approach | 16:21 |
pbourke | bah messed up PM | 16:21 |
pbourke | you get the idea :P | 16:21 |
*** jmccarthy has quit IRC | 16:26 | |
sdake_ | rhallisey thoe io error exceptions would be unexpected, so make sure to print the e value out | 16:31 |
sdake_ | in you patch | 16:31 |
rhallisey | roger | 16:31 |
sdake_ | since you will be checkingfor the existance of the file first | 16:31 |
*** britthouser has joined #kolla | 16:32 | |
openstackgerrit | Steven Dake proposed stackforge/kolla: WIP: Make keystone work on centos+binary https://review.openstack.org/218075 | 16:36 |
sdake_ | rhallisey if you wnat a hard review need love on https://review.openstack.org/#/c/205528/ | 16:40 |
sdake_ | no need to test it just insepct with yuor eyeballs | 16:41 |
sdake_ | pbourke same story :) | 16:41 |
*** unicell has joined #kolla | 16:46 | |
*** jruano has joined #kolla | 16:57 | |
*** akwasnie has quit IRC | 16:59 | |
sdake_ | rhallisey do you happen to be working on the next rev of your patch? | 17:11 |
sdake_ | there are more comments i just added to the last rev | 17:11 |
sdake_ | pbourke would appreciate someone that is a python pro taking a look at rhallisey's patch ;) | 17:12 |
sdake_ | that pro being you | 17:12 |
sdake_ | rhallisey re your patch, when you get wrapped on these first two, json everything that is an ansiblee service | 17:15 |
sdake_ | as separate patches | 17:15 |
sdake_ | put them as seprat commits in one patch stream | 17:15 |
sdake_ | and i'll -2 the first unti all have had reviews | 17:15 |
sdake_ | if it lands before the 3rd it lands as one piece | 17:15 |
sdake_ | if it doesnt' land before 3rd, it lands after liberty-3 | 17:16 |
sdake_ | i want to be able to check out the entire stream and test it | 17:17 |
sdake_ | if you need git help with this let me know | 17:17 |
sdake_ | git rebase -i is your friend ;-) | 17:17 |
sdake_ | pbourke https://review.openstack.org/#/c/216874/ | 17:18 |
rhallisey | sdake_, I'm still trying to understand the ansible copy | 17:19 |
*** sdake_ is now known as sdake | 17:19 | |
sdake | rhallisey is copy not working correctly? | 17:19 |
rhallisey | copy won't copy anywhere but /opt/kolla/service/ | 17:19 |
sdake | wher do you need it copied/ | 17:20 |
sdake | ? | 17:20 |
rhallisey | /opt/kolla | 17:20 |
sdake | /opt/kolla isn't bindmounted | 17:20 |
sdake | so you wont see it inside your container | 17:20 |
sdake | /opt/kolla is a shared directory on the target | 17:20 |
sdake | you can't copy to that directory anyway | 17:20 |
sdake | you need to copy to /opt/kolla/service | 17:20 |
rhallisey | ugh that's why | 17:20 |
sdake | we are not bindmounting /opt/kolla in every container anyway ;:) | 17:21 |
rhallisey | ok so then I need the service param | 17:21 |
sdake | os.environ | 17:21 |
sdake | you don't need a param | 17:21 |
rhallisey | ok that works | 17:23 |
rhallisey | ugh now it all makes sence | 17:23 |
sdake | i have pt in 1 hour for about 1 hour | 17:23 |
sdake | hopefully ou canget all the patches in the queue today y:) | 17:23 |
rhallisey | I was wtfing for like 3 hours there | 17:23 |
sdake | when i get stuck i ask for hlep ;) | 17:24 |
rhallisey | asked sam, he went to sleep | 17:24 |
sdake | i told you that yesterday iirc ;) | 17:24 |
rhallisey | you did, but I didn't understand | 17:25 |
sdake | yup | 17:25 |
sdake | the copy operation copies to the target, not to the container | 17:25 |
sdake | remember use git rebase -i to edit your first commit | 17:26 |
sdake | and keep them in one chain | 17:26 |
rhallisey | ya will do | 17:27 |
sdake | use git add to add changed files | 17:27 |
sdake | then git rebase --continue | 17:27 |
sdake | use the edit operation when git rebase -i brings up the commit list | 17:27 |
sdake | do NOT use git commit --amend -a | 17:28 |
*** sdake_ has joined #kolla | 17:31 | |
*** sdake has quit IRC | 17:34 | |
*** SourabhP has joined #kolla | 17:38 | |
*** loth has joined #kolla | 17:50 | |
*** jasonsb has joined #kolla | 17:52 | |
*** achanda has joined #kolla | 17:56 | |
SamYaple | rhallisey: hey | 17:57 |
SamYaple | sorry i dont know the issue you are tlaking about | 17:57 |
rhallisey | SamYaple, hey | 17:57 |
rhallisey | figured it out | 17:57 |
rhallisey | sdake did | 17:57 |
SamYaple | ah i see | 17:58 |
*** unicell has quit IRC | 17:58 | |
*** unicell1 has joined #kolla | 17:58 | |
SamYaple | sdake_: ping | 17:58 |
SamYaple | heat questino | 17:59 |
openstackgerrit | Merged stackforge/kolla: Rename support to control in Vagrantfile https://review.openstack.org/218352 | 18:06 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Add Ansible support for Heat https://review.openstack.org/205528 | 18:39 |
*** achanda has quit IRC | 18:44 | |
*** jruano has quit IRC | 18:48 | |
sdake_ | yo | 19:01 |
sdake_ | samyaple just got back from pt whats up | 19:01 |
*** sdake_ is now known as sdake | 19:01 | |
SamYaple | heat issue resolved without revert upstream | 19:02 |
SamYaple | but revert will probably still happen | 19:02 |
SamYaple | the question is now, do you want to pip install the newer package into the base distro temporarily? | 19:03 |
SamYaple | for centos binary i mean | 19:03 |
*** sdake_ has joined #kolla | 19:03 | |
*** jasonamyers has joined #kolla | 19:05 | |
*** sdake has quit IRC | 19:07 | |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Add Ansible support for Heat https://review.openstack.org/205528 | 19:08 |
sdake_ | ya i saw your message about mysql-python | 19:09 |
sdake_ | in the review | 19:09 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Remove unneeded pip install in centos-binary https://review.openstack.org/218424 | 19:11 |
*** achanda has joined #kolla | 19:11 | |
SamYaple | well there best of both worlds | 19:11 |
SamYaple | merge it fixed and working now | 19:11 |
SamYaple | second patch has a depends on upstream | 19:11 |
SamYaple | rhallisey: did you want to merge your ceph work now? The ansible can be done in a seperate patch. That has typically been our model | 19:12 |
rhallisey | SamYaple, no it's not ready | 19:12 |
SamYaple | i mean the containers are... They have the right packages | 19:13 |
rhallisey | SamYaple, ya I guess it's fine then | 19:14 |
sdake_ | just make sure its bulding plz | 19:15 |
SamYaple | rhallisey: do you want to rebase it or me? i included some whitespace in the last patch that needs to be removed | 19:15 |
rhallisey | let me finish current patch stuff then will go for it | 19:15 |
SamYaple | awesome! im going to leave a comment on the things i messed up real quick then | 19:16 |
SamYaple | comments left, thanks rhallisey | 19:17 |
rhallisey | cool thanks sam | 19:17 |
*** jruano has joined #kolla | 19:20 | |
*** jasonsb has quit IRC | 19:21 | |
*** jasonsb has joined #kolla | 19:22 | |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Add Ansible support for Heat https://review.openstack.org/205528 | 19:25 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Remove unneeded pip install in centos-binary https://review.openstack.org/218424 | 19:25 |
*** jasonsb has quit IRC | 19:26 | |
sdake_ | the build is definately slower | 19:29 |
sdake_ | it will ve nice to get rid of config-external.sh throughout the source base ;) | 19:30 |
sdake_ | rhallisy hows the progress coming? :) | 19:30 |
rhallisey | sdake_, going ok. Getting tired of the long build times | 19:39 |
sdake_ | just build one image? | 19:40 |
rhallisey | I only do | 19:40 |
sdake_ | build.py options imagename | 19:40 |
rhallisey | build rebuilding base is slow | 19:40 |
rhallisey | I'm close to another push | 19:41 |
SamYaple | rhallisey: are you centos-binary? | 19:41 |
rhallisey | ya | 19:41 |
*** devlaps has joined #kolla | 19:41 | |
SamYaple | yea that base can use some lovbe | 19:41 |
SamYaple | all those deps in teh base | 19:41 |
SamYaple | esh | 19:41 |
sdake_ | it needs to be broken apart | 19:41 |
sdake_ | that wouldn'tt help rhallisey tho ;-) | 19:42 |
SamYaple | ok anyone need anything | 19:43 |
SamYaple | going back to bed | 19:43 |
SamYaple | otherwise | 19:43 |
*** achanda has quit IRC | 19:43 | |
sdake_ | rhallisey needs a review on his json patch | 19:43 |
sdake_ | so he can get the work ready for merging in l3 hopefullly ;-) | 19:43 |
*** achanda has joined #kolla | 19:47 | |
rhallisey | ok one more build | 19:47 |
devlaps | hi folks | 19:48 |
sdake_ | hey devlaps | 19:48 |
devlaps | just diving into the developer quickstart now.. | 19:49 |
devlaps | where do you folks keep your reviews? | 19:49 |
sdake_ | https://review.openstack.org/#/q/is:watched+status:open+project:stackforge/kolla,n,z | 19:49 |
sdake_ | add kolla to your watched project list in review.openstack.org ui | 19:49 |
devlaps | will do. thanks. | 19:49 |
*** sdake_ is now known as sdake | 19:51 | |
* sdake wtb faster building | 19:51 | |
*** cfarquhar has quit IRC | 19:52 | |
openstackgerrit | Ryan Hallisey proposed stackforge/kolla: WIP:Replace config-external with a JSON file https://review.openstack.org/216874 | 19:52 |
* rhallisey forgot to rebase | 19:53 | |
openstackgerrit | Ryan Hallisey proposed stackforge/kolla: WIP:Replace config-external with a JSON file https://review.openstack.org/216874 | 19:54 |
*** cfarquhar has joined #kolla | 19:55 | |
*** cfarquhar has quit IRC | 19:55 | |
*** cfarquhar has joined #kolla | 19:55 | |
SamYaple | rhallisey: you rebased a bit wrong | 19:57 |
SamYaple | still have teh docker_templates dir | 19:57 |
rhallisey | ya I noticed just a sec | 19:57 |
SamYaple | I left other comments as well | 19:57 |
rhallisey | oh kk | 19:57 |
SamYaple | some in patchset 10, but then you pushed a bunch of patches :) | 19:57 |
*** jasonamyers has quit IRC | 19:58 | |
SamYaple | can someone tell me which package mysql_config comes from on centos? | 19:59 |
SamYaple | using your rpm tools i would guess | 19:59 |
sdake | rhallisey samyaple has comments n patchset#10 https://review.openstack.org/#/c/216874/10/docker_templates/base/set_configs.py | 20:00 |
rhallisey | ya addressing | 20:00 |
sdake | i think my if block could much simpler | 20:00 |
SamYaple | sdake: centos request above, can you provide? | 20:00 |
sdake | for k in key_list: | 20:01 |
SamYaple | sdake: i suggested that too i believe | 20:01 |
sdake | if k in data | 20:01 |
*** jasonamyers has joined #kolla | 20:01 | |
sdake | SamYaple provide hwich | 20:01 |
SamYaple | 20:00 < SamYaple> can someone tell me which package mysql_config comes from on centos? | 20:01 |
sdake | oh | 20:01 |
SamYaple | 20:00 < SamYaple> using your rpm tools i would guess | 20:01 |
sdake | secon | 20:01 |
SamYaple | thank you | 20:01 |
sdake | mariadb-devel-10.0.17-1.fc21.x86_64 | 20:02 |
SamYaple | hmm | 20:02 |
SamYaple | we need that then to fix heat until upstream | 20:02 |
SamYaple | are you ok with that in base for centos-binary sdake? | 20:02 |
SamYaple | until upstream merge | 20:02 |
sdake | doesn't mattr to me | 20:03 |
SamYaple | cool | 20:03 |
sdake | i suspect upstream will just changef the global rquirements | 20:03 |
sdake | that is wht I'd do ;-) | 20:03 |
SamYaple | global require has no version | 20:03 |
SamYaple | most distros only provide the older | 20:04 |
SamYaple | i doubt it | 20:04 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Add Ansible support for Heat https://review.openstack.org/205528 | 20:04 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Remove unneeded pip install in centos-binary https://review.openstack.org/218424 | 20:04 |
SamYaple | ok gotta go to sleep | 20:05 |
SamYaple | date tonight | 20:05 |
SamYaple | im going bowling | 20:05 |
sdake | real60m23.909s | 20:06 |
sdake | with a push | 20:06 |
sdake | terribad | 20:06 |
SamYaple | sdake: i blame your registry | 20:06 |
SamYaple | without push i build in 13 minutes | 20:06 |
SamYaple | with push 27 minutes | 20:07 |
sdake | i see | 20:07 |
SamYaple | but i am also doing ubuntu-source | 20:07 |
SamYaple | so grain of salt and all | 20:07 |
SamYaple | i would check your real image sizes | 20:07 |
SamYaple | also are you using the v2 registry? | 20:07 |
sdake | no | 20:08 |
SamYaple | ok dont | 20:08 |
SamYaple | it pushes each image in full each time, it doesnt catch the layer has already been uploaded | 20:08 |
SamYaple | its why docker pushes take 10 hours now :) | 20:08 |
sdake | samyaple quick q | 20:09 |
sdake | is there a run_once but per node not per deployment? | 20:09 |
SamYaple | everything is run_once per node | 20:10 |
sdake | i want to run bootstrap on every node one time for a certain type of bootstrap operation | 20:10 |
SamYaple | dont specify run_once? | 20:10 |
SamYaple | i think you may be confused as to what the option does | 20:10 |
sdake | i only want it to run one time on init | 20:10 |
sdake | yes possible the usptream docs are unclear | 20:10 |
SamYaple | when : something | 20:10 |
SamYaple | we skip all bootstrap tasks on the second time around | 20:11 |
sdake | yes but the db bootstrapping only occurs on one node right? | 20:11 |
SamYaple | yes as it should be | 20:11 |
SamYaple | that chown is not needed dammit! | 20:11 |
sdake | yes i want to bootstrap something else not a db | 20:11 |
SamYaple | :P | 20:11 |
sdake | you are worng | 20:11 |
sdake | but this is not about that | 20:11 |
sdake | i want to run once per node | 20:12 |
sdake | only one time | 20:12 |
SamYaple | you know ansible has no memory of past runs right? | 20:12 |
sdake | yes | 20:12 |
SamYaple | run_once means run on the first non-failed inventory host in the running group | 20:12 |
SamYaple | that is a single node | 20:12 |
sdake | i dont understand how it detect bootstrap has been done | 20:12 |
SamYaple | however you want | 20:13 |
sdake | i see | 20:13 |
sdake | i am making a data container for nova | 20:13 |
SamYaple | what ever condtion means 'bootstrap is done' is what you want to chec kfor | 20:13 |
sdake | since if you upgrade libvirt your data goes -byeybe! | 20:13 |
*** alisonh has quit IRC | 20:13 | |
SamYaple | not true | 20:13 |
SamYaple | but possible | 20:13 |
SamYaple | the way docker-py recreates the container the volume is saved when switching images | 20:14 |
SamYaple | but data container is what we need, agreed | 20:14 |
SamYaple | and i was just kidding about the chown sdake you might be right. i havent tested centos | 20:14 |
SamYaple | seems wierd | 20:14 |
sdake | ya i think it may be centos only | 20:15 |
sdake | i'll get into that once i fix up keystone | 20:15 |
SamYaple | or maybe im shit at testing | 20:15 |
SamYaple | idk | 20:15 |
SamYaple | this is why i want a gate | 20:15 |
openstackgerrit | Ryan Hallisey proposed stackforge/kolla: WIP:Replace config-external with a JSON file https://review.openstack.org/216874 | 20:15 |
SamYaple | im good at automaing! | 20:15 |
*** alisonh has joined #kolla | 20:15 | |
openstackgerrit | Ryan Hallisey proposed stackforge/kolla: WIP: Replace config-external with a JSON file for Galera https://review.openstack.org/217880 | 20:17 |
rhallisey | if the Galera template is good will be build that into the rest of the services | 20:17 |
*** jasonsb has joined #kolla | 20:27 | |
*** achanda has quit IRC | 20:28 | |
sdake | rhallisey galera looks pretty good | 20:30 |
sdake | but the json doesn't match the config-external | 20:30 |
rhallisey | I also need to put kolla-common back in | 20:31 |
sdake | rhallisey reviewed both your patches | 20:38 |
sdake | still needs love | 20:38 |
SamYaple | i have reviewed them as well | 20:38 |
SamYaple | sdake: i dont think this heat thing is going to work for centos-binary | 20:39 |
SamYaple | now i would need to instal lgcc | 20:39 |
SamYaple | in centos-binary | 20:39 |
SamYaple | huge images | 20:39 |
SamYaple | im just going to leave it | 20:39 |
SamYaple | centos-binary will be broken for heat-delete until heat is fixed | 20:39 |
sdake | just apply a patch | 20:40 |
sdake | from upstream | 20:40 |
SamYaple | what? | 20:40 |
sdake | cherry-pick their patch and apply it - we have done it before for horizon | 20:40 |
rhallisey | ya I like the loop better | 20:40 |
*** jasonamyers has quit IRC | 20:40 | |
SamYaple | sdake: alright well either way im removing that from the ansible patch right now | 20:40 |
SamYaple | ansible patch has like 30 revisions | 20:40 |
sdake | removing which | 20:41 |
SamYaple | the pip install stuff | 20:41 |
sdake | ok | 20:41 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Add Ansible support for Heat https://review.openstack.org/205528 | 20:41 |
rhallisey | devlaps, what's a module doctoring? | 20:42 |
devlaps | hi rhallisey.. apologies, that's my computer's autocorrect :( | 20:43 |
devlaps | "docstring" was the intention :) | 20:43 |
rhallisey | devlaps, hey no problem | 20:43 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Keystone container listens on port 80 by default https://review.openstack.org/218075 | 20:48 |
sdake | there keystone working in master now | 20:50 |
sdake | anyone doing centos+binary installs, you will need the above patch | 20:50 |
sdake | [sdake@bigiron kolla]$ sudo docker logs horizon | 20:51 |
sdake | chown: invalid spec: 'horizon:' | 20:51 |
jpeeler | fix this? httpd: Could not reliably determine the server's fully qualified domain name, using fe80::f816:3eff:fef5:c841 | 20:51 |
SamYaple | jpeeler: yea that could be fixed | 20:52 |
sdake | no jpeeler the prt where the container restarts ove rand over | 20:52 |
devlaps | thanks sdave. it's on my list, but not there yet. | 20:52 |
SamYaple | its not a breaking thing though | 20:52 |
sdake | jpeeler your using docker dir - i am using docker_templates images | 20:52 |
sdake | if you rebuild your images right now keystone is busted | 20:52 |
jpeeler | i thought docker_templates is the docker dir, but i'll approve and rebuild. running into other weirdness | 20:53 |
sdake | jpeeler yes but have you rebulit based upon what was in the docker-teplates dir? | 20:56 |
sdake | or are you using old docker based images | 20:56 |
jpeeler | yeah, just rebuilt 5 min ago | 20:56 |
jpeeler | the docker-templates patch got merged | 20:56 |
sdake | if your running into wierdness on an ansible deploy its probablybecause keystone goes out to lunch | 20:56 |
openstackgerrit | Ryan Hallisey proposed stackforge/kolla: WIP:Replace config-external with a JSON file https://review.openstack.org/216874 | 20:57 |
sdake | jpeeler to fix that problem you have | 20:58 |
sdake | put your ip addrss in /etc/hosts | 20:58 |
sdake | the contianer reads /etc/hosts by bindmounting into the container | 20:59 |
sdake | it will fix problems with galrea atleaest wit hthat same issue | 20:59 |
*** SourabhP has quit IRC | 21:01 | |
*** SourabhP has joined #kolla | 21:01 | |
*** jtriley has quit IRC | 21:02 | |
openstackgerrit | Ryan Hallisey proposed stackforge/kolla: WIP: Replace config-external with a JSON file for Galera https://review.openstack.org/217880 | 21:12 |
rhallisey | sdake, when do all these patch need to be done by? | 21:23 |
sdake | rhallisey why the export ? | 21:24 |
sdake | why not just do SERVICE="mariadb" | 21:24 |
sdake | as soon as librety-3 is functional and feature complete for the list of blueprints in liberty-3 milestone | 21:24 |
sdake | i am going to cut the release | 21:24 |
sdake | atm multinode doesnt work on centos+binary | 21:24 |
sdake | horizon doesn't work on centos+binary | 21:24 |
sdake | cinder is busted - pushign that to rc1 | 21:25 |
sdake | finally compute needs a data container | 21:25 |
sdake | so whenever all that work finishes is when i'm cutting the release | 21:25 |
sdake | it could be the 1st, coudl be the 3rd | 21:25 |
sdake | get em queued up and if they are ready to go we will merge em now | 21:26 |
rhallisey | sdake, because os.enviro looks at whats in env | 21:26 |
sdake | if not we will merge during rc1 | 21:26 |
openstackgerrit | Merged stackforge/kolla: Keystone container listens on port 80 by default https://review.openstack.org/218075 | 21:26 |
sdake | try it without eport | 21:26 |
rhallisey | just tried locally | 21:26 |
sdake | SERVICE=mariadb puts service into the environment | 21:27 |
sdake | and it didn't work? | 21:28 |
rhallisey | just did it locally it didn't make it to the environment | 21:28 |
rhallisey | that stays local | 21:28 |
sdake | paste error | 21:28 |
*** jasonsb has quit IRC | 21:29 | |
*** SourabhP has quit IRC | 21:30 | |
*** tummy has joined #kolla | 21:34 | |
*** SourabhP has joined #kolla | 21:38 | |
*** dimsum__ has quit IRC | 21:38 | |
*** jruano has quit IRC | 21:44 | |
*** achanda has joined #kolla | 21:48 | |
sdake | rhallisey there ya go - now other services please ;) | 21:50 |
sdake | start with the ansible ones in alphabetical order | 21:50 |
rhallisey | sdake, kk | 21:51 |
sdake | rhallisey just being realistic you will likely have to address the format problem | 21:54 |
rhallisey | ok | 21:54 |
sdake | let people spin onthat final review for a few days | 21:55 |
*** SourabhP has quit IRC | 21:55 | |
sdake | and then addess eveyrone's comments in one go | 21:55 |
sdake | m0ar services | 21:55 |
*** SourabhP has joined #kolla | 21:56 | |
*** dimsum__ has joined #kolla | 21:56 | |
*** dims_ has joined #kolla | 22:00 | |
*** SourabhP has quit IRC | 22:01 | |
*** dimsum__ has quit IRC | 22:02 | |
*** SourabhP has joined #kolla | 22:05 | |
rhallisey | sdake, testing as I add.. | 22:07 |
rhallisey | or would you prefer I guess so it doesn't take like 2 hours | 22:07 |
sdake | reference config-external | 22:07 |
sdake | just add it all | 22:07 |
rhallisey | I might just rush through this and test after | 22:08 |
sdake | as separate commits | 22:08 |
sdake | that is what i'd do | 22:08 |
sdake | and git rebase -i to the patch that is busted and fix it up | 22:08 |
*** SourabhP has quit IRC | 22:08 | |
openstackgerrit | Ryan Hallisey proposed stackforge/kolla: WIP:Replace config-external with a JSON file https://review.openstack.org/216874 | 22:11 |
openstackgerrit | Ryan Hallisey proposed stackforge/kolla: WIP: Replace config-external with a JSON file for Galera https://review.openstack.org/217880 | 22:11 |
openstackgerrit | Ryan Hallisey proposed stackforge/kolla: WIP: Replace config-external with a JSON file for Rabbitmq https://review.openstack.org/218496 | 22:11 |
*** ftcpops has quit IRC | 22:14 | |
*** jruano has joined #kolla | 22:16 | |
rhallisey | keystone is interestinf.. | 22:16 |
openstackgerrit | Ryan Hallisey proposed stackforge/kolla: WIP: Replace config-external with a JSON file for Galera https://review.openstack.org/218497 | 22:19 |
openstackgerrit | Ryan Hallisey proposed stackforge/kolla: WIP: Replace config-external with a JSON file for Keystone https://review.openstack.org/218497 | 22:20 |
sdake | get rid of the WIP: onthe fornt of your commits going forwrad plz | 22:22 |
sdake | you can't rebase reword ingerrit i think | 22:22 |
openstackgerrit | Ryan Hallisey proposed stackforge/kolla: WIP: Replace config-external with a JSON file for Glance https://review.openstack.org/218501 | 22:30 |
rhallisey | k | 22:30 |
rhallisey | sorry missed your comment | 22:30 |
openstackgerrit | Ryan Hallisey proposed stackforge/kolla: Replace config-external with a JSON file for Nova https://review.openstack.org/218505 | 22:42 |
rhallisey | sdake, I'm going to finish these up tmr | 22:44 |
rhallisey | and do some testing | 22:44 |
sdake | enjoy | 22:44 |
rhallisey | I have to be somewhere | 22:44 |
rhallisey | sdake, lights turned off and only other living thing here is this fly | 22:44 |
rhallisey | office is abandoned | 22:45 |
rhallisey | :) | 22:45 |
rhallisey | sdake, talk to ya tmr | 22:45 |
*** rhallisey has quit IRC | 22:45 | |
*** tummy has quit IRC | 23:34 | |
*** blahRus has quit IRC | 23:51 | |
*** dimsum__ has joined #kolla | 23:52 | |
sdake | kill deployment redeploy kill deployment redeploy all day long | 23:53 |
* sdake goes batty | 23:53 | |
*** dims_ has quit IRC | 23:55 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!