Tuesday, 2017-12-26

*** jianhua has joined #openstack-watcher00:38
*** AlexeyAbashkin has joined #openstack-watcher01:29
*** AlexeyAbashkin has quit IRC01:33
openstackgerritsuzhengwei proposed openstack/watcher master: use current weighted sd as min_sd when starting to simulate migrations  https://review.openstack.org/53001102:09
openstackgerritMerged openstack/watcher master: Add Datasource Abstraction  https://review.openstack.org/51885203:02
*** hidekazu has joined #openstack-watcher04:21
*** jianhua has quit IRC04:22
*** jianhua has joined #openstack-watcher04:29
openstackgerritMerged openstack/watcher master: Fix watcher audit list command  https://review.openstack.org/52869104:49
*** zhurong has joined #openstack-watcher04:52
openstackgerritHidekazu Nakamura proposed openstack/watcher master: Add zone migration strategy  https://review.openstack.org/52169104:57
openstackgerritMerged openstack/python-watcherclient master: Updated from global requirements  https://review.openstack.org/52892105:33
*** jianhua has quit IRC05:58
*** jianhua has joined #openstack-watcher05:59
*** zhurong has quit IRC07:00
*** suzhengwei has joined #openstack-watcher07:18
*** suzhengwei has quit IRC07:25
*** zhurong has joined #openstack-watcher07:39
*** AlexeyAbashkin has joined #openstack-watcher07:48
*** AlexeyAbashkin has quit IRC07:52
*** AlexeyAbashkin has joined #openstack-watcher07:52
openstackgerritlicanwei proposed openstack/watcher master: correct audit parameter typo  https://review.openstack.org/53006807:56
*** hidekazu has quit IRC08:03
*** alexchadin has joined #openstack-watcher08:27
*** alexchadin has quit IRC08:34
*** alexchadin has joined #openstack-watcher08:34
*** alexchadin has quit IRC08:52
*** alexchadin has joined #openstack-watcher08:53
openstackgerritAlexander Chadin proposed openstack/watcher master: Strategy requirements  https://review.openstack.org/52253808:54
openstackgerritAlexander Chadin proposed openstack/watcher master: Strategy requirements  https://review.openstack.org/52253808:55
openstackgerritYumengBao proposed openstack/watcher master: Add baremetal data model  https://review.openstack.org/48209309:05
*** suzhengwei has joined #openstack-watcher09:16
*** Yumeng has joined #openstack-watcher09:17
*** alexchadin has quit IRC09:22
*** alexchadin has joined #openstack-watcher09:22
alexchadinsuzhengwei: pong09:23
suzhengweiI want to talk about audit-api.09:29
alexchadinsuzhengwei: I'll be back in 45 minutes, excuse me09:30
suzhengweiok, wait for you09:30
*** alexchadin has quit IRC09:31
*** alexchadin has joined #openstack-watcher10:15
alexchadinsuzhengwei: hi10:16
alexchadinsuzhengwei: what exactly would you like to discuss?10:16
suzhengweithe AuditPostType10:17
suzhengweiThe goal or strategy in the AuditPostType, must be uuid, or can be name?10:18
suzhengweicurrently, it just accept uuid of the goal, strategy or audittemplate. The watcherclient will change the name to corresponding uuid when create audit.10:24
suzhengweiIf we directly call the api with name of goal, strategy, or audittemplate, it will raise exception when creating audit.10:25
alexchadinsuzhengwei: yes, currently it accepts only uuid10:26
suzhengweido you think we need change to type from wtypes.text to types.uuid for goal and strategy, if it only accepts uuid?10:29
suzhengweiOr we can refact it to work with both name and uuid?10:31
alexchadinsuzhengwei: let me think10:32
alexchadinsuzhengwei: haven't you tried to create audit with goal name and strategy name?10:38
alexchadinsuzhengwei: look, I tried to create audit without audittemplate10:52
alexchadinsuzhengwei: I used the following body: '{"audit_type": "ONESHOT", "auto_trigger": false, "goal": "dummy", "strategy": "dummy"}'10:53
alexchadinI got error here: https://github.com/openstack/watcher/blob/master/watcher/api/controllers/v1/audit.py#L12010:53
alexchadinso, if I specify name attribute and add it to body: '{"audit_type": "ONESHOT", "auto_trigger": false, "goal": "dummy", "strategy": "dummy", "name": "daudit"}' - it works10:54
suzhengweiI have found the reason.10:55
suzhengweiif you didn't give the audit name, it will produce a default name, which make this problem.10:56
alexchadinsuzhengwei: since all strategies have different names, I'd propose to start accepting name of strategy and goals too10:58
suzhengweiI think so10:58
alexchadinsuzhengwei: will you take it? :)11:01
suzhengweisure, i will give a patch11:01
*** AlexeyAbashkin has quit IRC11:13
*** suzhengwei has quit IRC11:19
*** alexchadin has quit IRC11:20
*** alexchadin has joined #openstack-watcher11:21
*** AlexeyAbashkin has joined #openstack-watcher11:22
*** alexchadin has quit IRC12:47
*** alexchadin has joined #openstack-watcher13:00
*** zhurong has quit IRC13:08
*** jianhua has quit IRC13:40
*** alexchadin has quit IRC14:01
openstackgerritsuzhengwei proposed openstack/watcher master: create audit failed  https://review.openstack.org/53010514:02
*** AlexeyAbashkin has quit IRC16:46
openstackgerritMerged openstack/puppet-watcher master: Run puppet-openstack-beaker-jobs  https://review.openstack.org/52978116:54
*** AlexeyAbashkin has joined #openstack-watcher18:05
*** harlowja has joined #openstack-watcher18:06
*** AlexeyAbashkin has quit IRC18:10
*** AlexeyAbashkin has joined #openstack-watcher18:26
*** AlexeyAbashkin has quit IRC18:30
*** AlexeyAbashkin has joined #openstack-watcher18:47
*** AlexeyAbashkin has quit IRC18:51
*** AlexeyAbashkin has joined #openstack-watcher19:27
*** AlexeyAbashkin has quit IRC19:31
*** AlexeyAbashkin has joined #openstack-watcher23:48
*** AlexeyAbashkin has quit IRC23:53

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