Skip to content

Conversation

jan-auer
Copy link
Member

@jan-auer jan-auer commented May 29, 2019

Creates an assemble-like endpoint for uploading artifact bundles.

Chunk Upload: Bundles are uploaded via the chunk upload. To accommodate for this, the options endpoint now returns "accept": ["debug_files", "release_files"] to indicate to sentry-cli that the endpoint is more capable now.

Assemble: Common code for chunk uploading and managing assemble state is moved into the sentry.tasks.assemble module. The cache key is now variable based on the task (but backward compatible).

Persistence: The artifact bundle is not persisted for now. Each file is individually unpacked and then stored as release file. Finally, the artifact bundle is removed from Sentry.

Bug Fixes: This PR also fixes a bug that might cause an endless poll in sentry-cli if an exception is thrown in the assemble task. This is achieved by catching all exceptions and setting the assemble status to a generic "internal server error".

Artifact Bundles are ZIP archives with the following structure:

manifest.json
files/
  index.js
  index.js.1       # name conflict
  index.js.map

Copy link
Member

@HazAT HazAT left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Note, we did an offline review for this earlier.
Only have one nitpick.

@jan-auer jan-auer requested a review from mitsuhiko May 30, 2019 18:51
* master: (30 commits)
  ref(ui): Avoid full-page load indicator after project creation (#12842)
  ref(events-v2) Use an object target instead building URLs (#13471)
  ref(onboarding): Improve alerts from wizard docs (#13469)
  feat(app-platform): integration feature models, serializer, endpoints (#13377)
  ref(onboarding): Improve platform picker hover color (#13468)
  chore: New SDK versions (#13465)
  feat(events-v2) Add rough sketch of event modal (#13415)
  chore(south) Update south state to use new jsonfield (#13466)
  fix(events-v2): Fix search filter (#13454)
  chore: Vendor JSONField to fix runtime warnings and reduce future upgrade pain (#13397)
  test(events-v2): Fix dynamic values in Percy (#13463)
  dx(notion): Upgrade from `notion` to `volta` (#13452)
  feat(onboarding): Add warning for when docs are missing examples (#13445)
  misc(metrics): First pass at un-sampled single metric paths. (#13434)
  fix(integrations) Fix missing identity link on re-install (#13450)
  fix(api): Fix bug when creating incident comments while logged in as another user.
  fix(api): Fix task error on incident creation
  chore(SDK's): Update various SDK versions/urls (#13225)
  chore: Make exports and imports match (#13449)
  feat(ui): Add message in Incidents list for creating an incident [SEN-694] (#13436)
  ...
@jan-auer jan-auer merged commit 209e42f into master May 31, 2019
@jan-auer jan-auer deleted the feat/release-artifacts-assemble branch May 31, 2019 08:33
@github-actions github-actions bot locked and limited conversation to collaborators Dec 20, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants