Status Update
Comments
lu...@gmail.com <lu...@gmail.com> #2
Hi Nasser, The comment you are trying to publish was created on the 7th of November:
commit aef9a6611d57d829f47b11f3c487d3ab11e4e2d9 (refs/draft-comments/34/1203734/1022030)
Author: Gerrit User 1022030 <1022030@d5d70762-12d0-45a1-890d-524b12d3f735>
Date: Thu Nov 7 19:54:39 2024 +0000
Update draft comments
You have then published the draft comments on the 14th of November on review.gerrithub.io:
pull_replication_log.2024-11-14.gz:[2024-11-14 23:14:47,597] Ref event received: DELETE on project All-Users:refs/draft-comments/34/1203734/1022030 - aef9a6611d57d829f47b11f3c487d3ab11e4e2d9 => 0000000000000000000000000000000000000000
Because of a bug on the multi-site plugin that I've found and
I've removed them manually now, and you shouldn't see them anymore.
na...@oss.qualcomm.com <na...@oss.qualcomm.com> #3
lu...@gmail.com <lu...@gmail.com> #4
Let me double-check.
na...@oss.qualcomm.com <na...@oss.qualcomm.com>
lu...@gmail.com <lu...@gmail.com> #5
This new comment has been published on eclipse.gerrithub.io yesterday:
sharedref_log.2024-11-20.gz:[2024-11-20 05:14:24,306] [FanOut-1[com.google.gerrit.server.notedb.AllUsersAsyncUpdate$$Lambda$5753/0x00007f8a1d478b58@1f146bf1]] INFO : {"ref_name":"refs/draft-comments/44/1202844/1022030","old_id":"9416130cb8257a340bb8aa887802d45d09a6af68","project_name":"All-Users","type":"DELETE_REF"}
I haven't rolled out yet the fix for the
If you try to do the same on
na...@oss.qualcomm.com <na...@oss.qualcomm.com> #6
Thanks. If it helps, this is what I get when I copy the request as cURL from Chrome dev tools:
curl 'https://review.gerrithub.io/changes/eclipse-jgit%2Fjgit~1202844/revisions/11/drafts' \
-X 'PUT' \
-H 'accept: */*' \
-H 'accept-language: en-US,en;q=0.9' \
-H 'cache-control: no-cache' \
-H 'content-type: application/json' \
-H 'cookie: scope=scopes; GerritAccount=<redacted>; XSRF_TOKEN=<redacted>; JSESSIONID=<redacted> \
-H 'origin: https://review.gerrithub.io' \
-H 'pragma: no-cache' \
-H 'priority: u=1, i' \
-H 'sec-ch-ua: "Chromium";v="130", "Google Chrome";v="130", "Not?A_Brand";v="99"' \
-H 'sec-ch-ua-mobile: ?0' \
-H 'sec-ch-ua-platform: "macOS"' \
-H 'sec-fetch-dest: empty' \
-H 'sec-fetch-mode: cors' \
-H 'sec-fetch-site: same-origin' \
-H 'user-agent: Mozilla/5.0 (Macintosh; Intel Mac OS X 10_15_7) AppleWebKit/537.36 (KHTML, like Gecko) Chrome/130.0.0.0 Safari/537.36' \
-H 'x-gerrit-auth: <redacted>' \
--data-raw '{"message":"nit: should be `packs`\n\n```suggestion\n\t * tag refs into packed-refs. If {@code all} flag is set, this packs all the\n```","unresolved":true,"path":"org.eclipse.jgit/src/org/eclipse/jgit/internal/storage/file/RefDirectory.java","side":"REVISION","line":296,"range":{"start_line":296,"start_character":64,"end_line":296,"end_character":68}}'
lu...@gmail.com <lu...@gmail.com> #8
I'm doing an emergency upgrade of the 1st site serving
na...@oss.qualcomm.com <na...@oss.qualcomm.com> #9
Does this issue also affect has:draft
somehow? I'm seeing on both eclipse.gerrithub.io and review.gerrithub.io that
ma...@gmail.com <ma...@gmail.com> #10
I also have non-existing draft comments listed for the following changes: 1199465 1201738 1203402
lu...@gmail.com <lu...@gmail.com> #11
@Matthias @Nasser yes, it does affect them. It's a long story that started when we realised that Gerrit was missing events for draft comments. As soon as the rollout is complete on all sites, this problem will finally disappear :-)
Description
GerritHub/GitHub username: nasser-quicinc
GerritHub/GitHub projects names: eclipse-jgit/jgit
Full URL exhibiting the problem:https://eclipse.gerrithub.io/c/eclipse-jgit/jgit/+/1203734/4..7/org.eclipse.jgit/src/org/eclipse/jgit/diff/UserDiff.java
Expected behavior: Saving draft comment completes and allows continued navigation on the change
Observed behavior: Draft widget shows "Draft (Failed to save)" and pop-up with:
Timestamp when the error occurred (include timezone): 2024-11-19T18:38:27.302Z
Has this worked before? If yes, when? Yes. 2024-11-18.
If applicable: screenshot of erroneous UI element:
If applicable: JavaScript console log (chrome: F12 > Console):
If applicable: Browser (name, version, operating system): Chrome, Version 130.0.6723.117, MacOS
If using git command-line: output of "which git && git --version"