test: Add nginx to the image request

osbuild · Dec 12, 2023 · f5968ef · f5968ef
1 parent 7d57892
commit f5968ef
Show file tree

Hide file tree

Showing 5 changed files with 101 additions and 9 deletions.
diff --git a/internal/dnfjson/dnfjson_test.go b/internal/dnfjson/dnfjson_test.go
@@ -86,6 +86,11 @@ func TestMakeDepsolveRequest(t *testing.T) {
 		Name:     "user-repo-2",
 		BaseURLs: []string{"https://example.org/user-repo-2"},
 	}
+	modHotfixRepo := rpmmd.RepoConfig{
+		Name:           "nginx",
+		BaseURLs:       []string{"https://example.org/nginx"},
+		ModuleHotfixes: common.ToPtr(true),
+	}
 	tests := []struct {
 		packageSets []rpmmd.PackageSet
 		args        []transactionArgs
@@ -171,6 +176,44 @@ func TestMakeDepsolveRequest(t *testing.T) {
 				},
 			},
 		},
+		// module hotfixes flag is passed
+		{
+			packageSets: []rpmmd.PackageSet{
+				{
+					Include:      []string{"pkg1"},
+					Exclude:      []string{"pkg2"},
+					Repositories: []rpmmd.RepoConfig{baseOS, appstream, modHotfixRepo},
+				},
+			},
+			args: []transactionArgs{
+				{
+					PackageSpecs: []string{"pkg1"},
+					ExcludeSpecs: []string{"pkg2"},
+					RepoIDs:      []string{baseOS.Hash(), appstream.Hash(), modHotfixRepo.Hash()},
+				},
+			},
+			wantRepos: []repoConfig{
+				{
+					ID:       baseOS.Hash(),
+					Name:     "baseos",
+					BaseURLs: []string{"https://example.org/baseos"},
+					repoHash: "fdc2e5bb6cda8e113308df9396a005b81a55ec00ec29aa0a447952ad4248d803",
+				},
+				{
+					ID:       appstream.Hash(),
+					Name:     "appstream",
+					BaseURLs: []string{"https://example.org/appstream"},
+					repoHash: "71c280f63a779a8bf53961ec2f15d51d052021de024a4e06ae499b8029701808",
+				},
+				{
+					ID:             modHotfixRepo.Hash(),
+					Name:           "nginx",
+					BaseURLs:       []string{"https://example.org/nginx"},
+					ModuleHotfixes: common.ToPtr(true),
+					repoHash:       "6ab05f54094ff2a0ee86facecae3e75e4065a01cc8caffbbbeb7505c6bfac283",
+				},
+			},
+		},
 		// 2 transactions + no package set specific repos
 		{
 			packageSets: []rpmmd.PackageSet{

diff --git a/test/cases/api.sh b/test/cases/api.sh
@@ -438,6 +438,36 @@ EOF
 )
 export FIREWALL_CUSTOMIZATION_BLOCK
 
+if [ "$TEST_MODULE_HOTFIXES" = 1 ]; then
+  if [ "$ARCH" = "x86_64"]; then
+    NGINX_REPO_URL="https://rpmrepo.osbuild.org/v2/mirror/public/el8/el8-x86_64-nginx-20231207"
+  else
+    NGINX_REPO_URL="https://rpmrepo.osbuild.org/v2/mirror/public/el8/el8-aarch64-nginx-20231207"
+  fi
+  EXTRA_PAYLOAD_REPOS_BLOCK=$(cat <<EOF
+,
+      {
+        "baseurl": "$NGINX_REPO_URL",
+        "check_gpg": false,
+        "check_repo_gpg": false,
+        "rhsm": false,
+        "module_hotfixes": true
+      }
+EOF
+)
+
+  EXTRA_PACKAGES_BLOCK=$(cat <<EOF
+,
+      "nginx",
+      "nginx-module-njs"
+EOF
+)
+
+fi
+
+export EXTRA_PAYLOAD_REPOS_BLOCK
+export EXTRA_PACKAGES_BLOCK
+
 # generate a temp key for user tests
 ssh-keygen -t rsa-sha2-512 -f "${WORKDIR}/usertest" -C "usertest" -N ""
 
@@ -582,9 +612,16 @@ function waitForImgState() {
 
 #
 # Make sure that requesting a non existing paquet results in failure
+# or if TEST_MODULE_HOTFIXES flag is passed, we verify, that without that flag the resolve fails
 #
+
+if [ "$TEST_MODULE_HOTFIXES" != 1 ]; then
+  jq_edit = '.customizations.packages = [ "jesuisunpaquetquinexistepas" ]'
+else
+  jq_edit = 'del(.customizations.payload_repositories[] | select(.baseurl == "https://rpmrepo.osbuild.org/v2/mirror/public/el8/el8-$ARCH-nginx-20231207") | .module_hotfixes)'
+fi
 REQUEST_FILE2="${WORKDIR}/request2.json"
-jq '.customizations.packages = [ "jesuisunpaquetquinexistepas" ]' "$REQUEST_FILE" > "$REQUEST_FILE2"
+jq $jq_edit "$REQUEST_FILE" > "$REQUEST_FILE2"
 
 greenprint  "Sending compose: Fail test"
 sendCompose "$REQUEST_FILE2"

diff --git a/test/cases/api/aws.sh b/test/cases/api/aws.sh
@@ -47,11 +47,11 @@ function createReqFile() {
     "payload_repositories": [
       {
         "baseurl": "$PAYLOAD_REPO_URL"
-      }
+      }${EXTRA_PAYLOAD_REPOS_BLOCK}
     ],
     "packages": [
       "postgresql",
-      "dummy"
+      "dummy"${EXTRA_PACKAGES_BLOCK}
     ],
     "users":[
       {

diff --git a/test/cases/api/common/s3.sh b/test/cases/api/common/s3.sh
@@ -21,11 +21,11 @@ function createReqFileEdge() {
     "payload_repositories": [
       {
         "baseurl": "$PAYLOAD_REPO_URL"
-      }
+      }${EXTRA_PAYLOAD_REPOS_BLOCK}
     ],
     "packages": [
       "postgresql",
-      "dummy"
+      "dummy"${EXTRA_PACKAGES_BLOCK}
     ],
     "users":[
       {
@@ -67,11 +67,11 @@ function createReqFileGuest() {
     "payload_repositories": [
       {
         "baseurl": "$PAYLOAD_REPO_URL"
-      }
+      }${EXTRA_PAYLOAD_REPOS_BLOCK}
     ],
     "packages": [
       "postgresql",
-      "dummy"
+      "dummy"${EXTRA_PACKAGES_BLOCK}
     ],
     "users":[
       {
@@ -108,11 +108,11 @@ function createReqFileVSphere() {
     "payload_repositories": [
       {
         "baseurl": "$PAYLOAD_REPO_URL"
-      }
+      }${EXTRA_PAYLOAD_REPOS_BLOCK}
     ],
     "packages": [
       "postgresql",
-      "dummy"
+      "dummy"${EXTRA_PACKAGES_BLOCK}
     ]${SUBSCRIPTION_BLOCK}${DIR_FILES_CUSTOMIZATION_BLOCK}
 ${TIMEZONE_CUSTOMIZATION_BLOCK}${FIREWALL_CUSTOMIZATION_BLOCK}
   },

diff --git a/tools/test-case-generators/repos.json b/tools/test-case-generators/repos.json
@@ -274,6 +274,12 @@
         "image_type_tags": [
           "azure-eap7-rhui"
         ]
+      },
+      {
+        "name": "nginx",
+        "baseurl": "https://rpmrepo.osbuild.org/v2/mirror/public/el8/el8-x86_64-nginx-20231207",
+        "check_gpg": false,
+        "module_hotfixes": true
       }
     ],
     "aarch64": [
@@ -302,6 +308,12 @@
         "image_type_tags": [
           "azure-rhui"
         ]
+      },
+      {
+        "name": "nginx",
+        "baseurl": "https://rpmrepo.osbuild.org/v2/mirror/public/el8/el8-aarch64-nginx-20231207",
+        "check_gpg": false,
+        "module_hotfixes": true
       }
     ],
     "s390x": [