Change a few 'errors.New' to 'fmt.Errorf'

These were not found by a linter but just something I noticed where it
would be better to include a bit more info.

Signed-off-by: Carlo Teubner <[email protected]>

pkg/server/server.go

@@ -520,7 +520,7 @@ func (s *Server) CheckHealth() health.State {
 func (s *Server) tryGetBundle() error {
 	client, err := server_util.NewServerClient(s.config.BindLocalAddress)
 	if err != nil {
-		return errors.New("cannot create registration client")
+		return fmt.Errorf("cannot create registration client: %w", err)
 	}
 	defer client.Release()
 
@@ -531,7 +531,7 @@ func (s *Server) tryGetBundle() error {
 	// As currently coded however, the API isn't served until after
 	// the server CA has been signed by upstream.
 	if _, err := bundleClient.GetBundle(context.Background(), &bundlev1.GetBundleRequest{}); err != nil {
-		return errors.New("unable to fetch bundle")
+		return fmt.Errorf("unable to fetch bundle: %w", err)
 	}
 	return nil
 }

test/integration/setup/node-attestation/client.go

@@ -230,7 +230,7 @@ func doX509popStep(ctx context.Context) error {
 
 	// Ban agent
 	if err := banAgent(ctx, client, svidResp.Id); err != nil {
-		return errors.New("failed to ban agent")
+		return fmt.Errorf("failed to ban agent: %w", err)
 	}
 
 	// Reattest banned agent, it MUST fail