Wednesday, 2018-08-08

*** gkadam_ has joined #openstack-mistral02:24
*** gkadam__ has joined #openstack-mistral02:26
*** gkadam_ has quit IRC02:29
*** zhurong has joined #openstack-mistral03:00
*** jaosorior has joined #openstack-mistral03:59
*** zhurong has quit IRC04:20
*** jaosorior has quit IRC05:27
*** gkadam__ has quit IRC05:42
*** gkadam has joined #openstack-mistral05:44
*** hardikjasani has joined #openstack-mistral05:53
*** zhurong has joined #openstack-mistral05:58
*** apetrich has joined #openstack-mistral06:08
*** gkadam has quit IRC06:09
*** gkadam has joined #openstack-mistral06:24
*** jaosorior has joined #openstack-mistral06:44
*** jaosorior has quit IRC07:38
*** jaosorior has joined #openstack-mistral07:38
*** jaosorior has quit IRC07:41
*** jaosorior has joined #openstack-mistral07:41
*** jaosorior has quit IRC07:42
*** jaosorior has joined #openstack-mistral07:43
*** zhurong has quit IRC08:26
*** openstackgerrit has joined #openstack-mistral08:35
openstackgerritThomas Herve proposed openstack/mistral master: Update workbook namespace to '' on migration  https://review.openstack.org/58951008:35
d0ugaltherve: What is somebody is already using migration 28 and has some namespaces in their workbooks...08:47
d0ugalWonder if we should set it to an empty string only when it is None08:47
therveYou're killing me here :)08:47
d0ugalYeah, I know08:47
d0ugalThis is why I was okay with updating the old migration, because it avoided this issue08:48
openstackgerritThomas Herve proposed openstack/mistral master: Update workbook namespace to '' on migration  https://review.openstack.org/58951008:49
therveAh there is a comment08:51
d0ugaloh yeah, thanks hardikjasani :)08:51
openstackgerritThomas Herve proposed openstack/mistral master: Update workbook namespace to '' on migration  https://review.openstack.org/58951008:51
therveI think I need to ignore the pep8 error08:52
thervesigh08:52
openstackgerritThomas Herve proposed openstack/mistral master: Update workbook namespace to '' on migration  https://review.openstack.org/58951008:54
therveOK! Hopefully this is the one08:55
d0ugaltherve: heh, sqlalchemy doesn't so "is None"?08:55
therved0ugal: Yeah I think something dumb happen when you do that08:56
d0ugalGood to know.08:56
d0ugalLGTM.08:56
d0ugaltherve: is there a way to have tripleo ci run the upgrade against this?08:57
therved0ugal: Maybe if we make https://review.openstack.org/#/c/589520/ depends on it?08:57
therveAh no it doesn't do an upgrade08:57
therveI'll try with a tht patch09:01
openstackgerritRenat Akhmerov proposed openstack/mistral master: WIP: designing a new scheduler  https://review.openstack.org/58901509:12
d0ugaltherve: http://logs.openstack.org/10/589510/5/check/mistral-tox-unit-mysql/bf62d0a/job-output.txt.gz09:18
d0ugalhttp://logs.openstack.org/10/589510/5/check/mistral-tox-unit-mysql/bf62d0a/job-output.txt.gz#_2018-08-08_09_03_06_35219509:19
d0ugalhmm09:19
therveAh I'm missing a c09:21
openstackgerritThomas Herve proposed openstack/mistral master: Update workbook namespace to '' on migration  https://review.openstack.org/58951009:22
d0ugaloh yeah09:22
d0ugalmy sqlalchemy is rustier than I thought.09:22
d0ugalCI results look promising so far :)10:11
*** jaosorior has quit IRC10:58
*** jaosorior has joined #openstack-mistral10:59
openstackgerritChandan Kumar proposed openstack/mistral master: Add tripleo-ci-centos-7-undercloud-containers job  https://review.openstack.org/58952011:35
*** thrash|g0ne is now known as thrash12:35
*** toure|gone is now known as toure12:43
d0ugalOnce this patch lands: https://review.openstack.org/58951013:09
d0ugalI am going to start the Rocky RC release13:10
d0ugalunless I hear any objections :)13:10
d0ugalIs there anything else I should be waiting for?13:10
*** hardikjasani has quit IRC13:12
*** rbrady has joined #openstack-mistral13:12
therved0ugal: Wasn't able to verify it against tripleo :/13:24
d0ugaltherve: damn, so - fingers crossed? :)13:24
therveSee https://review.openstack.org/589765 it didn't use the change AFAICT13:24
therveYeah I guess :)13:24
d0ugalthrash: is this done? https://bugs.launchpad.net/mistral/+bug/172079713:27
openstackLaunchpad bug 1720797 in Mistral "Migrate Mistral CI jobs to new Zuul v3" [High,In progress] - Assigned to Brad P. Crochet (brad-9)13:27
thrashd0ugal: Yeah. I think we can mark that as complete.13:28
openstackgerritMerged openstack/python-mistralclient master: Add namespace support for workbooks to client  https://review.openstack.org/58531914:02
openstackgerritMerged openstack/mistral master: Update workbook namespace to '' on migration  https://review.openstack.org/58951014:36
*** gkadam has quit IRC14:43
d0ugalyay14:46
d0ugalRC1 on its way!14:46
*** bobh has joined #openstack-mistral14:47
openstackgerritDougal Matthews proposed openstack/mistral master: Use trusts by default in OpenStack action calls  https://review.openstack.org/57244815:01
*** gkadam has joined #openstack-mistral15:24
openstackgerritHuang.Xiangdong proposed openstack/python-mistralclient master: fix mistral client failure  https://review.openstack.org/58995415:56
*** thrash is now known as thrash|biab16:01
*** thrash|biab is now known as thrash16:46
*** gkadam has quit IRC17:01
*** gkadam has joined #openstack-mistral17:46
*** rbrady is now known as rbrady-afk17:52
*** apetrich has quit IRC18:46
*** apetrich has joined #openstack-mistral18:47
*** gkadam has quit IRC19:18
*** apetrich has quit IRC19:28
*** rbrady-afk has quit IRC20:21
*** apetrich has joined #openstack-mistral20:24
*** apetrich has quit IRC20:56
*** bobh has quit IRC21:41
*** jtomasek_ has joined #openstack-mistral22:54
*** jtomasek has quit IRC23:02

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