kata-agent: Remove GRPCSock unused variable.

We already save the URL used to connect to the agent in the `state.URL` this variable is the used to connect the shim to agnet independently the socket type (VSOCK or serial) Signed-off-by: Jose Carlos Venegas Munoz <[email protected]>
kata-containers · Jul 31, 2018 · f389b94 · f389b94
1 parent 9b28325
commit f389b94
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 5 deletions.
diff --git a/virtcontainers/kata_agent.go b/virtcontainers/kata_agent.go
@@ -59,7 +59,6 @@ var (
 // KataAgentConfig is a structure storing information needed
 // to reach the Kata Containers agent.
 type KataAgentConfig struct {
-	GRPCSocket   string
 	LongLiveConn bool
 }
 

diff --git a/virtcontainers/kata_agent_test.go b/virtcontainers/kata_agent_test.go
@@ -671,13 +671,11 @@ func TestAgentPathAPI(t *testing.T) {
 	assert.Nil(err)
 	assert.Equal(k1, k2)
 
-	c.GRPCSocket = "unixsocket"
 	err = k1.generateVMSocket(id, c)
 	assert.Nil(err)
 	_, ok := k1.vmSocket.(Socket)
 	assert.True(ok)
 
-	c.GRPCSocket = "vsock:100:200"
 	err = k2.generateVMSocket(id, c)
 	assert.Nil(err)
 	_, ok = k2.vmSocket.(kataVSOCK)
@@ -692,7 +690,7 @@ func TestAgentConfigure(t *testing.T) {
 
 	k := &kataAgent{}
 	h := &mockHypervisor{}
-	c := KataAgentConfig{GRPCSocket: "vsock:100:200"}
+	c := KataAgentConfig{}
 	id := "foobar"
 
 	invalidAgent := HyperConfig{}
@@ -702,7 +700,6 @@ func TestAgentConfigure(t *testing.T) {
 	err = k.configure(h, id, dir, true, c)
 	assert.Nil(err)
 
-	c.GRPCSocket = "foobarfoobar"
 	err = k.configure(h, id, dir, true, c)
 	assert.Nil(err)