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

Status for propose-downstream job #1293

Closed
Tracked by #1082
lachmanfrantisek opened this issue Nov 24, 2021 · 0 comments · Fixed by #1435
Closed
Tracked by #1082

Status for propose-downstream job #1293

lachmanfrantisek opened this issue Nov 24, 2021 · 0 comments · Fixed by #1435
Labels
area/fedora Related to Fedora ecosystem area/user-experience Usability issue kind/feature New feature or a request for enhancement.

Comments

@lachmanfrantisek
Copy link
Member

lachmanfrantisek commented Nov 24, 2021

Use check-runs/commit-status to show users the state of the propose-downstream job in the place it was triggered (release commit or a new commit for the branch push once #1284 is done).

For propose-downstream triggered by an issue comment, create a comment as well as the commit status.

Relates to #1292 and packit/dashboard#155 (link the dashboard view once it's implemented).

@lachmanfrantisek lachmanfrantisek added kind/feature New feature or a request for enhancement. area/user-experience Usability issue area/fedora Related to Fedora ecosystem labels Nov 24, 2021
softwarefactory-project-zuul bot added a commit that referenced this issue Apr 12, 2022
Propose downstream reporting

Fixes #1293
As for the helper classes, I extracted the common logic to the BaseJobHelper and then added propose-downstream specific logic into ProposeDownstreamJobHelper. Since we already have also TestingFarmJobHelper which was placed directly in the worker directory, I created a dedicated helpers directory and moved these + the build dir including build helpers there. It is still not ideal (especially the testing farm inheriting from BaseBuildJobHelper but being placed outside of build dir),  but current class hierarchy is:

BaseJobHelper

BaseBuildJobHelper

CoprBuildJobHelper
KojiBuildJobHelper
TestingFarmJobHelper


ProposeDownstreamHelper



We can still discuss some better structure.

RELEASE NOTES BEGIN
TBD
RELEASE NOTES END

Reviewed-by: Tomas Tomecek <tomas@tomecek.net>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area/fedora Related to Fedora ecosystem area/user-experience Usability issue kind/feature New feature or a request for enhancement.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant