abnormal exit when do config_management

[Austin Sun]

Brief Description
-----------------
When run config_management in virtual machine(KVM/libvirt), it will throw
         Retrieving neighbor details... Traceback (most recent call last):
         File "/usr/bin/config_management", line 9, in <module>
         load_entry_point('controllerconfig==1.0.0', 'console_scripts', 'config_management')()
         File "/usr/lib64/python2.7/site-packages/controllerconfig/config_management.py", line 163, in main
         configure_management()
          File "/usr/lib64/python2.7/site-packages/controllerconfig/config_management.py", line 70, in configure_management
           lldp_interfaces = json.loads(lldpcli_show_output)['lldp'][0]['interface']
         KeyError: 'interface'
and exit abnormally

Severity
--------
Provide the severity of the defect.
<Minor: System/Feature is usable with minor issue>

Steps to Reproduce
------------------
Write down steps to reproduce the issue
install StarlingX ISO
run config_management command in Virtual Machine, then this issue is shown.

Expected Behavior
------------------
Write down what was expected after taking the steps written above
can continue config even lldp return null

Actual Behavior
----------------
The command throw exception and exit

Reproducibility
---------------
<Reproducible/Intermittent>

in libvirt/VM , it is 100%
System Configuration
--------------------
AIO/Standard Controller

Branch/Pull Time/Commit
-----------------------
master
Timestamp/Logs
--------------
Provide a snippet of logs if available and the timestamp when issue was seen.
Please indicate the unique identifier in the logs to highlight the problem
Provide a pointer to the logs for debugging (use attachments in Launchpad or paste.openstack.org)

 This bug is a security vulnerability
The security group for StarlingX will be notified.

[OpenStack Infra (hudson-openstack)]

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

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/629375
Committed: https://git.openstack.org/cgit/openstack/stx-config/commit/?id=a6c51fe49e7f0703849cd2bde822ade793d7755f
Submitter: Zuul
Branch: master

commit a6c51fe49e7f0703849cd2bde822ade793d7755f
Author: Sun Austin <email address hidden>
Date: Tue Jan 8 11:41:46 2019 +0800

    Make lldp check neighbors more robust for config_management

    it can not find neighbors in some env and will throw exception
    when config_management is running. this change will catch all exception
    and not break config.

    Closes-bug: 1811040

    Change-Id: I11b73a647addddf198723bc2654dac444d82d165
    Signed-off-by: Sun Austin <email address hidden>

[OpenStack Infra (hudson-openstack)]

Fix proposed to branch: f/stein
Review: https://review.openstack.org/631263

[OpenStack Infra (hudson-openstack)]

Reviewed: https://review.openstack.org/631263
Committed: https://git.openstack.org/cgit/openstack/stx-config/commit/?id=603cc28734e17be92269832a7b041d414db9a331
Submitter: Zuul
Branch: f/stein

commit 3b4acbfc38f01f9e600d7fb41f0dbd55ba2166e4
Author: Wei Zhou <email address hidden>
Date: Fri Jan 11 15:26:20 2019 -0500

    /etc/ceph/ceph.client.None.keyring not created on standby controller

    This commit ensures that file /etc/ceph/ceph.client.None.keyring is
    created on both controllers after ceph backend is added.

    Change-Id: I49c7b540caca18bada7166c9e7fb2f4b3475a5d3
    Closes-Bug: 1810440
    Signed-off-by: Wei Zhou <email address hidden>

commit 1b8a344edd3c69c671ecce7c65a3c6822fff91f8
Author: Bart Wensley <email address hidden>
Date: Tue Jan 15 09:46:51 2019 -0600

    Ensure /etc/kubernetes/manifests directory exists

    Ensure the /etc/kubernetes/manifests directory exists before
    starting kubelet. This will avoid repeating kubelet logs on
    worker nodes, complaining that the directory does not exist.

    Also removing an unused yum config file for the kubernetes
    repo.

    Change-Id: Id63287481f9fe063f1b8c993d973a4b4517abd81
    Story: 2004520
    Task: 28908
    Signed-off-by: Bart Wensley <email address hidden>

commit 7dd943fe46681ff361533b65807416fc44adf971
Author: Eric MacDonald <email address hidden>
Date: Mon Jan 14 15:22:06 2019 -0500

    Fix mtce.pp to handle missing /etc/rmonfiles.d directory

    https://review.openstack.org/#/c/628687/ stopped packaging the
    query_ntp_servers.sh script. However, since there were no other
    files being packaged into that directory the spec file choose
    not to create an empty directory.

    When config controller called the mtce.pp manifest to install
    dynamic files into /etc/rmonfiles.d it could not. So it failed.

    This update adds a directory check block to the mtce.pp file
    to create the directoy if its not present.

    Testing: Install AIO SX in SM1

    Change-Id: Ib2dfadb261be6f9ebbaa7213eb6669b25158c779
    Closes-Bug: 1811693
    Signed-off-by: Eric MacDonald <email address hidden>

commit f44717154a78add0a0a2497048e03ab536dc615b
Author: Alex Kozyrev <email address hidden>
Date: Thu Jan 10 18:43:14 2019 -0500

    Add Barbican bootstrap and runtime manifests

    Barbican service is needed during bootstrap phase for StarlingX.
    Implement bootstrap and runtime manifests to achieve that.

    Change-Id: I6c22ebddacf8aec3a731f7f6d7a762f79f511c78
    Story: 2003108
    Task: 27700
    Signed-off-by: Alex Kozyrev <email address hidden>

commit f2a734a5f1d07603cde45dab675000688e87890a
Author: Joseph Richard <email address hidden>
Date: Tue Jan 8 11:41:07 2019 -0500

    Set bridge-nf-call-arptables

    This commit sets net.bridge.bridge-nf-call-arptables=1, which was
    found to be required for a containerized neutron deployment.

    Story: 2004520
    Task: 28789
    Change-Id: I0163535ed6bac7e775a63dfa3d54727c60f4208d
    Signed-off-by: Joseph Richard <email address hidden>

commit 8d14f3cd52392e1b3341f4d36a...

[Ghada Khalil (gkhalil)]

Adding stx.2019.05 tag as this fix was merged for that release