getsentry/sentry-dart-plugin

integration-tests-sentry-cli.yml failing

marandaneto opened this issue · 7 comments

error: API request failed
caused by: sentry reported an error: unknown error (http status: 501)
DEBUG 2023-03-23 08:17:21.033209 +01:00 skipping update nagger because session is not attended

stdout: > Rewriting sources

Adding source map references
Bundled 0 files for upload
Uploaded release files to Sentry

Stopping the test-server ... /STOP request failed: Unable to read data from the transport connection: Connection reset by peer. - killing the server process instead
ParentContainsErrorRecordException: /Users/user/Github/sentry-dart-plugin/integration-test/scripts/test.ps1:72
Line |
72 | $Callback.Invoke()
| ~~~~~~~~~~~~~~~~~~
| Exception calling "Invoke" with "0" argument(s): "The running command stopped because the preference variable "ErrorActionPreference" or common parameter is set to Stop: Source map not uploaded"

@vaind do you know if something changed in the CLI? didn't check it yet but the integrations are failing.
IIRC upload-dif is now an alias to debug-files upload maybe the APIs changed too? Related to getsentry/team-mobile#97
Just a hunch tho, have not investigated it yet.

vaind commented

Do you mean this change from the latest release? https://github.com/getsentry/sentry-cli/releases/tag/2.15.2

fix: Make sourcemap injection on upload opt-in (https://github.com/getsentry/sentry-cli/pull/1534)

sinde the error you've pasted says Source map not uploaded

Do you mean this change from the latest release? https://github.com/getsentry/sentry-cli/releases/tag/2.15.2

fix: Make sourcemap injection on upload opt-in (https://github.com/getsentry/sentry-cli/pull/1534)

sinde the error you've pasted says Source map not uploaded

No no, I meant that the problem could be related to CLI changes and I just didn't investigate yet but I know one of the changes are related to the upload dif command, it does not mean its this one specifically.

I believe its because:
elif self.isApi('/api/0/projects/{}/{}/releases/{}@{}/files/'.format(apiOrg, apiProject, appIdentifier, version)):

Isn't a match for:

api/0/projects/{}/{}/releases/{}@{}/files/?cursor=&checksum=20c25a4dacf5f92b173ec6645b2c90ce862ce702&checksum=64c32dd2a18966f31e217f9a9f9887190763ae18&checksum=78f4970003bbc31ffb641fc962558f5529ef6646&checksum=94a6c2b75d172363ad9a6b9d225e6e87b0c9673d&checksum=ac1d5af84f0baa65ee952fc9c98c3e635568a5ea&checksum=cb6efe38afc5dfb12cd96fad00fddf88bf85e6d1

There were no query parameters so now the server returns:

DEBUG 2023-03-23 11:14:30.457614 +01:00 body: Error: no API matched GET '/api/0/projects/sentry-sdks/sentry-dart-plugin/releases/project@1.1.0/files/?cursor=&checksum=20c25a4dacf5f92b173ec6645b2c90ce862ce702&checksum=64c32dd2a18966f31e217f9a9f9887190763ae18&checksum=78f4970003bbc31ffb641fc962558f5529ef6646&checksum=94a6c2b75d172363ad9a6b9d225e6e87b0c9673d&checksum=ac1d5af84f0baa65ee952fc9c98c3e635568a5ea&checksum=cb6efe38afc5dfb12cd96fad00fddf88bf85e6d1'

@denrase this part is still very flaky.

Checking symbol server output
Verifying debug symbol upload...
app.so: count=3 - OK
libapp.so: count=3 - OK
libflutter.so: count=6 - OK
libhello_santry.so: count=12 - MISSING
Server received '3' instead.
libhello_santry.so.sym: count=4 - OK
Error: Process completed with exit code 1.

For some reason, libhello_santry.so is constantly changing, not sure why.

vaind commented

@denrase this part is still very flaky.

Checking symbol server output
Verifying debug symbol upload...
app.so: count=3 - OK
libapp.so: count=3 - OK
libflutter.so: count=6 - OK
libhello_santry.so: count=12 - MISSING
Server received '3' instead.
libhello_santry.so.sym: count=4 - OK
Error: Process completed with exit code 1.

For some reason, libhello_santry.so is constantly changing, not sure why.

I'd suggest looking at the reusable workflow... I did away with the strict checks. See a sample test script and CI job in sentry-dotnet.

The test script is invoked using https://github.com/pester/Pester so you can refer to it's docs to understand the available syntax

Closes #118