Skip to content
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

Missing installation ID after repo rename #4426

Closed
premun opened this issue Feb 6, 2025 · 2 comments
Closed

Missing installation ID after repo rename #4426

premun opened this issue Feb 6, 2025 · 2 comments
Assignees

Comments

@premun
Copy link
Member

premun commented Feb 6, 2025

We have another case of repo renaming and BAR missing its installation ID.
When this happens, users are unable to create subscriptions because the DB is missing the installation ID row for the renamed repository.

We can actually go and query the ID easily so we can do that for the user when they try to create the first subscription for the new repo:
https://docs.github.com/en/rest/apps/apps?apiVersion=2022-11-28#get-a-repository-installation-for-the-authenticated-app

We need the installation ID to get a token against the repository through the GitHub app.

@premun premun self-assigned this Feb 6, 2025
premun added a commit to premun/arcade-services that referenced this issue Feb 6, 2025
@premun
Copy link
Member Author

premun commented Feb 7, 2025

@dkurepa please roll this out and try creating a subscription targeting https://github.com/dotnet/yarp (they asked for Arcade subscription specifically).

@premun premun assigned dkurepa and unassigned premun Feb 7, 2025
@dkurepa
Copy link
Member

dkurepa commented Feb 10, 2025

The feature implemented in #4427 worked. I was able to create a subscription dotnet/yarp, without manually adding the subscription ID in BAR

@dkurepa dkurepa closed this as completed Feb 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants