Merge pull request #1614 from ibuildthecloud/dev-ports

Add dev ports and app locking

Author: ibuildthecloud

Dockerfile

@@ -8,7 +8,7 @@ FROM ghcr.io/acorn-io/images-mirror/rancher/klipper-lb:v0.3.5 AS klipper-lb
 FROM ghcr.io/acorn-io/images-mirror/golang:1.20-alpine AS helper
 WORKDIR /usr/src
 RUN apk -U add curl
-RUN curl -sfL https://github.com/loft-sh/devspace/archive/refs/tags/v5.18.5.tar.gz | tar xzf - --strip-components=1
+RUN curl -sfL https://github.com/loft-sh/devspace/archive/refs/tags/v6.3.2.tar.gz | tar xzf - --strip-components=1
 RUN --mount=type=cache,target=/go/pkg --mount=type=cache,target=/root/.cache/go-build CGO_ENABLED=0 go build -o /usr/local/bin/acorn-helper -ldflags "-s -w" ./helper
 
 FROM ghcr.io/acorn-io/images-mirror/golang:1.20 AS build

go.mod

@@ -81,7 +81,7 @@ func TestDev(t *testing.T) {
 	}
 
 	_, err = appWatcher.ByName(ctx, ns.Name, "test-app", func(app *v1.AppInstance) (bool, error) {
-		return app.Spec.Image != oldImage && app.Spec.Image != "", nil
+		return app.Spec.Image == oldImage && app.Status.DevSession != nil && app.Status.DevSession.SpecOverride.Image != "", nil
 	})
 	if err != nil {
 		t.Fatal(err)

go.sum

@@ -21,6 +21,7 @@ import (
 	"github.com/acorn-io/acorn/pkg/server"
 	"github.com/acorn-io/acorn/pkg/system"
 	"github.com/acorn-io/baaah/pkg/restconfig"
+	minkserver "github.com/acorn-io/mink/pkg/server"
 	"github.com/google/go-containerregistry/pkg/registry"
 	uuid2 "github.com/google/uuid"
 	"github.com/sirupsen/logrus"
@@ -126,13 +127,8 @@ func StartAPI(t *testing.T) *rest.Config {
 		t.Fatal(err)
 	}
 
-	srv := server.New()
-	srv.Options.SecureServing.Listener = l
-	srv.Options.Authentication.TolerateInClusterLookupFailure = true
-	cfg, err := srv.NewConfig("dev")
-	if err != nil {
-		t.Fatal(err)
-	}
+	opts := minkserver.DefaultOpts()
+	opts.SecureServing.Listener = l
 
 	kubeconfig := clientcmdapi.Config{
 		Clusters: map[string]*clientcmdapi.Cluster{
@@ -158,14 +154,20 @@ func StartAPI(t *testing.T) *rest.Config {
 	restConfig = restconfig.SetScheme(restConfig, scheme.Scheme)
 	restConfig.GroupVersion = &apiv1.SchemeGroupVersion
 
-	if err == nil {
-		go func() {
-			cfg.LocalRestConfig = restConfig
-			err := srv.Run(context.Background(), cfg)
-			t.Log("failed to start api", err)
-		}()
+	s, err := server.New(server.Config{
+		LocalRestConfig:    restConfig,
+		DefaultOpts:        opts,
+		IgnoreStartFailure: true,
+	})
+	if err != nil {
+		t.Fatal(err)
 	}
 
+	go func() {
+		err := s.Run(context.Background())
+		t.Log("failed to start api", err)
+	}()
+
 	restClient, err := rest.RESTClientFor(restConfig)
 	if err != nil {
 		t.Fatal(err)

integration/dev/dev_test.go

@@ -10,8 +10,8 @@ jobs: {
 			{
 			    "type": "basic",
 			    "data": {
-			        "key": "$(echo -n value | base64)",
-			        "pass": "$(echo -n $PASS | base64)"
+			        "key": "value",
+			        "pass": "$PASS"
 			    }
 			}
 			EOF

integration/helper/controller.go

@@ -65,6 +65,8 @@ func AddToSchemeWithGV(scheme *runtime.Scheme, schemeGroupVersion schema.GroupVe
 		&ImageAllowRuleList{},
 		&Event{},
 		&EventList{},
+		&DevSession{},
+		&DevSessionList{},
 	)
 
 	// Add common types

integration/secrets/testdata/generated-json/Acornfile

@@ -26,6 +26,8 @@ func (in *App) GetRegion() string {
 	return in.Status.Defaults.Region
 }
 
+type Acornfile v1.AppSpec
+
 // +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object
 
 type AppList struct {
@@ -571,3 +573,15 @@ type EventList struct {
 	metav1.ListMeta `json:"metadata,omitempty"`
 	Items           []Event `json:"items"`
 }
+
+// +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object
+
+type DevSession v1.DevSessionInstance
+
+// +k8s:deepcopy-gen:interfaces=k8s.io/apimachinery/pkg/runtime.Object
+
+type DevSessionList struct {
+	metav1.TypeMeta `json:",inline"`
+	metav1.ListMeta `json:"metadata,omitempty"`
+	Items           []DevSession `json:"items"`
+}

pkg/apis/api.acorn.io/v1/scheme.go

@@ -89,7 +89,6 @@ type AppInstanceSpec struct {
 	Annotations         []ScopedLabel    `json:"annotations,omitempty"`
 	Image               string           `json:"image,omitempty"`
 	Stop                *bool            `json:"stop,omitempty"`
-	DevMode             *bool            `json:"devMode,omitempty"`
 	Profiles            []string         `json:"profiles,omitempty"`
 	Volumes             []VolumeBinding  `json:"volumes,omitempty"`
 	Secrets             []SecretBinding  `json:"secrets,omitempty"`
@@ -119,12 +118,8 @@ func (in *AppInstanceSpec) GetNotifyUpgrade() bool {
 	return in.NotifyUpgrade != nil && *in.NotifyUpgrade
 }
 
-func (in *AppInstanceSpec) GetDevMode() bool {
-	return in.DevMode != nil && *in.DevMode
-}
-
-func (in *AppInstanceSpec) GetProfiles() []string {
-	if in.GetDevMode() {
+func (in *AppInstanceSpec) GetProfiles(devMode bool) []string {
+	if devMode {
 		found := false
 		for _, profile := range in.Profiles {
 			if profile == "dev?" {
@@ -190,7 +185,12 @@ type AppColumns struct {
 	Created   string `json:"created,omitempty" column:"name=Created,jsonpath=.metadata.creationTimestamp"`
 }
 
+func (a AppInstanceStatus) GetDevMode() bool {
+	return a.DevSession != nil
+}
+
 type AppInstanceStatus struct {
+	DevSession             *DevSessionInstanceSpec    `json:"devSession,omitempty"`
 	ObservedGeneration     int64                      `json:"observedGeneration,omitempty"`
 	ObservedImageDigest    string                     `json:"observedImageDigest,omitempty"`
 	Columns                AppColumns                 `json:"columns,omitempty"`