blender/flamenco
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

Cleanup: manager, make workerDBtoAPI(w) use workerSummary(w)

Browse Source 
This makes the `workerDBtoAPI(w)` and `workerSummary(w)` functions
consistent, and makes the former use the latter.
This commit is contained in:
Sybren A. Stüvel 2022-06-02 12:10:53 +02:00
parent 3c1b0e0539
commit 487a31624f
1 changed files with 6 additions and 16 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
internal/manager/api_impl/worker_mgt.go
View File

@ -51,7 +51,7 @@ func (f *Flamenco) FetchWorker(e echo.Context, workerUUID string) error {
}
logger.Debug().Msg("fetched worker")
apiWorker := workerDBtoAPI(dbWorker)
apiWorker := workerDBtoAPI(*dbWorker)
return e.JSON(http.StatusOK, apiWorker)
}
@ -117,22 +117,12 @@ func workerSummary(w persistence.Worker) api.WorkerSummary {
return summary
}
func workerDBtoAPI(dbWorker *persistence.Worker) api.Worker {
func workerDBtoAPI(w persistence.Worker) api.Worker {
apiWorker := api.Worker{
WorkerSummary: api.WorkerSummary{
Id: dbWorker.UUID,
Nickname: dbWorker.Name,
Status: dbWorker.Status,
Version: dbWorker.Software,
},
IpAddress: dbWorker.Address,
Platform: dbWorker.Platform,
SupportedTaskTypes: dbWorker.TaskTypes(),
}
if dbWorker.StatusRequested != "" {
apiWorker.StatusRequested = &dbWorker.StatusRequested
apiWorker.LazyStatusRequest = &dbWorker.LazyStatusRequest
WorkerSummary: workerSummary(w),
IpAddress: w.Address,
Platform: w.Platform,
SupportedTaskTypes: w.TaskTypes(),
}
return apiWorker