Skip to content

Inverted pulling of head/tags on latest builds #12029

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
davfsa opened this issue Feb 26, 2025 · 6 comments · Fixed by #12121
Closed

Inverted pulling of head/tags on latest builds #12029

davfsa opened this issue Feb 26, 2025 · 6 comments · Fixed by #12121
Assignees
Labels
Accepted Accepted issue on our roadmap Bug A bug

Comments

@davfsa
Copy link

davfsa commented Feb 26, 2025

Details

When fetching the branch/tags for latest builds, the method for fetching branches and tags is inversed, leading to failures when fetching them

Building a branch as latest
https://readthedocs.org/projects/hikari-py/builds/27320926/
Image

Building a tag as latest
https://readthedocs.org/projects/hikari-py/builds/27320945/
Image

Expected Result

Fetch succeeds

Actual Result

Fetch fails

@humitos
Copy link
Member

humitos commented Feb 26, 2025

Can you fill the template with the URL of you build, etc?

@davfsa
Copy link
Author

davfsa commented Feb 26, 2025

Can you fill the template with the URL of you build, etc?

Meant to do that and then seem to have forgotten when adding the images, sorry!

Updated now @humitos!

@humitos
Copy link
Member

humitos commented Feb 26, 2025

This seems like a very basic case that it should work out of the box and that it should have been working for a long time. @stsewd did we change anything recently?

@davfsa
Copy link
Author

davfsa commented Feb 26, 2025

This seems like a very basic case that it should work out of the box and that it should have been working for a long time

Funnily enough, this is something that has broken in the past (see
#10735). It seems to be (at least) a bit unexpected behaviour and not really something that is thought about a lot when implementing it (ref: #10735 (comment)).

But this seems like some boolean got inverted somewhere. It is weird that nobody else is complaining about it, since I would expect this to break in more cases

@davfsa
Copy link
Author

davfsa commented Mar 3, 2025

Found out some more information on this issue.

I decided to check the builds for our page and it seems like latest is pulling correctly now. So it seems that updates coming from the branch receiving pushes are pulled properly, the issue comes when you update the default branch option (for reference, I am using the new portal at https://app.readthedocs.org/dashboard/hikari-py/edit/)

Image

Screenshots of me trying this out. Failed builds are caused by me changing the value of the option and the successful ones are from pushes to the master branch. Also interesting to note, if I press the "Rebuild version" button, the build retries and succeeds this time (which is that build that is loading)

EDIT: This seems to only happen when switching from a tag to a branch. When switching from a branch to a branch or tag to tag, this doesn't happen

Image

Hope this helps track the bug down, as I tried to have a look and I am stumped on what could be the code causing it

@stsewd
Copy link
Member

stsewd commented Apr 17, 2025

Hi, this is indeed a problem when changing the default version from a branch to a tag or vice versa, when triggering a build we resync all versions and the state gets corrected.

@stsewd stsewd added Bug A bug Accepted Accepted issue on our roadmap labels Apr 17, 2025
@humitos humitos moved this from Planned to Needs review in 📍Roadmap Apr 22, 2025
@github-project-automation github-project-automation bot moved this to Planned in 📍Roadmap Apr 22, 2025
@github-project-automation github-project-automation bot moved this from Needs review to Done in 📍Roadmap Apr 22, 2025
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
Projects
Archived in project
Development

Successfully merging a pull request may close this issue.

3 participants