devfaz | yoctozepto: yeah, im just starting quite early :) | 05:35 |
---|---|---|
*** vishalmanchanda has joined #openstack-masakari | 06:49 | |
*** brinzhang has joined #openstack-masakari | 08:09 | |
*** mgoddard has quit IRC | 08:10 | |
*** mgoddard has joined #openstack-masakari | 08:11 | |
*** jopdorp has quit IRC | 08:33 | |
*** uneek has quit IRC | 08:33 | |
*** jopdorp has joined #openstack-masakari | 08:39 | |
*** uneek has joined #openstack-masakari | 08:39 | |
openstackgerrit | Vishal Manchanda proposed openstack/masakari-dashboard master: Cleanup for Refactor-error-messages https://review.opendev.org/748165 | 11:46 |
openstackgerrit | Vishal Manchanda proposed openstack/masakari-dashboard master: Cleanup for Refactor-error-messages https://review.opendev.org/748165 | 12:44 |
*** bhagyashris is now known as bhagyashris|away | 13:06 | |
*** brinzhang has quit IRC | 13:41 | |
*** brinzhang has joined #openstack-masakari | 13:41 | |
*** brinzhang has quit IRC | 13:48 | |
*** brinzhang has joined #openstack-masakari | 13:48 | |
noonedeadpunk | yoctozepto: so... regarding exception I'm wondering why HypervisorNotFoundByName is needed actually | 16:53 |
noonedeadpunk | https://opendev.org/openstack/masakari/src/branch/master/masakari/exception.py#L275-L280 | 16:53 |
yoctozepto | noonedeadpunk: could use analysing their usage; I'm not yet well-versed in "our" source code :-) | 16:54 |
yoctozepto | I guess we need to translate the nova exception for sure | 16:54 |
yoctozepto | but obviously "hypervisor" is no longer true | 16:54 |
noonedeadpunk | I know that HypervisorNotFoundByNameis used only in that context | 16:55 |
noonedeadpunk | of hypervisor reqest which is not the case anymore | 16:55 |
noonedeadpunk | and HostNotFoundByName for cases where nova evacuate can't be done or smth... | 16:56 |
yoctozepto | noonedeadpunk: yeah, maybe it is right to monitor the hypervisor's name | 16:57 |
noonedeadpunk | I mean they're really very close by meaning | 16:57 |
yoctozepto | noonedeadpunk: anyway, what do you have in corosync as host names? | 16:57 |
noonedeadpunk | I have them equal to the opensatck compute service list | 16:58 |
noonedeadpunk | The main thing is, that when engine issues evacuate call to nova it must pass service name | 16:58 |
noonedeadpunk | not hypervisor name | 16:58 |
openstackgerrit | Merged openstack/masakari master: Remove the unused coding style modules https://review.opendev.org/747580 | 16:58 |
openstackgerrit | Merged openstack/masakari master: Remove monotonic usage https://review.opendev.org/725420 | 16:59 |
yoctozepto | noonedeadpunk: yeah, I know that part | 16:59 |
yoctozepto | noonedeadpunk: but see, it could monitor *hypervisor* and then translate that name for the nova call | 16:59 |
noonedeadpunk | How would it do it? | 17:00 |
noonedeadpunk | Nova name is set in config ? which can be whatever | 17:00 |
noonedeadpunk | with no reall mapping between hypervisor name and compute service | 17:00 |
yoctozepto | noonedeadpunk: nova knows | 17:01 |
yoctozepto | (I guess) | 17:01 |
yoctozepto | but it's hard to justify this extra work now that it's broken in either scenario :-) | 17:02 |
openstackgerrit | Merged openstack/masakari master: Switch from unittest2 compat methods to Python 3.x methods https://review.opendev.org/739640 | 17:10 |
openstackgerrit | Merged openstack/masakari master: Remove translation sections from setup.cfg https://review.opendev.org/728034 | 17:10 |
openstackgerrit | Merged openstack/masakari-dashboard master: Stop to use the __future__ module. https://review.opendev.org/732859 | 17:36 |
openstackgerrit | Merged openstack/masakari-monitors master: Fix hacking min version to 3.0.1 https://review.opendev.org/727513 | 17:49 |
openstackgerrit | Merged openstack/masakari-monitors master: Remove elementtree deprecated methods https://review.opendev.org/737474 | 17:49 |
openstackgerrit | Dmitriy Rabotyagov (noonedeadpunk) proposed openstack/masakari master: Search in nova services instead of hypervisors https://review.opendev.org/728629 | 17:51 |
openstackgerrit | Merged openstack/masakari-monitors master: repeated parsing https://review.opendev.org/733428 | 18:02 |
*** brinzhang_ has joined #openstack-masakari | 18:21 | |
*** brinzhang has quit IRC | 18:25 | |
*** vishalmanchanda has quit IRC | 20:37 | |
*** brinzhang0 has joined #openstack-masakari | 22:00 | |
*** brinzhang_ has quit IRC | 22:04 | |
*** brinzhang0 has quit IRC | 23:22 | |
*** brinzhang0 has joined #openstack-masakari | 23:23 |
Generated by irclog2html.py 2.17.2 by Marius Gedminas - find it at https://mg.pov.lt/irclog2html/!