Friday, 2017-01-13

*** thorst_ has joined #openstack-watcher00:38
*** thorst_ has quit IRC00:43
*** zhurong has joined #openstack-watcher00:55
*** thorst_ has joined #openstack-watcher00:59
*** thorst_ has quit IRC01:03
*** sferna1_ has quit IRC01:04
*** thorst_ has joined #openstack-watcher01:04
*** thorst_ has quit IRC01:04
*** thorst_ has joined #openstack-watcher02:05
openstackgerritZhenyu Zheng proposed openstack/watcher: Should use glanceclient to get images  https://review.openstack.org/41444502:07
openstackgerritZhenyu Zheng proposed openstack/watcher: Enable coverage report in console output  https://review.openstack.org/41864002:07
*** thorst_ has quit IRC02:08
openstackgerritlicanwei proposed openstack/watcher: node.state should be node.status  https://review.openstack.org/41971602:21
openstackgerritlicanwei proposed openstack/watcher: resolve KeyError exception  https://review.openstack.org/41972702:51
openstackgerritJeremy Liu proposed openstack/python-watcherclient: Enable coverage report in console output  https://review.openstack.org/41972802:51
*** yuanying has quit IRC02:52
openstackgerritMerged openstack/puppet-watcher: Allow python-memcache install from authtoken class  https://review.openstack.org/41920103:01
*** absubram has joined #openstack-watcher03:11
*** diga has joined #openstack-watcher03:15
*** absubram_ has joined #openstack-watcher03:18
*** absubram has quit IRC03:19
*** absubram_ is now known as absubram03:19
*** thorst_ has joined #openstack-watcher03:35
*** thorst_ has quit IRC03:37
*** zhurong has quit IRC03:39
*** yuanying has joined #openstack-watcher03:52
*** atuly_ has quit IRC04:15
*** sferna1_ has joined #openstack-watcher04:19
*** atuly_ has joined #openstack-watcher04:29
*** diga has quit IRC05:01
*** atuly_ has quit IRC05:21
*** atuly_ has joined #openstack-watcher05:33
*** thorst_ has joined #openstack-watcher05:37
*** adisky_ has joined #openstack-watcher05:38
*** thorst_ has quit IRC05:42
*** yuanying has quit IRC06:06
*** yuanying has joined #openstack-watcher06:07
*** yuanying has quit IRC06:11
*** yuanying_ has joined #openstack-watcher06:11
*** yuanying_ has quit IRC06:12
*** yuanying has joined #openstack-watcher06:12
*** yuanying has quit IRC06:17
*** jwcroppe has quit IRC06:23
openstackgerritJeremy Liu proposed openstack/python-watcherclient: Remove unused files  https://review.openstack.org/41978206:41
*** zhurong has joined #openstack-watcher07:05
*** yuanying has joined #openstack-watcher07:19
*** hidekazu has joined #openstack-watcher07:25
*** tkaczynski has quit IRC07:28
*** adisky_ has quit IRC07:29
*** adisky_ has joined #openstack-watcher07:36
*** tkaczynski has joined #openstack-watcher07:36
*** thorst_ has joined #openstack-watcher07:39
*** thorst_ has quit IRC07:43
*** alexchadin has joined #openstack-watcher08:10
openstackgerritJeremy Liu proposed openstack/python-watcherclient: Use keystoneauth instead of keystoneclient  https://review.openstack.org/41980608:17
*** vincentfrancoise has joined #openstack-watcher08:30
openstackgerritDavid TARDIVEL proposed openstack/watcher: Enable notification for vm task state update  https://review.openstack.org/41956208:37
openstackgerritVincent Françoise proposed openstack/watcher: Fix multinode tempest test failure  https://review.openstack.org/41954408:49
alexchadinvincentfrancoise: hi08:54
*** zhurong has quit IRC09:16
openstackgerritDavid TARDIVEL proposed openstack/watcher: Enable notification for vm task state update  https://review.openstack.org/41956209:45
openstackgerritAlexander Chadin proposed openstack/watcher: New default planner  https://review.openstack.org/40699109:46
vincentfrancoisealexchadin: hi09:46
*** akuleshov1 has joined #openstack-watcher09:46
alexchadinvincentfrancoise: I have some questions about your weightplanner09:47
openstackgerritVincent Françoise proposed openstack/watcher: Fix multinode tempest test failure  https://review.openstack.org/41954409:48
vincentfrancoisealexchadin: what is it you want to know?09:48
alexchadinvincentfrancoise: take a look http://pastebin.com/qwcbFiVT09:49
alexchadinvincentfrancoise: we have discussed some time ago that all migrations can be executed at the same time09:49
vincentfrancoisealexchadin: my default weights are not good, that's why I have commented the TODOs left for me to do09:50
vincentfrancoisealexchadin: my parallelization param was configured just for me to test it quickly so we change the defaults to make them more logical09:51
alexchadinvincentfrancoise: btw, how planner will know that some resize action of vm1 has to be launched only after migration of vm1 is done?09:52
vincentfrancoisealexchadin: so if you set parallelization for migrate to 3 or more, we should end up with the same result09:52
vincentfrancoisealexchadin: 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 time09:54
alexchadinvincentfrancoise: 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
vincentfrancoisealexchadin: exactly09:55
vincentfrancoisealexchadin: it's basic but that should do it for simple parallelization09:55
alexchadinvincentfrancoise: I had different point of view on this process:)09:55
alexchadinvincentfrancoise: 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
vincentfrancoisealexchadin: yes09:58
openstackgerritMerged openstack/python-watcherclient: Remove unused files  https://review.openstack.org/41978209:58
alexchadinvincentfrancoise: In my implementation, we could run resize-1 action while migration-3 is executing09:59
vincentfrancoisealexchadin: 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
vincentfrancoisealexchadin: I know but I really don't want to break the separation of concern between the decision-engine/planner and the applier10:00
vincentfrancoisealexchadin: that's why I'm saying that it's a bit too specific10:01
vincentfrancoisealexchadin: your planner is more opimized but it requires you to know the actions you want to schedule to make it work10:02
alexchadinyeah10:02
vincentfrancoisealexchadin: my planner works without the knowledge of the actions but is less optimized10:03
alexchadinshould I assign this BP to you?10:04
vincentfrancoisealexchadin: no no, because your code does the DB work10:04
vincentfrancoisealexchadin: maybe we can be co-authors, that would be much fairer10:06
vincentfrancoisealexchadin: using Co-Authored-By: name <name@example.com>10:06
vincentfrancoise(https://wiki.openstack.org/wiki/GitCommitMessages)10:07
alexchadinis it gerrit tag?10:07
vincentfrancoiseyes10:07
vincentfrancoiseso can I let you integrate both our work and add my name as co-author on it?10:07
alexchadinyes, of course10:08
vincentfrancoisealexchadin: thanks a lot!10:08
vincentfrancoisealexchadin: also don't throw your code away10:08
openstackgerritDavid TARDIVEL proposed openstack/python-watcherclient: changes to make consistent with other openstack component  https://review.openstack.org/40343410:10
vincentfrancoisealexchadin: as I said, we can use it once we can associate a strategy to a planner10:10
alexchadinvincentfrancoise: 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.py10:13
alexchadinvincentfrancoise: each action type will have its own class in w_stab_planner.py10:14
alexchadinvincentfrancoise: and mark it as non-production10:14
alexchadinvincentfrancoise: we will not use it for now, but it will be 95% completed when we get planner-selector10:16
vincentfrancoisealexchadin: sounds good to me altough I would make the name more explicit like [...]/planner/workload_stabilization.py10:16
vincentfrancoisealexchadin: 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
alexchadinvincentfrancoise: ok10:17
alexchadinvincentfrancoise: btw, we need to delete default planner10:17
alexchadinvincentfrancoise: since there is no first_action_id field anymore10:18
*** openstackgerrit has quit IRC10:18
vincentfrancoisealexchadin: yes, my weight planner should become the default one ;)10:20
vincentfrancoisealexchadin: I renamed it because "default" doesn't say how it works10:20
*** hidekazu has quit IRC10:20
vincentfrancoisealexchadin: and because we will have many planners, it's better to give them relevant names10:21
alexchadinvincentfrancoise: +110:22
vincentfrancoisealexchadin: also, have you started the parrallel applier code?10:24
alexchadinvincentfrancoise: not yet, will start it today10:24
vincentfrancoisealexchadin: sweet!10:25
alexchadinvincentfrancoise: there should be less work10:25
alexchadinthan with planner10:25
vincentfrancoisealexchadin: yeah I agree10:25
vincentfrancoisealexchadin: it's just that we should merge it before the end of ocata-310:25
alexchadinmain logic is in pseudo-code10:25
alexchadinwhen end of it?10:25
vincentfrancoisealexchadin: I know, but in case you didn't have time, I would have had to do it myself10:26
vincentfrancoisealexchadin: 23rd Jan10:26
vincentfrancoiseso we should try to get it merged before next friday10:26
alexchadinvincentfrancoise: I will do it ASAP10:27
vincentfrancoisealexchadin: I like the spirit :D10:27
alexchadinvincentfrancoise: There is a little work I need to do, a little work to do, a little work to do, OH GOD A LOT OF WORK10:30
vincentfrancoisealexchadin: hahaha, I know this feeling well too!10:31
alexchadinvincentfrancoise: where did it come from?:D10:32
vincentfrancoisealexchadin: I'm trying to fix the watcher multinode gate job10:32
alexchadinvincentfrancoise: I see10:32
vincentfrancoisealexchadin: with dtardivel10:32
vincentfrancoiseand it's a pain10:32
alexchadinthere is no debug and you can only imagine how it works?10:34
vincentfrancoisealexchadin: just need to read the log veeeeery carefully10:38
vincentfrancoisealexchadin: but looks like it woks now! nova changed a config param that deactivated some notifications we were expecting10:38
alexchadinvincentfrancoise: debugging of openstack is unforgettable feelings:D10:40
vincentfrancoisealexchadin: you mean the pain? :D10:40
*** dtardivel has joined #openstack-watcher10:40
alexchadinvincentfrancoise: some sort of it:)10:40
vincentfrancoisealexchadin: can you +2+1 https://review.openstack.org/#/c/419562/ please?10:45
vincentfrancoisealexchadin: that's for the multinode gate fix ;)10:45
alexchadinvincentfrancoise: LGTM :)10:46
vincentfrancoisealexchadin: thx10:46
*** absubram has quit IRC10:47
*** openstackgerrit has joined #openstack-watcher10:52
openstackgerritMerged openstack/watcher: Enable notification for vm task state update  https://review.openstack.org/41956210:52
openstackgerritVincent Françoise proposed openstack/watcher: Remove obsolete Resource element  https://review.openstack.org/41897410:55
openstackgerritVincent Françoise proposed openstack/watcher: Graph cluster model instead of mapping one  https://review.openstack.org/41843410:56
*** alexchadin has quit IRC11:18
*** sferna1_ has quit IRC11:37
*** thorst_ has joined #openstack-watcher11:40
*** thorst_ has quit IRC11:45
*** openstackgerrit has quit IRC11:47
*** alexchadin has joined #openstack-watcher11:58
*** adisky_ has quit IRC12:09
*** alexchadin has quit IRC12:13
*** alexchadin has joined #openstack-watcher12:13
*** thorst_ has joined #openstack-watcher12:41
*** openstackgerrit has joined #openstack-watcher13:04
openstackgerritVincent Françoise proposed openstack/watcher: Remove obsolete Resource element  https://review.openstack.org/41897413:04
openstackgerritVincent Françoise proposed openstack/watcher: Graph cluster model instead of mapping one  https://review.openstack.org/41843413:04
*** thorst_ has quit IRC13:05
openstackgerritMerged openstack/python-watcherclient: changes to make consistent with other openstack component  https://review.openstack.org/40343413:08
*** thorst_ has joined #openstack-watcher13:09
*** absubram has joined #openstack-watcher13:40
*** sferna1_ has joined #openstack-watcher13:49
*** edleafe is now known as figleaf14:00
openstackgerritDavid TARDIVEL proposed openstack/watcher: Added tempest test for workload_balance  https://review.openstack.org/39335614:20
openstackgerritVincent Françoise proposed openstack/watcher: Remove obsolete Resource element  https://review.openstack.org/41897414:20
openstackgerritVincent Françoise proposed openstack/watcher: Graph cluster model instead of mapping one  https://review.openstack.org/41843414:20
*** michaelgugino has quit IRC14:21
*** michaelgugino has joined #openstack-watcher14:47
openstackgerritMerged openstack/python-watcherclient: Use keystoneauth instead of keystoneclient  https://review.openstack.org/41980615:06
openstackgerritAlexander Chadin proposed openstack/watcher: New default planner  https://review.openstack.org/40699115:07
*** alexchadin has quit IRC15:09
openstackgerritAlexander Chadin proposed openstack/python-watcherclient: Add functional tests to watcherclient  https://review.openstack.org/38918115:10
openstackgerritAlexander Chadin proposed openstack/python-watcherclient: Support parents field along with planner changes  https://review.openstack.org/40912615:10
*** jwcroppe has joined #openstack-watcher15:17
*** danpawli1 has quit IRC15:33
*** danpawlik has joined #openstack-watcher15:33
openstackgerritDavid TARDIVEL proposed openstack/watcher: Added tempest test for workload_stabilization  https://review.openstack.org/39335515:41
openstackgerritVincent Françoise proposed openstack/watcher: Remove obsolete Resource element  https://review.openstack.org/41897415:51
openstackgerritVincent Françoise proposed openstack/watcher: Graph cluster model instead of mapping one  https://review.openstack.org/41843415:51
*** akuleshov1 has left #openstack-watcher16:31
openstackgerritMerged openstack/python-watcherclient: Add param 'goal' and 'strategy' in list()  https://review.openstack.org/41779016:38
*** lamt has joined #openstack-watcher16:43
*** lamt has quit IRC16:52
openstackgerritMerged openstack/watcher: Add auto_trigger support to watcher  https://review.openstack.org/40371817:07
*** vincentfrancoise has quit IRC17:14
*** lamt has joined #openstack-watcher17:18
*** ri0 has joined #openstack-watcher17:43
*** ri0 has quit IRC17:48
*** wootehfoot has joined #openstack-watcher17:54
*** jwcroppe has quit IRC17:56
*** ri0 has joined #openstack-watcher17:57
*** ri0 has quit IRC17:58
*** jwcroppe has joined #openstack-watcher17:59
*** wootehfoot has quit IRC18:29
*** jwcroppe has quit IRC18:48
*** jwcroppe has joined #openstack-watcher18:53
*** openstack has joined #openstack-watcher18:58
*** wootehfoot has joined #openstack-watcher19:09
*** dtardivel has quit IRC19:34
*** openstack has joined #openstack-watcher19:57
*** openstack has joined #openstack-watcher20:27
*** wootehfoot has joined #openstack-watcher20:58
*** absubram has quit IRC21:05
*** wootehfoot has quit IRC21:40
*** thorst_ has quit IRC22:02
*** thorst has joined #openstack-watcher22:03
*** thorst has quit IRC22:07
*** thorst has joined #openstack-watcher22:28
*** thorst has quit IRC22:31
*** michaelgugino has quit IRC22:51
*** thorst has joined #openstack-watcher23:32
*** thorst has quit IRC23:37
*** figleaf is now known as edleafe23:43
*** jwcroppe has quit IRC23:47
*** harlowja has quit IRC23:56

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