Wednesday, 2017-12-06

*** threestrands has quit IRC00:25
*** yangyapeng has joined #openstack-mistral01:14
*** zhurong has joined #openstack-mistral01:49
*** bobh has joined #openstack-mistral02:18
*** bobh has quit IRC02:20
*** bobh has joined #openstack-mistral02:34
*** bobh has quit IRC02:36
*** thrash is now known as thrash|g0ne02:50
*** bobh has joined #openstack-mistral02:52
*** zhurong has quit IRC03:02
*** zhurong has joined #openstack-mistral03:10
*** bobh has quit IRC03:12
*** haint has joined #openstack-mistral03:52
*** bobh has joined #openstack-mistral03:53
*** nguyentrihai has quit IRC03:55
*** bobh has quit IRC04:08
*** bobh has joined #openstack-mistral04:16
*** bobh has quit IRC04:17
*** zhurong has quit IRC04:21
*** daidv has quit IRC04:41
*** daidv has joined #openstack-mistral04:42
*** zhurong has joined #openstack-mistral05:08
*** threestrands has joined #openstack-mistral05:09
*** threestrands has quit IRC05:09
*** AlexeyAbashkin has joined #openstack-mistral06:07
*** AlexeyAbashkin has quit IRC06:11
*** AlexeyAbashkin has joined #openstack-mistral06:12
*** AlexeyAbashkin has quit IRC06:26
*** gkadam has joined #openstack-mistral06:54
*** jtomasek has joined #openstack-mistral07:10
*** jtomasek has quit IRC07:10
*** jtomasek has joined #openstack-mistral07:11
rakhmerovd0ugal: hi, how is everything going?07:22
rakhmerovI'm back07:22
chandankumarrakhmerov: hello welcome back07:22
rakhmerovthanks )07:22
chandankumarrakhmerov: need some ack on https://review.openstack.org/524868 https://review.openstack.org/52486907:22
chandankumarto move further07:22
rakhmerovok-ok, I'll look at it when I have a chance..07:23
rakhmerova pile of things to do after vacation07:23
chandankumarsure take your time :-)07:24
apetrichrakhmerov, hey! welcome back. How was the holidays?07:37
rakhmerovhi Adriano )07:37
rakhmerovIt was good, very good )07:38
rakhmerovhow have you been?07:38
apetrichNot bad. My mom is visiting so I took last week off. Did a bit of driving around.07:38
rakhmerovooh, I see07:39
rakhmerovapetrich: may be it's too early to ask but are you going to participate the next PTG in Dublin?07:39
apetrichrakhmerov, d0ugal, toure and I put our names up from the workflow:dfg. It is very likely that d0ugal and I are going and we have to see about budget for toure to go07:41
apetrichAFAIK07:41
apetrichI'm kind of counting on going07:41
rakhmerovok, good07:41
rakhmerovvery good07:41
apetrichrakhmerov, I made some progress on that blueprint btw.07:42
rakhmerovooh, really?07:43
rakhmerovhave a WIP?07:43
openstackgerritAdriano Petrich proposed openstack/mistral master: WIP add extra yaql thinguies  https://review.openstack.org/52592607:43
rakhmerov:)07:43
rakhmerovgood timing!07:43
apetrichI think I need a bit more work on it. but putting it up for visibility07:44
rakhmerovok, sure07:44
rakhmerovawesome07:44
rakhmerovgood news )07:44
apetrichs/think/know/ :)07:44
d0ugalHey rakhmerov, welcome back!07:46
d0ugalBack shortly. I need breakfast :)07:46
rakhmerovsure ))07:46
rakhmerovhave it!07:46
* apetrich craves for breakfast too07:48
*** oidgar has joined #openstack-mistral08:00
*** shardy has joined #openstack-mistral08:08
*** csatari has quit IRC08:09
d0ugalapetrich: do it08:11
d0ugalrakhmerov: we had one big bug while you were away, but otherwise things have been quiet08:12
*** csatari has joined #openstack-mistral08:12
rakhmerovooh08:12
rakhmerovwhat was that?08:12
rakhmerovalready fixed?08:12
d0ugalrakhmerov: https://review.openstack.org/#/c/522822/08:13
d0ugalThe related bug is a bit confusing, because it came from the tripleo pipeline08:14
rakhmerovok08:14
d0ugalbut the tl;dr is that we were using oslo.context incorrectly and then released a new version that then broke how we do keystone auth08:14
rakhmerovI see08:14
d0ugalso all the actions etc. stopped working - it was very confusing08:14
rakhmerovyeah08:14
rakhmerovcoverage, coverage..08:15
d0ugalrakhmerov: well, the tempest gates caught it but since they don't vote nobody noticed08:15
d0ugalwe need to fix that08:15
rakhmerovyes08:15
d0ugalself.get_logging_values() in https://review.openstack.org/#/c/522822/1/mistral/context.py08:15
d0ugalThat was the problem line if you are interested08:15
rakhmerovI am, yep08:16
rakhmerovok, I'll look at it08:16
d0ugalwe were using a method to get logging values, but we were using it not for logging08:16
d0ugalso oslo.context added a mask to the token, because you don't want to log the real token08:16
d0ugalwhen they done that we had "***" and not the actual token08:16
rakhmerovok08:17
rakhmerovI see08:17
d0ugalMy suggestion is to make them voting and then disable the unstable tests. Then we will have at least some of the tests and we can fix the broken ones one at a time.08:19
d0ugalIt looks quite nice with almost everything voting :) https://review.openstack.org/#/c/522880/08:19
rakhmerov:)08:21
rakhmerovd0ugal: looked at the patch. Yeah, the bug was pretty stupid )08:26
rakhmerovI remember I was really surprised why we needed to use this method08:26
d0ugalYeah, we should have questioned that during rview08:26
rakhmerovsomebody explained but I didn't really get it08:27
d0ugallol08:27
d0ugalI think I missed it. I forget if I reviewed the original08:27
d0ugalrakhmerov: do you think we should backport it to pike? it should be okay, but if oslo.context backports it there could be problems.08:28
d0ugaland I think it is safe to backport.08:28
d0ugalhttps://review.openstack.org/523096 - I opened a backport, but wasn't sure.08:28
rakhmerovyes, I think we can do it08:29
rakhmerovapproved it08:31
d0ugalrakhmerov: thanks08:31
rakhmerovnp08:31
*** AlexeyAbashkin has joined #openstack-mistral08:32
d0ugalrakhmerov: My next goal is to make the tempest jobs stable enough to be voting.08:32
rakhmerovgood goal, yes08:32
rakhmerovI think we can disable at least one known test for now08:33
rakhmerovwe discussed that before but it was never done08:33
d0ugalrakhmerov: which one? They all pass sometimes.08:33
rakhmerovSSHProxied blablabla08:33
d0ugaloh yes, I have seen that one fail lots08:33
*** zhurong has quit IRC08:33
rakhmerovit fails often because the test itself08:33
d0ugalI am going to open a bug now and start noting down which tests fail.08:33
rakhmerovyes, good08:33
*** AlexeyAbashkin has quit IRC08:38
*** AlexeyAbashkin has joined #openstack-mistral08:44
*** jtomasek has quit IRC08:45
*** AlexeyAbashkin has quit IRC08:49
*** zhurong has joined #openstack-mistral08:53
*** jpich has joined #openstack-mistral09:03
*** vishwanathj has quit IRC09:13
d0ugalrakhmerov: if you can, reviews of these would be great: https://review.openstack.org/#/q/topic:bug/1718353+status:open+project:openstack/mistral09:14
d0ugaljust not on the one I -2'ed (see my comment there for the reason)09:14
*** vishwanathj has joined #openstack-mistral09:14
*** AlexeyAbashkin has joined #openstack-mistral09:14
d0ugalI am ignoring the -1 I got from somebody, because I don't think it is important - but let me know if you think I should change anything.09:14
d0ugalapetrich: ^ if you get a chance to review them too that'd be great.09:15
d0ugalThe are finally passing CI, other than the unrelated tempest issues :)09:15
apetrichd0ugal, looking09:15
d0ugalYou did review one already :)09:16
d0ugaltwo actually09:20
openstackgerritDougal Matthews proposed openstack/mistral master: Disable unstable tempest multi_vim_authentication test  https://review.openstack.org/52600209:26
openstackgerritDougal Matthews proposed openstack/mistral master: Make more CI jobs voting  https://review.openstack.org/52288009:26
*** d0ugal has quit IRC09:37
openstackgerritDougal Matthews proposed openstack/mistral master: Add type definitions to support on-complete, on-error, on-success  https://review.openstack.org/49979009:40
*** d0ugal has joined #openstack-mistral09:42
rakhmerovd0ugal: yes, sure, I'll look at them10:04
rakhmerovbut I think not today, tomorrow morning..10:04
d0ugalrakhmerov: okay, thanks :)10:06
openstackgerritMerged openstack/python-mistralclient master: Updated from global requirements  https://review.openstack.org/52051410:21
openstackgerritMerged openstack/mistral master: Allow filtering executions by their root_execution_id  https://review.openstack.org/49056210:30
*** katkapilatova has joined #openstack-mistral10:30
*** zhurong has quit IRC10:37
*** d0ugal has quit IRC10:39
*** jtomasek has joined #openstack-mistral10:40
openstackgerritMerged openstack/mistral stable/pike: Don't use oslo context get_logging_values  https://review.openstack.org/52309610:55
*** d0ugal has joined #openstack-mistral11:02
openstackgerritDougal Matthews proposed openstack/mistral master: Make more CI jobs voting  https://review.openstack.org/52288011:24
openstackgerritDougal Matthews proposed openstack/mistral master: Disable unstable tempest test_create_action_execution_sync test  https://review.openstack.org/52603411:24
chandankumard0ugal: please donot push tempest tests to mistral repo we can push to new repo11:27
chandankumard0ugal: https://review.openstack.org/#/c/524869/ is about to merge11:27
d0ugalchandankumar: I need to stop thm failing. These are not new tests.11:27
chandankumard0ugal: ack, then11:27
d0ugalchandankumar: but let me know when the gerrit project is there and I'll move my patches over.11:28
d0ugalIt should be very easy :)11:29
*** yangyapeng has quit IRC11:36
*** yangyapeng has joined #openstack-mistral11:36
openstackgerritDougal Matthews proposed openstack/mistral master: Disable unstable tempest multi_vim_authentication test  https://review.openstack.org/52600211:44
openstackgerritDougal Matthews proposed openstack/mistral master: Disable unstable tempest test_create_action_execution_sync test  https://review.openstack.org/52603411:44
openstackgerritDougal Matthews proposed openstack/mistral master: Make more CI jobs voting  https://review.openstack.org/52288011:44
*** AlexeyAbashkin has quit IRC11:58
*** jkilpatr has joined #openstack-mistral12:00
*** openstackgerrit has quit IRC12:03
*** bobh has joined #openstack-mistral12:18
*** yangyapeng has quit IRC12:18
*** yangyapeng has joined #openstack-mistral12:18
*** yangyapeng has quit IRC12:25
*** thrash|g0ne is now known as thrash12:46
*** AlexeyAbashkin has joined #openstack-mistral12:46
*** yangyapeng has joined #openstack-mistral12:50
*** openstackgerrit has joined #openstack-mistral13:13
openstackgerritDougal Matthews proposed openstack/mistral master: Make more CI jobs voting  https://review.openstack.org/52288013:13
*** dprince has joined #openstack-mistral13:14
*** jkilpatr has quit IRC13:16
*** bobh has quit IRC13:31
*** bobh has joined #openstack-mistral13:40
*** jkilpatr has joined #openstack-mistral13:43
*** bobh has quit IRC13:45
*** bobh has joined #openstack-mistral14:03
*** bobh has quit IRC14:07
*** bobh has joined #openstack-mistral14:15
*** gkadam has quit IRC14:18
*** bobh has quit IRC14:20
openstackgerritDougal Matthews proposed openstack/mistral master: Re-work the direct action call tempest test  https://review.openstack.org/52608214:20
*** bobh has joined #openstack-mistral14:22
*** bobh has quit IRC14:27
*** bobh has joined #openstack-mistral14:43
*** bobh has quit IRC14:48
openstackgerritBrad P. Crochet proposed openstack/mistral master: DNM: TEST Zuul v3 jobs  https://review.openstack.org/50942814:49
*** bobh has joined #openstack-mistral14:54
openstackgerritAndrey Kurilin proposed openstack/mistral master: [rally] use the proper task file name  https://review.openstack.org/52608914:58
*** bobh has quit IRC14:59
openstackgerritAdriano Petrich proposed openstack/mistral master: Add executions yaql filter  https://review.openstack.org/52592615:03
d0ugalthrash: FYI chandankumar is working on the mistral tempest plugin repo15:09
chandankumarhttps://review.openstack.org/#/q/project:openstack/mistral-tempest-plugin15:09
chandankumarhttps://github.com/openstack/mistral-tempest-plugin15:10
chandankumarhttps://review.openstack.org/#/c/524952/15:10
chandankumarplease wait till above review get merged15:10
openstackgerritAndrey Kurilin proposed openstack/mistral master: [rally] attempt to fix the job  https://review.openstack.org/52609615:11
*** bobh has joined #openstack-mistral15:13
thrashchandankumar: cool. I have zuul jobs for that.15:15
*** bobh has quit IRC15:17
*** bobh has joined #openstack-mistral15:18
openstackgerritAdriano Petrich proposed openstack/mistral master: Add executions yaql filter  https://review.openstack.org/52592615:19
openstackgerritBrad P. Crochet proposed openstack/mistral master: DNM: TEST Zuul v3 jobs  https://review.openstack.org/50942815:40
openstackgerritBrad P. Crochet proposed openstack/mistral master: DNM: TEST Migrate the jobs to native Zuul v3 format  https://review.openstack.org/50942815:41
openstackgerritBrad P. Crochet proposed openstack/python-mistralclient master: Migrate python-mistralclient jobs to Zuul v3  https://review.openstack.org/50918415:52
openstackgerritBrad P. Crochet proposed openstack/mistral master: DNM: TEST Migrate the jobs to native Zuul v3 format  https://review.openstack.org/50942815:59
openstackgerritToure Dunnon proposed openstack/mistral-extra master: Add Ansible actions to Mistral  https://review.openstack.org/51436016:05
openstackgerritBrad P. Crochet proposed openstack/mistral master: Fix swift endpoint  https://review.openstack.org/52611816:28
thrashd0ugal: ^^^^16:28
thrashrakhmerov: ^^^16:28
rakhmerovthrash: hey, what's up?16:28
thrashrakhmerov: swift actions are busted.16:29
thrash:D16:29
rakhmerovI'm here coincidentally for several minsz16:29
thrashand I'm not sure that fixed it... :P16:29
rakhmerovbusted?16:29
rakhmerovooh..16:29
rakhmerovreason?16:29
thrashNever mind... I need to figure out exactly what is going on. Getting the wrong endpoint...16:29
rakhmerovok16:30
thrashFailed to run action [action_ex_id=e4d9f3ef-ab5a-4b1a-aacb-3d19a7ea36fa, action_cls='<class 'mistral.actions.action_factory.SwiftAction'>', attributes='{u'client_method_name': u'get_object'}', params='{u'obj': u'network_data.yaml', u'container': u'overcloud'}']\n SwiftAction.get_object failed: Object GET failed: http://192.168.24.1:5000/v3/overcloud/network_data.yaml 404 Not Found  [first 60 chars16:30
thrashof response] {\"error\": {\"message\": \"(http://192.168.24.1:5000/v3/overclou16:30
rakhmerovhah16:30
thrashYou can see that it is using keystone endpoint for some reason.16:30
rakhmerovsome Keystone catalog problems maybe?16:30
rakhmerovook16:30
thrashI'll have an update in a minute I think.16:34
thrashHaha... was testing my change, and I restarted swift instead of mistral lol16:36
openstackgerritBrad P. Crochet proposed openstack/mistral master: Fix swift endpoint  https://review.openstack.org/52611816:39
thrashok... THAT fixes it.16:39
*** AlexeyAbashkin has quit IRC16:54
*** katkapilatova has quit IRC16:59
*** dprince has quit IRC17:01
openstackgerritBrad P. Crochet proposed openstack/mistral master: DNM: TEST Migrate the jobs to native Zuul v3 format  https://review.openstack.org/50942817:16
*** thrash is now known as thrash|biab17:16
*** jtomasek has quit IRC17:22
d0ugalthrash|biab: do we need the same change in tripleo-common?17:23
d0ugalthrash|biab: why wasn't tripleo CI failing before?17:23
*** jpich has quit IRC17:43
*** Kevin_Zheng has quit IRC17:46
*** shardy has quit IRC17:54
*** haint has quit IRC18:04
*** bobh has quit IRC18:13
*** nguyentrihai has joined #openstack-mistral18:14
*** thrash|biab is now known as thrash18:16
*** bobh has joined #openstack-mistral18:21
*** bobh has quit IRC18:25
*** bobh has joined #openstack-mistral18:31
*** gkadam has joined #openstack-mistral18:32
*** bobh has quit IRC18:36
*** dprince has joined #openstack-mistral18:36
*** bobh has joined #openstack-mistral18:45
*** bobh has quit IRC18:50
*** bobh has joined #openstack-mistral18:53
*** bobh has quit IRC18:58
openstackgerritBrad P. Crochet proposed openstack/mistral master: DNM: TEST Migrate the jobs to native Zuul v3 format  https://review.openstack.org/50942819:02
*** oidgar has quit IRC19:04
*** harlowja has quit IRC19:32
*** bobh has joined #openstack-mistral19:32
*** harlowja has joined #openstack-mistral19:36
*** bobh has quit IRC19:38
*** bobh has joined #openstack-mistral20:01
*** bobh has quit IRC20:06
*** gkadam has quit IRC20:14
openstackgerritBrad P. Crochet proposed openstack/mistral master: Migrate the jobs to native Zuul v3 format  https://review.openstack.org/50942820:15
*** dprince has quit IRC20:21
*** kong has quit IRC20:31
*** zhenguo has quit IRC20:31
*** rbrady has quit IRC20:31
*** FL1SK has quit IRC20:31
*** bobh has joined #openstack-mistral20:32
*** bobh has quit IRC20:38
*** kong has joined #openstack-mistral20:47
*** zhenguo has joined #openstack-mistral20:47
*** rbrady has joined #openstack-mistral20:47
*** FL1SK has joined #openstack-mistral20:47
*** livelace-link has quit IRC20:49
*** livelace-link has joined #openstack-mistral20:51
*** bobh has joined #openstack-mistral20:52
*** livelace-link has quit IRC20:53
*** jkilpatr has quit IRC20:54
*** livelace-link has joined #openstack-mistral20:54
*** livelace-link has quit IRC20:56
*** bobh has quit IRC20:56
*** livelace-link has joined #openstack-mistral20:57
*** livelace-link has quit IRC20:59
*** harlowja has quit IRC21:00
*** harlowja has joined #openstack-mistral21:03
*** bobh has joined #openstack-mistral21:03
*** bobh has quit IRC21:07
*** bobh has joined #openstack-mistral21:11
*** bobh has quit IRC21:16
*** harlowja has quit IRC21:32
*** bobh has joined #openstack-mistral21:32
*** bobh has quit IRC21:37
*** AlexeyAbashkin has joined #openstack-mistral21:38
*** openstackgerrit has quit IRC21:38
*** breton has quit IRC21:38
*** bobh has joined #openstack-mistral21:52
*** bobh has quit IRC21:53
*** bobh has joined #openstack-mistral21:53
*** openstackgerrit has joined #openstack-mistral22:01
*** breton has joined #openstack-mistral22:01
*** jkilpatr has joined #openstack-mistral22:21
*** harlowja has joined #openstack-mistral22:36
*** AlexeyAbashkin has quit IRC22:43
*** igormarnat has quit IRC23:06
*** igormarnat has joined #openstack-mistral23:06
*** openstack has joined #openstack-mistral23:29
*** ChanServ sets mode: +o openstack23:29

Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!