| *** dtardivel has quit IRC | 01:08 | |
| *** jwcroppe has joined #openstack-watcher | 01:54 | |
| *** thorst has joined #openstack-watcher | 04:16 | |
| *** thorst has quit IRC | 04:20 | |
| *** thorst has joined #openstack-watcher | 04:20 | |
| *** thorst has quit IRC | 04:20 | |
| *** thorst has joined #openstack-watcher | 04:21 | |
| *** thorst has quit IRC | 04:30 | |
| *** jwcroppe has quit IRC | 05:10 | |
| *** thorst has joined #openstack-watcher | 05:27 | |
| *** thorst has quit IRC | 05:34 | |
| *** openstack has joined #openstack-watcher | 13:20 | |
| vincentfrancoise | Hi seanmurphy | 13:35 |
|---|---|---|
| seanmurphy | hi vincentfrancoise | 13:35 |
| vincentfrancoise | I just had a look at the work you guys did and it looks good :) | 13:36 |
| seanmurphy | great - thanks | 13:36 |
| seanmurphy | i see you’re putting some comments on some commits | 13:36 |
| vincentfrancoise | I made some quick comments | 13:37 |
| seanmurphy | thanks | 13:37 |
| vincentfrancoise | not really on the strategy itself though | 13:37 |
| seanmurphy | ok | 13:37 |
| seanmurphy | our modus is to work on our own github for now | 13:38 |
| seanmurphy | and we will submit a version for review soon | 13:38 |
| seanmurphy | end of this week or early next week | 13:38 |
| seanmurphy | but if you would prefer an alternative process, we can do that too | 13:39 |
| seanmurphy | we just don’t want to spam the system with code which is not ready yet | 13:39 |
| vincentfrancoise | The only thing you have to think about doing it this way is the size of the patchset when you'll have to switch onto gerrit | 13:39 |
| vincentfrancoise | so it doesn't get to big, but I don't think it will | 13:40 |
| seanmurphy | ok - no it will not be so big i think | 13:48 |
| *** acabot_ has joined #openstack-watcher | 13:52 | |
| *** acabot__ has joined #openstack-watcher | 13:59 | |
| acabot__ | weekly meeting will start on #openstack-meeting-4 | 13:59 |
| jed56 | Hi | 14:00 |
| *** brunograz has quit IRC | 14:01 | |
| *** bruno_ has joined #openstack-watcher | 14:01 | |
| *** bruno_ is now known as brunograz | 14:01 | |
| *** acabot_ has quit IRC | 14:02 | |
| *** vtech_ has quit IRC | 14:09 | |
| *** seanmurphy has quit IRC | 14:18 | |
| *** acabot__ has quit IRC | 14:55 | |
| openstackgerrit | daniel.pawlik proposed openstack/watcher: Added support for live migration on non-shared storage https://review.openstack.org/284164 | 15:03 |
| *** tkaczynski has quit IRC | 15:03 | |
| *** acabot__ has joined #openstack-watcher | 15:04 | |
| *** acabot__ has quit IRC | 15:06 | |
| *** acabot__ has joined #openstack-watcher | 15:06 | |
| *** acabot__ has quit IRC | 15:08 | |
| *** brunograz has left #openstack-watcher | 15:20 | |
| *** pav0 has joined #openstack-watcher | 15:38 | |
| *** lugh has quit IRC | 15:50 | |
| pav0 | Hello, I have made small patch for watcher applier that support instances which are not on shared storage: https://bugs.launchpad.net/watcher/+bug/1549307 . If you can, pls review my code https://review.openstack.org/#/c/284164 | 16:02 |
| openstack | Launchpad bug 1549307 in watcher "Live migration of instances on non-shared storage is not supported" [Undecided,In progress] - Assigned to daniel.pawlik (daniel-pawlik) | 16:02 |
| vincentfrancoise | Hi pav0 | 16:08 |
| vincentfrancoise | thanks for contributing | 16:08 |
| vincentfrancoise | I'll have a look at it | 16:09 |
| pav0 | ok :) | 16:09 |
| vincentfrancoise | Ok so for what I can see | 16:13 |
| vincentfrancoise | you are doing some work to support the block_migration option in live migrations | 16:14 |
| vincentfrancoise | Following the comments you made the other day on this channel we had some discussion on the issue | 16:14 |
| pav0 | yap | 16:16 |
| vincentfrancoise | and I believe there is a blueprint that is about to be made on the subject to address the problem of configurable actions | 16:16 |
| vincentfrancoise | Basically, we need to provide a way to configure an action regarding at the platform level | 16:17 |
| vincentfrancoise | so we can set the migration action with a parameter like | 16:17 |
| vincentfrancoise | migration_type=live at the strategy level with block_migration=True/False in the watcher.conf file | 16:18 |
| pav0 | But I think, sometime block migration is not important | 16:19 |
| pav0 | Better way will be add in watcher.conf file new parameter: for example: only_block_migrate = True/False | 16:20 |
| pav0 | or something like this | 16:20 |
| vincentfrancoise | yeah that could be it | 16:20 |
| vincentfrancoise | the exact specs still have to be written down | 16:20 |
| vincentfrancoise | but in the meantime we can do it the way you did | 16:21 |
| vincentfrancoise | It's just that this will only be a temporary solution | 16:21 |
| pav0 | I understand | 16:22 |
| pav0 | It will be much easier, if nova knows if instance is on shared storage or not and send some information. But nova didn't know it, only compute host knows. | 16:23 |
| pav0 | So should I make new parameter in watcher.conf etc. or leave this solution and make some unit tests? | 16:25 |
| vincentfrancoise | Well, I guess you can leave it like this and write the tests. The config params mechanism we need to add will have to be generic so if you want to do it, you would have to write a full BP+specs to make sure everyone agree | 16:28 |
| vincentfrancoise | we'll see in the review if someone else has a different view on the matter anyway | 16:30 |
| pav0 | Ok | 16:31 |
| openstackgerrit | Merged openstack/watcher: Useless return statement in validate_sort_dir https://review.openstack.org/281949 | 16:49 |
| *** thorst has quit IRC | 17:02 | |
| *** thorst has joined #openstack-watcher | 17:03 | |
| *** thorst has quit IRC | 17:03 | |
| *** vincentfrancoise has quit IRC | 17:36 | |
| *** thorst has joined #openstack-watcher | 18:13 | |
| *** jwcroppe has quit IRC | 18:26 | |
| *** Guest80 has joined #openstack-watcher | 18:54 | |
| *** Guest80 is now known as blmannin | 18:54 | |
| *** blmannin has left #openstack-watcher | 18:54 | |
| *** jwcroppe has joined #openstack-watcher | 19:13 | |
| *** jwcroppe has quit IRC | 19:25 | |
| *** acabot_ has joined #openstack-watcher | 20:00 | |
| *** wootehfoot has joined #openstack-watcher | 20:03 | |
| *** dtardivel has quit IRC | 20:38 | |
| *** jwcroppe has joined #openstack-watcher | 20:54 | |
| *** thorst is now known as thorst_afk | 21:10 | |
| *** Guest80 has joined #openstack-watcher | 21:39 | |
| *** jwcroppe has quit IRC | 21:57 | |
| *** acabot_ has quit IRC | 22:24 | |
| *** thorst_afk is now known as thorst | 23:01 | |
| *** wootehfoot has quit IRC | 23:02 | |
| *** mestery has quit IRC | 23:30 | |
| *** Guest80 has quit IRC | 23:34 | |
| *** mestery has joined #openstack-watcher | 23:35 | |
| *** thorst has quit IRC | 23:48 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!