| *** chandan_kumar has joined #openstack-climate | 05:20 | |
| Nikolay_St | guys | 05:31 |
|---|---|---|
| Nikolay_St | please take a look at https://review.openstack.org/#/c/58964/ it looks fine and I think we can merge it | 05:31 |
| *** ddyachkov has joined #openstack-climate | 05:34 | |
| *** bauzas has quit IRC | 06:08 | |
| *** ddyachkov has quit IRC | 06:33 | |
| *** DinaBelova_ is now known as DinaBelova | 06:39 | |
| *** saju_m has joined #openstack-climate | 06:48 | |
| *** DinaBelova is now known as DinaBelova_ | 07:00 | |
| *** ppetit has joined #openstack-climate | 08:01 | |
| *** DinaBelova_ is now known as DinaBelova | 08:10 | |
| *** bauzas has joined #openstack-climate | 08:34 | |
| Nikolay_St | one more time | 08:41 |
| Nikolay_St | we need one more +2 and approve at https://review.openstack.org/#/c/58964/ to test it is really simple | 08:41 |
| Nikolay_St | hi all | 08:41 |
| bauzas | I'm on it | 08:53 |
| DinaBelova | o/ | 08:54 |
| bauzas | +2/A | 08:55 |
| DinaBelova | Yuriy & I found issue with my trusts creation errors | 08:55 |
| bauzas | cool | 08:56 |
| DinaBelova | they are about commit merged to keystoneclient several days ago | 08:56 |
| DinaBelova | https://review.openstack.org/61247 | 08:56 |
| DinaBelova | this one | 08:56 |
| DinaBelova | if keystoneclient is using master (like with devstack) now it will fail cause of this change | 08:56 |
| DinaBelova | there is simple solution for it, I'm checking it now | 08:57 |
| *** chandankumar_ has joined #openstack-climate | 09:02 | |
| *** chandan_kumar has quit IRC | 09:03 | |
| openstackgerrit | A change was merged to stackforge/climate: Add pylint check for Climate project https://review.openstack.org/58964 | 09:06 |
| DinaBelova | that means that climate now is not working with keystoneclient 0.6.0 and later | 09:16 |
| DinaBelova | although I have now some more errors | 09:16 |
| *** DinaBelova is now known as DinaBelova_ | 09:33 | |
| *** DinaBelova_ is now known as DinaBelova | 09:33 | |
| *** saju_m has quit IRC | 09:37 | |
| *** saju_m has joined #openstack-climate | 09:38 | |
| *** saju_m has quit IRC | 09:46 | |
| *** saju_m has joined #openstack-climate | 10:05 | |
| swann | DinaBelova: indeed, I received an Unauthorized from Keystone | 10:27 |
| *** chandan_kumar has joined #openstack-climate | 10:30 | |
| swann | initial feeling .. seems the trust creation call is weird .. there is no token and no password sent | 10:30 |
| DinaBelova | swann, everything is ok with that call - there is password and user passed | 10:31 |
| DinaBelova | although, that might be fixed with onw line, it's ok | 10:31 |
| DinaBelova | one* | 10:31 |
| DinaBelova | problem is much more | 10:32 |
| DinaBelova | I'll check oslo.messaging releases now | 10:32 |
| DinaBelova | cause it was commit to it too some days ago | 10:33 |
| DinaBelova | and that makes Climate fail | 10:34 |
| *** chandankumar_ has quit IRC | 10:34 | |
| DinaBelova | owowo | 10:34 |
| DinaBelova | bad news | 10:34 |
| DinaBelova | that change is already in 1.3.0.a7 oslo.messaging release | 10:34 |
| DinaBelova | soooo | 10:34 |
| DinaBelova | it's not working now too | 10:34 |
| DinaBelova | ok, two critical bugs in one day :D | 10:37 |
| *** DinaBelova is now known as DinaBelova_ | 10:37 | |
| *** ppetit has quit IRC | 11:15 | |
| *** DinaBelova_ is now known as DinaBelova | 12:10 | |
| *** pafuent has joined #openstack-climate | 12:25 | |
| *** casanch1 has joined #openstack-climate | 12:42 | |
| *** casanch1 has quit IRC | 12:49 | |
| *** casanch1 has joined #openstack-climate | 12:50 | |
| *** casanch1 has joined #openstack-climate | 12:51 | |
| *** casanch1 has quit IRC | 12:54 | |
| *** casanch1 has joined #openstack-climate | 12:55 | |
| openstackgerrit | Pablo Andres Fuente proposed a change to stackforge/climate: Initial devstack support implemented https://review.openstack.org/60465 | 13:04 |
| *** casanch1 has joined #openstack-climate | 13:08 | |
| openstackgerrit | Dina Belova proposed a change to stackforge/climate: Fix issue with new oslo.messaging https://review.openstack.org/74375 | 13:26 |
| openstackgerrit | Dina Belova proposed a change to stackforge/climate: Fix issue with new python-keystoneclient https://review.openstack.org/74376 | 13:31 |
| pafuent | bauzas: Hi | 13:34 |
| DinaBelova | pafuent, o/ | 13:36 |
| pafuent | DinaBelova: Hi | 13:36 |
| DinaBelova | do you have nice morning now? :) | 13:36 |
| pafuent | DinaBelova: Yes, and you? | 13:37 |
| DinaBelova | wet and finally sunny evening :) | 13:37 |
| DinaBelova | pafuent, I've tested several times your devstack impl | 13:37 |
| DinaBelova | it looks really good now | 13:38 |
| DinaBelova | although, I could find only oe issue | 13:38 |
| DinaBelova | one* | 13:38 |
| pafuent | DinaBelova: Which one? | 13:38 |
| pafuent | pafuent: BTW, Thanks for test it | 13:38 |
| DinaBelova | if user says 'unstack', climate-m is still running | 13:38 |
| DinaBelova | all other processes are killed as usually | 13:38 |
| DinaBelova | but climate-manager - no | 13:38 |
| pafuent | DinaBelova: There is a specific way to reproduce that? | 13:39 |
| DinaBelova | pafuent, yes :) | 13:39 |
| DinaBelova | run devstack | 13:39 |
| DinaBelova | say ./unstack.sh | 13:39 |
| pafuent | DinaBelova: I did stack/unstack and always is killed | 13:39 |
| DinaBelova | hm | 13:39 |
| DinaBelova | may you say ps aux | grep climate-manager | 13:39 |
| pafuent | DinaBelova: Yes | 13:39 |
| DinaBelova | if there are more than one now - it's not really good | 13:39 |
| pafuent | DinaBelova: So, you didn't do anything between stack/unstack? | 13:40 |
| DinaBelova | no | 13:40 |
| pafuent | DinaBelova: I'll try again, but I did that several times and the climate-manager was killed every time | 13:41 |
| DinaBelova | hm, really strange.... | 13:41 |
| * pafuent is stacking | 13:42 | |
| DinaBelova | pafuent, look at my cli logs | 13:42 |
| DinaBelova | http://paste.openstack.org/show/f8txF78VmJNt73kGQGDB/ | 13:42 |
| DinaBelova | ps aux when devstack is running | 13:43 |
| DinaBelova | unstack | 13:43 |
| DinaBelova | and then one more grep | 13:43 |
| DinaBelova | with climate-manager still running | 13:43 |
| DinaBelova | I guess that's quite strange, but I had this issue from the time we tried to implement devstack integration firstly | 13:44 |
| *** chmartinez has joined #openstack-climate | 13:45 | |
| pafuent | DinaBelova: I know this is a stupid question, but which OS and screen version are you using? Maybe is related to screen. | 13:46 |
| DinaBelova | ubuntu server 12.04 | 13:46 |
| DinaBelova | and as for screen | 13:46 |
| DinaBelova | one moment | 13:46 |
| DinaBelova | Screen version 4.00.03jw4 | 13:46 |
| DinaBelova | hm, that was installed by default I guess | 13:47 |
| DinaBelova | cause that's new ubuntu vm with only update and upgrade ran on it | 13:47 |
| pafuent | DinaBelova: Almost the same environment. I use Ubuntu 12.04 (no server) | 13:47 |
| DinaBelova | :( | 13:48 |
| * pafuent is ps | 13:48 | |
| * pafuent is ps aux | 13:48 | |
| DinaBelova | :D | 13:48 |
| * pafuent can't believe that hit the issue | 13:49 | |
| pafuent | DinaBelova: I can see the manager after the unstack | 13:49 |
| DinaBelova | wow, at least i'm not alone :) | 13:49 |
| pafuent | DinaBelova: YEs | 13:49 |
| bauzas | pafuent: gi | 13:49 |
| bauzas | pafuent: hi | 13:49 |
| bauzas | pafuent: sorry, was around for other concers | 13:50 |
| bauzas | concerns | 13:50 |
| DinaBelova | bauzas, o/ | 13:50 |
| bauzas | I'm pretty busy these days | 13:50 |
| pafuent | DinaBelova: I'll do it again but I'll do a climate lease-list to see what happens | 13:50 |
| DinaBelova | pafuent, issues with multiple managers might not be seen really (that's question of probability here) | 13:50 |
| *** saju_m has quit IRC | 13:50 | |
| pafuent | bauzas: NP, Hi | 13:50 |
| pafuent | bauzas: Are you working on schema-data-migration-with-alembic? | 13:51 |
| * pafuent is stacking againg | 13:52 | |
| bauzas | pafuent: not yet | 13:57 |
| bauzas | pafuent: it's due to be done right after the Pecan move | 13:58 |
| pafuent | bauzas: Ok. I'm asking because I was playing with Alembic and I would like to implement it, if don't mind you. | 13:59 |
| bauzas | well, I think the priority is maybe on other things | 14:00 |
| bauzas | the data scheme is quite stable now | 14:00 |
| SergeyLukjanov | btw, folks, you can find several examples of writing tests for migrations in savanna | 14:01 |
| pafuent | SergeyLukjanov: Thanks :) | 14:01 |
| pafuent | bauzas: I was interested in that because I'll be in paternity leave (if that really exist) two weeks starting on Friday, and seems alembic seems to be not so hard to get it working. And, as you say, the priority isn't high, so I can start working on try to get it done before my leave. | 14:07 |
| bauzas | well, I don't think it's reasonable to plan to have it delivered by Friday | 14:08 |
| bauzas | think about review time, and appropriate new patchsets | 14:08 |
| pafuent | bauzas: Yes I know. I really want to be helpful during this 3 days, and start working on something with a higher priority will be risky | 14:10 |
| bauzas | that one is much more a quickwin to me : https://bugs.launchpad.net/climate/+bug/1237293 | 14:10 |
| pafuent | bauzas: Ok, I'll check that. | 14:11 |
| bauzas | that's really a pain having DB exceptions not raised as ClimateException subclasses | 14:11 |
| bauzas | there is also one bug which is really bad : | 14:12 |
| bauzas | well, I can't find the bug number | 14:12 |
| bauzas | lemme find the wrong line | 14:13 |
| pafuent | bauzas: Ok | 14:13 |
| bauzas | https://github.com/stackforge/climate/blob/master/climate/manager/service.py#L206 | 14:13 |
| bauzas | that one | 14:13 |
| bauzas | that's related to the bug I mentioned earlier | 14:13 |
| bauzas | we smoothly hide the exception to the user, without informing it something wrong happened | 14:14 |
| bauzas | that's horrible | 14:14 |
| bauzas | exceptions should not have something like except myexc: log.something() | 14:14 |
| bauzas | that's obfuscating the error | 14:15 |
| pafuent | bauzas: Yes. Will be hard for the user to know what happened | 14:15 |
| bauzas | in other terms, there are so much things to do with exceptions in Climate | 14:15 |
| bauzas | we're not really verifying the results before passing them to the CLI | 14:15 |
| bauzas | we're on Tuesday | 14:16 |
| bauzas | that means you should be able to propose something by tomorrow, and we would have 2 days for reviewing/patching | 14:16 |
| bauzas | that's comfortabler | 14:16 |
| bauzas | and that's really improving our error management | 14:17 |
| pafuent | bauzas: Ok. I'll try to do something about it. | 14:18 |
| bauzas | cool thanks | 14:18 |
| pafuent | DinaBelova: Regarding the manager, it's quite strange | 14:26 |
| DinaBelova | pafuent, ? Did you find reason? | 14:26 |
| pafuent | DinaBelova: One time the manager is killed, and then not | 14:26 |
| DinaBelova | really strange, yes... | 14:27 |
| DinaBelova | maybe it's good idea to grep this service after its finishing and kill it if it's still alive | 14:27 |
| DinaBelova | 'casue I dunno exact reason for such rehaviour | 14:28 |
| pafuent | DinaBelova: Nope, I think that was related to not use the manager (which is stupid, but I learned that is better to test the stupid early), but not, happens randomly | 14:28 |
| DinaBelova | :( | 14:28 |
| pafuent | DinaBelova: I'll add the kill in a new patch | 14:29 |
| DinaBelova | thanks | 14:29 |
| *** chmartinez_ has joined #openstack-climate | 14:55 | |
| *** chmartinez has quit IRC | 14:57 | |
| pafuent | DinaBelova: pgrep is installed by default in Linux distros? | 15:09 |
| pafuent | DinaBelova: Or should I use grep and awk to get the pid of the manager | 15:10 |
| pafuent | ? | 15:10 |
| DinaBelova | pafuent, I suppose pgrep is default thing | 15:10 |
| pafuent | DinaBelova: Ok, I'll use it | 15:11 |
| *** chmartinez_ has quit IRC | 15:21 | |
| bauzas | pafuent: please use awk for this :) | 15:31 |
| bauzas | pgrep is not common | 15:31 |
| openstackgerrit | Pablo Andres Fuente proposed a change to stackforge/climate: Initial devstack support implemented https://review.openstack.org/60465 | 15:33 |
| pafuent | bauzas: Ok. I did't read your message. I'll upload a new patch. | 15:34 |
| openstackgerrit | Christian Martinez proposed a change to stackforge/climate: Adding tests for sqlalchemy class https://review.openstack.org/72943 | 15:36 |
| openstackgerrit | Pablo Andres Fuente proposed a change to stackforge/climate: Initial devstack support implemented https://review.openstack.org/60465 | 15:43 |
| *** chandan_kumar has quit IRC | 15:43 | |
| *** pafuent has quit IRC | 15:49 | |
| *** DinaBelova is now known as DinaBelova_ | 16:03 | |
| *** bauzas has quit IRC | 16:14 | |
| *** pafuent has joined #openstack-climate | 16:41 | |
| *** DinaBelova_ is now known as DinaBelova | 16:48 | |
| *** bauzas has joined #openstack-climate | 17:40 | |
| pafuent | bauzas: The fix for the line in service.py, should be fixed in a separate branch, right? | 17:52 |
| pafuent | bauzas: In other words, one different to the one to fix https://bugs.launchpad.net/climate/+bug/1237293 | 17:52 |
| *** chmartinez has joined #openstack-climate | 17:58 | |
| *** bauzas has quit IRC | 18:29 | |
| *** pafuent has quit IRC | 19:20 | |
| *** pafuent has joined #openstack-climate | 19:21 | |
| openstackgerrit | Cristian A Sanchez proposed a change to stackforge/climate: Update openstack.common with latest oslo-incubator https://review.openstack.org/73363 | 19:39 |
| *** bauzas has joined #openstack-climate | 19:57 | |
| *** chmartinez has quit IRC | 20:03 | |
| *** pafuent has quit IRC | 20:10 | |
| *** pafuent has joined #openstack-climate | 20:11 | |
| *** pafuent has quit IRC | 20:17 | |
| *** DinaBelova is now known as DinaBelova_ | 21:00 | |
| *** casanch1 has quit IRC | 21:14 | |
| *** bauzas has quit IRC | 23:00 | |
| *** openstackgerrit has quit IRC | 23:16 | |
| *** openstackgerrit has joined #openstack-climate | 23:18 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!