| *** harlowja_ has joined #openstack-watcher | 00:27 | |
| *** harlowja has quit IRC | 00:30 | |
| *** wootehfoot has quit IRC | 02:55 | |
| *** thorst has quit IRC | 02:56 | |
| *** thorst has joined #openstack-watcher | 02:57 | |
| *** thorst has quit IRC | 03:02 | |
| *** thorst has joined #openstack-watcher | 03:17 | |
| *** cjvolzka has joined #openstack-watcher | 03:38 | |
| *** cjvolzka has quit IRC | 03:41 | |
| *** Kevin_Zheng has joined #openstack-watcher | 03:57 | |
| *** thorst has quit IRC | 04:04 | |
| *** thorst has joined #openstack-watcher | 04:04 | |
| *** thorst has quit IRC | 04:13 | |
| *** thorst has joined #openstack-watcher | 05:10 | |
| *** thorst has quit IRC | 05:18 | |
| *** thorst has joined #openstack-watcher | 06:16 | |
| *** thorst has quit IRC | 06:23 | |
| *** jwcroppe has quit IRC | 06:38 | |
| *** thorst has joined #openstack-watcher | 07:21 | |
| *** thorst has quit IRC | 07:28 | |
| *** vincentfrancoise has joined #openstack-watcher | 08:23 | |
| *** thorst has joined #openstack-watcher | 08:26 | |
| openstackgerrit | Merged openstack/watcher: Remove useless Meta-Action https://review.openstack.org/263727 | 08:30 |
|---|---|---|
| *** dtardivel has joined #openstack-watcher | 08:33 | |
| *** thorst has quit IRC | 08:33 | |
| *** thorst has joined #openstack-watcher | 09:31 | |
| *** thorst has quit IRC | 09:38 | |
| *** jwcroppe has joined #openstack-watcher | 09:39 | |
| *** jwcroppe has quit IRC | 09:43 | |
| openstackgerrit | Jean-Emile DARTOIS proposed openstack/watcher: Provide a more dynamic Actions management solution https://review.openstack.org/263715 | 09:48 |
| *** thorst has joined #openstack-watcher | 10:36 | |
| *** thorst has quit IRC | 10:43 | |
| *** Biswa has joined #openstack-watcher | 11:00 | |
| openstackgerrit | Jean-Emile DARTOIS proposed openstack/watcher: Provide a more dynamic Actions management solution https://review.openstack.org/263715 | 11:16 |
| *** vincentfrancoise has quit IRC | 11:19 | |
| openstackgerrit | Jean-Emile DARTOIS proposed openstack/watcher: Provide a more dynamic Actions management solution https://review.openstack.org/263715 | 11:22 |
| *** jwcroppe has joined #openstack-watcher | 11:24 | |
| openstackgerrit | Jean-Emile DARTOIS proposed openstack/watcher: Provide a more dynamic Actions management solution https://review.openstack.org/263715 | 11:38 |
| openstackgerrit | Jean-Emile DARTOIS proposed openstack/watcher: Provide a more dynamic Actions management solution https://review.openstack.org/263715 | 11:39 |
| openstackgerrit | Jean-Emile DARTOIS proposed openstack/watcher: Provide a more dynamic Actions management solution https://review.openstack.org/263715 | 11:40 |
| *** thorst has joined #openstack-watcher | 11:41 | |
| *** jwcroppe has quit IRC | 11:42 | |
| openstackgerrit | Jean-Emile DARTOIS proposed openstack/watcher: Provide a more dynamic Actions management solution https://review.openstack.org/264142 | 11:47 |
| *** thorst has quit IRC | 11:48 | |
| openstackgerrit | Jean-Emile DARTOIS proposed openstack/watcher: Provide a more dynamic Actions management solution https://review.openstack.org/264142 | 12:54 |
| *** vincentfrancoise has joined #openstack-watcher | 12:59 | |
| *** thorst has joined #openstack-watcher | 13:02 | |
| openstackgerrit | Jean-Emile DARTOIS proposed openstack/watcher: Provide a more dynamic Actions management solution https://review.openstack.org/263715 | 13:28 |
| Biswa | https://bugs.launchpad.net/watcher/+bug/1525854 error: invalid command 'ostestr' despite virtualenvs have been recreated after adding new requiremen | 13:36 |
| openstack | Launchpad bug 1525854 in watcher "Use os-testr instead of testr" [Wishlist,New] - Assigned to biswajeeban (biswajeeban) | 13:36 |
| *** jwcroppe has joined #openstack-watcher | 13:38 | |
| *** wootehfoot has joined #openstack-watcher | 13:39 | |
| vincentfrancoise | Hi Biswa | 13:40 |
| openstackgerrit | Jean-Emile DARTOIS proposed openstack/watcher: Provide a more dynamic Actions management solution https://review.openstack.org/264175 | 13:40 |
| vincentfrancoise | I was actually making a comment on your bug and I just submitted it. So you can go have a look | 13:40 |
| *** alexstav has joined #openstack-watcher | 13:42 | |
| Biswa | Hi vincent | 13:46 |
| *** jwcroppe has quit IRC | 13:50 | |
| *** brunograz has joined #openstack-watcher | 13:51 | |
| vincentfrancoise | Did you have a look at the patchset link I've put? Did it help? | 13:51 |
| *** alexstav has quit IRC | 13:52 | |
| Biswa | yes I have lookd at the links you provided | 13:52 |
| *** alexstav has joined #openstack-watcher | 13:53 | |
| Biswa | oh just recently you have sent me a link .Okay let me go through it once again | 13:54 |
| Biswa | @vincent I will do the needful as you suggested. Thank you very much for your kind guidance | 13:57 |
| vincentfrancoise | Biswa: you're welcome | 13:58 |
| sballe | morning | 13:59 |
| *** bzhou has joined #openstack-watcher | 13:59 | |
| *** jwcroppe has joined #openstack-watcher | 13:59 | |
| acabot | morning | 14:00 |
| Biswa | @Viincent .I am just beginning with openstack to learn , to explore and to contribute . So sometimes I may surely appear very stupid. hope you will guide me the way | 14:00 |
| *** vtech has joined #openstack-watcher | 14:01 | |
| *** seanmurphy has joined #openstack-watcher | 14:01 | |
| openstackgerrit | Jean-Emile DARTOIS proposed openstack/watcher: Provide a more dynamic Actions management solution https://review.openstack.org/263715 | 14:04 |
| *** vmahe has joined #openstack-watcher | 14:05 | |
| *** cjvolzka has joined #openstack-watcher | 14:07 | |
| *** wootehfoot has quit IRC | 14:41 | |
| *** Biswa has quit IRC | 14:48 | |
| alexstav | jed56: what code removing you were talking about? :) | 15:08 |
| *** bzhou has quit IRC | 15:09 | |
| jed56 | https://www.irccloud.com/pastebin/GuGKUHlO/ | 15:10 |
| alexstav | jed56: but there are another problem in applier/mapping/default.py | 15:14 |
| alexstav | https://github.com/openstack/watcher/blob/master/watcher/applier/mapping/default.py#L37 | 15:14 |
| alexstav | jed56: why COLD_MIGRATE, but not LIVE_MIGRATE? | 15:15 |
| jed56 | I agree but the mapping will also be removed | 15:16 |
| alexstav | :D | 15:16 |
| jed56 | This is urgent ? | 15:16 |
| jed56 | if yes you can make a fix | 15:16 |
| alexstav | No :) | 15:16 |
| alexstav | But it doesn't work on my stands without this fix. But I don't think it is urgent. | 15:17 |
| alexstav | Thanks for the explanation. | 15:17 |
| jed56 | you are welcome! | 15:22 |
| *** srwilkers has joined #openstack-watcher | 15:27 | |
| edleafe | Just had a discussion in #openstack-nova regarding calling the scheduler when moving instances | 15:33 |
| edleafe | The clear answer to using RPC was "no". | 15:34 |
| edleafe | there was no alternative, though | 15:34 |
| edleafe | There is a spec for adding a check_destination() call to migrations, though, which will run the target host through the scheduler to verify that it is valid | 15:35 |
| edleafe | http://specs.openstack.org/openstack/nova-specs/specs/mitaka/approved/check-destination-on-migrations.html | 15:35 |
| edleafe | So probably the only option for us now is to select a host and call the migration. Eventually nova will be improved to disallow that if the host would not have accepted that instance originally | 15:36 |
| edleafe | But for now we are limited by what nova can do. | 15:36 |
| tpeoples | i think the notion of only being able to choose a host and have nova disallow it would be very inefficient | 15:39 |
| tpeoples | jwcroppe: ^ see above, and see nova channel | 15:39 |
| edleafe | tpeoples: yeah, but that's slightly better than choosing a host that violates the original request | 15:46 |
| tpeoples | edleafe: true. regardless, either way looks like we have some work to do with the nova scheduler that we'll need to try and land in N to make sure we can get something that fits our needs | 15:51 |
| *** vincentfrancoise has left #openstack-watcher | 15:57 | |
| *** vincentfrancoise has joined #openstack-watcher | 15:59 | |
| openstackgerrit | Taylor Peoples proposed openstack/watcher: Implement DevStack plugin https://review.openstack.org/254584 | 16:05 |
| jwcroppe | trying to follow... nova supports untargeted migration, which does call nova filters to ensure they 'pass' ? | 16:05 |
| edleafe | jwcroppe: nope. Currently nova blindly migrates to whatever host you tell it to | 16:06 |
| jwcroppe | edleafe: yes, targeted... but it also has untargeted via an extension IIRC | 16:06 |
| edleafe | the spec involves persisting the request so that nova can re-run the filters to make sure that the target is OK | 16:07 |
| jwcroppe | edleafe: i.e., if you select host X, it will try to do the migration. But I'm saying that in nova, if you do: nova live-migration <vm>, it picks a host for you | 16:07 |
| edleafe | currently nova doesn't persist the original request, so things such as (anti)affinity, etc., are lost | 16:07 |
| edleafe | jwcroppe: yes, based solely on "does it fit" | 16:08 |
| jwcroppe | edleafe: yeah, and any other filters that may be enabled though? And you're right on the affinity - we had to actually create our own proprietary filter to go back to the DB to check those | 16:09 |
| edleafe | jwcroppe: all that "extra" stuff, like PCI, NUMA, etc., is not persisted now, so that causes problems. | 16:09 |
| jwcroppe | maybe an intermediate compromise here is to allow Watch to provide a filter extension point and folks can insert their own filter logic however they want, until Nova has an 'officially accepted' way to do this | 16:10 |
| edleafe | This patch is adding the persistence: https://review.openstack.org/#/c/258628/ | 16:10 |
| edleafe | you can see the patch series involved | 16:10 |
| jwcroppe | cool, I'll look. | 16:10 |
| jwcroppe | My main concern is that we provide some mechanism for folks to allow Watcher to easily filter candidate hosts for migration if they know they're not acceptable | 16:11 |
| *** alexstav has quit IRC | 16:48 | |
| *** jwcroppe has quit IRC | 16:48 | |
| *** alexstav has joined #openstack-watcher | 16:55 | |
| *** vtech has quit IRC | 17:07 | |
| *** seanmurphy has quit IRC | 17:15 | |
| *** vmahe has quit IRC | 17:27 | |
| openstackgerrit | Alexandr Stavitskiy proposed openstack/watcher: Making configurable [user|project]_domain_id https://review.openstack.org/264285 | 17:29 |
| *** vtech has joined #openstack-watcher | 17:31 | |
| alexstav | Guys, check please my patchset, that's my first experience working with gerrit. Is all ok? :) | 17:31 |
| *** jwcroppe has joined #openstack-watcher | 17:48 | |
| *** alexstav has quit IRC | 17:57 | |
| openstackgerrit | Steve Wilkerson proposed openstack/watcher: Use graduated checked for doc formatting https://review.openstack.org/264294 | 18:18 |
| *** harlowja_ has quit IRC | 19:18 | |
| *** harlowja has joined #openstack-watcher | 19:19 | |
| *** seanmurphy has joined #openstack-watcher | 19:26 | |
| *** seanmurphy has quit IRC | 19:26 | |
| *** wootehfoot has joined #openstack-watcher | 20:00 | |
| *** vtech has quit IRC | 20:47 | |
| *** srwilkers has quit IRC | 20:53 | |
| *** alexstav has joined #openstack-watcher | 21:43 | |
| alexstav | tpeoples: Hi! How can I write comments in gerrit? As I see, a can write only Drafts. Do you see them? :) | 21:44 |
| tpeoples | alexstav: hey, you have to hit the reply button from the main page of the specific review | 21:47 |
| tpeoples | and then "post" | 21:47 |
| *** dtardivel has quit IRC | 21:47 | |
| alexstav | tpeoples: I clicked on your yellow comment, than push 'reply'. It creates 'draft'. Than I can only 'Save' and 'Discard' it. | 21:49 |
| tpeoples | alexstav: save that comment. then go back to the main page of the review and click the reply button | 21:50 |
| openstackgerrit | Alexandr Stavitskiy proposed openstack/watcher: Make [user|project]_domain_id configurable https://review.openstack.org/264285 | 21:51 |
| alexstav | tpeoples: awesome! Thanks a lot! :) | 21:51 |
| tpeoples | alexstav: np, it doesn't look like your comments got published though, only an edited patch set | 21:53 |
| alexstav | tpeoples: hm. I see only one 'reply' button at top of patch page. A clicked on it, then 'post' and nothing happens. | 21:56 |
| tpeoples | alexstav: http://i.imgur.com/GybOEG3.png | 22:01 |
| tpeoples | if you don't have any comments then post probably won't do anything | 22:02 |
| *** Kevin_Zheng_ has joined #openstack-watcher | 22:04 | |
| *** Kevin_Zheng has quit IRC | 22:04 | |
| *** Kevin_Zheng_ is now known as Kevin_Zheng | 22:04 | |
| *** junjie has quit IRC | 22:05 | |
| *** junjie has joined #openstack-watcher | 22:07 | |
| alexstav | tpeoples: Thanks a lot again :) The problem was, that I edited patch firstly. | 22:10 |
| alexstav | Ana now my comments stuck in first patch version :D | 22:10 |
| tpeoples | alexstav: there is a test that you'll need to fix with your patch, that's why jenkins is giving it a -1 | 22:25 |
| tpeoples | watcher.tests.common.test_keystone.TestKeystone.test_get_credentials | 22:25 |
| tpeoples | need to change the expected values from *_name to *_id | 22:25 |
| alexstav | tpeoples: yep, I have already fix that. Now the problem is how to push new file there. :D | 22:26 |
| alexstav | To gerrit | 22:26 |
| tpeoples | did you amend your commit locally already? | 22:26 |
| alexstav | No. I should change first commit? | 22:27 |
| tpeoples | well, tbh i have never used the edit patchset feature in the new gerrit like you did yet. you may need to pull down PS2 unless you have that locally? | 22:28 |
| tpeoples | there's a link provided under "Download" on gerrit | 22:29 |
| tpeoples | s/link/command | 22:29 |
| alexstav | I amend commit. Maybe I should try to pull? :) | 22:31 |
| alexstav | push | 22:31 |
| tpeoples | does your local commit have the commit message changes? | 22:31 |
| tpeoples | see http://docs.openstack.org/infra/manual/developers.html#updating-a-change | 22:31 |
| alexstav | Maybe I can remove PS2 in the gerrit and push it from local repo? | 22:34 |
| tpeoples | hmm i don't think gerrit allows you to remove a PS | 22:35 |
| openstackgerrit | Alexandr Stavitskiy proposed openstack/watcher: Make [user|project]_domain_id configurable https://review.openstack.org/264285 | 22:35 |
| tpeoples | that looks like it worked | 22:36 |
| * tpeoples notes that SERVERS_CONSOLIDATION is still used in some places when he thinks it should be BASIC_CONSOLIDATION now | 22:37 | |
| alexstav | tpeoples: thx a lot. I'm sorry for stupid questions :) | 22:37 |
| tpeoples | alexstav: thanks for helping | 22:40 |
| *** alexstav has quit IRC | 22:42 | |
| *** alexstav has joined #openstack-watcher | 22:43 | |
| *** alexstav has quit IRC | 22:48 | |
| *** thorst has quit IRC | 22:54 | |
| *** cjvolzka has quit IRC | 23:08 | |
| *** thorst has joined #openstack-watcher | 23:40 | |
| *** alexstav has joined #openstack-watcher | 23:44 | |
| *** thorst has quit IRC | 23:45 | |
| *** alexstav has quit IRC | 23:50 | |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!
