Feature Request: Option to NOT expose build logs about checks-api-plugin

[timja]

It's what is known as a Job behaviour or trait.

It's done at job level configuration (or organisation / multi branch folder level)

[timja]

I thought about this when it was added, makes sense to add configuration hooks for it.

There shouldn't be any sensitive info in the build logs, but it can happen...

[mrginglymus]

Ahh, I think I've grokked your second point. I'd think there's not much value (currently) in allowing users to disable the pipeline summary altogether; at that point you're left with a check that just has a link to the job, at which point you may be better off suppressing the checks for job status altogether and just using the status API (whilst still retaining the ability to use checks api for e.g. junit, warnings-ng etc).

[dduportal]

Thanks a lot for this! I still have a (newbie) question though: how can I use the flag to suppress Log (from #93)?

I'm not sure to understand if it is a property to set on my Jenkins administration settings, on the jobs (GH Org. Folder in my case) or on the Pipeline itself? I'm asking the question because I never, ever did anything to enable or specify the Checks (except switching from a Github Oauth to Github App). It's really really cool to have this enabled by default!

But I'm not sure where to go once the new released plugin will be installed on my instance.

(I volunteer to write something on the plugin, or Jenkins doc to show the case of course, but I need initial pointers).

[jglick]

https://github.com/jenkinsci/github-checks-plugin/blob/65a3ebf0923ebb3993414103d5269c2f536ffa0c/src/main/java/io/jenkins/plugins/checks/github/status/GitHubSCMSourceStatusChecksTrait.java#L121 IIUC.

[timja]

Yes that's right

[dduportal]

It's what is known as a Job behaviour or trait.
It's done at job level configuration (or organisation / multi branch folder level)

Ok, so I'll have to update my job through UI, or my Job-DSL config (with the new version installed of course). Is that correct?

https://github.com/jenkinsci/github-checks-plugin/blob/65a3ebf0923ebb3993414103d5269c2f536ffa0c/src/main/java/io/jenkins/plugins/checks/github/status/GitHubSCMSourceStatusChecksTrait.java#L121 IIUC.

Is this related to my question or to the PR? I'm not sure to understand (more than it's some code from the plugin github-checks-plugin that might be related to the current plugin, but I don't have the skills to links this code to an actionable element in Jenkins to be fair.

[jglick]

Means that in the multibranch config GUI you would select Status Checks Properties to customize, and from job-dsl you would need to find some way to select GitHubSCMSourceStatusChecksTrait. Seems that the github-checks plugin would need something to match #93 though right?

[timja]

Yeah looks like you're right, this was the last property added: jenkinsci/github-checks-plugin@65a3ebf

cc @mrginglymus

[dduportal]

Thanks a lot @jglick ! I did not know about this item. It's clear now. Looks like you're correct and that something will have to be made on github-checks if my understanding is correct. I'll defer to @timja .

[mrginglymus]

I have a half-done change for the other half of this...

[mrginglymus]

jenkinsci/github-checks-plugin#138

[XiongKezhi]

released in https://github.com/jenkinsci/checks-api-plugin/releases/tag/v1.6.0