sentry-cli icon indicating copy to clipboard operation
sentry-cli copied to clipboard

HTTP 504 when trying to set commits to a release

Open bartek-wesolowski opened this issue 5 years ago • 8 comments

I'm using setnry-cli 1.55.2. I'm running the following command: sentry-cli releases set-commits <my version here> --auto

I'm getting this result:

error: API request failed
  caused by: sentry reported an error: gateway timeout (http status: 504)

This is probably because the number of commits I'm trying to send is quite large (hundreds).

I filed another issue recently in which only 50 commits were attached to a release even if there were more of them. After fixing this, perhaps the server can't handle such big commit numbers.

bartek-wesolowski avatar Sep 01 '20 12:09 bartek-wesolowski

We are seeing the issue as well. Our CI sometimes fails on the first sentry command, but even if it succeeds it always fails on the second.

First:

sentry-cli releases new -p <project> <release>
error: API request failed
  caused by: sentry reported an error: gateway timeout (http status: 504)

Second:

+ sentry-cli releases set-commits --auto <release>
+--------------+--------------+
| Repository   | Revision     |
+--------------+--------------+
| xxxxxxxxxxx | <xxxxxxxxxxx> |
+--------------+--------------+
error: API request failed
  caused by: sentry reported an error: gateway timeout (http status: 504)

alacoste avatar Sep 17 '20 19:09 alacoste

Closing the issue as a part of large repository cleanup, due to it being inactive and/or outdated. Please do not hesitate to ping me if it is still relevant, and I will happily reopen it. Cheers!

kamilogorek avatar Oct 13 '21 14:10 kamilogorek

This still seems to be an issue, a user on Discoed reported that they run into this problem with a release containing 1200+ commits.

cleptric avatar Apr 23 '24 13:04 cleptric

Hi. I am the user on discord 😅 . Yes I'd like to create the first release of the project with 1200+ commits. I am getting the 504 :/. Available if need more details of course.

Neirda24 avatar Apr 23 '24 13:04 Neirda24

@Neirda24, thank you for reaching out to us!

#2026 appears to be related (possibly a duplicate?), but @ankitghosh, the user who opened #2026, was reporting an error with HTTP status 413.

Regardless of whether #2026 is a duplicate, it seems the fix for both issues will be the same: we likely need to introduce an option to limit the number of commits included in a release.

Since both issues could be resolved with such a fix, we can make this a priority!

szokeasaurusrex avatar Apr 24 '24 09:04 szokeasaurusrex

I retried this morning and it worked. all 1284 commits were pushed to sentry. Don't know why it failed yesterday.

Neirda24 avatar Apr 24 '24 09:04 Neirda24

Hmm, strange, glad it worked out for you though, @Neirda24!

szokeasaurusrex avatar Apr 24 '24 11:04 szokeasaurusrex

This issue has gone three weeks without activity. In another week, I will close it.

But! If you comment or otherwise update it, I will reset the clock, and if you label it Status: Backlog or Status: In Progress, I will leave it alone ... forever!


"A weed is but an unloved flower." ― Ella Wheeler Wilcox 🥀

github-actions[bot] avatar May 16 '24 00:05 github-actions[bot]