Add reverse domain name for annotation keys

Signed-off-by: robert-cronin <[email protected]>

Author: robert-cronin

integration/multiarch/patch_test.go

@@ -351,8 +351,8 @@ func verifyAnnotations(t *testing.T, patchedRef string, platforms []string, repo
 
 	// Check index-level annotations (Copa metadata)
 	assert.NotEmpty(t, manifest.Annotations, "index-level annotations should not be empty")
-	assert.Equal(t, "true", manifest.Annotations["copacetic.patched"], "should have Copa patched annotation")
-	assert.NotEmpty(t, manifest.Annotations["copacetic.patched.timestamp"], "should have Copa timestamp annotation")
+	assert.Equal(t, "true", manifest.Annotations["sh.copa.patched"], "should have Copa patched annotation")
+	assert.NotEmpty(t, manifest.Annotations["sh.copa.patched.timestamp"], "should have Copa timestamp annotation")
 	assert.NotEmpty(t, manifest.Annotations["org.opencontainers.image.created"], "should have created annotation")
 
 	t.Logf("found %d index-level annotations", len(manifest.Annotations))

pkg/patch/patch.go

@@ -48,11 +48,12 @@ import (
 )
 
 const (
-	copaProduct     = "copa"
-	defaultRegistry = "docker.io"
-	defaultTag      = "latest"
-	LINUX           = "linux"
-	trueString      = "true"
+	copaProduct             = "copa"
+	defaultRegistry         = "docker.io"
+	defaultTag              = "latest"
+	LINUX                   = "linux"
+	trueString              = "true"
+	copaAnnotationKeyPrefix = "sh.copa"
 )
 
 // for testing.
@@ -133,13 +134,13 @@ func createMultiPlatformManifest(
 			// add a custom annotation to indicate this image was patched by Copa
 			patchedKey := exptypes.AnnotationKey{
 				Type: exptypes.AnnotationIndex,
-				Key:  "copacetic.patched",
+				Key:  copaAnnotationKeyPrefix + ".patched",
 			}
 			annotations[patchedKey] = trueString
 
 			patchedTimestampKey := exptypes.AnnotationKey{
 				Type: exptypes.AnnotationIndex,
-				Key:  "copacetic.patched.timestamp",
+				Key:  copaAnnotationKeyPrefix + ".patched.timestamp",
 			}
 			annotations[patchedTimestampKey] = time.Now().UTC().Format(time.RFC3339)
 
@@ -155,13 +156,13 @@ func createMultiPlatformManifest(
 
 			patchedKey := exptypes.AnnotationKey{
 				Type: exptypes.AnnotationIndex,
-				Key:  "copacetic.patched",
+				Key:  copaAnnotationKeyPrefix + ".patched",
 			}
 			annotations[patchedKey] = trueString
 
 			patchedTimestampKey := exptypes.AnnotationKey{
 				Type: exptypes.AnnotationIndex,
-				Key:  "copacetic.patched.timestamp",
+				Key:  copaAnnotationKeyPrefix + ".patched.timestamp",
 			}
 			annotations[patchedTimestampKey] = time.Now().UTC().Format(time.RFC3339)
 		}