Page MenuHomePhabricator

[tbs.cli]: toolforge build-show shows unknown for steps that are actually ok
Closed, ResolvedPublic3 Estimated Story Points

Description

For example:

dcaro@toolsbeta-sgebastion-05:~$ toolforge build-show dcaro-buildpacks-pipelinerun-h2t88
...
            Step: build - unknown({'container': 'step-build', 'imageID': 'docker-pullable://docker-registry.tools.wmflabs.org/toolforge-buster0-builder@sha256:70724160bd6c4ba38e750969d00aa69b4f032c3a5a20e140b25fbed5d054375f', 'name': 'build', 'terminated': {'containerID': 'docker://dd46fc5a11b2d70d0441ee0986dff9e53c9b60842c0e4d9c78cfaff0ded79d7a', 'exitCode': 0, 'finishedAt': '2022-04-04T10:41:10Z', 'reason': 'Completed', 'startedAt': '2022-04-04T10:40:22Z'}})
...

This unkown string is being set in the _get_step_details_lines function.
https://gerrit.wikimedia.org/r/plugins/gitiles/cloud/toolforge/toolforge-cli/+/refs/heads/main/toolforge_cli/cli.py#113

Event Timeline

dcaro updated the task description. (Show Details)
Slst2020 added a project: User-Slst2020.
Slst2020 moved this task from To Do to Doing on the User-Slst2020 board.
Slst2020 triaged this task as Medium priority.Aug 26 2022, 8:33 AM
Slst2020 moved this task from Backlog to To be Discussed on the Toolforge Build Service board.

Is this just a matter of changing the string value from "unknown" to e.g. "completed"?

Not really, it's understanding why it's not going to the 'ok' if code branch, and fixing the conditions if needed.

@dcaro I'm not sure that I'm seeing the same output as you. Below is the result of running toolforge build-show locally on a successful pipeline run.

Screenshot 2022-10-06 at 11.15.57.png (546×964 px, 231 KB)

Editing to add: I can reproduce this on toolsbeta, but not in the local dev environment. When did we last deploy to toolsbeta?

Tested the latest version of the code (toolforge-cli) and it works ok :), so probably we just need to deploy a newer version of it in toolsbeta \o/

show_ok.png (916×1 px, 359 KB)

This has indeed been resolved by deploying the latest version of toolforge-cli