Revert "Also resolve the source code commit for IDEs in ide-service"

This reverts commit 06aee00.

Author: filiptronicek

components/gitpod-protocol/src/ide-protocol.ts

@@ -149,14 +149,4 @@ export interface IDEOption {
      * LatestImageVersion the semantic version of the latest IDE image.
      */
     latestImageVersion?: string;
-
-    /**
-     * ImageCommit the source code commit SHA of the IDE image.
-     */
-    imageCommit?: string;
-
-    /**
-     * LatestImageCommit the source code commit SHA of the latest IDE image.
-     */
-    latestImageCommit?: string;
 }

components/ide-service-api/go/config/ideconfig.go

@@ -63,10 +63,6 @@ type IDEOption struct {
 	ImageVersion string `json:"imageVersion,omitempty"`
 	// LatestImageVersion the semantic version of the latest IDE image.
 	LatestImageVersion string `json:"latestImageVersion,omitempty"`
-	// ImageCommit the source code commit SHA of the IDE image.
-	ImageCommit string `json:"imageCommit,omitempty"`
-	// LatestImageCommit the source code commit SHA of the latest IDE image.
-	LatestImageCommit string `json:"latestImageCommit,omitempty"`
 	// ImageLayers for additional ide layers and dependencies
 	ImageLayers []string `json:"imageLayers,omitempty"`
 	// LatestImageLayers for latest additional ide layers and dependencies

components/ide-service/pkg/ocitool/resolve.go

@@ -71,44 +71,40 @@ func interactiveFetchManifestOrIndex(ctx context.Context, res remotes.Resolver,
 	return "", nil, nil
 }
 
-func ResolveIDEVersion(ctx context.Context, ref string) (ManifestLabels, error) {
+func ResolveIDEVersion(ctx context.Context, ref string) (string, error) {
 	newCtx, cancel := context.WithTimeout(ctx, time.Second*30)
 	defer cancel()
 	res := docker.NewResolver(docker.ResolverOptions{})
 
 	name, mf, err := interactiveFetchManifestOrIndex(newCtx, res, ref)
 	if err != nil {
-		return ManifestLabels{}, err
+		return "", err
 	}
 
 	fetcher, err := res.Fetcher(ctx, name)
 	if err != nil {
-		return ManifestLabels{}, err
+		return "", err
 	}
 
 	cfgin, err := fetcher.Fetch(ctx, mf.Config)
 	if err != nil {
-		return ManifestLabels{}, err
+		return "", err
 	}
 	defer cfgin.Close()
 
 	var tmp ManifestJSON
 
 	err = json.NewDecoder(cfgin).Decode(&tmp)
 	if err != nil {
-		return ManifestLabels{}, err
+		return "", nil
 	}
-
-	return tmp.Config.Labels, nil
-}
-
-type ManifestLabels struct {
-	Version string  `json:"io.gitpod.ide.version"`
-	Commit  *string `json:"io.gitpod.ide.commit,omitempty"`
+	return tmp.Config.Labels.Version, nil
 }
 
 type ManifestJSON struct {
 	Config struct {
-		Labels ManifestLabels `json:"Labels"`
+		Labels struct {
+			Version string `json:"io.gitpod.ide.version"`
+		} `json:"Labels"`
 	} `json:"config"`
 }

components/ide-service/pkg/ocitool/resolve_test.go

@@ -43,7 +43,7 @@ func TestResolveIDEVersion(t *testing.T) {
 				t.Errorf("ResolveIDEVersion() error = %v, wantErr %v", err, tt.wantErr)
 				return
 			}
-			if got.Version != tt.want {
+			if got != tt.want {
 				t.Errorf("ResolveIDEVersion() = %v, want %v", got, tt.want)
 			}
 		})

components/ide-service/pkg/server/ideconfig.go

@@ -72,13 +72,10 @@ func ParseConfig(ctx context.Context, b []byte) (*config.IDEConfig, error) {
 				option.Image = resolved
 			}
 		}
-		if resolvedLabels, err := oci_tool.ResolveIDEVersion(ctx, option.Image); err != nil {
+		if resolvedVersion, err := oci_tool.ResolveIDEVersion(ctx, option.Image); err != nil {
 			log.WithError(err).Error("ide config: cannot get version from image")
 		} else {
-			option.ImageVersion = resolvedLabels.Version
-			if resolvedLabels.Commit != nil {
-				option.ImageCommit = *resolvedLabels.Commit
-			}
+			option.ImageVersion = resolvedVersion
 		}
 		if option.LatestImage != "" {
 			if resolved, err := oci_tool.Resolve(ctx, option.LatestImage); err != nil {
@@ -87,13 +84,10 @@ func ParseConfig(ctx context.Context, b []byte) (*config.IDEConfig, error) {
 				log.WithField("ide", id).WithField("image", option.LatestImage).WithField("resolved", resolved).Info("ide config: resolved latest image digest")
 				option.LatestImage = resolved
 			}
-			if resolvedLabels, err := oci_tool.ResolveIDEVersion(ctx, option.LatestImage); err != nil {
+			if resolvedVersion, err := oci_tool.ResolveIDEVersion(ctx, option.LatestImage); err != nil {
 				log.WithError(err).Error("ide config: cannot get version from image")
 			} else {
-				option.LatestImageVersion = resolvedLabels.Version
-				if resolvedLabels.Commit != nil {
-					option.LatestImageCommit = *resolvedLabels.Commit
-				}
+				option.LatestImageVersion = resolvedVersion
 			}
 		}
 		cfg.IdeOptions.Options[id] = option