diff options
author | James E. Blair <jim@acmegating.com> | 2021-12-14 10:52:36 -0800 |
---|---|---|
committer | James E. Blair <jim@acmegating.com> | 2021-12-14 11:00:40 -0800 |
commit | a2714c370a05d80f1d947c6344eda89a4902ec42 (patch) | |
tree | f05206f6979ffe272e05476e25bb0dddeb75a216 /tests/fixtures/layouts | |
parent | 9a2fd432f1ee87c8b9cb9fbb89e15590c544b844 (diff) | |
download | zuul-a2714c370a05d80f1d947c6344eda89a4902ec42.tar.gz |
Fix merging reconfiguration events
Reconfiguration events for the same tenant are supposed to be merged
so that if many branches are created, only one reconfiguration happens.
However, an error existed in the merge method which raised an exception
preventing the merge, and the exception handler around the merge method
was too broad and therefore suppressed the error.
This is well tested, but with tests that look more like "unit" tests
rather than functional tests. They did not contain the branch_cache_ltime
dict which triggered the error.
This patch corrects the error and updates the test to be more realistic.
Change-Id: I0b5c7628a0580db55b56eec8c081ebee4d31d989
Diffstat (limited to 'tests/fixtures/layouts')
0 files changed, 0 insertions, 0 deletions