feature: use 'uname -n' instead of 'hostname' to get hostname

fixes: #2194 - The `hostname` command may need to be downloaded on some machines, but the `uname` command is always present. - add comment for GetHostName Signed-off-by: Pengda Yang <daz-3ux@proton.me>
sealerio · Apr 25, 2023 · 05fcaa8 · 05fcaa8
1 parent 035b662
commit 05fcaa8
Show file tree

Hide file tree

Showing 2 changed files with 2 additions and 2 deletions.
diff --git a/pkg/infradriver/infradriver.go b/pkg/infradriver/infradriver.go
@@ -89,7 +89,7 @@ type InfraDriver interface {
 
 	// GetPlatform Get remote platform
 	GetPlatform(host net.IP) (v1.Platform, error)
-
+	// Gethostname Get remote host name
 	GetHostName(host net.IP) (string, error)
 	// Ping Ping remote host
 	Ping(host net.IP) error

diff --git a/pkg/infradriver/ssh_infradriver.go b/pkg/infradriver/ssh_infradriver.go
@@ -306,7 +306,7 @@ func (d *SSHInfraDriver) GetClusterLaunchApps() []string {
 }
 
 func (d *SSHInfraDriver) GetHostName(hostIP net.IP) (string, error) {
-	hostName, err := d.CmdToString(hostIP, nil, "hostname", "")
+	hostName, err := d.CmdToString(hostIP, nil, "uname -n", "")
 	if err != nil {
 		return "", err
 	}