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

Manager: improve logging when workers register

Browse Source 
The info message that a worker registered now also includes its UUID.
Any failure hashing the password will now also log the worker name + UUID.
This commit is contained in:
Sybren A. Stüvel 2023-04-03 18:07:47 +02:00
parent e2559b1181
commit 28cc7b7a3f
1 changed files with 7 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

10
internal/manager/api_impl/workers.go
View File

@ -33,8 +33,12 @@ func (f *Flamenco) RegisterWorker(e echo.Context) error {
} }
// TODO: validate the request, should at least have non-empty name, secret, and platform. // TODO: validate the request, should at least have non-empty name, secret, and platform.
workerUUID := uuid.New()
logger.Info().Str("name", req.Name).Msg("registering new worker") logger = logger.With().
Str("name", req.Name).
Str("uuid", workerUUID).
Logger()
logger.Info().Msg("registering new worker")
hashedPassword, err := passwordHasher.GenerateHashedPassword([]byte(req.Secret)) hashedPassword, err := passwordHasher.GenerateHashedPassword([]byte(req.Secret))
if err != nil { if err != nil {
@ -43,7 +47,7 @@ func (f *Flamenco) RegisterWorker(e echo.Context) error {
} }
dbWorker := persistence.Worker{ dbWorker := persistence.Worker{
UUID: uuid.New(), UUID: workerUUID,
Name: req.Name, Name: req.Name,
Secret: string(hashedPassword), Secret: string(hashedPassword),
Platform: req.Platform, Platform: req.Platform,