diff options
| author | James E. Blair <jim@acmegating.com> | 2022-03-28 14:08:08 -0700 |
|---|---|---|
| committer | James E. Blair <jim@acmegating.com> | 2022-03-28 15:08:45 -0700 |
| commit | 38770a7867f2cf9ed18a2aa6607e655377bf42e1 (patch) | |
| tree | e072a22d13c99bc6972a0455cc23070c670bc249 /releasenotes | |
| parent | 9bd930110955438ca4b6721c39b6e685e33e1fab (diff) | |
| download | zuul-38770a7867f2cf9ed18a2aa6607e655377bf42e1.tar.gz | |
Fix recursive Gerrit change query
We have a history memo list which we use to avoid infinite recursion
in the Gerrit change query, however instead of mutating it, we copy
it. This avoids infinite recursion in some circumstances, but at the
cost of exponential repitition. It may also allow infinite recursion
depending on the order of results in Gerrit query responses.
To correct this, mutate the memo list instead of duplicating it.
Change-Id: Ic892cf5aa2db2f5e11f8562f857d9ec0a6b60c0a
Diffstat (limited to 'releasenotes')
| -rw-r--r-- | releasenotes/notes/submitted-together-query-f2a04313193dd101.yaml | 5 |
1 files changed, 5 insertions, 0 deletions
diff --git a/releasenotes/notes/submitted-together-query-f2a04313193dd101.yaml b/releasenotes/notes/submitted-together-query-f2a04313193dd101.yaml new file mode 100644 index 000000000..4cad1389f --- /dev/null +++ b/releasenotes/notes/submitted-together-query-f2a04313193dd101.yaml @@ -0,0 +1,5 @@ +--- +fixes: + - | + Fixed an issue in the Gerrit driver introduced in 5.2.0 which + could cause infinite loops while querying changes. |