Ensure a smooth upgrade path after adv svc split

[Numan Siddique]

check-grenade-dsvm-neutron is failing after the services split. An example of a traceback is [1].

This is because no thin shims have been provided for the drivers, as done for plugins in [2]. Thin shims should be temporary just to provide a bw compat upgrade path, but they should be replaced by a more effective mechanism like 1) make load_drivers() use stevedore, and 2) add entry points for bw compatibility.

[1] http://logs.openstack.org/51/140851/19/check/check-grenade-dsvm-neutron/da95f11/logs/new/screen-q-svc.txt.gz?level=TRACE
[2] https://review.openstack.org/#/c/140515/

[Clark Boylan (cboylan)]

I am removing openstack-ci from this bug for two reasons. First this is an issue with neutron and grenade. They will need to sort out an upgrade path for these services. Second OpenStack infra bugs are now filed at: https://storyboard.openstack.org/#!/project_group/55

[Armando Migliaccio (armando-migliaccio)]

Interim fix for grenade: https://review.openstack.org/#/c/141256/

[Armando Migliaccio (armando-migliaccio)]

@Ihar: when did I volunteer for the Neutron fix? :P

[Ihar Hrachyshka (ihar-hrachyshka)]

Nah, sorry, that was a miscommunication prolly. ;)
I'm moving the bug back to my plate then. :)

[OpenStack Infra (hudson-openstack)]

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

[Armando Migliaccio (armando-migliaccio)]

No worries :)

That said, once the Neutron change lands, I would imagine that the follow-up Grenade patch would be to drop the upgrade script, correct?

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

[Kyle Mestery (mestery)]

Moving to Kilo-2 since we have to cut Kilo-1 today.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

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

commit 21842feeae0c1a0a59242d28d74765f5cc761c8c
Author: Jakub Libosvar <email address hidden>
Date: Tue Dec 16 17:33:23 2014 +0100

    Backward compatibility for advanced services

    Patch implements translation from class paths to neutron to class paths
    to neutron_<adv_service>. It's achieved by defining entry point in
    setup.cfg which is translated by stevedore.

    There will be needed patches in advanced services tree calling
    get_provider_driver_class() function before importing class.

    This patch specifically fixes loading service plugins and
    drivers for service plugin. Patches for agents are still needed in
    neutron repo and adv services repos.

    Alternative and better solution would be implementing new DriverType
    to oslo.config, which will have callback to
    get_provider_driver_class()-like function.

    Co-Authored-By: Ihar Hrachyshka <email address hidden>
    Change-Id: I76af175c4387326a4e5ff95c2f15d8b866dedab3
    Partial-Bug: 1401895

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/142474
Committed: https://git.openstack.org/cgit/openstack/neutron-fwaas/commit/?id=8d1c1c4a32c95396e6b8f5f32f229b74fbb105e1
Submitter: Jenkins
Branch: master

commit 8d1c1c4a32c95396e6b8f5f32f229b74fbb105e1
Author: Jakub Libosvar <email address hidden>
Date: Wed Dec 17 16:56:36 2014 +0100

    Backward compatibility for fwaas

    Fwaas driver class is changed to one from this repo in case it attempts
    to be loaded from neutron repo.

    This change depends on I76af175c4387326a4e5ff95c2f15d8b866dedab3

    Change-Id: If850adb3ec9db0a9ff0d43cd7445264c168d241c
    Closes-Bug: 1401895

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/142482
Committed: https://git.openstack.org/cgit/openstack/neutron-lbaas/commit/?id=52d54dd7d063689f7eae73a98c8c542a6901decd
Submitter: Jenkins
Branch: master

commit 52d54dd7d063689f7eae73a98c8c542a6901decd
Author: Jakub Libosvar <email address hidden>
Date: Wed Dec 17 17:15:55 2014 +0100

    Backward compatibility for lbaas

    Lbaas driver class is changed to one from this repo in case it attempts
    to be loaded from neutron repo.

    This change depends on I76af175c4387326a4e5ff95c2f15d8b866dedab3

    Change-Id: If850adb3ec9db0a9ff0d43cd7445264c168d241c
    Closes-Bug: 1401895

[OpenStack Infra (hudson-openstack)]

Change abandoned by Jakub Libosvar (<email address hidden>) on branch: master
Review: https://review.openstack.org/142484
Reason: This patch is not needed anymore after agent refactor: https://review.openstack.org/#/c/140918

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/142484
Committed: https://git.openstack.org/cgit/openstack/neutron-vpnaas/commit/?id=30fe57e4a883bc64b42f858dbf02d3d79ee84018
Submitter: Jenkins
Branch: master

commit 30fe57e4a883bc64b42f858dbf02d3d79ee84018
Author: Jakub Libosvar <email address hidden>
Date: Wed Dec 17 17:23:47 2014 +0100

    Backward compatibility for vpnaas

    Vpnaas driver class is changed to one from this repo in case it attempts
    to be loaded from neutron repo.

    This change depends on I76af175c4387326a4e5ff95c2f15d8b866dedab3

    Change-Id: I2986a9724396920071a3a22121e5b5a5a08ed059
    Closes-Bug: 1401895

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/142852
Committed: https://git.openstack.org/cgit/openstack-dev/grenade/commit/?id=ad79234af5744e4bd586de6f1ffd9fd829f9a619
Submitter: Jenkins
Branch: master

commit ad79234af5744e4bd586de6f1ffd9fd829f9a619
Author: armando-migliaccio <email address hidden>
Date: Thu Dec 18 09:36:36 2014 -0800

    Drop Kilo upgrade hack for Neutron

    This patch removes the need for a special upgrade procedure
    as outlined in [1]. The Neutron codebases have been tweaked
    [2,3,4,5] to allow for a backward compatible upgrade.

    [1] https://wiki.openstack.org/wiki/ReleaseNotes/Kilo#Upgrade_Notes_6
    [2] https://review.openstack.org/#/c/142484/
    [3] https://review.openstack.org/#/c/142482/
    [4] https://review.openstack.org/#/c/142474/
    [5] https://review.openstack.org/#/c/142150/

    Closes-bug: #1401895

    Change-Id: I82b8aa4215f9669bb622548f27f8bb53544c9792