Thursday, 2017-09-28

*** thorst has joined #openstack-watcher00:07
*** thorst has quit IRC00:08
*** hidekazu has joined #openstack-watcher00:14
*** zhurong has joined #openstack-watcher00:47
*** thorst has joined #openstack-watcher01:09
*** thorst has quit IRC01:23
*** thorst has joined #openstack-watcher01:23
*** thorst has quit IRC01:26
openstackgerritMerged openstack/watcher master: [Doc] Fix host option  https://review.openstack.org/50743702:06
*** thorst has joined #openstack-watcher02:26
openstackgerritHidekazu Nakamura proposed openstack/watcher master: Add exception log when migrate action failed  https://review.openstack.org/50803502:57
*** nicolasbock has quit IRC03:06
*** zhurong has quit IRC04:07
hidekazuadisky: hi04:38
adiskyhidekazu: hi04:40
hidekazuadisky: i found another bug today. very critical !04:41
hidekazuadisky: https://bugs.launchpad.net/watcher/+bug/172005404:41
openstackLaunchpad bug 1720054 in watcher "migrate action failed with TypeError" [Undecided,New]04:41
hidekazuadisky: we can not migrate at all.04:41
hidekazuadisky: if do you have environment, check this is true or not please.04:42
adiskyhidekazu: ok, checking04:44
adiskyhidekazu: i don't have environment04:47
adiskybut this should not break04:47
hidekazuadisky: we really need tempest test.04:48
hidekazuadisky: ok, i will propose patch today04:48
adiskyhidekazu: your nova api version??04:49
hidekazuadisky: yes, in nova 2.25, disk_over_commit was removed.04:49
adiskywe are using disk over commit, from long and when i was implementing actionplan cancel, it was working goog04:49
adisky*good04:50
adiskyohhk, so in 2.25 they removed04:50
hidekazuadisky: currently 2.53 is default nova version in watcher.04:50
adiskyhidekazu: ok, i think we can support both??04:51
hidekazucurrently we do not support both.04:52
adiskyhidekazu: ok are you talking about integrating tempest to gate jobs??04:53
adiskyohh i see we dont have full coverge in tempest04:54
hidekazuadisky: yes, but honestly, i do not know we have enouh tempest test04:54
adiskyhidekazu: i checked here04:54
adiskyhttps://github.com/openstack/watcher-tempest-plugin/04:54
adiskyand i think no test for applier04:55
hidekazuadiky: test_execute_basic_optim.py seems really execute action plan.04:58
adiskyhidekazu: ok, i have tested actionplan cancel with 2.24...badluck05:00
hidekazuadisky: no watcher user at all, so we have to use by ourselves05:02
openstackgerritHidekazu Nakamura proposed openstack/watcher master: Fix migrate action failure  https://review.openstack.org/50805805:07
adiskyhidekazu: hmm05:10
*** zhurong has joined #openstack-watcher06:38
openstackgerritAlexander Chadin proposed openstack/watcher-specs master: Cleanup approved folder of Ocata and Pike  https://review.openstack.org/50808608:20
*** alexchadin has joined #openstack-watcher08:22
adiskyhidekazu: hi08:26
adiskyalexchadin: ping08:27
alexchadinadisky: hi08:27
adiskyalexchadin: i want to discuss multiple datasource08:27
hidekazuadisky: hi08:28
adiskyhidekazu: notification action plan cancel, all other things were fine??08:28
alexchadinadisky: sure08:29
adiskyalexchadin: To we really want to support, multiple datasource at same time??08:30
hidekazuhttps://review.openstack.org/#/c/492949/08:30
hidekazuadisky: partially not.08:30
adiskyhidekazu: ya apart from this, i am asking?/08:31
adiskyhidekazu: i have check this08:31
hidekazuadisky: ?08:32
adiskyhidekazu: I mean i have checked your comment, and will fix this08:32
adiskyi am asking all other were ok??08:32
hidekazuadisky: okay08:32
adiskyalexchadin: i want to split blueprint in two parts08:32
hidekazuadisky: i am not checking cancel error.08:32
hidekazulets discuss at first :)08:33
adiskyadisky:  okay, which error??08:33
hidekazuadisky: i am not checking actionplan.cancel.error case yet.08:34
adiskyhidekazu: okay!!08:34
hidekazuadisky: alexchadin: please go on.08:35
alexchadinadisky: let me read this bp again08:37
adiskyalexchadin: I want to split it in two blueprints, one for abstraction08:37
adiskyanother one for multiple datasource08:37
alexchadinfirst of all, I'd like to propose to set Gnocchi as default datasource for Watcher08:40
alexchadinadisky: ^08:40
adiskyalexchadin: yes it should be08:40
adiskyand i see abstraction is atmost needed, because every strategy uses datasource in its own way08:41
alexchadinadisky: good idea, start with abstraction layer for datasource08:44
adiskyalexchadin: ok, one more thing08:44
adiskyi dont know who filed blueprint, but he mentions entries in setup.cfg08:45
adiskyso do we want to make datasource loadable??08:45
*** alexchadin has quit IRC08:50
*** alexchadin has joined #openstack-watcher08:51
*** alexchadin has quit IRC08:51
*** alexchadin has joined #openstack-watcher08:51
alexchadinadisky: sorry for disconnecting08:51
alexchadinadisky: I'd prefer don't make datasource loadable08:55
*** hidekazu has quit IRC08:58
*** hidekazu has joined #openstack-watcher08:59
*** alexchadin has quit IRC09:01
*** hidekazu has left #openstack-watcher09:03
*** alexchadin has joined #openstack-watcher09:03
*** alexchad_ has joined #openstack-watcher09:28
*** alexchadin has quit IRC09:30
*** alexchad_ has quit IRC09:32
*** alexchadin has joined #openstack-watcher09:33
openstackgerritAlexander Chadin proposed openstack/watcher-specs master: Spec for check strategy requirements  https://review.openstack.org/50608309:45
*** suzhengwei has joined #openstack-watcher09:46
openstackgerritsuzhengwei proposed openstack/watcher-specs master: listen to all necessary notifications  https://review.openstack.org/50810510:21
openstackgerritsuzhengwei proposed openstack/watcher-specs master: listen to all necessary notifications  https://review.openstack.org/50810510:23
openstackgerritsuzhengwei proposed openstack/watcher master: listen to 'compute.instance.rebuild.end' event  https://review.openstack.org/50354910:24
openstackgerritsuzhengwei proposed openstack/watcher master: listen to 'compute.instance.resize.confirm.end' event  https://review.openstack.org/50357010:24
*** zhurong has quit IRC10:27
*** alexchadin has quit IRC10:32
openstackgerritsuzhengwei proposed openstack/watcher master: filter exclude instances during migration  https://review.openstack.org/50810810:42
*** suzhengwei has quit IRC10:45
*** thorst has quit IRC11:16
*** thorst has joined #openstack-watcher12:03
*** ianychoi_ is now known as ianychoi13:06
*** alexchadin has joined #openstack-watcher13:28
*** alexchadin has quit IRC14:48
*** thorst has quit IRC20:26
*** thorst has joined #openstack-watcher20:43
*** thorst has quit IRC21:00
*** thorst has joined #openstack-watcher21:00
*** thorst has quit IRC21:11
*** thorst has joined #openstack-watcher21:32
*** thorst has quit IRC21:36
*** thorst has joined #openstack-watcher22:13
*** thorst has quit IRC22:17
*** thorst has joined #openstack-watcher22:23
*** thorst has quit IRC22:35
*** thorst has joined #openstack-watcher22:36
*** thorst has quit IRC22:42
*** thorst has joined #openstack-watcher22:45
*** thorst has quit IRC22:58
*** thorst has joined #openstack-watcher22:58
*** thorst has quit IRC23:31
*** thorst has joined #openstack-watcher23:39
*** hidekazu has joined #openstack-watcher23:40
*** thorst has quit IRC23:54
*** thorst has joined #openstack-watcher23:55

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