*** dschroeder has quit IRC | 00:12 | |
*** drwho has joined #openstack-freezer | 00:13 | |
*** drwho has quit IRC | 00:17 | |
*** sputnik13 has joined #openstack-freezer | 00:31 | |
sputnik13 | helloooo | 00:31 |
---|---|---|
*** sputnik13 has quit IRC | 01:18 | |
*** hidekazu has joined #openstack-freezer | 01:42 | |
*** hidekazu has left #openstack-freezer | 01:43 | |
*** zhonghua has joined #openstack-freezer | 02:50 | |
*** zhonghua-lee has quit IRC | 02:54 | |
*** drwho has joined #openstack-freezer | 04:16 | |
*** drwho has quit IRC | 04:20 | |
openstackgerrit | daiki kato proposed openstack/freezer: Fix typo: my scheduled backup 6 => schedule_backups 6 https://review.openstack.org/251710 | 05:05 |
*** vannif has quit IRC | 05:42 | |
*** zhonghua has quit IRC | 06:19 | |
*** zhonghua-lee has joined #openstack-freezer | 06:20 | |
*** drwho has joined #openstack-freezer | 07:55 | |
*** drwho has quit IRC | 08:00 | |
*** reldan has joined #openstack-freezer | 09:58 | |
*** openstackgerrit has quit IRC | 10:32 | |
*** openstackgerrit has joined #openstack-freezer | 10:32 | |
*** szaher has joined #openstack-freezer | 10:53 | |
m3m0 | sputnik13 hello :) | 11:53 |
*** reldan has quit IRC | 11:53 | |
*** reldan has joined #openstack-freezer | 12:13 | |
*** daemontool has joined #openstack-freezer | 13:18 | |
daemontool | I've setup my environment, looking at http://logs.openstack.org/68/252568/1/check/gate-freezer-pylint/4d7d9a2/console.html | 13:24 |
daemontool | all, that issue is happening most probably because pbr is not installing pylint | 13:34 |
reldan | daemontool: 2015-12-02 19:42:19.410 | pylint installed: | 13:36 |
daemontool | reldan: are you sure that is installed from a previous install? | 13:37 |
daemontool | because when you execute | 13:37 |
daemontool | tox -r -v | 13:37 |
daemontool | it should install pylint in the virtualenv that tox generate | 13:37 |
daemontool | but it isn't | 13:37 |
daemontool | in my test | 13:37 |
daemontool | not even pbr is installed | 13:37 |
reldan | I don’t know I just watching the log | 13:37 |
daemontool | I'm just trying this on a fresh vm | 13:37 |
daemontool | ok | 13:38 |
daemontool | sure, I might be wrong | 13:38 |
daemontool | but in my case that can be one of the possible issues | 13:38 |
reldan | Problem importing module classes.py: cannot import name InferenceContext | 13:38 |
reldan | 2015-12-03 18:13:30.121 | Problem importing module classes.pyc: cannot import name InferenceContext | 13:38 |
reldan | I don’t know what it is | 13:38 |
*** drwho has joined #openstack-freezer | 13:40 | |
daemontool | classes.py | 13:40 |
daemontool | is a file part of pylint | 13:40 |
daemontool | afaik | 13:40 |
reldan | what is astroid? https://bitbucket.org/logilab/pylint/issues/660/python35-problem-importing-module | 13:41 |
daemontool | good question is a pylint dep | 13:42 |
daemontool | I think our issue is related | 13:43 |
daemontool | as we have since we changed our requirements aligned to Liberty? | 13:44 |
*** drwho has quit IRC | 13:44 | |
daemontool | Slashme: did you see this issue per chance also with Kilo? | 13:45 |
daemontool | reldan: pylint is installed on the tox env | 13:50 |
reldan | don’t understand what is wrong | 13:50 |
daemontool | we are 2 | 13:52 |
daemontool | :) | 13:52 |
daemontool | reldan: check here https://github.com/openstack/requirements/blob/stable/liberty/global-requirements.txt | 13:55 |
daemontool | check the last commit message | 13:55 |
daemontool | Jenkins Merge "Cap astroid to unbreak pylint." into stable/liberty | 13:55 |
reldan | astroid<1.4.0 # breaks pylint 1.4.4 | 13:55 |
reldan | but we have astroid 1.4.4 | 13:55 |
reldan | in our repo | 13:55 |
reldan | not in repo | 13:55 |
reldan | in log | 13:56 |
reldan | pylint installed: appdirs==1.4.0,APScheduler==3.0.4,astroid==1.4.1, | 13:56 |
reldan | ah no | 13:56 |
reldan | 1.4.1 we have | 13:56 |
reldan | but anyway it is higher than 1.4.0 | 13:56 |
daemontool | astroid<1.4.0 | 13:57 |
reldan | Yes, but in our log we are installing 1.4.1. So probably we just need to retry commits | 13:57 |
daemontool | that's the astroid from the global-requirements.txt in liberty | 13:57 |
daemontool | one sec | 13:58 |
daemontool | by changing that locally it works | 13:59 |
daemontool | let's see if it pass upstream ci | 13:59 |
reldan | Yes, let’s retry. Do you know how I can rebuild my pull requests? | 13:59 |
daemontool | rebuild? | 14:00 |
daemontool | clone master | 14:00 |
daemontool | then | 14:00 |
daemontool | git fetch https://review.openstack.org/openstack/freezer refs/changes/68/252568/1 && git checkout FETCH_HEAD | 14:00 |
daemontool | change whatever you need to change | 14:00 |
daemontool | git commit --amend | 14:00 |
daemontool | git review | 14:01 |
reldan | I see | 14:01 |
daemontool | git add would go before fgit commit | 14:01 |
reldan | Thank you, just I thought I can leave a comment with some magic word like “rebuild” and it will be retriggered | 14:01 |
daemontool | ah like rebase? | 14:02 |
daemontool | well nope | 14:02 |
daemontool | :) | 14:02 |
openstackgerrit | Eldar Nugaev proposed openstack/freezer: Removing old integration tests https://review.openstack.org/253117 | 14:06 |
reldan | Sitll doesn’t work and uses 1.4.1 http://logs.openstack.org/17/253117/2/check/gate-freezer-pylint/105ae9b/console.html | 14:16 |
*** daemontool has quit IRC | 14:20 | |
openstackgerrit | Eldar Nugaev proposed openstack/freezer: Removing old integration tests https://review.openstack.org/253117 | 14:26 |
*** daemontool has joined #openstack-freezer | 14:46 | |
daemontool | brilliant reldan! (thumbsup) | 14:47 |
daemontool | after that change, everybody should rebase | 14:47 |
daemontool | Slashme: ^^ | 14:47 |
reldan | https://review.openstack.org/#/c/253117/3/requirements.txt | 14:47 |
reldan | But I had to add this line | 14:47 |
reldan | I hope it is ok | 14:47 |
reldan | I can create separated commit - only with cnange in requirements.txt to accept it faster | 14:48 |
reldan | Should I? | 14:48 |
daemontool | yes that's the solution | 14:48 |
daemontool | I think it is better | 14:48 |
daemontool | I did that | 14:48 |
daemontool | but looks like my laptop | 14:48 |
daemontool | cannot connect to the gerrit ssh port :( | 14:48 |
reldan | Ok, let me create it ) | 14:49 |
openstackgerrit | Eldar Nugaev proposed openstack/freezer: Fixed astroid version to fix pylint https://review.openstack.org/253536 | 14:50 |
szaher | daemontool m3m0 can we merge elder patch asap ? I need to resend mine | 14:57 |
daemontool | szaher: +2ed | 14: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 sorted | 15:01 |
szaher | daemontool thanks | 15:02 |
daemontool | jokke_: ok | 15:04 |
daemontool | ty | 15:04 |
daemontool | too late | 15:05 |
daemontool | :( | 15:05 |
szaher | https://review.openstack.org/253536 Merged Thanks Vanni | 15:07 |
szaher | sorry jokke_ I didn't see your message, I was asking the guys to merge it as we need it for another patch | 15:08 |
openstackgerrit | Merged openstack/freezer: Fixed astroid version to fix pylint https://review.openstack.org/253536 | 15:18 |
openstackgerrit | Eldar Nugaev proposed openstack/freezer: Removing obsolete parameters from freezer args (swift related). https://review.openstack.org/253106 | 15:24 |
*** smekel has quit IRC | 15:43 | |
*** smekel has joined #openstack-freezer | 15:46 | |
*** dschroeder has joined #openstack-freezer | 16:13 | |
*** reldan has quit IRC | 16:20 | |
openstackgerrit | Saad 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/252568 | 16:25 |
*** sputnik13 has joined #openstack-freezer | 16:27 | |
openstackgerrit | Fausto Marzi proposed openstack/freezer: Fix typo: my scheduled backup 6 => schedule_backups 6 https://review.openstack.org/251710 | 17:08 |
*** szaher has quit IRC | 17:58 | |
*** daemontool has quit IRC | 18:23 | |
*** sputnik13 has quit IRC | 18:24 | |
*** sputnik13 has joined #openstack-freezer | 18:26 | |
*** drwho has joined #openstack-freezer | 20:52 | |
*** drwho has quit IRC | 20:57 | |
*** openstack has joined #openstack-freezer | 23:39 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!