Merge "Revert "Introduce macro to remove Py3 only plugins on CentOS 7""

docker/horizon/Dockerfile.j2

@@ -163,7 +163,6 @@ RUN ln -s horizon-source/* horizon \
     && cp -r /horizon/openstack_dashboard/conf/* /etc/openstack-dashboard/ \
     && cp /horizon/openstack_dashboard/local/local_settings.py.example /etc/openstack-dashboard/local_settings \
     && cp /horizon/manage.py /var/lib/kolla/venv/bin/manage.py \
-    {{ macros.remove_py3_only_plugins_for_py2(['searchlight-ui', 'neutron-vpnaas-dashboard']) }}
     && if [ "$(ls /plugins)" ]; then \
            {{ macros.install_pip(horizon_plugins_pip_packages) }}; \
        fi \

docker/macros.j2

@@ -97,13 +97,3 @@ RUN usermod --append --home {{ homedir }} --groups kolla {{ name }} \
 {{ enable_repos(repos) }}
 
 {% endmacro %}
-
-# NOTE(hrw): ending \ has to be at the end as we generate one line and there
-# can be multiple plugins to remove
-{% macro remove_py3_only_plugins_for_py2(plugins_to_remove) -%}
-    {% if distro_python_version == '2.7' -%}
-        {% for plugin in plugins_to_remove %}&& rm -rf /plugins/{{ plugin }}* {% endfor %} \
-    {%- else -%}
-    \
-    {%- endif %}
-{%- endmacro %}

docker/neutron/neutron-server-opendaylight/Dockerfile.j2

@@ -18,6 +18,8 @@ USER root
         ] %}
         {% else %}
         {% set neutron_server_opendaylight_packages = [
+            'python-networking-bgpvpn',
+            'python-networking-bgpvpn-heat',
             'python-networking-l2gw',
             'python-networking-odl'
         ] %}
@@ -56,7 +58,6 @@ ADD plugins-archive /
 
 RUN if [ "$(ls /plugins)" ]; then \
         sed -i 's|^ceilometer===.*$||g' requirements/upper-constraints.txt \
-        {{ macros.remove_py3_only_plugins_for_py2(['networking-bgpvpn']) }}
         && {{ macros.install_pip(neutron_server_opendaylight_plugins_pip_packages) }}; \
     fi
 

kolla/image/build.py

@@ -140,10 +140,6 @@ UNBUILDABLE_IMAGES = {
         "ovsdpdk",
     },
 
-    'centos+source': {
-        "searchlight-base",
-    },
-
     'debian': {
         "bifrost-base",  # tries to install 'mysql-server' which is not in
                          # Debian 'buster'