DevFW-CICD/garm
15
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions 2

Error when deleting a pool with runners

Browse source
This commit is contained in:
Gabriel Adrian Samfira 2022-05-10 16:10:02 +00:00
parent 8a7396151f
commit 4e4ab691c8
1 changed files with 12 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

12
runner/pools.go
View file

@ -52,6 +52,18 @@ func (r *Runner) DeletePoolByID(ctx context.Context, poolID string) error {
return runnerErrors.ErrUnauthorized
}
pool, err := r.store.GetPoolByID(ctx, poolID)
if err != nil {
if !errors.Is(err, runnerErrors.ErrNotFound) {
return errors.Wrap(err, "fetching pool")
}
return nil
}
if len(pool.Instances) > 0 {
return runnerErrors.NewBadRequestError("pool has runners")
}
if err := r.store.DeletePoolByID(ctx, poolID); err != nil {
return errors.Wrap(err, "fetching pool")
}