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

[Support] nbgitpuller link breaks after update - University of Toronto #3603

Closed
GeorgianaElena opened this issue Jan 10, 2024 · 11 comments
Closed
Assignees
Labels
support Issues that track Freshdesk tickets

Comments

@GeorgianaElena
Copy link
Member

The Freshdesk ticket link

https://2i2c.freshdesk.com/a/tickets/1229

Ticket request type

Something is not working

Ticket impact

🟥 Critical

Short ticket description

Nbgitpuller links not working.

When the hub is:

(Optional) Investigation results

No investigation happened yet as I wanted to first open the issue for visibility.

@GeorgianaElena GeorgianaElena added the support Issues that track Freshdesk tickets label Jan 10, 2024
@GeorgianaElena GeorgianaElena changed the title [Support] {{ Ticket name }} [Support] nbgitpuller link breaks after update - University of Toronto Jan 10, 2024
@GeorgianaElena
Copy link
Member Author

nbgitpuller works with https://r.datatools.utoronto.ca/ if the repo is public.

However the repo in the ticket is private and it was reported that it worked on Monday, I believe before the migration?

@GeorgianaElena
Copy link
Member Author

I am confused of the fact that I don't see evidence that private GitHub pulling was setup for the toronto hub https://infrastructure.2i2c.org/howto/features/private-nbgitpuller/#allow-nbgitpuller-to-pull-from-private-github-repos

@GeorgianaElena
Copy link
Member Author

GeorgianaElena commented Jan 10, 2024

Ahh, the GitHub app is not under the 2i2c org, but in https://github.com/organizations/utoronto-2i2c/settings/apps/ and this is why I couldn't find it.

git-credentials-helper is setup at

gitconfig:
mountPath: /etc/gitconfig
# app-id comes from https://github.com/organizations/utoronto-2i2c/settings/apps/utoronto-jupyterhub-private-cloner
stringData: |
[credential "https://github.com"]
helper = !git-credential-github-app --app-key-file /etc/github/github-app-private-key.pem --app-id 93515
useHttpPath = true

@GeorgianaElena
Copy link
Member Author

git-credentials-helper is available in the original utoronto hub image https://github.com/2i2c-org/utoronto-image/blob/14320bae73a0570aa1ab1b19c68758894b291979/environment.yml#L42 but not in the new r-image.

2i2c-org/utoronto-r-image#17 should make it available

@GeorgianaElena
Copy link
Member Author

Opened #3604 to move the setup of the https://github.com/organizations/utoronto-2i2c/settings/apps/utoronto-jupyterhub-private-cloner app under the default hub, as I believe we need a new one for the R hub

@GeorgianaElena
Copy link
Member Author

Hmm, the weird thing is that using nbgitpuller with a private repo on https://jupyter.utoronto.ca doesn't work either, although it seems to have everything setup 🤔

@GeorgianaElena
Copy link
Member Author

I've opened #3605 that registeres the new GitHub app created to be used for pulling by the r hub https://github.com/organizations/utoronto-2i2c/settings/apps/utoronto-r-hub-private-cloner

@GeorgianaElena
Copy link
Member Author

yuvipanda added a commit to yuvipanda/pilot-hubs that referenced this issue Jan 10, 2024
@yuvipanda
Copy link
Member

yuvipanda commented Jan 10, 2024

I merged #3604 and then reverted #3606 - it turns out 2i2c-org/utoronto-r-image#17 was actually the only change needed! It does mean that the same GitHub app is used for both jupyter.utoronto.ca and r.datatools.utoronto.ca, but that's actually ok as it makes the migration from jupyter.utoronto.ca to r.datatools.utoronto.ca easier - people will not have to reinstall.

@yuvipanda
Copy link
Member

I opened #3608 to document. @GeorgianaElena do you think it's ok to delete the github app created for the R hub?

We should probably try to bring the github app for the jupyter app to this org! But idk if that means everyone will have to reinstall the app?

@GeorgianaElena
Copy link
Member Author

Thank you @yuvipanda for sorting this out! I've deleted the new app and will close this issue as it's now resolved 🚀

@GeorgianaElena GeorgianaElena added support Issues that track Freshdesk tickets and removed support Issues that track Freshdesk tickets labels Jan 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
support Issues that track Freshdesk tickets
Projects
No open projects
Status: Done 🎉
Development

No branches or pull requests

2 participants