extracted parts of pkg/gameservers that deal with SDK server into pkg…

…/sdkserver
googleforgames · Dec 21, 2018 · d43f4a7 · d43f4a7
1 parent d4b1d10
commit d43f4a7
Show file tree

Hide file tree

Showing 9 changed files with 22 additions and 11 deletions.
diff --git a/cmd/sdk-server/main.go b/cmd/sdk-server/main.go
@@ -25,8 +25,8 @@ import (
 
 	"agones.dev/agones/pkg"
 	"agones.dev/agones/pkg/client/clientset/versioned"
-	"agones.dev/agones/pkg/gameservers"
 	"agones.dev/agones/pkg/sdk"
+	"agones.dev/agones/pkg/sdkserver"
 	"agones.dev/agones/pkg/util/runtime"
 	"agones.dev/agones/pkg/util/signals"
 	gwruntime "github.com/grpc-ecosystem/grpc-gateway/runtime"
@@ -108,8 +108,8 @@ func main() {
 			logger.WithError(err).Fatalf("Could not create the agones api clientset")
 		}
 
-		var s *gameservers.SDKServer
-		s, err = gameservers.NewSDKServer(viper.GetString(gameServerNameEnv),
+		var s *sdkserver.SDKServer
+		s, err = sdkserver.NewSDKServer(viper.GetString(gameServerNameEnv),
 			viper.GetString(podNamespaceEnv), kubeClient, agonesClient)
 		if err != nil {
 			logger.WithError(err).Fatalf("Could not start sidecar")
@@ -146,7 +146,7 @@ func registerLocal(grpcServer *grpc.Server, ctlConf config) error {
 		}
 	}
 
-	local, err := gameservers.NewLocalSDKServer(filePath)
+	local, err := sdkserver.NewLocalSDKServer(filePath)
 	if err != nil {
 		return err
 	}

diff --git a/pkg/gameservers/controller_test.go b/pkg/gameservers/controller_test.go
@@ -1106,3 +1106,14 @@ func newFakeController() (*Controller, agtesting.Mocks) {
 	c.recorder = m.FakeRecorder
 	return c, m
 }
+
+func newSingleContainerSpec() v1alpha1.GameServerSpec {
+	return v1alpha1.GameServerSpec{
+		Ports: []v1alpha1.GameServerPort{{ContainerPort: 7777, HostPort: 9999, PortPolicy: v1alpha1.Static}},
+		Template: corev1.PodTemplateSpec{
+			Spec: corev1.PodSpec{
+				Containers: []corev1.Container{{Name: "container", Image: "container/image"}},
+			},
+		},
+	}
+}
diff --git a/pkg/gameservers/helper_test.go → pkg/sdkserver/helper_test.go b/pkg/gameservers/helper_test.go → pkg/sdkserver/helper_test.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package gameservers
+package sdkserver
 
 import (
 	"context"

diff --git a/pkg/gameservers/localsdk.go → pkg/sdkserver/localsdk.go b/pkg/gameservers/localsdk.go → pkg/sdkserver/localsdk.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package gameservers
+package sdkserver
 
 import (
 	"io"

diff --git a/pkg/gameservers/localsdk_test.go → pkg/sdkserver/localsdk_test.go b/pkg/gameservers/localsdk_test.go → pkg/sdkserver/localsdk_test.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package gameservers
+package sdkserver
 
 import (
 	"encoding/json"

diff --git a/pkg/gameservers/sdk.go → pkg/sdkserver/sdk.go b/pkg/gameservers/sdk.go → pkg/sdkserver/sdk.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package gameservers
+package sdkserver
 
 import (
 	"agones.dev/agones/pkg/apis/stable/v1alpha1"

diff --git a/pkg/gameservers/sdk_test.go → pkg/sdkserver/sdk_test.go b/pkg/gameservers/sdk_test.go → pkg/sdkserver/sdk_test.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package gameservers
+package sdkserver
 
 import (
 	"testing"

diff --git a/pkg/gameservers/sdkserver.go → pkg/sdkserver/sdkserver.go b/pkg/gameservers/sdkserver.go → pkg/sdkserver/sdkserver.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package gameservers
+package sdkserver
 
 import (
 	"io"

diff --git a/pkg/gameservers/sdkserver_test.go → pkg/sdkserver/sdkserver_test.go b/pkg/gameservers/sdkserver_test.go → pkg/sdkserver/sdkserver_test.go
@@ -12,7 +12,7 @@
 // See the License for the specific language governing permissions and
 // limitations under the License.
 
-package gameservers
+package sdkserver
 
 import (
 	"net/http"