[BUG] Not all VMs migrate properly #7
Loading…
Reference in New Issue
No description provided.
Delete Branch "%!s(<nil>)"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
Some VMs are left running after flush and are never updated. Investigate cause.
I think this is caused because the
domain_list
ends up missing hosts for some reason. Should have the keepalive re-add any missing hosts to the list (it should be already, but shrug).mentioned in commit
778eff2d7d
So my initial hunch was not correct. It seems that consistently, it's always the second element in the
domain_list
that is missed. For example:eb2bfdcc-ba79-47a2-b500-98bcee885ebc
is the second element of the array, and it is skipped during migration without even beginning (theSelecting target
message is the very first thing in the loop). Something weird is up here.closed via commit
7f3caa2859
For posterity: The cause was me misinterpreting how Python handled list objects during looping and modification. In short, I was removing elements from the
domain_list
list object, but since I was also looping over that object, it was getting thrown off by one - only the second entry would be missed but it was consistently the second, which makes sense considering the element removed was the first, which in turn meant that on the next loop, object "2" would actually be the 3rd element, and the second was missed.Fixed it up by doing an explicit
copy()
from thedomain_list
list before looping over it, ensuring the modifications coming from the main daemon process didn't affect the loop.