fix golangci-lint

Signed-off-by: Arjun Raja Yogidas <[email protected]>

Author: coderbirju

cmd/nerdctl/image/image_convert_linux_test.go

@@ -102,7 +102,7 @@ func TestImageConvert(t *testing.T) {
 				Command: func(data test.Data, helpers test.Helpers) test.TestableCommand {
 					return helpers.Command("image", "convert", "--soci",
 						"--soci-span-size", "2097152",
-						"--soci-min-layer-size", "20971520",
+						"--soci-min-layer-size", "0",
 						testutil.CommonImage, data.Identifier("converted-image"))
 				},
 				Expected: test.Expects(0, nil, nil),

docs/soci.md

@@ -4,6 +4,22 @@ SOCI Snapshotter is a containerd snapshotter plugin. It enables standard OCI ima
 
 See https://github.com/awslabs/soci-snapshotter to learn further information.
 
+## SOCI Index Manifest Versions
+
+SOCI supports two index manifest versions:
+
+- **v1**: Original format using OCI Referrers API (disabled by default in SOCI v0.10.0+)
+- **v2**: New format that packages SOCI index with the image (default in SOCI v0.10.0+)
+
+To enable v1 indices in SOCI v0.10.0+, add to `/etc/soci-snapshotter-grpc/config.toml`:
+```toml
+[pull_modes]
+  [pull_modes.soci_v1]
+    enable = true
+```
+
+For detailed information about the differences between v1 and v2, see the [SOCI Index Manifest v2 documentation](https://github.com/awslabs/soci-snapshotter/blob/main/docs/soci-index-manifest-v2.md).
+
 ## Prerequisites
 
 - Install containerd remote snapshotter plugin (`soci-snapshotter-grpc`) from https://github.com/awslabs/soci-snapshotter/blob/main/docs/getting-started.md
@@ -46,10 +62,12 @@ nerdctl push --snapshotter=soci --soci-span-size=2097152 --soci-min-layer-size=2
 ```
 --soci-span-size and --soci-min-layer-size are two properties to customize the SOCI index. See [Command Reference](https://github.com/containerd/nerdctl/blob/377b2077bb616194a8ef1e19ccde32aa1ffd6c84/docs/command-reference.md?plain=1#L773) for further details.
 
+> **Note**: With SOCI v0.10.0+, `nerdctl push` creates and pushes v2 indices by default. For v1 indices, enable them in the snapshotter config as described in the section above.
+
 
 ## Enable SOCI for `nerdctl image convert`
 
-| :zap: Requirement | nerdctl >= 2.2.0 |
+| :zap: Requirement | nerdctl >= 2.1.3 |
 | ----------------- | ---------------- |
 
 | :zap: Requirement | soci-snapshotter >= 0.10.0 |
@@ -59,4 +77,6 @@ nerdctl push --snapshotter=soci --soci-span-size=2097152 --soci-min-layer-size=2
 ```console
 nerdctl image convert --soci --soci-span-size=2097152 --soci-min-layer-size=20971520 public.ecr.aws/my-registry/my-repo:latest public.ecr.aws/my-registry/my-repo:soci
 ```
---soci-span-size and --soci-min-layer-size are two properties to customize the SOCI index. See [Command Reference](https://github.com/containerd/nerdctl/blob/377b2077bb616194a8ef1e19ccde32aa1ffd6c84/docs/command-reference.md?plain=1#L773) for further details.
+--soci-span-size and --soci-min-layer-size are two properties to customize the SOCI index. See [Command Reference](https://github.com/containerd/nerdctl/blob/377b2077bb616194a8ef1e19ccde32aa1ffd6c84/docs/command-reference.md?plain=1#L773) for further details.
+
+The `image convert` command with `--soci` flag creates SOCI-enabled images using SOCI Index Manifest v2, which combines the SOCI index and the original image into a single artifact.

go.mod

@@ -136,7 +136,7 @@ require (
 	go.opentelemetry.io/otel/metric v1.35.0 // indirect
 	go.opentelemetry.io/otel/trace v1.35.0 // indirect
 	golang.org/x/exp v0.0.0-20241108190413-2d47ceb2692f // indirect
-	golang.org/x/mod v0.25.0 // indirect
+	golang.org/x/mod v0.25.0
 	google.golang.org/genproto/googleapis/rpc v0.0.0-20250218202821-56aae31c358a // indirect
 	//gomodjail:unconfined
 	google.golang.org/grpc v1.72.2 // indirect

pkg/snapshotterutil/sociutil.go

@@ -26,15 +26,38 @@ import (
 	"strconv"
 	"strings"
 
-	"github.com/Masterminds/semver/v3"
+	"golang.org/x/mod/semver"
+
 	"github.com/containerd/containerd/v2/client"
 	"github.com/containerd/log"
 
 	"github.com/containerd/nerdctl/v2/pkg/api/types"
 )
 
+// setupSociCommand creates and sets up a SOCI command with common configuration
+func setupSociCommand(gOpts types.GlobalCommandOptions) (*exec.Cmd, error) {
+	sociExecutable, err := exec.LookPath("soci")
+	if err != nil {
+		log.L.WithError(err).Error("soci executable not found in path $PATH")
+		log.L.Info("you might consider installing soci from: https://github.com/awslabs/soci-snapshotter/blob/main/docs/install.md")
+		return nil, err
+	}
+
+	sociCmd := exec.Command(sociExecutable)
+	sociCmd.Env = os.Environ()
+
+	// #region for global flags.
+	if gOpts.Address != "" {
+		sociCmd.Args = append(sociCmd.Args, "--address", gOpts.Address)
+	}
+	if gOpts.Namespace != "" {
+		sociCmd.Args = append(sociCmd.Args, "--namespace", gOpts.Namespace)
+	}
+
+	return sociCmd, nil
+}
+
 // CheckSociVersion checks if the SOCI binary version is at least the required version
-// This function can be used by both production code and tests
 func CheckSociVersion(requiredVersion string) error {
 	sociExecutable, err := exec.LookPath("soci")
 	if err != nil {
@@ -58,50 +81,19 @@ func CheckSociVersion(requiredVersion string) error {
 		return fmt.Errorf("failed to parse SOCI version from output: %s", versionStr)
 	}
 
-	// Extract version number
-	installedVersion := matches[1]
+	// Extract version number and add 'v' prefix required by semver package
+	// The regex captures only the numeric part, so we always need to add the 'v'
+	installedVersion := "v" + matches[1]
+	requiredSemver := "v" + requiredVersion
 
-	// Compare versions using semver
-	v1, err := semver.NewVersion(installedVersion)
-	if err != nil {
-		return fmt.Errorf("failed to parse installed version %s: %v", installedVersion, err)
-	}
-
-	v2, err := semver.NewVersion(requiredVersion)
-	if err != nil {
-		return fmt.Errorf("failed to parse minimum required version %s: %v", requiredVersion, err)
-	}
-
-	if v1.LessThan(v2) {
-		return fmt.Errorf("SOCI version %s is lower than the required version %s for the convert operation", installedVersion, requiredVersion)
+	// Use semver package to compare versions
+	if semver.Compare(installedVersion, requiredSemver) < 0 {
+		return fmt.Errorf("SOCI version %s is lower than the required version %s for the convert operation", strings.TrimPrefix(installedVersion, "v"), requiredVersion)
 	}
 
 	return nil
 }
 
-// setupSociCommand creates and sets up a SOCI command with common configuration
-func setupSociCommand(gOpts types.GlobalCommandOptions) (*exec.Cmd, error) {
-	sociExecutable, err := exec.LookPath("soci")
-	if err != nil {
-		log.L.WithError(err).Error("soci executable not found in path $PATH")
-		log.L.Info("you might consider installing soci from: https://github.com/awslabs/soci-snapshotter/blob/main/docs/install.md")
-		return nil, err
-	}
-
-	sociCmd := exec.Command(sociExecutable)
-	sociCmd.Env = os.Environ()
-
-	// #region for global flags.
-	if gOpts.Address != "" {
-		sociCmd.Args = append(sociCmd.Args, "--address", gOpts.Address)
-	}
-	if gOpts.Namespace != "" {
-		sociCmd.Args = append(sociCmd.Args, "--namespace", gOpts.Namespace)
-	}
-
-	return sociCmd, nil
-}
-
 // ConvertSociIndexV2 converts an image to SOCI format and returns the converted image reference with digest
 func ConvertSociIndexV2(ctx context.Context, client *client.Client, srcRef string, destRef string, gOpts types.GlobalCommandOptions, platforms []string, sOpts types.SociOptions) (string, error) {
 	// Check if SOCI version is at least 0.10.0 which is required for the convert operation