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/329219 /git.openstack. org/cgit/ openstack/ charm-nova- cloud-controlle r/commit/ ?id=b570e36cba9 3affe67594ef820 afae731b746ad7
Committed: https:/
Submitter: Jenkins
Branch: master
commit b570e36cba93aff e67594ef820afae 731b746ad7
Author: David Ames <email address hidden>
Date: Mon Jun 13 15:55:40 2016 -0700
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: Ie37a4d98de9c5e 7bd26304e096796 ce6287ea52b Bug:#1588462
Related-