This change fixes the obvious race for a status_set() between
check_optional_interfaces() and assess_status() as the later calls the former
which calls status_set(), returns the status, which is then potentially set
again by the assess_status() function. This cleans up the code so that only a
single status_set() is performed when calling assess_status().
Reviewed: https:/ /review. openstack. org/325650 /git.openstack. org/cgit/ openstack/ charm-glance/ commit/ ?id=1ab2c809ad7 251f6d8a30c053d 8c7818ac051547
Committed: https:/
Submitter: Jenkins
Branch: master
commit 1ab2c809ad7251f 6d8a30c053d8c78 18ac051547
Author: Alex Kavanagh <email address hidden>
Date: Sun Jun 5 12:57:53 2016 +0000
Fix for status-set race - related to bug 1588462
This change fixes the obvious race for a status_set() between optional_ interfaces( ) and assess_status() as the later calls the former
check_
which calls status_set(), returns the status, which is then potentially set
again by the assess_status() function. This cleans up the code so that only a
single status_set() is performed when calling assess_status().
Change-Id: I06efbcaade8f0c 99b5931104e6887 d24cb10e35a Bug:#1588462
Related-