Comment 2 for bug 1954751

Reviewed: https://review.opendev.org/c/openstack/neutron/+/822051
Committed: https://opendev.org/openstack/neutron/commit/c6a6c5ae127a97856c880da3f82257977a06a571
Submitter: "Zuul (22348)"
Branch: master

commit c6a6c5ae127a97856c880da3f82257977a06a571
Author: Slawek Kaplonski <email address hidden>
Date: Thu Dec 16 22:25:27 2021 +0100

    [Functional] Fix expected number of the enqueue_state_change calls

    In the HA router's keepalived state change monitor tests, it was
    expected that enqueue_state_change method will be called 3 or 4 times.
    But after some changes in the keepalived_state_change monitor which were
    done some time ago, it may be now that it will be called just 2 or 3
    times:
    - 2 when initial status will be "primary" and it will be just
      transition to "backup",
    - 3 when initial status will be "backup", then it will transition to
      "primary" and finally to "backup" again.

    To reflect those 2 possibilities, test was changed that it will expect
    2 or 3 calls and will check only that last 2 will be always transition
    to "primary" and then to "backup".

    Additionally this patch adds some extra logging in that test so it will
    be easier to check what was going on in that test.

    Closes-Bug: #1954751
    Change-Id: Ib5de7e65839f52c35c43801969e3f0c16dead5bb