From 8c0f47103d8cca3cc88a402da1174daf6ab66d97 Mon Sep 17 00:00:00 2001 From: ethan Date: Tue, 18 Nov 2025 13:16:53 +1100 Subject: [PATCH] fix(provisioner): preserve actual error when getting provisioner key details Previously, when GetProvisionerKey() failed, the actual error was swallowed and replaced with a generic 'unable to get provisioner key details' message. This made it very difficult to diagnose issues such as missing URLs, authentication failures, or network problems. Now the actual error is preserved using error wrapping, so users can see the real cause of the failure (e.g., 404 Not Found, connection refused, invalid key, etc.). Fixes #20781 --- enterprise/cli/provisionerdaemonstart.go | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/enterprise/cli/provisionerdaemonstart.go b/enterprise/cli/provisionerdaemonstart.go index 4c0ed003a18df..b15e56d8ab385 100644 --- a/enterprise/cli/provisionerdaemonstart.go +++ b/enterprise/cli/provisionerdaemonstart.go @@ -107,7 +107,7 @@ func (r *RootCmd) provisionerDaemonStart() *serpent.Command { if provisionerKey != "" { pkDetails, err := client.GetProvisionerKey(ctx, provisionerKey) if err != nil { - return xerrors.New("unable to get provisioner key details") + return xerrors.Errorf("unable to get provisioner key details: %w", err) } for k, v := range pkDetails.Tags {