Monday, 2015-12-21

« Sunday, 2015-12-20 Index Tuesday, 2015-12-22 »
*** salv-orlando has quit IRC00:03
*** tzn has quit IRC00:16
*** salv-orl_ has quit IRC00:36
*** mbound_ has quit IRC00:40
*** dims has joined #kolla00:44
*** dims has quit IRC01:02
*** sdake_ has joined #kolla01:03
*** sdake has quit IRC01:04
openstackgerrithardik proposed openstack/kolla: Added dockerfiles for mistral.  https://review.openstack.org/25889001:19
openstackgerrithardik proposed openstack/kolla: Added dockerfiles for mistral.  https://review.openstack.org/25889001:20
*** kjelly has quit IRC01:23
*** salv-orlando has joined #kolla01:37
*** mbound_ has joined #kolla01:41
*** mbound_ has quit IRC01:46
*** tfukushima has joined #kolla01:53
*** dims has joined #kolla02:07
*** jasonsb has joined #kolla02:07
*** jasonsb has quit IRC02:07
*** jasonsb has joined #kolla02:09
*** salv-orlando has quit IRC02:12
*** sdake_ has quit IRC02:47
*** dave-mccowan has quit IRC02:51
*** jasonsb has quit IRC03:01
*** sdake has joined #kolla03:04
*** stvnoyes has quit IRC03:04
*** stvnoyes has joined #kolla03:05
*** sdake_ has joined #kolla03:07
*** sdake has quit IRC03:09
*** ssurana has joined #kolla03:18
*** coolsvap|away is now known as coolsvap03:23
openstackgerritSwapnil Kulkarni (coolsvap) proposed openstack/kolla: Add trove to aux in build.py  https://review.openstack.org/25982903:28
*** dims has quit IRC03:32
*** coolsvap is now known as coolsvap|away03:33
*** coolsvap|away is now known as coolsvap03:40
*** klint has joined #kolla03:43
Guest55431hi is anyone around who I can talk to about mariadb module? I have found an issue with multinode04:25
*** Guest55431 is now known as _RA04:26
_RAI have a fix04:27
*** tfukushima has quit IRC04:37
*** tfukushima has joined #kolla04:38
*** tfukushima has quit IRC04:43
*** tfukushima has joined #kolla04:53
*** tfukushima has quit IRC04:58
*** ssurana has quit IRC05:08
*** tfukushima has joined #kolla05:31
*** yuanying has joined #kolla05:56
*** nihilifer has joined #kolla06:00
*** _RA has quit IRC06:17
*** sdake_ has quit IRC07:17
*** gfidente has joined #kolla07:29
*** gfidente has joined #kolla07:29
*** kjelly has joined #kolla07:35
*** unicell has quit IRC07:38
*** unicell has joined #kolla07:38
*** akwasnie has joined #kolla07:52
*** akwasnie has quit IRC08:03
*** salv-orlando has joined #kolla08:11
*** salv-orlando has quit IRC08:14
*** Jeffrey4l has quit IRC08:15
*** Jeffrey4l has joined #kolla08:22
*** unicell1 has joined #kolla08:23
*** unicell has quit IRC08:24
*** kproskurin has joined #kolla08:29
*** kproskurin has quit IRC08:32
*** kproskurin has joined #kolla08:34
*** shardy has joined #kolla08:37
*** mbound_ has joined #kolla08:53
*** cloudnautique has quit IRC08:55
*** athomas has joined #kolla09:01
*** akwasnie has joined #kolla09:07
*** akwasnie has quit IRC09:28
*** salv-orlando has joined #kolla09:34
*** openstack has joined #kolla15:35
*** sdake has joined #kolla15:36
*** sdake has quit IRC15:45
*** sdake has joined #kolla15:48
*** Jeffrey4l has quit IRC15:49
Guest9750Thanks <ajafo> 1.5.0 worked now it failed further but it seems to be related to build15:51
*** sdake_ has joined #kolla15:52
*** sdake has quit IRC15:53
ajafoGuest9750: n/p15:53
*** dave-mccowan has quit IRC15:53
*** cloudnautique has joined #kolla15:55
*** cloudnautique has quit IRC15:55
*** cloudnautique has joined #kolla15:56
openstackgerritArtur Zarzycki proposed openstack/kolla: Ubuntu binary containers.  https://review.openstack.org/26006915:59
dmsimardSamYaple: looks like there are issues in Mitaka (at least RDO from what I can tell) due to Kolla using mysqldb versus pymsql16:00
*** sdake has joined #kolla16:00
dmsimardSamYaple: upstream is going full pymysql, puppet-openstack already made the move16:01
*** blahRus has joined #kolla16:01
dmsimardgetting failed migrations in jobs right now, i.e, http://logs.openstack.org/17/258017/7/check/gate-kolla-dsvm-deploy-centos-binary/6a2bf73/console.html#_2015-12-21_15_42_27_40516:01
*** smartshader has joined #kolla16:02
*** sdake_ has quit IRC16:02
dmsimardfrom poking around, I can't see where kolla would be explicitely installing anything else (mysqldb) to try and change that16:03
*** dave-mccowan has joined #kolla16:07
SamYapledmsimard: we should make the move too, patch incoming16:10
dmsimardSamYaple: what, already ?16:10
dmsimardIt looks like it involves explicitely installing pymysql and changing the connection strings in the conf files16:10
SamYapledmsimard: yea will take two seconds16:10
dmsimardSamYaple: ok i'll rebase on top of it to see what happens, you're awesome <316:13
SamYapledmsimard: there wont be testing on my end just yet, only testing in the gate until i get some more free time later today16:14
openstackgerritSam Yaple proposed openstack/kolla: Convert to pymysql  https://review.openstack.org/26008016:16
SamYaplesyntax looks correct there, well see what the gate says16:16
sdakehey folks I'm on pto this week16:17
sdakeI will be in wednesday to cut mitaka-116:17
sdakebut otherwise, expect me to be spotty in and out16:18
nihiliferSamYaple: what about binary installation? are there packages for pymysql?16:21
openstackgerritArtur Zarzycki proposed openstack/kolla: Ubuntu binary containers.  https://review.openstack.org/26006916:26
dmsimardSamYaple: don't think gate will pass, the package name is not the right one16:28
dmsimarder, yeah, I kinda meant to make the same comment as nihilifer - I put my comment in the pip section16:29
dmsimardthe pip install was hidden as part of the skipped common lines in the diff :/16:30
SamYaplenihilifer: i just threw it up a second ago, i can add that thanks16:34
openstackgerritSam Yaple proposed openstack/kolla: Convert to pymysql  https://review.openstack.org/26008016:36
SamYaplenihilifer: re: https://review.openstack.org/#/c/259624/16:49
SamYaplenihilifer: ansible _requires_ somethings that break pep816:49
SamYaplewe could choose to ignore those specific tests, but ignoring the folder is what we have been doing up until this point16:50
SamYaplepersonally I would prefer just to ignore the breaking tests (like H101 i believe is one) and open the ansible code up to almost full pep8 tests16:50
nihiliferhmmm... but many things are breaking there?16:51
nihiliferexcept of wildcard import?16:51
nihiliferi can check it in a free time of course16:51
SamYapleright well its written in the coding style of ansible16:53
SamYaplethats the whole deal16:53
SamYaplewe agreed to do it the ansible way which doesnt follow pep8 at all, if we want to change that (and I am in favor of that), we need to ignore the required, but breaking things ansible does16:54
SamYapleits not just teh wildcard imports16:54
openstackgerritSam Yaple proposed openstack/kolla: Simplify config creation  https://review.openstack.org/25962416:58
nihiliferif only wildcard is a problem, then i'd propose to do just "# noqa" on them16:59
nihiliferif something else, then we probably to ignore moar things in tox.ini17:00
SamYaplenihilifer: its not the only thing17:00
SamYaplei remember at least 1 other17:00
SamYapledoes "# noqa" make tox ignore it?17:00
nihiliferSamYaple: afaik, yes17:01
SamYapleok so here is what i propose17:01
SamYaplea new patchset based on this one where we filter just a few tests from tox and check all the ansible modules17:01
SamYaplethe tests we would filter would be the things ansible requires to work17:01
SamYaplebut past that the files are still checked17:01
SamYapleH303 and F403 will be broken for sure17:03
*** akwasnie has joined #kolla17:03
nihiliferSamYaple: cool, let's try this17:05
openstackgerritSam Yaple proposed openstack/kolla: pep8 the ansible modules  https://review.openstack.org/26010117:05
SamYaplenihilifer: ^ there you go. that will break 20 ways til sunday, but ill update it with coding fixes in a minute17:05
nihiliferthx17:06
*** akwasnie has quit IRC17:20
Guest9750While doing kolla-ansible deploy got this17:33
Guest9750TASK: [common | Starting Ansible container] ***********************************  failed: [localhost] => {"changed": true, "failed": true} msg: Docker API Error: Cannot start container bf1e1e384c90fa676b0cb03d315800034ce9fd5e41e56e667c2a4bb30d972c32: [8] System error: not a directory  FATAL: all hosts have already failed -- aborting17:33
*** kproskurin has quit IRC17:33
*** unicell1 has quit IRC17:33
SamYapleGuest9750: ive seen that once or twice before. An issue with docker17:38
*** akwasnie has joined #kolla17:38
SamYapleif i recall the only way I could fix it was to purge the /var/lib/docker folder, but first I would recommend just removing the container in question and restarting17:39
*** akwasnie has quit IRC17:39
Guest9750@SamYaple do I need to build again then?17:40
SamYapleGuest9750: if you remove the container, no, if you remove the /var/lib/docker folder you would17:41
Guest9750okay17:41
SamYapleits a docker problem here, not a kolla one17:41
Guest9750I removed the particular container but same issue persists17:44
Guest9750I guess have to remove /var/lib/docker17:44
*** athomas has quit IRC17:53
*** mbound_ has quit IRC17:55
*** salv-orl_ has joined #kolla18:00
*** salv-orlando has quit IRC18:03
*** barthalion has joined #kolla18:08
*** athomas has joined #kolla18:10
*** gfidente is now known as gfidente|afk18:14
*** unicell has joined #kolla18:23
*** tzn has joined #kolla18:26
*** unicell has quit IRC18:27
*** unicell has joined #kolla18:34
*** unicell has quit IRC18:34
*** unicell1 has joined #kolla18:34
*** ssurana has joined #kolla18:49
*** shardy is now known as shardy_afk18:58
openstackgerritSidharth Surana proposed openstack/kolla: Make galeradb bootstraping robust  https://review.openstack.org/25496118:58
*** mbound_ has joined #kolla19:09
*** mbound_ has quit IRC19:11
*** mbound_ has joined #kolla19:11
*** tzn has quit IRC19:17
openstackgerritDavid Moreau Simard proposed openstack/kolla: Convert to pymysql  https://review.openstack.org/26008019:48
openstackgerritDavid Moreau Simard proposed openstack/kolla: Make the yum repositories configurable in the base image  https://review.openstack.org/25801719:48
dmsimardAw crap, I didn't mean to do that19:48
barthaliongit review of death19:48
dmsimardmeant to rebase my patch on the of the pymysql review :(19:49
dmsimardSamYaple: ^ didn't change anything in the pymysql patchset, it's probably just rebased on top of master19:49
nihiliferyou did "git review" without "-R"? or something more wild? ;)19:49
dmsimardnihilifer: I rebased manually19:50
* dmsimard frowns in shame19:52
dmsimardwhew, at least gerrit recognized that it was just a rebase19:53
*** tzn has joined #kolla19:54
*** dwalsh has quit IRC19:55
*** zeus` has joined #kolla19:56
zeus`hi all!!19:57
zeus`I'm looking for some code for the ansible-ceilometer since there's no code to run the ceilometer just the docker files, or we have to update the config files by hand to enable ceilometer on nova and cinder?19:59
dmsimarddependencies were clearer in the old gerrit UI :(20:00
openstackgerritDavid Moreau Simard proposed openstack/kolla: Make the yum repositories configurable in the base image  https://review.openstack.org/25801720:00
*** ssurana has quit IRC20:08
SamYapledmsimard: not to worry. itll be fine20:11
*** ssurana has joined #kolla20:19
openstackgerritSam Yaple proposed openstack/kolla: pep8 the ansible modules  https://review.openstack.org/26010120:29
openstackgerritSam Yaple proposed openstack/kolla: Simplify config creation  https://review.openstack.org/25962420:29
*** ashestakov has joined #kolla20:34
*** neilus has joined #kolla20:44
openstackgerritSidharth Surana proposed openstack/kolla: Make galeradb bootstraping robust  https://review.openstack.org/25496121:14
*** shardy_afk is now known as shardy21:19
openstackgerritSam Yaple proposed openstack/kolla: pep8 the ansible modules  https://review.openstack.org/26010121:19
openstackgerritSam Yaple proposed openstack/kolla: pep8 the ansible modules  https://review.openstack.org/26010121:24
openstackgerritSam Yaple proposed openstack/kolla: Simplify config creation  https://review.openstack.org/25962421:24
openstackgerritSidharth Surana proposed openstack/kolla: Make galeradb bootstraping robust  https://review.openstack.org/25496121:26
*** dims_ has quit IRC21:33
*** dims has joined #kolla21:33
openstackgerritMerged openstack/kolla: Add trove to aux in build.py  https://review.openstack.org/25982921:37
ashestakovcoolsvap: ping21:39
*** dims has quit IRC21:43
openstackgerritMerged openstack/kolla: Fix retrieving api_interface  https://review.openstack.org/24799921:44
*** athomas has quit IRC21:48
*** dims has joined #kolla21:49
dmsimardSamYaple: for the any() block, it's purpose is to set the defaults if no parameters are set21:52
dmsimardI guess I could get rid of it entirely and put the default logic wherever the config from file is loaded instead21:53
SamYapledmsimard: i understand that, but if you look at it you are testing the rpm_repo_url param in the any function, since you _always_ use rpm_repo_url anyway, why test it?21:55
dmsimardthe rpm_repo_url is outside of the if becase it will always run since a default is set21:55
SamYapleso why test it?21:55
dmsimardbecause if rpm_repo_url is set, I don't want to use the default21:55
dmsimardbut I have no way to tell if a value is set other than testing all three values21:56
dmsimard(unless I'm mistaken)21:56
SamYapleso all three values must be set?21:56
dmsimardif no values are set in the three vars, I want to set the default21:56
dmsimardif at least one is set, don't use the default21:56
SamYapleright, but if i set rpm_file and rpm_repo_file  but _NOT_ rpm_repo_url, rpm_repo_url will never get set21:57
dmsimardand that's fine21:57
SamYapleno its not, because it wont get a default21:57
SamYapleread the code21:57
dmsimardif there are no rpm_repo_url, it shouldn't be setting any21:58
dmsimardit's a default to be used only if no repositories are set at all21:58
dmsimardlet me rephrase that21:58
SamYaplewhat it looks like you need to do is set the rpm_repo_url as the default value for the variable and do an 'if rpm_file and rpm_repo_url:' instead of that any()22:00
dmsimardIf there is a rpm_file set (i.e, trunk.rdoproject/release.rpm), the only repository will be that one single rpm_file. rpm_repo_url will be empty and that's fine, it will not be setting up any repositories from URL'd .repos.22:00
dmsimardhowever, if I don't have any settings for all three parameters, I want to set a sane default22:01
dmsimardit's quite possible my logic (in the code) is flawed, I'll try to see if I can make it better22:01
SamYaplewell that can be worked into argparse stuff22:01
dmsimardbut I had to remove the args from argparse :P22:02
SamYapleno you didnt22:02
SamYapleyou still parse them with argparse22:02
SamYapleoh wait nvm wrong patchset22:02
SamYapleyea youre right22:03
dmsimardyou remind me that I screwed up though, I should remove those keys from the defaults var22:03
dmsimardsince I removed the args22:03
SamYapleso let me ask, do you need to use rpm_file _and_ rpm_repo_file at the same time?22:03
dmsimardI don't *need* to, but it should work22:04
SamYapleim not sure the difference between the two then22:04
dmsimardgiven a, b, c, I should be able to use any combination22:04
dmsimardrpm_file is any .rpm file22:04
SamYapleoh. hmm22:04
dmsimardlike this: https://www.rdoproject.org/repos/rdo-release.rpm or /tmp/rdo-release.rpm22:04
SamYaplethen why have an if statement at all?22:05
dmsimardrpm_repo_file is a .repo file, like this: http://trunk.rdoproject.org/centos7/delorean-deps.repo22:05
SamYaplejust use a for loop22:05
SamYapleif the variable is empty, the for loop doesnt do anything22:05
dmsimardwhen a config isn't set in configparser, what does it yield for a key ? Is it set (i.e, defaultdict) or does it give None ?22:06
SamYapledoesnt matter, if its an empty string/list/dict or None it still skips the for loop22:07
dmsimardhmm, checking22:07
SamYapletry it out real quick22:07
SamYaplefor i in '':22:07
SamYaple  print('nothing)22:07
SamYapleit wont print22:07
SamYaplesame with [],{},None22:08
dmsimardIt can iterate over an empty string, an empty list but not none22:08
*** dims_ has joined #kolla22:08
dmsimardkeyerror none is not iterable22:08
dmsimardbut I can default them to an empty string instead of none I guess22:08
*** dims has quit IRC22:08
SamYaplei think they are default to ''22:10
SamYapleyoure right about iterable and None being bad22:11
*** shardy has quit IRC22:12
*** dims has joined #kolla22:12
dmsimardyeah the for loops work fine, but how do I set up the repo_url only if none of the three settings are set without a loop ?22:13
*** dims_ has quit IRC22:13
SamYapleSet it in the default dict with the appropriate default (it will get overridden by the config file)22:17
dmsimardokay, I'll try to test that22:18
*** sdake has quit IRC22:21
openstackgerritDavid Moreau Simard proposed openstack/kolla: Make the yum repositories configurable in the base image  https://review.openstack.org/25801722:23
*** dims has quit IRC22:29
*** jtriley has quit IRC22:40
*** tzn has quit IRC22:49
*** neilus has quit IRC22:51
*** neilus has joined #kolla22:52
*** neilus has quit IRC22:52
*** neilus has joined #kolla22:52
*** ashestakov has quit IRC22:53
*** blahRus has quit IRC23:01
*** neilus has quit IRC23:01
*** neilus has joined #kolla23:01
*** Jeffrey4l has joined #kolla23:15
*** neilus has quit IRC23:21
*** dims has joined #kolla23:21
*** neilus has joined #kolla23:22
*** tzn has joined #kolla23:24
*** tzn has quit IRC23:30
*** dims has quit IRC23:30
*** dims has joined #kolla23:32
*** ssurana has quit IRC23:32
*** sdake has joined #kolla23:45
« Sunday, 2015-12-20 Index Tuesday, 2015-12-22 »

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