@@ -632,6 +632,7 @@ func (c *StoreReconciler) provision(
632632
633633func (c * StoreReconciler )TrackResourceReplacement (ctx context.Context ,workspaceID ,buildID ,claimantID uuid.UUID ,replacements []* sdkproto.ResourceReplacement ) {
634634// Set authorization context since this may be called in the background (i.e. with a bare context).
635+ // nolint:gocritic // Necessary to query all the required data.
635636ctx = dbauthz .AsSystemRestricted (ctx )
636637// Since this may be called in a fire-and-forget fashion, we need to give up at some point.
637638trackCtx ,trackCancel := context .WithTimeout (ctx ,time .Minute )
@@ -642,6 +643,7 @@ func (c *StoreReconciler) TrackResourceReplacement(ctx context.Context, workspac
642643}
643644}
644645
646+ // nolint:revive // Shut up it's fine.
645647func (c * StoreReconciler )trackResourceReplacement (ctx context.Context ,workspaceID ,buildID ,claimantID uuid.UUID ,replacements []* sdkproto.ResourceReplacement )error {
646648if err := ctx .Err ();err != nil {
647649return err
@@ -683,12 +685,12 @@ func (c *StoreReconciler) trackResourceReplacement(ctx context.Context, workspac
683685// Use the claiming build here (not prebuild) because both should be equivalent, and we might as well spot inconsistencies now.
684686templateVersion ,err := c .store .GetTemplateVersionByID (ctx ,build .TemplateVersionID )
685687if err != nil {
686- return xerrors .Errorf ("fetch template version %q: %w" ,build .TemplateVersionID .String ())
688+ return xerrors .Errorf ("fetch template version %q: %w" ,build .TemplateVersionID .String (), err )
687689}
688690
689691org ,err := c .store .GetOrganizationByID (ctx ,workspace .OrganizationID )
690692if err != nil {
691- return xerrors .Errorf ("fetch org %q: %w" ,workspace .OrganizationID .String ())
693+ return xerrors .Errorf ("fetch org %q: %w" ,workspace .OrganizationID .String (), err )
692694}
693695
694696// Track resource replacement in Prometheus metric.