| *** Guest28904 has quit IRC | 00:34 | |
| *** adiantum has quit IRC | 00:37 | |
| *** ianychoi has joined #openstack-watcher | 02:00 | |
| openstackgerrit | Merged openstack/watcher master: Remove unused utilities file https://review.opendev.org/659487 | 03:36 |
|---|---|---|
| *** ianychoi has quit IRC | 03:49 | |
| *** ianychoi has joined #openstack-watcher | 03:49 | |
| *** ianychoi has quit IRC | 03:51 | |
| *** ianychoi has joined #openstack-watcher | 03:52 | |
| *** adiantum has joined #openstack-watcher | 05:44 | |
| *** adiantum has quit IRC | 05:54 | |
| openstackgerrit | Dantali0n proposed openstack/watcher master: Improve exceptions and logging in ds manager https://review.opendev.org/658127 | 06:06 |
| openstackgerrit | Zhenyu Zheng proposed openstack/watcher master: Fix typo in ceilometer datasource https://review.opendev.org/659726 | 07:07 |
| openstackgerrit | Zhenyu Zheng proposed openstack/watcher master: Fix typo in ceilometer datasource https://review.opendev.org/659726 | 08:03 |
| *** chenke has joined #openstack-watcher | 08:12 | |
| chenke | Dantalion: Hi | 08:12 |
| chenke | are you here? | 08:13 |
| *** tssurya has joined #openstack-watcher | 08:38 | |
| openstackgerrit | chenker proposed openstack/watcher master: support-keystoneclient-option https://review.opendev.org/658102 | 08:40 |
| Dantalion | chenke: Yes I am | 08:46 |
| chenke | i guess you are here. | 08:48 |
| chenke | haha | 08:49 |
| chenke | https://review.opendev.org/#/c/659042/ and https://review.opendev.org/#/c/659062/ please help review | 08:51 |
| chenke | after these commit, i will commit about the bp(Add resource_name in action input parameter field), and i will study another open source project logstash. | 08:53 |
| Dantalion | chenke: Are you looking into using logstash as potential datasource for Watcher? | 08:53 |
| chenke | no | 08:54 |
| Dantalion | chenke: I can check those patches today somewhere | 08:54 |
| chenke | ok, thanks. | 08:55 |
| chenke | i will use filebeat replace logstash. | 08:55 |
| chenke | Logstash consumes more resources | 08:56 |
| chenke | But they have nothing to do with watcher | 08:56 |
| Dantalion | I see | 08:56 |
| Dantalion | okaj, good luck :) | 08:57 |
| chenke | thanks~ | 08:57 |
| chenke | bye~ | 08:57 |
| *** chenke has quit IRC | 09:11 | |
| licanwei | Dantalion: https://review.opendev.org/#/c/656622/14 | 09:19 |
| licanwei | Dantalion: the two tempest tests failure | 09:20 |
| licanwei | Dantalion: the tempest code need to be modified, | 09:20 |
| licanwei | Dantalion: I'll fix it | 09:22 |
| openstackgerrit | licanwei proposed openstack/watcher-tempest-plugin master: Fix tempest failure https://review.opendev.org/659765 | 09:29 |
| openstackgerrit | licanwei proposed openstack/watcher-tempest-plugin master: Fix tempest failure https://review.opendev.org/659765 | 09:31 |
| openstackgerrit | licanwei proposed openstack/watcher master: formal datasource interface implementation https://review.opendev.org/656622 | 09:33 |
| licanwei | Dantalion: I added a Depends-on in https://review.opendev.org/#/c/656622 | 09:34 |
| Dantalion | licanwei: Thank you :) yes that makes sense. I do not know if depends-on is bi-directional but will try by issueing recheck | 09:35 |
| openstackgerrit | Merged openstack/watcher master: Fix_inappropriate_name https://review.opendev.org/659042 | 10:11 |
| openstackgerrit | Merged openstack/watcher master: Use base_strategy's add_action_migrate method https://review.opendev.org/659062 | 10:13 |
| openstackgerrit | Merged openstack/watcher master: pass default_config_dirs variable for config initialization. https://review.opendev.org/658348 | 10:20 |
| openstackgerrit | Dantali0n proposed openstack/watcher master: formal datasource interface implementation https://review.opendev.org/656622 | 11:18 |
| openstackgerrit | Dantali0n proposed openstack/watcher master: formal datasource interface implementation https://review.opendev.org/656622 | 11:18 |
| *** mriedem has joined #openstack-watcher | 13:21 | |
| mriedem | Dantalion: what is the backport policy in watcher? https://review.opendev.org/#/c/659726/ has been broken since queens it looks like (i'm not sure how no one noticed until now), but shouldn't that be backported? and if so, there should probably be a bug for it yeah? | 13:49 |
| Dantalion | mriedem: Backport yes, bug seems fine to track it. Honestly for a moment I thought I was the culprit since we are changing the parameter name with the new datasource interface | 13:52 |
| mriedem | unless someone has a time machine i don't think that's possible :) | 13:52 |
| mriedem | the tests for that code are also worthless, i've suggested a way to make them at least a bit better | 13:54 |
| mriedem | i'm sure kevin won't mind, we're on the same team :) | 13:54 |
| Dantalion | mriedem: I think your right we can merge the fix and backport and i'll just rebase the new datasource interface | 13:56 |
| Dantalion | I can't merge the datasource interface currently anyway because tempest keeps being intermittend | 13:57 |
| mriedem | ok i'll see if i can tweak kevin's change to make the tests better and we can backport (i'm stable-maint-core so can help with that as well) | 13:59 |
| mriedem | i think he ran into this trying to get watcher working with our queens-based product | 13:59 |
| *** tssurya has quit IRC | 15:36 | |
| openstackgerrit | Matt Riedemann proposed openstack/watcher master: Fix typo in ceilometer datasource https://review.opendev.org/659726 | 16:06 |
| openstackgerrit | Matt Riedemann proposed openstack/watcher master: Fix typo in ceilometer datasource https://review.opendev.org/659726 | 16:06 |
| *** mriedem is now known as mriedem_away | 16:49 | |
| *** mriedem_away is now known as mriedem | 18:23 | |
| *** adiantum has joined #openstack-watcher | 18:34 | |
| *** adiantum has quit IRC | 19:38 | |
| *** adiantum has joined #openstack-watcher | 20:43 | |
| openstackgerrit | Matt Riedemann proposed openstack/watcher master: Optimize NovaClusterDataModelCollector.add_instance_node https://review.opendev.org/659688 | 20:49 |
| *** adiantum has quit IRC | 20:54 | |
| openstackgerrit | Matt Riedemann proposed openstack/watcher master: Optimize NovaClusterDataModelCollector.add_instance_node https://review.opendev.org/659688 | 20:56 |
| *** mriedem has quit IRC | 22:20 | |
| *** adiantum has joined #openstack-watcher | 22:25 | |
| *** adiantum has quit IRC | 22:29 | |
Generated by irclog2html.py 2.15.3 by Marius Gedminas - find it at mg.pov.lt!