Public vip didn't relocate after connection loss on the public nic of primary controller

Shutting down interfaces with ifup/ifdown is not a valid test of network outage.

The proper scenario is to remove interface from bridge and not ifdown/ipup

Did as Sergii suggested -- same result.
(updated scenario)

Have got the same on ubuntu.

Kirill, please write down what you did

Kirill, please, write down specific commands that you applied

3. Simulate connection loose on "public" nic of primary controller
by removing the corresponding interface of the bridge on the host

There was a similar issue with split brain scenarios reproducing and there was a conclusion made that in virtual environments manipulations with interfaces are really bad way to simulate real HW cases (there are no carrier down notifications). AFAIK, the decision made was to put down the bridge, not an interface - but I'm not sure if putting it out of bridge would work as well. So please, try to reproduce the same issue by putting the public bridge in down state or even better - at real HW, if it is possible

In case when Public VIP is inaccessible, but interface and link are UP on the problem controller (for example firewall/routing issues on the switch side) - it won't be migrated. It won't break your OpenStack cloud, but will make Public VIP and services you expect to access via Public VIP (like Horizon) unavailable.
I beleive we won't be able to fix it in 5.1 due to HCF, so we need to put a record in release notes about it. We'll address this in 6.0

Fix proposed to branch: master
Review: https://review.openstack.org/122587

Reviewed: https://review.openstack.org/122587
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=04b2df08b0717d7b091a9a3425d4c6af979afbd3
Submitter: Jenkins
Branch: master

commit 04b2df08b0717d7b091a9a3425d4c6af979afbd3
Author: Aleksandr Didenko <email address hidden>
Date: Thu Sep 18 18:05:43 2014 -0700

    Tie pub vip with public gw ping

    Add new pacemaker cloned "ping" primitive and tie it with public
    vip via "location" in order to migrate public VIP in case
    controller can't ping public gateway.

    Closes-bug: 1370510
    Change-Id: Ibf4fedee536ac9eb106206060b58a6039558ef85

Fix proposed to branch: stable/5.1
Review: https://review.openstack.org/123587

I noticed that the fix for 5.1 is been on review for about a month. I think it may be reasonable to add several reviewers explicitly there.

Reviewed: https://review.openstack.org/123587
Committed: https://git.openstack.org/cgit/stackforge/fuel-library/commit/?id=636ceb39174f390c927d06749c355138ed846c84
Submitter: Jenkins
Branch: stable/5.1

commit 636ceb39174f390c927d06749c355138ed846c84
Author: Aleksandr Didenko <email address hidden>
Date: Thu Sep 18 18:05:43 2014 -0700

    Tie pub vip with public gw ping

    Add new pacemaker cloned "ping" primitive and tie it with public
    vip via "location" in order to migrate public VIP in case
    controller can't ping public gateway.

    Partial-bug: 1370510
    Change-Id: Ibf4fedee536ac9eb106206060b58a6039558ef85
    (cherry picked from commit 04b2df08b0717d7b091a9a3425d4c6af979afbd3)

Why was this bug reset from Fix Committed back to In Progress? What work remains to be done here?

Target milestone moved to 5.1.2. Resetting status to In Progress in 6.0.x to be consistent with 5.1.x, until it's clarified which one it should be.

Verified

{"build_id": "2014-12-03_01-07-36", "ostf_sha": "64cb59c681658a7a55cc2c09d079072a41beb346", "build_number": "48", "auth_required": true, "api": "1.0", "nailgun_sha": "500e36d08a45dbb389bf2bd97673d9bff48ee84d", "production": "docker", "fuelmain_sha": "7626c5aeedcde77ad22fc081c25768944697d404", "astute_sha": "ef8aa0fd0e3ce20709612906f1f0551b5682a6ce", "feature_groups": ["mirantis"], "release": "5.1.1", "release_versions": {"2014.1.3-5.1.1": {"VERSION": {"build_id": "2014-12-03_01-07-36", "ostf_sha": "64cb59c681658a7a55cc2c09d079072a41beb346", "build_number": "48", "api": "1.0", "nailgun_sha": "500e36d08a45dbb389bf2bd97673d9bff48ee84d", "production": "docker", "fuelmain_sha": "7626c5aeedcde77ad22fc081c25768944697d404", "astute_sha": "ef8aa0fd0e3ce20709612906f1f0551b5682a6ce", "feature_groups": ["mirantis"], "release": "5.1.1", "fuellib_sha": "a3043477337b4a0a8fd166dc83d6cd5d504f5da8"}}}, "fuellib_sha": "a3043477337b4a0a8fd166dc83d6cd5d504f5da8"}