Gerrit changes no longer syncing to Github [351843807]

Fixed

Bug

Host-Gerrithub

Status Update

No update yet.

Description

hj...@cornell.edu

created issue #1

Jul 9, 2024 01:25AM

** If you have a GerritForge Enterprise Support contract, file a bug through

https://support.gerritforge.com instead **
** If you are interested in activating a new Support contract, use the

https://gerritforge.com/contact form instead **
** This is for bugs in Gerrit sites hosted at *.

gerrithub.io **

GerritHub/GitHub username:
arklian

GerritHub/GitHub projects names:
arklian/patina
arklian/patina2
arklian/TestRepo

Full URL exhibiting the problem:

https://gerrithub.io/plugins/gitiles/arklian/TestRepo/+log/3e0806be0e26e99626d110f56491ac3f75ac9477/project.config

https://github.com/arklian/TestRepo/commits/3e0806be0e26e99626d110f56491ac3f75ac9477/project.config

https://github.com/arklian/TestRepo/commits/refs/meta/config

https://github.com/arklian/TestRepo/commits/main/

https://gerrithub.io/q/project:arklian/TestRepo

Expected behavior:
Github should sync changes from Gerrit.

Observed behavior:
Both refs/meta/config and main are no longer syncing in all 3 repos.

Timestamp when the error occurred (include timezone):
Approx. 3AM EST

Has this worked before? If yes, when?
Before I cloned arklian/patina, the syncs seemed to be working fine for both TestRepo and arklian/patina2

Thanks!

Comments

lu...@gmail.com <lu...@gmail.com> #2Jul 9, 2024 11:50AM

Accepted by lu...@gmail.com.

This issue was caused by a bug in the replication plugin on master. I've manually patched production and looking at the fix to apply to the replication plugin code.

lu...@gmail.com <lu...@gmail.com> #3Jul 9, 2024 06:50PM

Marked as fixed.

Fixed with https://gerrit-review.googlesource.com/c/plugins/replication/+/432879

hj...@cornell.edu <hj...@cornell.edu> #4Jul 9, 2024 07:34PM

Thanks for the quick fix! Can confirm that the replication looks fixed!

ap...@google.com <ap...@google.com> #5Jul 10, 2024 02:38PM

Project: plugins/replication
Branch: master

commit 3982574d28f254b525b093c2ecae8caa47825910
Author: Luca Milanesio <luca.milanesio@gmail.com>
Date: Tue Jul 09 19:46:42 2024

Fix FanoutConfigurationResource update API

When updating the FanoutConfigurationResource, the remote subsection
needs to be removed as the subsection name is part of the config
filename.

Bug:

Issue 351843807

Change-Id: Ib434a434a82de7e237387c7d7df8eb0e78cfa5a6

M src/main/java/com/googlesource/gerrit/plugins/replication/FanoutConfigResource.java
M src/test/java/com/googlesource/gerrit/plugins/replication/FanoutConfigResourceTest.java

https://gerrit-review.googlesource.com/432879