| *** apetrich has quit IRC | 03:15 | |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Stop using deprecated keystone_authtoken/auth_uri https://review.openstack.org/594187 | 04:09 |
|---|---|---|
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Stop using deprecated keystone_authtoken/auth_uri https://review.openstack.org/594187 | 04:31 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Stop using deprecated keystone_authtoken/auth_uri https://review.openstack.org/594187 | 04:46 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Release note for fixing YAQL data context preparation https://review.openstack.org/637127 | 05:14 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Release note for fixing YAQL data context preparation https://review.openstack.org/637127 | 05:15 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral stable/rocky: Fix how Mistral prepares data for evaluating a YAQL expression https://review.openstack.org/637130 | 06:08 |
| *** jtomasek has joined #openstack-mistral | 07:00 | |
| *** pgaxatte has joined #openstack-mistral | 07:09 | |
| rakhmerov | hi all, help wanted to understand how to address http://logs.openstack.org/87/594187/19/check/openstack-tox-docs/1cadddf/job-output.txt.gz#_2019-02-15_04_57_51_069517 | 07:18 |
| rakhmerov | the patch is: https://review.openstack.org/#/c/594187/ | 07:19 |
| rakhmerov | requirements-check is now fixed but I'm not sure how to fix this issue with docs | 07:19 |
| rakhmerov | d0ugal: ^ | 07:19 |
| *** apetrich has joined #openstack-mistral | 07:30 | |
| apetrich | Morning | 07:31 |
| apetrich | we have weird version conflict | 07:31 |
| apetrich | (dogpile.cache 0.7.1 (/home/zuul/src/git.openstack.org/openstack/mistral/.tox/docs/lib/python3.5/site-packages), Requirement.parse('dogpile.cache<0.7.0,>=0.6.2'), {'oslo.cache'}) | 07:31 |
| apetrich | I saw that here https://review.openstack.org/#/c/594187/ on the failed docs gate | 07:31 |
| rakhmerov | apetrich: hi, yes | 07:39 |
| rakhmerov | what I don't understand is why it happens only on this patch? | 07:39 |
| rakhmerov | we used to have it on other patches but they it was somehow magically fixed | 07:39 |
| apetrich | mmm | 07:40 |
| rakhmerov | I'm not sure how | 07:40 |
| apetrich | let me test it locally and see what happens. | 07:40 |
| rakhmerov | it looks like a problem in oslo.cache rather than in Mistral | 07:40 |
| apetrich | aye | 07:40 |
| rakhmerov | seems like oslo.cache has a conflicting dependency | 07:40 |
| rakhmerov | but Mistral pulls oslo.cache in turn | 07:41 |
| apetrich | maybe the dogpile 0.7.1 is installed on the gates for something else? | 07:41 |
| rakhmerov | yes, that's what it seems like | 07:42 |
| rakhmerov | but why does it happen only for this patch? :) | 07:43 |
| rakhmerov | and how to fix it | 07:43 |
| rakhmerov | apetrich: https://github.com/openstack/oslo.cache/blob/master/requirements.txt#L5 | 07:43 |
| apetrich | aye it happens locally | 07:43 |
| apetrich | oslo-cache 1.32.0 has requirement dogpile.cache<0.7.0,>=0.6.2, but you'll have dogpile-cache 0.7.1 which is incompatible. | 07:43 |
| rakhmerov | yeah | 07:44 |
| apetrich | aye I agree it is their problem | 07:44 |
| rakhmerov | apetrich: any idea why it doesn't occur on other patches? | 07:45 |
| rakhmerov | e.g. we merged another patch yesterday and it was ok | 07:46 |
| apetrich | No idea | 07:48 |
| rakhmerov | aah.. I think I'm starting to understand | 07:56 |
| apetrich | rakhmerov, do we need dogpile for mistral | 07:56 |
| rakhmerov | apetrich: https://review.openstack.org/#/c/594187/19/requirements.txt | 07:56 |
| apetrich | exactly | 07:56 |
| rakhmerov | I deleted the upper version for dogpile | 07:56 |
| rakhmerov | and that's why it can now pull 0.7.1 I think.. | 07:56 |
| apetrich | if you add that line it works | 07:56 |
| apetrich | yeah | 07:56 |
| rakhmerov | but if I don't delete it then there's a problem with requirements check | 07:57 |
| rakhmerov | so I still don't understand what to do | 07:57 |
| rakhmerov | apetrich: well, it's a good question where we need it or not | 07:57 |
| apetrich | ignore the question | 07:57 |
| rakhmerov | I don't remember that we use it explicitly somewhere | 07:57 |
| apetrich | I was getting on that change | 07:58 |
| rakhmerov | ? | 07:58 |
| apetrich | dogpile is refered only in the requirements | 07:58 |
| rakhmerov | hah.. | 07:58 |
| rakhmerov | then why do we have it in requirements.tx? | 07:58 |
| rakhmerov | interesting | 07:58 |
| rakhmerov | let me try to remove it | 07:58 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Stop using deprecated keystone_authtoken/auth_uri https://review.openstack.org/594187 | 08:02 |
| apetrich | rakhmerov, we might have to remove from lower-requirements also | 08:06 |
| rakhmerov | yeah, let's see | 08:06 |
| rakhmerov | I'd like to experiment | 08:06 |
| rakhmerov | requirements-check now passed | 08:08 |
| rakhmerov | apetrich: doc again failed | 08:12 |
| rakhmerov | with the same error | 08:12 |
| rakhmerov | do you think removing a low constraint may help? | 08:12 |
| rakhmerov | apetrich: ^ | 08:13 |
| apetrich | shouldn't | 08:16 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Stop using deprecated keystone_authtoken/auth_uri https://review.openstack.org/594187 | 08:16 |
| apetrich | rakhmerov, ^ if it is the 0.7.1 again | 08:16 |
| rakhmerov | yeah | 08:16 |
| rakhmerov | it shouldn't | 08:16 |
| rakhmerov | but I decided to see if it changes anything | 08:16 |
| apetrich | I'm fine with that | 08:19 |
| apetrich | it is a very odd bug | 08:19 |
| apetrich | we might need the dogpile.cache in the requirements to avoid that issue. if you remove both it fails locally | 08:26 |
| apetrich | bound with the upperversion at 0.7.0 | 08:26 |
| rakhmerov | yeah | 08:35 |
| rakhmerov | yeah, so removing it didn't help | 08:36 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Stop using deprecated keystone_authtoken/auth_uri https://review.openstack.org/594187 | 08:39 |
| apetrich | do you remember the issue that was happening on keeping the <=0.7.0 ? | 08:39 |
| d0ugal | rakhmerov, apetrich: Here now. Any luck figuring it out? | 08:43 |
| rakhmerov | d0ugal: hi, not yet | 08:45 |
| rakhmerov | we keep trying ) | 08:45 |
| rakhmerov | I rolled it back to where it was now | 08:45 |
| rakhmerov | and requirements-check now failed again, of course | 08:46 |
| rakhmerov | aah.. this is insane | 08:46 |
| rakhmerov | d0ugal: it looks like a mystery really | 08:46 |
| rakhmerov | the only way to pass requirements-check that I found is to remove the upper bound in requirements.txt | 08:47 |
| rakhmerov | but if we do it then docs start failing because oslo.cache pulls some weird version | 08:47 |
| rakhmerov | 0.7.1 | 08:47 |
| rakhmerov | I guess I'm misunderstanding something.. | 08:48 |
| rakhmerov | question #1) Given that it's now rebased properly, why is this the only patch where it happens? It has nothing to do with this dogpile.cache | 08:49 |
| d0ugal | We hit something like this before | 08:49 |
| d0ugal | https://review.openstack.org/#/c/626532/ | 08:49 |
| rakhmerov | yes | 08:49 |
| rakhmerov | d0ugal: do you remember what the solution was? | 08:49 |
| d0ugal | It was me that pinned dogpile to <0.7 to fix it at the time | 08:49 |
| rakhmerov | aah, yes | 08:49 |
| rakhmerov | so you updated oslo.cache | 08:49 |
| d0ugal | it conflicted with openstacksdk | 08:50 |
| rakhmerov | yeah | 08:50 |
| d0ugal | https://github.com/openstack/openstacksdk/commit/b25e4b8be57e582cb133be310f2789271bc1e8d4 | 08:50 |
| d0ugal | so maybe that has just been released? | 08:50 |
| d0ugal | Yeah, openstacksdk was released 11 days ago | 08:50 |
| d0ugal | so maybe we are just seeing that come through now? | 08:50 |
| rakhmerov | ooh damn.. | 08:50 |
| d0ugal | I'll propose a revert to my change in oslo cache | 08:51 |
| rakhmerov | I see | 08:51 |
| rakhmerov | wait a sec.. | 08:51 |
| rakhmerov | d0ugal: why revert of your change? | 08:51 |
| rakhmerov | what exactly do you want to do? | 08:51 |
| d0ugal | rakhmerov: to remove the limit from oslo.cache? | 08:51 |
| d0ugal | Maybe that isn't the correct fix | 08:51 |
| rakhmerov | aah.. | 08:52 |
| d0ugal | It was also removed from global requirements | 08:52 |
| d0ugal | https://github.com/openstack/requirements/commit/d6576d9a6e0629c5c7059aa8c9e36e9c0994ed5b | 08:52 |
| rakhmerov | "maybe" doesn't sound too encouraging :) | 08:52 |
| d0ugal | so both global and sdk removed the limit | 08:52 |
| d0ugal | lol | 08:52 |
| rakhmerov | yeah | 08:52 |
| rakhmerov | ok, please go ahead | 08:52 |
| d0ugal | We should also remove it from mistral | 08:52 |
| rakhmerov | I think this is the right direction, yes | 08:52 |
| rakhmerov | yep | 08:52 |
| rakhmerov | when I removed boundaries it was still failing | 08:53 |
| rakhmerov | requirements-check I mean | 08:53 |
| rakhmerov | when I removed only the upper bound then this gate was OK | 08:53 |
| rakhmerov | but doc was failing | 08:53 |
| rakhmerov | when it was w/o any bounds then it'd say "Low bound is not set" or something like this | 08:54 |
| d0ugal | hmm | 08:55 |
| d0ugal | I guess we will see what happens | 08:55 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: Update dogpile.cache to match global requirements https://review.openstack.org/637145 | 08:55 |
| rakhmerov | ok | 08:55 |
| d0ugal | I proposed a new patch instead of a revert | 08:57 |
| *** akovi has joined #openstack-mistral | 09:02 | |
| openstackgerrit | Vlad Gusev proposed openstack/mistral master: Stop using deprecated keystone_authtoken/auth_uri https://review.openstack.org/594187 | 09:08 |
| rakhmerov | d0ugal: both jobs failed | 09:09 |
| d0ugal | damn | 09:09 |
| openstackgerrit | Vlad Gusev proposed openstack/mistral master: Stop using deprecated keystone_authtoken/auth_uri https://review.openstack.org/594187 | 09:10 |
| d0ugal | Then I'm really not sure | 09:11 |
| *** shardy has joined #openstack-mistral | 09:11 | |
| rakhmerov | yeah | 09:13 |
| rakhmerov | that's the thing | 09:13 |
| rakhmerov | :) | 09:13 |
| rakhmerov | d0ugal: Can you please try to remove the upper bound only? | 09:14 |
| rakhmerov | That's what I saw in other projects | 09:14 |
| rakhmerov | d0ugal: openstacksdk has dogpile.cache>=0.6.2 | 09:18 |
| d0ugal | yeah | 09:18 |
| d0ugal | http://codesearch.openstack.org/?q=dogpile.cache&i=nope&files=requirements.txt&repos= | 09:18 |
| rakhmerov | interestingly, a dependency w/o an upper bound still remains compatible with global requirements | 09:21 |
| rakhmerov | I wonder if it was always the case or it was changed not long ago | 09:21 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: Update dogpile.cache to match global requirements https://review.openstack.org/637145 | 09:22 |
| d0ugal | https://github.com/openstack/requirements/blob/master/upper-constraints.txt#L402 | 09:23 |
| d0ugal | I don't think we should ever get higher than 0.6.8 because of that anyway | 09:23 |
| d0ugal | This is such a confusing mess | 09:23 |
| rakhmerov | hm.. yes | 09:24 |
| rakhmerov | it is a mess | 09:24 |
| d0ugal | I miss the requirements bot updating everything. It created noise but it seemed to work | 09:24 |
| rakhmerov | ooh yeah | 09:24 |
| rakhmerov | btw, why was it removed? | 09:24 |
| rakhmerov | I don't quite remember | 09:24 |
| d0ugal | I can't remember either | 09:24 |
| rakhmerov | or just missed that | 09:24 |
| rakhmerov | ok | 09:24 |
| apetrich | me neither | 09:28 |
| *** vgvoleg has quit IRC | 09:29 | |
| rakhmerov | d0ugal: doc failed, requirements-check passed | 09:30 |
| rakhmerov | same error | 09:30 |
| rakhmerov | pkg_resources.ContextualVersionConflict: (dogpile.cache 0.7.1 (/home/zuul/src/git.openstack.org/openstack/mistral/.tox/docs/lib/python3.5/site-packages), Requirement.parse('dogpile.cache<0.7.0,>=0.6.2'), {'oslo.cache'}) | 09:30 |
| rakhmerov | it seems like we can't do anything in Mistral to fix it | 09:31 |
| rakhmerov | so may be reverting your patch to oslo.cache is the only solution | 09:34 |
| rakhmerov | d0ugal: ^ | 09:35 |
| *** vgvoleg has joined #openstack-mistral | 09:49 | |
| d0ugal | Yeah, worth trying | 09:49 |
| *** vgvoleg has quit IRC | 09:51 | |
| rakhmerov | d0ugal: can you please do that? | 10:07 |
| d0ugal | My olso.cache patch is essentially a revert I think? | 10:08 |
| rakhmerov | yeah | 10:13 |
| rakhmerov | well, not sure | 10:14 |
| *** vgvoleg has joined #openstack-mistral | 10:52 | |
| openstackgerrit | Vlad Gusev proposed openstack/mistral master: Release note for fixing event-engines HA https://review.openstack.org/637167 | 10:56 |
| openstackgerrit | Chandan Kumar proposed openstack/mistral master: Revert "Fix how Mistral prepares data for evaluating a YAQL expression" https://review.openstack.org/637170 | 11:01 |
| akovi | Do you have some info on why the YAQL change is being reverted? | 12:59 |
| *** jistr is now known as jistr|mtg | 15:32 | |
| *** jistr|mtg is now known as jistr | 16:04 | |
| *** pgaxatte has quit IRC | 16:34 | |
| *** akovi has quit IRC | 17:06 | |
| openstackgerrit | Corey Bryant proposed openstack/mistral master: add python 3.7 unit test job https://review.openstack.org/637268 | 19:06 |
| openstackgerrit | Corey Bryant proposed openstack/mistral-lib master: add python 3.7 unit test job https://review.openstack.org/637269 | 19:07 |
| openstackgerrit | Corey Bryant proposed openstack/python-mistralclient master: add python 3.7 unit test job https://review.openstack.org/637270 | 19:07 |
| *** shardy has quit IRC | 19:43 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!