fix: remove the downloaded packages when install failed (#66)

* fix: remove the downloaded packages when install failed

Signed-off-by: shilinlee <[email protected]>

Author: shilinlee

README.md

@@ -9,8 +9,8 @@
 
 ## Documentation
 
-- [English](https://docs.opengemini.org/guide/reference/gemix/gemix_documentation_guide.html)
-- [简体中文](https://docs.opengemini.org/zh/guide/reference/gemix/gemix_documentation_guide)
+- [English]()
+- [简体中文](https://docs.opengemini.org/zh/guide/reference/gemix_manual.html)
 
 ## Installation
 
@@ -32,6 +32,11 @@ gemix playground
 gemix cluster insall opengemini_demo v1.1.1 ./topology.yaml --skip-create-user -p
 ```
 
+Tips:
+
+> If there are network issues with the automatic download of the installation package, 
+> please place the installation package under `~/.gemix/storage/cluster/packages/`
+
 ## Usage
 
 After installing `gemix`, you can use it to install binaries of openGemini components and create clusters.

pkg/cluster/operation/download.go

@@ -45,7 +45,10 @@ func Download(prefix, component, nodeOS, arch, version string) error {
 
 	if component == spec.ComponentGrafana {
 		// FIXME: download from opengemini.org
-		fileName = fmt.Sprintf("%s-%s.%s-%s.tar.gz", component, ver.GrafanaVersion, nodeOS, arch)
+		if nodeOS == "darwin" {
+			arch = "amd64"
+		}
+		fileName = fmt.Sprintf("%s-enterprise-%s.%s-%s.tar.gz", component, ver.GrafanaVersion, nodeOS, arch)
 		componentUrl = strings.Join([]string{"https://dl.grafana.com/oss/release", fileName}, "/")
 	}
 	dstPath := spec.ProfilePath(spec.OpenGeminiPackageCacheDir, fileName)
@@ -55,9 +58,8 @@ func Download(prefix, component, nodeOS, arch, version string) error {
 
 	if utils2.IsExist(dstPath) {
 		if component == spec.ComponentOpenGemini {
-			if err := repository.VerifyComponent(version, dstPath); err != nil {
-				_ = os.Remove(dstPath) // nolint
-			}
+			// TODO: do not remote package
+			_ = repository.VerifyComponent(version, dstPath) // nolint
 		}
 	}
 

pkg/repository/repo.go

@@ -61,7 +61,8 @@ func VerifyComponent(version, target string) error {
 	checksums := spec.ProfilePath(spec.OpenGeminiPackageCacheDir, CHECKSUMS)
 	if utils.IsNotExist(checksums) {
 		if err := tryToDownloadCheckSumsFile(version); err != nil {
-			return errors.WithStack(err)
+			// FIXME: assume that the user downloaded the component without problems
+			return nil
 		}
 	}