| *** dshulyak_ has joined #openstack-solar | 08:00 | |
| *** salmon_ has joined #openstack-solar | 08:33 | |
| openstackgerrit | Lukasz Oles proposed openstack/solar: Move all text from README to documentation https://review.openstack.org/270062 | 08:58 |
|---|---|---|
| dshulyak_ | hello | 09:14 |
| salmon_ | hi dshulyak_ | 09:15 |
| pigmej | hello guys :) | 09:26 |
| pigmej | salmon_: -1 for pbr again :D | 09:27 |
| openstackgerrit | Lukasz Oles proposed openstack/solar: Move all text from README to documentation https://review.openstack.org/270062 | 09:27 |
| salmon_ | pigmej: pbr? where? | 09:27 |
| pigmej | because it requires README.md | 09:28 |
| pigmej | ;p | 09:28 |
| salmon_ | I reanamed it ti .rst | 09:28 |
| pigmej | 2016-01-20 09:09:13.342 | IOError: [Errno 2] No such file or directory: 'README.md' | 09:28 |
| pigmej | ;P | 09:28 |
| salmon_ | yeahm I changed setup.cfg now | 09:30 |
| pigmej | :) | 09:33 |
| openstackgerrit | Bogdan Dobrelya proposed openstack/solar: Fix 80 chars wraps for docs https://review.openstack.org/270100 | 10:11 |
| pigmej | dshulyak_: the problem is that afair we need to enforce all tables, all metadata etc BEFORE creating multiple clients | 10:12 |
| pigmej | I had similar fuckups with it but afair that was a solution (at least worked for me) | 10:13 |
| pigmej | BUT it's probably perfectly fine to use file based sqlite for tests :) | 10:13 |
| salmon_ | I will test this one https://review.openstack.org/#/c/269018/ | 10:18 |
| pigmej | cool :) | 10:20 |
| pigmej | salmon, ideally you should test 2 in the same tim | 10:20 |
| pigmej | time | 10:20 |
| salmon_ | 2? | 10:20 |
| salmon_ | how? | 10:20 |
| pigmej | https://review.openstack.org/#/c/269238/ + https://review.openstack.org/#/c/269018/ | 10:20 |
| salmon_ | ah | 10:20 |
| pigmej | lock fixes just part of problems, the other part is fixed by counter | 10:21 |
| salmon_ | ok | 10:21 |
| salmon_ | and I need to change how celery starts? | 10:21 |
| pigmej | you can test them separately BUT you will need to know what errors are caused by what :) | 10:21 |
| pigmej | revert https://review.openstack.org/#/c/267769/1/utils/solar-celery | 10:21 |
| salmon_ | ok | 10:22 |
| pigmej | salmon_ + bogdando you should agree which patch will be merged first :) | 10:24 |
| bogdando | pigmej, I lost the context | 10:24 |
| salmon_ | pigmej: bogdano patch was first | 10:25 |
| pigmej | https://review.openstack.org/#/c/270100/ vs https://review.openstack.org/#/c/269673/ | 10:26 |
| pigmej | ? | 10:26 |
| salmon_ | yup | 10:26 |
| bogdando | IMHO the https://review.openstack.org/#/c/270100/ | 10:32 |
| salmon_ | yup | 10:32 |
| pigmej | bogdando: thanks for patch :) | 10:34 |
| bogdando | np, let's W+1 | 10:34 |
| openstackgerrit | Merged openstack/solar: Fix 80 chars wraps for docs https://review.openstack.org/270100 | 10:39 |
| pigmej | salmon_: how is testing going ? | 10:52 |
| pigmej | dshulyak_: you wanted to understand / check why lock is complicated now, isn't it? | 10:53 |
| salmon_ | pigmej: it's going :) | 10:53 |
| pigmej | k | 10:54 |
| salmon_ | riak example worked | 10:55 |
| pigmej | make '+' then :) | 10:58 |
| salmon_ | I will check all examples ;) | 11:00 |
| dshulyak_ | pigmej: yes, i will just dig into +- thing after i will fix sqlite | 11:09 |
| pigmej | k | 11:10 |
| openstackgerrit | Bogdan Dobrelya proposed openstack/solar: Add tox checkniceness,checksyntax for docs https://review.openstack.org/269559 | 11:35 |
| bogdando | pigmej, salmon_ all works now | 11:35 |
| bogdando | no more unrequired deps installing | 11:35 |
| bogdando | and main tests ain't affected at all | 11:36 |
| pigmej | bogdando: I still don't get why `envlist = pep8,py27,docs,checkniceness,checksyntax` | 11:37 |
| bogdando | that is for default "tox" invocation | 11:37 |
| bogdando | we want it to do 1) generic tests, 2) build docs, 3) check style for docs | 11:37 |
| bogdando | what looks most reasonable, at least for me... | 11:38 |
| bogdando | as a dev you may use only -e py27,pep8 | 11:38 |
| bogdando | and nothing will change for you | 11:39 |
| pigmej | yaeh but the question is do we really need "doc test" on default tox invocation? | 11:39 |
| bogdando | why not? | 11:39 |
| bogdando | tox shall build docs as well and check them. I can't get your point, why not? | 11:40 |
| pigmej | Maybe I'm too python oriented or too dev oriented | 11:41 |
| pigmej | but I don't see a point of building docs on 'tox' invocation | 11:41 |
| pigmej | but it's my personal opinion :) | 11:41 |
| pigmej | and feeling. | 11:41 |
| bogdando | devs impact: replace "tox" to "tox -e py27,pep8". users impact: instead tox -l, and consequent tox -e docs && tox -e checkniceness && tox -e checkstyle .. just tox! | 11:41 |
| bogdando | just let's think about users and tech writers as well ;) | 11:42 |
| pigmej | well | 11:42 |
| pigmej | if one *just* plays with docs, he for sure don't want our full test suite | 11:42 |
| bogdando | ok, I can remove epy27,pep8 from envlist :D | 11:42 |
| *** openstackgerrit has quit IRC | 11:43 | |
| bogdando | but I believe that is the most expected defautls for just "tox" to do all it can | 11:43 |
| pigmej | I think we should keep there py27,pep8 OR docs related things, not mixing them together :) | 11:43 |
| bogdando | I don't agree, but if you insist... | 11:44 |
| -openstackstatus- NOTICE: review.openstack.org is being restarted to apply patches | 11:44 | |
| *** ChanServ changes topic to "review.openstack.org is being restarted to apply patches" | 11:44 | |
| pigmej | bogdando: well, dont' change it becaues I said it :) | 11:44 |
| *** openstackgerrit has joined #openstack-solar | 11:44 | |
| pigmej | let's wait for other opininos :) | 11:44 |
| salmon_ | pigmej: Tested with all our examples. All worked | 11:47 |
| pigmej | salmon_: perfect, not that I expected something other ;P | 11:47 |
| salmon_ | :D | 11:47 |
| openstackgerrit | Lukasz Oles proposed openstack/solar: Increase riak timeout - time depends on download speed https://review.openstack.org/270147 | 11:49 |
| pigmej | salmon_: is there any other way to not base on this timeout but some dynamic polling? | 11:52 |
| salmon_ | pigmej: we coud change it to "timeout for time since last change" | 11:53 |
| salmon_ | but current timeout is quite ok and on CI they have faster net :) | 11:54 |
| pigmej | k | 11:54 |
| pigmej | faster than mine too? | 11:54 |
| *** ChanServ changes topic to "#openstack-solar" | 11:55 | |
| -openstackstatus- NOTICE: Restart done, review.openstack.org is available | 11:55 | |
| salmon_ | pigmej: no idea | 11:55 |
| pigmej | :P | 11:55 |
| openstackgerrit | Lukasz Oles proposed openstack/solar: Update Wordpress tutorial https://review.openstack.org/269673 | 11:59 |
| salmon_ | bogdando: what is the link to the solar dashboard? | 12:36 |
| salmon_ | on gerrit | 12:36 |
| bogdando | https://review.openstack.org/#/dashboard/?foreach=%28project%3Aopenstack%2Fsolar%29+status%3Aopen+NOT+label%3AWorkflow%3C%3D%2D1+NOT+label%3ACode%2DReview%3C%3D%2D2&title=Solar+Review+Inbox&My+Patches+Requiring+Attention=owner%3Aself+%28label%3AVerified%2D1+OR+label%3ACode%2DReview%2D1+OR+label%3AWorkflow%3C%3D%2D1%29&Needs+Approval=NOT+label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D%2D1+NOT | 12:37 |
| bogdando | +label%3AVerified%3C%3D%2D1+NOT+owner%3Aself+label%3ACode%2DReview%3E%3D2+NOT+label%3ACode%2DReview%3C%3D%2D1&Ready+for+Core+Reviewers=NOT+label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D%2D1+NOT+label%3AVerified%3C%3D%2D1+label%3AVerified%3E%3D1+label%3ACode%2DReview%3E%3D1+NOT+label%3ACode%2DReview%3E%3D2+NOT+label%3ACode%2DReview%3C%3D%2D1&5+Days+Without+Feedback=NOT+label%3AWorkflow%3E%3D | 12:37 |
| bogdando | 1+NOT+label%3AWorkflow%3C%3D%2D1+NOT+label%3AVerified%3C%3D%2D1+NOT+label%3ACode%2DReview%3C%3D2+age%3A5d&No+Feedback=NOT+label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D%2D1+NOT+label%3AVerified%3C%3D%2D1+NOT+label%3ACode%2DReview%3C%3D2+NOT+age%3A5d+limit%3A50&Disagreement=NOT+label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D%2D1+NOT+label%3AVerified%3C%3D%2D1+label%3ACode%2DReview%3C%3D%2 | 12:37 |
| bogdando | D1+label%3ACode%2DReview%3E%3D1&Negative+Feedback=NOT+label%3AWorkflow%3E%3D1+NOT+label%3AWorkflow%3C%3D%2D1+NOT+label%3AVerified%3C%3D%2D1+label%3ACode%2DReview%2D1+NOT+label%3ACode%2DReview%3C%3D%2D2+NOT+label%3ACode%2DReview%3E%3D1+limit%3A20 | 12:37 |
| salmon_ | what a long url :D | 12:38 |
| salmon_ | thx | 12:38 |
| openstackgerrit | Merged openstack/solar: Update Wordpress tutorial https://review.openstack.org/269673 | 12:40 |
| dshulyak_ | can be merged https://review.openstack.org/#/c/269238/ ? | 13:03 |
| pigmej | dshulyak_: I think so, it just misses workflow flag | 13:04 |
| openstackgerrit | Dmitry Shulyak proposed openstack/solar: Use isolated filebased sqlite for each test process https://review.openstack.org/270181 | 13:08 |
| openstackgerrit | Dmitry Shulyak proposed openstack/solar: Use isolated filebased sqlite for each test process https://review.openstack.org/270181 | 13:08 |
| pigmej | dshulyak_: why do you make gevent non optional ? | 13:10 |
| pigmej | I dislike that fact. | 13:10 |
| pigmej | also that --clean sounds to me like bad idea. | 13:11 |
| dshulyak_ | thats for ci environments | 13:11 |
| dshulyak_ | why is it bad? | 13:11 |
| pigmej | because you don't check if it's sqlite | 13:11 |
| pigmej | and you will remove any file that matches 'database' name | 13:11 |
| dshulyak_ | i check if thats file | 13:12 |
| dshulyak_ | ah | 13:12 |
| pigmej | I can have 'riak' filename or any other that matches databasename | 13:12 |
| pigmej | :) | 13:12 |
| pigmej | dshulyak_: what about gevent ? why you moved it to dblayer back ? | 13:13 |
| openstackgerrit | Merged openstack/solar: Fixing concurrency problems in history counter https://review.openstack.org/269238 | 13:13 |
| dshulyak_ | no | 13:13 |
| dshulyak_ | i moved only dblayer part | 13:13 |
| dshulyak_ | i can wrap it in similar try/except | 13:13 |
| dshulyak_ | and i moved it because it imports too much | 13:14 |
| pigmej | but why you moved it there ? | 13:14 |
| dshulyak_ | because it is dblayer patches? | 13:14 |
| pigmej | it also patches utils | 13:15 |
| dshulyak_ | i need to remove them from solar/__init__.py | 13:15 |
| dshulyak_ | and dblayer/__init__ seems to be the right place | 13:15 |
| pigmej | we had it in dblayer before, and it turned out that it's too late to have it there | 13:15 |
| dshulyak_ | no | 13:15 |
| dshulyak_ | i remember that problem | 13:15 |
| dshulyak_ | it wasnt related even :) | 13:15 |
| pigmej | u sure ? | 13:16 |
| dshulyak_ | you jsut moved it :) | 13:16 |
| pigmej | well, but then you need to move utils patch from there | 13:16 |
| pigmej | because it's possible that utils is not patched correctly | 13:16 |
| dshulyak_ | hm, what they are doing in dblayer :) | 13:17 |
| pigmej | that's another question :P | 13:18 |
| dshulyak_ | should i patch them in utils? | 13:18 |
| pigmej | they should be patched together with the rest of our patches | 13:18 |
| pigmej | because they depend | 13:19 |
| pigmej | but if your intention was to just move that paches then ok, BUT it has to be conditional | 13:20 |
| dshulyak_ | this 2 shouldnt depend on dbalyer | 13:21 |
| dshulyak_ | _patch(utils, 'get_local', get_local) | 13:21 |
| dshulyak_ | _patch(utils, 'get_current_ident', get_current_ident) | 13:21 |
| pigmej | but they were needed only for dblyaer | 13:22 |
| pigmej | but yeah | 13:22 |
| pigmej | they shouldn't now probably | 13:22 |
| dshulyak_ | ok, i will leave them in place for now, they wont be used wo importing db first anyway | 13:24 |
| pigmej | kinda | 13:24 |
| pigmej | :) | 13:24 |
| openstackgerrit | Dmitry Shulyak proposed openstack/solar: Use isolated filebased sqlite for each test process https://review.openstack.org/270181 | 13:24 |
| pigmej | https://github.com/openstack/solar/blob/master/solar/dblayer/gevent_patches.py#L29-L35 | 13:25 |
| pigmej | dshulyak_: how this format will work when someone will test on riak ? | 13:25 |
| dshulyak_ | format wont have any effect if there is no {PID} in url | 13:28 |
| pigmej | let's assume that noone will abuse this :D | 13:28 |
| pigmej | dshulyak_: and with this everything works for you ? | 13:29 |
| dshulyak_ | with file based sqlite tests for new worker are fine, i will rebase on top of this patch in a moment | 13:30 |
| pigmej | cool :) | 13:30 |
| pigmej | btw can you test this locking, because it's now blocker for release | 13:30 |
| openstackgerrit | Dmitry Shulyak proposed openstack/solar: Zerorpc worker for orchestration modules https://review.openstack.org/269166 | 13:31 |
| dshulyak_ | i will review in 20-30 mins | 13:41 |
| pigmej | perfect :) | 13:44 |
| *** bogdando has quit IRC | 13:56 | |
| *** bogdando has joined #openstack-solar | 14:02 | |
| pigmej | ok guys, I think it's nonsense to "improve" errors in repository | 14:04 |
| pigmej | `OSError: [Errno 2] No such file or directory: '/var/lib/solar/repositories/resources/ratarstar/2.0.0' | 14:05 |
| pigmej | ` this is better than any custom made error, isn't it? | 14:05 |
| pigmej | it's very easy to use it as API then (standard python errors), I can wrap the errors in some RepositoryException BUT... I'm not sure if it's correct idea | 14:05 |
| pigmej | salmon_: dshulyak_ opinions ? | 14:05 |
| pigmej | It will require a lot of work, with a lot of errno.E* things, and we will probably gain nothing except more pretty string error | 14:06 |
| salmon_ | pigmej: depends when do you see this error. On what action? | 14:06 |
| pigmej | `solar repository` cli actions | 14:06 |
| pigmej | I could hide traceback in CLI, and print str in red | 14:07 |
| pigmej | this one | 14:09 |
| pigmej | https://bpaste.net/show/3e695d20b692 | 14:09 |
| pigmej | is also pretty clear | 14:10 |
| salmon_ | good ux means not exceptions :) | 14:10 |
| salmon_ | *no | 14:10 |
| pigmej | salmon_: this one is python API usage => exception is ok | 14:10 |
| salmon_ | I'm talking about cli | 14:11 |
| pigmej | https://bpaste.net/show/1c4b492f2d52 | 14:11 |
| pigmej | what would you like to see in this case? | 14:11 |
| salmon_ | https://bpaste.net/show/4774387e1191 | 14:13 |
| pigmej | I'm pretty sure that we should catch errors like that on CLI level | 14:13 |
| salmon_ | yup | 14:13 |
| pigmej | so I'm not touching errors in repository then | 14:13 |
| pigmej | except one place where I need to consolidate things | 14:13 |
| openstackgerrit | Jedrzej Nowak proposed openstack/solar: Unified exception in repo create https://review.openstack.org/270208 | 14:15 |
| salmon_ | for sure it should be only on CLI level | 14:15 |
| dshulyak_ | pigmej: what is the handling for KeyError in who_is_locking and change_locking_start doing? | 14:26 |
| pigmej | dblayer :D | 14:27 |
| dshulyak_ | i mean when it is possible | 14:27 |
| pigmej | wait I need to look into code :) | 14:27 |
| pigmej | you mean that identity keyerror? | 14:28 |
| pigmej | it's needed because ensemble uses your 'old' method | 14:28 |
| pigmej | where you set `identity` | 14:28 |
| pigmej | so keyerror is possible when no identity is set | 14:30 |
| pigmej | are you talking about that KeyERror or other ? | 14:31 |
| dshulyak_ | yes, i see | 14:31 |
| pigmej | It's kinda fleaky api | 14:32 |
| pigmej | we can simplify it later (we can use riak types on riak, and this implementation on sqlite), and get rid of this "identity" completely, or make special class for ensemble only etc | 14:34 |
| dshulyak_ | then i think i understood how it works, even if 2 concurrent threads will change lockers with 1 - they will agree on what thread is erroneous because of sorting in sum_all | 14:36 |
| pigmej | this too, | 14:37 |
| pigmej | but with n_val=1 first will win, second will fail no matter of 'sorting' order. This implementation prevents storing "old" value after storing new | 14:37 |
| pigmej | so our "delete" case is no-go now :) | 14:38 |
| pigmej | https://bpaste.net/show/233e13105f27 | 15:06 |
| pigmej | salmon_: ^ | 15:06 |
| salmon_ | :) | 15:10 |
| dshulyak_ | i think we should just remove traceback somewhere in click hooks | 15:12 |
| pigmej | dshulyak_: yeah working on it :) | 15:13 |
| pigmej | (see --debug) :) | 15:13 |
| dshulyak_ | i missed discussion :) | 15:13 |
| pigmej | though I created separate group | 15:13 |
| pigmej | btw salmon_ , dshulyak_ one of you need set workflow flag :) | 15:20 |
| pigmej | https://review.openstack.org/#/c/269018/ | 15:20 |
| openstackgerrit | Jedrzej Nowak proposed openstack/solar: Added EGroup click group which can handle errors https://review.openstack.org/270247 | 15:20 |
| dshulyak_ | is it possible to set debug in main ? | 15:23 |
| pigmej | no but it could be possible | 15:24 |
| pigmej | genrally click is stupid | 15:24 |
| pigmej | because you will need to call it | 15:24 |
| pigmej | solar repository --debug import resources -l | 15:24 |
| pigmej | not solar repository import resources -l --debug | 15:24 |
| pigmej | so with main it will be even worse | 15:25 |
| pigmej | solar --debug repository import resources -l | 15:25 |
| dshulyak_ | i think it is better to set solar —debug | 15:26 |
| salmon_ | +1 | 15:26 |
| pigmej | k | 15:27 |
| openstackgerrit | Merged openstack/solar: CRDTish lock to avoid concurrent update/delete https://review.openstack.org/269018 | 15:30 |
| openstackgerrit | Jedrzej Nowak proposed openstack/solar: Added EGroup click group which can handle errors https://review.openstack.org/270247 | 15:31 |
| pigmej | dshulyak_: salmon_ done | 15:31 |
| openstackgerrit | Dmitry Shulyak proposed openstack/solar: Use isolated filebased sqlite for each test process https://review.openstack.org/270181 | 16:18 |
| openstackgerrit | Dmitry Shulyak proposed openstack/solar: Zerorpc worker for orchestration modules https://review.openstack.org/269166 | 16:18 |
| openstackgerrit | Merged openstack/solar: Increase riak timeout - time depends on download speed https://review.openstack.org/270147 | 16:28 |
| openstackgerrit | Bogdan Dobrelya proposed openstack/solar: Add tox checkniceness,checksyntax for docs https://review.openstack.org/269559 | 16:38 |
| pigmej | dshulyak_: can you remind me how to test new workers? | 17:18 |
| dshulyak_ | solar-worker | 17:19 |
| dshulyak_ | there is not init script right now, i’m just running it in foreground | 17:19 |
| pigmej | ah right :) | 17:19 |
| pigmej | btw salmon_ dshulyak_ I responded: https://review.openstack.org/#/c/269653/3 :) | 17:25 |
| pigmej | hmmmmmmmmmm, I'm starting really to dislike git, or I have some stupid defaults, I will update this PR with things that should be there. | 17:38 |
| pigmej | (I will fix that stderr to dshulyak_ ) | 17:38 |
| openstackgerrit | Jedrzej Nowak proposed openstack/solar: Unify returned transport results https://review.openstack.org/269653 | 17:41 |
| openstackgerrit | Jedrzej Nowak proposed openstack/solar: Unify returned transport results https://review.openstack.org/269653 | 17:52 |
| pigmej | dshulyak_: good job with that new worker :) | 17:59 |
| pigmej | dshulyak_: but I have one question, could we later add some notification state about locking ? (in addition to this sleep) ? | 18:00 |
| pigmej | or at least we should randomize sleep, | 18:00 |
| dshulyak_ | what kind of notification? | 18:00 |
| pigmej | "someone released lock" etc | 18:01 |
| pigmej | dshulyak_: hmm, I'm having a lot of siblings errors though | 18:01 |
| dshulyak_ | i think it is not rebased after recent merges | 18:02 |
| pigmej | ah | 18:02 |
| dshulyak_ | or maybe it is :) | 18:02 |
| pigmej | then bad | 18:02 |
| pigmej | https://bpaste.net/show/3d7b0a7ca6a4 | 18:02 |
| dshulyak_ | yeah, i think it is rebased | 18:02 |
| pigmej | maybe you somehow abuse cache ? | 18:03 |
| pigmej | because I don't see errors on first execution | 18:03 |
| pigmej | but it kinda /always/ fails on second | 18:03 |
| pigmej | yeah, solar o restart => always error, but when I will restart worker first, and then solar o restart then everything is fine | 18:04 |
| dshulyak_ | yeah, atleast it is reproducible :) | 18:06 |
| dshulyak_ | should be east | 18:07 |
| dshulyak_ | easy | 18:07 |
| dshulyak_ | ah crap | 18:07 |
| dshulyak_ | i broke session_start/session_end | 18:07 |
| dshulyak_ | in my hooks mechanism i always pass context everywhere | 18:08 |
| dshulyak_ | and session_start/session_end doesnt accept it | 18:08 |
| dshulyak_ | there should be warning | 18:08 |
| pigmej | ;D:D | 18:09 |
| pigmej | well, it can't know wtf | 18:09 |
| pigmej | I mean how would you like to know that? | 18:12 |
| dshulyak_ | sorry, to know what? | 18:13 |
| pigmej | "<dshulyak_> there should be warning" | 18:14 |
| dshulyak_ | in logs there should be a warning that ModelMeta.session_start crashed | 18:14 |
| dshulyak_ | actually it should be error | 18:14 |
| dshulyak_ | hm, now i have another error :) | 18:15 |
| dshulyak_ | SiblingsError: No empty object for resolution not sure what to do with siblings: ['{"status": "PENDING", "task_type": "solar_resource", "target": "5168ef2d4b1177d9726eaaca75d852d3", "args": ["hosts_file1", "run"], "childs": [], "parents": ["system_log:1545e573-270d-4e57-b6c0-14c22d867bc6~node2.run", "system_log:1545e573-270d-4e57-b6c0-14c22d867bc6~node1.run"], "execution": "system_log:1545e573-270d-4e57-b6c0-14c22d867bc6", "errmsg": "", "name": | 18:15 |
| dshulyak_ | "hosts_file1.run"}', '{"status": "SUCCESS", "task_type": "solar_resource", "target": "5168ef2d4b1177d9726eaaca75d852d3", "args": ["hosts_file1", "run"], "childs": [], "parents": ["system_log:1545e573-270d-4e57-b6c0-14c22d867bc6~node2.run", "system_log:1545e573-270d-4e57-b6c0-14c22d867bc6~node1.run"], "execution": "system_log:1545e573-270d-4e57-b6c0-14c22d867bc6", "errmsg": "", "name": "hosts_file1.run"}'] | 18:15 |
| pigmej | yeah i's conflict | 18:16 |
| pigmej | you reset it | 18:16 |
| *** salmon_1 has joined #openstack-solar | 18:17 | |
| dshulyak_ | you think thats the cache issue? | 18:18 |
| pigmej | yeah | 18:19 |
| pigmej | I never had it when I'm restarting worker after each execution | 18:20 |
| *** bogdando has quit IRC | 18:22 | |
| *** salmon_ has quit IRC | 18:22 | |
| pigmej | btw what did you mean by context ? | 18:22 |
| *** bogdando has joined #openstack-solar | 18:28 | |
| dshulyak_ | i information that is related to orchestration there | 18:32 |
| pigmej | aha | 18:33 |
| dshulyak_ | strange.. i am certainly using session_end now | 18:34 |
| pigmej | it's always in the same greenlet ? | 18:35 |
| pigmej | maybe you call session_end in incorrect greenlet? | 18:36 |
| pigmej | but can you confirm that it works 'once' ? | 18:38 |
| dshulyak_ | yeah, after reset it starts to throw errors | 18:40 |
| pigmej | yeah, but if you reset AND restart solar-worker, then everything is fine | 18:41 |
| pigmej | at least for me | 18:41 |
| dshulyak_ | yes, the problem is still with session, im not calling it for some reason.. | 18:43 |
| pigmej | "good" at least it's easy to debug:) | 18:44 |
| openstackgerrit | Dmitry Shulyak proposed openstack/solar: Zerorpc worker for orchestration modules https://review.openstack.org/269166 | 18:47 |
| dshulyak_ | thanks, i was just stupid :) instead of calling function - i assigned it | 18:47 |
| pigmej | :D:D:D | 18:48 |
| pigmej | shit happens :D | 18:49 |
| *** dshulyak_ has quit IRC | 19:22 | |
| *** dshulyak_ has joined #openstack-solar | 20:18 | |
| *** dshulyak_ has quit IRC | 20:28 | |
| *** dshulyak_ has joined #openstack-solar | 20:42 | |
| *** dshulyak_ has quit IRC | 20:53 | |
| *** salmon_1 has quit IRC | 22:58 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!