Remove /debug/jobs/list?v=1 endpoint

Overview

When the listen address is configured we expose an endpoint host:post/debug/jobs/list which by default outputs the logs below (v1), but if you pass in the query parameter ?v=2 so like host:post/debug/jobs/list?v=2 it would out more detailed values. The reason we had v=2 is in case someone depended on the data of v1 and we didn't want to cause a breaking change.

v1
id=405521380 url=https://gitlab.com/gitlab-org/gitlab.git state=running stage=build_script executor_stage=docker_run
id=405521386 url=https://gitlab.com/gitlab-org/gitlab.git state=running stage=build_script executor_stage=docker_run
v2
url=https://gitlab.com/gitlab-org/gitlab/-/jobs/405521380 state=running stage=build_script executor_stage=docker_run duration=1h23m5.876837725s
url=https://gitlab.com/gitlab-org/gitlab/-/jobs/405521386 state=running stage=build_script executor_stage=docker_run duration=1h23m5.391078748s

Proposal

Remove the v1 output and make v2 output the default to have less code. The impact of this should be low since not a lot of users depend on it and this endpoint is used for monitoring so if it "breaks" something it will break monitoring not running any job.

Edited Jan 16, 2020 by Steve Xuereb
Assignee Loading
Time tracking Loading