| *** zhurong has joined #openstack-mistral | 00:23 | |
| *** harlowja has quit IRC | 00:26 | |
| *** bobh has quit IRC | 00:28 | |
| *** zhurong has quit IRC | 01:26 | |
| *** bobh has joined #openstack-mistral | 02:13 | |
| *** bobh has quit IRC | 02:34 | |
| *** hardikjasani has joined #openstack-mistral | 04:37 | |
| *** jaosorior has quit IRC | 05:26 | |
| *** apetrich has joined #openstack-mistral | 06:01 | |
| *** jaosorior has joined #openstack-mistral | 06:05 | |
| openstackgerrit | Pierre Gaxatte proposed openstack/mistral master: Increase delayed_calls_v2.auth_context https://review.openstack.org/589377 | 06:41 |
|---|---|---|
| *** gkadam has joined #openstack-mistral | 07:04 | |
| *** xavierhardy has joined #openstack-mistral | 07:14 | |
| *** xavierhardy has quit IRC | 07:19 | |
| openstackgerrit | Pierre Gaxatte proposed openstack/mistral master: Increase delayed_calls_v2.auth_context https://review.openstack.org/589377 | 07:43 |
| *** jaosorior has quit IRC | 07:49 | |
| *** apetrich has quit IRC | 07:56 | |
| *** jaosorior has joined #openstack-mistral | 08:02 | |
| *** jaosorior has quit IRC | 09:02 | |
| *** josecastroleon has joined #openstack-mistral | 09:15 | |
| *** jaosorior has joined #openstack-mistral | 10:46 | |
| *** apetrich has joined #openstack-mistral | 11:53 | |
| *** jaosorior has quit IRC | 11:59 | |
| *** jaosorior has joined #openstack-mistral | 12:19 | |
| *** jaosorior has quit IRC | 12:27 | |
| *** thrash|g0ne is now known as thrash | 12:33 | |
| *** rbrady has joined #openstack-mistral | 12:36 | |
| *** jaosorior has joined #openstack-mistral | 12:42 | |
| *** rakhmerov has quit IRC | 12:50 | |
| *** weshay has quit IRC | 12:50 | |
| *** bobh has joined #openstack-mistral | 12:54 | |
| *** hardikjasani has quit IRC | 13:05 | |
| *** thrash is now known as thrash|biab | 14:14 | |
| *** jrist has quit IRC | 14:15 | |
| therve | d0ugal: I think there is an issue with https://review.openstack.org/#/c/582482/ | 14:25 |
| d0ugal | darn | 14:26 |
| d0ugal | therve: with the migration? | 14:27 |
| therve | d0ugal: No I don't think so | 14:27 |
| therve | Not sure what that is, but maybe just noise | 14:27 |
| therve | Like the cron is running before the db is updated or something | 14:27 |
| d0ugal | oh, weird | 14:27 |
| *** gkadam_ has joined #openstack-mistral | 14:27 | |
| *** jrist has joined #openstack-mistral | 14:27 | |
| d0ugal | therve: so cron triggers are failing? | 14:28 |
| therve | d0ugal: Right but possibly intermittently | 14:28 |
| therve | The real failure is something else | 14:28 |
| *** gkadam has quit IRC | 14:29 | |
| therve | The heartbeat error happens on successful runs | 14:30 |
| d0ugal | Right, a race condition with the upgrade seems likely | 14:30 |
| d0ugal | Is there a bug for this? | 14:30 |
| therve | Not yet no | 14:33 |
| *** josecastroleon has quit IRC | 14:35 | |
| *** thrash|biab is now known as thrash | 14:44 | |
| therve | d0ugal: I may be wrong, but I thin the issue is that '' != None | 14:47 |
| therve | Surprising right | 14:47 |
| d0ugal | therve: Which file? | 14:47 |
| therve | d0ugal: Which file? | 14:48 |
| therve | Which which file? :) | 14:48 |
| d0ugal | hrm | 14:48 |
| d0ugal | Where are you seeing '' != None? | 14:48 |
| therve | d0ugal: So, we list workbooks, but we delete it, it's not there anymore | 14:49 |
| therve | The workbook was create before namespace existed | 14:49 |
| therve | So it's in the db, it's null | 14:49 |
| d0ugal | gotcha | 14:49 |
| therve | But we query '' as namespace, so we can't find it | 14:49 |
| therve | Ah | 14:49 |
| therve | d0ugal: https://github.com/openstack/mistral/blob/master/mistral/db/sqlalchemy/migration/alembic_migrations/versions/022_namespace_support.py#L111 | 14:49 |
| therve | We miss that in the 28 migration | 14:49 |
| d0ugal | Right | 14:50 |
| d0ugal | Doing that in a new migration is more tricky, since some namespaces will be set | 14:51 |
| d0ugal | This is a bit of a pain now :) | 14:51 |
| d0ugal | therve: I gotta head out shortly - but I can look into this more tomorrow (if you don't beat me to it) | 14:51 |
| therve | d0ugal: Sure pushing a patch | 14:52 |
| *** bobh has quit IRC | 14:56 | |
| d0ugal | Thanks | 14:58 |
| openstackgerrit | Thomas Herve proposed openstack/mistral master: Update workbook namespace to '' on migration https://review.openstack.org/589510 | 14:58 |
| *** jaosorior has quit IRC | 15:01 | |
| *** bobh has joined #openstack-mistral | 15:17 | |
| *** bobh has quit IRC | 15:17 | |
| openstackgerrit | Chandan Kumar proposed openstack/mistral master: Add tripleo-ci-centos-7-undercloud-containers job https://review.openstack.org/589520 | 15:37 |
| *** bobh has joined #openstack-mistral | 15:45 | |
| *** rbrady is now known as rbrady-afk | 16:01 | |
| d0ugal | therve: Is it generally considered okay to change migrations? | 16:17 |
| d0ugal | therve: I know it isn't fully released - but I'm wondering what that'll do to anyone running master | 16:18 |
| d0ugal | ... but I guess they will also have the issue you found | 16:18 |
| d0ugal | so ¯\_(ツ)_/¯ | 16:18 |
| therve | d0ugal: It depends on your policy :) | 16:19 |
| d0ugal | therve: I'm not sure I have a policy. | 16:19 |
| therve | master chasing exists, but... here the migration is broken, so I don't know | 16:19 |
| d0ugal | Can you re-apply a migration in sqlalchemy somehow? | 16:20 |
| d0ugal | Wondering if we can at least provide a workaround for anyone that hits the problem | 16:20 |
| d0ugal | I'll ponder it and see what the other cores think. Thanks for the quick patch :) | 16:21 |
| d0ugal | Right, I'm off again :) | 16:21 |
| *** gkadam_ has quit IRC | 16:38 | |
| *** harlowja has joined #openstack-mistral | 17:31 | |
| *** harlowja has quit IRC | 17:43 | |
| *** rbrady-afk is now known as rbrady | 18:05 | |
| *** apetrich has quit IRC | 18:35 | |
| openstackgerrit | Merged openstack/mistral master: Add a debug log for the webhook publisher https://review.openstack.org/587001 | 18:46 |
| *** openstackgerrit has quit IRC | 18:49 | |
| *** bobh has quit IRC | 19:01 | |
| *** bobh has joined #openstack-mistral | 19:18 | |
| *** bobh has quit IRC | 19:23 | |
| *** apetrich has joined #openstack-mistral | 19:33 | |
| *** bobh has joined #openstack-mistral | 19:59 | |
| *** apetrich has quit IRC | 20:45 | |
| *** thrash is now known as thrash|g0ne | 20:51 | |
| *** rbrady has quit IRC | 21:02 | |
| *** toure is now known as toure|gone | 22:10 | |
| *** bobh has quit IRC | 22:12 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!