#236 Pull Request CI: Put the Fedora version/release information to the job name
Opened 3 years ago by churchyard. Modified 3 years ago

Please, could you put the Fedora version/release in the job name?

Pagure now lists all the CI results for a commit and if I open 4 pull requests from a single branch, all I get is this and I need to read the logs to figure about what Fedora version failed.

png

Thanks


If I understood well, pagure does not care of the PR context to display CI results ? CI status should not be mixed between PRs.

Zuul set the PR flag on the related PR: https://opendev.org/zuul/zuul/src/branch/master/zuul/driver/pagure/pagureconnection.py#L423
Not sure I can easily change this behavior tbh.

The Zuul comment with CI result detail should be accurate.

Is this considered as an issue in Pagure ?

CI status should not be mixed between PRs.

I agree.

Not sure I can easily change this behavior tbh.

The behavior is OK-ish, but please include the Fedora version and/or PR number in the flag username and/or comment.

The Zuul comment with CI result detail should be accurate.

The Zuul comment makes this easier, yes.
The "Fedora CI" jobs don't post comments.

Is this considered as an issue in Pagure ?

I consider it as such, https://pagure.io/pagure/issue/5104
In the meantime, I've opened this.

Fedora CI flags are added by toddlers: https://pagure.io/fedora-infra/toddlers/blob/main/f/toddlers/plugins/flag_ci_pr.py#_87

I don't see Zuul there, so Zuul is probably talking to the Pagure API directly (?)

I don't see Zuul there, so Zuul is probably talking to the Pagure API directly (?)

Yes, see:

Zuul set the PR flag on the related PR: https://opendev.org/zuul/zuul/src/branch/master/zuul/driver/pagure/pagureconnection.py#L423

Log in to comment on this ticket.

Metadata
Attachments 1
png
Attached 3 years ago View Comment