Tuesday, 2015-07-07

*** sdake_ has joined #kolla00:00
*** dims has joined #kolla00:01
*** sdake has quit IRC00:02
*** Slower has joined #kolla00:07
*** sdake has joined #kolla00:09
*** sdake_ has quit IRC00:13
*** bmace is now known as bmace_away00:15
*** thetrav has joined #kolla00:24
*** dolpher has joined #kolla00:35
*** dims has quit IRC00:50
*** dims has joined #kolla00:58
*** thetrav has quit IRC01:02
*** jasonsb has quit IRC01:06
*** daneyon has joined #kolla01:06
*** jasonsb has joined #kolla01:06
*** jasonsb has quit IRC01:11
*** erkules has joined #kolla01:25
*** erkules_ has quit IRC01:28
openstackgerritMerged stackforge/kolla: Fixing tox tests - split out validation tests  https://review.openstack.org/19892201:46
openstackgerritMerged stackforge/kolla: Add --no-install-recommends option to Ubuntu image  https://review.openstack.org/19748101:50
*** jasonsb has joined #kolla01:56
*** daneyon has quit IRC02:08
*** daneyon has joined #kolla02:08
*** vinkman has quit IRC02:27
*** jtriley has joined #kolla02:41
*** vinkman has joined #kolla02:42
*** daneyon has quit IRC02:45
*** jtriley has quit IRC02:52
*** vinkman1 has joined #kolla03:02
*** vinkman has quit IRC03:03
openstackgerritMerged stackforge/kolla: Fix incorrect database_address variable in globals.yml  https://review.openstack.org/19875803:21
SamYaplemorning03:23
sdakemorning sam03:23
sdakesay can you respond on that mailing list thread with the name of the modules you need and the location they need to be installed to03:23
SamYaplesdake: ill read it03:24
sdakeyou already read it03:36
sdakeits the one with [tc] in the title03:36
sdakethe one where greg said its a-ok to fork03:36
sdakeone silly q, will thee modules work with ansible 1.9 do you think if we just copy them in?03:37
SamYaplesdake: i use them with 1.9.203:39
SamYaplethey are very loosely coupled03:39
*** dims has quit IRC03:39
sdakecool I'll do that project tomorrow if you get me the info today/before morning tomorrow03:57
sdakeor start it atleast03:57
SamYaplesdake: can you ack https://review.openstack.org/#/c/198534/04:04
sdakei can look at it04:04
sdakein future can you use "Closes-Bug: #XYZ"04:05
sdakevs Closes-Bug: XYZ"04:05
sdakeit doesn't stop me from approving this bug04:05
sdakejust more consistent04:05
sdakenot a big deal if you can't remember04:05
SamYapleno04:05
SamYapleyou change04:05
SamYaplehttp://docs.openstack.org/infra/manual/developers.html04:05
sdakeso data containers are going to sleep rather then exit?04:05
SamYaplethis doesnt change the existing data containers04:06
SamYapleon when running with ansible04:06
sdakeImplements: blueprint authentication04:06
sdakeCloses-Bug: #12345604:06
openstackbug 123456 in xine-lib (Ubuntu) "podcast crashes amarok" [Undecided,Fix released] https://launchpad.net/bugs/12345604:06
sdakeChange-Id: I4946a16d27f712ae2adf8441ce78e6c0bb0bb65704:06
SamYapleso it doesnt break idempotency04:06
sdakewhat is restart policy?04:07
SamYaplesdake: i can do it however you want :) i was just being difficult. i dont actually care04:07
SamYapleyou know, restart always04:07
sdakei was just following the guidelines :)04:07
sdakeso we aren't restaritng with this change?04:07
SamYaplewe arent restarting the bootstrap containers04:08
sdakegot it04:08
sdakenot all container just bootstrap04:08
SamYaplethe global restart policy is controlled in globals.yml04:08
SamYaplewhich is default to always04:08
sdakethanks just trying to learn04:08
openstackgerritMerged stackforge/kolla: Change restart policy for bootstrap container  https://review.openstack.org/19853404:09
SamYapleim always here to answer questions04:09
openstackgerritSam Yaple proposed stackforge/kolla: Add new build script  https://review.openstack.org/19661704:26
openstackgerritHui Kang proposed stackforge/kolla: Fix the ansible tags for individual services  https://review.openstack.org/19895504:33
sdakehui kang hi!04:33
sdakenew developer yay :)04:33
*** thetrav has joined #kolla04:39
*** dims has joined #kolla04:40
*** dims_ has joined #kolla04:41
*** dims has quit IRC04:45
*** dims_ has quit IRC04:46
*** bmace_away is now known as bmace04:55
*** nihilifer has joined #kolla04:59
*** bmace has quit IRC05:02
nihilifero/05:05
SamYaple\o05:05
*** bmace has joined #kolla05:13
*** juggler_ has joined #kolla05:46
openstackgerritDigambar proposed stackforge/kolla: Add config-internal/config-external to Magnum containers  https://review.openstack.org/19821305:48
*** juggler has quit IRC05:49
*** dasm has quit IRC05:53
sdakehey nih :)05:54
*** dasm has joined #kolla05:55
*** vinkman has joined #kolla06:08
*** vinkman1 has quit IRC06:10
*** thetrav has quit IRC06:12
*** bmace is now known as bmace_away06:23
*** vinkman has quit IRC07:11
sdakelet mmandre around07:15
sdakemandre around07:15
openstackgerritSam Yaple proposed stackforge/kolla: Add new build script  https://review.openstack.org/19661707:20
sdakehttps://github.com/openstack/magnum/blob/master/requirements.txt#L1507:21
sdakecool looks good07:21
sdakethanks sam07:21
sdakemandre if youc ould ack that would super appreciat it ;)07:21
*** juggler_ is now known as juggler07:23
SamYaplebuilding and push can also happen in parallel with this script07:26
sdakesam can you get on that list of modules today so I can have some real work to do tomorrow ;)07:26
SamYaplei have it already07:27
SamYaplei should send that to you07:27
sdakesned to ml thread plz07:27
sdakeso its permanent for others to find07:27
sdakebd time for me07:28
sdaketuesday meeting funday07:28
* sdake groans07:28
sdakenight folks07:28
SamYaplehehe he made a "merge-it-all" branch07:28
sdakeya I saw that07:28
sdakeI don't even know what it does07:28
SamYapleall the openstack module bits07:29
sdaketoo tired to think atm :)07:29
sdakejust send me list plz07:29
SamYapleyup07:29
sdakethanks fine sir07:29
sdakei'll start working on that tomorrow07:29
sdakeI think the rest of our work is fairly well distributed at this point07:30
sdakeconvince a core reviewer to approve that patch07:30
sdakeits had enough commentary afaik07:30
sdakepossibly rhallllisey will be the first person you will see :)07:31
sdakemandre is usually aroudn this hour as well07:31
SamYaplemandre literally just +2 it before you -1 again07:31
SamYaple>:|07:31
sdakeanyway if I don't get some r&r my virus isn't going to get any better07:32
SamYaplego07:32
sdakelater07:32
sdake:)07:32
*** dims has joined #kolla07:43
*** dims has quit IRC07:48
*** erkules has quit IRC08:06
*** erkules has joined #kolla08:06
*** kevsi has joined #kolla08:08
*** shardy has joined #kolla08:17
*** zhiwei has joined #kolla08:20
openstackgerritMerged stackforge/kolla: Add new build script  https://review.openstack.org/19661708:22
mandreSamYaple, I acked your python build script08:22
mandreand now i'm off :)08:22
SamYaplew00t thanks mandre08:22
SamYapletry it out08:23
*** zhiwei has left #kolla08:36
*** jmccarthy has joined #kolla08:41
*** inc0 has joined #kolla08:50
inc0hello08:51
inc0missed me?:)08:51
SamYaplehey welcome new contributor!08:51
*** gfidente has joined #kolla08:52
*** gfidente is now known as gfidente|afk08:52
inc0anything happened in last week I should know about?08:52
inc0btw, transilvania is cool place!08:53
SamYapleinc0: https://github.com/stackforge/kolla/blob/master/tools/build.py08:53
SamYapledo it08:53
inc0o, new builder?08:54
inc0and its in python! sounds like fun08:54
SamYapleyea i wrote it after you left08:54
SamYaplei literally just merged minutes ago08:55
inc0cool stuff, I'll give it a try08:55
*** shardy_ has joined #kolla09:00
*** athomas has joined #kolla09:01
*** shardy has quit IRC09:02
*** shardy_ has quit IRC09:05
*** shardy has joined #kolla09:06
SamYapleinc0: my build times are 10 minutes09:18
inc0I'm not sure I'll achieve results like that due to internal network configuration (corpo \o/)09:19
inc0but I'll give it a try as soon as I dig myself out of openstack-dev09:19
*** mstachow has joined #kolla09:23
SamYapleinc0: i have an internal cache for packages09:26
*** pbourke has quit IRC09:27
*** pbourke has joined #kolla09:27
*** dolpher has quit IRC09:32
mstachowo/09:35
*** dims has joined #kolla09:44
inc0damn it seems I won't attend to midcycle09:45
*** dims_ has joined #kolla09:45
inc0which infuriates me tbh09:45
inc0I'll try to fight a bit more09:45
*** dims__ has joined #kolla09:46
pbourkeinc0: san jose is a long way for 2 days :)09:46
inc0it is09:47
inc0but I don't think we'll get much time in Tokyo09:47
pbourkeI wonder what the plan is wrt big tent09:47
inc0yeah, me too09:48
pbourkeIm assuming if kolla gets in then we get proper dev session time09:48
*** dims has quit IRC09:48
*** dims_ has quit IRC09:50
*** dims__ has quit IRC09:50
SamYaplethe plan is to submit to the TC at the end of L2, assuming ansible is done09:52
inc0anything good happend with gates?09:53
inc0I thought that was one of problems09:54
SamYapleinc0: functional tests require ansible09:54
*** shadower has joined #kolla09:56
*** rbergeron has quit IRC10:02
inc0SamYaple, camel case? really?:(10:06
inc0:P10:06
inc0method names in build.py10:06
SamYapleCamelCase for Class names are prefered10:06
inc0but not in methods10:06
*** shardy_ has joined #kolla10:06
SamYaplelowerCamelCase is though10:07
SamYaplethere is a pep thing for this10:07
inc0its acceptable, but _ is still prefered10:07
nihiliferin Python standard library lowerCamelCase is used mostly10:07
nihiliferlook at unittest methods ;)10:07
*** shardy has quit IRC10:08
SamYaplei do CamcelCase and lowerCamelCase for all my functions/classes10:08
SamYapleunderscores for variables10:08
SamYapleCAPS for globals (which i almost never use)10:08
nihiliferbut I'm not sure what hacking is saying about method naming. but we'll se when implementing some linting in future10:09
inc0https://www.python.org/dev/peps/pep-0008/#method-names-and-instance-variables10:09
nihiliferfor now, I don't care ;)10:09
SamYapledoes this work? is it consistent throughout? good10:11
*** shardy_ has quit IRC10:12
SamYaple_I_ use CamcelCase for class names, lowerCamelCase for function, underscroe for variables10:12
*** shardy has joined #kolla10:13
inc0and lowerCamelCase should be replaced by underscore according to pep810:13
SamYapledoes this work? is it consistent throughout? good10:13
SamYapleoh. that would explain why i do it that way10:17
SamYaplehttps://www.python.org/dev/peps/pep-0008/#function-names10:17
SamYaple"mixedCase is allowed only in contexts where that's already the prevailing style (e.g. threading.py), to retain backwards compatibility. "10:18
SamYaplei do it that way because ive always done it that way10:18
SamYapleand im going to continue to do it that way10:18
*** diga has quit IRC10:20
*** athomas has quit IRC10:26
*** gfidente|afk has quit IRC10:28
*** athomas has joined #kolla10:34
*** rbergeron has joined #kolla10:43
*** dims has joined #kolla10:46
*** dims has quit IRC10:51
*** jmccarthy has quit IRC11:10
*** jmccarthy has joined #kolla11:10
*** rhallisey has joined #kolla11:21
rhalliseymorning11:22
jmccarthyo/11:23
SamYaplehello rhallisey11:23
SamYaplemy new build script merged11:23
SamYaplefinally11:23
rhalliseyawesome!11:24
SamYaplegive it a try. i want to get it to default by L211:24
SamYapleim looking into a non-voting gate atm11:24
*** prad has joined #kolla11:45
openstackgerritSam Yaple proposed stackforge/kolla: Ansible Glance support  https://review.openstack.org/19849411:48
SamYapleany cores online that can review? https://review.openstack.org/#/c/198533/11:50
inc0hello rhallisey, did you submit your talk yet?11:50
rhalliseyinc0, not yet11:53
*** dims has joined #kolla12:02
*** diga has joined #kolla12:03
*** dims has quit IRC12:07
*** jmccarthy has quit IRC12:08
*** jmccarthy1 has joined #kolla12:08
*** diga_ has joined #kolla12:16
harmwSamYaple: you there for some more general ansible help?12:18
*** diga has quit IRC12:20
harmwoh lol, nvm, just found the filters page on docs.ansible12:21
openstackgerritPaul Bourke proposed stackforge/kolla: Document python mysqldb dependency for ansible deploys  https://review.openstack.org/19907712:32
openstackgerritMerged stackforge/kolla: Have bootstrap script create the initial endpoint  https://review.openstack.org/19853312:34
sdakepbourke hae you tried ou the ansible code yet ?12:36
SamYapleharmw: i am here!12:36
SamYaplesdake: yes he has12:36
SamYaplehe picked up on lots of docs issues12:36
pbourkesdake: to an extent. I nearly have the aio working, then will try multinode12:36
sdakeand the result was?12:36
sdakenice !12:36
sdakeso someone could duplicate the instal ;)12:36
pbourkesdake: I was having keystone issues, sam suggested it may be because I was using keystoneclient rather than openstackclient12:36
pbourkeso trying that now12:37
sdakepbourke I am making a package of depependneices to install (the keystone moduesl)12:37
harmwSamYaple: I'm having a little fight with regex_replace(), it for some reasons prints a weird char where I'd expect \112:37
harmwholygrail: "{{ ansible_hostname | regex_replace('(.+)', 'XX\\1') }}"12:38
SamYapleharmw: whatcha doing regex on the hostname for?12:38
harmwit's not kolla-related :)12:38
SamYapleah cool12:38
harmwbut given your the ansible-wizard (and I'm not in #ansible)12:39
SamYaplethe funny thing is im really not12:40
harmwthat hardly is funny12:40
harmw:p12:41
harmwSamYaple: https://groups.google.com/forum/#!topic/ansible-project/nQSRBAzPJks12:49
harmwthat was my problem, and fix12:49
SamYapleoh yea. complex args are where its at though12:50
SamYaplemuch better way of doing it12:50
SamYapleah thanks for the merge rhallisey12:55
rhalliseyno problem12:56
SamYaplewith that merge, database rabbitmq keystone should all come up and be functional12:56
SamYaplethen we get the keystone modules and we can drop in glance and the rest quickly12:57
openstackgerritSam Yaple proposed stackforge/kolla: Ansible Glance support  https://review.openstack.org/19849413:00
*** jruano has joined #kolla13:03
*** gfidente has joined #kolla13:05
pbourkeSamYaple: would you mind having a quick glance at this and see if you spot anything wrong? http://paste.openstack.org/show/351922/13:07
SamYaplepbourke: no. couple of things to check. are oyu listening on 192.168.0.50:35357 (ss -plnt)13:08
SamYapleare you sure you have the correct keystone image built13:08
SamYaple(it should be merge in repo as of about an hour ago)13:09
SamYapleim going to tets on a fresh install once i get home13:09
SamYapleharmw: would oyu mind rereviewing https://review.openstack.org/#/c/198519/ with the changes13:11
pbourkeall that should be correct. keystone is listening on that port but stacktracing in it's log13:11
pbourkewould be good to get confirmation it works for someone else as I dont think its me13:12
SamYaplepost stacktrace?13:12
SamYapleyoure probably right, but why does mine work, we need to find the difference13:12
pbourkehttp://paste.openstack.org/show/351945/13:12
SamYaplecan you post your wsgi script from /opt/kolla/keystone ?13:13
pbourkethat said it seems a difference trace than what I was getting yesterday13:13
SamYapleindeed13:13
*** inc0 has quit IRC13:13
pbourkehttp://paste.openstack.org/show/351946/13:14
SamYaplelgtm13:14
sdakeare you guys woring on getting horizon operational?13:14
pbourkeSamYaple: wrt docker-py, is installing from pip ok with you?13:19
SamYapleonly way t ogo in this case13:19
*** dims has joined #kolla13:19
pbourkesdake: who?13:19
SamYapledoubt >=1.2.0 is packaged13:19
pbourkeSamYaple: seems not13:19
SamYaplepbourke: i also have this error now13:22
SamYaplepackage upstream may have changed13:22
sdake,,,,,,,,,,,,,,,,,,,,pip is fine pbourke13:22
*** nihilifer has quit IRC13:23
sdakepbourke anyone woring on that problem13:23
pbourkesdake: looking into it13:23
sdakepbourke your looking into it?13:23
pbourkeyes13:23
sdakecool thanks for th e info13:24
openstackgerritPaul Bourke proposed stackforge/kolla: Document python mysqldb dependency for ansible deploys  https://review.openstack.org/19907713:24
SamYapleah pbourke you may be right about the permission issue. do me a favor `docker exec -it keystone chmod -R 777 /var/log/keystone; docker restart keystone`13:24
SamYaplegive it a few minutes and tell me if you get a 401 error13:25
SamYaplereally heading home this time, bbiaf13:26
openstackgerritPaul Bourke proposed stackforge/kolla: Document dependencies for Ansible deploys  https://review.openstack.org/19907713:26
jmccarthy1General question, ok so for components that we dockerise, they usually will run as what user ? I'm asking as it's giving me permission problems with other non-dockerised processes who are also trying to share a filesystem .. it seems like I need to create a swift user and group inside the container ?13:26
*** jruano has quit IRC13:27
*** jmccarthy1 has left #kolla13:27
*** jmccarthy has joined #kolla13:27
*** jruano has joined #kolla13:27
pbourkejmccarthy: for each component they should run as their own user. in the swift wip we're creating a swift user in swift-base13:28
pbourkejmccarthy: I think the permission problem is unique to our test setup13:28
pbourkeSamYaple: there's also timing issues with the ansible deploy, will log specific bugs when I get a chance but please keep an eye out :)13:34
pbourkeSamYaple: by timing issues I mean it will fail looking for a value or db connection (there are a few), then work on re-run13:34
jmccarthypbourke: Ok I see it now, yea there is an id mismatch13:37
pbourkeSamYaple: fyi https://bugs.launchpad.net/kolla/+bug/147227013:41
openstackLaunchpad bug 1472270 in kolla "Permission issues on /var/log/keystone breaking keystone container" [Undecided,New]13:41
*** ccrouch has joined #kolla13:42
*** jtriley has joined #kolla13:44
SamYaplepbourke: timing issues?13:44
SamYapleoh right yes13:44
SamYaplei know of that bug13:45
SamYapleit is a bug on AIO only13:45
pbourkeok13:45
SamYaplei need to file it13:45
pbourkeyes13:45
SamYapleits because of the when conditional13:45
pbourkealso Im guessing we know about this but really could use a CI check for container functionality13:45
SamYapleit doesnt restart the mariadb bootstrap container if its the first in the list until the playbooks are rerun13:45
pbourkerather than just build13:45
*** ccrouch has left #kolla13:45
SamYapleyea were waiting on OVS and or ansible13:46
pbourkeah13:46
SamYapleits not that the database is timing out, it is that the database isn't actually running (the bootstrap cotainer stops the database13:47
SamYapleor something like that13:47
*** jruano_ has joined #kolla13:49
*** jruano has quit IRC13:50
SamYaplepbourke: yup that process runs as the apache user13:51
SamYaplethe one doing the logging13:51
pbourkeyeah just checking13:51
SamYaplethats the problem13:51
SamYapleive got the patch done13:51
pbourkenice!13:51
pbourkeone step ahead13:51
*** nihilifer has joined #kolla13:51
*** jruano_ is now known as jruano13:52
openstackgerritSam Yaple proposed stackforge/kolla: Fix permissions on log directory for keystone  https://review.openstack.org/19910713:53
openstackgerritSam Yaple proposed stackforge/kolla: Fix permissions on log directory for keystone  https://review.openstack.org/19910713:53
openstackgerritSam Yaple proposed stackforge/kolla: Fix permissions on log directory for keystone  https://review.openstack.org/19910713:54
SamYapleoh god dont look13:54
SamYapledont code while tired13:54
openstackgerritMerged stackforge/kolla: Add min env vars doc validation to pep8 gate  https://review.openstack.org/19747413:55
*** jruano has quit IRC13:58
*** ccrouch has joined #kolla14:02
*** jruano has joined #kolla14:02
pbourkeSamYaple: that's not working. the issue is that the log files themselves are writable only by root14:04
pbourkenot sure why that's happening though14:04
*** jruano has quit IRC14:06
*** bmace_away is now known as bmace14:10
SamYaplepbourke: is your keystone db populated?14:12
*** pbourke has quit IRC14:14
*** sdake has quit IRC14:14
*** sdake has joined #kolla14:15
* SamYaple sighs14:15
*** pdb has joined #kolla14:20
pdbSamYaple: yup Im testing from sratch14:21
pdb*scratch14:21
SamYaplei know what happened14:22
SamYapletwo different patches that didnt touch the same files (so rebase wouldnt break it) merged14:23
SamYapleone in the wsgi script and one in the Dockerfile14:23
SamYaplethey conflict functionally14:23
openstackgerritPaul Bourke proposed stackforge/kolla: Document docker_pull_policy in Ansible docs  https://review.openstack.org/19876014:31
*** bmace has quit IRC14:36
SamYapleERROR: openstack 'NoneType' object does not support item assignment14:44
SamYaplethats what shows up from the current openstackclient in a container14:44
*** jtriley has quit IRC14:45
pdbI was installing it from pip14:47
SamYaplein the container14:47
SamYapleits not bootstraping (after the other isseu was fixed14:47
*** inc0 has joined #kolla14:49
*** bmace has joined #kolla14:50
*** jtriley has joined #kolla14:53
*** bradjones|away is now known as bradjones14:56
openstackgerritPaul Bourke proposed stackforge/kolla: Update centos-source-base from symlink to a real layer  https://review.openstack.org/19789715:01
openstackgerritPaul Bourke proposed stackforge/kolla: Keystone installation from source  https://review.openstack.org/19316615:05
*** jmccarthy has quit IRC15:09
*** jmccarthy has joined #kolla15:09
*** jtriley has quit IRC15:14
SamYaplepdb: hehe. we are also running into an ansible bug ontop of all of this15:15
SamYaple:D15:15
pdbo_O15:15
*** dasm has quit IRC15:15
openstackgerritSam Yaple proposed stackforge/kolla: Fix permissions on log directory for keystone  https://review.openstack.org/19910715:15
SamYapletry out that build please15:16
*** dims has quit IRC15:16
pdboh my15:17
SamYaplethat actually probably solves your race conditions (not for the db, thats a different one) but the rabbit one for sure15:18
*** jruano has joined #kolla15:19
pdbTASK: [keystone | Waiting for bootstrap container to exit15:20
pdbstderr: Error response from daemon: wait: no such id: bootstrap_keystone15:20
SamYaplethats a container that actually starts15:20
SamYapleyoull want to look into that15:20
SamYaplesince we dont clean it up until the next step it should be on your system...15:21
pdbit might be a result of the race15:21
SamYapledid you rebuilt the container?15:21
*** jruano has quit IRC15:22
pdb:/15:22
pdbwoops15:22
pdbmy bad15:22
rhalliseysdake, ok got the green light for san jose15:22
SamYaplepdb: i can testify in a brand new fresh environment this patch gets me from nothing to keystone responding outside teh container15:23
openstackgerritBorneMace proposed stackforge/kolla: Fixed pep8 failures in all tools/ scripts  https://review.openstack.org/19915115:25
bmacefyi sam, a lot of the pep8 fixes were in the new build script so it would be great if you could give it a look15:26
SamYaplebmace: was pep failing before?15:28
*** mstachow has quit IRC15:28
bmacepep was completely broken.  i have a couple of bps to fix it.. it wasn't doing any pep8 at all15:28
SamYaplebmace: https://review.openstack.org/#/c/198533/ looks like it passed all the pep8 checks15:28
SamYapleis our pep8 gate not working?15:28
bmaceit was just checking the ability to load yaml / json / dockerfile stuff15:28
bmaceexactly.. working on fixing it :/15:29
SamYapleah15:29
SamYapleand the build script passes the pep8 gate with your changes?15:29
bmaceit passes what the pep8 gate will look like, yeah :)15:29
SamYaplewfm15:29
bmacewhich is essentially just flake815:29
SamYaplewhen will it be in the jenkins checks?15:30
SamYaplewell probably do it non-voting at first15:30
bmacei'm working that stuff through the project-config guys right now15:30
bmaceyes, that is my plan, making pep8 non-voting. then fix it to actually use flake 815:31
pdbSamYaple: no joy unfortunately. http://paste.openstack.org/show/352276/ - from scratch. shows 3 runs, before it goes no further15:31
bmacei broke out our current checks into a separate check / job15:31
bmacebtw, pdb, can't make the tox changes in that review since it will break the gate.. this isn't a fix for all pep8 issues in the code base, just in tools/15:31
pdbbmace: makes sense, I guess what I should have said was maybe the commit msg should say Partially-Implements15:32
pdb?15:32
SamYaplepdb: that is you cannot connect to the datbase though... then you have an out of date globals.yml (missing openstack_region_name)15:33
SamYapleyou should probably fix those15:33
bmaceah, sure, i can change that if you want :)15:33
openstackgerritBorneMace proposed stackforge/kolla: Fixed pep8 failures in all tools/ scripts  https://review.openstack.org/19915115:34
SamYaplebmace: why dont you get the gate in and THEN fix the scripts?15:35
bmacewell i certainly don't want to get it in, and voting15:35
bmaceotherwise we will be broken..15:35
SamYapleno it starts non-voting15:35
SamYaplefix the scripts (until successful)15:36
SamYaplethen we make it voting15:36
bmaceand i have no idea how long the project-config guys are going to take15:36
pdbSamYaple: working now, thanks15:36
SamYaplelike fully working?15:36
bmaceyes, the way i have it set up now is that pep8 becomes non-voting and the old json / yaml / dockerfile stuff stays voting but is renamed.15:36
pdbSamYaple: well it still fails the first time trying to connect to db, but after the second run I can issue a token15:37
SamYaplebmace: can you walk me through the steps to add the gate? I want to add a gate for the new build.py (non-voting) and i was confused greatly by project-config15:37
pdbbmace: I guess the problem is you push this in now, by the time the gate goes we'll have to fix again15:37
SamYaplepdb: yea the first run fails due to the AIO issue15:37
bmacewhy would we need to fix it again? not that other issues might not come up, but the issues that i am fixing stay fixed.. especially in files that aren't changed much15:38
SamYaplebuild.py will be getting changed alot :)15:38
pdbif there's no auto check people are bound to violate the rules again15:38
bmaceSamYaple: as soon as those guys actually approve it i will be more than happy to show you how i did things.. i think i got it right this time, worked off of the pep8 standard template for the new test.15:38
pdbbmace: no problem with the change just I think that's what SamYaple is getting at15:39
bmacejust because you are going to sleep i your bed tonight doesn't mean there is no use in making it when you wake up, imho15:39
SamYaplebmace: agreed, if the gate was in (so we could fix it) but new things are going to break a gate that doesnt exist yet15:40
SamYapleso eniveitably well haveto patch again15:40
*** shardy_ has joined #kolla15:40
SamYaplesince you did the work im good with it, but lets hold off on more changes until the gate is in15:40
pdbbmace: lol15:40
bmaceok.  hopefully the infra guys look at it some time soon :/15:40
pdbif i have room service (the gate) I dont make it up15:41
*** shardy has quit IRC15:41
bmace:P15:42
SamYapleok these metaphors can end15:42
pdb:D15:42
SamYapleits like making your bed when you have room service15:42
SamYaple(that was a simile)15:43
*** dims has joined #kolla15:43
*** shardy_ has quit IRC15:45
*** jtriley has joined #kolla15:46
*** shardy has joined #kolla15:46
SamYaplehey guys. logging for the new build script15:46
SamYapleI have this idea15:46
SamYapleafter a check to see if tmux is installed on the system (so this would be option) I could use tmux to open up a new screen _per_ build15:47
SamYapleso you could tail the logs of each building image15:47
*** nihilifer has quit IRC15:47
SamYapleremove the tmux window when it successed in a build, keep it around if it fails15:47
SamYaplethat way you get an active idea of whats going on with what15:48
*** sdake_ has joined #kolla15:48
*** sdake has quit IRC15:48
pdbcrazy but could work15:49
SamYapleive done it before15:49
pdbI guess devstack does something similar with screen15:49
SamYapleyea i think they do but screen hasnt been developed for.... 15? years15:49
SamYapleso im not using it15:49
SamYapleeven15:50
SamYapleonce15:50
pdbyeah15:50
pdbtmux is better15:50
*** jtriley has quit IRC15:52
*** jtriley has joined #kolla15:55
openstackgerritPaul Bourke proposed stackforge/kolla: Document docker_pull_policy in Ansible docs  https://review.openstack.org/19876015:58
openstackgerritSam Yaple proposed stackforge/kolla: Fix permissions on log directory for keystone  https://review.openstack.org/19910715:59
*** daneyon has joined #kolla16:00
SamYaplehey team. https://review.openstack.org/#/c/199107/ fixes a blocking ansible issue and I would appreciate reviews on this. thanks!16:00
*** mfalatic has joined #kolla16:01
pdbjmccarthy: I'm going to push my latest work to the github branch. could you try pulling down and starting all the object containers?16:03
pdbjmccarthy: just getting some errors in the proxy server log and would be interested to see if you're getting the same16:03
*** vinkman has joined #kolla16:10
*** jmccarthy has quit IRC16:11
*** jmccarthy1 has joined #kolla16:11
*** inc0 has quit IRC16:11
*** absubram has joined #kolla16:16
jmccarthy1pdb: one sec16:16
pdbjmccarthy1: no rush16:16
*** jmccarthy1 has quit IRC16:17
*** jmccarthy has joined #kolla16:18
pdbjmccarthy: object server working! :)16:25
pdbjmccarthy: what was your test for the auditor so I can see what it's up to16:27
jmccarthyWell they all built ;)16:29
*** jasonsb has quit IRC16:29
jmccarthyFor auditor I was putting an object into the store, and then mucking with it directly on the filestore (on /srv) so then the hash would disagree - the auditor will get a copy from somewhere to 'fix' it if it can, in any case the mismatched object will end up in a quarantine store (also on /srv)16:31
pdbyup seems to work16:35
*** sdake_ is now known as sdae16:35
jmccarthysweet16:36
*** sdae is now known as sdake16:39
jmccarthygotta run, laters16:55
*** jmccarthy has left #kolla16:55
*** diogogmt has joined #kolla16:57
*** rbergeron has quit IRC17:05
*** gfidente has quit IRC17:11
*** jasonsb has joined #kolla17:15
*** pradk has joined #kolla17:21
*** jasonsb has quit IRC17:31
*** jasonsb has joined #kolla17:32
*** rbergeron has joined #kolla17:35
*** rbergeron has quit IRC17:37
*** rbergeron has joined #kolla17:37
*** athomas has quit IRC17:52
*** coolsvap has joined #kolla18:06
harmwcoolsvap: ping18:16
openstackgerritMerged stackforge/kolla: make merge_configs idempotent  https://review.openstack.org/19851918:16
coolsvapharmw, pong18:17
harmwcoolsvap: could you update https://etherpad.openstack.org/p/kolla-standard-start please, if neccesary?18:17
coolsvapharmw, yes, I have sent horizon patch for review, working on barbican, was travelling last couple of days and in conf18:19
coolsvapupdating it18:19
coolsvapthe ones with review comments, I will send patches after my day in conf is over18:21
coolsvaptoday eod18:21
bmacecould another core take a quick glance at https://review.openstack.org/#/c/199151/ ?  it is just pep8 formatting fixes.18:23
harmwcoolsvap: when is that, UTC :)18:28
harmwbmace: gimme a sec18:28
rhalliseybmace, all set18:31
coolsvapharmw, currently i am in EDT, UTC mostly after 22:0018:31
openstackgerritMerged stackforge/kolla: Fixed pep8 failures in all tools/ scripts  https://review.openstack.org/19915118:36
*** sdake has quit IRC18:36
bmacethanks much rhallisey / harmw18:38
*** sdake has joined #kolla18:52
*** dims has quit IRC18:53
*** sdake_ has joined #kolla18:56
*** dims has joined #kolla18:58
*** sdake has quit IRC19:00
*** nihilifer has joined #kolla19:09
*** vinkman has quit IRC19:18
*** vinkman1 has joined #kolla19:18
harmwsdake_: I won't make it tomorrow, meeting19:22
harmwsdake_: https://etherpad.openstack.org/p/kolla-standard-start is updated though19:22
sdake_harmw mind providing update now then plz19:22
harmwcoolsvap: could you pickup https://review.openstack.org/#/c/198223/ first?19:23
sdake_what is status19:23
harmwsdake_: working on doing just that19:23
harmw:)19:23
harmwgimme a sec19:23
harmwor two19:23
sdake_3 max19:23
sdake_3s up ;)19:23
harmwjeez19:23
harmwok19:23
* harmw bails19:24
sdake_jk ;)19:24
harmwhm, weird19:24
harmwall letter F's (lowercase) are looking like upside-down asses, farting19:24
harmwin gerrit, that is19:25
harmwfeels like I'm hacked by the Russians19:25
harmwcoolsvap: horizon only needs a +2, cool :)19:26
harmw(since I can do that)19:26
harmwreviewing that right now19:26
*** shardy_ has joined #kolla19:31
openstackgerritMerged stackforge/kolla: Add config-internal/config-external for horizon  https://review.openstack.org/19860019:32
*** shardy has quit IRC19:32
*** shardy_ has quit IRC19:36
*** shardy has joined #kolla19:37
harmwsdake_: work on kolla-standard-start is progressing, around the weekend 8/9 cat.1 containers have been merged. Cinder is underway. Out of 9 cat.2 containers that have seen action, 3 are completed and 1 is pending the first review. Additional patches are required from coolsvap and diga_. Work has not begun on mongodb and hautoproxy containers. If there are no volunteers, I will take care of those myself but that may be decided during tomorrow's meeting. Mari19:42
*** rbergeron has quit IRC19:43
*** dims has quit IRC19:48
*** rbergeron has joined #kolla19:49
*** jruano has joined #kolla19:49
*** rbergeron has quit IRC19:59
*** rbergeron has joined #kolla20:00
harmwsdake_: got that?20:04
coolsvapharmw, sdake_ will pickup the patchs in queue tonight20:24
sdake_harmw thanks bro20:25
*** daneyon has quit IRC20:26
sdake_coolsvap what are you talking about?20:30
sdake_tonight I am going to bed ;-)20:30
coolsvapsdake_, I was travelling for last two days and in conf today, was talking the same with harmw20:31
sdake_what do you mean pikcup patches in the queue?20:32
coolsvapI will pick up the patches which have review comments for kolla-start20:32
sdake_I am actually working on installing modules for ansible20:32
sdake_oh you will20:32
sdake_I toughtyou were talking aboutwhat I would do :)20:32
coolsvap:)20:32
sdake_everyone that works on kolla is mater of their own time20:33
sdake_if someone doesn't want to work on something they are not forced to - since how cold I possibly force them anyway? :)20:33
coolsvap:)20:34
*** coolsvap is now known as coolsvap|away20:39
harmwyou can't sdake_ ?20:39
harmwthen why is there this magical axe up over my head everytime your name pops up in here?20:39
sdake_can you photo it ;)20:40
harmwoh I once tried, but it killed my phone20:41
sdake_hah20:41
harmwkinda like snapchat, but different20:41
harmwand that underscore is quite the pointy thing as well :>20:43
harmwnn guys20:46
*** daneyon has joined #kolla21:10
*** dims has joined #kolla21:13
*** sdake_ is now known as sdake21:20
*** bmace has quit IRC21:21
*** dims has quit IRC21:30
*** pradk has quit IRC21:32
*** bmace has joined #kolla21:33
*** dims has joined #kolla21:35
*** dims has quit IRC21:35
*** Guest7393 has joined #kolla21:35
*** sdake_ has joined #kolla21:48
*** jtriley has quit IRC21:51
*** sdake has quit IRC21:52
*** jtriley has joined #kolla21:52
*** jtriley has quit IRC21:58
*** nihilifer has quit IRC22:05
*** prad has quit IRC22:16
*** daneyon has quit IRC22:34
*** jtriley has joined #kolla22:38
*** jtriley has quit IRC22:49
*** jruano has quit IRC22:58
*** absubram has quit IRC23:11
*** britthouser has quit IRC23:23
*** jtriley has joined #kolla23:29
*** diogogmt has quit IRC23:30
*** diogogmt has joined #kolla23:35

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