Skip to content

Commit d4637ce

Browse files
vyasgunpraveenkumar
authored andcommitted
Return ErrNotImplemented from unixgramListener on windows and linux
1 parent a95cba9 commit d4637ce

File tree

4 files changed

+16
-10
lines changed

4 files changed

+16
-10
lines changed

cmd/crc/cmd/daemon.go

Lines changed: 9 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -14,17 +14,17 @@ import (
1414
"syscall"
1515
"time"
1616

17-
"github.com/crc-org/crc/v2/pkg/crc/api/client"
18-
"github.com/crc-org/crc/v2/pkg/crc/daemonclient"
19-
2017
"github.com/containers/gvisor-tap-vsock/pkg/types"
2118
"github.com/containers/gvisor-tap-vsock/pkg/virtualnetwork"
2219
"github.com/crc-org/crc/v2/pkg/crc/adminhelper"
2320
"github.com/crc-org/crc/v2/pkg/crc/api"
21+
"github.com/crc-org/crc/v2/pkg/crc/api/client"
2422
"github.com/crc-org/crc/v2/pkg/crc/api/events"
2523
crcConfig "github.com/crc-org/crc/v2/pkg/crc/config"
2624
"github.com/crc-org/crc/v2/pkg/crc/constants"
25+
"github.com/crc-org/crc/v2/pkg/crc/daemonclient"
2726
"github.com/crc-org/crc/v2/pkg/crc/logging"
27+
"github.com/crc-org/machine/libmachine/drivers"
2828
"github.com/docker/go-units"
2929
"github.com/gorilla/handlers"
3030
"github.com/pkg/errors"
@@ -214,11 +214,16 @@ func run(configuration *types.Configuration) error {
214214
for {
215215
ctx, cancel := context.WithCancel(context.Background())
216216
conn, err := unixgramListener(ctx, vn)
217+
if err != nil && errors.Is(err, drivers.ErrNotImplemented) {
218+
cancel()
219+
break
220+
}
217221
if err != nil && !errors.Is(err, net.ErrClosed) {
218222
logging.Errorf("unixgramListener error: %v", err)
219223
}
224+
220225
if oldCancel != nil {
221-
logging.Warnf("New connection from %s. Closing old connection", conn.LocalAddr().String())
226+
logging.Warnf("New connection to %s. Closing old connection", conn.LocalAddr().String())
222227
oldCancel()
223228
}
224229
oldCancel = cancel

cmd/crc/cmd/daemon_darwin.go

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -39,7 +39,7 @@ func unixgramListener(ctx context.Context, vn *virtualnetwork.VirtualNetwork) (*
3939
if err != nil {
4040
return conn, errors.Wrap(err, "failed to listen unixgram")
4141
}
42-
logging.Infof("listening on %s:", constants.UnixgramSocketPath)
42+
logging.Infof("listening on %s", constants.UnixgramSocketPath)
4343
vfkitConn, err := transport.AcceptVfkit(conn)
4444
if err != nil {
4545
return conn, errors.Wrap(err, "failed to accept vfkit connection")

cmd/crc/cmd/daemon_linux.go

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -8,11 +8,11 @@ import (
88

99
"github.com/containers/gvisor-tap-vsock/pkg/transport"
1010
"github.com/containers/gvisor-tap-vsock/pkg/virtualnetwork"
11-
"github.com/crc-org/crc/v2/pkg/crc/constants"
12-
"github.com/crc-org/crc/v2/pkg/crc/logging"
13-
1411
"github.com/coreos/go-systemd/v22/activation"
1512
"github.com/coreos/go-systemd/v22/daemon"
13+
"github.com/crc-org/crc/v2/pkg/crc/constants"
14+
"github.com/crc-org/crc/v2/pkg/crc/logging"
15+
"github.com/crc-org/machine/libmachine/drivers"
1616
"github.com/mdlayher/vsock"
1717
)
1818

@@ -127,7 +127,7 @@ func httpListener() (net.Listener, error) {
127127
}
128128

129129
func unixgramListener(_ context.Context, _ *virtualnetwork.VirtualNetwork) (*net.UnixConn, error) {
130-
return nil, nil
130+
return nil, drivers.ErrNotImplemented
131131
}
132132

133133
func startupDone() {

cmd/crc/cmd/daemon_windows.go

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -9,6 +9,7 @@ import (
99
"github.com/containers/gvisor-tap-vsock/pkg/virtualnetwork"
1010
"github.com/crc-org/crc/v2/pkg/crc/constants"
1111
"github.com/crc-org/crc/v2/pkg/crc/logging"
12+
"github.com/crc-org/machine/libmachine/drivers"
1213
)
1314

1415
func vsockListener() (net.Listener, error) {
@@ -38,7 +39,7 @@ func checkIfDaemonIsRunning() (bool, error) {
3839
}
3940

4041
func unixgramListener(_ context.Context, _ *virtualnetwork.VirtualNetwork) (*net.UnixConn, error) {
41-
return nil, nil
42+
return nil, drivers.ErrNotImplemented
4243
}
4344

4445
func startupDone() {

0 commit comments

Comments
 (0)