tide status does not inform about pending jobs

[0xmichalis]

When a PR has all the labels set but waits on one or more pending jobs, tide just reports that the PR is in the merge pool. It should pick a pending job and state that it's waiting for it to finish.

/area prow
/kind bug

@cjwagner @stevekuznetsov

[stevekuznetsov]

/assign

[0xmichalis]

Still an issue.

[stevekuznetsov]

Link? Where? How?

[0xmichalis]

I haven't seen it work anywhere.

Example: openshift/origin#19288

[0xmichalis]

Hrm, the jobs are failed and not pending. Maybe a separate issue?

[stevekuznetsov]

Hmm, @cjwagner in that linked case, we were on old tide and upgraded to new tide. We should have updated the statuses since the wanted state (pending) doesn't match the actual state (successful). So the bug is in update, not this bug specifically.

[stevekuznetsov]

@Kargakis see

{"client":"github","component":"tide","level":"info","msg":"CreateStatus(openshift, openshift-ansible, 1d6173922d5e68b1fec7734d24ecd24b4eed7830, {pending https://deck-ci.svc.ci.openshift.org/tide.html Not mergeable. Job ci/openshift-jenkins/gcp has not succeeded. tide})","time":"2018-04-10T17:39:56Z"}

[stevekuznetsov]

The issue is that we would only tell you about jobs if you are LGTM+Approved and in that case you're in the pool. Issue is #7279

[stevekuznetsov]

/close

[0xmichalis]

Btw, do we know what's the size limit for the status description?

…

On Tue, Apr 10, 2018 at 7:43 PM, Steve Kuznetsov ***@***.***> wrote: @Kargakis <https://github.com/kargakis> see {"client":"github","component":"tide","level":"info","msg":"CreateStatus(openshift, openshift-ansible, 1d6173922d5e68b1fec7734d24ecd24b4eed7830, {pending https://deck-ci.svc.ci.openshift.org/tide.html Not mergeable. Job ci/openshift-jenkins/gcp has not succeeded. tide})","time":"2018-04-10T17:39:56Z"} — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#7449 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ADuFfzRpR7L2hzjrrHAxcZ3iXU51AFoxks5tnO8wgaJpZM4S-cA7> .

[cjwagner]

Btw, do we know what's the size limit for the status description?

We're just estimating the size limit. It seems to be somewhere around 80 characters, but it could be display dependent.
The spaced used for the description text is shared with the context name and state so less space is available if the status is pending. Hovering with the cursor shows the full description text even if the description is truncated.