Friday, 2015-12-04

*** dschroeder has quit IRC00:12
*** drwho has joined #openstack-freezer00:13
*** drwho has quit IRC00:17
*** sputnik13 has joined #openstack-freezer00:31
sputnik13helloooo00:31
*** sputnik13 has quit IRC01:18
*** hidekazu has joined #openstack-freezer01:42
*** hidekazu has left #openstack-freezer01:43
*** zhonghua has joined #openstack-freezer02:50
*** zhonghua-lee has quit IRC02:54
*** drwho has joined #openstack-freezer04:16
*** drwho has quit IRC04:20
openstackgerritdaiki kato proposed openstack/freezer: Fix typo: my scheduled backup 6 => schedule_backups 6  https://review.openstack.org/25171005:05
*** vannif has quit IRC05:42
*** zhonghua has quit IRC06:19
*** zhonghua-lee has joined #openstack-freezer06:20
*** drwho has joined #openstack-freezer07:55
*** drwho has quit IRC08:00
*** reldan has joined #openstack-freezer09:58
*** openstackgerrit has quit IRC10:32
*** openstackgerrit has joined #openstack-freezer10:32
*** szaher has joined #openstack-freezer10:53
m3m0sputnik13 hello :)11:53
*** reldan has quit IRC11:53
*** reldan has joined #openstack-freezer12:13
*** daemontool has joined #openstack-freezer13:18
daemontoolI've setup my environment, looking at http://logs.openstack.org/68/252568/1/check/gate-freezer-pylint/4d7d9a2/console.html13:24
daemontoolall, that issue is happening most probably because pbr is not installing pylint13:34
reldandaemontool: 2015-12-02 19:42:19.410 | pylint installed:13:36
daemontoolreldan:  are you sure that is installed from a previous install?13:37
daemontoolbecause when you execute13:37
daemontooltox -r -v13:37
daemontoolit should install pylint in the virtualenv that tox generate13:37
daemontoolbut it isn't13:37
daemontoolin my test13:37
daemontoolnot even pbr is installed13:37
reldanI don’t know I just watching the log13:37
daemontoolI'm just trying this on a fresh vm13:37
daemontoolok13:38
daemontoolsure, I might be wrong13:38
daemontoolbut in my case that can be one of the possible issues13:38
reldanProblem importing module classes.py: cannot import name InferenceContext13:38
reldan2015-12-03 18:13:30.121 | Problem importing module classes.pyc: cannot import name InferenceContext13:38
reldanI don’t know what it is13:38
*** drwho has joined #openstack-freezer13:40
daemontoolclasses.py13:40
daemontoolis a file part of pylint13:40
daemontoolafaik13:40
reldanwhat is astroid? https://bitbucket.org/logilab/pylint/issues/660/python35-problem-importing-module13:41
daemontoolgood question is a pylint dep13:42
daemontoolI think our issue is related13:43
daemontoolas we have since we changed our requirements aligned to Liberty?13:44
*** drwho has quit IRC13:44
daemontoolSlashme: did you see this issue per chance also with Kilo?13:45
daemontoolreldan:  pylint is installed on the tox env13:50
reldandon’t understand what is wrong13:50
daemontoolwe are 213:52
daemontool:)13:52
daemontoolreldan: check here https://github.com/openstack/requirements/blob/stable/liberty/global-requirements.txt13:55
daemontoolcheck the last commit message13:55
daemontoolJenkins Merge "Cap astroid to unbreak pylint." into stable/liberty13:55
reldanastroid<1.4.0 # breaks pylint 1.4.413:55
reldanbut we have astroid 1.4.413:55
reldanin our repo13:55
reldannot in repo13:55
reldanin log13:56
reldanpylint installed: appdirs==1.4.0,APScheduler==3.0.4,astroid==1.4.1,13:56
reldanah no13:56
reldan1.4.1 we have13:56
reldanbut anyway it is higher than 1.4.013:56
daemontoolastroid<1.4.013:57
reldanYes, but in our log we are installing 1.4.1. So probably we just need to retry commits13:57
daemontoolthat's the astroid from the global-requirements.txt in liberty13:57
daemontoolone sec13:58
daemontoolby changing that locally it works13:59
daemontoollet's see if it pass upstream ci13:59
reldanYes, let’s retry. Do you know how I can rebuild my pull requests?13:59
daemontoolrebuild?14:00
daemontoolclone master14:00
daemontoolthen14:00
daemontoolgit fetch https://review.openstack.org/openstack/freezer refs/changes/68/252568/1 && git checkout FETCH_HEAD14:00
daemontoolchange whatever you need to change14:00
daemontoolgit commit --amend14:00
daemontoolgit review14:01
reldanI see14:01
daemontoolgit add would go before fgit commit14:01
reldanThank you, just I thought I can leave a comment with some magic word like “rebuild” and it will be retriggered14:01
daemontoolah like rebase?14:02
daemontoolwell nope14:02
daemontool:)14:02
openstackgerritEldar Nugaev proposed openstack/freezer: Removing old integration tests  https://review.openstack.org/25311714:06
reldanSitll doesn’t work and uses 1.4.1 http://logs.openstack.org/17/253117/2/check/gate-freezer-pylint/105ae9b/console.html14:16
*** daemontool has quit IRC14:20
openstackgerritEldar Nugaev proposed openstack/freezer: Removing old integration tests  https://review.openstack.org/25311714:26
*** daemontool has joined #openstack-freezer14:46
daemontoolbrilliant reldan! (thumbsup)14:47
daemontoolafter that change, everybody should rebase14:47
daemontoolSlashme:  ^^14:47
reldanhttps://review.openstack.org/#/c/253117/3/requirements.txt14:47
reldanBut I had to add this line14:47
reldanI hope it is ok14:47
reldanI can create separated commit - only with cnange in requirements.txt to accept it faster14:48
reldanShould I?14:48
daemontoolyes that's the solution14:48
daemontoolI think it is better14:48
daemontoolI did that14:48
daemontoolbut looks like my laptop14:48
daemontoolcannot connect to the gerrit ssh port :(14:48
reldanOk, let me create it )14:49
openstackgerritEldar Nugaev proposed openstack/freezer: Fixed astroid version to fix pylint  https://review.openstack.org/25353614:50
szaherdaemontool m3m0 can we merge elder patch asap ? I need to resend mine14:57
daemontoolszaher: +2ed14:59
jokke_please do not +A patches atm.15:00
jokke_you should get openstackannounce here as well ;)15:00
jokke_there's some issues infra is battling against and they sent announcement this morning to refrain approving patches before those are sorted15:01
szaherdaemontool thanks15:02
daemontooljokke_:  ok15:04
daemontoolty15:04
daemontooltoo late15:05
daemontool:(15:05
szaher https://review.openstack.org/253536 Merged Thanks Vanni15:07
szahersorry jokke_ I didn't see your message, I was asking the guys to merge it as we need it for another patch15:08
openstackgerritMerged openstack/freezer: Fixed astroid version to fix pylint  https://review.openstack.org/25353615:18
openstackgerritEldar Nugaev proposed openstack/freezer: Removing obsolete parameters from freezer args (swift related).  https://review.openstack.org/25310615:24
*** smekel has quit IRC15:43
*** smekel has joined #openstack-freezer15:46
*** dschroeder has joined #openstack-freezer16:13
*** reldan has quit IRC16:20
openstackgerritSaad Zaher proposed openstack/freezer: Switch freezer-scheduler to oslo.config and oslo.log switch freezer-scheduler to use oslo.config and switch from native python logging module to oslo.log This commit includes: - using oslo.config for parsing cli and config files options - using oslo.log i  https://review.openstack.org/25256816:25
*** sputnik13 has joined #openstack-freezer16:27
openstackgerritFausto Marzi proposed openstack/freezer: Fix typo: my scheduled backup 6 => schedule_backups 6  https://review.openstack.org/25171017:08
*** szaher has quit IRC17:58
*** daemontool has quit IRC18:23
*** sputnik13 has quit IRC18:24
*** sputnik13 has joined #openstack-freezer18:26
*** drwho has joined #openstack-freezer20:52
*** drwho has quit IRC20:57
*** openstack has joined #openstack-freezer23:39

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