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

Deleted repository not detected #2412

Closed
beruic opened this issue Sep 13, 2016 · 5 comments · Fixed by #7310
Closed

Deleted repository not detected #2412

beruic opened this issue Sep 13, 2016 · 5 comments · Fixed by #7310
Labels
Accepted Accepted issue on our roadmap Bug A bug Needed: design decision A core team decision is required Sprintable Small enough to sprint on
Milestone

Comments

@beruic
Copy link

beruic commented Sep 13, 2016

Details

I have deleted the repository at https://github.com/beruic/bootstrap-tagsinput, but it does not disappear when i refresh in the repository importer at https://readthedocs.org/dashboard/import/.

@humitos humitos added the Support Support question label Feb 23, 2017
@humitos
Copy link
Member

humitos commented May 30, 2017

I confirm that I'm able to reproduce this. I created a repository called 1 in GitHub and I imported without problem to RTD. Then I deleted the project from RTD and also from GitHub. After that I hit re-sync in RTD and I still seeing my 1 repository to be able to import.

If I try to re-imported after deleted from GitHub I receive this error: https://readthedocs.org/projects/my1/builds/5489577/

Failed to import project: Failed to get code from 'https://github.com/humitos/1.git' (git fetch): 128 Stderr: fatal: could not read Username for 'https://github.com': No such device or address

@humitos humitos added Bug A bug and removed Support Support question labels May 30, 2017
@agjohnson
Copy link
Contributor

Correct, our sync repository function doesn't remove existing RemoteRepository objects, nor does it attempt to discover which should be removed.

Some decisions to make here:

  • Is deleting the RemoteRepository object the most correct thing?
  • Should we also disable the project if the repository goes away?

I would say deleting is fine, as long as we ensure it doesn't cascade to connected projects.

I would also say that disabling the project doesn't make sense, as this could simply be a matter of losing access to a fork of a repository. Perhaps we need to think more on this case though.

@agjohnson
Copy link
Contributor

agjohnson commented Sep 19, 2018

Also, related: #2759. Not sure if we block on this issue, but #2759 changes some of the decisions here.

@agjohnson agjohnson added this to the Admin UX milestone Sep 19, 2018
@agjohnson agjohnson added the Accepted Accepted issue on our roadmap label Sep 19, 2018
@beruic
Copy link
Author

beruic commented Sep 21, 2018

I just checked BTW. The deleted repository still appears in my list.

@humitos
Copy link
Member

humitos commented Jul 3, 2020

After #7183, GitHub integration in cleaning up old RemoteRepository from that user only if it's not connected to a Project. This will have impact in the "Import Project" page that won't show stale repositories that do not exist anymore.

The same work/code needs to be ported to GitLab and Bitbucket to be able to close this issue.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Accepted Accepted issue on our roadmap Bug A bug Needed: design decision A core team decision is required Sprintable Small enough to sprint on
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants