Commit4cbc292

committed

Log lock acquisition time

Setting default claimer to avoid panicsSigned-off-by: Danny Kopping <danny@coder.com>

1 parent9dd9fed commit4cbc292Copy full SHA for 4cbc292

File tree

-1

lines changed

-1

lines changed

Lines changed: 1 addition & 0 deletions

Original file line number	Diff line number	Diff line change
`@@ -566,6 +566,7 @@ func New(options Options) API {`
`566`	`566`	`f:=appearance.NewDefaultFetcher(api.DeploymentValues.DocsURL.String())`
`567`	`567`	`api.AppearanceFetcher.Store(&f)`
`568`	`568`	`api.PortSharer.Store(&portsharing.DefaultPortSharer)`
	`569`	`+api.PrebuildsClaimer.Store(&prebuilds.DefaultClaimer)`
`569`	`570`	`buildInfo:= codersdk.BuildInfoResponse{`
`570`	`571`	`ExternalURL:buildinfo.ExternalURL(),`
`571`	`572`	`Version:buildinfo.Version(),`

Lines changed: 6 additions & 1 deletion

Original file line number	Diff line number	Diff line change
`@@ -111,14 +111,19 @@ func (c Controller) reconcile(ctx context.Context, templateID uuid.UUID) {`
`111`	`111`	`default:`
`112`	`112`	`}`
`113`	`113`
	`114`	`+logger.Debug(ctx,"starting reconciliation")`
	`115`	`+`
`114`	`116`	`// get all templates or specific one requested`
`115`	`117`	`err:=c.store.InTx(func(db database.Store)error {`
	`118`	`+start:=time.Now()`
`116`	`119`	`err:=db.AcquireLock(ctx,database.LockIDReconcileTemplatePrebuilds)`
`117`	`120`	`iferr!=nil {`
`118`		`-logger.Warn(ctx,"failed to acquire top-level prebuilds lock; likely running on another coderd replica",slog.Error(err))`
	`121`	`+logger.Warn(ctx,"failed to acquire top-level prebuildsreconciliationlock; likely running on another coderd replica",slog.Error(err))`
`119`	`122`	`returnnil`
`120`	`123`	`}`
`121`	`124`
	`125`	`+deferlogger.Debug(ctx,"acquired top-level prebuilds reconciliation lock",slog.F("acquire_wait_secs",fmt.Sprintf("%.4f",time.Since(start).Seconds())))`
	`126`	`+`
`122`	`127`	`innerCtx,cancel:=context.WithTimeout(ctx,time.Second*30)`
`123`	`128`	`defercancel()`
`124`	`129`

Comments

(0)