*** thorst_ has joined #openstack-watcher | 00:38 | |
*** thorst_ has quit IRC | 00:43 | |
*** zhurong has joined #openstack-watcher | 00:55 | |
*** thorst_ has joined #openstack-watcher | 00:59 | |
*** thorst_ has quit IRC | 01:03 | |
*** sferna1_ has quit IRC | 01:04 | |
*** thorst_ has joined #openstack-watcher | 01:04 | |
*** thorst_ has quit IRC | 01:04 | |
*** thorst_ has joined #openstack-watcher | 02:05 | |
openstackgerrit | Zhenyu Zheng proposed openstack/watcher: Should use glanceclient to get images https://review.openstack.org/414445 | 02:07 |
---|---|---|
openstackgerrit | Zhenyu Zheng proposed openstack/watcher: Enable coverage report in console output https://review.openstack.org/418640 | 02:07 |
*** thorst_ has quit IRC | 02:08 | |
openstackgerrit | licanwei proposed openstack/watcher: node.state should be node.status https://review.openstack.org/419716 | 02:21 |
openstackgerrit | licanwei proposed openstack/watcher: resolve KeyError exception https://review.openstack.org/419727 | 02:51 |
openstackgerrit | Jeremy Liu proposed openstack/python-watcherclient: Enable coverage report in console output https://review.openstack.org/419728 | 02:51 |
*** yuanying has quit IRC | 02:52 | |
openstackgerrit | Merged openstack/puppet-watcher: Allow python-memcache install from authtoken class https://review.openstack.org/419201 | 03:01 |
*** absubram has joined #openstack-watcher | 03:11 | |
*** diga has joined #openstack-watcher | 03:15 | |
*** absubram_ has joined #openstack-watcher | 03:18 | |
*** absubram has quit IRC | 03:19 | |
*** absubram_ is now known as absubram | 03:19 | |
*** thorst_ has joined #openstack-watcher | 03:35 | |
*** thorst_ has quit IRC | 03:37 | |
*** zhurong has quit IRC | 03:39 | |
*** yuanying has joined #openstack-watcher | 03:52 | |
*** atuly_ has quit IRC | 04:15 | |
*** sferna1_ has joined #openstack-watcher | 04:19 | |
*** atuly_ has joined #openstack-watcher | 04:29 | |
*** diga has quit IRC | 05:01 | |
*** atuly_ has quit IRC | 05:21 | |
*** atuly_ has joined #openstack-watcher | 05:33 | |
*** thorst_ has joined #openstack-watcher | 05:37 | |
*** adisky_ has joined #openstack-watcher | 05:38 | |
*** thorst_ has quit IRC | 05:42 | |
*** yuanying has quit IRC | 06:06 | |
*** yuanying has joined #openstack-watcher | 06:07 | |
*** yuanying has quit IRC | 06:11 | |
*** yuanying_ has joined #openstack-watcher | 06:11 | |
*** yuanying_ has quit IRC | 06:12 | |
*** yuanying has joined #openstack-watcher | 06:12 | |
*** yuanying has quit IRC | 06:17 | |
*** jwcroppe has quit IRC | 06:23 | |
openstackgerrit | Jeremy Liu proposed openstack/python-watcherclient: Remove unused files https://review.openstack.org/419782 | 06:41 |
*** zhurong has joined #openstack-watcher | 07:05 | |
*** yuanying has joined #openstack-watcher | 07:19 | |
*** hidekazu has joined #openstack-watcher | 07:25 | |
*** tkaczynski has quit IRC | 07:28 | |
*** adisky_ has quit IRC | 07:29 | |
*** adisky_ has joined #openstack-watcher | 07:36 | |
*** tkaczynski has joined #openstack-watcher | 07:36 | |
*** thorst_ has joined #openstack-watcher | 07:39 | |
*** thorst_ has quit IRC | 07:43 | |
*** alexchadin has joined #openstack-watcher | 08:10 | |
openstackgerrit | Jeremy Liu proposed openstack/python-watcherclient: Use keystoneauth instead of keystoneclient https://review.openstack.org/419806 | 08:17 |
*** vincentfrancoise has joined #openstack-watcher | 08:30 | |
openstackgerrit | David TARDIVEL proposed openstack/watcher: Enable notification for vm task state update https://review.openstack.org/419562 | 08:37 |
openstackgerrit | Vincent Françoise proposed openstack/watcher: Fix multinode tempest test failure https://review.openstack.org/419544 | 08:49 |
alexchadin | vincentfrancoise: hi | 08:54 |
*** zhurong has quit IRC | 09:16 | |
openstackgerrit | David TARDIVEL proposed openstack/watcher: Enable notification for vm task state update https://review.openstack.org/419562 | 09:45 |
openstackgerrit | Alexander Chadin proposed openstack/watcher: New default planner https://review.openstack.org/406991 | 09:46 |
vincentfrancoise | alexchadin: hi | 09:46 |
*** akuleshov1 has joined #openstack-watcher | 09:46 | |
alexchadin | vincentfrancoise: I have some questions about your weightplanner | 09:47 |
openstackgerrit | Vincent Françoise proposed openstack/watcher: Fix multinode tempest test failure https://review.openstack.org/419544 | 09:48 |
vincentfrancoise | alexchadin: what is it you want to know? | 09:48 |
alexchadin | vincentfrancoise: take a look http://pastebin.com/qwcbFiVT | 09:49 |
alexchadin | vincentfrancoise: we have discussed some time ago that all migrations can be executed at the same time | 09:49 |
vincentfrancoise | alexchadin: my default weights are not good, that's why I have commented the TODOs left for me to do | 09:50 |
vincentfrancoise | alexchadin: my parallelization param was configured just for me to test it quickly so we change the defaults to make them more logical | 09:51 |
alexchadin | vincentfrancoise: btw, how planner will know that some resize action of vm1 has to be launched only after migration of vm1 is done? | 09:52 |
vincentfrancoise | alexchadin: so if you set parallelization for migrate to 3 or more, we should end up with the same result | 09:52 |
vincentfrancoise | alexchadin: because only a single action type can be executed at a time, which means that you can never have a migrate and a resize being executed at the same time | 09:54 |
alexchadin | vincentfrancoise: as I understand your algorithm is something like: set of migrations to be done then set of resizes to be done and so on, right? | 09:54 |
vincentfrancoise | alexchadin: exactly | 09:55 |
vincentfrancoise | alexchadin: it's basic but that should do it for simple parallelization | 09:55 |
alexchadin | vincentfrancoise: I had different point of view on this process:) | 09:55 |
alexchadin | vincentfrancoise: what if we have 3 migrations, where 1 is running too long. Does it mean that resize actions won't be executed until all 3 migrations is done? | 09:57 |
vincentfrancoise | alexchadin: yes | 09:58 |
openstackgerrit | Merged openstack/python-watcherclient: Remove unused files https://review.openstack.org/419782 | 09:58 |
alexchadin | vincentfrancoise: In my implementation, we could run resize-1 action while migration-3 is executing | 09:59 |
vincentfrancoise | alexchadin: IMHO, your planner is a bit too specific/optimized so it would make a lot more sense to have it if we can associate a strategy to a planner (see https://blueprints.launchpad.net/watcher/+spec/watcher-planner-selector) | 09:59 |
vincentfrancoise | alexchadin: I know but I really don't want to break the separation of concern between the decision-engine/planner and the applier | 10:00 |
vincentfrancoise | alexchadin: that's why I'm saying that it's a bit too specific | 10:01 |
vincentfrancoise | alexchadin: your planner is more opimized but it requires you to know the actions you want to schedule to make it work | 10:02 |
alexchadin | yeah | 10:02 |
vincentfrancoise | alexchadin: my planner works without the knowledge of the actions but is less optimized | 10:03 |
alexchadin | should I assign this BP to you? | 10:04 |
vincentfrancoise | alexchadin: no no, because your code does the DB work | 10:04 |
vincentfrancoise | alexchadin: maybe we can be co-authors, that would be much fairer | 10:06 |
vincentfrancoise | alexchadin: using Co-Authored-By: name <name@example.com> | 10:06 |
vincentfrancoise | (https://wiki.openstack.org/wiki/GitCommitMessages) | 10:07 |
alexchadin | is it gerrit tag? | 10:07 |
vincentfrancoise | yes | 10:07 |
vincentfrancoise | so can I let you integrate both our work and add my name as co-author on it? | 10:07 |
alexchadin | yes, of course | 10:08 |
vincentfrancoise | alexchadin: thanks a lot! | 10:08 |
vincentfrancoise | alexchadin: also don't throw your code away | 10:08 |
openstackgerrit | David TARDIVEL proposed openstack/python-watcherclient: changes to make consistent with other openstack component https://review.openstack.org/403434 | 10:10 |
vincentfrancoise | alexchadin: as I said, we can use it once we can associate a strategy to a planner | 10:10 |
alexchadin | vincentfrancoise: I would like to rename watcher/decision_engine/planner/default.py to watcher/decision_engine/planner/w_stab_planner and move all validate_parents() methods to watcher/decision_engine/planner/w_stab_planner.py | 10:13 |
alexchadin | vincentfrancoise: each action type will have its own class in w_stab_planner.py | 10:14 |
alexchadin | vincentfrancoise: and mark it as non-production | 10:14 |
alexchadin | vincentfrancoise: we will not use it for now, but it will be 95% completed when we get planner-selector | 10:16 |
vincentfrancoise | alexchadin: sounds good to me altough I would make the name more explicit like [...]/planner/workload_stabilization.py | 10:16 |
vincentfrancoise | alexchadin: and can you also create the BP for it with the dependency on watcher-planner-selector <https://blueprints.launchpad.net/watcher/+spec/watcher-planner-selector>? | 10:17 |
alexchadin | vincentfrancoise: ok | 10:17 |
alexchadin | vincentfrancoise: btw, we need to delete default planner | 10:17 |
alexchadin | vincentfrancoise: since there is no first_action_id field anymore | 10:18 |
*** openstackgerrit has quit IRC | 10:18 | |
vincentfrancoise | alexchadin: yes, my weight planner should become the default one ;) | 10:20 |
vincentfrancoise | alexchadin: I renamed it because "default" doesn't say how it works | 10:20 |
*** hidekazu has quit IRC | 10:20 | |
vincentfrancoise | alexchadin: and because we will have many planners, it's better to give them relevant names | 10:21 |
alexchadin | vincentfrancoise: +1 | 10:22 |
vincentfrancoise | alexchadin: also, have you started the parrallel applier code? | 10:24 |
alexchadin | vincentfrancoise: not yet, will start it today | 10:24 |
vincentfrancoise | alexchadin: sweet! | 10:25 |
alexchadin | vincentfrancoise: there should be less work | 10:25 |
alexchadin | than with planner | 10:25 |
vincentfrancoise | alexchadin: yeah I agree | 10:25 |
vincentfrancoise | alexchadin: it's just that we should merge it before the end of ocata-3 | 10:25 |
alexchadin | main logic is in pseudo-code | 10:25 |
alexchadin | when end of it? | 10:25 |
vincentfrancoise | alexchadin: I know, but in case you didn't have time, I would have had to do it myself | 10:26 |
vincentfrancoise | alexchadin: 23rd Jan | 10:26 |
vincentfrancoise | so we should try to get it merged before next friday | 10:26 |
alexchadin | vincentfrancoise: I will do it ASAP | 10:27 |
vincentfrancoise | alexchadin: I like the spirit :D | 10:27 |
alexchadin | vincentfrancoise: There is a little work I need to do, a little work to do, a little work to do, OH GOD A LOT OF WORK | 10:30 |
vincentfrancoise | alexchadin: hahaha, I know this feeling well too! | 10:31 |
alexchadin | vincentfrancoise: where did it come from?:D | 10:32 |
vincentfrancoise | alexchadin: I'm trying to fix the watcher multinode gate job | 10:32 |
alexchadin | vincentfrancoise: I see | 10:32 |
vincentfrancoise | alexchadin: with dtardivel | 10:32 |
vincentfrancoise | and it's a pain | 10:32 |
alexchadin | there is no debug and you can only imagine how it works? | 10:34 |
vincentfrancoise | alexchadin: just need to read the log veeeeery carefully | 10:38 |
vincentfrancoise | alexchadin: but looks like it woks now! nova changed a config param that deactivated some notifications we were expecting | 10:38 |
alexchadin | vincentfrancoise: debugging of openstack is unforgettable feelings:D | 10:40 |
vincentfrancoise | alexchadin: you mean the pain? :D | 10:40 |
*** dtardivel has joined #openstack-watcher | 10:40 | |
alexchadin | vincentfrancoise: some sort of it:) | 10:40 |
vincentfrancoise | alexchadin: can you +2+1 https://review.openstack.org/#/c/419562/ please? | 10:45 |
vincentfrancoise | alexchadin: that's for the multinode gate fix ;) | 10:45 |
alexchadin | vincentfrancoise: LGTM :) | 10:46 |
vincentfrancoise | alexchadin: thx | 10:46 |
*** absubram has quit IRC | 10:47 | |
*** openstackgerrit has joined #openstack-watcher | 10:52 | |
openstackgerrit | Merged openstack/watcher: Enable notification for vm task state update https://review.openstack.org/419562 | 10:52 |
openstackgerrit | Vincent Françoise proposed openstack/watcher: Remove obsolete Resource element https://review.openstack.org/418974 | 10:55 |
openstackgerrit | Vincent Françoise proposed openstack/watcher: Graph cluster model instead of mapping one https://review.openstack.org/418434 | 10:56 |
*** alexchadin has quit IRC | 11:18 | |
*** sferna1_ has quit IRC | 11:37 | |
*** thorst_ has joined #openstack-watcher | 11:40 | |
*** thorst_ has quit IRC | 11:45 | |
*** openstackgerrit has quit IRC | 11:47 | |
*** alexchadin has joined #openstack-watcher | 11:58 | |
*** adisky_ has quit IRC | 12:09 | |
*** alexchadin has quit IRC | 12:13 | |
*** alexchadin has joined #openstack-watcher | 12:13 | |
*** thorst_ has joined #openstack-watcher | 12:41 | |
*** openstackgerrit has joined #openstack-watcher | 13:04 | |
openstackgerrit | Vincent Françoise proposed openstack/watcher: Remove obsolete Resource element https://review.openstack.org/418974 | 13:04 |
openstackgerrit | Vincent Françoise proposed openstack/watcher: Graph cluster model instead of mapping one https://review.openstack.org/418434 | 13:04 |
*** thorst_ has quit IRC | 13:05 | |
openstackgerrit | Merged openstack/python-watcherclient: changes to make consistent with other openstack component https://review.openstack.org/403434 | 13:08 |
*** thorst_ has joined #openstack-watcher | 13:09 | |
*** absubram has joined #openstack-watcher | 13:40 | |
*** sferna1_ has joined #openstack-watcher | 13:49 | |
*** edleafe is now known as figleaf | 14:00 | |
openstackgerrit | David TARDIVEL proposed openstack/watcher: Added tempest test for workload_balance https://review.openstack.org/393356 | 14:20 |
openstackgerrit | Vincent Françoise proposed openstack/watcher: Remove obsolete Resource element https://review.openstack.org/418974 | 14:20 |
openstackgerrit | Vincent Françoise proposed openstack/watcher: Graph cluster model instead of mapping one https://review.openstack.org/418434 | 14:20 |
*** michaelgugino has quit IRC | 14:21 | |
*** michaelgugino has joined #openstack-watcher | 14:47 | |
openstackgerrit | Merged openstack/python-watcherclient: Use keystoneauth instead of keystoneclient https://review.openstack.org/419806 | 15:06 |
openstackgerrit | Alexander Chadin proposed openstack/watcher: New default planner https://review.openstack.org/406991 | 15:07 |
*** alexchadin has quit IRC | 15:09 | |
openstackgerrit | Alexander Chadin proposed openstack/python-watcherclient: Add functional tests to watcherclient https://review.openstack.org/389181 | 15:10 |
openstackgerrit | Alexander Chadin proposed openstack/python-watcherclient: Support parents field along with planner changes https://review.openstack.org/409126 | 15:10 |
*** jwcroppe has joined #openstack-watcher | 15:17 | |
*** danpawli1 has quit IRC | 15:33 | |
*** danpawlik has joined #openstack-watcher | 15:33 | |
openstackgerrit | David TARDIVEL proposed openstack/watcher: Added tempest test for workload_stabilization https://review.openstack.org/393355 | 15:41 |
openstackgerrit | Vincent Françoise proposed openstack/watcher: Remove obsolete Resource element https://review.openstack.org/418974 | 15:51 |
openstackgerrit | Vincent Françoise proposed openstack/watcher: Graph cluster model instead of mapping one https://review.openstack.org/418434 | 15:51 |
*** akuleshov1 has left #openstack-watcher | 16:31 | |
openstackgerrit | Merged openstack/python-watcherclient: Add param 'goal' and 'strategy' in list() https://review.openstack.org/417790 | 16:38 |
*** lamt has joined #openstack-watcher | 16:43 | |
*** lamt has quit IRC | 16:52 | |
openstackgerrit | Merged openstack/watcher: Add auto_trigger support to watcher https://review.openstack.org/403718 | 17:07 |
*** vincentfrancoise has quit IRC | 17:14 | |
*** lamt has joined #openstack-watcher | 17:18 | |
*** ri0 has joined #openstack-watcher | 17:43 | |
*** ri0 has quit IRC | 17:48 | |
*** wootehfoot has joined #openstack-watcher | 17:54 | |
*** jwcroppe has quit IRC | 17:56 | |
*** ri0 has joined #openstack-watcher | 17:57 | |
*** ri0 has quit IRC | 17:58 | |
*** jwcroppe has joined #openstack-watcher | 17:59 | |
*** wootehfoot has quit IRC | 18:29 | |
*** jwcroppe has quit IRC | 18:48 | |
*** jwcroppe has joined #openstack-watcher | 18:53 | |
*** openstack has joined #openstack-watcher | 18:58 | |
*** wootehfoot has joined #openstack-watcher | 19:09 | |
*** dtardivel has quit IRC | 19:34 | |
*** openstack has joined #openstack-watcher | 19:57 | |
*** openstack has joined #openstack-watcher | 20:27 | |
*** wootehfoot has joined #openstack-watcher | 20:58 | |
*** absubram has quit IRC | 21:05 | |
*** wootehfoot has quit IRC | 21:40 | |
*** thorst_ has quit IRC | 22:02 | |
*** thorst has joined #openstack-watcher | 22:03 | |
*** thorst has quit IRC | 22:07 | |
*** thorst has joined #openstack-watcher | 22:28 | |
*** thorst has quit IRC | 22:31 | |
*** michaelgugino has quit IRC | 22:51 | |
*** thorst has joined #openstack-watcher | 23:32 | |
*** thorst has quit IRC | 23:37 | |
*** figleaf is now known as edleafe | 23:43 | |
*** jwcroppe has quit IRC | 23:47 | |
*** harlowja has quit IRC | 23:56 |
Generated by irclog2html.py 2.14.0 by Marius Gedminas - find it at mg.pov.lt!