| *** http_GK1wmSU has joined #openstack-mistral | 00:33 | |
| *** http_GK1wmSU has left #openstack-mistral | 00:33 | |
| openstackgerrit | Merged openstack/mistral master: Fix auth in actions if there is no auth_uri in context https://review.openstack.org/489561 | 00:45 |
|---|---|---|
| *** jrist has joined #openstack-mistral | 02:11 | |
| *** gkadam-afk has joined #openstack-mistral | 03:37 | |
| EmilienM | cores, please review https://review.openstack.org/#/c/481947/ | 04:51 |
| *** sharatss has joined #openstack-mistral | 05:53 | |
| *** livelace has joined #openstack-mistral | 05:58 | |
| *** mwhahaha has quit IRC | 06:02 | |
| *** gkadam-afk is now known as gkadam | 06:22 | |
| rakhmerov | d0ugal: hi | 06:31 |
| rakhmerov | on RDO problem, yes, 3.1.1 is the latest but it's not present there in packages | 06:32 |
| rakhmerov | (it wasn't at least) | 06:32 |
| rakhmerov | on https://review.openstack.org/#/c/481738/, well, it's Winson's style. He works with Mistral all the time and knows it very well but I can't convince him that the community life is important :) | 06:33 |
| rakhmerov | so, I just thoroughly reviewed it and it seemed fine to me | 06:33 |
| rakhmerov | and we had some discussion around tricky places | 06:33 |
| rakhmerov | but yes, we didn't have any discussions around it anyway | 06:34 |
| rakhmerov | that is a problem | 06:34 |
| rakhmerov | EmilienM: yes, ok, we'll try | 06:35 |
| rakhmerov | sorry for the delay | 06:35 |
| rakhmerov | d0ugal: how did you see that there hasn't been an RDO promotion for 11 days? | 06:43 |
| apetrich | rakhmerov, I talked to the rdo ci folks (I used to be one of them :) and they are scrambling to get it promoted | 07:08 |
| rakhmerov | apetrich: ok :) | 07:09 |
| *** jtomasek has joined #openstack-mistral | 07:09 | |
| apetrich | it does not help that some cores are away for a ci meeting this week | 07:09 |
| *** Dinesh_Bhor has quit IRC | 07:10 | |
| rakhmerov | apetrich: so can you please explain how this promotion works in general? | 07:13 |
| rakhmerov | for clients it's based on new versions? | 07:13 |
| rakhmerov | and for servers it | 07:13 |
| rakhmerov | it's done on a periodic bases? | 07:13 |
| rakhmerov | that's my current (possible wrong) understanding | 07:14 |
| rakhmerov | I'm just trying to understand what we did wrong so that we have this problem now | 07:14 |
| apetrich | I didn't work on promotions so my knowledge can be a bit wrong. here is what I understand out of it | 07:14 |
| apetrich | for master there are some stages on trunk.rdoproject there's current current current-tripleo current-tripleo-rdo and current-passed-ci | 07:16 |
| *** Dinesh_Bhor has joined #openstack-mistral | 07:16 | |
| apetrich | current-passed-ci is the important one. is the one that is cut to be released. (I'm not sure the distintion between current-tripleo current-tripleo-rdo) | 07:17 |
| openstackgerrit | Merged openstack/python-mistralclient master: mistral execution-list -f value should be empty https://review.openstack.org/489726 | 07:17 |
| apetrich | so for the master branch (so it is packages that do not have a hash associated it is build from git branch master) stuff is constantly built into current | 07:18 |
| apetrich | and a periodic job tries to test the current stuff | 07:18 |
| apetrich | it is tested here: https://ci.centos.org/view/rdo/view/promotion-pipeline/job/rdo_trunk-promote-master-current-tripleo/ | 07:18 |
| apetrich | if all that passes the current packages are moved into current-passed-ci and then into a release when a release is due | 07:19 |
| apetrich | ci is done using the current-passed-ci so until we get a promotion we are testing against old packages | 07:19 |
| apetrich | stuff is different for past versions those are all pinned to a hash, so they are build the same until we backport something and do a backported release | 07:20 |
| rakhmerov | apetrich: ok | 07:26 |
| rakhmerov | apetrich: not sure I understand everything though ) | 07:27 |
| rakhmerov | will re-read shortly.. | 07:27 |
| apetrich | when we merge something in master it goes into https://trunk.rdoproject.org/centos7-master/current/ once a day they try what is in current and if that is good it moves into current-passed-ci | 07:28 |
| apetrich | that is the promotion | 07:29 |
| apetrich | all our ci is done over current-passed-ci so we are testing over old libraries until promotion | 07:29 |
| apetrich | current-passed-ci is always able to be released (in theory and pretty much in practice) | 07:30 |
| apetrich | when we release I think current-passed-ci is tested and moved into a release | 07:30 |
| apetrich | in rhos there's more stuff done to current-passed-ci before release though | 07:31 |
| *** lkannan has quit IRC | 07:40 | |
| d0ugal | rakhmerov: https://dashboards.rdoproject.org/rdo-dev | 07:48 |
| d0ugal | rakhmerov: 12 days now. | 07:48 |
| d0ugal | I think the key thing to understand with promotions is that packages are only promoted when they are all proven to work together. | 07:53 |
| d0ugal | So that means mistralclient wont be promoted if there is something completely unrelated that is broken in tripleo | 07:53 |
| d0ugal | and this is why it can take so long, because so many packages are built from master there is a high chance something will break. | 07:54 |
| *** mgershen has joined #openstack-mistral | 07:58 | |
| rakhmerov | d0ugal: aah! It's not be been done not only for the mistral client?? | 08:02 |
| *** mwhahaha has joined #openstack-mistral | 08:03 | |
| rakhmerov | d0ugal: I saw this dashboard but I was looking for something related to mistral specifically and didn't realize you meant all packages | 08:03 |
| rakhmerov | apetrich: yeah, it makes more sense now to me | 08:03 |
| rakhmerov | thanks | 08:03 |
| d0ugal | Yeah, it makes sense I guess but I can see it is frustraiting if you only want one package update. | 08:03 |
| rakhmerov | yep | 08:04 |
| rakhmerov | but at least now we know the answer (in general) | 08:04 |
| apetrich | d0ugal, Thanks! that was what I was assuming and forgot to mention | 08:04 |
| d0ugal | rakhmerov: if you have a moment, I wanted to run an idea by you | 08:04 |
| rakhmerov | we thought we did something wrong on releasing mistral client or something else | 08:04 |
| rakhmerov | d0ugal: sure | 08:05 |
| rakhmerov | I do have a moment ) | 08:05 |
| d0ugal | great :) | 08:05 |
| d0ugal | You may remember my idea about root_execution (i.e. mistral execution-list --root-execution UUID) | 08:05 |
| d0ugal | I started to look into this, and I think the best way to do it is by adding the root execution UUID to the initial workflow execution - otherwise you need to recursively query and build up a tree which is very expensive. | 08:06 |
| d0ugal | I started doing it here: https://review.openstack.org/490063 | 08:06 |
| d0ugal | I just wanted to check that seemed like a reasonable approach before I spent too much more time on it. | 08:06 |
| rakhmerov | d0ugal: yeah, I remember | 08:07 |
| rakhmerov | well, I think it's ok | 08:09 |
| rakhmerov | just one (may be not so important) concern is that it won't allow to filter by not the root execution | 08:10 |
| d0ugal | hmm, I don't understand | 08:10 |
| rakhmerov | if we have say ex1 -> ex2 -> ex3 -> ex4 | 08:10 |
| rakhmerov | then we'll have a way to get ex2, ex3 and ex4 by specifying the root execution (the id of ex1) | 08:11 |
| rakhmerov | but won't be able to get ex3 and ex4 by specifying the id of ex2 | 08:11 |
| d0ugal | Right | 08:12 |
| rakhmerov | I mean we have to specify only the root execution | 08:12 |
| rakhmerov | not the second level or the third level | 08:12 |
| d0ugal | That is a limitation, it is hard to get the sub-section of the graph. | 08:12 |
| rakhmerov | but, I guess it may be a different task to do so | 08:12 |
| rakhmerov | yeah | 08:12 |
| d0ugal | I wanted to solve that, but I couldn't think of a good way to do it | 08:12 |
| rakhmerov | it doesn't fit well the relational model | 08:12 |
| d0ugal | Indeed | 08:12 |
| d0ugal | We need to move to a graph database ;) | 08:13 |
| rakhmerov | graph database is a good solution :) | 08:13 |
| rakhmerov | yes | 08:13 |
| rakhmerov | no kidding, I wanted to do it long ago ) | 08:13 |
| d0ugal | Okay, let me know when you port us over ;) | 08:13 |
| rakhmerov | sure! | 08:13 |
| rakhmerov | gimme a couple of days ;) | 08:13 |
| d0ugal | lol | 08:13 |
| rakhmerov | d0ugal: so Dougal, I think your solution is good enough for our use case | 08:14 |
| rakhmerov | given that we have an RDBMS | 08:14 |
| d0ugal | It is the simplest solution, which I like | 08:14 |
| rakhmerov | yes | 08:14 |
| d0ugal | I originally tried to do it without any model changes, but didn't get far before I decided it was a bad idea :) https://review.openstack.org/#/c/487391/ | 08:15 |
| rakhmerov | ok | 08:15 |
| d0ugal | okay, cool - I'll continue with this but keep looking for any more flexible solutions too | 08:16 |
| rakhmerov | ok | 08:16 |
| d0ugal | I think this will be useful for the cloudflow GUI btw | 08:16 |
| rakhmerov | yeah, this last one you posted looks complicated ) | 08:17 |
| rakhmerov | yes, sure | 08:17 |
| rakhmerov | btw, Guy's working very actively on various additions to the core functionality | 08:17 |
| d0ugal | and the one I abandoned only got ex1 and ex2, it wasn't recursive! | 08:17 |
| rakhmerov | he releases CloudFlow pretty often | 08:17 |
| rakhmerov | I believe it's already usable for much | 08:17 |
| d0ugal | Great, I seen you have a talk submitted for summit. | 08:17 |
| rakhmerov | yep | 08:18 |
| *** openstackgerrit has quit IRC | 08:18 | |
| d0ugal | I want to try it out in tripleo - the only thing stopping me is figuring out how to make it accessible :) | 08:19 |
| rakhmerov | sharatss: hi, are you working on something now? Or busy with internal stuff? | 08:19 |
| d0ugal | since I always ssh in | 08:19 |
| sharatss | rakhmerov, i am on leave.. anything needed? | 08:19 |
| rakhmerov | d0ugal: oh you mean environmental problem? | 08:19 |
| d0ugal | rakhmerov: yes | 08:20 |
| rakhmerov | d0ugal: I guess something like ssh tunnel or other trick could help ::) | 08:20 |
| rakhmerov | sharatss: ooh, ok, nothing needed, have a rest | 08:20 |
| d0ugal | yup, I just need to figure it out. I know some people do it for the tripleo GUI, so I could just ask them :) | 08:20 |
| d0ugal | (or read the docs!!) | 08:20 |
| rakhmerov | sharatss: I'll just finish that patch with --limit fix myself | 08:20 |
| rakhmerov | d0ugal: docs on what? ssh tunnel? :) | 08:21 |
| d0ugal | tripleo docs | 08:21 |
| rakhmerov | ooh, ok | 08:21 |
| rakhmerov | got it | 08:21 |
| rakhmerov | (hoping you weren't offending by me mentioning ssh tunnel docs) ;))) | 08:22 |
| rakhmerov | ..offended.. | 08:22 |
| d0ugal | haha, no | 08:22 |
| d0ugal | I would probably need to read that too, it has been a while. | 08:22 |
| sharatss | rakhmerov, oh.. ok. actually the weather is so bad that half of my team is on a sick leave | 08:22 |
| rakhmerov | d0ugal: yeah, but it's pretty easy anyway | 08:22 |
| rakhmerov | sharatss: np | 08:22 |
| *** openstackgerrit has joined #openstack-mistral | 08:24 | |
| openstackgerrit | Nikolay Mahotkin proposed openstack/mistral master: [Trusts] Fix deleting trust https://review.openstack.org/490049 | 08:24 |
| rakhmerov | apetrich: is this a critical comment? https://review.openstack.org/#/c/489217/4/mistralclient/api/v2/action_executions.py | 08:38 |
| rakhmerov | it's basically just what we have in other resource managers too | 08:38 |
| *** pratapagoutham has quit IRC | 08:41 | |
| openstackgerrit | Renat Akhmerov proposed openstack/python-mistralclient master: Fix how "--limit" is passed to the server for action executions https://review.openstack.org/489217 | 08:42 |
| openstackgerrit | wangxu proposed openstack/mistral master: Fixed crontrigger execution error. https://review.openstack.org/490395 | 08:56 |
| *** apetrich has quit IRC | 08:58 | |
| openstackgerrit | Renat Akhmerov proposed openstack/python-mistralclient master: Fix how "--limit" is passed to the server for action executions https://review.openstack.org/489217 | 09:19 |
| openstackgerrit | Merged openstack/mistral master: Fix drop index in version 022 DB upgrade script https://review.openstack.org/490217 | 09:27 |
| *** lkannan has joined #openstack-mistral | 09:38 | |
| *** apetrich has joined #openstack-mistral | 09:44 | |
| openstackgerrit | Nikolay Mahotkin proposed openstack/mistral master: [Trusts] Fix deleting trust https://review.openstack.org/490049 | 10:03 |
| apetrich | rakhmerov, I'm not a huge fan of the late if evaluation when reading code, but I'm fine if you don't mind. | 10:11 |
| apetrich | it is more of a style thing and I'm fine +1ing it | 10:11 |
| rakhmerov | ok | 10:17 |
| openstackgerrit | Renat Akhmerov proposed openstack/python-mistralclient master: Fix how "--limit" is passed to the server for action executions https://review.openstack.org/489217 | 10:21 |
| openstackgerrit | Ritesh proposed openstack/mistral master: Consider size of output_on_error https://review.openstack.org/490414 | 10:26 |
| openstackgerrit | Ritesh proposed openstack/mistral master: Consider size of output_on_error https://review.openstack.org/490414 | 10:28 |
| openstackgerrit | Renat Akhmerov proposed openstack/python-mistralclient master: Fix how "--limit" is passed to the server for action executions https://review.openstack.org/489217 | 10:30 |
| rakhmerov | mgershen, d0ugal: can you please review https://review.openstack.org/#/c/481947/ ? | 10:36 |
| openstackgerrit | Nikolay Mahotkin proposed openstack/mistral master: [Trusts] Fix deleting trust https://review.openstack.org/490049 | 10:42 |
| *** jkilpatr has quit IRC | 10:44 | |
| openstackgerrit | Sharat Sharma proposed openstack/mistral master: Small typo fix https://review.openstack.org/490420 | 10:51 |
| openstackgerrit | Merged openstack/python-mistralclient master: Add test for target parameters and fix requests lib error https://review.openstack.org/489237 | 11:07 |
| openstackgerrit | Renat Akhmerov proposed openstack/python-mistralclient master: Fix how "--limit" is passed to the server for action executions https://review.openstack.org/489217 | 11:09 |
| *** jkilpatr has joined #openstack-mistral | 11:17 | |
| *** apetrich has quit IRC | 11:41 | |
| rakhmerov | mgershen, d0ugal: please https://review.openstack.org/#/c/489217/ | 11:58 |
| rakhmerov | very much needed | 11:58 |
| *** dprince has joined #openstack-mistral | 12:12 | |
| *** catintheroof has joined #openstack-mistral | 12:36 | |
| *** thrash is now known as thrash|biab | 12:38 | |
| openstackgerrit | Nikolay Mahotkin proposed openstack/mistral master: [Trusts] Fix deleting trust https://review.openstack.org/490049 | 12:39 |
| *** gkadam has quit IRC | 12:50 | |
| *** apetrich has joined #openstack-mistral | 12:52 | |
| openstackgerrit | Nikolay Mahotkin proposed openstack/mistral master: Fix event-triggers workflow namespace https://review.openstack.org/490466 | 13:00 |
| openstackgerrit | Nikolay Mahotkin proposed openstack/mistral master: Fix event-triggers workflow namespace https://review.openstack.org/490466 | 13:01 |
| openstackgerrit | Merged openstack/python-mistralclient master: Fix how "--limit" is passed to the server for action executions https://review.openstack.org/489217 | 13:16 |
| *** sharatss has quit IRC | 13:31 | |
| *** sharatss has joined #openstack-mistral | 13:44 | |
| *** thrash|biab is now known as thrash | 13:52 | |
| *** bobh has joined #openstack-mistral | 13:57 | |
| *** sharatss has quit IRC | 14:11 | |
| *** thrash is now known as thrah|biab | 14:21 | |
| *** apetrich has quit IRC | 14:34 | |
| openstackgerrit | Merged openstack/mistral master: Use recommended function to setup auth middleware in devstack https://review.openstack.org/481947 | 14:46 |
| openstackgerrit | Merged openstack/mistral master: Set mistral-dashboard default branch to master https://review.openstack.org/489972 | 14:47 |
| openstackgerrit | Merged openstack/mistral master: Add Glare action pack https://review.openstack.org/473898 | 14:47 |
| openstackgerrit | Merged openstack/mistral master: Small typo fix https://review.openstack.org/490420 | 14:47 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: [WIP] Add root execution ID to workflow executions https://review.openstack.org/490063 | 14:54 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: [WIP] Add root execution ID to workflow executions https://review.openstack.org/490063 | 14:57 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: [WIP] Add root execution ID to workflow executions https://review.openstack.org/490063 | 15:03 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: Add root_execution_id to sub-workflow executions https://review.openstack.org/490063 | 15:11 |
| *** dprince has quit IRC | 15:27 | |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: Add root_execution_id to sub-workflow executions https://review.openstack.org/490063 | 15:29 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: Allow filter executions by their root_execution_id https://review.openstack.org/490562 | 15:29 |
| openstackgerrit | Michal Gershenzon proposed openstack/python-mistralclient master: Create and run a workflow within a namespace https://review.openstack.org/488531 | 15:39 |
| openstackgerrit | Nikolay Mahotkin proposed openstack/mistral master: Fix event-triggers workflow namespace https://review.openstack.org/490466 | 15:39 |
| *** dprince has joined #openstack-mistral | 15:40 | |
| openstackgerrit | Nikolay Mahotkin proposed openstack/mistral master: Fix event-triggers workflow namespace https://review.openstack.org/490466 | 15:51 |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: [WIP] Allow filter executions by their root_execution_id https://review.openstack.org/490562 | 15:59 |
| *** thrah|biab is now known as thrash | 16:00 | |
| openstackgerrit | Dougal Matthews proposed openstack/mistral master: [WIP] Allow filtering executions by their root_execution_id https://review.openstack.org/490562 | 16:03 |
| *** apetrich has joined #openstack-mistral | 16:10 | |
| *** vishwana_ has joined #openstack-mistral | 16:20 | |
| *** vishwanathj has quit IRC | 16:24 | |
| *** jtomasek has quit IRC | 16:25 | |
| *** gkadam has joined #openstack-mistral | 16:37 | |
| *** gkadam has quit IRC | 17:39 | |
| *** gkadam has joined #openstack-mistral | 17:39 | |
| openstackgerrit | Merged openstack/mistral master: Fixed crontrigger execution error. https://review.openstack.org/490395 | 17:59 |
| openstackgerrit | Nikolay Mahotkin proposed openstack/mistral master: Fix event-triggers workflow namespace https://review.openstack.org/490466 | 18:19 |
| *** gkadam has quit IRC | 18:29 | |
| *** livelace has quit IRC | 18:49 | |
| *** harlowja has quit IRC | 19:57 | |
| *** bobh has quit IRC | 19:59 | |
| *** jkilpatr has quit IRC | 20:48 | |
| *** bobh has joined #openstack-mistral | 20:55 | |
| *** harlowja has joined #openstack-mistral | 21:15 | |
| *** dprince has quit IRC | 21:54 | |
| *** bobh has quit IRC | 22:02 | |
| *** bobh has joined #openstack-mistral | 23:44 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!