*** itlinux has joined #openstack-mistral | 00:16 | |
*** AlexeyAbashkin has joined #openstack-mistral | 00:40 | |
*** AlexeyAbashkin has quit IRC | 00:44 | |
*** pengdake_ has joined #openstack-mistral | 00:55 | |
*** pengdake_ has quit IRC | 01:11 | |
*** pengdake_ has joined #openstack-mistral | 01:11 | |
*** pengdake has joined #openstack-mistral | 01:16 | |
*** pengdake_ has quit IRC | 01:16 | |
*** itlinux has quit IRC | 01:19 | |
*** itlinux has joined #openstack-mistral | 01:44 | |
*** pengdake has quit IRC | 01:59 | |
*** itlinux has quit IRC | 02:10 | |
*** itlinux has joined #openstack-mistral | 02:32 | |
*** pengdake has joined #openstack-mistral | 02:36 | |
*** itlinux has quit IRC | 02:44 | |
*** d0ugal has quit IRC | 02:49 | |
*** d0ugal has joined #openstack-mistral | 03:00 | |
*** gkadam has joined #openstack-mistral | 03:24 | |
*** harlowja has quit IRC | 03:30 | |
*** pengdake has quit IRC | 03:33 | |
*** pengdake has joined #openstack-mistral | 03:33 | |
*** d0ugal has quit IRC | 03:55 | |
*** d0ugal has joined #openstack-mistral | 04:00 | |
*** pengdake has quit IRC | 04:08 | |
*** harlowja has joined #openstack-mistral | 04:12 | |
*** gkadam_ has joined #openstack-mistral | 04:13 | |
*** gkadam_ has quit IRC | 04:14 | |
*** gkadam_ has joined #openstack-mistral | 04:14 | |
*** gkadam has quit IRC | 04:15 | |
*** ykarel_ has joined #openstack-mistral | 04:22 | |
*** harlowja has quit IRC | 04:30 | |
*** pengdake has joined #openstack-mistral | 05:01 | |
rakhmerov | d0ugal: hi, for some reason I don't see any support for root_execution_id on the client? Didn't we add it? I thought we did long ago.. | 05:04 |
---|---|---|
*** harlowja has joined #openstack-mistral | 05:14 | |
*** hardikjasani has joined #openstack-mistral | 06:00 | |
*** harlowja has quit IRC | 06:24 | |
*** ykarel_ is now known as ykarel | 06:31 | |
*** AlexeyAbashkin has joined #openstack-mistral | 07:16 | |
*** AlexeyAbashkin has quit IRC | 07:21 | |
*** AlexeyAbashkin has joined #openstack-mistral | 07:26 | |
*** ykarel_ has joined #openstack-mistral | 07:47 | |
*** ykarel has quit IRC | 07:48 | |
apetrich | rakhmerov, weird I do remember a patch for that. was it in tripleo-common instead? | 07:51 |
rakhmerov | apetrich: yeah, I thought I saw that patch as well | 07:51 |
apetrich | ugly regression patch seems fixed | 07:52 |
rakhmerov | not sure about triplo-common | 07:52 |
* apetrich is happier | 07:52 | |
*** ykarel__ has joined #openstack-mistral | 07:52 | |
rakhmerov | apetrich: what regression patch? | 07:52 |
apetrich | yeah so very tripleo-y :) | 07:52 |
apetrich | rakhmerov, it has been blocking promotion for a while. so we changed the python-mistralclient to return mistral exceptions but some other code were expecting keystone errors | 07:53 |
apetrich | https://review.openstack.org/#/c/543916/ | 07:53 |
rakhmerov | ok ) | 07:53 |
apetrich | the patches for changing to mistral exceptions were from november but some clients are just tested on release. hence the problem | 07:53 |
*** ykarel has joined #openstack-mistral | 07:54 | |
apetrich | it was like the second regression with a patch that had landed more than 4 months in a row | 07:54 |
apetrich | in mistralclient | 07:54 |
*** ykarel_ has quit IRC | 07:55 | |
apetrich | I think we will need some better actions to test that more frequently | 07:55 |
*** ykarel__ has quit IRC | 07:57 | |
apetrich | rakhmerov, not finding the root_execution_id patch. I guess we didn't have one? | 08:05 |
rakhmerov | apetrich: that's what I'd like to clarify with d0ugal | 08:05 |
rakhmerov | I failed to find it too although I seem to remember it | 08:05 |
rakhmerov | I just wanted to demo this functionality to someone | 08:06 |
rakhmerov | and realized that it's not supported by the client | 08:06 |
*** akazakov has joined #openstack-mistral | 08:35 | |
*** gkadam__ has joined #openstack-mistral | 08:50 | |
*** oidgar has joined #openstack-mistral | 08:51 | |
*** gkadam_ has quit IRC | 08:53 | |
*** jpich has joined #openstack-mistral | 08:59 | |
*** ykarel is now known as ykarel|lunch | 09:19 | |
d0ugal | rakhmerov: correct, I never added it to the client. | 09:50 |
rakhmerov | ooh, interesting.. | 09:50 |
d0ugal | well, it works via --filter doesn't it? | 09:50 |
d0ugal | I forget | 09:50 |
rakhmerov | hah! | 09:50 |
rakhmerov | yes, it actually should! %) | 09:51 |
d0ugal | so I don't think there is specific support - should there be? | 09:51 |
d0ugal | It might be easier if there was... | 09:51 |
rakhmerov | completely forgot about it.. | 09:51 |
d0ugal | or maybe the client could at least show the valid filter options | 09:51 |
rakhmerov | no-no, I think it is fine (if it works) but we just need to put a section about it in the doc | 09:51 |
rakhmerov | because it's very very useful | 09:52 |
rakhmerov | in many cases | 09:52 |
rakhmerov | showing the valid filter options is a good idea but we'll have to hardcode them on the client | 09:52 |
rakhmerov | which may not be a big deal though.. | 09:52 |
d0ugal | rakhmerov: see the commends on: https://review.openstack.org/#/c/543402/3 | 09:52 |
d0ugal | comments* | 09:53 |
d0ugal | brb | 09:53 |
rakhmerov | d0ugal: oh, so I need to send an email with the FFE request to myself or what? :) | 09:55 |
rakhmerov | let me send something anyway.. | 09:56 |
*** ykarel|lunch is now known as ykarel | 10:00 | |
*** shardy has joined #openstack-mistral | 10:02 | |
rakhmerov | d0ugal: just sent an email | 10:04 |
rakhmerov | "openstack-dev][mistral][release][ffe] Requesting FFE for supporting source execution id in the Mistral client" | 10:04 |
rakhmerov | you can also say a word or two to support this | 10:05 |
d0ugal | rakhmerov: great, thanks | 10:17 |
d0ugal | rakhmerov: sorry, I got called away longer than I thought. Today is a bit crazy | 10:18 |
d0ugal | I'll reply and agree, thanks for doing it. | 10:18 |
*** ykarel_ has joined #openstack-mistral | 10:26 | |
rakhmerov | np | 10:26 |
*** ykarel has quit IRC | 10:29 | |
*** ykarel_ is now known as ykarel | 10:37 | |
*** ykarel_ has joined #openstack-mistral | 10:52 | |
*** ykarel has quit IRC | 10:54 | |
*** AlexeyAbashkin has quit IRC | 11:05 | |
*** AlexeyAbashkin has joined #openstack-mistral | 11:38 | |
d0ugal | rakhmerov: Can we land this? https://review.openstack.org/#/c/543447/ | 11:50 |
d0ugal | It adds quite a bit of time to mistralclient patches, but we would have caught the regression we landed months ago. | 11:50 |
openstackgerrit | OpenStack Proposal Bot proposed openstack/mistral master: Updated from global requirements https://review.openstack.org/544267 | 12:10 |
*** gkadam has joined #openstack-mistral | 12:17 | |
*** gkadam__ has quit IRC | 12:19 | |
*** katkapilatova has joined #openstack-mistral | 12:31 | |
*** gkadam has quit IRC | 12:38 | |
*** thrash|g0ne is now known as thrash | 12:57 | |
*** weshay_PTO is now known as weshay | 13:01 | |
*** ykarel__ has joined #openstack-mistral | 13:03 | |
*** ykarel_ has quit IRC | 13:05 | |
openstackgerrit | Brad P. Crochet proposed openstack/python-mistralclient master: Migrate python-mistralclient jobs to Zuul v3 https://review.openstack.org/509184 | 13:18 |
*** oidgar has quit IRC | 13:21 | |
*** hardikjasani has quit IRC | 13:30 | |
openstackgerrit | OpenStack Release Bot proposed openstack/mistral master: Update reno for stable/queens https://review.openstack.org/544320 | 13:42 |
openstackgerrit | OpenStack Release Bot proposed openstack/mistral-dashboard master: Update reno for stable/queens https://review.openstack.org/544323 | 13:42 |
openstackgerrit | Merged openstack/mistral-dashboard master: Update reno for stable/queens https://review.openstack.org/544323 | 13:53 |
*** oidgar has joined #openstack-mistral | 14:16 | |
*** ykarel__ is now known as ykarel | 14:26 | |
*** bobh has joined #openstack-mistral | 15:05 | |
*** gkadam has joined #openstack-mistral | 15:12 | |
openstackgerrit | Merged openstack/mistral master: Update reno for stable/queens https://review.openstack.org/544320 | 15:16 |
*** AlexeyAbashkin has quit IRC | 15:17 | |
*** AlexeyAbashkin has joined #openstack-mistral | 15:42 | |
*** itlinux has joined #openstack-mistral | 15:50 | |
*** gkadam_ has joined #openstack-mistral | 15:52 | |
*** katkapilatova has quit IRC | 15:52 | |
*** katkapilatova has joined #openstack-mistral | 15:53 | |
*** katkapilatova has quit IRC | 15:57 | |
*** ykarel is now known as ykarel|away | 15:57 | |
*** gkadam has quit IRC | 16:04 | |
*** ykarel|away has quit IRC | 16:05 | |
d0ugal | apetrich: sigh. https://review.openstack.org/#/c/543447/ | 16:14 |
apetrich | d0ugal, test_executions_list_with_pagination again? | 16:16 |
d0ugal | apetrich: nah, another one | 16:16 |
d0ugal | apetrich: mistralclient.tests.functional.cli.v2.cli_tests_v2.TaskCLITests.test_task_list_with_filter | 16:16 |
apetrich | really? why I am seing thise http://logs.openstack.org/47/543447/5/gate/python-mistralclient-devstack-dsvm/fbb570d/job-output.txt.gz#_2018-02-14_13_04_10_493568 | 16:17 |
apetrich | oh also | 16:17 |
apetrich | yay | 16:17 |
d0ugal | huh | 16:17 |
d0ugal | Interesting | 16:17 |
d0ugal | I didn't notice there was more than one failure :( | 16:18 |
* apetrich cant type | 16:21 | |
* apetrich can't type "can't type" | 16:22 | |
apetrich | is that a race condition? because it smells like one | 16:23 |
d0ugal | apetrich: aye | 16:23 |
d0ugal | they all will be | 16:23 |
d0ugal | apetrich: in the test_task_list_with_filter it fails when listing the tasks, I assume there are tasks from an execution in a different test at that point | 16:23 |
apetrich | aye | 16:24 |
d0ugal | if tempest wasn't concurrent we would have less issues | 16:24 |
d0ugal | ... and slower tests | 16:24 |
apetrich | considering that we wait for ages to zuul to spin our tests how much time are we saving by deployment? | 16:24 |
d0ugal | apetrich: no idea, but I don't think we can change how people run tempest easily - so better to fix it at our end. | 16:26 |
d0ugal | IMO the tempest tests were just incorrectly done from the start and need to be rethought, but that is quite a big job | 16:27 |
d0ugal | Maybe I can try with all my spare time... | 16:27 |
*** itlinux has quit IRC | 16:41 | |
*** itlinux has joined #openstack-mistral | 16:52 | |
*** oidgar has quit IRC | 17:01 | |
*** AlexeyAbashkin has quit IRC | 17:07 | |
*** AlexeyAbashkin has joined #openstack-mistral | 17:15 | |
*** AlexeyAbashkin has quit IRC | 17:20 | |
*** jpich has quit IRC | 17:24 | |
*** thrash is now known as thrash|f00dz | 17:42 | |
openstackgerrit | OpenStack Proposal Bot proposed openstack/mistral stable/pike: Updated from global requirements https://review.openstack.org/544593 | 18:30 |
*** openstackgerrit has quit IRC | 18:33 | |
*** shardy has quit IRC | 18:38 | |
*** prometheanfire has joined #openstack-mistral | 18:58 | |
*** bobh has quit IRC | 18:58 | |
prometheanfire | we plan on removing mistral-extra from requirements, can we get an ack from the ptl? https://review.openstack.org/542907 | 18:58 |
apetrich | prometheanfire, why? if anything mistral-extra will be more present. the plan is to migrate openstack actions out of mistral into mistral-extra | 19:16 |
prometheanfire | apetrich: it's not used anywhere, http://codesearch.openstack.org/?q=mistral-extra&i=nope&files=.*requirements.*&repos= | 19:17 |
apetrich | yeah I know. | 19:18 |
apetrich | because releasing it was a pain. now that we have it released we are moving functionality into it this cycle | 19:18 |
apetrich | prometheanfire, ^^ | 19:18 |
prometheanfire | ok, if it's happening this cycle that works for me | 19:18 |
apetrich | we have a couple of cards to do so | 19:18 |
apetrich | cool | 19:19 |
prometheanfire | mind leaving a comment in the review? | 19:19 |
apetrich | I'm typing it | 19:19 |
prometheanfire | :D | 19:19 |
apetrich | cheers | 19:19 |
*** harlowja has joined #openstack-mistral | 19:21 | |
*** prometheanfire has left #openstack-mistral | 19:22 | |
*** bobh has joined #openstack-mistral | 19:38 | |
d0ugal | apetrich: gj | 20:23 |
*** thrash|f00dz is now known as thrash | 20:23 | |
*** bobh has quit IRC | 22:17 | |
*** threestrands has joined #openstack-mistral | 22:31 | |
*** itlinux has quit IRC | 22:53 |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!