| *** juggler has joined #kolla | 00:01 | |
| *** jasonsb has joined #kolla | 00:06 | |
| *** diogogmt_ has joined #kolla | 00:12 | |
| *** dims_ has quit IRC | 00:12 | |
| *** diogogmt has quit IRC | 00:12 | |
| *** diogogmt_ is now known as diogogmt | 00:12 | |
| sdake | jpeeler if your around can you ack that last change | 00:13 |
|---|---|---|
| *** tfukushima has joined #kolla | 00:19 | |
| *** tfukushima has quit IRC | 00:23 | |
| *** dolpher has joined #kolla | 00:32 | |
| *** banix has joined #kolla | 00:32 | |
| *** tfukushima has joined #kolla | 00:36 | |
| *** diogogmt has quit IRC | 00:37 | |
| *** achanda has quit IRC | 00:58 | |
| *** dims has joined #kolla | 01:04 | |
| *** ccrouch has quit IRC | 01:05 | |
| *** weiyu_ has joined #kolla | 01:13 | |
| *** sdake_ has joined #kolla | 01:20 | |
| *** sdake has quit IRC | 01:24 | |
| *** erkules has joined #kolla | 01:37 | |
| *** erkules_ has quit IRC | 01:40 | |
| *** diogogmt has joined #kolla | 01:42 | |
| *** zhiwei has joined #kolla | 01:43 | |
| *** banix has quit IRC | 01:44 | |
| *** diogogmt has quit IRC | 01:48 | |
| *** diogogmt has joined #kolla | 01:49 | |
| *** erkules has quit IRC | 01:52 | |
| *** daneyon_ has quit IRC | 01:53 | |
| *** erkules has joined #kolla | 01:55 | |
| *** banix has joined #kolla | 01:56 | |
| *** tfukushima has quit IRC | 02:00 | |
| *** tfukushima has joined #kolla | 02:00 | |
| *** maximedev has joined #kolla | 02:00 | |
| maximedev | Hi guys | 02:00 |
| *** achanda has joined #kolla | 02:02 | |
| *** tfukushima has quit IRC | 02:04 | |
| *** tummy has joined #kolla | 02:04 | |
| sdake_ | sup maximedev | 02:12 |
| *** sdake_ is now known as sdake | 02:12 | |
| *** vern has joined #kolla | 02:17 | |
| *** Haomeng|2 is now known as Haomeng | 02:41 | |
| *** dims has quit IRC | 02:55 | |
| sdake | bmace found this gem http://events.linuxfoundation.org/sites/events/files/slides/CloudOpen2015.pdf | 02:56 |
| *** alisonh has joined #kolla | 02:57 | |
| *** alisonh has quit IRC | 03:03 | |
| *** unicell has quit IRC | 03:11 | |
| *** diogogmt has quit IRC | 03:13 | |
| *** loth has quit IRC | 03:17 | |
| *** alisonh has joined #kolla | 03:21 | |
| *** achanda has quit IRC | 03:28 | |
| *** diga has joined #kolla | 03:33 | |
| SamYaple | sdake: you didnt change your vote https://review.openstack.org/#/c/215157/ | 03:38 |
| sdake | i wasn't asked to do so | 03:39 |
| sdake | ou asked on a different review which i did | 03:39 |
| sdake | where i made an error in the review | 03:39 |
| SamYaple | I responded to your comments in the review and said they were not accurate | 03:39 |
| sdake | ok well i didn't see that | 03:39 |
| sdake | i'll check now hang tight | 03:39 |
| sdake | samyaple done enjoy | 03:41 |
| sdake | thanks for the info about build-essential | 03:41 |
| sdake | I don't know alot about ubuntu so its hard for me to evaluate package deletions | 03:41 |
| SamYaple | yea feel free to comment on them, but ths is all tested stuff | 03:43 |
| sdake | i am mainly concerned about breaking centos-binary | 03:43 |
| sdake | but also trying to help you with the reviews on the ubuntu bits | 03:44 |
| sdake | like postgresql-devel I doubt that is in essential but i guess itdoesn't matter ;) | 03:44 |
| SamYaple | yea there are no changes to centos-binary | 03:44 |
| SamYaple | some new if blocks, but same end result for centos | 03:44 |
| sdake | in one of the reviews there was something | 03:44 |
| sdake | i dont recall which | 03:44 |
| SamYaple | no that was about rhel | 03:44 |
| SamYaple | not centos-binary | 03:44 |
| SamYaple | we dont do rhel anywhere | 03:45 |
| SamYaple | mostly | 03:45 |
| SamYaple | but i still fixed that | 03:45 |
| sdake | ya that was just an ask | 03:45 |
| sdake | because i can't remmber everything | 03:45 |
| sdake | old man syndrome :) | 03:45 |
| *** alisonh has quit IRC | 03:54 | |
| *** alisonh has joined #kolla | 03:59 | |
| sdake | samyaple if your interested in seeing how kolla got kicked off, have a look at this video https://www.youtube.com/watch?v=rZBkCovEIZI | 04:18 |
| *** dolpher has quit IRC | 04:20 | |
| *** achanda has joined #kolla | 04:20 | |
| sdake | sam re the redhat comment i just left | 04:22 |
| sdake | what i meant to say is red hat has guidelines about how their trademakr is to be ussd | 04:22 |
| sdake | and I don't know if "RedHat family" would fit their guideliens | 04:23 |
| sdake | and RPM is just easier way for me not to have to deal with finding out :) | 04:23 |
| sdake | but just fyi its Red Hat, not RedHat, they have a shit fit whenever someone says RedHat in legal in any press or code | 04:23 |
| sdake | the safest way to refer to red hat is "Red Hat, Inc." | 04:25 |
| Kennan | hi sdake: I am trying kolla on ubuntu, it said need kernel 3.19 but seems default can not get such version,/ | 04:26 |
| sdake | kennan you need utopic | 04:26 |
| *** weiyu_ has quit IRC | 04:26 | |
| sdake | kennan samyaple is your go to guy for getting rollign on ubuntu | 04:27 |
| sdake | perosnally i'd recommend running in a centos vm | 04:27 |
| Kennan | sdake: utopic seems kernel 3.16 or 3.14 | 04:27 |
| Kennan | it not 3.19 | 04:27 |
| sdake | kennan shows what I know, SamYaple said Utopic was 3.19 | 04:28 |
| sdake | samyaple with that recent change you made to the config dir, do all config-external files need to change | 04:29 |
| sdake | kennan samyaple is about but he is just getting his workday started so he may take a bit to respond | 04:30 |
| Kennan | ok sdake: thanks. for centos | 04:30 |
| Kennan | what version do you recommend? | 04:30 |
| sdake | centos 7 is minimum required version | 04:31 |
| Kennan | ok. sdake: let me find a centos 7 if possible | 04:31 |
| sdake | its a free iso download | 04:31 |
| sdake | kennan diversity results of the big tent: http://ur1.ca/ni02n -> http://paste.fedoraproject.org/257427/31574144 | 04:33 |
| Kennan | interesting sadke: | 04:39 |
| *** banix has quit IRC | 04:40 | |
| *** weiyu_ has joined #kolla | 04:52 | |
| *** sdake_ has joined #kolla | 04:52 | |
| *** sdake has quit IRC | 04:56 | |
| *** unicell has joined #kolla | 04:58 | |
| *** SourabhP has joined #kolla | 04:59 | |
| *** SourabhP_ has joined #kolla | 05:00 | |
| *** SourabhP has quit IRC | 05:04 | |
| *** SourabhP_ is now known as SourabhP | 05:04 | |
| *** unicell has quit IRC | 05:05 | |
| *** unicell has joined #kolla | 05:06 | |
| sdake_ | samyaple pbourke harmw wtb reviews on https://review.openstack.org/#/c/213537/ | 05:06 |
| sdake_ | its blocking the rest of the gating going live | 05:07 |
| *** weiyu_ has quit IRC | 05:10 | |
| *** absubram has quit IRC | 05:16 | |
| *** sdake_ has quit IRC | 05:23 | |
| SamYaple | Kennan: hello | 05:24 |
| SamYaple | Kennan: did you have a question about the kernel? | 05:24 |
| SamYaple | Kennan: i see you have a question about kernel 3.19, that is only if you are using AUFS | 05:25 |
| Kennan | hi SamYaple | 05:25 |
| Kennan | I tried to figure out what's requirements for ubuntu | 05:25 |
| Kennan | it said it needs kernel version 3.19 | 05:26 |
| SamYaple | What version of ubuntu? | 05:26 |
| Kennan | it is too higher | 05:26 |
| Kennan | https://wiki.ubuntu.com/UtopicUnicorn/ReleaseNotes | 05:26 |
| Kennan | it is 3.16 for 14.10 | 05:26 |
| SamYaple | Kennan: I would not recommend 14.10, since it is not an LTS. But in that case your only option is btrfs backing for Docker | 05:26 |
| Kennan | hi SamYaple: | 05:27 |
| Kennan | for 14.04 | 05:27 |
| SamYaple | in 14.04 you can install 3.19 kernel and use AUFS, but btrfs is still faster | 05:27 |
| Kennan | if I use btrfs, | 05:27 |
| Kennan | how to configure that in kolla ? | 05:29 |
| Kennan | I did not know where can configure to use btrfs | 05:29 |
| SamYaple | its docker that you would need t oconfigure | 05:29 |
| SamYaple | you dont need to configure anything for kolla | 05:29 |
| SamYaple | Kolla doesn't care about your Docker backend | 05:29 |
| Kennan | hi SamYaple so you means docker community have docs how to configure with btrfs ? | 05:30 |
| SamYaple | Yes Kennan | 05:30 |
| SamYaple | it is refered to as a docker 'storage driver' | 05:31 |
| SamYaple | So you can look up how to configure the docker storage driver for btrfs | 05:31 |
| Kennan | hi SamYaple what backend do you use for ubuntu ? | 05:32 |
| SamYaple | I use BTRFS and ZFS equally | 05:34 |
| SamYaple | but with the 3.19 kernel for Btrfs | 05:34 |
| Kennan | hi SamYaple so you compile kernel in ubuntu ? | 05:43 |
| *** tfukushima has joined #kolla | 05:45 | |
| *** dims has joined #kolla | 05:47 | |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Convert MD to RST to publish to docs.openstack.org https://review.openstack.org/215458 | 05:48 |
| *** tfukushima has quit IRC | 05:50 | |
| *** zhiwei has quit IRC | 05:52 | |
| *** dims has quit IRC | 05:52 | |
| *** zhiwei has joined #kolla | 05:53 | |
| *** dims has joined #kolla | 06:01 | |
| *** dims has quit IRC | 06:05 | |
| *** weiyu_ has joined #kolla | 06:06 | |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Prepare for MD to RST conversion https://review.openstack.org/215463 | 06:07 |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Run pandoc to convert the documentation https://review.openstack.org/215464 | 06:07 |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Run pandoc to convert the documentation https://review.openstack.org/215464 | 06:16 |
| *** inc0 has joined #kolla | 06:20 | |
| inc0 | morning | 06:20 |
| inc0 | we don't have single container build yet do we? | 06:20 |
| *** SourabhP has quit IRC | 06:26 | |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Run pandoc to convert the documentation https://review.openstack.org/215464 | 06:45 |
| *** tfukushima has joined #kolla | 06:46 | |
| *** tfukushima has quit IRC | 06:50 | |
| *** diga has quit IRC | 07:05 | |
| *** tfukushima has joined #kolla | 07:12 | |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Run pandoc to convert the documentation https://review.openstack.org/215464 | 07:14 |
| *** tfukushima has quit IRC | 07:16 | |
| *** jmccarthy has quit IRC | 07:24 | |
| *** jmccarthy has joined #kolla | 07:25 | |
| *** shardy has joined #kolla | 07:27 | |
| Haomeng | hi guys | 07:28 |
| Haomeng | I run vargant up to setup env, encountered this error - "The box 'puppetlabs/centos-7.0-64-puppet' could not be found.", any comments? | 07:29 |
| Haomeng | should I download this box first? | 07:34 |
| *** achanda has quit IRC | 07:35 | |
| *** zhiwei has quit IRC | 07:37 | |
| *** zhiwei has joined #kolla | 07:37 | |
| Kennan | HaoMeng which guide do you refer ? | 07:38 |
| Haomeng | Kennan: thanks for your helping | 07:38 |
| Haomeng | Kennan: this guide - https://github.com/stackforge/kolla/blob/master/docs/vagrant.md | 07:38 |
| Haomeng | Kennan: I think some pre-action should be taken | 07:38 |
| Haomeng | Kennan: but it is not covered by the guide | 07:39 |
| Kennan | do you use a physical machine to setup? | 07:39 |
| Haomeng | Kennan: I run on VM, will not working with vm? | 07:40 |
| Haomeng | Kennan: I installed vagrant and virtualbox | 07:40 |
| Kennan | I think vagrant can not work with VM well | 07:40 |
| Kennan | it seems need install on physical machine for vagrant to work well. You may need to check with SamYaple | 07:41 |
| Kennan | vagrant sometimes up action need rely on VT intel or AMD virtuail | 07:41 |
| Kennan | but VM not have such capability | 07:41 |
| Haomeng | Kennan: yes, but we can enable vm nested if it supported | 07:42 |
| Kennan | did you check if vagrant support that nested ? | 07:42 |
| Kennan | I did not think how can enable such nested VT | 07:42 |
| Haomeng | Kennan: another question, I just want to play with kolla to deploy openstack, so, vagrant is not nesserary | 07:42 |
| Haomeng | Kennan: ok | 07:42 |
| Haomeng | Kennan: so which guide is easy for me to try kolla first | 07:43 |
| Haomeng | Kennan: https://github.com/stackforge/kolla/tree/master/devenv , is this fine for my fist try? | 07:44 |
| Kennan | HaoMeng: I ma also new to kolla | 07:44 |
| Kennan | yes | 07:44 |
| Kennan | https://github.com/stackforge/kolla/blob/master/docs/dev-quickstart.md | 07:44 |
| Kennan | could be used , I remember sdake said that before | 07:44 |
| Haomeng | Kennan: ok, will try, thank you | 07:44 |
| Haomeng | Kennan: ok | 07:44 |
| Kennan | wcl | 07:44 |
| Haomeng | Kennan: the reason I use vargant, that because I think it should be more easier:) | 07:45 |
| *** zhiwei has quit IRC | 08:03 | |
| openstackgerrit | Michal Jastrzebski (inc0) proposed stackforge/kolla: build.py refactor https://review.openstack.org/215504 | 08:05 |
| *** diga has joined #kolla | 08:17 | |
| openstackgerrit | Mick Thomspon proposed stackforge/kolla: Added missing configuration to horizon docker template. https://review.openstack.org/215511 | 08:29 |
| *** inc0 has quit IRC | 08:35 | |
| *** inc0_ has joined #kolla | 08:35 | |
| *** inc0__ has joined #kolla | 08:37 | |
| *** inc0_ has quit IRC | 08:40 | |
| *** inc0_ has joined #kolla | 08:41 | |
| *** inc0__ has quit IRC | 08:45 | |
| *** tfukushima has joined #kolla | 08:45 | |
| *** diga has quit IRC | 08:49 | |
| *** inc0_ has quit IRC | 08:50 | |
| *** inc0_ has joined #kolla | 08:50 | |
| openstackgerrit | Merged stackforge/kolla: Workaround for existing keepalived.pid file https://review.openstack.org/215380 | 09:03 |
| openstackgerrit | Michal Jastrzebski (inc0) proposed stackforge/kolla: Build.py regex filtering https://review.openstack.org/215523 | 09:03 |
| openstackgerrit | Michal Jastrzebski (inc0) proposed stackforge/kolla: Build single image https://review.openstack.org/215524 | 09:03 |
| inc0_ | there you go, we all needed that. | 09:04 |
| *** vbel has quit IRC | 09:07 | |
| *** vbel has joined #kolla | 09:07 | |
| pbourke | inc0_: are your patches alternatives to jpeeler's? | 09:10 |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Run pandoc to convert the documentation https://review.openstack.org/215464 | 09:10 |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Prepare for MD to RST conversion https://review.openstack.org/215463 | 09:10 |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Rework documentaton to actually get people rolling https://review.openstack.org/215532 | 09:10 |
| inc0_ | pbourke, I haven't seen Jeffs patches | 09:11 |
| inc0_ | I just need this to work and it took me less than an hour | 09:11 |
| inc0_ | I'm in "fix all my painpoints with build script" frenzy right now | 09:11 |
| pbourke | sure thing | 09:11 |
| openstackgerrit | Merged stackforge/kolla: Add build from source and templating for Zaqar https://review.openstack.org/202441 | 09:11 |
| pbourke | just fyi he does have a patch | 09:11 |
| *** sdake has joined #kolla | 09:11 | |
| pbourke | so will have to choose | 09:11 |
| pbourke | maybe you could review his and see how it compares | 09:12 |
| pbourke | after the single image building my next biggest pain point would be the logging | 09:12 |
| pbourke | but im not sure how that could be solved with the paralell building | 09:12 |
| pbourke | multiple log files might be nice | 09:13 |
| pbourke | then you could tail them in separate windows | 09:13 |
| sdake | hey guys | 09:13 |
| sdake | docs are my biggest pain point :) | 09:13 |
| *** weiyu_ has quit IRC | 09:14 | |
| sdake | whats going on | 09:14 |
| inc0_ | pbourke, is this patch upstream already? | 09:14 |
| pbourke | inc0_: no its for review | 09:14 |
| inc0_ | ah, review | 09:14 |
| pbourke | https://review.openstack.org/#/c/215323/ | 09:14 |
| inc0_ | I must have missed it | 09:14 |
| pbourke | sdake: roger your comment on swift docs | 09:14 |
| pbourke | sdake: was thinking the same, will put something together | 09:14 |
| sdake | pbourke if yu get it into basic shape like a 10-15 liner i can make it int othe docs | 09:15 |
| sdake | hey inc0 jpeeler wrote tht acode already | 09:15 |
| inc0_ | logic is a bit different but we'll compare | 09:15 |
| inc0_ | hold on, next patch coming right up | 09:15 |
| inc0_ | (different one) | 09:16 |
| openstackgerrit | Mick Thomspon proposed stackforge/kolla: Added missing configuration to horizon docker template. https://review.openstack.org/215511 | 09:16 |
| Kennan | sdake: not sleep ? | 09:16 |
| sdake | god i need to be kennan | 09:17 |
| sdake | kennan try readin g the last commit for docs | 09:17 |
| sdake | it shoud lget you going | 09:17 |
| sdake | the one in the queue | 09:17 |
| Kennan | sdake: do you know why libvirt is needed in kolla ? | 09:17 |
| Kennan | Finally stop libvirt on the host machine. Only one copy of libvirt may be running at a time. | 09:17 |
| Kennan | I did not understand that | 09:18 |
| sdake | https://review.openstack.org/#/c/215532/ | 09:18 |
| sdake | libvirt runs in a container | 09:18 |
| sdake | if it were to run outside the container we coudln't do upgrades properly | 09:18 |
| sdake | nor support a feasible deployment model with orrect sharingof data with nova | 09:18 |
| Kennan | ok. you make libvirt run in a container | 09:19 |
| sdake | right | 09:19 |
| Kennan | so it can not run on the host | 09:19 |
| Kennan | right? | 09:19 |
| sdake | right | 09:19 |
| Kennan | ok got it. Thanks | 09:19 |
| sdake | that is why you have to turn it off on the host | 09:19 |
| sdake | the new docs are more clear on this point | 09:19 |
| sdake | coolsvap|away where u at bro :) | 09:21 |
| sdake | need reviews!! | 09:21 |
| sdake | kennan id you get aio going yet? | 09:22 |
| sdake | one thing missing from current master docs is the idea of building imges | 09:22 |
| sdake | you need to do that | 09:22 |
| sdake | its in my docs update | 09:22 |
| *** stvnoyes has joined #kolla | 09:23 | |
| Kennan | sdake: ok still fighting for that. just install kolla client now (as some other work in hands, did not do it all the time :)) | 09:23 |
| sdake | i think realistically a kolla evaluation should take less then 30 minutes | 09:23 |
| sdake | our heat and vagrant documentation need love | 09:24 |
| sdake | the heat documentation is focused around compose still | 09:24 |
| * sdake groans | 09:24 | |
| openstackgerrit | Michal Jastrzebski (inc0) proposed stackforge/kolla: Add custom code to base docker templates https://review.openstack.org/215541 | 09:24 |
| inc0_ | as I said, solve all the painpoints | 09:24 |
| pbourke | inc0_: does the regex patch not solve the single case as well? | 09:25 |
| pbourke | do we need both? | 09:25 |
| inc0_ | regex code builds deps | 09:25 |
| inc0_ | and I wanted to have option to skip building deps alltogether | 09:25 |
| Kennan | sdake: I used quick start guide. | 09:25 |
| inc0_ | and just do this single image with no regard of anything | 09:25 |
| Kennan | I only have VM in hands now, in your doc it seems need bare-metal | 09:26 |
| inc0_ | regex code is useful to "build me latest neutron" use case | 09:26 |
| pbourke | inc0_: but the cache generally makes it not an issue | 09:26 |
| pbourke | if deps don't need to be rebuilt they wont be | 09:26 |
| Kennan | is it a must requirements to meet ? | 09:26 |
| Kennan | sdake: | 09:26 |
| sdake | kennan vm will work | 09:26 |
| inc0_ | pbourke, it shoulding, but if I have custom base image, which I do (proxy and such) | 09:26 |
| inc0_ | this keeps biting me | 09:26 |
| sdake | i like hat include header change | 09:26 |
| pbourke | im not convinced :/ | 09:27 |
| inc0_ | pbourke, with include header this all will be solvable | 09:27 |
| pbourke | great work all the same | 09:27 |
| inc0_ | without single | 09:27 |
| sdake | kennan for vm case follow the bare metal model | 09:27 |
| inc0_ | feel free to decide what of it is worth merging | 09:28 |
| inc0_ | I personally thing regex is useful and convenient tho | 09:28 |
| sdake | you will need a libvirt override in /etc/kolla/nova.conf | 09:28 |
| Kennan | sdake: is it only one VM is OK for kolla test and deploy? I noticed in some kolla docs. said need more nodes | 09:28 |
| sdake | to specify that qemu should be used for hypervisor instead of kvm | 09:28 |
| inc0_ | as for single...well I need either this one or include one | 09:28 |
| pbourke | inc0_: love the regex part | 09:28 |
| sdake | kennan you can do an AIO deploy on one vm sure | 09:28 |
| inc0_ | I'm perfectly ok with abandoning single if it's not needed | 09:29 |
| sdake | kennan the first step is to get an AIO deployment going | 09:29 |
| pbourke | how are the single and header related? | 09:29 |
| inc0_ | nope | 09:29 |
| sdake | kennan the next step if you like that is to get a multinode deployment going | 09:29 |
| inc0_ | just both can solve same problem | 09:30 |
| sdake | baby steps | 09:30 |
| inc0_ | adding proxy support that is | 09:30 |
| pbourke | how can I solve the proxy support with --single? | 09:31 |
| inc0_ | if you build your own manually tinkered base image | 09:31 |
| inc0_ | beforehand | 09:31 |
| inc0_ | that's how I did till now | 09:31 |
| inc0_ | however include would be much better | 09:31 |
| inc0_ | https://review.openstack.org/#/c/215504/ and please, review this | 09:31 |
| inc0_ | unless this get merged I'll be in rebase hell | 09:32 |
| sdake | not reviewing at 2:30am aftrer being awake for 20 hurs | 09:32 |
| pbourke | but i have a manually tinkered base image, i use regex, the cache skips over the manual image and builds the one I specify | 09:32 |
| pbourke | sorry not a big deal just trying to be sure I understand | 09:32 |
| inc0_ | cache won't skip it if Dockerfile changes | 09:32 |
| inc0_ | and that happends with time | 09:32 |
| inc0_ | but, that's why I consider include better option | 09:33 |
| sdake | pbourke any chance you will have those swift comments fixed up in the review queue for today since its end of week? | 09:33 |
| sdake | I've got a demo wednesday and woudl like to do full monty with swift + heat + horizon | 09:33 |
| pbourke | sdake: lets say yes | 09:34 |
| sdake | cool and some minimal docs | 09:34 |
| sdake | doesn't even have to be consumable | 09:34 |
| sdake | justsome way for *me* to consume them | 09:34 |
| sdake | guys we are making fantastic progress :) | 09:35 |
| sdake | t - 10 days for liberty 3 :) | 09:36 |
| openstackgerrit | Vladislav Belogrudov proposed stackforge/kolla: Cinder ansible waits for bootstrap container on all hosts https://review.openstack.org/215549 | 09:36 |
| pbourke | inc0_: that patch is some slick python | 09:39 |
| inc0_ | yeah, but somethigs wrong | 09:39 |
| pbourke | inc0_: oh? | 09:39 |
| inc0_ | there, build tests fail, I'll debug it | 09:39 |
| sdake | how well does multinode deployment work with master | 09:39 |
| pbourke | inc0_: was about to ask you in you can convince me it works the same way | 09:39 |
| pbourke | s/in/if | 09:40 |
| inc0_ | (strange because I've asserted both logics to be the same) | 09:40 |
| inc0_ | it's supposed to, I've compared results from both methods | 09:40 |
| inc0_ | but I'll do that again with more options I guess | 09:40 |
| inc0_ | we need unit tests for build script | 09:42 |
| inc0_ | gonna file a bug for that | 09:42 |
| pbourke | +1 | 09:43 |
| inc0_ | https://bugs.launchpad.net/kolla/+bug/1487390 | 09:44 |
| openstack | Launchpad bug 1487390 in kolla "unittests for build script" [Undecided,New] | 09:44 |
| sdake | hey folks we are going to have docso n docsopenstack.org | 09:45 |
| sdake | everyone excited about that? :-) | 09:45 |
| pbourke | if it means more people will use kolla then yes ;) | 09:46 |
| pbourke | looks like my mail never made it to the ML :( | 09:46 |
| pbourke | want to get these neutron plugins off my list | 09:47 |
| sdake | my review-kolla folder has 3k emails in it unread | 09:51 |
| Kennan | hi sdake: for your doc changes | 09:55 |
| Kennan | I tried tools/build.py | 09:55 |
| Kennan | it failed with | 09:55 |
| Kennan | ConnectionError: ('Connection aborted.', error(2, 'No such file or directory')) | 09:55 |
| sdake | is docker running? | 09:55 |
| sdake | systemctl status docker-engine | 09:56 |
| inc0_ | omg...I've just noticed what's wrong | 09:57 |
| Kennan | by bad, docker not started up itself after curl bash script | 09:57 |
| Kennan | I manual start it now | 09:58 |
| sdake | thanks i'll modify the docs to address that in a new itrationkennan | 09:58 |
| *** dims has joined #kolla | 10:05 | |
| openstackgerrit | Michal Jastrzebski (inc0) proposed stackforge/kolla: build.py refactor https://review.openstack.org/215504 | 10:07 |
| inc0_ | this build script does some crazy memory jumping | 10:09 |
| inc0_ | ;) | 10:09 |
| *** akwasnie has joined #kolla | 10:17 | |
| sdake | ok guys im off to bed | 10:22 |
| sdake | nght | 10:22 |
| sdake | kennan plese leave feedbak on irc about your aio deploy | 10:22 |
| sdake | or sk other ppele your qs | 10:22 |
| sdake | we hae abuthalf our enginering in europe | 10:22 |
| sdake | engineering team that is | 10:22 |
| Kennan | ok sdake: still building image seems long time | 10:24 |
| vbel | I think most of building is IO-wait :) So parallelizing it does not bring much | 10:29 |
| vbel | unless you are on SSD and 1Gb net :) | 10:30 |
| vbel | night sdake | 10:31 |
| vbel | "1Gb"->"100Gb" | 10:35 |
| inc0_ | pbourke, it seems after fixes refactor works | 10:40 |
| inc0_ | now I need some git fun to rebase rest;) | 10:40 |
| pbourke | does anyone know how the perms are being set for data containers now | 10:46 |
| pbourke | e.g. /var/lib/rabbitmq seems to be rabbitmq:rabbitmq but cant find the part that does that | 10:46 |
| inc0_ | it should be in ansible roles | 10:46 |
| inc0_ | rabbitmq/tasks/start.yml | 10:46 |
| pbourke | i see nothing about perms in there | 10:47 |
| vbel | I am checking rabbitmq now | 10:48 |
| pbourke | its working for rabbit just want to know how to do the same for swift | 10:49 |
| pbourke | as right now vols are root:root | 10:49 |
| pbourke | *my vols | 10:49 |
| inc0_ | look in bootstrap as well | 10:49 |
| vbel | the problem with that is that this folder is mounted as root and does not have permissions to put anything inside as rabbitmq user | 10:49 |
| inc0_ | data containers are started there | 10:49 |
| inc0_ | and rabbitmq has single volume /var/lib/rabbitmq according to that | 10:49 |
| vbel | /var/lib/rabbitmq is from data container and belongs to root | 10:50 |
| inc0_ | and other rabbit containers have volumes-from | 10:50 |
| pbourke | vbel: | 10:50 |
| pbourke | $ docker exec -it rabbitmq ls -ld /var/lib/rabbitmq | 10:50 |
| pbourke | drwxr-xr-x 3 rabbitmq rabbitmq 40 Aug 20 10:37 /var/lib/rabbitmq | 10:50 |
| vbel | depending on your version of rabbit it can su to rabbitmq from the very start and will fail to put its pid into /var/lib/rabbitmq/mnesia | 10:50 |
| inc0_ | try changing /var/lib/rabbitmq to /var/lib/rabbitmq:/var/lib/rabbitmq | 10:51 |
| pbourke | no as I said | 10:51 |
| pbourke | rabbitmq works | 10:51 |
| pbourke | I just cant see how | 10:51 |
| vbel | yes, it is originally rabbitmq:rabbitmq, but if you mount it it is root:root | 10:51 |
| vbel | so if you run it like docker run --rm -ti --volumes-from rabbitmq_data centos-binary-rabbitmq:latest /bin/bash | 10:53 |
| vbel | you will see it becomes root:root | 10:53 |
| pbourke | found it | 10:53 |
| vbel | I am sure it failes in oracle linux | 10:53 |
| pbourke | its in config-rabbitmq.sh | 10:53 |
| vbel | yes, true, so it should be not in config-rabbitmq.sh but in run | 10:54 |
| vbel | in start.sh | 10:54 |
| *** stvnoyes has quit IRC | 10:55 | |
| vbel | otherwise only bootstrapping works | 10:55 |
| vbel | anyone to submit the fix? | 10:56 |
| Haomeng | anyone can help me, I want to have try to deploy openstack via kolla, what is the guide I should follow? thanks | 10:57 |
| Haomeng | I am not sure the steps to play with kolla | 10:58 |
| pbourke | Haomeng: hi, you can start here: https://github.com/stackforge/kolla/blob/master/docs/dev-quickstart.md | 10:58 |
| Haomeng | pbourke: thank you | 10:58 |
| Haomeng | pbourke: yes, I try with steps, but some break with fatal error - http://paste.openstack.org/show/423307/ | 11:00 |
| Haomeng | pbourke: Error: image kollaglue/centos-binary-kolla-ansible:latest not found | 11:00 |
| Haomeng | pbourke: should I buld image first? | 11:00 |
| pbourke | Haomeng: yes, sorry I think that image is missing | 11:01 |
| pbourke | from dockerhub | 11:01 |
| pbourke | if you build it it should work | 11:01 |
| Haomeng | pbourke: ok, np, thanks for your info | 11:01 |
| Haomeng | pbourke: run "./tools/build-all-docker-images" right? | 11:02 |
| pbourke | Haomeng: it would be quicker to run docker/centos/binary/kolla-ansible/build --release | 11:02 |
| Haomeng | pbourke: great thanks | 11:02 |
| pbourke | Haomeng: you will probably need to build base first | 11:03 |
| Haomeng | pbourke: ok | 11:03 |
| pbourke | docker/centos/binary/base/build --release | 11:03 |
| Haomeng | pbourke: ok, thank you:) | 11:03 |
| Haomeng | pbourke: error - The command '/bin/sh -c yum -d 10 -y install systemd systemd-libs systemd-devel && yum clean all' returned a non-zero code: 1 | 11:04 |
| *** inc0_ has quit IRC | 11:04 | |
| pbourke | are you building on ubuntu? | 11:04 |
| Haomeng | pbourke: my systems is ubuntu, have to run this on redhat/centos which supports yum? | 11:04 |
| *** chandankumar has joined #kolla | 11:04 | |
| Haomeng | pbourke: yes | 11:04 |
| Haomeng | pbourke: what is os type required | 11:05 |
| pbourke | Haomeng: from quickstart: | 11:05 |
| pbourke | "For Ubuntu based systems, do not use AUFS when starting Docker daemon unless you are running the Utopic (3.19) kernel. AUFS requires CONFIG_AUFS_XATTR=y set when building the kernel. On Ubuntu, versions prior to 3.19 did not set that flag. If you are unable to upgrade your kernel, you should use a different storage backend such as btrfs. | 11:05 |
| pbourke | " | 11:05 |
| Haomeng | pbourke: ok, got | 11:06 |
| Haomeng | pbourke: enable the CONFIG_AUFS_XATTR=y, still get same error - yum-3.4.3-125.el7.centos.noarch has missing requires of yum-plugin-fastestmirror | 11:08 |
| Haomeng | pbourke: what is the os requirements | 11:08 |
| Haomeng | pbourke: CentOS should be better? | 11:09 |
| pbourke | Haomeng: yeah im not sure on the specifics of changing the backend. I run ubuntu myself but tend to build in a centos vm | 11:09 |
| pbourke | Haomeng: hopefully some of this will be smoother after the liberty release | 11:10 |
| pbourke | Haomeng: we're in a state of flux atm | 11:10 |
| Haomeng | pbourke: ok, np:) just want to have a try, thank you | 11:12 |
| Haomeng | pbourke: I have an idea, should we have a *kolla docker image*, which is easy to run kolla:) | 11:13 |
| Haomeng | pbourke: :) | 11:13 |
| openstackgerrit | Vladislav Belogrudov proposed stackforge/kolla: Rabbitmq servers fail to start due to wrong permissions https://review.openstack.org/215582 | 11:13 |
| pbourke | :D | 11:13 |
| vbel | just checked with our ol - now starts properly | 11:14 |
| pbourke | vbel: why is it any different doing it in start.sh vs config-rabbit ? | 11:15 |
| vbel | bootstap container chmod it and exits. new container mounts it as root:root again | 11:16 |
| pbourke | but config-rabbitmq is sourced from start.sh | 11:16 |
| vbel | in case of KOLLA_BOOTSTRAP | 11:16 |
| pbourke | ah wait, i see it now | 11:16 |
| pbourke | sorry | 11:16 |
| pbourke | wonder why I didnt have this issue though | 11:17 |
| vbel | no problem, it was a bit of mystery :) | 11:18 |
| vbel | data containers - did we have them for rabbit? | 11:18 |
| pbourke | i was having a problem around this but rebuilding all containers fixed it | 11:19 |
| pbourke | sdake knows the reason | 11:19 |
| vbel | the reason is data container. it has root:root volume. Before we had rabbitmq as data donor with correct /var/lib/rabbitmq permissions | 11:21 |
| vbel | e.g. I do docker run --rm -ti --volumes-from rabbitmq centos-binary-rabbitmq:latest /bin/bash | 11:22 |
| vbel | and have correct permissions for rabbit in no time :) | 11:22 |
| vbel | so new 'data' container can be tricky | 11:23 |
| vbel | 1 sec | 11:25 |
| *** diga has joined #kolla | 11:26 | |
| Haomeng | pbourke: not sure where we can download these kolla base docker image, or have to build on local? | 11:28 |
| vbel | I think I have it :) sometimes it runs sometime does not. we do not wait for bootstrap container and kill it in next task | 11:30 |
| vbel | it runs perfectly with waiting for bootstrap. Bootstrap chmods the volume but sometimes cannot do it because we don't wait for it | 11:41 |
| *** rhallisey has joined #kolla | 11:43 | |
| openstackgerrit | Vladislav Belogrudov proposed stackforge/kolla: Rabbitmq servers fail to start due to wrong permissions https://review.openstack.org/215582 | 11:51 |
| vbel | anyone could run cinder? Running client gets ERROR: Malformed request url (HTTP 400) | 12:22 |
| vbel | also cinder_volume says unable to update stats, LVMVolumeDriver not initialized | 12:23 |
| *** inc0 has joined #kolla | 12:29 | |
| pbourke | Haomeng: its best to build everything locally | 12:31 |
| Haomeng | pbourke: ok, will try, thanks | 12:31 |
| inc0 | damn, our CI for builds is horrible | 12:31 |
| inc0 | no way to check what are errors | 12:31 |
| Haomeng | pbourke: now get this error - "failed: [localhost] => {"changes": ["{\"status\":\"Pulling repository docker.io/kollaglue/centos-binary-kolla-ansible\"", looks like there is no such kollaglue/centos-binary-kolla-ansible image in docker.io | 12:32 |
| pbourke | Haomeng: try tools/build.py | 12:33 |
| Haomeng | pbourke: docker search kollaglue/centos-binary-kolla-ansible return nothing | 12:33 |
| Haomeng | pbourke: ok | 12:33 |
| pbourke | Haomeng: it should build everything | 12:33 |
| Haomeng | pbourke: ok | 12:33 |
| Haomeng | pbourke: run tools/build.py, got "ERROR:__main__:could not find image: no such id: kollaglue/centos-binary-ceilometer-base:latest" | 12:34 |
| pbourke | inc0: seen this ^ | 12:34 |
| pbourke | ? | 12:34 |
| *** Guest94558 has joined #kolla | 12:35 | |
| Haomeng | http://paste.openstack.org/show/423366/ | 12:36 |
| Guest94558 | Hi Folks, we are looking at containerizing among other control-plane service, openstack too for our deployments | 12:36 |
| Guest94558 | and hit upon Kolla projects which aims to just do that | 12:36 |
| inc0 | docker images | search ceilometer-base | 12:36 |
| inc0 | is there image like that? | 12:37 |
| Guest94558 | so, a quick query: is kolla production ready ..? | 12:37 |
| inc0 | aslo, are you using ansible? | 12:37 |
| pbourke | inc0: he's just trying to build all images | 12:37 |
| Guest94558 | are there any reference of Kolla in production or such deployments ..? | 12:37 |
| inc0 | Guest94558, it is if you want to run master | 12:37 |
| Haomeng | inc0: sudo docker images | grep ceilometer-base return nothing | 12:37 |
| inc0 | so dependencies are busted somehow | 12:38 |
| rhallisey | Guest94558, I'm sure if there any reference production deployments | 12:38 |
| rhallisey | I"m not sure* | 12:38 |
| Haomeng | which command I should run to build all dependencies, it is "tools/build.py", right? | 12:38 |
| inc0 | that should work, right | 12:38 |
| inc0 | if it doesn't something is wrong | 12:38 |
| Haomeng | inc0: ok, let mey try | 12:38 |
| inc0 | try to run ./build.py -d | 12:38 |
| inc0 | with debug | 12:39 |
| inc0 | aaa | 12:39 |
| Haomeng | inc0: ok, thanks | 12:39 |
| Guest94558 | inc0: we are definitely interested in Kolla for the obvious benefits of micro-services based deployment | 12:39 |
| inc0 | ./build.py -d -T 1 | 12:39 |
| inc0 | single thread for better logging | 12:39 |
| Guest94558 | however not sure if it is production ready | 12:39 |
| pbourke | Guest94558: if you're looking to containerise the control plane I would say is the best solution available. As far as "production ready" it's still under quite heavy development | 12:39 |
| inc0 | Guest94558, problem (?) of Kolla is that we're master based right | 12:39 |
| inc0 | now | 12:39 |
| Guest94558 | pbourke: yes, we want in control-plane | 12:40 |
| inc0 | so if you're not willing to risk deployment of latest greatest bleeding edge master, Kolla might have problems | 12:40 |
| inc0 | however, kolla stable kilo is a thing and it works | 12:40 |
| pbourke | the liberty release at the end of the month is the first I would recommend for production | 12:40 |
| SamYaple | but it has no upgrade path inc0 | 12:40 |
| inc0 | but it's very old code and limited comparing to what we have now | 12:40 |
| Guest94558 | inc0: so, is there no Kilo/Liberty stable base for Kolla ..? | 12:40 |
| SamYaple | I would not recommend kilo Kolla to anyon | 12:41 |
| inc0 | Guest94558, there is no stable Liberty anywhere | 12:41 |
| inc0 | Liberty is still in development | 12:41 |
| SamYaple | pbourke: would you consider voting on https://review.openstack.org/#/c/215157/ ? | 12:41 |
| inc0 | as for production, I'd suggest to wait till Liberty is released, and run kolla then | 12:41 |
| SamYaple | I believe I have answered your questions | 12:41 |
| Guest94558 | inc0: yes, I mean plan for Liberty | 12:41 |
| Haomeng | inc0: can you help - http://paste.openstack.org/show/423367/ | 12:42 |
| inc0 | if you want to deploy liberty - kolla is an answer | 12:42 |
| Haomeng | inc0: thank you | 12:42 |
| Guest94558 | inc0: in production ..? | 12:42 |
| inc0 | yup | 12:42 |
| vbel | could anyone anytime run cinder service? | 12:42 |
| inc0 | It will be as stable as openstack underneath | 12:42 |
| pbourke | SamYaple: no forgiveness! | 12:42 |
| Guest94558 | inc0: ok, sounds good, thanks. | 12:42 |
| SamYaple | pbourke: :) | 12:42 |
| Haomeng | inc0: will run "./build.py -d -T 1" and collect log again | 12:43 |
| pbourke | SamYaple: i forgot we were using get-pip, so the upgrade is indeed redundant | 12:43 |
| SamYaple | I want to actually move all repos for centos into the base pbourke | 12:43 |
| SamYaple | its safest I think and keeps same package version | 12:43 |
| pbourke | yeah we should | 12:43 |
| pbourke | rhallisey: I remember you had done some docs on cinder etc, have they disappeared? | 12:46 |
| rhallisey | uh ya looks like they're gone.. | 12:46 |
| rhallisey | I think they were removed in the latest purge | 12:47 |
| rhallisey | not sure why | 12:47 |
| Haomeng | inc0: the first error is - error: unpacking of archive failed on file /usr/bin/systemd-detect-virt: cpio: cap_set_file | 12:47 |
| rhallisey | was there a question? | 12:47 |
| pbourke | Haomeng: this is the same ubuntu issue | 12:47 |
| SamYaple | rhallisey: re ceph | 12:48 |
| Haomeng | pbourke: yes, think so | 12:48 |
| Haomeng | pbourke: any idea to fix | 12:48 |
| pbourke | rhallisey: not really, just was doing to something short for swift, but then wondered where the other similar docs had gone | 12:48 |
| rhallisey | SamYaple, what's up | 12:48 |
| SamYaple | why dont you leave most of that start.sh out since that will need to be handled much different | 12:48 |
| pbourke | Haomeng: see my comments earlier | 12:48 |
| Haomeng | pbourke: ok | 12:48 |
| rhallisey | SamYaple, where would it be handled? | 12:48 |
| SamYaple | ceph is going to be tricky since that data can't only exist in a data container since it needs to be sync around to the other nodes | 12:48 |
| rhallisey | I've already dropped like half of it as it | 12:49 |
| SamYaple | ansible has to be able to fetch that data | 12:49 |
| SamYaple | all the generated keys need to be sync'd, the mounting needs to be handled external to the container | 12:49 |
| rhallisey | pbourke, ya not sure why they were removed | 12:49 |
| rhallisey | SamYaple, ok.. this will be tricy | 12:49 |
| SamYaple | indeed | 12:50 |
| SamYaple | i have it working in Yaodu, I can Kolla-fy the ansible code | 12:50 |
| SamYaple | but it will break some kolla 'best-practices' due to its very natrue | 12:50 |
| SamYaple | no way around that | 12:50 |
| rhallisey | I was looking at yours + ceph container example | 12:50 |
| rhallisey | the ceph community has it containerized, but it's not very good practice and we also have ansible | 12:51 |
| rhallisey | so I'm trying to decipher what goes where when I know nothing about ceph | 12:51 |
| rhallisey | :p | 12:51 |
| rhallisey | so it's wip | 12:51 |
| SamYaple | yea ive been over ceph-docker but we cant come to agreement on how to do that, i dont agree with them | 12:51 |
| SamYaple | rhallisey: just get the containers up, and we can do the start.sh in the ansible script | 12:52 |
| rhallisey | ya their solution is not very good | 12:52 |
| SamYaple | I keep telling them | 12:52 |
| SamYaple | ansible-ceph is decent | 12:52 |
| rhallisey | it's actually pretty horrible | 12:52 |
| * SamYaple remembers he has a pull request to finish for ansible-ceph | 12:52 | |
| SamYaple | rhallisey: yea im not disagreeing there, but im not running it either | 12:59 |
| SamYaple | interestingly rhallisey, using ceph in docker containers allows us to avoid several hack jobs that ture-ceph daemons do | 13:02 |
| SamYaple | the main one is the mount the OSD to read the OSD_ID and then REMOUNT IT to the correct location once thats found | 13:02 |
| SamYaple | we dont have too | 13:02 |
| SamYaple | much much cleaner | 13:02 |
| openstackgerrit | Merged stackforge/kolla: Fix up base for ubuntu https://review.openstack.org/215157 | 13:06 |
| openstackgerrit | Sam Yaple proposed stackforge/kolla: Fix MariaDB for ubuntu https://review.openstack.org/215159 | 13:06 |
| openstackgerrit | Merged stackforge/kolla: Add kolla-ansible for ubuntu https://review.openstack.org/215112 | 13:07 |
| openstackgerrit | Merged stackforge/kolla: Fix RabbitMQ for Ubuntu https://review.openstack.org/215158 | 13:08 |
| inc0 | sorry guys, was on emergency | 13:10 |
| openstackgerrit | Sam Yaple proposed stackforge/kolla: Fix MariaDB for ubuntu https://review.openstack.org/215159 | 13:10 |
| inc0 | (debug neutron and setup rack...it was productive 15 minutes) | 13:10 |
| SamYaple | coolsvap|away: sdake test your code! | 13:14 |
| SamYaple | Its literally the only requirement for templating | 13:15 |
| openstackgerrit | Michal Jastrzebski (inc0) proposed stackforge/kolla: Build.py regex filtering https://review.openstack.org/215523 | 13:15 |
| openstackgerrit | Michal Jastrzebski (inc0) proposed stackforge/kolla: build.py refactor https://review.openstack.org/215504 | 13:15 |
| openstackgerrit | Michal Jastrzebski (inc0) proposed stackforge/kolla: Build single image https://review.openstack.org/215524 | 13:15 |
| openstackgerrit | Sam Yaple proposed stackforge/kolla: Fix broken zaqar template https://review.openstack.org/215622 | 13:16 |
| SamYaple | hey rhallisey pbourke building templates is broke without https://review.openstack.org/215622 | 13:16 |
| *** dims has quit IRC | 13:16 | |
| SamYaple | if we could expidate that merge | 13:16 |
| rhallisey | kk | 13:16 |
| SamYaple | thanks guys | 13:17 |
| rhallisey | np | 13:17 |
| *** dims has joined #kolla | 13:17 | |
| openstackgerrit | Michal Jastrzebski (inc0) proposed stackforge/kolla: Add custom code to base docker templates https://review.openstack.org/215541 | 13:17 |
| *** stvnoyes has joined #kolla | 13:21 | |
| inc0 | I got lost in git.. | 13:25 |
| openstackgerrit | Michal Jastrzebski (inc0) proposed stackforge/kolla: Build.py regex filtering https://review.openstack.org/215523 | 13:25 |
| openstackgerrit | Michal Jastrzebski (inc0) proposed stackforge/kolla: build.py refactor https://review.openstack.org/215504 | 13:25 |
| openstackgerrit | Michal Jastrzebski (inc0) proposed stackforge/kolla: Build single image https://review.openstack.org/215524 | 13:25 |
| inc0 | SamYaple, about your -1 on refactor...it is way more clean and idiomatic approach | 13:26 |
| SamYaple | inc0: I disagree | 13:26 |
| inc0 | so respectfully, don't -1 refactoring | 13:26 |
| SamYaple | and no lamba ever | 13:26 |
| inc0 | lambda is part of language and it's extremaly useful | 13:26 |
| SamYaple | Im -2 on lamba | 13:27 |
| inc0 | old code was long, undreadable | 13:27 |
| inc0 | what's your argument? I don't accept "people don't understand lambda" | 13:27 |
| inc0 | they should | 13:27 |
| SamYaple | To bad, that is a valid argument | 13:27 |
| inc0 | it is not | 13:27 |
| SamYaple | Regardless, im -2 on lamba | 13:27 |
| inc0 | that's not magical code, that's idiomatic | 13:27 |
| *** diga has quit IRC | 13:29 | |
| inc0 | and in this case, map and filter funcions, its widely used | 13:30 |
| SamYaple | As the code stands, you dont have to know much about python. Im going to keep it that way. I am -2 on lamba. There will be no swaying that | 13:31 |
| inc0 | as coding stands, we should keep high standards | 13:31 |
| inc0 | and that's how python code should look like | 13:32 |
| SamYaple | I strongly disagree. | 13:32 |
| inc0 | https://www.jeffknupp.com/writing-idiomatic-python-ebook/ good read, I recommend it | 13:32 |
| SamYaple | sure thing. not going to change my mind on lamba | 13:33 |
| SamYaple | It is not easily understood by most people, I wont have it | 13:33 |
| pbourke | in fairness, inc0's approach is way cleaner | 13:35 |
| SamYaple | Agreed | 13:35 |
| SamYaple | but redo it without lamba | 13:35 |
| pbourke | that's ridiculous | 13:36 |
| pbourke | ops are not editing this code | 13:36 |
| inc0 | if they want to, better learn python beforehand anyway... | 13:37 |
| inc0 | in this use case (and that's pretty much only use case when I use lambda) it's most clean approach I know | 13:38 |
| *** jtriley has joined #kolla | 13:38 | |
| inc0 | declaring closure there, and that's what would take to remove lambda would harm readibility | 13:38 |
| SamYaple | Ive had the lamba fight. Ive heard all the sides. I am -2 on lamba | 13:39 |
| inc0 | I don't accept argument "lambda is too hard". Programming is hard. | 13:40 |
| inc0 | also, to be fair SamYaple, things you did with references out there will prove much harder to understand to anyone | 13:41 |
| SamYaple | Im ok with a refactor. Im not ok with lamba | 13:41 |
| SamYaple | Its time for me to sleep again. | 13:41 |
| inc0 | go to sleep. | 13:41 |
| inc0 | maybe after you get some rest it won't look that bad | 13:42 |
| inc0 | you're grumpy person | 13:42 |
| inc0 | now | 13:42 |
| inc0 | :P | 13:42 |
| SamYaple | :) | 13:42 |
| rhallisey | lol | 13:42 |
| SamYaple | I should clarify my first comment, Im -2 on lamba not the sort_images change | 13:42 |
| SamYaple | will comment as such | 13:42 |
| *** Guest94558 has quit IRC | 13:42 | |
| inc0 | and read this book I linked, really after 7yrs of python experience I've learned to care about that stuff | 13:43 |
| inc0 | and yes, lambdas are good if used correctly | 13:43 |
| SamYaple | I completely trust your python inc0. I dont trust everyone else | 13:43 |
| inc0 | then maybe that will make them consider this too hard and will ask someone who actually understands python | 13:44 |
| inc0 | that will prove beneficial for them | 13:44 |
| inc0 | anywayu | 13:44 |
| SamYaple | 'lets exlude people from understanding a glorified script' is all i hear | 13:44 |
| SamYaple | this is a refactor of a bash script, not a true program | 13:44 |
| inc0 | nope, I think this should be considered true program right now | 13:45 |
| inc0 | including unit tests and such | 13:45 |
| inc0 | in fact, this might be one of most critical code in whole project | 13:45 |
| SamYaple | it is certainly not | 13:45 |
| inc0 | let's treat it like that, not a bash script | 13:45 |
| inc0 | well, if this is broken, gates are broken | 13:46 |
| SamYaple | ok like i said its bedtime. I have added another comment clarifying my first | 13:46 |
| inc0 | if gates are broken, no new changes comes in | 13:46 |
| SamYaple | I would like to point out you cant merge the build.py code if it breaks the gate.... | 13:46 |
| inc0 | but it can break the gate afterwards | 13:47 |
| SamYaple | ugh im not getting drawn in again. I feel very strongly about no lambda and that isn't going to change | 13:47 |
| inc0 | you merge, something is wrong with dir structure, it breaks | 13:47 |
| SamYaple | im ok with a refactor past that your concerns are valid | 13:47 |
| inc0 | that's how bugs work | 13:47 |
| inc0 | and I think your approach to lambda is zealotus more than rational | 13:48 |
| inc0 | :P | 13:48 |
| inc0 | lambdas are cool. | 13:48 |
| inc0 | if used properly | 13:48 |
| SamYaple | perhaps, but zealotus is unlikely to change in a day if that is the situation | 13:48 |
| inc0 | that's why I'll ask for a second opinion:P | 13:48 |
| inc0 | when sdake wakes up | 13:48 |
| *** diogogmt has joined #kolla | 13:49 | |
| inc0 | let's ask Steven, if he asks me to remove lambda, I'm removing lambda | 13:49 |
| *** diogogmt has quit IRC | 13:50 | |
| rhallisey | I'm if it's hard to understand we can document the build script. If people push changes that are incorrect we can block | 13:51 |
| *** diogogmt has joined #kolla | 13:52 | |
| inc0 | rhallisey, https://review.openstack.org/#/c/215504/1 is it hard to understand? | 13:52 |
| SamYaple | fyi inc0 jpeeler has his image building code up https://review.openstack.org/#/c/215323/ | 13:53 |
| inc0 | I've seen that - I've submitted this one before I found out about his | 13:53 |
| inc0 | now we have 2 choices, I'm ok with either if it works | 13:53 |
| rhallisey | I think it's fine | 13:54 |
| inc0 | https://review.openstack.org/#/c/215541/ also, unrelated, please review this one - will solve lots of problems | 13:54 |
| inc0 | see SamYaple? Everyone thinks lambdas are ok;)_ | 13:54 |
| inc0 | I'll just submit python code when you're not around | 13:55 |
| SamYaple | yea thats a great thing to say, trying to subvert reviews | 13:55 |
| inc0 | your grumpiness killed rest of sense of humor in you | 13:56 |
| inc0 | :P | 13:56 |
| SamYaple | no that was the alcohol for the past 10 hours | 13:56 |
| SamYaple | but that isn't where the lambda thing comes from | 13:56 |
| SamYaple | i need some eggs. im going to sleep guys | 13:57 |
| SamYaple | inc0: I promise to rereview when i awake, but i wont promise a change | 13:57 |
| inc0 | I'll ask sdake to solve this issue | 13:58 |
| inc0 | let's agree to follow his decision | 13:58 |
| inc0 | ok? | 13:58 |
| SamYaple | No I wont agree to that, but I will have the discussion and I can be swayed. | 13:58 |
| SamYaple | in jpeeler's patch sdake mentions that this may be treated as a library in which case the conversation drastically chagnes for me | 13:59 |
| SamYaple | right now it is a glorified script in my mind, a script that all the ops guys I know can read an modify | 14:00 |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: Add Ansible role for memcached https://review.openstack.org/214593 | 14:03 |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: Implement Ansible role for Swift https://review.openstack.org/215065 | 14:03 |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: Implement Ansible role for Swift https://review.openstack.org/215065 | 14:05 |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: Implement Ansible role for Swift https://review.openstack.org/215065 | 14:08 |
| jpeeler | SamYaple: you mean you're opposed to it being treated as a library? | 14:08 |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: Implement Ansible role for Swift https://review.openstack.org/215065 | 14:09 |
| inc0 | jpeeler, I think that was about my script being unreadable for ops people | 14:09 |
| SamYaple | jpeeler: no thats just a different conversation | 14:09 |
| inc0 | and in case this turns into library, ops people won't read it at all | 14:10 |
| SamYaple | inc0: correct | 14:10 |
| SamYaple | that is my viewpoint | 14:10 |
| inc0 | SamYaple, then when we'll have CLI | 14:10 |
| inc0 | it will be library | 14:10 |
| inc0 | it should end up being library sooner or later | 14:10 |
| inc0 | I don't think we should treat this as glorified bash any more | 14:11 |
| SamYaple | i think you understand my points inc0, whether you agree or not is a different matter. If I am understood then we can communicate on it further (later) | 14:11 |
| jpeeler | so which patch should continue for the single image building? | 14:11 |
| SamYaple | jpeeler: yours is the non-dependant patch | 14:12 |
| SamYaple | thats the one we require for removing the docker dir | 14:12 |
| inc0 | jpeeler, https://review.openstack.org/#/c/215523/ this one adds a bit different functionality | 14:12 |
| jpeeler | it does, yes | 14:12 |
| inc0 | one futher on adds "just build that damn image" thing | 14:13 |
| SamYaple | sorry guys im out for real this time. im closing the window | 14:13 |
| inc0 | but I think regex building is useful | 14:13 |
| inc0 | "build me neutron plz" | 14:13 |
| inc0 | its convenient for that stuff | 14:13 |
| jpeeler | right | 14:13 |
| inc0 | bbl I'm moving home | 14:16 |
| inc0 | cyas | 14:16 |
| *** jasonamyers has joined #kolla | 14:20 | |
| *** inc0 has quit IRC | 14:20 | |
| openstackgerrit | Merged stackforge/kolla: Fix broken zaqar template https://review.openstack.org/215622 | 14:21 |
| *** diogogmt has quit IRC | 14:26 | |
| *** diogogmt has joined #kolla | 14:31 | |
| openstackgerrit | Jeff Peeler proposed stackforge/kolla: Add options for selective build and listing images https://review.openstack.org/215323 | 14:38 |
| *** banix has joined #kolla | 14:40 | |
| *** pbourke has quit IRC | 14:58 | |
| *** blahRus has joined #kolla | 14:58 | |
| *** pbourke has joined #kolla | 14:58 | |
| sdake | morning | 14:58 |
| sdake | pbourke ping | 14:59 |
| sdake | morningbtw :) (you dropped off) | 14:59 |
| pbourke | sdake: howdy | 14:59 |
| pbourke | sdake: you mustnt have slept long? | 14:59 |
| pbourke | sdake: i've addressed all but one comment on the swift patch and added docs | 15:00 |
| pbourke | sdake: its good to go imo, it depends how strongly you guys feel on the remaining comment | 15:00 |
| sdake | 5 hours? | 15:00 |
| sdake | typical 5-6 hours enerally | 15:00 |
| pbourke | very little | 15:00 |
| sdake | ok did you get it in the review queue | 15:01 |
| sdake | i typiccallyget 6 hours of sleep | 15:01 |
| pbourke | yup its there | 15:01 |
| sdake | so ya little light | 15:01 |
| sdake | i feel better when I get 8 but the summertime - i wake with the sun :( | 15:01 |
| sdake | in the winter I wke up at 9-10am :) | 15:02 |
| sdake | any kind of docs available | 15:02 |
| pbourke | yup | 15:03 |
| pbourke | https://review.openstack.org/#/c/215065/9/docs/swift-readme.rst | 15:03 |
| *** unicell1 has joined #kolla | 15:03 | |
| *** inc0 has joined #kolla | 15:03 | |
| inc0 | afternoon | 15:03 |
| sdake | pbourke where is the info on what comment is not fixed | 15:04 |
| sdake | just waking up and i know its your end of day | 15:04 |
| pbourke | sdake: see my most recent comment | 15:04 |
| sdake | so litttle slow | 15:04 |
| pbourke | on that review | 15:04 |
| *** unicell has quit IRC | 15:04 | |
| pbourke | np | 15:04 |
| inc0 | sdake, I need your opinion. | 15:04 |
| inc0 | https://review.openstack.org/#/c/215504/4 we have small conflict here with Sam | 15:05 |
| pbourke | honestly the thought of refactoring that config makes me want to go home | 15:05 |
| pbourke | but its sunny here and its that time of the week | 15:05 |
| inc0 | and I would like you to be arbitrator | 15:05 |
| sdake | oh not rainining for once ;) | 15:05 |
| sdake | dublin never know if its going to rain :) | 15:06 |
| pbourke | exactly! | 15:06 |
| inc0 | I thought you know when it's going to rain | 15:06 |
| inc0 | it will. | 15:06 |
| sdake | ok so this is tthe thign where the ring files isn't copied to the target nodes? | 15:06 |
| sdake | sorry if i seem dense again brain booting | 15:07 |
| bmace | you mean the lambda thing inc0? i think it is a valid python thing to use lambda, especially for sorts and that kind of thing. | 15:07 |
| inc0 | yeah, lambda thing | 15:07 |
| sdake | lambda is normal python | 15:07 |
| sdake | it is perfectly fine to use | 15:07 |
| pbourke | sdake: ive added ansible code to copy the ring files to all nodes. I recommend some coffee and reading the docs ;) | 15:07 |
| inc0 | it's not like wizadry...I can do wizadry, this isn't one | 15:07 |
| sdake | ok well whatis missing then? :) | 15:08 |
| pbourke | the generation of the actual rings | 15:08 |
| sdake | my inclination at this point is to techdebt bug it | 15:08 |
| pbourke | that has to be done manually | 15:08 |
| sdake | oh eneration | 15:08 |
| sdake | ya techdebt bug | 15:08 |
| pbourke | using my perfectly documented commands :p | 15:08 |
| sdake | we can tackle in rc1 | 15:08 |
| *** banix has quit IRC | 15:08 | |
| sdake | btw its not coffee that gts me going, i need pepsi ;) | 15:09 |
| bmace | pepsi-max! :) | 15:09 |
| bmace | maximum caffeine | 15:10 |
| sdake | i would like autogeneration for liberty if possible | 15:10 |
| sdake | but it doesn't have to be in this patchsset | 15:10 |
| sdake | bmace less cafffeine but more sugar :) | 15:10 |
| inc0 | sdake, mind saying that you're ok with lambda under review on which Sam threw -2 bomb? | 15:10 |
| sdake | i probaby wouldn't drink pepsi ifnt' i wasnt addicted to it | 15:10 |
| bmace | bleh.. i prefer the calories in caffeine over the calories in sugar. | 15:10 |
| sdake | ya but i respect other cores -2s | 15:11 |
| sdake | but i will say lamba is fine to use - got a link | 15:11 |
| inc0 | ever tried Yerba Mate? | 15:11 |
| inc0 | well, -2 is because there is lambda | 15:11 |
| sdake | what is that some drug? | 15:11 |
| sdake | I dont ned any more drugs | 15:11 |
| *** SourabhP has joined #kolla | 15:11 | |
| inc0 | no, it's kind of plant | 15:11 |
| sdake | gimme link to the review | 15:11 |
| *** banix has joined #kolla | 15:11 | |
| inc0 | very popular in southern america | 15:11 |
| inc0 | they make sort of tea out of it, good stuff | 15:12 |
| sdake | so is coca leaves | 15:12 |
| inc0 | https://review.openstack.org/#/c/215504/4 | 15:12 |
| sdake | sounds like drug to me :) | 15:12 |
| inc0 | no, it's juniper;) | 15:12 |
| inc0 | (plant, not the company) | 15:12 |
| inc0 | https://en.wikipedia.org/wiki/Yerba_mate | 15:12 |
| *** SourabhP_ has joined #kolla | 15:13 | |
| inc0 | they drink it all over souther ameria...and Poland | 15:13 |
| inc0 | its suprisingly popular in Poland | 15:13 |
| inc0 | that's what I drink when I need caffeine...or I take guarana-extract pills;) | 15:13 |
| inc0 | sdake, it doesn't use sorted..it uses filter and map | 15:16 |
| inc0 | and that's where lambda is | 15:16 |
| *** SourabhP has quit IRC | 15:16 | |
| *** SourabhP_ is now known as SourabhP | 15:16 | |
| sdake | inc0 that chemical you mentioned above looks safe enough - only xanthines which are actually known to be neuro-protectants | 15:18 |
| sdake | not sur eabout polyphenols tho | 15:19 |
| inc0 | well, its juniper infusion, and it's being drinked all over that place for several hundred years now | 15:19 |
| sdake | those may or may not be safe - so I'll stick to my terrile-for-me pepsi :) | 15:19 |
| sdake | people been smoking tobacco for hundreds of years too :) | 15:19 |
| inc0 | same about pot | 15:20 |
| inc0 | I think all should be legal and let evolution solve things | 15:20 |
| sdake | ya well imo pot is fairlly safe minus possible lung damage | 15:20 |
| sdake | inc0 for once we agree :) | 15:20 |
| sdake | inc0 you should dig into why samyaple doesn't like lambda | 15:21 |
| sdake | is it hte particular use, or lambda in general | 15:21 |
| inc0 | I did | 15:21 |
| inc0 | he considers build.py to be glorified bash script, therefore ops should edit it | 15:22 |
| inc0 | and ops are scared of lambda | 15:22 |
| sdake | ya that logic makes sense it is an advanced python concept | 15:22 |
| inc0 | that's just that. Lambda is not an if statement | 15:22 |
| inc0 | still, I beg to diffet that it's glorified bash script | 15:22 |
| sdake | ya well i really doubt operators will edit it | 15:23 |
| inc0 | I think we should unittest shit out of it and build lib out of it | 15:23 |
| inc0 | \\\\\\\\\\\\\\\\\\\\\\\\\\ | 15:23 |
| inc0 | sorry,...that was my cat | 15:23 |
| sdake | highly highly doubt | 15:23 |
| sdake | keyboard cat11 | 15:23 |
| inc0 | that was actual cat who decided to venture over backslash...and its close to enter on my keyboard | 15:24 |
| sdake | lets be honest, nobody will be able to edit that buid.py script unless they have 1-2 years of python experience | 15:24 |
| sdake | but lambda may be beyond 1-2 years of python experience | 15:24 |
| inc0 | yeah, especially when I actually understood what's happening out there | 15:24 |
| sdake | can it be done without lambda? | 15:24 |
| rhallisey | lol | 15:24 |
| sdake | i haven't looked at the patch | 15:24 |
| inc0 | I can define closure... | 15:24 |
| inc0 | it's lambda in map function | 15:24 |
| inc0 | so classical use case | 15:25 |
| sdake | well i would be ok with avoiding conplex language features of python in build.py and in general in kolla | 15:25 |
| sdake | one of the failures of python is it has too many language features | 15:25 |
| inc0 | about this.. https://github.com/stackforge/kolla/blob/master/tools/build.py#L272 this line alters self.images | 15:26 |
| sdake | lambda is one of those complex language features | 15:26 |
| sdake | inc0 too ealry for me to understand code sorry | 15:26 |
| inc0 | I know how references in python works, but thats way beyond 2yrs of python | 15:26 |
| pbourke | even inc0's cat can do it | 15:27 |
| sdake | line 272 is 1-2 years of python experience matieral | 15:27 |
| inc0 | I'll remove this for sake of Sam's mental health | 15:27 |
| *** SourabhP has quit IRC | 15:28 | |
| sdake | lambda was the last thing i learned in python if that helps :) | 15:28 |
| pbourke | Im opposed to removing it | 15:28 |
| pbourke | the arguments against are not sound | 15:28 |
| inc0 | sdake, metaclasses are fun | 15:28 |
| sdake | if the arguemnt is "ops will edit build.py" that is pure crack talk | 15:28 |
| *** ccrouch has joined #kolla | 15:29 | |
| inc0 | that's Sams concern as far as I know, and he confirmed that | 15:29 |
| pbourke | the code is elegant and is completely standard and valid python | 15:29 |
| *** ccrouch has quit IRC | 15:29 | |
| *** ccrouch has joined #kolla | 15:29 | |
| sdake | again I respect other cores -2s | 15:29 |
| sdake | if his mind can't be changed it can't be changed | 15:29 |
| inc0 | I think main difference between me and him is that I think wwe should treat build.by as normal python code which will be extended | 15:30 |
| inc0 | and therefore readability counts | 15:30 |
| pbourke | this is normal python code! | 15:30 |
| sdake | pbourke I think sam's real concern is the use of advanced langugage features | 15:30 |
| sdake | he brought this up during the jinja2 teplating thing too | 15:30 |
| sdake | as if operators will edit jinja2 containers | 15:31 |
| inc0 | yeah, lack of includes | 15:31 |
| inc0 | I have issues with that too, but not too much | 15:31 |
| pbourke | this script is beyond the scope of operators | 15:31 |
| sdake | turns out we dont really need anything besides conditionals | 15:31 |
| sdake | pbourke totally agre | 15:31 |
| *** ccrouch has quit IRC | 15:31 | |
| inc0 | sdake, thing is...we never did...its just for convenience and readibility | 15:31 |
| sdake | operators are nto going to edit the code base unless they have an engineeirng wizard on staff | 15:31 |
| *** ccrouch has joined #kolla | 15:31 | |
| inc0 | we can write every program with nxor, it's turing complete | 15:32 |
| sdake | inc0 lol | 15:32 |
| inc0 | NAND* sorry | 15:32 |
| sdake | well the openstack process is a -2 kills a review | 15:32 |
| inc0 | nand is turing complete | 15:32 |
| sdake | so pbourke recommend commenting if you think sam is wrong | 15:33 |
| inc0 | that's what we should talk with Sam about tbh...its 2nd time he does that to me | 15:33 |
| inc0 | -2 I mean | 15:33 |
| sdake | i gave you a -2 i think not sam :) | 15:33 |
| sdake | -2s are perfectly normal part o the process | 15:33 |
| inc0 | it was about L1 | 15:33 |
| sdake | L1? | 15:34 |
| inc0 | but he removed it;) for keepalive container | 15:34 |
| pbourke | inc0: for you regex patch, should this work? "tools/build.py --template -r *neutron*" | 15:34 |
| sdake | oh | 15:34 |
| inc0 | pbourke, actually I made this so -r neutron is enough | 15:34 |
| inc0 | no need for * | 15:34 |
| sdake | pbourke can you comment on the lambda thing if you feel strongly about it in the review | 15:34 |
| pbourke | doing so now | 15:34 |
| pbourke | inc0: thanks | 15:35 |
| inc0 | its re.search, otherwise it would need to be .*neutron.* | 15:35 |
| sdake | its also perfectly normal on a -2 for people to comment if they disagree with the core reviewer | 15:35 |
| inc0 | but you can do all sorts of regex magic there, it's pretty flexible | 15:35 |
| sdake | but if he rally doesn't want to remove his -2, then the review is dead as is | 15:35 |
| sdake | that is just the process | 15:35 |
| inc0 | unless I rebase it... | 15:36 |
| sdake | and we should all respect core reviewers -2s in general | 15:36 |
| inc0 | j/k ;) | 15:36 |
| sdake | if you rebase it without the offending code | 15:36 |
| sdake | yu mean | 15:36 |
| sdake | i think you mean refactor | 15:36 |
| inc0 | well, correct me if I wrong, but simple rebase will throw away -2 | 15:36 |
| sdake | no | 15:36 |
| sdake | -2 is locked in | 15:36 |
| inc0 | ah ok | 15:36 |
| sdake | the only way is to remove th change id and make a new patch | 15:37 |
| sdake | this is highly highy frowned upon | 15:37 |
| inc0 | as it should be | 15:37 |
| sdake | I would -2 to preserve sam's vote in that case ;) | 15:37 |
| inc0 | anyway, I'll keep refactoring build.py, I have issues in how wasteful for memory it is | 15:38 |
| sdake | ya a git rebase followed by a git review - the -2 vote stays | 15:38 |
| sdake | inc0 i would like to use build.py as a library | 15:38 |
| sdake | so while your refactoring try to keep that in mind | 15:39 |
| sdake | I'd like to bill it as such as well | 15:39 |
| sdake | using an entrypoint python magic ;) | 15:39 |
| sdake | long term - doesn't need to happen immediately | 15:40 |
| sdake | or maybe medium term | 15:40 |
| inc0 | if we'd make CLI client, that will come automatically | 15:40 |
| bmace | just hope nobody -2s magic ;) | 15:40 |
| sdake | for what its worth, my opinion of python is that it hsa too many language features and "python" code is "hard to read code" :) | 15:41 |
| sdake | although pbourke's recommendation to use .get was good - I liked that | 15:41 |
| inc0 | sdake, while I agree with lots of features...most of them are syntactic sugar which actually helps to read it | 15:41 |
| sdake | pythonic that is | 15:41 |
| pbourke | sdake: did you add that in the end? | 15:42 |
| pbourke | can't remember | 15:42 |
| sdake | yes i did | 15:42 |
| pbourke | nice ;) | 15:42 |
| sdake | although jpeeler had a patch which did same thing which doesn't have .get | 15:42 |
| sdake | i just didn't think of it when doing it | 15:42 |
| sdake | but of course that is th ecorrect way to do it | 15:42 |
| sdake | it does it in a different spot | 15:43 |
| sdake | bmace I found weikus's slides | 15:43 |
| sdake | looked solid | 15:43 |
| bmace | ah, from linuxcon? or openstack day? | 15:43 |
| sdake | linuxcon | 15:43 |
| bmace | it is mostly accurate. some minor syntax tweaking is still in the works. | 15:44 |
| pbourke | link? | 15:45 |
| *** banix has quit IRC | 15:46 | |
| *** absubram has joined #kolla | 15:47 | |
| jpeeler | ok so has the decision been made to abandon https://review.openstack.org/#/c/215323/ ? | 15:49 |
| jpeeler | i missed a pep thing, so wondering if it's worth fixing | 15:49 |
| sdake | jpeeler not that i am aware of | 15:50 |
| sdake | i havent even reviewed inc0's work | 15:50 |
| sdake | but i don't want a new argument and from the discussion it appears a -r argument was added | 15:50 |
| sdake | by arguemnt i mean to the command line ;) | 15:51 |
| inc0 | it's for regex filtering | 15:51 |
| sdake | it can just be without an argument | 15:51 |
| inc0 | I can make if default if you want | 15:51 |
| sdake | buid.py regex | 15:51 |
| sdake | yes | 15:51 |
| inc0 | ok | 15:51 |
| sdake | i asked same for jeff's work | 15:51 |
| sdake | does your patch rebuild parents | 15:51 |
| inc0 | it does | 15:52 |
| sdake | you should be able to do the following | 15:52 |
| sdake | build.py heat ceilometer | 15:52 |
| sdake | and anything heat and ceilometer elated is rebuilt | 15:52 |
| sdake | that is what id like to see | 15:53 |
| sdake | or buid.py heat_engine | 15:53 |
| sdake | nnd only heat-engine is build | 15:53 |
| sdake | jpeelers code doesn't build parents | 15:53 |
| openstackgerrit | Jeff Peeler proposed stackforge/kolla: Add options for selective build and listing images https://review.openstack.org/215323 | 15:53 |
| pbourke | inc0: dumb question but... could the lambda not be replaced with 'for img in this_tier: images_to_process.remove(img)' | 15:53 |
| pbourke | inc0: not saying it should | 15:53 |
| inc0 | it can ofc | 15:53 |
| jpeeler | i fixed it. feel free to abandon it if inc0's patch gets in | 15:54 |
| inc0 | I've used lambda to keep it 1liner | 15:54 |
| sdake | in that case, might as well jus tremove the lambda | 15:54 |
| pbourke | :( | 15:54 |
| sdake | that is simpler | 15:54 |
| inc0 | I will, no worried | 15:54 |
| sdake | but again the reason is pure crack sauce ;) | 15:54 |
| pbourke | its not worth arguing about, but I think its a shame | 15:54 |
| inc0 | just...we need to have this discussion | 15:54 |
| *** shardy_ has joined #kolla | 15:54 | |
| sdake | (that operators will edit buidl.py) | 15:55 |
| pbourke | is it going to be a requirement to only use basic code for kolla? | 15:55 |
| sdake | no | 15:55 |
| pbourke | if so that may put potential contributors off | 15:55 |
| sdake | pythonic code is good with me | 15:55 |
| sdake | i'll add to agenda for wednesday's meeting so we can have a vote for it | 15:56 |
| sdake | not that we have alot of python code atm | 15:56 |
| *** shardy has quit IRC | 15:56 | |
| inc0 | we will | 15:56 |
| sdake | i said atm | 15:56 |
| inc0 | sooner or later | 15:56 |
| inc0 | yeah | 15:56 |
| sdake | python code should be easy to understan | 15:57 |
| sdake | lambda is sometimes *required* to do the job | 15:57 |
| pbourke | tbh if an op can understand the tier building mechanism they will definitely understand the lambda | 15:57 |
| sdake | if its not rquired, my recommendation would be eto avoid it | 15:57 |
| sdake | lambda is an advanced language feature | 15:58 |
| pbourke | we're probably bikeshedding now | 15:58 |
| sdake | no we are setting policy | 15:58 |
| sdake | but we can do it wednesday | 15:58 |
| pbourke | the regex patch is awesome though | 15:59 |
| sdake | we have a policy of only using conditionals in .j2 files | 15:59 |
| sdake | that is a good policy | 15:59 |
| sdake | oh yu man bikeshedding this patch ;) | 15:59 |
| *** shardy_ has quit IRC | 16:00 | |
| sdake | pbourke bikeshedding which part, the use of -r as an option or lambda? | 16:00 |
| pbourke | the lambda | 16:00 |
| pbourke | I just dont like to see someone win because they shout the loudest | 16:00 |
| pbourke | as I said the arguments against are not good imo | 16:01 |
| *** shardy has joined #kolla | 16:01 | |
| sdake | agree and we are not setting a policy about lambda today | 16:01 |
| pbourke | the regex patch is dependant on it thouhg :( | 16:01 |
| sdake | and agree the arguemnts against are not sound | 16:01 |
| pbourke | inc0: can you make the regex patch standalone? | 16:01 |
| pbourke | and we'll postpone the other discussion | 16:01 |
| sdake | lets get it merged | 16:01 |
| sdake | i want stuff going in fast and furious | 16:01 |
| sdake | need individual image build like asap :) | 16:02 |
| pbourke | +1 | 16:02 |
| pbourke | come back inc0 :P | 16:03 |
| *** ccrouch has quit IRC | 16:03 | |
| *** ccrouch1 has joined #kolla | 16:04 | |
| *** ccrouch1 has quit IRC | 16:04 | |
| *** ccrouch has joined #kolla | 16:04 | |
| inc0 | I'm back | 16:14 |
| inc0 | :) | 16:14 |
| inc0 | I'll remove lambda from refactoring patch | 16:14 |
| pbourke | thought you'd rage quit | 16:14 |
| pbourke | ;) | 16:14 |
| inc0 | nah, I've played too much of online games to rage quit from something like that | 16:14 |
| pbourke | inc0: my suggestion. put up the pep8 refactor, regex change, and lambda as 3 independant changes | 16:15 |
| pbourke | we can merge the first two very easily | 16:15 |
| pbourke | then vote on the third next wed | 16:15 |
| openstackgerrit | Vladislav Belogrudov proposed stackforge/kolla: Rabbitmq servers fail to start due to wrong permissions https://review.openstack.org/215582 | 16:16 |
| sdake | we dont ned to vote on the patch that can be doen in the gerrit reiview | 16:17 |
| *** chandankumar has quit IRC | 16:17 | |
| sdake | we can vote on whether we accept pythonic code or not in the code base | 16:17 |
| sdake | including advanced language features | 16:17 |
| sdake | its a policy that might as well be set now since there is disagreemeent among the cores around advanced language features | 16:18 |
| inc0 | I'll just remove lambda here | 16:18 |
| inc0 | I can do it without lambda easily | 16:18 |
| inc0 | and we'll decide on future policy | 16:19 |
| inc0 | but...can I get reviews on this? | 16:20 |
| inc0 | https://review.openstack.org/#/c/215541/ | 16:20 |
| inc0 | that would make my life much easyier | 16:20 |
| sdake | inc0 that is a good course of action | 16:20 |
| inc0 | I'm not in texas yet, so I don't need to be redneck;) | 16:21 |
| inc0 | so I can go for a compromise | 16:22 |
| *** unicell1 has quit IRC | 16:24 | |
| sdake | inc0 that patch is reviewed | 16:25 |
| inc0 | thanks | 16:26 |
| *** dims is now known as dimsum__ | 16:26 | |
| vbel | inc: I request include_footer too :) | 16:28 |
| vbel | to clean up stuff :) | 16:28 |
| vbel | include_footer would be at the other end of container hierarchy | 16:30 |
| inc0 | sdake, you want to call ./build.py regex? | 16:30 |
| inc0 | but that makes regex required | 16:30 |
| openstackgerrit | Paul Bourke proposed stackforge/kolla: WIP: Add optional third party Neutron plugins https://review.openstack.org/207545 | 16:31 |
| *** daneyon has joined #kolla | 16:31 | |
| sdake | inc0 what is your question | 16:31 |
| sdake | i dont understand it | 16:31 |
| sdake | inc0 if nothign is specified, it should build all | 16:32 |
| sdake | look at jpeelers patch he made similiar change | 16:33 |
| sdake | tools/builld.py builds all just as happens now | 16:33 |
| pbourke | on that subject | 16:33 |
| pbourke | I would prefer no args to print usage | 16:33 |
| pbourke | principle of least surprise | 16:33 |
| inc0 | well, then to build all would be ./build.py . | 16:35 |
| *** vbel has quit IRC | 16:35 | |
| inc0 | which isn't horrible | 16:35 |
| *** ashishb has joined #kolla | 16:35 | |
| sdake | pbourke agree that would be better but thatwould b ea change in behavior | 16:36 |
| pbourke | true | 16:36 |
| sdake | i dont mind the changei nbehavior | 16:36 |
| pbourke | not that we've had any shortage of those in the run up to liberty ;) | 16:36 |
| sdake | lol | 16:36 |
| sdake | ya agree | 16:36 |
| inc0 | yeah, after liberty this will be harder | 16:36 |
| sdake | my consistency engine in my brain isnt working yet, still booting | 16:36 |
| pbourke | suggest: | 16:37 |
| pbourke | -r regex | 16:37 |
| pbourke | --all | 16:37 |
| pbourke | no args = print usage | 16:37 |
| sdake | i do not want a -r really | 16:37 |
| inc0 | or maybe ./build.py regex as default | 16:37 |
| inc0 | ./build.py --all for all | 16:37 |
| sdake | no need for it and w eneed to pass multiple regexes | 16:37 |
| inc0 | also, does --all means just centos binary? | 16:37 |
| sdake | --all is good idea | 16:37 |
| pbourke | inc0: i like that | 16:37 |
| sdake | inc0 yes just the default | 16:38 |
| inc0 | because you know that with regex we can ditch --type binary | 16:38 |
| pbourke | yes centos binary is default | 16:38 |
| sdake | inc0 i prefer nto make the building process require regex expereince | 16:38 |
| inc0 | ok | 16:38 |
| sdake | if people want to use some complex regex thats fine | 16:38 |
| inc0 | so ./build.py neutorn --type source will work | 16:38 |
| sdake | but i dont want to make it mandatory theybe a regex wizard | 16:38 |
| inc0 | well..this wont, but with "neutron" it would | 16:39 |
| sdake | the regex shuld be the last thing specified | 16:43 |
| sdake | and tht shoudl be in the hep documentation | 16:43 |
| sdake | the lsit of regexes rather | 16:43 |
| sdake | ./build.py --type source neutron heat ceilometer | 16:44 |
| sdake | should build all neutorn heat ceilometer related junk | 16:44 |
| sdake | pbourke thanks for the swift work :) | 16:47 |
| sdake | i'm really keen to try that out | 16:47 |
| pbourke | sdake: yw hope it works for you | 16:47 |
| sdake | i'll let you know | 16:47 |
| sdake | does master work atm? | 16:47 |
| pbourke | yeah I ran through the AIO from scratch and it worked well | 16:48 |
| pbourke | I should try multinode too I guess | 16:48 |
| sdake | did you rm -rf /opt/kolla first? | 16:48 |
| pbourke | yes | 16:48 |
| sdake | nice | 16:48 |
| openstackgerrit | Michal Jastrzebski (inc0) proposed stackforge/kolla: Build.py regex filtering https://review.openstack.org/215523 | 16:48 |
| openstackgerrit | Michal Jastrzebski (inc0) proposed stackforge/kolla: build.py refactor https://review.openstack.org/215504 | 16:48 |
| openstackgerrit | Michal Jastrzebski (inc0) proposed stackforge/kolla: Build single image https://review.openstack.org/215524 | 16:48 |
| sdake | ok brainstorming session real quick guys | 16:49 |
| sdake | I found someone that wants to make a big contribution | 16:49 |
| inc0 | soo...we want multiple regexes support? | 16:49 |
| sdake | inc0 yes | 16:49 |
| inc0 | k | 16:49 |
| sdake | so I'mgoing to point them at installable package | 16:49 |
| sdake | as in setup.py | 16:49 |
| sdake | where do we want files installed | 16:50 |
| openstackgerrit | Merged stackforge/kolla: Prepare for gating by distro, type, docker type https://review.openstack.org/213537 | 16:50 |
| inc0 | uhh let's hold on a moment | 16:50 |
| pbourke | off the top of my head most should go in /opt/kolla | 16:51 |
| pbourke | with confs in /etc/kolla | 16:51 |
| sdake | installable package is a dependency for pushing to docs.openstack.org so we are not holding on on this one | 16:51 |
| inc0 | if we install kolla packages, that means we eihter break python convention or put dockerifles to dist-packages | 16:51 |
| sdake | dockerfiles can go to /usr/share/kolla/dockerfiles_templates | 16:51 |
| inc0 | what that install would do anyway? | 16:51 |
| *** unicell has joined #kolla | 16:51 | |
| sdake | this is what we are brainstorming inc0 :) | 16:52 |
| inc0 | alias wherever/build.py kolla ? | 16:52 |
| inc0 | I mean...it will just move files from one place to another | 16:52 |
| inc0 | no PATH inclusions | 16:52 |
| sdake | build.py would have to searhch for docker files in /usr/share/kolla/dockerfile_templates | 16:52 |
| sdake | we can do build.py as a separate patch imo | 16:52 |
| sdake | the idea is to ge tthe structure in place | 16:52 |
| sdake | so I canget on with my sphinx work | 16:53 |
| inc0 | most of sphinx works would be on dockerfiles tho | 16:53 |
| inc0 | I mean should | 16:53 |
| inc0 | and manuals | 16:53 |
| inc0 | I'm not sure how sphinx handles docker | 16:53 |
| pbourke | sdake: i'm about to head off for the weekend. if you find issues I'll look at them first thing monday, Id be surprised if there's anythin major. Would also appreciate a look in on https://review.openstack.org/#/c/207545/ | 16:54 |
| inc0 | and ansible ofc | 16:54 |
| sdake | sphinx isn't going to build docker | 16:54 |
| sdake | pbourke can yu give me 5 minutes for brainstorm? | 16:54 |
| sdake | or do you need to roll | 16:54 |
| pbourke | no that cool | 16:54 |
| sdake | pbourke i wil check out the review | 16:54 |
| pbourke | is this on where to install stuff? | 16:55 |
| inc0 | we could use some ansible magic to make ansible-playbook lots-of-arguments something like kolla install --aio | 16:55 |
| sdake | ok so we have etc that needs installed we have ansible that needs installed we have kolla-ansible that needs installed | 16:55 |
| inc0 | this would mean python entrypoints wizadry | 16:55 |
| sdake | we have build.py that needs to be renamed | 16:55 |
| sdake | I would be satisified with something basic to begin with | 16:56 |
| sdake | and iterate | 16:56 |
| pbourke | there's all the tools | 16:56 |
| inc0 | /var/lib/kolla for dockerfiles and dist-packages for python? | 16:56 |
| sdake | /var/lib is for persistent files | 16:56 |
| pbourke | well, not all needed but dont forget them | 16:56 |
| inc0 | /opt is ugly for installed stuff I think | 16:57 |
| sdake | thinking /usr/share/kolla/ansible foransible bits | 16:57 |
| sdake | /usr/share/kolla/dockerfile_templates for docker templates | 16:57 |
| inc0 | usr/share/kolla/docker for docker then? | 16:57 |
| sdake | yup | 16:57 |
| sdake | /usr/share is for non variable data :) | 16:57 |
| inc0 | make it docker, we're moving to templates as default;) | 16:57 |
| *** jmccarthy has quit IRC | 16:57 | |
| sdake | ok so tools directory goes where? | 16:58 |
| sdake | /usr/bin? | 16:58 |
| pbourke | yeah | 16:58 |
| *** achanda has joined #kolla | 16:58 | |
| sdake | i dont know if we really want to install all of the tools | 16:58 |
| inc0 | usr/local/bin | 16:58 |
| pbourke | that's what i was saying | 16:58 |
| sdake | not /usrlocal/bin | 16:58 |
| pbourke | just some of them | 16:58 |
| sdake | this is for a poroper install | 16:58 |
| inc0 | which ones? | 16:58 |
| inc0 | besides build | 16:58 |
| sdake | ya lets make the lsit now | 16:58 |
| sdake | what wouold people prpose | 16:58 |
| sdake | kolla-ansible and build.py are what i prpose | 16:59 |
| sdake | anything else? | 16:59 |
| inc0 | how about we turn both of these to python libs and expose single entrypoint? | 16:59 |
| inc0 | kolla build, kolla deploy | 16:59 |
| sdake | kolla-ansible is a shell script | 16:59 |
| sdake | inc0 i am looking for something quick and dirty | 17:00 |
| inc0 | simple enough to be turn python in 15mins | 17:00 |
| sdake | not perfection | 17:00 |
| sdake | just to unblock my sphinx work | 17:00 |
| inc0 | but if we teach people to use cli in liberty, we'll be stuck with it for some time | 17:00 |
| sdake | inc0 unblock sphinx work is goal here | 17:00 |
| sdake | we can tackle rest of it in liberty-rc1 | 17:01 |
| inc0 | I'm ok with rc1 | 17:01 |
| inc0 | as long as we'll have more stable cli in liberty itselfd | 17:01 |
| sdake | so i need basic infrastructure right now though | 17:01 |
| sdake | to get docs publishedon docs.openstack.org | 17:01 |
| sdake | ok | 17:01 |
| sdake | so entrypoints are future followon work | 17:01 |
| sdake | what to do about etc directory | 17:02 |
| sdake | just replace what is in /etc/kolla? | 17:04 |
| inc0 | yeah, it works nice | 17:04 |
| sdake | ok so whicch toosl do we need installed besides build and kolla-ansible | 17:07 |
| *** jmccarthy has joined #kolla | 17:08 | |
| *** ccrouch has quit IRC | 17:09 | |
| *** ccrouch1 has joined #kolla | 17:09 | |
| openstackgerrit | Michal Jastrzebski (inc0) proposed stackforge/kolla: Build.py regex filtering https://review.openstack.org/215523 | 17:12 |
| openstackgerrit | Michal Jastrzebski (inc0) proposed stackforge/kolla: Build single image https://review.openstack.org/215524 | 17:12 |
| inc0 | I think that's it | 17:12 |
| *** ccrouch1 has quit IRC | 17:17 | |
| *** ccrouch has joined #kolla | 17:21 | |
| openstackgerrit | Michal Jastrzebski (inc0) proposed stackforge/kolla: Add custom code to base docker templates https://review.openstack.org/215541 | 17:22 |
| openstackgerrit | Michal Jastrzebski (inc0) proposed stackforge/kolla: Add custom code to base docker templates https://review.openstack.org/215541 | 17:25 |
| inc0 | 11 lines of code, 3 bps in partially implements | 17:25 |
| inc0 | next one will implement more blueprints than lines of code | 17:25 |
| inc0 | ok, I think I've done enough for today | 17:26 |
| inc0 | kick SamYaple from me when he wakes up, for that -2 | 17:26 |
| inc0 | good night everyone | 17:26 |
| *** inc0 has quit IRC | 17:27 | |
| *** ccrouch has quit IRC | 17:29 | |
| *** jmccarthy has quit IRC | 17:30 | |
| *** ccrouch has joined #kolla | 17:30 | |
| *** tfukushima has quit IRC | 17:32 | |
| *** ccrouch has quit IRC | 17:34 | |
| *** ccrouch1 has joined #kolla | 17:34 | |
| *** bradjones has quit IRC | 17:37 | |
| *** ccrouch1 has quit IRC | 17:37 | |
| *** ccrouch has joined #kolla | 17:38 | |
| *** bradjones has joined #kolla | 17:38 | |
| *** bradjones has quit IRC | 17:38 | |
| *** bradjones has joined #kolla | 17:38 | |
| *** akwasnie has quit IRC | 17:39 | |
| *** ccrouch has quit IRC | 17:41 | |
| *** achanda has quit IRC | 17:44 | |
| openstackgerrit | Merged stackforge/kolla: Added missing configuration to horizon docker template. https://review.openstack.org/215511 | 17:46 |
| *** jasonamyers has quit IRC | 17:46 | |
| *** loth has joined #kolla | 17:48 | |
| *** apuimedo has quit IRC | 17:48 | |
| *** tummy has quit IRC | 17:55 | |
| *** ashishb has quit IRC | 18:06 | |
| *** achanda has joined #kolla | 18:07 | |
| *** achanda has quit IRC | 18:08 | |
| *** achanda has joined #kolla | 18:08 | |
| *** achanda has quit IRC | 18:09 | |
| openstackgerrit | Michal Jastrzebski (inc0) proposed stackforge/kolla: Add custom code to base docker templates https://review.openstack.org/215541 | 18:21 |
| *** ccrouch has joined #kolla | 18:36 | |
| *** absubram has quit IRC | 18:38 | |
| *** absubram has joined #kolla | 18:41 | |
| *** tfukushima has joined #kolla | 18:42 | |
| *** ccrouch has quit IRC | 18:43 | |
| *** sdake has quit IRC | 18:46 | |
| *** ccrouch has joined #kolla | 18:46 | |
| *** ccrouch has quit IRC | 18:47 | |
| *** ccrouch has joined #kolla | 18:47 | |
| *** tummy has joined #kolla | 19:00 | |
| SamYaple | alright having read the scrollback I want to point out that whether operators will modify the build.py code or not, they _will_ be reading it | 19:00 |
| SamYaple | reading openstack source is basically part of the job | 19:00 |
| SamYaple | there is no reason to use lambda in this case it is not well know how to understand it and it makes it less readbale in my opinon | 19:00 |
| SamYaple | all while saving 1 single line | 19:00 |
| *** banix has joined #kolla | 19:01 | |
| SamYaple | 1 line.... | 19:01 |
| SamYaple | oh and you can do everything in python wihtout a lambda, there is nothing that requires it | 19:02 |
| SamYaple | after all it is just a function on the fly | 19:02 |
| *** sdake has joined #kolla | 19:03 | |
| *** tummy has quit IRC | 19:04 | |
| sdake | yo | 19:04 |
| SamYaple | wut wut | 19:04 |
| sdake | samyaple you left a comment i was missing data containres | 19:04 |
| sdake | did yu mean i forgot to remove some? | 19:04 |
| SamYaple | yes, was that not clear? | 19:05 |
| SamYaple | i dont remember what i said | 19:05 |
| sdake | yes just verifying | 19:05 |
| sdake | before i pull down the patch and rewor kit | 19:05 |
| sdake | samyaple i found someone to do setup.py | 19:06 |
| sdake | i need this for sphinx doc generaiton | 19:06 |
| *** tfukushima has quit IRC | 19:06 | |
| SamYaple | whats his name? | 19:06 |
| sdake | for docs.openstack.org | 19:06 |
| sdake | suhail | 19:06 |
| SamYaple | ah ok | 19:06 |
| sdake | internal dude | 19:06 |
| SamYaple | thought it might have been someone i know | 19:06 |
| sdake | have any requirements on that work yo uwould like to express | 19:06 |
| jpeeler | what exactly is getting packaged? | 19:07 |
| jpeeler | all the testr work done was written to avoid usage of setup.py :/ | 19:07 |
| sdake | we are putting ansible in /usr/locla/kolla/asible | 19:07 |
| sdake | all etc is going in /etc/kolla | 19:07 |
| SamYaple | jpeeler: the whole kolla project | 19:07 |
| SamYaple | pip install kolla | 19:07 |
| SamYaple | that would pull down the dockerfiles and everything | 19:07 |
| sdake | buildl.py and ansible-kolla are going in /usr/bin | 19:08 |
| SamYaple | and install /etc/ | 19:08 |
| sdake | docker is going in /usr/local/koall/docker | 19:08 |
| sdake | docker_teomplates is going in /usr/local/kolla/docker_templates | 19:08 |
| SamYaple | i have no opinions on this, as long as /etc stays I am good | 19:08 |
| SamYaple | directory wise | 19:08 |
| sdake | you mean existing etc doesn't get overwritten? | 19:09 |
| sdake | or etc gets installed? | 19:10 |
| SamYaple | /etc/kolla | 19:10 |
| *** tummy has joined #kolla | 19:10 | |
| sdake | the main goal of his work is to get sphinx in a runnable state | 19:10 |
| SamYaple | yup | 19:10 |
| sdake | so it may need a bit of rework after | 19:10 |
| sdake | could you clarify yup on etc, you want it to stay or be copied over? | 19:11 |
| SamYaple | i dont know whats being asked, i defer to you at this time | 19:11 |
| * SamYaple is a bit screwed up at the moment) | 19:11 | |
| sdake | the plan is to replae whatever is in /etc/kolla with what is in the epo | 19:11 |
| SamYaple | that will wipe settings and passwords, that is a bad idea | 19:12 |
| *** tummy has quit IRC | 19:12 | |
| sdake | ok woud you propose not copying /etc then? | 19:12 |
| SamYaple | no it just needs to be a smart copy. not overwrite if existing, and (maybe not immediately) but smart upgrading of existing files | 19:13 |
| SamYaple | adding new options/changing names | 19:13 |
| sdake | i dont know if there is a way to conditionally copy /etc | 19:13 |
| SamYaple | overwritting is really bad | 19:13 |
| sdake | agreed | 19:13 |
| sdake | lets leave etc out of this patch and file a tech debt bug for that | 19:14 |
| sdake | agree? | 19:14 |
| SamYaple | sure | 19:14 |
| *** absubram has quit IRC | 19:17 | |
| sdake | any other requirements for setup.py? | 19:21 |
| SamYaple | I dont think so | 19:22 |
| SamYaple | i dont know | 19:22 |
| sdake | we can todo any remaining stuff | 19:23 |
| sdake | like etc | 19:23 |
| sdake | and the special magic around loading /bin files | 19:24 |
| sdake | setup.cfg entrypoints is what i'm referring to | 19:24 |
| sdake | are there other binries we want to install besides kolla-ansible and build.py? | 19:25 |
| sdake | samyaple comment on https://review.openstack.org/#/c/215504/ for you needs attention if you wouldn't mind | 19:27 |
| sdake | -2 kills a review, i'd lie it unkilled with the proposed change you suggested | 19:27 |
| SamYaple | i -2 because of the talk of getting it merged behind my back :/ | 19:28 |
| sdake | that isn't going to happen | 19:28 |
| SamYaple | Well not _now_ | 19:28 |
| sdake | i'll address that in the wednesday meeting if you have logs | 19:29 |
| sdake | core reviewers should not be merging code behind peoples back | 19:29 |
| SamYaple | It wasn't cores saying this | 19:29 |
| SamYaple | its fine though, I will update the vote | 19:29 |
| sdake | tx | 19:29 |
| SamYaple | Are you saying -2 blcoks new patches? | 19:29 |
| SamYaple | i wasnt aware of that | 19:30 |
| sdake | nah but it kills the erview | 19:30 |
| sdake | people give up on -2 usually :( | 19:30 |
| SamYaple | I did say in the review I would remove it once that was removed | 19:30 |
| sdake | you are free to -2 as you like | 19:30 |
| sdake | but if we find a solution that is agreeable would like otes changed to -1 :) | 19:30 |
| openstackgerrit | Jeff Peeler proposed stackforge/kolla: Add options for selective build and listing images https://review.openstack.org/215323 | 19:30 |
| jpeeler | should i just abandon that? ^ | 19:30 |
| SamYaple | I just didnt appreciate the 'ill submit when you arent around' | 19:31 |
| sdake | ok fair then samyaple | 19:31 |
| sdake | i'll ddress it in the meeting | 19:31 |
| sdake | if there was that type of comment i'd keep a -2 on the patch until it was changed too | 19:31 |
| sdake | until the ptl sorted out the conflict | 19:31 |
| SamYaple | jpeeler: it looks like inc0 just pulled in your changes that I wanted (free form args rather than --regex option) so its up to you | 19:32 |
| SamYaple | I am ok with your change, but i see the appeal of inc0 approach | 19:32 |
| sdake | jpeeler i think the dirction is to go with inc0's patch | 19:32 |
| sdake | jpeeler your patch is good too | 19:32 |
| sdake | its too bad you both did the same job -waste of time | 19:32 |
| jpeeler | well i read that he was going to wait for me | 19:32 |
| sdake | this is the problem with global blueprints for features | 19:32 |
| SamYaple | inc0 knew jpeeler was doing this :/ | 19:32 |
| jpeeler | so yes, it's a shame that it occurred | 19:33 |
| SamYaple | i was disappointed as well | 19:33 |
| sdake | i dont know what to tell you its not like I control other people's behavior :) | 19:33 |
| SamYaple | if I believe your statement, does that mean you are controling me? | 19:34 |
| sdake | influence and control are two different things | 19:35 |
| sdake | control means "do this" "it gets done" | 19:35 |
| sdake | infleunce means "can you do this" "ok I agree" | 19:35 |
| sdake | i do influence peoples behavior :) | 19:36 |
| sdake | just like you do :) | 19:36 |
| sdake | your influencing my behavior around the work suhaiil is doingaround setuppy for example | 19:36 |
| sdake | make sense? | 19:37 |
| sdake | samyaple can you approve my doc changes pls or jpeeler | 19:38 |
| SamYaple | link? | 19:39 |
| sdake | lets iterate on them | 19:39 |
| sdake | second | 19:39 |
| sdake | https://review.openstack.org/#/c/215532/ | 19:39 |
| sdake | and its children patches | 19:39 |
| SamYaple | sdake: left a review on the final patch https://review.openstack.org/#/c/215532/ | 19:45 |
| SamYaple | the others are good | 19:45 |
| openstackgerrit | Merged stackforge/kolla: Prepare for MD to RST conversion https://review.openstack.org/215463 | 19:47 |
| openstackgerrit | Merged stackforge/kolla: Run pandoc to convert the documentation https://review.openstack.org/215464 | 19:47 |
| sdake | thanks sam I will adjust | 19:48 |
| jpeeler | lot of spelling errors | 19:49 |
| jpeeler | as far as having multiple dev envs, i'd be happy to see vagrant be the preferred bootstrap tool | 19:51 |
| *** ccrouch has quit IRC | 19:51 | |
| SamYaple | waht are you takling about jpeeler? he spells the greetest! | 19:51 |
| jpeeler | heh | 19:51 |
| sdake | jpeeler add comments i will fix | 19:54 |
| jpeeler | i did | 19:54 |
| sdake | i dont mind fixing spellings errors i just dont want alot of retructure | 19:54 |
| sdake | i already did the restructure | 19:54 |
| sdake | if we do more restructure it can be new patches | 19:54 |
| sdake | i'd like vagrant to be the primary development environment butit only works with virutalbox | 19:55 |
| sdake | and the documentation around heat is not up to date | 19:55 |
| sdake | that is coming in a followon patch | 19:55 |
| sdake | i wrote those docs at 2am | 19:55 |
| jpeeler | sdake: i have a patch to make it work with libvirt | 19:55 |
| sdake | so was a bit bleary eyed | 19:55 |
| sdake | jpeeler fantastic!!!! | 19:55 |
| sdake | i can't wiat to test it out | 19:55 |
| jpeeler | but i'm having difficulty verifying multinode | 19:55 |
| jpeeler | so i've been holding off on posting it (as well as what the network should look like) | 19:56 |
| jpeeler | but yeah, libvirt should become the default in my opinion. great that it works with virtualbox too though | 19:56 |
| SamYaple | jpeeler: a docs patch for libvirt? | 19:56 |
| sdake | i'd like heat to be n option too but it needs updating | 19:56 |
| sdake | samyaple no its an implementation patch | 19:57 |
| jpeeler | SamYaple: not docs, libvirt requires a modified Vagrantfile and image | 19:57 |
| sdake | vagrant code atm is hard coded to only work with virtual box | 19:57 |
| SamYaple | oh i see | 19:57 |
| sdake | it should work with both | 19:57 |
| SamYaple | cool | 19:57 |
| sdake | if we need separate vagrant files put em in separate dirs | 19:57 |
| SamYaple | +1 on heat | 19:57 |
| SamYaple | I do most of my devel on instances right now | 19:58 |
| jpeeler | honestly i think it's weird to say, hack on openstack, but you need openstack to do it | 19:58 |
| sdake | yes we have two development environments plus one bare metal deploy doc | 19:58 |
| SamYaple | the instances are running on Openstack running Kolla! on my gear here | 19:58 |
| sdake | yup we have a cisco lab where i can test like 50 nodes with ehat | 19:58 |
| sdake | that would rock | 19:58 |
| sdake | with heat that is | 19:58 |
| sdake | but the heat template needs updating | 19:59 |
| jpeeler | SamYaple: do you know of a way to get rackspace instances for openstack dev? | 19:59 |
| sdake | its still based upon compose | 19:59 |
| sdake | maybe the docs just need updating | 19:59 |
| jpeeler | i remember seeing something a while back about $500 credit or something | 19:59 |
| sdake | jpeeler you gotta pay | 19:59 |
| SamYaple | jpeeler: with $$ | 19:59 |
| SamYaple | i mean ive got a free account for internal, but i work there ;) | 19:59 |
| SamYaple | I never actually use it | 19:59 |
| sdake | ok | 19:59 |
| SamYaple | jpeeler: have you see the OSIC stuff Intel and RAX are planning? | 20:01 |
| sdake | hey samyaple | 20:02 |
| SamYaple | hi sdake | 20:02 |
| sdake | feedbck from operators summit, wife said almsot every operator is using linuxbridge not ovs | 20:02 |
| jpeeler | SamYaple: i didn't, but i just read the blog announcement. 1000 node cluster should be useful | 20:03 |
| SamYaple | 2000 | 20:04 |
| openstackgerrit | Merged stackforge/kolla: Add custom code to base docker templates https://review.openstack.org/215541 | 20:05 |
| jpeeler | 1000*2 yeah. wonder how they'll grant access | 20:05 |
| SamYaple | sdake: who can blame them? very few people use the right versions of OVS, and ubuntu (the most used distro for openstack last i heard) has OVS 2.0.2 (a known broken devel version) in the repo | 20:05 |
| SamYaple | sdake: that being said DVR will never be on OVS and the neutron people only really care about OVS, linuxbridge was always a patch job onto of neutron | 20:05 |
| SamYaple | and it seems to always be a secondary thought | 20:06 |
| SamYaple | jpeeler: as a member of this community (big tent esspecially) youll be able to schedule time to access it | 20:06 |
| sdake | seems secondary to develdops but primary to operators | 20:06 |
| SamYaple | thats because linuxbridge 'just works' | 20:06 |
| sdake | pointbeing, good thing we have linuxbridge support ;) | 20:07 |
| SamYaple | but openvswitch has the features and performance needed for the future | 20:07 |
| SamYaple | indeed | 20:07 |
| sdake | or we would have to scramble to make it happen | 20:07 |
| SamYaple | Nah its a piece of cake | 20:07 |
| SamYaple | Wait until i setup the configs to do 1 click DVR deploy | 20:07 |
| SamYaple | lets see how many people use OVS then ;) | 20:08 |
| sdake | nice | 20:08 |
| sdake | anyway i thought that was cool feedback :) | 20:08 |
| SamYaple | it is good. that is suprising though since as late as havana i didnt know anyone using linuxbridge | 20:08 |
| sdake | are all the docker_templates patches merged? | 20:08 |
| SamYaple | i guess people go burned on early OVS | 20:09 |
| sdake | for centos + binary | 20:09 |
| SamYaple | looks like it | 20:09 |
| sdake | and has anyone tested it? | 20:09 |
| SamYaple | https://review.openstack.org/#/q/project:stackforge/kolla+branch:master+topic:bp/dockerfile-template,n,z | 20:09 |
| sdake | (I haven't) | 20:09 |
| SamYaple | i only run templates | 20:09 |
| SamYaple | been that way for a week | 20:09 |
| sdake | nice on centos binary? | 20:09 |
| SamYaple | yea, but i havent tested booting for a few days | 20:09 |
| sdake | cool thanks for the info | 20:10 |
| SamYaple | last time was on mon or tuesday | 20:10 |
| SamYaple | sdake: https://review.openstack.org/#/c/215159/ | 20:10 |
| SamYaple | that change work for you? | 20:10 |
| sdake | wtb project rename already! | 20:10 |
| sdake | looking | 20:10 |
| sdake | samyaple i dont have time to verify - willing to +2 / +a but need verification it works with centos-binary | 20:12 |
| sdake | havey ou tried that part of it out? | 20:12 |
| SamYaple | yea | 20:12 |
| sdake | cool approving then | 20:12 |
| SamYaple | this doenst change anything for centos | 20:12 |
| SamYaple | you can actually run the playbooks again without getting a change | 20:12 |
| sdake | its just all that curdini stuff | 20:13 |
| sdake | the removals | 20:13 |
| sdake | i was worried about that in galera.cnf.j2 | 20:13 |
| SamYaple | oh that was cleanup from removing config-interal | 20:13 |
| sdake | and that definately works post rebuild? | 20:13 |
| SamYaple | i could have done a seperate patch, it just happened | 20:13 |
| SamYaple | yea | 20:13 |
| sdake | ok | 20:13 |
| SamYaple | we never used those functions for config-external | 20:13 |
| sdake | didn't know that | 20:13 |
| SamYaple | i only integrated the two that are left with config-external | 20:14 |
| *** tummy has joined #kolla | 20:14 | |
| *** tummy has quit IRC | 20:16 | |
| openstackgerrit | Merged stackforge/kolla: Fix MariaDB for ubuntu https://review.openstack.org/215159 | 20:18 |
| openstackgerrit | Jeff Peeler proposed stackforge/kolla: Add templated dockerfiles for ironic https://review.openstack.org/215770 | 20:33 |
| sdake | jpeeler rockin dude ;) | 20:36 |
| sdake | progress ftw | 20:36 |
| SamYaple | jpeeler: is this a new container, or migrations frok docker folder? | 20:37 |
| jpeeler | untested, but they build | 20:37 |
| jpeeler | SamYaple: new | 20:37 |
| SamYaple | ok cool | 20:38 |
| jpeeler | SamYaple: so remove the run line or the entire if endif section? | 20:40 |
| SamYaple | jpeeler: just the entire section | 20:40 |
| SamYaple | thats what we do with the others | 20:40 |
| openstackgerrit | Jeff Peeler proposed stackforge/kolla: Add templated dockerfiles for ironic https://review.openstack.org/215770 | 20:45 |
| *** jtriley has quit IRC | 20:53 | |
| sdake | harmw ping | 20:57 |
| harmw | shoot me! | 20:59 |
| sdake | ;) | 20:59 |
| sdake | horizon | 20:59 |
| harmw | yep, tomorrow | 20:59 |
| sdake | in the forecast? | 20:59 |
| harmw | saturdayevening | 20:59 |
| sdake | nice | 20:59 |
| sdake | start with the patch up in gerrit | 20:59 |
| harmw | yep | 20:59 |
| sdake | you can use the cherry-pick tab to fetch it | 21:00 |
| sdake | on top of master | 21:00 |
| sdake | it probably needs a rebae | 21:00 |
| harmw | yes sir | 21:00 |
| sdake | and rework to remove the start.yml and stuff abstractions | 21:00 |
| sdake | whatever state you get it into before ou head off to bed | 21:00 |
| sdake | please push it | 21:00 |
| sdake | rather git revew it | 21:00 |
| harmw | oh ok | 21:00 |
| sdake | i may pick it up so we can work it 24 hours :) | 21:00 |
| harmw | fair enough | 21:01 |
| harmw | :) | 21:01 |
| sdake | and then i'll push | 21:01 |
| sdake | and youcan pick up possibly sunday :) | 21:01 |
| harmw | lets see how far I can get tomorrow | 21:01 |
| sdake | basically what i'm say8ing is elts work it together | 21:01 |
| harmw | I hear you | 21:01 |
| sdake | i'd like it merged before tuesday if at all possible | 21:01 |
| harmw | ill keep that in mind | 21:02 |
| *** banix has quit IRC | 21:22 | |
| *** weiyu_ has joined #kolla | 21:24 | |
| *** ccrouch has joined #kolla | 21:33 | |
| *** ccrouch has quit IRC | 21:33 | |
| *** ccrouch has joined #kolla | 21:33 | |
| *** dolpher has joined #kolla | 21:36 | |
| *** tummy has joined #kolla | 21:42 | |
| *** bradjones has quit IRC | 21:42 | |
| *** bradjones has joined #kolla | 21:43 | |
| *** bradjones has quit IRC | 21:43 | |
| *** bradjones has joined #kolla | 21:43 | |
| *** jruano has joined #kolla | 21:56 | |
| *** ccrouch has quit IRC | 21:57 | |
| *** dolpher1 has joined #kolla | 22:01 | |
| *** dolpher has quit IRC | 22:03 | |
| *** blahRus has quit IRC | 22:05 | |
| *** weiyu_ has quit IRC | 22:12 | |
| openstackgerrit | Steven Dake proposed stackforge/kolla: Rework documentaton to actually get people rolling https://review.openstack.org/215532 | 22:17 |
| sdake | jpeeler around? | 22:25 |
| *** achanda has joined #kolla | 22:38 | |
| *** dolpher has joined #kolla | 22:42 | |
| *** dolpher1 has quit IRC | 22:44 | |
| *** banix has joined #kolla | 22:47 | |
| *** jasonsb has quit IRC | 22:49 | |
| *** tummy has quit IRC | 22:49 | |
| *** banix has quit IRC | 23:05 | |
| *** dolpher1 has joined #kolla | 23:05 | |
| *** dolpher has quit IRC | 23:07 | |
| *** stvnoyes has quit IRC | 23:25 | |
| *** daneyon has quit IRC | 23:34 | |
| *** jruano has quit IRC | 23:37 | |
| *** banix has joined #kolla | 23:53 | |
| *** jruano has joined #kolla | 23:55 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!