*** 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!