product: Consider using logger and capture stdout/stderr in GetVersion

[radeksimko]

Currently getting product version may fail in some rare cases and while we would report this as an error, we don't explicitly capture the stdout or stderr to make debugging of such problems easier.

hc-install/product/terraform.go

Lines 29 to 49 in 7e86c53

	GetVersion: func(ctx context.Context, path string) (*version.Version, error) {
	cmd := exec.CommandContext(ctx, path, "version")
	out, err := cmd.Output()
	if err != nil {
	return nil, err
	}
	stdout := strings.TrimSpace(string(out))
	submatches := terraformVersionOutputRe.FindStringSubmatch(stdout)
	if len(submatches) != 2 {
	return nil, fmt.Errorf("unexpected number of version matches %d for %s", len(submatches), stdout)
	}
	v, err := version.NewVersion(submatches[1])
	if err != nil {
	return nil, fmt.Errorf("unable to parse version %q: %w", submatches[1], err)
	}
	return v, err
	},

See also hashicorp/terraform-plugin-sdk#823