Functional test_models_sync in VPNaaS (*aaS?) is failing due to neutron model mismatch

[Akihiro Motoki]

VPNaaS test_models_sync in gate-neutron-vpnaas-dsvm-functional is failing due to neutron model mismatch.
For example, http://logs.openstack.org/09/261109/3/check/gate-neutron-vpnaas-dsvm-functional/a26256d/

It seems it is triggered by the review https://review.openstack.org/#/c/212213/.
This commit forgot to add dns_db to migration/models/head.py.

Ideally it should be detected by neutron gate functional test.
The reason it was not detected in our func test is being investigated under bug 1537510.

The reason I filed this as a separate bug is because a subproject does not need to test neutron model sync.
Instead it should check model sync in each subproject.

I am not very confident that my above statement is correct, but I think it is worth filed.

[OpenStack Infra (hudson-openstack)]

Related fix proposed to branch: master
Review: https://review.openstack.org/271865

[Ann Taraday (akamyshnikova)]

@amotoki

Neutron migrations are applied before migrations of other projects migrations applied, so we cannot check just subproject model migration sync, as some tables are created by Neutron migrations.

[OpenStack Infra (hudson-openstack)]

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

[Akihiro Motoki (amotoki)]

Ann, thanks for the clarification.
I just wondered we can ignore neutron model mismatch in model sync check in a subproject, but it is not so easy..

I decided to use this bug to track the gate failure in neutron-vpnaas (and other sub-projects if any though I am not aware of them).

We can use bug 1537510 to track why the model mismatch was not detected in our gate.

Does it sound good?

[OpenStack Infra (hudson-openstack)]

Change abandoned by Akihiro Motoki (<email address hidden>) on branch: master
Review: https://review.openstack.org/271865
Reason: The jobs for https://review.openstack.org/#/c/271773/ have not completed yet, but the functional tests succeeded. This patch is no longer needed.

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/271918
Committed: https://git.openstack.org/cgit/openstack/neutron/commit/?id=d9693d787812347133e9ff765428389a2c332cdf
Submitter: Jenkins
Branch: master

commit d9693d787812347133e9ff765428389a2c332cdf
Author: Akihiro Motoki <email address hidden>
Date: Mon Jan 25 16:46:08 2016 +0900

    Add dns_db to models/head.py

    commit 93ac8b3a3318c0b762d399fa78b8ce981d44b3d9 adds dns_db
    but it forgot to add it to models/heads.
    As a result, db migration model sync check is failing.

    Closes-Bug: #1537578
    Closes-Bug: #1537510

    Note that it was not detected in our neutron functional tests.
    It is being investigated in bug 1537510.

    Change-Id: Ica15afb1fd2335e5fe686ec36b05513a46427bcd

[Thierry Carrez (ttx)]

This issue was fixed in the openstack/neutron 8.0.0.0b3 development milestone.