Skip to content
Snippets Groups Projects

Fix wrongly generated `Content-Range` header for `PATCH /api/v4/jobs/:id/trace` request

Merged Tomasz Maczukin requested to merge 3275-fix-content-range-for-trace-patch into master
All threads resolved!
3 files
+ 39
6
Compare changes
  • Side-by-side
  • Inline
Files
3
+ 3
3
@@ -304,15 +304,15 @@ func (n *GitLabClient) UpdateJob(config common.RunnerConfig, jobCredentials *com
result, statusText, _, response := n.doJSON(&config.RunnerCredentials, "PUT", fmt.Sprintf("jobs/%d", jobInfo.ID), http.StatusOK, &request, nil)
n.requestsStatusesMap.Append(config.RunnerCredentials.ShortDescription(), APIEndpointUpdateJob, result)
keepAliveResponse := NewKeepAliveResponse(response)
remoteJobStateResponse := NewRemoteJobStateResponse(response)
log := config.Log().WithFields(logrus.Fields{
"code": result,
"job": jobInfo.ID,
"job-status": keepAliveResponse.RemoteState,
"job-status": remoteJobStateResponse.RemoteState,
})
switch {
case keepAliveResponse.IsAborted():
case remoteJobStateResponse.IsAborted():
log.Warningln("Submitting job to coordinator...", "aborted")
return common.UpdateAbort
case result == http.StatusOK:
Loading