Skip to content

Commit b140c00

Browse files
committed
xhyve: add .plist exist check and better log message
Signed-off-by: Koichi Shiraishi <zchee.io@gmail.com>
1 parent dc71617 commit b140c00

File tree

2 files changed

+21
-4
lines changed

2 files changed

+21
-4
lines changed

vmnet/vmnet.go

Lines changed: 15 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ package vmnet
55
import (
66
"fmt"
77
"net"
8+
"os"
89
"os/exec"
910
"strings"
1011
)
@@ -15,7 +16,17 @@ const (
1516
NET_MASK_KEY = "Shared_Net_Mask"
1617
)
1718

19+
// isExist returns whether the filename is exists.
20+
func isExist(filename string) bool {
21+
_, err := os.Stat(filename)
22+
return err == nil
23+
}
24+
1825
func GetNetAddr() (net.IP, error) {
26+
if !isExist(CONFIG_PLIST + ".plist") {
27+
return nil, fmt.Errorf("Does not exist %s", CONFIG_PLIST+".plist")
28+
}
29+
1930
out, err := exec.Command("defaults", "read", CONFIG_PLIST, NET_ADDR_KEY).Output()
2031
if err != nil {
2132
return nil, err
@@ -28,6 +39,10 @@ func GetNetAddr() (net.IP, error) {
2839
}
2940

3041
func getNetMask() (net.IPMask, error) {
42+
if !isExist(CONFIG_PLIST + ".plist") {
43+
return nil, fmt.Errorf("Does not exist %s", CONFIG_PLIST+".plist")
44+
}
45+
3146
out, err := exec.Command("defaults", "read", CONFIG_PLIST, NET_MASK_KEY).Output()
3247
if err != nil {
3348
return nil, err

xhyve/xhyve.go

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -376,7 +376,7 @@ func (d *Driver) Create() error {
376376
return err
377377
}
378378

379-
log.Infof("Extracting kernel and initrd from %s...", isoFilename)
379+
log.Infof("Extracting %s and %s from %s...", d.Vmlinuz, d.Initrd, isoFilename)
380380
if err := d.extractKernelImages(); err != nil {
381381
return err
382382
}
@@ -413,7 +413,7 @@ func (d *Driver) Create() error {
413413
log.Infof("Convert UUID to MAC address...")
414414
rawUUID, err := d.getMACAdress()
415415
if err != nil {
416-
return err
416+
return fmt.Errorf("Could not convert the UUID to MAC address: %s", err.Error())
417417
}
418418
d.MacAddr = trimMacAddress(rawUUID)
419419
log.Debugf("Converted MAC address: %s", d.MacAddr)
@@ -606,14 +606,16 @@ func (d *Driver) extractKernelImages() error {
606606
vmlinuz := filepath.Join(volumeRootDir, "boot", d.Vmlinuz)
607607
initrd := filepath.Join(volumeRootDir, "boot", d.Initrd)
608608

609-
log.Debugf("Extracting kernel into %s", d.ResolveStorePath("."))
609+
log.Debugf("Extracting %s into %s", d.Vmlinuz, d.ResolveStorePath(d.Vmlinuz))
610610
if err := mcnutils.CopyFile(vmlinuz, d.ResolveStorePath(d.Vmlinuz)); err != nil {
611611
return err
612612
}
613-
log.Debugf("Extracting initrd into %s", d.ResolveStorePath("."))
613+
614+
log.Debugf("Extracting %s into %s", d.Initrd, d.ResolveStorePath(d.Initrd))
614615
if err := mcnutils.CopyFile(initrd, d.ResolveStorePath(d.Initrd)); err != nil {
615616
return err
616617
}
618+
617619
log.Debugf("Unmounting %s", isoFilename)
618620
if err := hdiutil("detach", volumeRootDir); err != nil {
619621
return err

0 commit comments

Comments
 (0)