testing do not merge

Signed-off-by: Arjun Raja Yogidas <arjunry@amazon.com>

Author: coderbirju

.github/workflows/build.yml

@@ -28,25 +28,25 @@ jobs:
   setup:
     uses: ./.github/workflows/setup.yml
 
-  test:
-    needs: setup
-    runs-on: ${{ fromJSON(needs.setup.outputs.runner-labels)[matrix.os] }}
-    strategy:
-      matrix:
-        os: ${{ fromJSON(needs.setup.outputs.available-runners) }}
-    timeout-minutes: 15
-    steps:
-      - uses: actions/checkout@v6
-      - uses: actions/setup-go@v6
-        with:
-          go-version: ${{ env.GO_VERSION }}
-      - name: Install zlib static on AL2 ARM instances
-        if: matrix.os == 'al2-arm'
-        run: dnf install zlib-static.aarch64 -y
-      - run: make
-      - run: make test-with-coverage
-      - name: Show test coverage
-        run: make show-test-coverage
+  # test:
+  #   needs: setup
+  #   runs-on: ${{ fromJSON(needs.setup.outputs.runner-labels)[matrix.os] }}
+  #   strategy:
+  #     matrix:
+  #       os: ${{ fromJSON(needs.setup.outputs.available-runners) }}
+  #   timeout-minutes: 15
+  #   steps:
+  #     - uses: actions/checkout@v6
+  #     - uses: actions/setup-go@v6
+  #       with:
+  #         go-version: ${{ env.GO_VERSION }}
+  #     - name: Install zlib static on AL2 ARM instances
+  #       if: matrix.os == 'al2-arm'
+  #       run: dnf install zlib-static.aarch64 -y
+  #     - run: make
+  #     - run: make test-with-coverage
+  #     - name: Show test coverage
+  #       run: make show-test-coverage
 
   integration:
     needs: setup
@@ -72,6 +72,6 @@ jobs:
           if [[ "${{ matrix.os }}" == "ubuntu-x86" ]]; then
             SKIP_SYSTEMD_TESTS=1
           fi
-          SKIP_SYSTEMD_TESTS=$SKIP_SYSTEMD_TESTS make integration-with-coverage
-      - name: Show test coverage
-        run: make show-integration-coverage
+          SKIP_SYSTEMD_TESTS=$SKIP_SYSTEMD_TESTS GO_TEST_FLAGS="-run TestConvert/convert_and_replace" make integration
+      # - name: Show test coverage
+      #   run: make show-integration-coverage

integration/convert_test.go

@@ -200,24 +200,28 @@ func validateConversion(t *testing.T, sh *shell.Shell, originalDigest, converted
 
 		// Get GC labels
 		manifestLabelOutput := sh.O("ctr", "content", "label", manifestDesc.Digest.String())
+		fmt.Printf("running ctr content label on manifest digest -> %v", manifestDesc.Digest.String())
 		manifestLabels := strings.Split(strings.TrimSpace(string(manifestLabelOutput)), ",")
+		fmt.Printf("length of maifestLabels -> %v", len(manifestLabels))
 		// verify GC lables exists
 		if len(manifestLabels) <= 0 {
 			t.Errorf("manifest does not contain labels, got %d labels", len(manifestLabels))
 		}
 		// verify config label exists
-		// verify config label exists
-		hasConfigLabel := false
+		var configInfor ocispec.Descriptor
 		for _, label := range manifestLabels {
 			parts := strings.Split(label, "=")
 			if len(parts) == 2 && parts[0] == "containerd.io/gc.ref.content.config" {
-				hasConfigLabel = true
-				break
+				fmt.Printf("If it goes here it should be failing -> %v", parts[1])
+				configBytes := sh.O("ctr", "content", "label", parts[1])
+				if err := json.Unmarshal(configBytes, &configInfor); err != nil {
+					t.Errorf("failed to decode manifest: %v", err)
+					continue
+				}
+
 			}
 		}
-		if !hasConfigLabel {
-			t.Errorf("manifest does not contain required config label 'containerd.io/gc.ref.content.config'")
-		}
+
 	}
 
 }