Skip to content

Commit a37e414

Browse files
committed
Remove unnecessary annotations check
Signed-off-by: robert-cronin <[email protected]>
1 parent 2c651e1 commit a37e414

File tree

1 file changed

+0
-22
lines changed

1 file changed

+0
-22
lines changed

integration/multiarch/patch_test.go

Lines changed: 0 additions & 22 deletions
Original file line numberDiff line numberDiff line change
@@ -404,28 +404,6 @@ func verifyAnnotations(t *testing.T, patchedRef string, platforms []string, repo
404404
if isPatchablePlatform(platformStr, platforms, reportDir) {
405405
t.Logf("checking manifest annotations for patched platform %s", platformStr)
406406

407-
// Verify that if original nginx annotations exist, they are preserved
408-
commonAnnotations := []string{
409-
"org.opencontainers.image.source",
410-
"org.opencontainers.image.url",
411-
"org.opencontainers.image.version",
412-
"org.opencontainers.image.revision",
413-
"org.opencontainers.image.base.name",
414-
"org.opencontainers.image.base.digest",
415-
}
416-
417-
foundAnnotations := 0
418-
for _, expectedKey := range commonAnnotations {
419-
if value, exists := manifestEntry.Annotations[expectedKey]; exists {
420-
assert.NotEmpty(t, value, "annotation %s should not be empty for platform %s", expectedKey, platformStr)
421-
t.Logf("platform %s has annotation %s=%s", platformStr, expectedKey, value)
422-
foundAnnotations++
423-
}
424-
}
425-
426-
// We expect at least some annotations to be preserved for nginx images
427-
assert.Greater(t, foundAnnotations, 0, "platform %s should have at least some preserved annotations", platformStr)
428-
429407
// The created timestamp should be updated for patched platforms
430408
if createdTime, exists := manifestEntry.Annotations["org.opencontainers.image.created"]; exists {
431409
assert.NotEmpty(t, createdTime, "created timestamp should not be empty for patched platform %s", platformStr)

0 commit comments

Comments
 (0)