commit 4eb61941f811a8ab96745a5ea07e6a4a89fe764c
Author: Ben Nemec <email address hidden>
Date: Wed Jan 23 15:13:12 2019 +0000
Avoid double-setting event
In commit cc8b51e1e16f6bdc7d6c0e571e2002e70cde098d we added a send
on the old event when clearing an EventletEvent. However, this was
done unconditionally, which means if the event was already sent
then we attempt to send it again. This fails with:
AssertionError: Trying to re-send() an already-triggered event.
Similar to 14a53c4d8a9d5605b14a503803859df0d6d4b820, we should check
if self._set is True and if so then we know that the event was
already sent and we don't need to do it again.
Reviewed: https:/ /review. openstack. org/632758 /git.openstack. org/cgit/ openstack/ oslo.utils/ commit/ ?id=4eb61941f81 1a8ab96745a5ea0 7e6a4a89fe764c
Committed: https:/
Submitter: Zuul
Branch: master
commit 4eb61941f811a8a b96745a5ea07e6a 4a89fe764c
Author: Ben Nemec <email address hidden>
Date: Wed Jan 23 15:13:12 2019 +0000
Avoid double-setting event
In commit cc8b51e1e16f6bd c7d6c0e571e2002 e70cde098d we added a send
on the old event when clearing an EventletEvent. However, this was
done unconditionally, which means if the event was already sent
then we attempt to send it again. This fails with:
AssertionError: Trying to re-send() an already-triggered event.
Similar to 14a53c4d8a9d560 5b14a503803859d f0d6d4b820, we should check
if self._set is True and if so then we know that the event was
already sent and we don't need to do it again.
Change-Id: I660601383072d1 1e4a077aada8c1b 8c30b9d8d1d
Closes-Bug: 1812922