| opendevreview | Takashi Kajinami proposed openstack/aetos master: Support client cert/key for TLS connection with Prometheus https://review.opendev.org/c/openstack/aetos/+/956983 | 01:05 |
|---|---|---|
| opendevreview | Merged openstack/aetos master: Validate input of [DEFAULT] auth_mode https://review.opendev.org/c/openstack/aetos/+/956650 | 06:36 |
| opendevreview | Callum Dickinson proposed openstack/ceilometer master: Add vcpus pollster https://review.opendev.org/c/openstack/ceilometer/+/957214 | 09:38 |
| opendevreview | Callum Dickinson proposed openstack/ceilometer master: Add memory pollster https://review.opendev.org/c/openstack/ceilometer/+/957216 | 10:09 |
| opendevreview | Callum Dickinson proposed openstack/ceilometer master: Add memory pollster https://review.opendev.org/c/openstack/ceilometer/+/957216 | 11:15 |
| opendevreview | Jaromír Wysoglad proposed openstack/aetos master: Add configuration for project label name https://review.opendev.org/c/openstack/aetos/+/957224 | 12:20 |
| tkajinam | hmm I suspect that current power_state pollster may conflict with resize (or cold migration) which keeps the original domain at source until the operation is confirmed | 12:30 |
| tkajinam | https://review.opendev.org/c/openstack/ceilometer/+/957214 | 12:30 |
| tkajinam | maybe even live migration can interfere | 12:38 |
| tkajinam | jwysogla, ^^^ you might want to discuss this with Juan | 12:38 |
| tkajinam | The potential problem I'm pointing is that current power_state pollster publishes metrics regardless of domain status. During resize or migration we expect two domains defined in source and dest so we may get two samples from both these two nodes | 12:40 |
| opendevreview | Takashi Kajinami proposed openstack/ceilometer master: Disable power.state meter https://review.opendev.org/c/openstack/ceilometer/+/957231 | 12:50 |
| jwysogla | I'll try, but I'll need to do some studying as I have 0 knowledge on the topic. Not sure if Juan is much better. | 12:52 |
| jwysogla | Also I won't see Juan until September because of our overlapping PTOs, but I'll try to point him to this conversation. | 12:53 |
| tkajinam | ok | 12:53 |
| tkajinam | jwysogla, if you can leave this link (maybe in internal chat or something) for him then that might be helpful https://bugs.launchpad.net/ceilometer/+bug/2120560 | 12:58 |
| mrunge | this is tricky. I wonder if there is a better solution to the issue. | 12:59 |
| tkajinam | the only way I can think of atm is to publish only specific state such as running and probably crashed. | 13:05 |
| tkajinam | though I'm not sure that does not break the original motivation to add that meter | 13:05 |
| tkajinam | https://libvirt.org/html/libvirt-libvirt-domain.html#virDomainState | 13:06 |
| tkajinam | publishing shutoff is definitely tricky. also afair domain enters paused state during migration so we properly exclude that case or avoid publishing that state | 13:07 |
| tkajinam | the others might be ok | 13:07 |
| tkajinam | for total cpu/ram we can avoid this problem by publishing these meters only when the domain is active but power state requires the other state by its nature... so again it's tricky | 13:10 |
| jwysogla | ok, I think I understand the issue. I agree with disabling it by default until we figure out how to have the metric in a not confusing way in the scenarios you pointed out. Regarding possible solutions and original motivation, I think it'd be good to wait for Juan as he knows more about the metric. | 13:19 |
| tkajinam | yeah | 13:31 |
| opendevreview | Merged openstack/python-observabilityclient master: Fix PrometheusAPIClient set_client_cert https://review.opendev.org/c/openstack/python-observabilityclient/+/957185 | 13:56 |
| opendevreview | Merged openstack/ceilometer master: Disable power.state meter https://review.opendev.org/c/openstack/ceilometer/+/957231 | 18:51 |
| opendevreview | Jaromír Wysoglad proposed openstack/aodh master: Rbac restrictions to prometheus alarms https://review.opendev.org/c/openstack/aodh/+/957279 | 19:23 |
| opendevreview | Jaromír Wysoglad proposed openstack/aodh master: Rbac restrictions to prometheus alarms https://review.opendev.org/c/openstack/aodh/+/957279 | 19:24 |
Generated by irclog2html.py 4.0.0 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!