| *** dangtrinhnt has quit IRC | 01:20 | |
| *** openstackgerrit has joined #openstack-mistral | 03:53 | |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Simplify workflow and join completion logic https://review.openstack.org/613551 | 03:53 |
|---|---|---|
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Simplify workflow and join completion logic https://review.openstack.org/613551 | 04:24 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: WIP: improve integrity checker https://review.openstack.org/615720 | 04:24 |
| *** jtomasek has joined #openstack-mistral | 05:14 | |
| *** akovi has joined #openstack-mistral | 05:28 | |
| *** nguyenhai has quit IRC | 05:40 | |
| *** nguyenhai has joined #openstack-mistral | 05:41 | |
| rakhmerov | d0ugal: good morning! | 05:52 |
| rakhmerov | d0ugal: we've got another problem with CI | 05:53 |
| rakhmerov | let's discuss how to fix all this when you're available | 05:53 |
| *** jtomasek has quit IRC | 06:39 | |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Fix usage of cachetools in lookup_utils https://review.openstack.org/615739 | 06:53 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Fix usage of cachetools in lookup_utils https://review.openstack.org/615739 | 07:04 |
| rakhmerov | d0ugal: I suggest we merge https://review.openstack.org/615739 which fixes the issue with the new version of cachetools and disables that test_generator test for now | 07:08 |
| rakhmerov | and fix the senlin issue after that | 07:08 |
| rakhmerov | because our CI queue is fully blocked now | 07:09 |
| rakhmerov | I guess we can temporarily live w/o senlin actions | 07:09 |
| rakhmerov | d0ugal: what do you think? | 07:09 |
| *** gkadam_ has joined #openstack-mistral | 07:45 | |
| *** josecastroleon has joined #openstack-mistral | 07:46 | |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Simplify workflow and join completion logic https://review.openstack.org/613551 | 07:47 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: WIP: improve integrity checker https://review.openstack.org/615720 | 07:47 |
| josecastroleon | hi @rakhmerov | 07:48 |
| josecastroleon | the error in the build for https://review.openstack.org/#/c/614211/ | 07:49 |
| josecastroleon | it's linked to a new version of cachetools that removed the missing attribute in the init creation of the cache | 07:49 |
| josecastroleon | https://github.com/tkem/cachetools/commit/5b85b3f523e92913fd742d35fb409e1caf9cec57 | 07:49 |
| therve | josecastroleon: https://review.openstack.org/615739 | 07:55 |
| josecastroleon | sorry for the noise | 07:55 |
| rakhmerov | josecastroleon: already fixed | 07:57 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Fix usage of cachetools in lookup_utils https://review.openstack.org/615739 | 08:38 |
| *** jtomasek has joined #openstack-mistral | 08:41 | |
| d0ugal | rakhmerov: sure, that plan sounds fine to me | 08:54 |
| rakhmerov | d0ugal: ok | 08:54 |
| openstackgerrit | Renat Akhmerov proposed openstack/mistral master: Add batch size for integrity checker https://review.openstack.org/615720 | 09:24 |
| therve | Is there a way to know in a task error whether it failed because of timeout? | 09:57 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: Fix senlin fake client creation https://review.openstack.org/614211 | 10:16 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: Fix senlin fake client creation https://review.openstack.org/614211 | 10:19 |
| d0ugal | therve: No, I don't think there is a way | 10:24 |
| therve | d0ugal: I think we can checking the task state info string | 10:24 |
| therve | But it's a bit weird | 10:24 |
| d0ugal | Yeah, that is a bit nasty | 10:24 |
| d0ugal | We should probably have some failure codes | 10:24 |
| therve | d0ugal: https://review.openstack.org/615653 ftr | 10:24 |
| therve | d0ugal: Did you find the right openstacksdk version btw | 10:25 |
| d0ugal | therve: I don't think so :) | 10:26 |
| therve | OK because the patch was still failing I think | 10:26 |
| d0ugal | Yeah, I am just about to try and figure it out | 10:26 |
| therve | d0ugal: I think you found out that 0.19 broke mistral again :) | 10:28 |
| d0ugal | lol | 10:28 |
| d0ugal | Going for the most recent version seemed like the easy option | 10:29 |
| d0ugal | sigh | 10:29 |
| therve | At least, my fix doesn't work with it | 10:29 |
| therve | Well that's good otherwise we'd have the issue in a month again | 10:29 |
| therve | Yeah it works with 17.2 but not newer... | 10:31 |
| *** pgaxatte has quit IRC | 10:36 | |
| therve | That's so rubbish... | 10:40 |
| d0ugal | Yeah :/ | 10:40 |
| therve | I vote to remove senlin from the face of earth | 10:41 |
| d0ugal | +2 | 10:41 |
| * d0ugal doesn't even know what Senlin is/does | 10:41 | |
| therve | Autoscaling | 10:41 |
| d0ugal | oh, that sounds fairly useful | 10:42 |
| d0ugal | huh, TIL Blizzard contribute to OpenStack | 10:42 |
| therve | Yep! | 10:43 |
| therve | d0ugal: http://paste.openstack.org/show/734195/ wdyt? | 10:46 |
| therve | That's horrible, but... | 10:47 |
| d0ugal | If it works, that is fine | 10:47 |
| d0ugal | the fake client method is allowed to be horrible IMO | 10:47 |
| therve | Alright let's try that then | 10:48 |
| *** pgaxatte has joined #openstack-mistral | 10:48 | |
| d0ugal | I prefer class_ over klass tho :) | 10:49 |
| therve | Ah :) | 10:50 |
| openstackgerrit | Thomas Herve proposed openstack/mistral master: Fix senlin fake client creation https://review.openstack.org/614211 | 10:51 |
| therve | One sec | 10:52 |
| openstackgerrit | Thomas Herve proposed openstack/mistral master: Fix senlin fake client creation https://review.openstack.org/614211 | 10:52 |
| therve | Alright | 10:53 |
| *** s10 has joined #openstack-mistral | 10:57 | |
| *** pgaxatte has quit IRC | 11:01 | |
| d0ugal | therve: LGTM, thanks! | 11:14 |
| *** gkadam_ has quit IRC | 11:46 | |
| *** gkadam has joined #openstack-mistral | 11:46 | |
| *** andreykurilin has joined #openstack-mistral | 12:47 | |
| *** pgaxatte has joined #openstack-mistral | 12:57 | |
| *** thrash|g0ne is now known as thrash | 13:19 | |
| andreykurilin | hi folks | 13:26 |
| andreykurilin | can anyone help me with investigation of mistral@devstack integration ? | 13:26 |
| andreykurilin | I see failures in several patches now | 13:27 |
| andreykurilin | the trace looks like - http://logs.openstack.org/15/615815/2/check/rally-task-mistral/d358cc9/job-output.txt.gz#_2018-11-06_12_02_08_501782 | 13:27 |
| andreykurilin | ok, it looks like recent release of cachetools affected your code | 13:29 |
| andreykurilin | https://pypi.org/project/cachetools/#history | 13:29 |
| andreykurilin | https://github.com/tkem/cachetools/blob/master/CHANGES.rst#v300-2018-11-04 there is a breaking change in the latest version | 13:30 |
| andreykurilin | https://github.com/tkem/cachetools/issues/58 | 13:32 |
| andreykurilin | https://github.com/openstack/requirements/commit/95fad145d6c49d32ff0ce70d901301a6bf6be880 | 13:32 |
| d0ugal | andreykurilin: You probably want this: https://review.openstack.org/615739 | 13:40 |
| andreykurilin | d0ugal: great, thanks! | 13:41 |
| d0ugal | np | 13:41 |
| *** bobh has joined #openstack-mistral | 14:19 | |
| openstackgerrit | Merged openstack/mistral master: Fix usage of cachetools in lookup_utils https://review.openstack.org/615739 | 14:22 |
| *** toure is now known as toure|biab | 14:51 | |
| *** s10 has quit IRC | 15:04 | |
| *** akovi has quit IRC | 15:08 | |
| *** toure|biab is now known as toure | 15:19 | |
| *** gkadam has quit IRC | 15:30 | |
| openstackgerrit | Jose Castro Leon proposed openstack/mistral master: Removes insecure parameter from barbican client https://review.openstack.org/614487 | 15:40 |
| openstackgerrit | Jose Castro Leon proposed openstack/mistral master: Add back the secrets_store action into mapping.json https://review.openstack.org/614488 | 15:41 |
| openstackgerrit | Jose Castro Leon proposed openstack/mistral master: Adds secrets_retrieve to the list of available actions in barbican https://review.openstack.org/614491 | 15:41 |
| *** thrash is now known as thrash|biab | 17:41 | |
| *** apetrich has quit IRC | 18:03 | |
| *** thrash|biab is now known as thrash | 18:12 | |
| *** jtomasek has quit IRC | 18:18 | |
| *** bobh has quit IRC | 18:55 | |
| *** shardy has quit IRC | 19:00 | |
| *** bobh has joined #openstack-mistral | 20:11 | |
| *** jtomasek has joined #openstack-mistral | 20:21 | |
| *** jtomasek has quit IRC | 20:24 | |
| *** mmethot_ is now known as mmethot | 20:38 | |
| *** bobh has quit IRC | 21:20 | |
| *** apetrich has joined #openstack-mistral | 21:29 | |
| *** bobh has joined #openstack-mistral | 21:46 | |
| *** bobh has quit IRC | 22:02 | |
| *** bobh has joined #openstack-mistral | 22:42 | |
| *** bobh has quit IRC | 22:45 | |
| *** bobh has joined #openstack-mistral | 22:46 | |
| *** bobh has quit IRC | 22:50 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!