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.

Sign up for GitHub

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

Jump to bottom

Implement retriggering Bodhi updates via dist-git pull-request comments #1541

Closed
4 tasks
Tracked by #1537
lbarcziova opened this issue Jun 9, 2022 · 4 comments · Fixed by #1729
Closed
4 tasks
Tracked by #1537

Implement retriggering Bodhi updates via dist-git pull-request comments #1541

lbarcziova opened this issue Jun 9, 2022 · 4 comments · Fixed by #1729
Assignees
@majamassarini @nikromen

Comments

@lbarcziova
Copy link
Member

lbarcziova commented Jun 9, 2022 •
edited
Loading

  • retriggering will be done via dedicated Packit command - e.g /packit create-update
  • we need to listen to org.fedoraproject.prod.pagure.pull-request.comment.added messages in fedmsg (example message)
  • user who triggers the jobs should have write access to the dist-git repository
  • get the last build for the given branch
@lbarcziova lbarcziova mentioned this issue Jun 9, 2022
Closed
2 tasks
@nikromen nikromen self-assigned this Jul 11, 2022
@nikromen nikromen mentioned this issue Jul 19, 2022
Merged
@TomasTomecek
Copy link
Member

I spoke with @thozza yesterday and he said he's looking forward to this functionality :)

@thozza
Copy link

thozza commented Aug 5, 2022 •
edited
Loading

Yes, this would be awesome for all of the Image Builder components 😍
We are releasing them in upstream by scheduled automation and rely on Packit to handle most of the Fedora workflow.

@lachmanfrantisek lachmanfrantisek moved this to 📋 Backlog 🔟 in Packit Kanban Board Sep 6, 2022
@lachmanfrantisek lachmanfrantisek moved this from backlog to in-progress in Packit Kanban Board Sep 6, 2022
@nikromen nikromen mentioned this issue Sep 14, 2022
Closed
@majamassarini majamassarini mentioned this issue Oct 28, 2022
Merged
1 task
@stale
Copy link

stale bot commented Nov 2, 2022

This issue has been marked as stale because it hasn't seen any
activity for the last 60 days.

Stale issues are closed after 14 days, unless the label is removed
by a maintainer or someone comments on it.

This is done in order to ensure that open issues are still relevant.

Thank you for your contribution! 🦄 🚀 🤖

(Note: issues labeled with pinned or EPIC are
never marked as stale.)

@stale stale bot added the stale Is the issue still valid? label Nov 2, 2022
@lbarcziova
Copy link
Member Author

This is in progress.

@lbarcziova lbarcziova removed the stale Is the issue still valid? label Nov 2, 2022
@majamassarini majamassarini self-assigned this Nov 3, 2022
@majamassarini majamassarini moved this from in-progress [10] to in-review in Packit Kanban Board Nov 4, 2022
softwarefactory-project-zuul bot added a commit that referenced this issue Nov 10, 2022
@softwarefactory-project-zuul
Merge pull request #1729 from majamassarini/retrigger-bodhi
a53ff50 
Feat(bodhi): Retrigger bodhi update via dist-git PR command

I have rebased the Jirka K. work and I have used Mixins to manage Koji data needed to create a new bodhi update.
This comes from #1654
Fixes #1541
Merge after packit/packit#1764

 call is_packager as aked by Laura: #1654 (comment)

Note: since I am using this method if for any reason the actor is None then the body update will succeed, I don't know if this could be a problem.

RELEASE NOTES BEGIN
Packit now allows to re-trigger Bodhi update via dist-git PR comment /packit create-update.
RELEASE NOTES END

Reviewed-by: Tomas Tomecek <[email protected]>
Reviewed-by: Laura Barcziová <None>
Reviewed-by: Maja Massarini <None>
Repository owner moved this from in-review to done in Packit Kanban Board Nov 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@majamassarini majamassarini

@nikromen nikromen

Labels
None yet
Projects
Packit Kanban Board
Archived in project
Milestone
No milestone
5 participants
@TomasTomecek @majamassarini @thozza @lbarcziova @nikromen