| SamYaple | sdake: on it | 00:00 |
|---|---|---|
| openstackgerrit | Sam Yaple proposed stackforge/kolla: Remove config-internal for Horizon https://review.openstack.org/210577 | 00:02 |
| *** thumpba has quit IRC | 00:03 | |
| SamYaple | sdake: youre stuff could be merged if you didnt have 10 patch long dependacy chains | 00:07 |
| sdake | my stuff was already mreged samyaple | 00:19 |
| sdake | those are new patches | 00:20 |
| *** thumpba has joined #kolla | 00:20 | |
| SamYaple | you had those changes in originally for those patches though | 00:21 |
| SamYaple | they got removed right? | 00:21 |
| sdake | these are new bug fixes to master for left over stuff | 00:22 |
| sdake | I dont know what happened with ceilometer | 00:22 |
| sdake | I think it was left undone | 00:22 |
| SamYaple | no https://review.openstack.org/#/c/210520/4/docker/centos/source/ceilometer/ceilometer-alarm/Dockerfile | 00:22 |
| SamYaple | i think one of your rebases screwed things up | 00:22 |
| SamYaple | or someone elses | 00:22 |
| SamYaple | you clearly did it | 00:23 |
| sdake | probably was just broken to begin with | 00:23 |
| SamYaple | eh anyway | 00:23 |
| SamYaple | this seems to break builds... | 00:23 |
| sdake | with my patch set almost all containers build again ;-) | 00:24 |
| sdake | http://ur1.ca/nedq9 -> http://paste.fedoraproject.org/253283/14391663 | 00:25 |
| openstackgerrit | Sam Yaple proposed stackforge/kolla: Removed config-internal check.sh for glance https://review.openstack.org/210872 | 00:26 |
| sdake | its config-glance.sh that is the problem | 00:27 |
| sdake | it was deleted out of common/base | 00:27 |
| openstackgerrit | Sam Yaple proposed stackforge/kolla: Removed config-internal check.sh for cinder https://review.openstack.org/210873 | 00:29 |
| SamYaple | do we use it at all? | 00:29 |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Remove config-internal for glance https://review.openstack.org/210874 | 00:30 |
| sdake | check.sh? Nah we had planned to but I dont know how we would actually use it | 00:30 |
| sdake | with config-external, we for the most part don't need config-service.sh bash scripts | 00:31 |
| sdake | we should remove all that are still left | 00:31 |
| sdake | according to my build that was pretty much just glance | 00:31 |
| openstackgerrit | Sam Yaple proposed stackforge/kolla: Removed config-internal check.sh for neutron https://review.openstack.org/210875 | 00:31 |
| SamYaple | we do need it | 00:32 |
| SamYaple | maybe not glance | 00:32 |
| SamYaple | but for rabbitmq and mariadb | 00:32 |
| sdake | ok | 00:32 |
| SamYaple | i specifically used some functions from them | 00:32 |
| sdake | no openstack service should be using it I think | 00:32 |
| SamYaple | we can reintegrate them back into the start.sh if you want, but dont just delete them | 00:33 |
| SamYaple | i beleive thats right | 00:33 |
| sdake | i didn't delete anything from rabbitmq or mariadb | 00:33 |
| sdake | you did those containers | 00:33 |
| sdake | we dont need config-glance.sh that was only used by config-internal | 00:33 |
| SamYaple | I want to propose we rename CONFIG_EXTERNAL_COPY_ONCE to COPY_ONCE | 00:33 |
| SamYaple | i didn't delete them from rabbitmq or mariadb. just letting you know so you didnt ;) | 00:34 |
| sdake | roger | 00:34 |
| sdake | that neutron set-configs should be some special version of set_configs that takes a parameter imo ;-) | 00:35 |
| SamYaple | sdake: nah we can just wait for thin containers | 00:36 |
| SamYaple | no big thing | 00:36 |
| sdake | if we rename copy_once we have to rename copy_always | 00:37 |
| sdake | if we are going to do that, lets do it before liberty-3 please | 00:37 |
| sdake | i dont see the point but i dont care strongly | 00:37 |
| *** thumpba has quit IRC | 00:38 | |
| *** jpeeler has joined #kolla | 00:38 | |
| sdake | one area for gold plating is removing config-external.sh, inject a yaml file with the files source/target locations to copy, and make set-configs process it in a standard location | 00:39 |
| sdake | this is sort of based on an idea you may ahve had previously mentioned, not certain if it was you or someone else | 00:40 |
| SamYaple | i suggested json since we do a similiar thing with swift | 00:41 |
| SamYaple | additionally it will need information about the owner and permissions | 00:41 |
| sdake | i find json difficult to use typically | 00:41 |
| SamYaple | i would prefer it not to be yaml | 00:41 |
| SamYaple | well no one will be using it directly | 00:41 |
| SamYaple | it will be generated | 00:42 |
| SamYaple | yaml is a pita to geenrate and we already generate json | 00:42 |
| sdake | generate from what? | 00:42 |
| sdake | yaml is easy to generate last I tried ;-) | 00:42 |
| SamYaple | Ansible will generate the file | 00:42 |
| SamYaple | the container will read the file and do the actions | 00:42 |
| sdake | oh i see | 00:43 |
| SamYaple | we do the simliar thing with swift and we use json | 00:43 |
| sdake | thats a pretty good approach | 00:43 |
| SamYaple | i want to keep it consistent | 00:43 |
| sdake | gold plating but unfortunately affects our abi | 00:43 |
| sdake | i'd like our abi to be stable after l3 | 00:43 |
| SamYaple | if you convince pbourke to switch swift to yaml, i wouldn't care | 00:43 |
| sdake | i didn't gt the autogneration parth with ansible at runtime | 00:44 |
| sdake | i thought we woudl inject at container create | 00:44 |
| sdake | but runtime injection is much better | 00:44 |
| SamYaple | yea buildtime is the least flexible | 00:44 |
| sdake | it does put a burden on those that integrate with the containers to comprehend the format | 00:44 |
| sdake | atm there is no such burden | 00:45 |
| SamYaple | there is with swift | 00:45 |
| SamYaple | and json or yaml is still going to be a format | 00:45 |
| sdake | what i mean is config-external atm is hard coded | 00:45 |
| SamYaple | yea i get that | 00:45 |
| sdake | i dont know about swift | 00:45 |
| SamYaple | im not for or against yaml _except_ json is builtin for python and yaml is not, and we already use json | 00:46 |
| SamYaple | with that in mind we should just generate json | 00:46 |
| SamYaple | hmmm | 00:46 |
| SamYaple | if we do generated yaml that puts yaml deps on the destination hosts | 00:46 |
| sdake | the nice thing bout that is we ould get rid of all config-external.sh files | 00:47 |
| SamYaple | yep | 00:47 |
| sdake | yaml in python is built in i think | 00:47 |
| SamYaple | why does it require libyaml | 00:48 |
| sdake | i dont feel strongly about json vs yaml if its autogenerated | 00:48 |
| sdake | if its human generated i have a very strong preference for yaml | 00:48 |
| SamYaple | agreed | 00:49 |
| sdake | i spent a year plus generating templates in json for heat, and it was super painful | 00:49 |
| sdake | and then steve baker added yaml support and i was olike "wow wtf why didn't we start with that. :) | 00:49 |
| sdake | samyaple can you work out a prototype keystone container with the yaml copy stuff so we can see what it looks like | 00:51 |
| sdake | i want to make sure the tripleoguys are onboard with that | 00:51 |
| sdake | if its autogenerated, where are the original file data stored, in yaml in ansible? | 00:53 |
| *** Slower has quit IRC | 00:53 | |
| SamYaple | you mean the json? | 00:56 |
| sdake | containers containing less and less code - I ike that | 00:56 |
| SamYaple | since im not doing yaml | 00:56 |
| SamYaple | swift uses json | 00:56 |
| sdake | so we have to put the data in manually in some how right? | 00:56 |
| SamYaple | not neccesarily | 00:56 |
| sdake | i would like to see the implementation top to bottom for keysstone for eg | 00:57 |
| *** zhiwei has joined #kolla | 00:57 | |
| SamYaple | yea got alot of stuff going on atm. itll be a while | 00:57 |
| sdake | ok | 00:57 |
| SamYaple | yaml is not builtin for python | 00:58 |
| SamYaple | it is a hard -2 since it will add deps to teh destination nodes | 00:58 |
| sdake | yes but ansible uses yaml on the deployment host | 00:58 |
| SamYaple | which is not where the code runs | 00:58 |
| SamYaple | we went ofver this | 00:58 |
| sdake | i know i got it | 00:59 |
| sdake | what i'm saying is the human generated part can be in yaml | 00:59 |
| sdake | if there is any to be | 00:59 |
| SamYaple | it cannot | 00:59 |
| SamYaple | it would still be parsed on the destination node | 00:59 |
| sdake | if its an argument to merge_configs ? | 01:00 |
| sdake | (not yaml but just straight arguments) | 01:00 |
| sdake | just trying to think through how you get the uid and permissions in | 01:00 |
| sdake | or if we care to specify those at all | 01:00 |
| SamYaple | well those can be assumed to be keystone (or service) and 0644 i think | 01:00 |
| SamYaple | the reason we set it at all is the container <--> host pid mapping issues | 01:01 |
| sdake | cool so we need not specify them | 01:01 |
| *** Slower has joined #kolla | 01:01 | |
| SamYaple | it shoudl be an option. im sure well need it | 01:01 |
| SamYaple | but thats not a problem | 01:01 |
| SamYaple | sdake: im sorry i missed your point | 01:03 |
| SamYaple | yea as an option from yaml into a module it should work | 01:03 |
| SamYaple | there is an issue with that in that you would need to specify lots of things and then merge the json | 01:04 |
| SamYaple | multiple files means multiple runs | 01:04 |
| sdake | multiple runs of what | 01:05 |
| SamYaple | neutron.conf ml2_conf.ini l3_agent.ini etc | 01:06 |
| SamYaple | that would spit out multiple jsons that would need to merge | 01:06 |
| SamYaple | ah ive got it | 01:07 |
| sdake | just copy the raw ini files to /var/lib/kolla an dcopy those in | 01:07 |
| sdake | all you need is the metadata | 01:07 |
| sdake | in json | 01:07 |
| SamYaple | you dont understand what im saying | 01:07 |
| SamYaple | but thats ok | 01:07 |
| SamYaple | we can do something different | 01:07 |
| SamYaple | we can have a dict we build in yaml (the defaults/main.yml for each service) and use an ansible filter to convert that to json | 01:08 |
| SamYaple | itll be simple | 01:08 |
| SamYaple | i think that makes everyone happy. good | 01:09 |
| *** dolpher has joined #kolla | 01:09 | |
| sdake | as long as the containers don't take a rocket scientiest to figure how to configure i'm happy ;) | 01:09 |
| sdake | my 845 tubes arrive tomorrow - hopefullly this time they aren'tbusted | 01:10 |
| sdake | samyaple can you ack this missed comment change - there ws actually a separate complaint bug on this problem https://review.openstack.org/#/c/210850/ | 01:15 |
| *** daneyon_ has quit IRC | 01:24 | |
| *** erkules has joined #kolla | 01:35 | |
| *** Slower has quit IRC | 01:35 | |
| *** erkules_ has quit IRC | 01:37 | |
| *** Slower has joined #kolla | 01:44 | |
| *** Slower has quit IRC | 01:49 | |
| *** erkules has quit IRC | 01:52 | |
| *** erkules has joined #kolla | 01:54 | |
| *** thumpba has joined #kolla | 02:08 | |
| *** banix has joined #kolla | 02:09 | |
| *** thumpba has quit IRC | 02:15 | |
| *** Slower has joined #kolla | 02:18 | |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Remove check.sh from linuxbridge-agent https://review.openstack.org/210951 | 02:20 |
| *** diogogmt has quit IRC | 02:30 | |
| *** diogogmt has joined #kolla | 02:42 | |
| SamYaple | sdake: how are upgrades coming along? | 03:10 |
| SamYaple | anything new to share ideas wise? | 03:10 |
| *** thumpba has joined #kolla | 03:12 | |
| *** thumpba has quit IRC | 03:17 | |
| *** unicell1 has joined #kolla | 03:19 | |
| *** unicell has quit IRC | 03:21 | |
| openstackgerrit | Sam Yaple proposed stackforge/kolla: Update nova.conf and ports https://review.openstack.org/210960 | 03:35 |
| openstackgerrit | Sam Yaple proposed stackforge/kolla: Update nova.conf and ports https://review.openstack.org/210960 | 03:37 |
| openstackgerrit | Sam Yaple proposed stackforge/kolla: Update nova.conf and ports https://review.openstack.org/210960 | 03:47 |
| openstackgerrit | Sam Yaple proposed stackforge/kolla: Update neutron confs and ports https://review.openstack.org/210962 | 04:01 |
| *** jpeeler has quit IRC | 04:10 | |
| *** unicell has joined #kolla | 04:21 | |
| *** unicell1 has quit IRC | 04:21 | |
| *** banix has quit IRC | 04:34 | |
| *** klint has joined #kolla | 04:40 | |
| *** banix has joined #kolla | 04:41 | |
| openstackgerrit | Sam Yaple proposed stackforge/kolla: Update glance confs and ports https://review.openstack.org/210969 | 04:44 |
| *** achanda has joined #kolla | 04:53 | |
| coolsvap | SamYaple, around? | 04:55 |
| *** banix has quit IRC | 04:55 | |
| SamYaple | coolsvap: always | 04:58 |
| coolsvap | SamYaple, are we moving the rabbitmq related settings to global with the update config bp? | 04:59 |
| SamYaple | what do you mean? | 04:59 |
| coolsvap | https://review.openstack.org/#/c/210969/1/ansible/roles/glance/templates/glance-api.conf.j2 | 04:59 |
| SamYaple | no | 04:59 |
| coolsvap | i see the rabbit mq configuraiton is removed | 05:00 |
| SamYaple | in the case of that its just removing them | 05:00 |
| SamYaple | look at the commit message | 05:00 |
| *** sdake has quit IRC | 05:01 | |
| coolsvap | oh okay, sorry missed it, was looking at the change set so there will be one more update for the if block | 05:01 |
| SamYaple | glance is the only time well be removing rabbit in the configs fyi | 05:01 |
| SamYaple | everything else uses it pretty heavily | 05:01 |
| SamYaple | keystone and glance only use it when ceilometer is involved | 05:01 |
| openstackgerrit | Sam Yaple proposed stackforge/kolla: Update rabbitmq confs and ports https://review.openstack.org/210971 | 05:04 |
| openstackgerrit | Swapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add ansible support for Cinder https://review.openstack.org/208868 | 05:06 |
| *** thumpba has joined #kolla | 05:14 | |
| *** thumpba has quit IRC | 05:18 | |
| openstackgerrit | Sam Yaple proposed stackforge/kolla: Update mariadb confs and ports https://review.openstack.org/210980 | 05:58 |
| openstackgerrit | Sam Yaple proposed stackforge/kolla: Update mariadb confs and ports https://review.openstack.org/210980 | 05:59 |
| *** thumpba has joined #kolla | 06:15 | |
| *** thumpba has quit IRC | 06:19 | |
| *** bmace has joined #kolla | 06:30 | |
| openstackgerrit | Sam Yaple proposed stackforge/kolla: Update mariadb confs and ports https://review.openstack.org/210980 | 06:36 |
| SamYaple | pbourke: you around? | 06:37 |
| bmace | its still a bit early for him, maybe in another hour or two. | 06:40 |
| coolsvap | yes | 06:41 |
| coolsvap | or maybe not | 06:42 |
| SamYaple | slacker | 06:42 |
| SamYaple | sleep is for the weak! | 06:42 |
| *** dolpher has quit IRC | 06:47 | |
| *** dolpher has joined #kolla | 06:48 | |
| *** dolpher1 has joined #kolla | 06:57 | |
| *** dolpher has quit IRC | 06:59 | |
| *** diga has quit IRC | 07:02 | |
| *** diga has joined #kolla | 07:03 | |
| *** thumpba has joined #kolla | 07:15 | |
| *** achanda has quit IRC | 07:16 | |
| *** achanda has joined #kolla | 07:18 | |
| *** thumpba has quit IRC | 07:20 | |
| *** mstachow has joined #kolla | 07:24 | |
| *** achanda has quit IRC | 07:28 | |
| *** achanda has joined #kolla | 07:31 | |
| *** jmccarthy has quit IRC | 07:40 | |
| *** jmccarthy has joined #kolla | 07:41 | |
| *** achanda has quit IRC | 07:46 | |
| *** gfidente has joined #kolla | 07:52 | |
| *** athomas has joined #kolla | 08:04 | |
| pbourke | morning | 08:31 |
| *** tfukushima has joined #kolla | 08:47 | |
| pbourke | SamYaple: you remember we removed the json in swift in the end | 08:50 |
| pbourke | SamYaple: its plain env variables now | 08:50 |
| *** inc0 has joined #kolla | 08:51 | |
| inc0 | hello | 08:51 |
| pbourke | looks like you guys had a busy weekend | 08:54 |
| inc0 | yeah, still going | 08:55 |
| inc0 | trying hard to setup relatively working openstack | 08:55 |
| *** vbel has quit IRC | 08:59 | |
| *** vbel has joined #kolla | 09:00 | |
| pbourke | woohoo no more min-env | 09:06 |
| coolsvap | pbourke, :) | 09:16 |
| *** thumpba has joined #kolla | 09:17 | |
| *** thumpba has quit IRC | 09:22 | |
| *** dolpher1 has quit IRC | 09:22 | |
| *** weiyu_ has joined #kolla | 09:52 | |
| openstackgerrit | Merged stackforge/kolla: Remove config-internal for Horizon https://review.openstack.org/210577 | 09:58 |
| *** zhiwei has quit IRC | 10:01 | |
| openstackgerrit | Merged stackforge/kolla: Remove Ansible abstraction for Glance https://review.openstack.org/210693 | 10:06 |
| *** mstachow has quit IRC | 10:32 | |
| *** shardy has joined #kolla | 10:41 | |
| inc0 | SamYaple, did you try latest greatest ansible 2.0 with docker? | 10:51 |
| *** pbourke has quit IRC | 10:54 | |
| SamYaple | inc0: the kolla-ansible container is running 2.0 | 10:54 |
| vbel | we have ansible 2.0 in kolla-ansible container | 10:54 |
| SamYaple | its required to make it work | 10:54 |
| *** pbourke has joined #kolla | 10:54 | |
| vbel | Thanks, SamYaple :) | 10:54 |
| SamYaple | :D | 10:55 |
| inc0 | meh, I'm getting all sorts of strange errors | 10:55 |
| SamYaple | inc0: from the contaienr? | 10:55 |
| SamYaple | i wouldn't recommend running it on the deploy host | 10:55 |
| inc0 | yeah I guess | 10:55 |
| vbel | will roles/start.yml be eliminated soon? | 10:55 |
| SamYaple | i mean im saying dont. i know its broke there | 10:55 |
| SamYaple | vbel: im still working on the patchsets | 10:55 |
| SamYaple | only keystone and glance has merged | 10:55 |
| inc0 | I'm out for few days and...kolla-ansible container?:O | 10:56 |
| vbel | I see, just wonder general trend | 10:56 |
| SamYaple | inc0: yea all the dependacies are containerized :D | 10:56 |
| *** akwasnie has joined #kolla | 11:03 | |
| *** weiyu_ has quit IRC | 11:08 | |
| *** tfukushima has quit IRC | 11:08 | |
| *** rhallisey has joined #kolla | 11:10 | |
| *** sdake has joined #kolla | 11:28 | |
| sdake | morning | 11:29 |
| inc0 | morning sdake | 11:29 |
| rhallisey | morning | 11:30 |
| *** sdake_ has joined #kolla | 11:32 | |
| *** sdake has quit IRC | 11:36 | |
| sdake_ | pbourke can you ahve a look at https://review.openstack.org/#/c/210848/ again plz | 11:37 |
| sdake_ | rhallisey can you have a look at https://review.openstack.org/#/c/210874/ | 11:41 |
| openstackgerrit | Merged stackforge/kolla: Removed config-internal check.sh for neutron https://review.openstack.org/210875 | 11:42 |
| sdake_ | and approve that review i asked pbourke to look at ? :-) | 11:42 |
| sdake_ | our builds are broken without these patches | 11:45 |
| sdake_ | jpeeler you around? | 11:46 |
| *** sdake has joined #kolla | 11:49 | |
| *** sdake_ has quit IRC | 11:52 | |
| sdake | rhallisey ping re compose | 11:59 |
| rhallisey | sdake, ya what it? | 12:04 |
| sdake | do you plan to use compose for tripleo | 12:04 |
| sdake | with config-external | 12:04 |
| rhallisey | possibly | 12:05 |
| sdake | make a decision i either want to remove it from the repo or rework it to support config-external | 12:05 |
| rhallisey | at this point it won't look much different that just a docke rurn | 12:05 |
| sdake | as is, it is broken and confusing | 12:05 |
| sdake | to do the rework is a couple hours - so rather not burn time on it if not necessary | 12:05 |
| sdake | this is my removal patch https://review.openstack.org/#/c/210694/ | 12:06 |
| sdake | that took a couple hours to make in the first place | 12:06 |
| rhallisey | I think we will. At least right now we are | 12:06 |
| sdake | ok rework it is then | 12:07 |
| openstackgerrit | Vladislav Belogrudov proposed stackforge/kolla: Ansible should wait bootstrap container on first host https://review.openstack.org/211120 | 12:07 |
| *** tfukushima has joined #kolla | 12:09 | |
| vbel | can anyone from ansible fans check ^^^ :) | 12:11 |
| *** tfukushima has quit IRC | 12:14 | |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Use docker-engine in setup_gate https://review.openstack.org/211123 | 12:16 |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Use docker-engine in setup_gate https://review.openstack.org/211123 | 12:18 |
| *** tfukushima has joined #kolla | 12:37 | |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Use docker-engine in setup_gate https://review.openstack.org/211123 | 12:37 |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Use docker-engine in setup_gate https://review.openstack.org/211123 | 12:39 |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Add centos-src-build tox gate https://review.openstack.org/211128 | 12:43 |
| openstackgerrit | Merged stackforge/kolla: Remove config-internal from galera https://review.openstack.org/210848 | 12:43 |
| openstackgerrit | Merged stackforge/kolla: Remove config-internal from ceilometer https://review.openstack.org/210849 | 12:44 |
| openstackgerrit | Merged stackforge/kolla: Clean up comment in rabbitmq https://review.openstack.org/210850 | 12:44 |
| openstackgerrit | Merged stackforge/kolla: Remove config-internal from cinder-backup https://review.openstack.org/210851 | 12:44 |
| openstackgerrit | Merged stackforge/kolla: Remove config-internal from neutron agent start files https://review.openstack.org/210852 | 12:44 |
| openstackgerrit | Merged stackforge/kolla: Remove most kolla-common.sh code https://review.openstack.org/210838 | 12:44 |
| openstackgerrit | Merged stackforge/kolla: Remove config-internal for glance https://review.openstack.org/210874 | 12:44 |
| openstackgerrit | Vladislav Belogrudov proposed stackforge/kolla: Ansible should wait bootstrap container on first host https://review.openstack.org/211120 | 13:01 |
| SamYaple | thanks vbel | 13:01 |
| *** dwalsh has joined #kolla | 13:02 | |
| vbel | SamYaple: you are welcome :) I have another bug in queue, checking/testing it. I did not specify run_once for clean-up since we could fail to do so in previous runs. So clean-up runs on all hosts | 13:03 |
| *** klint has quit IRC | 13:03 | |
| *** thumpba has joined #kolla | 13:19 | |
| *** diogogmt has quit IRC | 13:33 | |
| openstackgerrit | Vladislav Belogrudov proposed stackforge/kolla: Keystone bootstrap fails if keystone service is running https://review.openstack.org/211158 | 13:35 |
| *** jtriley has joined #kolla | 13:37 | |
| *** diogogmt has joined #kolla | 13:38 | |
| openstackgerrit | Vladislav Belogrudov proposed stackforge/kolla: Keystone bootstrap fails if keystone service is running https://review.openstack.org/211158 | 13:41 |
| *** banix_ has joined #kolla | 13:43 | |
| *** jtriley has quit IRC | 13:45 | |
| pbourke | vbel: will the new block execute on a fresh system? | 13:45 |
| *** banix_ is now known as banix | 13:46 | |
| pbourke | vbel: i.e. will it say 'cleaning up from previous run' even if there is no previous run | 13:46 |
| vbel | if no runs - no problem. We just tell ansible no keystone should run | 13:46 |
| openstackgerrit | Alicja Kwasniewska proposed stackforge/kolla: Added template for base image, added new method generating Dockerfiles from templates https://review.openstack.org/210288 | 13:47 |
| vbel | I know left over keystone is somewhat unreal but it happened for me because I wanted to re-run bootstrapping | 13:48 |
| vbel | unfortunately we don't have status from bootstrapping container. I will close the bug if we decide user must take care him/herself | 13:49 |
| sdake | vbel I think you need a different solution for https://bugs.launchpad.net/kolla/+bug/1483259 | 13:50 |
| openstack | Launchpad bug 1483259 in kolla "kestone bootstrap fails if previous keystone running" [Medium,In progress] - Assigned to Vladislav Belogrudov (vlad-belogrudov) | 13:50 |
| sdake | I left it as valid and marked it for l3, but the proper approach would be a new integration point rather then jamming into the current playbook runs | 13:50 |
| sdake | vbel my theory on software dev is if it happens in development it will happen in deployment | 13:51 |
| sdake | so we should strive to find solutions for these sorts of problems | 13:51 |
| vbel | ok, I have abandoned the change but left bug open | 13:52 |
| vbel | Thanks for reviewing! | 13:52 |
| sdake | a better approach would be to keep the review open, and ust modify the commit for the change id that way the comments are not lost | 13:53 |
| sdake | but ymmv ;-) | 13:53 |
| sdake | it would also help if you put in the bug how you got to this state | 13:54 |
| vbel | will update the bug, thanks | 13:55 |
| openstackgerrit | Alicja Kwasniewska proposed stackforge/kolla: Added template for base image, added new method generating Dockerfiles from templates https://review.openstack.org/210288 | 13:55 |
| sdake | so someone else can confirm, as per our bug triage process | 13:55 |
| *** jtriley has joined #kolla | 13:56 | |
| sdake | jtriley around? | 13:57 |
| *** jpeeler has joined #kolla | 14:08 | |
| sdake | jpeeler | 14:11 |
| sdake | any chnace you can sort out build.py in the gate | 14:11 |
| jpeeler | there a chance yes heh | 14:12 |
| jpeeler | i haven't forgotten about it though | 14:12 |
| sdake | i am working on source builds in the gate | 14:12 |
| sdake | and they need parallel to build in time | 14:12 |
| *** diogogmt has quit IRC | 14:14 | |
| sdake | akwasnie your template looks correct but seems too complex imo | 14:15 |
| sdake | from source build takes real86m17.636s | 14:16 |
| sdake | and 8 containers didn't build | 14:16 |
| openstackgerrit | Merged stackforge/kolla: Ansible should wait bootstrap container on first host https://review.openstack.org/211120 | 14:17 |
| sdake | akwasnie strive for simplicity in the templates not optimization ;-) | 14:17 |
| *** unicell1 has joined #kolla | 14:18 | |
| *** unicell has quit IRC | 14:20 | |
| sdake | samyaple does osad take 90 minutes to deploy? | 14:22 |
| *** sdake_ has joined #kolla | 14:23 | |
| *** banix has quit IRC | 14:26 | |
| *** sdake has quit IRC | 14:27 | |
| *** jtriley has quit IRC | 14:29 | |
| sdake_ | i can see why the docker cats don't put conditionals in docker building | 14:31 |
| vbel | They will end up with Makefiles :) | 14:33 |
| *** sdake has joined #kolla | 14:34 | |
| *** sdake_ has quit IRC | 14:37 | |
| sdake | jpeeler which toool did you use to determine we take up #4 spot in image sizes in the docker hub? | 14:38 |
| jpeeler | i didn't do any calculations, it was reported on the blog | 14:38 |
| jpeeler | and it was number of repos, just FYI | 14:39 |
| sdake | which blog | 14:39 |
| jpeeler | https://labs.ctl.io/docker-hub-top-10/ | 14:39 |
| sdake | looks like scratch wuld make a good data container source ;-) | 14:43 |
| *** banix has joined #kolla | 14:48 | |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: Implement Murano source images https://review.openstack.org/209905 | 14:49 |
| pbourke | hmm I wonder if we've been a bit hasty in removing config-internal for all services? | 14:53 |
| pbourke | some don't have ansible roles yet | 14:53 |
| sdake | the solution to that problem is ansible roles | 14:53 |
| pbourke | i know, but my point is some containers are now unusable | 14:54 |
| pbourke | until ansible is implemented | 14:54 |
| sdake | it is more important to have a stable abi for liberty 3 then full ansible support for supplemental services | 14:54 |
| sdake | agree we need to get cracking on the ansible implementation | 14:55 |
| pbourke | one step back, 2 forward I suppose :) | 14:55 |
| sdake | but the thing is, if you here using config-internal prior, it wasn't compatible with ansible | 14:55 |
| sdake | so you could test the container but not deployment | 14:55 |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: Fix missing config in horizon source image https://review.openstack.org/209415 | 14:55 |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: Fix missing config in horizon source image https://review.openstack.org/209415 | 14:56 |
| sdake | rhallisey pretty much need horizon asap for ansible ;-) | 14:56 |
| sdake | your assigned, you plan to tackle that soon? | 14:56 |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: Fix missing config in horizon source image https://review.openstack.org/209415 | 14:57 |
| *** blahRus has joined #kolla | 14:57 | |
| rhallisey | ya I was doing ceph atm | 14:57 |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Remove tox -egenconfig since its not needed for cinder https://review.openstack.org/211221 | 15:02 |
| sdake | pbourke which containers are you concerned with? | 15:04 |
| sdake | horizon ? | 15:04 |
| *** Verilium has quit IRC | 15:04 | |
| pbourke | sdake: I dont mind too much, fully support the config-external method | 15:04 |
| *** daneyon has joined #kolla | 15:05 | |
| pbourke | sdake: just seeing some tweaks getting left behind when throwing out config-internal that might be useful to reference when doing the ansible role | 15:05 |
| *** daneyon has quit IRC | 15:05 | |
| *** daneyon has joined #kolla | 15:06 | |
| sdake | agree, we can reference the l2 tag for that however | 15:06 |
| sdake | but if there are ansible roles you ned first lets prioritize those | 15:07 |
| sdake | since your cracking on murano which would go in supplemental, you may want horizon | 15:07 |
| *** diogogmt has joined #kolla | 15:07 | |
| sdake | https://blueprints.launchpad.net/kolla/+spec/restructure-template | 15:08 |
| sdake | pbourke ^^ | 15:10 |
| pbourke | wha's supp | 15:11 |
| pbourke | :p | 15:11 |
| sdake | supplemental is what it stands for | 15:11 |
| sdake | stuff not required for a basic operating openstack cluster | 15:12 |
| pbourke | I know just being stupid | 15:12 |
| pbourke | im assuming we want to avoid the word core? | 15:12 |
| pbourke | ah yeah I see SamYaple's comment | 15:13 |
| sdake | the top description says "basic" | 15:13 |
| sdake | i just didn't fix the cut and paste below | 15:13 |
| sdake | there fixed | 15:14 |
| *** jtriley has joined #kolla | 15:16 | |
| *** diogogmt has quit IRC | 15:18 | |
| *** diogogmt has joined #kolla | 15:19 | |
| *** shakamunyi has joined #kolla | 15:27 | |
| *** jruano has joined #kolla | 15:29 | |
| *** thumpba has quit IRC | 15:30 | |
| *** daneyon has quit IRC | 15:32 | |
| *** daneyon has joined #kolla | 15:32 | |
| *** thumpba has joined #kolla | 15:32 | |
| sdake | jtriley around? | 15:33 |
| *** SourabhP has joined #kolla | 15:35 | |
| *** thumpba has quit IRC | 15:37 | |
| *** thumpba has joined #kolla | 15:37 | |
| *** thumpba has quit IRC | 15:38 | |
| *** thumpba has joined #kolla | 15:38 | |
| *** thumpba has quit IRC | 15:40 | |
| *** thumpba has joined #kolla | 15:41 | |
| *** thumpba has quit IRC | 15:44 | |
| *** thumpba has joined #kolla | 15:45 | |
| *** jtriley has quit IRC | 15:48 | |
| *** tfukushima has quit IRC | 15:57 | |
| *** tfukushima has joined #kolla | 15:58 | |
| *** tfukushima has quit IRC | 15:58 | |
| *** tfukushima has joined #kolla | 15:58 | |
| *** unicell1 has quit IRC | 16:03 | |
| *** athomas_ has joined #kolla | 16:04 | |
| *** jtriley has joined #kolla | 16:06 | |
| *** SourabhP has quit IRC | 16:06 | |
| *** athomas has quit IRC | 16:07 | |
| *** pbourke has quit IRC | 16:08 | |
| *** pbourke has joined #kolla | 16:08 | |
| pbourke | can anyone tell me, what's the purpose of this directory? https://github.com/stackforge/kolla/tree/master/etc/kolla/config | 16:12 |
| bmace | i think you can add stuff to those and they get merged into the resulting config that gets pulled into the container? | 16:13 |
| pbourke | ah yeah, they're the augment files | 16:15 |
| sdake | right | 16:16 |
| pbourke | maybe we should put some comments in those or something | 16:17 |
| pbourke | looks a little strange having a dir full of empty files | 16:17 |
| *** jasonsb has quit IRC | 16:20 | |
| *** rhallisey has quit IRC | 16:21 | |
| *** shardy has quit IRC | 16:25 | |
| *** tfukushima has quit IRC | 16:28 | |
| pbourke | this is a bit weird :/ | 16:29 |
| pbourke | why are there like 5 config sources for each of the glance services | 16:29 |
| pbourke | https://github.com/stackforge/kolla/blob/master/ansible/roles/glance/tasks/config.yml#L6-L10 | 16:30 |
| *** athomas_ has quit IRC | 16:32 | |
| vbel | merge one into another? | 16:34 |
| pbourke | starting to see it now but it seems overkill | 16:34 |
| pbourke | one template + one augment per service would be enough | 16:34 |
| *** vbel has quit IRC | 16:36 | |
| *** unicell has joined #kolla | 16:36 | |
| *** tfukushima has joined #kolla | 16:37 | |
| *** tfukushima has quit IRC | 16:37 | |
| *** jruano has quit IRC | 16:38 | |
| sdake | pbourke database eg - you specify one database config and that merges to all services | 16:38 |
| sdake | pbourke everything gets merged into the j2 template on line 6 | 16:39 |
| sdake | iirc we shoudl have a messaging config file per service as well that loads globally | 16:39 |
| sdake | then glance.conf is the default set of config files | 16:39 |
| sdake | and glance/glance.conf.aug is an augment | 16:39 |
| sdake | not sur why samyaple has two there | 16:39 |
| sdake | global.conf iirc contains port ids and stuff | 16:40 |
| pbourke | sdake: thanks yeah I see it now. just took me longer than I expected! | 16:40 |
| pbourke | the glance.conf threw me off also | 16:40 |
| sdake | regarding comments, samyaple did say he wanted to add comments | 16:41 |
| sdake | but hasn't got to it | 16:41 |
| sdake | so glance.conf is for all glance services (a global config augment) | 16:42 |
| sdake | glance registry.conf is for registry only | 16:42 |
| sdake | can someone approve this rhallisey jpeeler https://review.openstack.org/#/c/211123/ | 16:43 |
| *** rhallisey has joined #kolla | 16:49 | |
| *** britthou_ has joined #kolla | 17:04 | |
| *** sdake_ has joined #kolla | 17:04 | |
| jpeeler | sdake: is there really a problem with the fedora build being in the gate? | 17:04 |
| *** jruano has joined #kolla | 17:05 | |
| sdake_ | jpeeler i am not sure but i am not keen on running red hat's hacked version of the gate for source builds | 17:05 |
| sdake_ | rather gate/docker | 17:06 |
| sdake_ | it definately wont work for push | 17:06 |
| sdake_ | because it asks you to confirm a push operation with [Y/n] | 17:06 |
| sdake_ | the main issue is our docs recommend using the docker inc version | 17:07 |
| sdake_ | which ends upw ith docker group vs dockerroot | 17:07 |
| sdake_ | and tox -e doen't work with docker inc's vresion | 17:07 |
| *** shardy has joined #kolla | 17:07 | |
| *** britthouser has quit IRC | 17:07 | |
| sdake_ | because we use sudo -g dockerrot vs sudo -g docker | 17:07 |
| *** sdake has quit IRC | 17:07 | |
| sdake_ | i dont get why rht has variance in docker | 17:08 |
| sdake_ | its a standard implementation ffs | 17:08 |
| jpeeler | for what it's worth, i think --confirm-def-push=false turns off the confirmation. i don't get the deviation | 17:08 |
| jpeeler | ok well, can we just the yum provided builds here instead of curling it? http://yum.dockerproject.org/repo/main/ | 17:09 |
| *** SourabhP has joined #kolla | 17:09 | |
| *** britthou_ has quit IRC | 17:10 | |
| *** jasonsb has joined #kolla | 17:10 | |
| *** shardy has quit IRC | 17:12 | |
| *** jasonsb has quit IRC | 17:14 | |
| *** jasonsb has joined #kolla | 17:14 | |
| *** sdake_ is now known as sdake | 17:14 | |
| *** achanda has joined #kolla | 17:14 | |
| *** banix has quit IRC | 17:23 | |
| sdake | interesting i was wondering how to turn tht annying feature off | 17:23 |
| sdake | jpeeler yum provided builds work for me although the nice thing about adding the repository to the gate is yum figures ou thte version # automatically | 17:24 |
| sdake | I'm unclear of our version of fedora can change or not | 17:24 |
| sdake | of/if | 17:25 |
| *** inc0 has quit IRC | 17:29 | |
| *** bmace has quit IRC | 17:31 | |
| *** dwalsh has quit IRC | 17:31 | |
| *** sdake_ has joined #kolla | 17:32 | |
| *** akwasnie has quit IRC | 17:33 | |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Add centos-src-build tox gate https://review.openstack.org/211128 | 17:34 |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Use docker-engine in setup_gate https://review.openstack.org/211123 | 17:34 |
| *** sdake has quit IRC | 17:35 | |
| *** gfidente is now known as gfidente|afk | 17:35 | |
| *** jruano has quit IRC | 17:39 | |
| *** jruano has joined #kolla | 17:43 | |
| *** britthouser has joined #kolla | 17:45 | |
| *** dwalsh has joined #kolla | 17:47 | |
| *** britthouser has quit IRC | 17:50 | |
| *** britthouser has joined #kolla | 17:50 | |
| *** bmace has joined #kolla | 17:50 | |
| *** SourabhP has quit IRC | 17:57 | |
| *** dwalsh has quit IRC | 17:58 | |
| jpeeler | hrm, i didn't realize that the docker.io bash script would set up the fedora rpm | 17:58 |
| jpeeler | i mean repo | 17:58 |
| jpeeler | will comment on the review | 17:59 |
| *** dwalsh has joined #kolla | 18:01 | |
| *** inc0 has joined #kolla | 18:04 | |
| sdake_ | its all good i did what you wanted ;) | 18:05 |
| sdake_ | if you want me to do something else just le tme know | 18:05 |
| sdake_ | rhallisey do you mind if I take horizon blueprint? | 18:05 |
| sdake_ | rhallisey you have 12 bueprints, and no offense, but your not superman, I don't have high confidence youc an get em all done ;) | 18:05 |
| rhallisey | sdake_, ya sure if you have work for it | 18:05 |
| rhallisey | I'm like 1/2 way maybe | 18:05 |
| sdake_ | can you post your review | 18:06 |
| sdake_ | and i'll take it over | 18:06 |
| rhallisey | kk | 18:06 |
| sdake_ | i just want to get cracking on stuff | 18:06 |
| rhallisey | I'm not super man | 18:06 |
| sdake_ | dont want us single threading | 18:06 |
| sdake_ | i am looking for work | 18:06 |
| sdake_ | i am in the process of pushing stuff to mitaka as well | 18:06 |
| sdake_ | we can discuss at the team meeting - also expect an email on the ml about it and priorities | 18:06 |
| sdake_ | we barely got 20 blueprints finished last time | 18:07 |
| sdake_ | and our community hasn't grown since last cycle by more then 1 or 2 people | 18:07 |
| sdake_ | and we have more bug testing/filing going on ;) | 18:08 |
| *** jtriley has quit IRC | 18:08 | |
| sdake_ | jpeeler are you working on ansibilizing ironic | 18:08 |
| sdake_ | or just making a container for it | 18:09 |
| jpeeler | i thought you wanted me to do both | 18:09 |
| jpeeler | but i don't know ansible, so if you want to do that that's fine | 18:09 |
| sdake_ | if you think you can tackle it wfm | 18:09 |
| *** banix has joined #kolla | 18:09 | |
| sdake_ | i dont want to do th eansible part no | 18:09 |
| *** britthou_ has joined #kolla | 18:09 | |
| sdake_ | we have 3 weeks, just looking to unload work to mitaka | 18:09 |
| *** inc0 has quit IRC | 18:10 | |
| openstackgerrit | Ryan Hallisey proposed stackforge/kolla: Add ansible support for Horizon https://review.openstack.org/211286 | 18:10 |
| sdake_ | wonder what ever happened to fang | 18:10 |
| sdake_ | hope he is ok | 18:10 |
| rhallisey | sdake_, ^ go ahead | 18:10 |
| rhallisey | was dealing with httpd config | 18:10 |
| sdake_ | thanks rhallisey | 18:11 |
| sdake_ | you actually ahve a bunch of work done ;) | 18:11 |
| sdake_ | nice job | 18:11 |
| rhallisey | ya I put down getting ceph cluster going | 18:11 |
| rhallisey | now I can go back to that! | 18:11 |
| *** britthou_ has quit IRC | 18:12 | |
| *** jtriley has joined #kolla | 18:12 | |
| *** britthouser has quit IRC | 18:12 | |
| sdake_ | oh sorry | 18:14 |
| sdake_ | if you want to finish the job on horizon that wfm | 18:14 |
| sdake_ | i am just tyring to unload your plate :) | 18:14 |
| sdake_ | i'd like folks to hve no more then 6 blueprints if possible | 18:15 |
| *** britthouser has joined #kolla | 18:16 | |
| harmw | i'd just like more time in a day, think you can get me that sdake_ ? | 18:20 |
| rhallisey | sdake_, I don't care | 18:20 |
| sdake_ | harmw cant work that magic - you figure out how to make the moon on a greater orbit, i'll do it ;) | 18:20 |
| harmw | got some other projects that need my attention, hence my absence | 18:20 |
| sdake_ | harmw ya very few kolla devs are f ull time | 18:20 |
| harmw | yea but even then | 18:21 |
| sdake_ | but we appreciate your part time help ;) | 18:21 |
| harmw | :) | 18:21 |
| *** britthouser has quit IRC | 18:21 | |
| sdake_ | especially reviews becuase our core reviewer team is too small | 18:21 |
| harmw | did I mention I realy hate building websites? | 18:21 |
| sdake_ | we need atleaset 1 more cat imo | 18:21 |
| sdake_ | is that what you do for a living? | 18:21 |
| harmw | well I do try to keep reviewing :) | 18:21 |
| sdake_ | harmw to be honest I dont see why you dont get an openstack job dude | 18:21 |
| harmw | no, god no | 18:21 |
| harmw | uhm, websites building is spare time | 18:22 |
| sdake_ | your an openstack developer and have good karma in the commnity ;) | 18:22 |
| sdake_ | did I mention I hate computers ;-) | 18:22 |
| harmw | openstackjob... maybe someday :) | 18:22 |
| harmw | lol | 18:22 |
| sdake_ | when does mandre reappear :) | 18:22 |
| * sdake_ wtb's mandre | 18:22 | |
| rhallisey | end of August | 18:22 |
| rhallisey | like 1 week | 18:22 |
| harmw | you guys heard of softwarecircus.io btw? | 18:23 |
| sdake_ | rhallisey I cranked up ceph to essential, I really want that done for liberty | 18:23 |
| rhallisey | sdake_, kk I'll switch my attention to that unless you want me to take horizon back | 18:24 |
| sdake_ | nah i'll take horizon | 18:24 |
| rhallisey | k | 18:24 |
| sdake_ | i kicked magnum to mitaka | 18:24 |
| sdake_ | that is what i was going to work on next | 18:24 |
| sdake_ | while also working on source gating ;) | 18:24 |
| sdake_ | down to a manageble 24 blueprints ;) | 18:27 |
| rhallisey | I need to fix ansible cinder to get this working since config internal is gone | 18:30 |
| rhallisey | coolsvap, you cool if I send you patches for cinder-ansible? | 18:31 |
| rhallisey | oh nvm looks like you made some changes | 18:32 |
| *** SourabhP has joined #kolla | 18:32 | |
| *** sdake_ is now known as sdake | 18:35 | |
| sdake | rhallisey if you https://blueprints.launchpad.net/kolla/+spec/ceph-container have started on that mark it started plz | 18:39 |
| rhallisey | sdake, yup, all set | 18:40 |
| sdake | rhallisey you will be doing ansible for ceph as well? | 18:40 |
| rhallisey | ya I'll handle it | 18:40 |
| rhallisey | I'm going to try and help finish off cinder ansible then move to that | 18:41 |
| sdake | rhallisey do you need the data containers for tripleo? | 18:43 |
| rhallisey | no, I just thought we needed them in general | 18:44 |
| *** gfidente|afk is now known as gfidente | 18:44 | |
| *** jruano has quit IRC | 18:44 | |
| openstackgerrit | Jeff Peeler proposed stackforge/kolla: Improve logging and prepare for gate https://review.openstack.org/202216 | 18:47 |
| jpeeler | sdake: prerequisite to getting build.py in gate ^ | 18:47 |
| sdake | jpeeler let me look at the last review and make sure the issues have been solved | 18:48 |
| sdake | it will be after i finish the job on this launchpad pounding | 18:48 |
| sdake | so give me 1 hour or so | 18:49 |
| jpeeler | no rush | 18:52 |
| *** jruano has joined #kolla | 18:52 | |
| rhallisey | coolsvap, around? | 18:52 |
| *** bmace has quit IRC | 18:55 | |
| sdake | rhallisey if you dont need them for tirpleo, i am goin to remove them | 18:56 |
| rhallisey | all the data containers? | 18:56 |
| sdake | and you can use the api container with a different name for the data container | 18:56 |
| rhallisey | are we going to have one? | 18:56 |
| sdake | thats the plan | 18:56 |
| sdake | I want to lock down the container ABI for liberty | 18:56 |
| sdake | after liberty I'd like minimal change unless we fubared something | 18:57 |
| sdake | that way tripleo has a stable base for liberty/mitaka to get the job done | 18:57 |
| rhallisey | k sounds good | 18:59 |
| *** britthouser has joined #kolla | 19:02 | |
| sdake | if we fubared something we will change it | 19:03 |
| sdake | but its getting down to very basic container content now | 19:03 |
| sdake | oru containers copy a file in and install packages | 19:04 |
| sdake | can't get much more simple then that | 19:04 |
| sdake | rhallisey note I changed https://blueprints.launchpad.net/kolla/+spec/ceph-container | 19:05 |
| rhallisey | noted | 19:05 |
| *** britthouser has quit IRC | 19:26 | |
| *** britthouser has joined #kolla | 19:26 | |
| *** sdake_ has joined #kolla | 19:29 | |
| *** bmace has joined #kolla | 19:31 | |
| *** sdake has quit IRC | 19:33 | |
| *** sdake has joined #kolla | 19:35 | |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Add centos-src-build tox gate https://review.openstack.org/211128 | 19:36 |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Use docker-engine in setup_gate https://review.openstack.org/211123 | 19:36 |
| *** britthouser has quit IRC | 19:36 | |
| sdake | ok jpeeler looking at your compelx review now ;-) | 19:37 |
| *** sdake_ has quit IRC | 19:38 | |
| bmace | whats up sdake? :) | 19:39 |
| *** loth has left #kolla | 19:39 | |
| *** britthouser has joined #kolla | 19:40 | |
| *** echoingumesh has joined #kolla | 19:43 | |
| *** echoingu_ has joined #kolla | 19:44 | |
| sdake | jpeeler can you expalin the threads thing plz | 19:45 |
| sdake | i dont get why you special case threads=1 | 19:45 |
| *** echoingumesh has quit IRC | 19:47 | |
| jpeeler | sdake: if you specify just one thread, that allows realtime output from the docker building | 19:49 |
| sdake | oh didn't know that | 19:49 |
| sdake | you mean each docker operation prints out? | 19:50 |
| sdake | or just the completed docker op | 19:50 |
| jpeeler | due to the thread handling, that's not possible with multiple builds occurring. so for each build, when multiple threads are in use the logs are not outputted until completely finished | 19:50 |
| sdake | I would expect line 92 to be in a loop for tha t to work? | 19:50 |
| sdake | for each item in the log to be printed | 19:51 |
| sdake | rather each utput of each docker operation | 19:51 |
| sdake | i see its in a for loop | 19:52 |
| sdake | got it | 19:52 |
| jpeeler | yeah | 19:52 |
| sdake | I just assume that works - have no idea how they actually implemented that in python ;-) | 19:52 |
| jpeeler | last time i tested it it did | 19:53 |
| sdake | cool well i -1'ed your review | 19:53 |
| sdake | but not for that reason | 19:53 |
| sdake | that is a good improvement | 19:53 |
| sdake | I recommend a comment | 19:53 |
| sdake | and adding something to indicate running with threads=1 will provide realtime output | 19:53 |
| sdake | that could come in handy | 19:53 |
| sdake | (to the hep of -T) | 19:53 |
| sdake | help of that is | 19:54 |
| *** jruano has quit IRC | 19:55 | |
| *** echoingu_ has quit IRC | 19:57 | |
| *** echoingumesh has joined #kolla | 19:59 | |
| *** dwalsh has quit IRC | 20:41 | |
| *** echoingumesh has quit IRC | 20:46 | |
| *** jasonsb has quit IRC | 20:46 | |
| *** jasonsb has joined #kolla | 21:01 | |
| openstackgerrit | Jeff Peeler proposed stackforge/kolla: Improve logging and prepare for gate https://review.openstack.org/202216 | 21:02 |
| * jpeeler is slow today | 21:02 | |
| openstackgerrit | Ryan Hallisey proposed stackforge/kolla: Add ansible support for Cinder https://review.openstack.org/208868 | 21:05 |
| *** achanda has quit IRC | 21:07 | |
| *** mfalatic has joined #kolla | 21:09 | |
| *** thumpba has quit IRC | 21:12 | |
| rhallisey | jpeeler, it's better than being Slower | 21:15 |
| rhallisey | :P | 21:15 |
| *** bmace has quit IRC | 21:16 | |
| *** rhallisey has quit IRC | 21:18 | |
| *** bmace has joined #kolla | 21:33 | |
| *** achanda has joined #kolla | 21:40 | |
| *** bmace has quit IRC | 21:47 | |
| *** tfukushima has joined #kolla | 21:47 | |
| *** echoingumesh has joined #kolla | 21:54 | |
| *** echoingumesh has quit IRC | 21:59 | |
| *** bmace has joined #kolla | 21:59 | |
| *** sdake has quit IRC | 22:01 | |
| *** echoingumesh has joined #kolla | 22:02 | |
| *** jtriley_ has joined #kolla | 22:07 | |
| *** jtriley has quit IRC | 22:10 | |
| *** jtriley_ has quit IRC | 22:11 | |
| *** echoingumesh has quit IRC | 22:28 | |
| *** banix has quit IRC | 22:50 | |
| *** diogogmt has quit IRC | 22:56 | |
| *** jtriley has joined #kolla | 22:59 | |
| *** jtriley has quit IRC | 23:11 | |
| *** echoingumesh has joined #kolla | 23:19 | |
| *** jasonsb has quit IRC | 23:30 | |
| *** jasonsb has joined #kolla | 23:31 | |
| *** sdake has joined #kolla | 23:31 | |
| *** jtriley has joined #kolla | 23:31 | |
| *** tfukushima has quit IRC | 23:32 | |
| sdake | yo | 23:34 |
| bmace | yo indeed | 23:34 |
| *** jasonsb has quit IRC | 23:35 | |
| SamYaple | hello | 23:42 |
| *** blahRus has quit IRC | 23:42 | |
| SamYaple | sdake: i have a ping from you but cant find it in scrollback | 23:42 |
| sdake | samyaple no idea bro | 23:43 |
| sdake | probably no tworth tlakingabout ;) | 23:43 |
| sdake | nothing i can recall I wanted to discuss | 23:44 |
| *** daneyon has quit IRC | 23:44 | |
| sdake | so have physical therapy 3 days a week groan | 23:44 |
| *** daneyon has joined #kolla | 23:44 | |
| sdake | so i'll be out mwf likely for a couple hours | 23:45 |
| SamYaple | lol ok | 23:47 |
| *** tfukushima has joined #kolla | 23:52 | |
| *** sdake has quit IRC | 23:53 | |
| openstackgerrit | Sam Yaple proposed stackforge/kolla: Update nova.conf and ports https://review.openstack.org/210960 | 23:55 |
| *** sdake has joined #kolla | 23:56 | |
| *** tfukushima has quit IRC | 23:57 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!