chore(build): increase resource request and limit to be consistent with master; Fix GitHub Webhook response parsing (#1280)(#1281)

Signed-off-by: Julian Raufelder <[email protected]>
Co-authored-by: Julian Raufelder <[email protected]>

Author: chengfang

config/install.yaml

@@ -891,10 +891,10 @@ spec:
         resources:
           limits:
             cpu: 500m
-            memory: 128Mi
+            memory: 1024Mi
           requests:
-            cpu: 10m
-            memory: 64Mi
+            cpu: 250m
+            memory: 512Mi
         securityContext:
           allowPrivilegeEscalation: false
           capabilities:

config/manager/manager.yaml

@@ -200,10 +200,10 @@ spec:
         resources:
           limits:
             cpu: 500m
-            memory: 128Mi
+            memory: 1024Mi
           requests:
-            cpu: 10m
-            memory: 64Mi
+            cpu: 250m
+            memory: 512Mi
         volumeMounts:
           - mountPath: /app/config
             name: image-updater-conf

pkg/webhook/ghcr.go

@@ -110,7 +110,7 @@ func (g *GHCRWebhook) Parse(r *http.Request) (*argocd.WebhookEvent, error) {
 	}
 
 	// Only process container packages
-	if payload.Package.PackageType != "container" {
+	if strings.ToLower(payload.Package.PackageType) != "container" {
 		return nil, fmt.Errorf("unsupported package type: %s", payload.Package.PackageType)
 	}
 

pkg/webhook/ghcr_test.go

@@ -146,7 +146,7 @@ func TestGHCRWebhook_Parse(t *testing.T) {
 		expectError  bool
 	}{
 		{
-			name: "valid registry package published event",
+			name: "valid registry package published event with lower case package_type",
 			payload: `{
 				"action": "published",
 				"package": {
@@ -169,6 +169,54 @@ func TestGHCRWebhook_Parse(t *testing.T) {
 			expectedTag:  "v1.0.0",
 			expectError:  false,
 		},
+		{
+			name: "another valid registry package published event with upper case package_type",
+			payload: `{
+				"action": "published",
+				"package": {
+					"name": "myapp",
+					"package_type": "CONTAINER",
+					"owner": {
+						"login": "myuser"
+					},
+					"package_version": {
+						"name": "v1.0.0",
+						"container_metadata": {
+							"tag": {
+								"name": "v1.0.0"
+							}
+						}
+					}
+				}
+			}`,
+			expectedRepo: "myuser/myapp",
+			expectedTag:  "v1.0.0",
+			expectError:  false,
+		},
+		{
+			name: "another valid registry package published event with camel case package_type",
+			payload: `{
+				"action": "published",
+				"package": {
+					"name": "myapp",
+					"package_type": "Container",
+					"owner": {
+						"login": "myuser"
+					},
+					"package_version": {
+						"name": "v1.0.0",
+						"container_metadata": {
+							"tag": {
+								"name": "v1.0.0"
+							}
+						}
+					}
+				}
+			}`,
+			expectedRepo: "myuser/myapp",
+			expectedTag:  "v1.0.0",
+			expectError:  false,
+		},
 		{
 			name: "valid registry package published event with latest tag",
 			payload: `{