Distro layer build failes on python-neutronclient

[Scott Little]

Brief Description
-----------------
Distro layer build failes on package python-neutronclient.

Missing dependency on oslo_log

Severity
--------
Provide the severity of the defect.
Critical

Steps to Reproduce
------------------
In distro layer ...
build-pkgs

Expected Behavior
------------------
build passes

Actual Behavior
----------------
build failes

Reproducibility
---------------
100%

System Configuration
--------------------
NA

Branch/Pull Time/Commit
-----------------------
20200501T012136Z

Last Pass
---------
April 24

Timestamp/Logs
--------------
Exception occurred:
  File "/builddir/build/BUILD/python-neutronclient-6.14.0/neutronclient/osc/v2/logging/network_log.py", line 23, in <module>
    from oslo_log import log as logging
ImportError: No module named oslo_log

Test Activity
-------------
Build

Workaround
----------
None

[Al Bailey (albailey1974)]

I do not see this error. The %check section of the spec file does not care if the unit tests fail.
Is it possible that some other component is causing the break.

[Stefan Dinescu (stefandinescu)]

A colleague of mine saw this error, while I didn't. It's possible that in a parallel build it depends on which mock environment the neutronclient is being built in and if another package pulled this dependency before neutronclient.

[Ghada Khalil (gkhalil)]

stx.4.0 / critical - build failure

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/724792
Committed: https://git.openstack.org/cgit/starlingx/upstream/commit/?id=8e1bc6dc344cf03b67c11fd5bc4b8759076d6fb4
Submitter: Zuul
Branch: master

commit 8e1bc6dc344cf03b67c11fd5bc4b8759076d6fb4
Author: Scott Little <email address hidden>
Date: Thu Apr 30 23:21:43 2020 -0400

    Fix missing build dependency in python-neutronclient

    Closes-Bug: 1876229
    Change-Id: I61cb4f37888d4a0f1122ff5d6e15a2de3544dc1f
    Signed-off-by: Scott Little <email address hidden>

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: f/centos8
Review: https://review.opendev.org/729828

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.opendev.org/729828
Committed: https://git.openstack.org/cgit/starlingx/upstream/commit/?id=253f9ec5506cb87f7d928249e39ed8349be0fd63
Submitter: Zuul
Branch: f/centos8

commit 8e1bc6dc344cf03b67c11fd5bc4b8759076d6fb4
Author: Scott Little <email address hidden>
Date: Thu Apr 30 23:21:43 2020 -0400

    Fix missing build dependency in python-neutronclient

    Closes-Bug: 1876229
    Change-Id: I61cb4f37888d4a0f1122ff5d6e15a2de3544dc1f
    Signed-off-by: Scott Little <email address hidden>

commit 3b04384192301098aba2cc56831d7d25b8737433
Author: Andy Ning <email address hidden>
Date: Thu Apr 30 09:15:24 2020 -0400

    Set service's self url based on settings for Horizon

    The intention of the original patch is to remove Horizon's reference to
    internal endpoint so that the default endpoint type specified in settings
    (OPENSTACK_ENDPOINT_TYPE) can take effect uniformly across all services.
    But the patched function base.get_url_for_service() take 3 mandatory
    parameters. So removing the third param causes Horizon traceback when
    accessing API Access page.

    This commit updated the patch so that services's self url is set to
    the value of OPENSTACK_ENDPOINT_TYPE in settings (or default to
    "internalURL" if OPENSTACK_ENDPOINT_TYPE is not set).

    Change-Id: I19604d702f5f2de80dc4d0f426154fb5a0b9ad78
    Closes-Bug: 1874902
    Signed-off-by: Andy Ning <email address hidden>

commit 602aa48d826d4fafdbd5a82e862bc684d2932af6
Author: Chen, Yan <email address hidden>
Date: Wed Mar 25 02:59:20 2020 +0000

    [PATCH 12/16] stx-upstream: upgrade neutronclient for Train

    Current Version: 6.14.0

    Story#2007589 is created to track the task to clean up the dependency
    of neutronclient, so that we can remove it safely.
    The cleanup task may impact:
     - sysinv
     - stx-gui
     - document for provisioning StarlingX

    Story: 2007502
    Task: 39261

    Depends-on: https://review.opendev.org/#/c/716818
    Change-Id: Ib93605829ccc48f08157a9d37b0d1658bb7d0d78
    Signed-off-by: Chen, Yan <email address hidden>

commit 3a324b66e1e9c601b627d7aa35e1548ead35aee0
Author: chron.xie <email address hidden>
Date: Tue Apr 7 15:36:03 2020 +0800

    update upstream project for horizon train upgrade.

    This change is based on openstack stable/train.

    Story: 2007502
    Task: 39268

    Depends-on: https://review.opendev.org/#/c/716954/1
    Change-Id: Ifd5adbcbd7ccd124c27b859619f21b3a7cd55de7
    Signed-off-by: Xie Quqiang <email address hidden>

commit d5f2693324302998f725898a4e05dc8c2c9d849b
Author: Tee Ngo <email address hidden>
Date: Fri Apr 17 19:27:31 2020 -0400

    Remove hardcoded endpoint type for keystone

    In this commit, Horizon in upstream repo is patched to
    remove the hardcoded endpoint type for keystone so that
    the default endpoint type specified in settings
    (OPENSTACK_ENDPOINT_TYPE) can take effect uniformly across
    all services. This change is also required in order to
    support intercloud communications securely and uniformly
    over mgmt interface via amdin endpoint type.
  ...