Skip to content

Call new endpoints #102

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
Jul 1, 2025
Merged

Call new endpoints #102

merged 1 commit into from
Jul 1, 2025

Conversation

NicoHinderling
Copy link
Contributor

@NicoHinderling NicoHinderling commented Jun 30, 2025

Add endpoint calling interface. The test scripts are just temporary references for now. Once my kafka PRs land, we can do this on the latest main branches of the two projects:

  1. (optional) run make reset-db in sentry dir for a clean slate
  2. Run sentry locally: sentry devserver --workers
  3. Open http://127.0.0.1:8000/settings/sentry/auth-tokens and create an auth token.
  4. Open the launchpad repo and run make serve to ensure the launchpad server is also running locally
  5. Open sentry-cli and run ./target/release/sentry-cli --log-level=debug --auth-token {your_auth_token} mobile-app upload {any_apk_or_xcarchive} --org sentry --project internal. This should successfully upload the artifact to your local sentry instance. You can check in your local postgres: there should now be a row in the PreprodArtifact and sentry_file tables

Step #4 will also submit a kafka message which will then trigger the processing task in launchpad.

My next PR will wire up the processing task to include the network calls back to the monolith

For now we can test these endpoints with the make commands I've added

note: we've updated the sentry's devservices config so technically you can also now run launchpad as a dependency of sentry by running devservices up --mode launchpad in the sentry repo

@NicoHinderling NicoHinderling force-pushed the call-new-endpoints branch 2 times, most recently from f526e1c to 191ab4e Compare June 30, 2025 22:41
@NicoHinderling NicoHinderling marked this pull request as ready for review July 1, 2025 20:41
@NicoHinderling NicoHinderling force-pushed the call-new-endpoints branch 2 times, most recently from c769652 to 7905d2f Compare July 1, 2025 21:03
Copy link
Contributor Author

NicoHinderling commented Jul 1, 2025

This stack of pull requests is managed by Graphite. Learn more about stacking.

@NicoHinderling NicoHinderling changed the base branch from main to graphite-base/102 July 1, 2025 21:05
@NicoHinderling NicoHinderling changed the base branch from graphite-base/102 to replace-kafka-consumer-with-more-arroyo July 1, 2025 21:05
@NicoHinderling NicoHinderling force-pushed the replace-kafka-consumer-with-more-arroyo branch from e33a9be to 60e1f4e Compare July 1, 2025 21:26
@NicoHinderling NicoHinderling changed the base branch from replace-kafka-consumer-with-more-arroyo to graphite-base/102 July 1, 2025 22:18
@NicoHinderling NicoHinderling changed the base branch from graphite-base/102 to main July 1, 2025 22:18
@NicoHinderling NicoHinderling merged commit dbc7bb6 into main Jul 1, 2025
6 checks passed
@NicoHinderling NicoHinderling deleted the call-new-endpoints branch July 1, 2025 22:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants