*** jasonsb has quit IRC | 00:00 | |
-openstackstatus- NOTICE: Zuul was restarted due to an error; events (such as approvals or new patchsets) since 23:01 UTC have been lost and affected changes will need to be rechecked | 00:04 | |
*** diogogmt has quit IRC | 00:12 | |
sdake | anyone know a way to print 19 spaces in python without pressing the space key 19 times | 00:16 |
---|---|---|
sdake | and without using '%-19s'% '' | 00:17 |
bmace | you could do it in a for loop? :) | 00:21 |
*** jruano has quit IRC | 00:23 | |
*** bmace has quit IRC | 00:32 | |
*** achanda_ has joined #kolla | 00:35 | |
*** zhiwei has joined #kolla | 00:36 | |
*** ccrouch has quit IRC | 00:37 | |
*** achanda has quit IRC | 00:38 | |
*** achanda_ has quit IRC | 00:39 | |
*** thumpba has quit IRC | 00:42 | |
*** bmace has joined #kolla | 00:52 | |
*** thumpba has joined #kolla | 00:56 | |
*** thumpba has quit IRC | 00:59 | |
*** thumpba has joined #kolla | 01:08 | |
*** thumpba_ has joined #kolla | 01:13 | |
*** thumpba has quit IRC | 01:14 | |
*** thumpba_ has quit IRC | 01:15 | |
*** thumpba has joined #kolla | 01:15 | |
sdake | been working on diversity script a bit: | 01:20 |
sdake | http://paste.openstack.org/show/412638/ | 01:20 |
*** weiyu has joined #kolla | 01:20 | |
sdake | if you want to play with it you can pull itfrom here https://review.openstack.org/211799 | 01:21 |
*** wshao has joined #kolla | 01:23 | |
*** diogogmt has joined #kolla | 01:28 | |
*** britthouser has quit IRC | 01:30 | |
*** erkules has joined #kolla | 01:32 | |
*** jasonsb has joined #kolla | 01:34 | |
*** erkules_ has quit IRC | 01:35 | |
*** tobe_ has joined #kolla | 01:39 | |
*** britthouser has joined #kolla | 01:39 | |
*** jruano has joined #kolla | 01:40 | |
*** erkules has quit IRC | 01:52 | |
*** erkules has joined #kolla | 01:54 | |
SamYaple | heyo | 01:59 |
*** Haomeng|2 has joined #kolla | 02:00 | |
*** Haomeng has quit IRC | 02:03 | |
*** thumpba has quit IRC | 02:04 | |
*** thumpba has joined #kolla | 02:05 | |
*** thumpba has quit IRC | 02:05 | |
*** Haomeng has joined #kolla | 02:15 | |
*** Haomeng|2 has quit IRC | 02:15 | |
*** unicell has quit IRC | 02:20 | |
SamYaple | sdake: https://review.openstack.org/#/c/202216/ | 02:23 |
SamYaple | pleez | 02:23 |
sdake | samyaple looking | 02:24 |
openstackgerrit | Merged stackforge/kolla: Improve logging and prepare for gate https://review.openstack.org/202216 | 02:29 |
*** jruano has quit IRC | 02:34 | |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Add Dockerfile template for rabbitmq https://review.openstack.org/211817 | 02:51 |
SamYaple | sdake: much cleaner, this dockerfile is | 02:51 |
*** dolpher has joined #kolla | 02:52 | |
*** britthouser has quit IRC | 02:52 | |
sdake | let me light up a kill stick and i'll have a look | 02:54 |
sdake | sam can you do me a solid and put how to build the containers in the etherpad | 03:00 |
SamYaple | ./build.py --template ? | 03:00 |
sdake | because I'm not really sure how to build templated containers | 03:00 |
sdake | and i'm sure everyone will ask | 03:00 |
SamYaple | its literally in the help file | 03:00 |
sdake | that is it? | 03:00 |
SamYaple | ./build.py -h | 03:00 |
SamYaple | yea | 03:00 |
*** dolpher1 has joined #kolla | 03:00 | |
sdake | -t rdo or -t source? | 03:01 |
SamYaple | those all work, but we havent been implementing the rdo tag | 03:01 |
SamYaple | binary and source are all we implemented | 03:01 |
sdake | that is good | 03:01 |
sdake | so -t binary? | 03:01 |
SamYaple | which is default, but yes | 03:01 |
sdake | how do you select the distro base? | 03:02 |
SamYaple | please run ./build.py -h | 03:02 |
sdake | i'll just looka t the source | 03:02 |
sdake | i have to disconnec t to do that | 03:02 |
SamYaple | :/ | 03:02 |
*** ccrouch has joined #kolla | 03:02 | |
SamYaple | helpfile is there for a reason | 03:02 |
sdake | ya this vpn drives me crazy | 03:02 |
*** jruano has joined #kolla | 03:03 | |
sdake | holy shit jpeleer grew facial hair | 03:03 |
SamYaple | im getting google fiber in san antonio | 03:03 |
SamYaple | funny i am clean shaven at the moment | 03:03 |
sdake | i've got fibre from cox coming end of month | 03:04 |
*** dolpher has quit IRC | 03:04 | |
*** banix has quit IRC | 03:08 | |
sdake | do we hae a working debian base image? | 03:12 |
sdake | samyaple ^^ | 03:13 |
*** dolpher has joined #kolla | 03:13 | |
SamYaple | techincially it should work, yes | 03:14 |
SamYaple | i dont think its been tested no | 03:14 |
*** tobe_ has quit IRC | 03:15 | |
*** dolpher1 has quit IRC | 03:16 | |
*** tobe_ has joined #kolla | 03:16 | |
sdake | samyaple to speed up the process have a look at my instructions https://etherpad.openstack.org/p/kolla-dockerfile-template | 03:21 |
*** weiyu has quit IRC | 03:24 | |
SamYaple | id remove debian | 03:25 |
SamYaple | also the source deploy for ubuntu isnt reall large | 03:26 |
SamYaple | really only source+centos is needed | 03:26 |
SamYaple | ubuntu only has 2 or 3 services with "source" | 03:26 |
SamYaple | also fedora | 03:26 |
SamYaple | i dont think fedora needs to be built and tested riht now | 03:27 |
*** achanda has joined #kolla | 03:27 | |
SamYaple | we know its broke (hence the bug yo uahve in your name) | 03:27 |
*** weiyu has joined #kolla | 03:29 | |
*** diogogmt has quit IRC | 03:29 | |
sdake | i'll fix fedora tonight | 03:31 |
sdake | as far as from source i'd like to make sure the builds work for ubuntu else it will create big rework later | 03:32 |
sdake | but ack on the lack of services | 03:32 |
SamYaple | funny thing is some of the ubuntu builds dont work right now | 03:32 |
SamYaple | so im not so worried | 03:32 |
SamYaple | I am about to go through all teh ubuntu container since i need them now | 03:33 |
SamYaple | but it can wait fro the templating to be dont | 03:33 |
SamYaple | done | 03:33 |
sdake | what is done | 03:34 |
sdake | reconnecting bbiaf | 03:34 |
*** sdake_ has joined #kolla | 03:35 | |
sdake_ | -T=1 is a cool feature | 03:38 |
sdake_ | oraclelinux | 03:39 |
*** sdake has quit IRC | 03:39 | |
sdake_ | INFO:__main__:base:No package epel-release available. | 03:39 |
openstackgerrit | Swapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add ansible support for Cinder https://review.openstack.org/208868 | 03:41 |
sdake_ | ubuntu builds | 03:42 |
coolsvap | sdake_, i might take a dig at ubuntu builds once i am finished with current work in hand | 03:43 |
coolsvap | regarding the the updated images on dockerhub, i think it might be worth a try to assign individuals for specific builds | 03:44 |
sdake_ | debian fails to build | 03:44 |
sdake_ | coolsvap i am not suggesting pushing | 03:44 |
coolsvap | maybe have a bug or something at each milestone to track status | 03:44 |
sdake_ | just building | 03:44 |
sdake_ | i can make one script that pushes all the changes per milestone | 03:45 |
sdake_ | it will probably take 10 hours to run or more | 03:45 |
sdake_ | i should have caught that epel thing in review | 03:46 |
* sdake_ ughs | 03:46 | |
coolsvap | yes, i would still suggest to break it down since it will help us track status and a point person who can provide updates | 03:46 |
*** sdake_ is now known as sdake | 03:46 | |
sdake | i am not concerned about docker registry builds atm | 03:47 |
coolsvap | because no one probably would track the build/push status for 10 hours | 03:47 |
sdake | i have a rax account where i run the build | 03:47 |
sdake | i dont want to disribute that job - its easy to put in one script and have one person care for it | 03:48 |
sdake | ubuntu builds from source and from binary for rabbitmq | 03:49 |
sdake | debian is busted | 03:49 |
sdake | centos and ubuntu build | 03:56 |
sdake | INFO:__main__:base:Yum command has been deprecated, redirecting to '/usr/bin/dnf install -y https://rdo.fedorapeople.org/rdo-release.rpm'. | 03:58 |
* sdake shakes fists at fedora | 03:59 | |
sdake | samyaple do you have a tidy solution for that problem? | 03:59 |
sdake | doen't hae to be now | 04:00 |
sdake | just before its removed | 04:00 |
*** jruano has quit IRC | 04:03 | |
sdake | INFO:__main__:base:/usr/share/man/man5/cert8.db.5.gz: No such file or directory | 04:05 |
sdake | cannot reconstruct rpm from disk files | 04:05 |
sdake | I don't even know what to make of that | 04:05 |
sdake | so here is the score, oraclelinux, fedora, and debian are broken | 04:06 |
sdake | only ubuntu and centos work | 04:06 |
*** unicell has joined #kolla | 04:06 | |
SamYaple | sdake: yea, the solution is an alias | 04:09 |
SamYaple | but once the tools really start to diverge the options will start to change | 04:09 |
sdake | we use install and update | 04:10 |
sdake | i guess they could change that | 04:10 |
sdake | but unlikely | 04:10 |
sdake | and clean all | 04:10 |
sdake | that could possibly change | 04:10 |
* sdake groans | 04:10 | |
sdake | those idiots said they were going to keep the cli intact | 04:11 |
SamYaple | then why change the name ;) | 04:12 |
sdake | oraclelinux from source fails to build | 04:14 |
sdake | INFO:__main__:rabbitmq:/bin/sh: /usr/lib/rabbitmq/bin/rabbitmq-plugins: No such file or directory | 04:14 |
SamYaple | i dont think oraclelinux has any from source stuff | 04:15 |
sdake | their entire stuff is from source | 04:16 |
SamYaple | https://github.com/stackforge/kolla/tree/master/docker/ol | 04:16 |
sdake | ya i know | 04:16 |
SamYaple | theres nothing there dude | 04:16 |
sdake | i just saw | 04:16 |
sdake | i wonder if they are only building centos or something | 04:17 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Add Dockerfile template for rabbitmq https://review.openstack.org/211838 | 04:17 |
sdake | need to ask pbourke | 04:17 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Add Dockerfile template for keystone https://review.openstack.org/211838 | 04:17 |
SamYaple | wrong name | 04:17 |
SamYaple | once more | 04:17 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Add Dockerfile template for keystone https://review.openstack.org/211838 | 04:17 |
sdake | base file and keystone are inconsistent | 04:23 |
sdake | one is if distro if type | 04:23 |
sdake | other is if type if distro | 04:23 |
*** weiyu has quit IRC | 04:25 | |
sdake | infra is if distro | 04:25 |
sdake | is there a good reason for openstack services using if type if distro? | 04:25 |
SamYaple | The other way requires alot of duplicate? | 04:26 |
SamYaple | the source part is the same across distros | 04:26 |
sdake | ok makes sense | 04:26 |
sdake | would have to have same if source block twice | 04:27 |
SamYaple | yea | 04:27 |
sdake | thanks for answering my q ;) | 04:27 |
SamYaple | the case is opposite for the base image since the depluicate is within the same distro rather than without | 04:27 |
sdake | ya well i didn't actually implement the stuff so thats why i asked | 04:28 |
SamYaple | not a problem im just busy | 04:28 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Add Dockerfile template for keystone https://review.openstack.org/211838 | 04:32 |
sdake | samyaple looks like you c aught my review comments | 04:33 |
sdake | over ESP | 04:33 |
SamYaple | and you complain when you dont understand me | 04:33 |
sdake | where does keystone.tar come from? | 04:43 |
sdake | samyaple ^^ trying to build yoru containers fails for me with keystone | 04:44 |
SamYaple | yea the source installs dont work with the build.py script yet | 04:45 |
sdake | I can see why you recommended no testing earlier ;-) | 04:46 |
SamYaple | yep | 04:47 |
sdake | that was sarcasm ;) | 04:47 |
sdake | what needs to change for build.py to build the tarballs? | 04:47 |
sdake | run a script or something? | 04:47 |
SamYaple | the way the current build scripts do it is run the code in .buildconf | 04:48 |
SamYaple | thats not teh way it should be done for the python code | 04:48 |
SamYaple | but no one has written the python code yet | 04:48 |
SamYaple | or added what it should look like | 04:49 |
sdake | you mean .buildinfo? | 04:49 |
SamYaple | yea | 04:49 |
SamYaple | .build* something | 04:49 |
sdake | why should it not be done that way? | 04:49 |
SamYaple | to get away from bash code and hidden files (the purpose f the new build script) | 04:49 |
SamYaple | a conf file or the like was suggested | 04:50 |
sdake | expand please | 04:50 |
sdake | like keystone.build.conf ? | 04:50 |
SamYaple | if python reads the conf it would have all the info on how and where to clone/pull/cp the tarball and then the methods for the python code would need ot be written | 04:51 |
SamYaple | no like a global one | 04:51 |
SamYaple | we are trying ot get away from each directory having config stuff | 04:51 |
sdake | ok, so global one containing info for every package? | 04:51 |
SamYaple | every openstack service, yes | 04:52 |
*** tobe_ has quit IRC | 04:53 | |
sdake | ini format or yaml? | 04:53 |
SamYaple | i dont mind yaml, however that does place a yaml requirement on the build server | 04:54 |
sdake | is ini built into python? | 04:54 |
SamYaple | yea | 04:54 |
SamYaple | configparser | 04:54 |
coolsvap | SamYaple, sdake a question regarding templates, can we skip the ubuntu/debian related stuff in template if its not there? | 04:57 |
coolsvap | with a note to add it later | 04:57 |
SamYaple | coolsvap: yea you dont have to implement anything that doesnt already exist | 04:57 |
SamYaple | just keep the structre there | 04:57 |
sdake | nothing exists for ubuntu right? | 04:57 |
* sdake can't keep up with the state of change of the repo :) | 04:57 | |
SamYaple | there is stuff for ubuntu | 04:58 |
sdake | let me look second | 04:58 |
SamYaple | actually coolsvap can you put 'RUN /bin/fasle' in there if it doesnt exist? | 04:58 |
SamYaple | to force the container ot not build | 04:58 |
sdake | only rabbitmq nova-base and keystone | 04:58 |
sdake | oh wrongdir | 04:58 |
sdake | no correct dir | 04:59 |
sdake | there are no services implemented from source for ubuntu | 04:59 |
sdake | and little from binary | 04:59 |
SamYaple | keystone | 04:59 |
coolsvap | sdake, its on the list but not prioritized :-/ | 04:59 |
SamYaple | oh i guess its not | 04:59 |
sdake | what is on the list coolsvap | 05:00 |
coolsvap | add build from source for ubuntu and test | 05:00 |
sdake | is there a blueprint for that? | 05:00 |
coolsvap | no not yet, need to create one | 05:01 |
sdake | i thought there was for some reason | 05:01 |
sdake | ubuntu falls into the same category as rhel | 05:01 |
SamYaple | how so? | 05:01 |
SamYaple | no licensing issues | 05:01 |
coolsvap | there was for ubuntu but its no longer active | 05:01 |
coolsvap | i meant implemented | 05:02 |
sdake | it clearly wasn't implemented ;) | 05:02 |
coolsvap | sdake, yes | 05:02 |
sdake | got a link | 05:02 |
sdake | samyaple meaning we do it after we do centos | 05:02 |
SamYaple | sdake: im about to do ubuntu full time so that wil lchange in the next week or so | 05:02 |
SamYaple | ubuntu from source even | 05:03 |
sdake | wfm | 05:03 |
SamYaple | since ubuntu has no 'delorean' type repo | 05:03 |
sdake | coolsvap plz link the blueprint and i'll pull it into l3 | 05:03 |
coolsvap | https://blueprints.launchpad.net/kolla/+spec/install-from-ubuntu | 05:03 |
sdake | ya policy i set unilaterally :) is if there is no binary link to from source | 05:03 |
SamYaple | what? | 05:03 |
*** tobe_ has joined #kolla | 05:04 | |
sdake | we will discuss at tomorrow meeting | 05:05 |
sdake | well that really bums me out i thought we had ubuntu sorted out | 05:07 |
SamYaple | me too | 05:07 |
SamYaple | i just hadnt looked | 05:07 |
sdake | really all we have sorted out is centos | 05:07 |
sdake | coolsvap do as sam suggested keep structure but run bin false | 05:09 |
* SamYaple crasks the whip | 05:09 | |
coolsvap | yes will do | 05:09 |
coolsvap | sdake, i might take a dig at ubuntu once i am done with ceilometer-ansible and templating | 05:10 |
sdake | ya dont overwork yourself :) | 05:10 |
sdake | so guys lets brainstorm ini format | 05:10 |
sdake | we need to get this fixed asap | 05:10 |
sdake | because i want from source and binary at minimum tested on centos | 05:11 |
sdake | need to keep parity with what we have | 05:11 |
sdake | http://ur1.ca/neza0 -> http://paste.fedoraproject.org/254187/35628914 | 05:11 |
sdake | there is the buildinfo | 05:11 |
*** weiyu has joined #kolla | 05:11 | |
sdake | how is COMPONENT used there? | 05:12 |
sdake | how about | 05:14 |
coolsvap | its mostly used for temp variable | 05:14 |
sdake | [containername] | 05:14 |
sdake | git_master= | 05:14 |
sdake | git_stable= | 05:14 |
sdake | tarball+master= | 05:14 |
SamYaple | what? | 05:14 |
sdake | tarball_stable= | 05:14 |
SamYaple | why all those things? | 05:14 |
SamYaple | we need three things. tarball, git clone, raw copy from filesystem | 05:15 |
SamYaple | where to download/clone/copy should be configurable | 05:15 |
SamYaple | so pull_method and pull_source should be enough | 05:15 |
sdake | been up since 5am | 05:15 |
SamYaple | no problem | 05:15 |
SamYaple | i have a few more templates to hammer out then ill handle that sdake | 05:16 |
*** Haomeng has quit IRC | 05:16 | |
sdake | nice | 05:16 |
sdake | this is a blocker for our work | 05:16 |
SamYaple | well its a blocker to remove the other builds stuff | 05:16 |
sdake | leave some of the easy templates for peopel in the community to learn on sam | 05:16 |
SamYaple | but as long as its there when the template work is done its fine | 05:16 |
SamYaple | yea sdake its just ones i already started on | 05:16 |
SamYaple | glance and mariadb | 05:16 |
SamYaple | before your email | 05:17 |
sdake | oh ya thats fine | 05:17 |
SamYaple | i just did keystone and rabbitmq cause yo uasked | 05:17 |
sdake | ya mariadb would have been good in place of rabbitmq ;) | 05:17 |
sdake | I just randomly pickd that | 05:17 |
SamYaple | eh its done | 05:18 |
SamYaple | lots of RAX work at the moment | 05:18 |
sdake | yar | 05:19 |
coolsvap | sdake, i have another approach to reduce the number of lines in ini and make it hierarchical | 05:20 |
sdake | why do we need hiearcy | 05:20 |
sdake | only the base image or 1 image will ever need a tarball I think | 05:21 |
coolsvap | sdake, yes | 05:21 |
sdake | if there is a use case f or hiearcy lets hear it now because we can't use ini in that case | 05:21 |
coolsvap | i think the git-clone urls are mostly same | 05:22 |
SamYaple | coolsvap: right but mostly the same != the same | 05:22 |
SamYaple | im ok with a bit of duplication there | 05:22 |
sdake | i dont want a complex format for config | 05:22 |
SamYaple | yea | 05:22 |
SamYaple | a static "here is where you get this file" is fine with me | 05:23 |
SamYaple | any changes across the file is a quick sed command away | 05:23 |
coolsvap | SamYaple, yes its not same always, i was just trying to put templatize the ini | 05:23 |
sdake | coolsvap interesting idea but lets not do that to our users :) | 05:24 |
sdake | its hard enough on devs to deal with all that | 05:24 |
sdake | but i expect devs to be able to handle it | 05:24 |
coolsvap | sdake, it would be easy than the sed, just put explicity what you need, rest is done by default | 05:25 |
sdake | the prototype template files for dockerfiles are simpel enough for devs to understand but I dont know if a non dev would ge tit | 05:25 |
sdake | ya overrides inheritence | 05:25 |
sdake | i hear what your saying | 05:25 |
sdake | dont' like :) | 05:25 |
coolsvap | hmm | 05:26 |
coolsvap | np :) | 05:26 |
*** weiyu has quit IRC | 05:27 | |
sdake | samyaple how do we select between git and uri? | 05:28 |
sdake | yet another cli option? | 05:29 |
sdake | i think better might be a default section | 05:29 |
sdake | with a type field or something similar | 05:30 |
coolsvap | sdake, https://github.com/stackforge/kolla/blob/master/tools/gen-source-tar.sh#L5 | 05:30 |
sdake | lets try not to add more stuff to the cli if at all possible | 05:30 |
coolsvap | https://github.com/stackforge/kolla/blob/master/tools/gen-source-tar.sh#L5 | 05:30 |
coolsvap | sorry https://github.com/stackforge/kolla/blob/master/tools/gen-source-tar.sh#L32 | 05:30 |
sdake | so its hard coded | 05:30 |
coolsvap | yes | 05:34 |
sdake | samyaple for git we also need a branch I think | 05:38 |
sdake | or tag | 05:38 |
sdake | tag | 05:38 |
sdake | maybe we can get that out of the uri | 05:38 |
sdake | ok time to pass the foxtrot out | 05:39 |
sdake | night | 05:39 |
sdake | best wishes on that build..py work sam ;) | 05:39 |
sdake | night coolsvap | 05:39 |
*** akwasnie has joined #kolla | 05:47 | |
*** weiyu has joined #kolla | 05:47 | |
*** diga_ has joined #kolla | 05:50 | |
coolsvap | night sdake | 05:50 |
coolsvap | SamYaple, still around? | 06:04 |
coolsvap | have a look at http://paste.openstack.org/show/412651 just trying to sort out the template for ceilometer-base | 06:06 |
SamYaple | coolsvap: hey | 06:13 |
SamYaple | in the source section you should have any if blocks | 06:13 |
SamYaple | the CMD shouldnt even be in a base image like this | 06:14 |
coolsvap | http://paste.openstack.org/show/412652/ | 06:17 |
coolsvap | SamYaple, ^^ | 06:18 |
SamYaple | im not sure what youre asking me here | 06:18 |
coolsvap | just look once if this is correct | 06:19 |
SamYaple | no | 06:19 |
SamYaple | there is no binary stuff | 06:20 |
SamYaple | the source stuff should work for everything | 06:20 |
SamYaple | you dont need to have bit/ture like that | 06:20 |
SamYaple | /bin/true | 06:20 |
SamYaple | we can do this in a review | 06:20 |
coolsvap | in ceilometer base there is nothing | 06:20 |
coolsvap | https://github.com/stackforge/kolla/blob/master/docker/centos/binary/ceilometer/ceilometer-base/Dockerfile | 06:20 |
coolsvap | otherwise i wouldnt have asked | 06:21 |
SamYaple | right but doesnt that mean we just need to have one if source section? | 06:21 |
SamYaple | man our images really need updating | 06:22 |
coolsvap | yes, thats what i wanted to confirm, but then we will not have the ceilometer-base for binary | 06:22 |
coolsvap | isnt it? | 06:23 |
*** echoingumesh has joined #kolla | 06:23 | |
SamYaple | no it needs to be there. but we _should_ be installing something like ceilometer-common in the base but dont worry about it for right now | 06:24 |
SamYaple | http://paste.openstack.org/show/412653/ | 06:25 |
SamYaple | coolsvap: ^ | 06:25 |
*** unicell1 has joined #kolla | 06:28 | |
*** unicell has quit IRC | 06:28 | |
coolsvap | SamYaple, alrighty | 06:29 |
*** echoingumesh has quit IRC | 06:30 | |
sdake | the reaosn we have a ceilometer-base is to install ceilomter-config | 06:33 |
sdake | but that went away | 06:33 |
sdake | coolsvap | 06:33 |
SamYaple | yea sdake but we should be installing the ceilometer-common package | 06:33 |
SamYaple | otherwise we have really large images | 06:33 |
sdake | may possibly make sense | 06:34 |
SamYaple | i really need to do the image tuning i did for yaodu on kolla | 06:34 |
SamYaple | those images were super tiny | 06:34 |
sdake | just explaining why it is seemingy empty :) | 06:34 |
SamYaple | like 1.2gb size across the whoe things | 06:34 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Add Dockerfile template for mariadb https://review.openstack.org/211864 | 06:34 |
*** sdake_ has joined #kolla | 06:35 | |
*** sdake has quit IRC | 06:39 | |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Add Dockerfile template for glance https://review.openstack.org/211871 | 06:51 |
SamYaple | jpeeler: do yo ureally live in SC? | 06:57 |
*** fangfenghua has joined #kolla | 07:00 | |
*** inc0 has joined #kolla | 07:05 | |
inc0 | good morning | 07:06 |
SamYaple | morning inc0 | 07:06 |
akwasnie | hi:) | 07:07 |
SamYaple | hello akwasnie | 07:07 |
akwasnie | SamYaple which service can I take for templates? Are u working on sth more than glance, mariadb, keystone and rabbit? | 07:08 |
SamYaple | akwasnie: no those are all done now | 07:08 |
SamYaple | there is an etherpad wit ha list | 07:09 |
SamYaple | akwasnie: https://etherpad.openstack.org/p/kolla-dockerfile-template | 07:09 |
SamYaple | any one not claimed just throw your name beside it! | 07:09 |
*** diga_ has quit IRC | 07:09 | |
akwasnie | ok | 07:10 |
openstackgerrit | Swapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add Dockerfile template for glance https://review.openstack.org/211878 | 07:29 |
SamYaple | coolsvap: glance? that looks like ceilometer to me! | 07:30 |
* coolsvap bad | 07:31 | |
SamYaple | coolsvap: unfortunately i know there is no way you tested that | 07:31 |
SamYaple | rabbitmq has the code needed to template teh FROM section | 07:31 |
*** achanda has quit IRC | 07:31 | |
coolsvap | SamYaple, yes the build.py update | 07:32 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Add Dockerfile template for glance https://review.openstack.org/211871 | 07:35 |
*** achanda has joined #kolla | 07:36 | |
*** jmccarthy has joined #kolla | 07:37 | |
*** tobe_ has quit IRC | 07:44 | |
*** jmccarthy has quit IRC | 07:46 | |
*** achanda has quit IRC | 07:46 | |
openstackgerrit | Swapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add Dockerfile template for Ceilometer https://review.openstack.org/211878 | 07:47 |
*** tobe_ has joined #kolla | 07:50 | |
*** jmccarthy has joined #kolla | 08:00 | |
*** tobe_ has quit IRC | 08:00 | |
*** sdake_ has quit IRC | 08:04 | |
*** tobe_ has joined #kolla | 08:05 | |
*** diga_ has joined #kolla | 08:14 | |
*** Haomeng has joined #kolla | 08:14 | |
*** gfidente has joined #kolla | 08:33 | |
*** tfukushima has joined #kolla | 08:34 | |
openstackgerrit | Swapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add Dockerfile template for Ceilometer https://review.openstack.org/211878 | 08:40 |
*** jmccarthy has quit IRC | 09:02 | |
*** vbel has quit IRC | 09:08 | |
*** vbel has joined #kolla | 09:09 | |
openstackgerrit | Swapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add Dockerfile template for Cinder https://review.openstack.org/211919 | 09:16 |
*** jmccarthy has joined #kolla | 09:21 | |
openstackgerrit | Merged stackforge/kolla: Glance bootstrap container mounts wrong config folder https://review.openstack.org/211587 | 09:22 |
openstackgerrit | Merged stackforge/kolla: Add Dockerfile template for rabbitmq https://review.openstack.org/211817 | 09:26 |
openstackgerrit | Merged stackforge/kolla: Add Dockerfile template for mariadb https://review.openstack.org/211864 | 09:26 |
SamYaple | pbourke: ping | 09:26 |
*** jmccarthy has quit IRC | 09:26 | |
pbourke | SamYaple: morning | 09:26 |
SamYaple | hows it go | 09:26 |
pbourke | good :) | 09:26 |
SamYaple | got some questions for oy | 09:27 |
pbourke | wasnt me | 09:27 |
SamYaple | im implementing the source config stuff in python build script | 09:27 |
SamYaple | lies! | 09:27 |
SamYaple | so because we are trying to remove all config-in-each-folder type stuff that leads to only one real option, a global config | 09:27 |
pbourke | ok | 09:29 |
SamYaple | this is what i have: http://paste.openstack.org/show/412672/ | 09:29 |
SamYaple | can you forsee any issues here? | 09:29 |
SamYaple | file or local should also be a type | 09:29 |
pbourke | oo | 09:30 |
pbourke | nice | 09:30 |
pbourke | no that looks pretty nifty | 09:30 |
SamYaple | ok. This leaves the build.py to do the implementation | 09:30 |
SamYaple | ive got the tarball part done | 09:30 |
SamYaple | but the git part :/ | 09:30 |
SamYaple | may just end up wrapping `git` | 09:31 |
pbourke | you may not need dest_filename | 09:32 |
pbourke | depends on the implementation | 09:32 |
pbourke | whats the issue with git? | 09:32 |
SamYaple | git support with python | 09:32 |
SamYaple | without external libs | 09:33 |
SamYaple | the dest_filename is needed because i use it to name the file in the final location | 09:33 |
SamYaple | i really want to pull that information dynamically though | 09:34 |
SamYaple | can't quite figure it out yet | 09:34 |
pbourke | can you not use the section name? | 09:35 |
pbourke | also I would just subprocess.call the git command | 09:36 |
SamYaple | ah another caveat | 09:36 |
SamYaple | the section name will need to be the name of the image where i install it | 09:36 |
SamYaple | like cinder-base glance-base | 09:36 |
SamYaple | since i use the image name to fetch the configs | 09:36 |
pbourke | coolsvap: question on https://review.openstack.org/#/c/211919/1/docker_templates/cinder-app/cinder-scheduler/Dockerfile.j2 if you have time | 09:38 |
coolsvap | pbourke, sure | 09:39 |
pbourke | SamYaple: not following that last part | 09:39 |
coolsvap | pbourke, we are skipping building the images which are not available currently | 09:39 |
SamYaple | pbourke: the /bin/false is becase the ubuntu stuff has no install | 09:39 |
*** fangfenghua has quit IRC | 09:41 | |
pbourke | would something like this be better? | 09:43 |
pbourke | RUN echo '{{ base_distro }} not yet supported' && /bin/false | 09:43 |
SamYaple | yea | 09:43 |
SamYaple | as long as it breaks the image i dont care | 09:43 |
SamYaple | or rather, i dont have an opinion | 09:44 |
SamYaple | I was considering just catching /bin/false in the build.py script and having the failure type be 'unsupported' | 09:44 |
SamYaple | since the logs are eh | 09:44 |
openstackgerrit | Alicja Kwasniewska proposed stackforge/kolla: Add Dockerfile template for horizon https://review.openstack.org/211934 | 09:46 |
*** jmccarthy has joined #kolla | 09:46 | |
*** dolpher has quit IRC | 09:50 | |
*** erkules has quit IRC | 09:54 | |
*** erkules has joined #kolla | 09:54 | |
openstackgerrit | Merged stackforge/kolla: Add Dockerfile template for keystone https://review.openstack.org/211838 | 09:59 |
inc0 | SamYaple, neutron binary still not working right? we need to use source with your patch? | 09:59 |
*** zhiwei has quit IRC | 10:04 | |
SamYaple | inc0: it works for me now | 10:07 |
SamYaple | whats wrong with it? | 10:07 |
inc0 | I'll try when I'll build it | 10:07 |
inc0 | guys, do we want to make release a variable for source? | 10:17 |
*** gfidente has quit IRC | 10:18 | |
inc0 | like downloading nova-{{ release }}.tar? | 10:18 |
SamYaple | inc0: not with this new build.py script | 10:18 |
SamYaple | the conf file is simple | 10:18 |
SamYaple | but its all modifiable in one location | 10:19 |
SamYaple | so a single thread | 10:19 |
SamYaple | sing sed* | 10:19 |
SamYaple | whatever | 10:19 |
*** diga_ has quit IRC | 10:19 | |
inc0 | so first sed all .buildconfs and replace master to whatever? | 10:19 |
SamYaple | no | 10:20 |
SamYaple | all thats going away | 10:20 |
inc0 | how do I deploy kilo for example? | 10:20 |
SamYaple | well from master that wont be supported | 10:20 |
inc0 | wait, I thought we agree it will | 10:21 |
inc0 | on midcycle | 10:21 |
SamYaple | no we agreed people will do that | 10:21 |
SamYaple | we wont block it | 10:21 |
SamYaple | but its not an option in the kolla repo | 10:21 |
inc0 | I kinda have issues with that. especially if it's this simple | 10:21 |
SamYaple | its not though | 10:22 |
openstackgerrit | Swapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add Dockerfile template for Ceilometer https://review.openstack.org/211878 | 10:22 |
*** fangfenghua has joined #kolla | 10:22 | |
inc0 | building docker images is | 10:22 |
inc0 | deploying with ansible, not so, but that's just deployment | 10:22 |
inc0 | and not-so only with ansible generating configs | 10:23 |
inc0 | if you have working config, that's doable | 10:23 |
SamYaple | right but nothing stops you from building images for kilo right now by tweaking hte buildinfo things | 10:24 |
SamYaple | on the from source side | 10:24 |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Implement source fetching for build.py https://review.openstack.org/211954 | 10:31 |
SamYaple | pbourke: https://review.openstack.org/211954 | 10:31 |
SamYaple | feedback needed | 10:31 |
openstackgerrit | Swapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add Dockerfile template for Ceilometer https://review.openstack.org/211878 | 10:33 |
openstackgerrit | Alicja Kwasniewska proposed stackforge/kolla: Add Dockerfile template for horizon https://review.openstack.org/211934 | 10:35 |
*** jmccarthy has quit IRC | 10:35 | |
openstackgerrit | Swapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add Dockerfile template for Cinder https://review.openstack.org/211919 | 10:38 |
pbourke | looking | 10:38 |
*** jmccarthy has joined #kolla | 10:59 | |
*** tobe_ has quit IRC | 11:03 | |
*** tobe_ has joined #kolla | 11:04 | |
*** inc0_ has joined #kolla | 11:04 | |
*** inc0 has quit IRC | 11:04 | |
*** tobe_ has quit IRC | 11:09 | |
*** Haomeng|2 has joined #kolla | 11:11 | |
*** juggler has quit IRC | 11:13 | |
pbourke | SamYaple: it works anyway | 11:15 |
*** Haomeng has quit IRC | 11:15 | |
pbourke | and implementation looks clean | 11:15 |
SamYaple | pbourke: yea but suggestions? | 11:17 |
*** inc0_ has quit IRC | 11:17 | |
*** rhallisey has joined #kolla | 11:17 | |
pbourke | i dont think I have anything for you other than the usual like a little more error checking, maybe the ability to specify a custom build.ini, etc | 11:18 |
*** weiyu has quit IRC | 11:20 | |
openstackgerrit | Ryan Hallisey proposed stackforge/kolla: Glance-registry is missing a messaging config https://review.openstack.org/211616 | 11:20 |
*** fangfenghua has quit IRC | 11:20 | |
SamYaple | pbourke: yea i was going to do a cli option for an override config | 11:23 |
*** inc0 has joined #kolla | 11:29 | |
*** tobe_ has joined #kolla | 11:33 | |
*** inc0 has quit IRC | 11:35 | |
*** tobe_ has quit IRC | 11:39 | |
*** inc0 has joined #kolla | 11:43 | |
*** inc0_ has joined #kolla | 11:44 | |
*** gfidente has joined #kolla | 11:46 | |
*** inc0 has quit IRC | 11:47 | |
*** britthouser has joined #kolla | 11:53 | |
*** britthou_ has joined #kolla | 11:54 | |
openstackgerrit | Sam Yaple proposed stackforge/kolla: Add Dockerfile template for glance https://review.openstack.org/211871 | 11:55 |
*** britthouser has quit IRC | 11:57 | |
openstackgerrit | Swapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add Dockerfile template for Ceilometer https://review.openstack.org/211878 | 12:04 |
*** jmccarthy has quit IRC | 12:07 | |
*** jmccarthy has joined #kolla | 12:07 | |
openstackgerrit | Swapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add Dockerfile template for Cinder https://review.openstack.org/211919 | 12:08 |
openstackgerrit | Swapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add Dockerfile template for Ceilometer https://review.openstack.org/211878 | 12:10 |
*** diogogmt has joined #kolla | 12:11 | |
*** diogogmt has quit IRC | 12:24 | |
openstackgerrit | Swapnil Kulkarni (coolsvap) proposed stackforge/kolla: Add Dockerfile template for Cinder https://review.openstack.org/211919 | 12:24 |
*** 7GHAAXY5L has joined #kolla | 12:32 | |
*** dwalsh has joined #kolla | 12:36 | |
*** jruano has joined #kolla | 12:38 | |
*** inc0__ has joined #kolla | 12:46 | |
*** inc0_ has quit IRC | 12:49 | |
*** 7GHAAXY5L has quit IRC | 12:50 | |
*** inc0__ has quit IRC | 12:53 | |
openstackgerrit | Alicja Kwasniewska proposed stackforge/kolla: Add Dockerfile template for horizon https://review.openstack.org/211934 | 13:03 |
*** weiyu_ has joined #kolla | 13:07 | |
*** britthou_ has quit IRC | 13:14 | |
rhallisey | anyone have an idea why cinder can't connect to the db | 13:32 |
rhallisey | even though I can manually connect to it and it gets created.. | 13:33 |
rhallisey | https://bugs.launchpad.net/kolla/+bug/1481928 | 13:34 |
openstack | Launchpad bug 1481928 in kolla "Cinder is unable to connect to the database" [Critical,Triaged] - Assigned to Ryan Hallisey (rthall14) | 13:34 |
*** blahRus has joined #kolla | 13:39 | |
*** jtriley has joined #kolla | 13:43 | |
jpeeler | SamYaple: yeah, I do live in SC. why? | 13:45 |
rhallisey | ok I know.. | 13:46 |
*** rhallisey has quit IRC | 13:47 | |
*** dwalsh has quit IRC | 13:50 | |
openstackgerrit | Vladislav Belogrudov proposed stackforge/kolla: Glance cannot store images due to incomplete configuration https://review.openstack.org/212033 | 13:50 |
vbel | hello, can we check ^^ so that glance finally works out of box? :) | 13:51 |
*** weiyu_ has quit IRC | 13:51 | |
vbel | cinder is my next service to torture :) | 13:52 |
*** diogogmt has joined #kolla | 13:55 | |
*** diogogmt has quit IRC | 13:55 | |
*** rhallisey has joined #kolla | 13:56 | |
vbel | I could finally store and download cirros images from glance without loss :) | 13:57 |
*** sdake_ has joined #kolla | 14:00 | |
sdake_ | morning | 14:01 |
sdake_ | such as it is | 14:01 |
vbel | morning sdake | 14:01 |
openstackgerrit | Jeff Peeler proposed stackforge/kolla: Add new test script for parallel builds https://review.openstack.org/212040 | 14:01 |
*** absubram has joined #kolla | 14:04 | |
*** jmccarthy has quit IRC | 14:07 | |
openstackgerrit | Jeff Peeler proposed stackforge/kolla: Add new test script for parallel builds https://review.openstack.org/212040 | 14:09 |
*** thumpba has joined #kolla | 14:15 | |
*** dwalsh has joined #kolla | 14:16 | |
*** jmccarthy has joined #kolla | 14:17 | |
*** jruano has quit IRC | 14:17 | |
*** banix has joined #kolla | 14:17 | |
*** britthouser has joined #kolla | 14:21 | |
*** rhallisey_ has joined #kolla | 14:23 | |
*** dwalsh has quit IRC | 14:23 | |
*** rhallisey has quit IRC | 14:27 | |
sdake_ | jpeeler can you get a gat job going in openstack-infra to run that build script | 14:31 |
vbel | sdake: https://review.openstack.org/212033 ? To get things going :) It is a basic config to use glance out of box. If users want another store they can re-configure it | 14:34 |
*** rhallisey_ is now known as rhallisey | 14:35 | |
sdake_ | vbel how does oracle buidl ol images | 14:37 |
sdake_ | or only ben focusing on the centos images? | 14:37 |
vbel | we have ol source based images similar to centos-source- | 14:38 |
sdake_ | how doy ou build them? | 14:38 |
sdake_ | what is your process, is what i'm after | 14:38 |
vbel | docker 1.8 with build-env patch | 14:38 |
sdake_ | do you build the ol/base image? | 14:39 |
vbel | yes | 14:39 |
*** banix has quit IRC | 14:39 | |
sdake_ | ok well we are doing this new ninja2 templating thing | 14:39 |
sdake_ | and i'd like the oracle images to work | 14:39 |
sdake_ | one question i had is i notice there is no other package besides base, do you just hack the build script to rename the centos images to oraclelinx o rsomething? | 14:40 |
vbel | we have mirror in house, should be able to push upstream some day | 14:40 |
sdake_ | mean mirror of the git repo? | 14:41 |
vbel | some stuff is very easy - like linking ol-source-heat to centos-source-heat | 14:41 |
vbel | yes | 14:41 |
sdake_ | probably not a mirror :) | 14:41 |
sdake_ | but thanks that answers my q | 14:41 |
vbel | well, it is enhanced mirror :) | 14:41 |
vbel | some day I will start adding stuff to docker/ol/source/ | 14:42 |
sdake_ | what other divergence is there besides linking ol-source-heat and all the other projects | 14:42 |
sdake_ | vbel we are deprecating ht docker dir | 14:42 |
sdake_ | going to docker_templates | 14:42 |
vbel | so we will fetch/merge them | 14:43 |
sdake_ | regarding glance i dont get why that changeis necessary, glance wfm when i tested it, but I don't have objection to it so approving | 14:43 |
vbel | or rather adopt since there is not much to merge | 14:43 |
vbel | it does not work when trying to push image into it because config section is missing | 14:44 |
sdake_ | that did work for me | 14:44 |
sdake_ | at end of l2 | 14:44 |
sdake_ | perhaps new change in glance | 14:44 |
sdake_ | i manually tested the tag before pushing the tag | 14:45 |
vbel | I added simple file store as in openstack manuals. Could be | 14:45 |
vbel | I thought by default it should be local file store but suddenly it wanted the config explicitly | 14:45 |
vbel | is it 2015.1.1 or 2015.1.0 that was tested? | 14:46 |
sdake_ | master | 14:47 |
vbel | ok | 14:47 |
vbel | I tested with 2015.1.1 (stable kilo bugfix) | 14:47 |
sdake_ | thats the difference then | 14:47 |
vbel | kolla uses 2015.1.0 by default | 14:47 |
vbel | wrong... | 14:48 |
sdake_ | rdo uses master only | 14:48 |
vbel | yes, correct | 14:48 |
jpeeler | sdake_: once the review gets approved, i'll add it as an experimental job | 14:48 |
openstackgerrit | Merged stackforge/kolla: Glance cannot store images due to incomplete configuration https://review.openstack.org/212033 | 14:48 |
sdake_ | pbourke can you approve peeler's patch or rhallisey | 14:48 |
pbourke | sdake_: which one | 14:49 |
sdake_ | according to the governance repos official tool for teamstats, our team size is 14 committers and 14 reviewers :) | 14:49 |
jpeeler | pbourke: https://review.openstack.org/#/c/212040/ , thanks in advance! | 14:49 |
sdake_ | jpeeler i see you grew facial hair :) | 14:50 |
jpeeler | Ha, yeah the other picture was approaching 10 years old | 14:50 |
sdake_ | akwasnie my bad for not inviting you to the kolla-drivers team | 14:50 |
sdake_ | its a mostly open team - (kolla devs and whatnot are all welcome) | 14:51 |
sdake_ | I'll add you now | 14:51 |
sdake_ | you actually look like not a boy in that picture ;) | 14:51 |
sdake_ | i hate to use the word man because your still a young buck :) | 14:51 |
jpeeler | like i said, long overdue | 14:52 |
sdake_ | its a pretty solid picture | 14:52 |
sdake_ | i see your hair is doing the same shit mine is doing, thinning :) | 14:52 |
jpeeler | yes :/ but it's who i am! | 14:52 |
sdake_ | i was losing my hair badly at 25 or so, so i took propecia for about 1 year | 14:52 |
sdake_ | and that stopped it | 14:53 |
sdake_ | and then neverhad to do anything else | 14:53 |
jpeeler | really? | 14:53 |
* rhallisey goes to photoshop picture with beard | 14:53 | |
sdake_ | yup propecia seems permanent | 14:53 |
sdake_ | even though people take it forever | 14:53 |
sdake_ | my hair isn't like that dude from ancient alien astronauths but its not terrible ;) | 14:53 |
sdake_ | https://en.wikipedia.org/wiki/Giorgio_A._Tsoukalos | 14:55 |
sdake_ | that cat has the best hair in the business :) | 14:55 |
sdake_ | ya I highy recommend 1 year propecia treatment for folks that want to keep their hair at the 25-28 age rang when it starts to fall out :) | 14:57 |
*** diogogmt has joined #kolla | 14:57 | |
sdake_ | doctors will want to keep you on it forever but once it does its thing it seems permanent - several people have told me this as well after the fact | 14:57 |
jpeeler | how expensive is this drug | 14:57 |
sdake_ | its cheap | 14:58 |
sdake_ | its a pill rather then a topical too which is nice | 14:58 |
jpeeler | interesting | 14:58 |
sdake_ | iirc it does require blood tests | 14:58 |
sdake_ | every 3 months or so | 14:58 |
sdake_ | thats the part that sucke | 14:59 |
sdake_ | d | 14:59 |
sdake_ | i used to pass out on blood draws but now pretty resilient to the blood loss :) | 14:59 |
jpeeler | just need one more review sdake | 15:01 |
*** diogogmt has quit IRC | 15:03 | |
*** banix has joined #kolla | 15:03 | |
*** diogogmt has joined #kolla | 15:04 | |
jpeeler | docker 1.8.0 came out, i can't keep up | 15:04 |
sdake_ | nice | 15:04 |
sdake_ | that is why i wanted to use the curl thing ;) | 15:04 |
sdake_ | jpeeler just fyi docker is on a 2 month release schedule | 15:05 |
jpeeler | since when? | 15:05 |
sdake_ | since forever | 15:05 |
jpeeler | good to know | 15:05 |
sdake_ | since i got involved in docker 1.5 | 15:05 |
sdake_ | probably before then | 15:05 |
sdake_ | they are nt super strict on the schedule but try to keep to it | 15:06 |
* coolsvap remember looking at a review for cinder-app -> cinder or was i dreaming of having that review | 15:17 | |
rhallisey | looks like only cinder-api gets cinder.conf... great | 15:27 |
*** thumpba has quit IRC | 15:31 | |
*** thumpba has joined #kolla | 15:33 | |
*** banix has quit IRC | 15:48 | |
sdake_ | coolsvap I wrote we need to rename it, but never did the rename because people were busy in the space | 15:48 |
sdake_ | feel free to submit a rename - cinder-app is out of place | 15:48 |
sdake_ | bybusy in the space I mean busy in the cinder dir during config-internal deletion | 15:49 |
coolsvap | sdake_, hmm sometime later | 15:49 |
sdake_ | are there any cinder commits out? If not I'll do right now | 15:49 |
sdake_ | both coolsvap and rhallisey would know :) | 15:50 |
coolsvap | sdake_, later | 15:50 |
rhallisey | still trying fix the ansible piece | 15:50 |
sdake_ | this wont affect that unless your changing docker/cinder-app | 15:50 |
sdake_ | docker/*/*/cinder_app | 15:51 |
coolsvap | sdake_, there are multiple changes, i think we can do it at once | 15:51 |
rhallisey | ya should be fine | 15:51 |
coolsvap | either way i am not much worried about it now | 15:51 |
coolsvap | i was just digging sometime earlier i could not find the change set or bu | 15:52 |
coolsvap | so just asked | 15:52 |
sdake_ | i am already in action | 15:52 |
sdake_ | ;) | 15:52 |
*** thumpba has quit IRC | 15:57 | |
*** thumpba has joined #kolla | 15:57 | |
*** thumpba has quit IRC | 15:58 | |
*** thumpba has joined #kolla | 15:58 | |
openstackgerrit | Steven Dake proposed stackforge/kolla: Rename cinder-app to cinder to be more consistent https://review.openstack.org/212088 | 16:05 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Rename cinder-app to cinder to be more consistent https://review.openstack.org/212088 | 16:07 |
sdake_ | i coudln't tst that because i deved it on a mac | 16:08 |
*** pbourke has quit IRC | 16:08 | |
sdake_ | but lets get it merged and if it breaks the build we can get it fixed | 16:09 |
sdake_ | rhallisey can you review | 16:09 |
sdake_ | i'll add a bug id now sec | 16:09 |
*** pbourke has joined #kolla | 16:09 | |
pbourke | just realised I need ansible memcache as a precursor to swift | 16:09 |
pbourke | does memcache need to run in some kind of cluster or can it be one instance per server | 16:09 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Rename cinder-app to cinder to be more consistent https://review.openstack.org/212088 | 16:10 |
sdake_ | pbourke pbourke rhallisey please review that before we need further cinder changes | 16:10 |
sdake_ | I dont want to have to rebase it :) | 16:10 |
pbourke | sdake_: been wanting that change for ages | 16:10 |
sdake_ | pbourke I have no idea how memcache works | 16:11 |
pbourke | why are there so many deletes? | 16:11 |
pbourke | did you use git mv? | 16:11 |
sdake_ | rhallisey actually review this first: https://review.openstack.org/#/c/211221/ | 16:12 |
sdake_ | yes I used git mv | 16:12 |
sdake_ | but I had to relink a whole bunch of shit | 16:12 |
pbourke | ah | 16:12 |
sdake_ | because the links were cinder-app not cinder | 16:12 |
pbourke | probably just gerrit not being pretty | 16:12 |
sdake_ | please lets get that other cinder review merged first | 16:12 |
* sdake_ doens't want to erbase ;-) | 16:13 | |
*** sdake_ is now known as sdake | 16:13 | |
sdake | rhallisey or jpeeler this could use a rubber stamp https://review.openstack.org/#/c/210823/ | 16:13 |
rhallisey | my stamp is made of gold | 16:14 |
rhallisey | lol jks | 16:14 |
sdake | so besides diversity the other thing that is used to measure how stable a project is is the team size | 16:15 |
sdake | a larger team naturally is more resilient | 16:15 |
sdake | we have 14 reviewers and 14 commiters according to the teamstats.py tool in the governance repo (renamed from diversity.py) | 16:16 |
sdake | we have about 30 poeple that contributed code in the last 3 months | 16:16 |
sdake | MIN_REVIEWS = 30 | 16:16 |
sdake | MIN_COMMITS = 6 | 16:16 |
sdake | these are the counts to hit an increment in either reviewers or committers | 16:17 |
sdake | jpeeler https://review.openstack.org/#/c/212040/ needs some minor edits - can you crank it out so we canget er merged | 16:22 |
sdake | the faster we have the gate running the better | 16:22 |
*** unicell1 has quit IRC | 16:25 | |
*** jmccarthy has quit IRC | 16:28 | |
openstackgerrit | Merged stackforge/kolla: Remove tox -egenconfig since its not needed for cinder https://review.openstack.org/211221 | 16:29 |
openstackgerrit | Merged stackforge/kolla: Remove mariadb container https://review.openstack.org/210823 | 16:31 |
*** akwasnie has quit IRC | 16:33 | |
sdake | pbourke or rhallisey can one of you review https://review.openstack.org/#/c/211934/ | 16:33 |
sdake | need m0ar reviewers! | 16:34 |
sdake | can't wait until mandre is back | 16:34 |
sdake | rhalllisey rubber stamping on https://review.openstack.org/#/c/212088/ appreciated ;) | 16:35 |
sdake | then if you can build it and see if it still works once merged that would be extra+ credit | 16:36 |
*** diga_ has joined #kolla | 16:40 | |
openstackgerrit | Merged stackforge/kolla: Add Dockerfile template for horizon https://review.openstack.org/211934 | 16:43 |
*** diga_ has quit IRC | 16:46 | |
*** gfidente has quit IRC | 16:48 | |
*** unicell has joined #kolla | 16:48 | |
*** mfalatic has quit IRC | 16:58 | |
*** mfalatic has joined #kolla | 16:59 | |
harmw | sdake: ill make sure I can do some reviews this friday evening | 17:00 |
harmw | that worked out in the past quite well | 17:00 |
harmw | for me atleast :p | 17:00 |
*** mfalatic has quit IRC | 17:02 | |
*** mfalatic has joined #kolla | 17:03 | |
*** achanda has joined #kolla | 17:13 | |
*** britthou_ has joined #kolla | 17:27 | |
*** britthouser has quit IRC | 17:30 | |
*** banix has joined #kolla | 17:34 | |
*** Haomeng has joined #kolla | 17:36 | |
*** Haomeng|2 has quit IRC | 17:38 | |
coolsvap | sdake, SamYaple can please review https://review.openstack.org/211442 https://review.openstack.org/211445 https://review.openstack.org/211878 https://review.openstack.org/211919 | 17:42 |
rhallisey | cinder ansible -> https://review.openstack.org/#/c/208868/ | 17:43 |
rhallisey | sdake, just need another +2 | 17:43 |
*** bmace has quit IRC | 17:43 | |
*** absubram has quit IRC | 17:55 | |
*** absubram has joined #kolla | 17:57 | |
*** banix has quit IRC | 18:01 | |
openstackgerrit | Merged stackforge/kolla: Add ansible support for Cinder https://review.openstack.org/208868 | 18:06 |
openstackgerrit | Merged stackforge/kolla: Glance-registry is missing a messaging config https://review.openstack.org/211616 | 18:07 |
openstackgerrit | Merged stackforge/kolla: Update neutron confs and ports https://review.openstack.org/210962 | 18:07 |
openstackgerrit | Ryan Hallisey proposed stackforge/kolla: Ansible Cinder needs to create the v2 endpoint https://review.openstack.org/212140 | 18:21 |
rhallisey | that's a little closer... | 18:21 |
*** banix has joined #kolla | 18:22 | |
*** tfukushima has quit IRC | 18:27 | |
*** tfukushima has joined #kolla | 18:28 | |
*** tfukushima has quit IRC | 18:28 | |
openstackgerrit | Jeff Peeler proposed stackforge/kolla: Add new test script for parallel builds https://review.openstack.org/212040 | 18:30 |
*** absubram has quit IRC | 18:35 | |
*** absubram has joined #kolla | 18:38 | |
*** absubram has quit IRC | 18:48 | |
*** absubram has joined #kolla | 18:54 | |
sdake | back | 18:56 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Rename cinder-app to cinder to be more consistent https://review.openstack.org/212088 | 19:05 |
sdake | pbourke you around? | 19:05 |
sdake | rhallisey can you re-rubber stamp that - it had a link error | 19:05 |
sdake | harmw if you could rubber stamp the above - it was alredy reviewed twice ^^ | 19:07 |
sdake | just had an error in one of the links | 19:08 |
sdake | or jpeeler ^^ | 19:08 |
* jpeeler got it | 19:09 | |
sdake | jpeeler thanks | 19:09 |
sdake | jpeeler suggestion for new gate naming | 19:09 |
sdake | as you make new gates for each of the distro types | 19:09 |
sdake | or atleast the ones you are interested in | 19:09 |
sdake | actually I was sort of doing that work | 19:09 |
sdake | anyway | 19:09 |
jpeeler | i actually already have it ready for review | 19:09 |
jpeeler | but haven't posted it | 19:09 |
sdake | i think what makes sense is gate-build-images-centos-src | 19:09 |
sdake | gate-build-images-centos-bin | 19:10 |
sdake | gate-build-images-oraclelinux-src | 19:10 |
sdake | etc | 19:10 |
jpeeler | i currently have kolla-parallel-build-f21... | 19:10 |
sdake | ya lets rename it | 19:11 |
sdake | we are doing gates for every distro + src/bin combo | 19:11 |
sdake | make sense? | 19:12 |
jpeeler | it does | 19:12 |
jpeeler | once i get this in the gate, i probably won't work on getting all the other distro combinations in. but they should be super easy to add | 19:13 |
jpeeler | sdake: i guess i'll use "rdo" since that was used for the tox target | 19:17 |
sdake | wfm | 19:17 |
sdake | please link so i can add centos from src | 19:18 |
sdake | or alternatively follow up the first patch with a ssecond | 19:18 |
sdake | i'd like to get those two gates working since we are doing this tempalting work now | 19:18 |
jpeeler | the project-config review? i haven't submitted it yet since the tox review hasn't merged | 19:18 |
sdake | can you link and ask rhallisey to review it plz | 19:19 |
rhallisey | wow.. just figured out the bug.. | 19:19 |
sdake | needs to happen soon its already the 12th :) | 19:19 |
jpeeler | https://review.openstack.org/#/c/212040/ | 19:19 |
rhallisey | that sucked | 19:19 |
jpeeler | sdake: i had all this staged before i left last month, but reviews didn't get done :/ | 19:19 |
sdake | rhallisey can you look that over | 19:19 |
sdake | its a little complicated | 19:20 |
sdake | its all good jpeeler | 19:20 |
rhallisey | nobody every spell something wrong in config-external again please | 19:20 |
sdake | we need more core revierers - we produce more code then reviews typically :) | 19:20 |
rhallisey | ya looking | 19:20 |
jpeeler | i'm bad about reviewing if i'm not actively submitting code | 19:20 |
rhallisey | getting over bug hangover here | 19:21 |
jpeeler | i guess i need to learn ansible a lot more too | 19:24 |
sdake | ya jpeeler you ought to take on a service - its esay to leanr took me about 4 hours to do heat | 19:24 |
sdake | although I ran into trobule and the work is stalled atm | 19:25 |
sdake | i wouldn't necessarily start with ironic either, that may b edifficult to do ansible with or may not not sure | 19:25 |
jpeeler | yeah i will once my back log gets cleared up | 19:25 |
sdake | if you just do api and conductor it hsould be easy enough | 19:25 |
jpeeler | (this gate stuff part of that) | 19:25 |
openstackgerrit | Ryan Hallisey proposed stackforge/kolla: All the Cinder services need to reference the correct config https://review.openstack.org/212157 | 19:28 |
sdake | rhallisey can you hold off on that one | 19:29 |
sdake | and wait until cinder-app finishes | 19:30 |
sdake | it will be easy for you to rebase | 19:30 |
rhallisey | ya | 19:30 |
sdake | it will be a pita for me to rebase | 19:30 |
rhallisey | I have one more patch too but I will hold the merge | 19:30 |
sdake | for you to rebase git pull --rebase should figure out the renames | 19:30 |
rhallisey | there was a spelling error in config-external | 19:30 |
sdake | but git doens't work in revrse, if I have a rename in my repo it doesn't rebase your work into it | 19:31 |
rhallisey | so cinder-backups was never configured O.o | 19:31 |
rhallisey | that drove me crazy | 19:31 |
rhallisey | it was just one letter :( | 19:31 |
rhallisey | sdake, no worries i'll -1 until yours merges | 19:31 |
*** jasonsb has quit IRC | 19:33 | |
*** jasonsb has joined #kolla | 19:34 | |
openstackgerrit | Ryan Hallisey proposed stackforge/kolla: Cinder-backup is not being configured because config-external spelling error https://review.openstack.org/212160 | 19:35 |
rhallisey | will -1 that also | 19:35 |
sdake | not sure how we miss these things :( | 19:36 |
sdake | we have a plan to remove the config-external.sh script entirely | 19:37 |
sdake | and specify the config files to load up via ansible and a common json file | 19:37 |
rhallisey | that bug sucked | 19:37 |
rhallisey | because it was undetectable | 19:37 |
rhallisey | oh well | 19:37 |
*** jasonsb has quit IRC | 19:38 | |
*** echoingumesh has joined #kolla | 19:39 | |
*** echoingumesh has quit IRC | 19:41 | |
*** echoingumesh has joined #kolla | 19:42 | |
*** echoingumesh has quit IRC | 19:46 | |
*** bmace has joined #kolla | 19:52 | |
openstackgerrit | Merged stackforge/kolla: Rename cinder-app to cinder to be more consistent https://review.openstack.org/212088 | 19:56 |
sdake | ok rhallisey a quick rebae and i'll look at your patches | 19:59 |
openstackgerrit | Ryan Hallisey proposed stackforge/kolla: Cinder-backup is not being configured because config-external spelling error https://review.openstack.org/212160 | 19:59 |
rhallisey | that's the only one that needed a rebase | 20:00 |
rhallisey | now I think cinder bootstrap needs to be fixed.. | 20:01 |
rhallisey | gettin there | 20:01 |
openstackgerrit | Merged stackforge/kolla: Removed config-internal check.sh for glance https://review.openstack.org/210872 | 20:07 |
openstackgerrit | Merged stackforge/kolla: Glance source based containers cannot write to /var/log/glance https://review.openstack.org/211580 | 20:09 |
*** achanda has quit IRC | 20:15 | |
openstackgerrit | Merged stackforge/kolla: Update glance confs and ports https://review.openstack.org/210969 | 20:22 |
openstackgerrit | Merged stackforge/kolla: Update rabbitmq confs and ports https://review.openstack.org/210971 | 20:22 |
openstackgerrit | Merged stackforge/kolla: Update nova.conf and ports https://review.openstack.org/210960 | 20:22 |
*** echoingumesh has joined #kolla | 20:23 | |
*** jruano has joined #kolla | 20:28 | |
*** jasonsb has joined #kolla | 20:36 | |
openstackgerrit | Merged stackforge/kolla: Add new test script for parallel builds https://review.openstack.org/212040 | 20:36 |
*** thumpba has quit IRC | 20:40 | |
openstackgerrit | Ryan Hallisey proposed stackforge/kolla: Ansible Cinder needs to create the v2 endpoint https://review.openstack.org/212140 | 20:42 |
*** rhallisey has quit IRC | 20:53 | |
jpeeler | infra changed check/gate config naming/convenctions | 20:57 |
sdake | rhallisey i hae an emergency | 20:57 |
sdake | if i am not back by meeting start time please get it started | 20:57 |
sdake | ok? | 20:57 |
sdake | i expect to be back | 20:58 |
sdake | i didn't update the agenda | 20:58 |
sdake | so ymmv there :) | 20:58 |
sdake | bmace ^^ | 20:58 |
*** jruano has quit IRC | 21:00 | |
*** bmace has quit IRC | 21:01 | |
*** achanda has joined #kolla | 21:01 | |
*** echoingumesh has quit IRC | 21:02 | |
*** echoingumesh has joined #kolla | 21:03 | |
*** echoingumesh has quit IRC | 21:04 | |
*** echoingumesh has joined #kolla | 21:04 | |
*** bmace has joined #kolla | 21:19 | |
*** jtriley has quit IRC | 21:34 | |
jpeeler | sdake: hope everything's okay | 21:38 |
jpeeler | https://review.openstack.org/212194 - build.py in gate review | 21:39 |
sdake | backin time | 21:53 |
sdake | ya just car problem | 21:53 |
sdake | nothing major | 21:53 |
*** rhallisey has joined #kolla | 21:56 | |
sdake | meeting now in openstack-meeting-4 | 22:01 |
jpeeler | sdake: do whatever you have to do for that gate review above when you get a chance | 22:06 |
sdake | cool i am going to append to it a new review | 22:07 |
sdake | opefully it doens't damage yours | 22:07 |
sdake | i think gerrit will leave yours intact | 22:07 |
jpeeler | as long as you make it dependent on it, it won't damage it | 22:07 |
sdake | i make it dependent by pulling it and then applying mine as a new patch | 22:07 |
jpeeler | right | 22:07 |
sdake | rhallisey asked for the teamstats output, so here it is for everyone to look at: | 22:19 |
sdake | http://paste.openstack.org/raw/412795/ | 22:19 |
sdake | note kolla is one of 15 projects with the team:diversity flag | 22:19 |
sdake | ;) | 22:19 |
sdake | the cutoffs are 30 reviews or 6 commits | 22:20 |
sdake | in the script | 22:20 |
*** diogogmt has quit IRC | 22:22 | |
sdake | already 3pm and I dont think I've done a single thing today | 22:32 |
* sdake wtfawks | 22:32 | |
sdake | jpeeler is the build with tox -e images-centos-rdo parallel or serial? | 22:40 |
jpeeler | parallel | 22:40 |
sdake | nice | 22:40 |
sdake | why did you decide to add -T=1 (which is a fantastic improvement btw) in that change then? | 22:40 |
jpeeler | really curious to see how fast it goes | 22:40 |
*** achanda has quit IRC | 22:40 | |
jpeeler | sdake: i had a stalled build when testing and was like, this sucks waiting to see what's happening | 22:41 |
sdake | roger | 22:42 |
*** vinkman has joined #kolla | 22:50 | |
sdake | what -T level did you use | 22:52 |
sdake | once the gate commit goes live we ought to play with that paramemter to see what produces best results | 22:53 |
sdake | i think the machines are 4 cpus avaialble but it may jsut be single cpu | 22:53 |
jpeeler | the default is 8 i believe | 22:54 |
sdake | 8gb ram | 22:54 |
jpeeler | it's easy to play with once the gate config merges | 22:54 |
openstackgerrit | Steven Dake proposed stackforge/kolla: Test commit - do not merge https://review.openstack.org/212214 | 22:56 |
*** weiyu has joined #kolla | 22:56 | |
jpeeler | ha | 22:57 |
*** bmace has quit IRC | 22:58 | |
sdake | jpeeler whre is that gate job running for what I just submitted | 22:59 |
sdake | it prints out /proc/cpuinfo | 22:59 |
jpeeler | sdake: if i recall correctly it doesn't enter the pipeline until a vote is submitted | 22:59 |
jpeeler | i was typing that before you pinged me heh | 22:59 |
jpeeler | just add +1 to it | 22:59 |
sdake | it gets gated automaticaly without votes | 22:59 |
sdake | rather checked | 23:00 |
sdake | the check jobs run on commit not vote | 23:00 |
jpeeler | well i bet it speeds it up then | 23:00 |
jpeeler | i don't know where the job is then... | 23:00 |
jpeeler | there it is | 23:01 |
jpeeler | 8 cores | 23:02 |
jpeeler | actually i'm not sure how to read it | 23:03 |
jpeeler | does 8 processors mean it necessarily has 8 cores? | 23:03 |
*** weiyu has quit IRC | 23:04 | |
jpeeler | the reason i'm confused is because i wasn't sure how hyperthreading affects things (if at all) | 23:04 |
sdake | its actually virtualized | 23:09 |
sdake | so its not like you get a 1:1 mapping | 23:09 |
sdake | openstack allows overcommit | 23:09 |
sdake | so the raw cpus get shared | 23:09 |
sdake | its a nova config option | 23:09 |
sdake | but it looks like 8 cores to the os | 23:10 |
sdake | we could probably run with -T 1000 then | 23:10 |
sdake | I have 16 cores and rn with that | 23:10 |
*** weiyu has joined #kolla | 23:11 | |
*** vinkman has left #kolla | 23:13 | |
sdake | jepeeler can yu paste the link | 23:15 |
*** blahRus has quit IRC | 23:16 | |
*** achanda has joined #kolla | 23:16 | |
sdake | jpeeler xeon 8 core cpu without hyperthreading | 23:19 |
sdake | surprised ht isn't turned on | 23:19 |
sdake | jpeeler trying to figure out how to template-ize your review | 23:20 |
sdake | for my change | 23:20 |
sdake | jpeeler do you know how tox is templatized? We have an example of using parameters with tox in our repo | 23:20 |
jpeeler | tox? i don't off the top of my head. headed to dinner now, but if you just want to test the different thread numbers, you can change the args in test_build.py | 23:26 |
*** banix has quit IRC | 23:36 | |
*** devlaps has joined #kolla | 23:57 | |
*** achanda has quit IRC | 23:59 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!