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

After 5.7.3 release, some updates with bugzillas never finish loading #4317

Closed
AdamSaleh opened this issue Dec 13, 2021 · 3 comments · Fixed by #4319
Closed

After 5.7.3 release, some updates with bugzillas never finish loading #4317

AdamSaleh opened this issue Dec 13, 2021 · 3 comments · Fixed by #4319

Comments

@AdamSaleh
Copy link
Contributor

Cause of the issue is not yet known.

One such example: https://bodhi.fedoraproject.org/updates/FEDORA-2021-c2b61f2725

Currently investigating.

Will @mattiaverga have some idea?

@AdamSaleh
Copy link
Contributor Author

Running in bodhi-shell:

u = m.Update.query.filter_by(alias='FEDORA-2021-c2b61f2725').one()
[b.update_details() for b in u.bugs]
m.Session().commit()

did manually fix the issue. This could suggest problem is somewhere in celery-worker?

@mattiaverga
Copy link
Contributor

@AdamSaleh yeah, I know the cause: creating sidetag rawhide updates never calls the work_on_bugs celery task.
We should find an appropriate place where to call the task for this type of updates, like we do for the other, or we can find a place where to call it for all types (maybe in the update.init() method, which I think it's the only common place for all).

This has been on my todo list for a while, but always postponed since it's not causing any real issue apart of the missing title in the web UI (the loading animation is just a gif, nothing is really loading in the background).

@AdamSaleh
Copy link
Contributor Author

Good! I will mention it to releng guys, that this shouldn't signify an issue, more of a UX problem.

I wonder if we should have such hook directly in the model somewhere? Or is that a bad idea?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants