Skip to content

Tag named "latest" can conflict with RTD "latest" #534

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

Closed
metalivedev opened this issue Nov 7, 2013 · 7 comments · Fixed by #3913
Closed

Tag named "latest" can conflict with RTD "latest" #534

metalivedev opened this issue Nov 7, 2013 · 7 comments · Fixed by #3913
Labels
Improvement Minor improvement to code Needed: design decision A core team decision is required

Comments

@metalivedev
Copy link

I found that our repo was stuck on an old version because we had a tag named "latest"

@metalivedev
Copy link
Author

Suggested fix:

RTD should issue a warning if the user's repo includes a tag named latest as this can apparently conflict with RTD's definition of "latest".

The suggested fix is to remove the tag from the repo and wipe the repo:

# Delete the tag locally
git tag -d latest
# Push the tag change upstream
git push origin :refs/tags/latest

then for the wipe, see the RTD docs

@gregmuellegger gregmuellegger added Improvement Minor improvement to code Needed: design decision A core team decision is required labels Jul 21, 2015
@agjohnson
Copy link
Contributor

Agreed. However if we do accept a latest version, we should probably have a method for reverting the latest version as we originally saw it -- a branch -- to our automatic increment latest version.

@humitos humitos self-assigned this Dec 29, 2017
@humitos humitos removed their assignment Jan 23, 2018
@humitos
Copy link
Member

humitos commented Jan 23, 2018

I see two different ideas here:

  1. suggest the user that latest tag/branch is incompatible with RTD and won't make RTD work properly
  2. give the user the ability to choose if he/she wants to use the latest from his/her repository or the auto-created by RTD

Which one is the best one? or both?

@humitos
Copy link
Member

humitos commented Apr 4, 2018

From my understanding this is happening because we don't remove the Version instances that are not matching to a branch/tag.

So, if the original branch was called latest we create a Version instance with the attribute machine=False (which means that this is not a version automatically created by RTD). Then, if you remove the branch from the git repository, we don't remove&create that Versions with the machine=True and it gets stuck in an inconsistent state.

There are a similar issue regarding not deleting/syncing the Version properly under #3887 with more information about this.

@humitos
Copy link
Member

humitos commented Apr 9, 2018

@stsewd does the fix that we talked the other day will fix this problem also, right? (we do have those 2 special cases to consider in the sync method: stable and latest)

If you haven't done yet, could you add a test case for the latest case also in #3887? Thanks.

@stsewd
Copy link
Member

stsewd commented Apr 10, 2018

@humitos yeah, I think this is the same bug

@stsewd
Copy link
Member

stsewd commented May 10, 2018

Closing in favor of #3887

@stsewd stsewd closed this as completed May 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Improvement Minor improvement to code Needed: design decision A core team decision is required
Projects
None yet
Development

Successfully merging a pull request may close this issue.

5 participants