(PA-6677) Restore support for AL2 x86_64 in manifests

Commit 5c3ffe8 fixed AL2 aarch64, but broke AL2
x86_64.

This commit restores the previous behavior of installing AL2 packages on AL2
aarch64.

It preserves the existing behavior of installing EL6/7 packages on AL1/2 x86_64,
respectively.

Author: joshcooper

manifests/osfamily/redhat.pp

@@ -19,8 +19,11 @@
       }
       'Amazon': {
         $major_version = $facts['os']['release']['major']
-        $amz_el_version = "${major_version}" ? {
-          /^(2017|2018)$/  => '6',
+        $arch = $facts['os']['architecture']
+        $amz_el_version = "${major_version}-${arch}" ? {
+          '2-aarch64'      => '2',
+          '2-x86_64'       => '7',
+          /^(2017|2018)-/  => '6',
           default          => $major_version,
         }
 
@@ -36,10 +39,16 @@
     # lint:endignore
     if ($puppet_agent::is_pe and (!$puppet_agent::use_alternate_sources)) {
       $pe_server_version = pe_build_version()
-      # Treat Amazon Linux just like Enterprise Linux
-      $pe_repo_dir = ($facts['os']['name'] == 'Amazon') ? {
-        true    => "el-${amz_el_version}-${facts['os']['architecture']}",
-        default => $facts['platform_tag'],
+      # Install amazon packages on AL2 (only aarch64) and 2003 and up (all arch)
+      if $facts['os']['name'] == 'Amazon' {
+        # lint:ignore:only_variable_string
+        $pe_repo_dir = "${amz_el_version}" ? {
+          /^(6|7)$/ => "el-${amz_el_version}-${facts['os']['architecture']}",
+          default   => $facts['platform_tag'],
+        }
+        # lint:endignore
+      } else {
+        $pe_repo_dir = $facts['platform_tag']
       }
       if $puppet_agent::source {
         $source = "${puppet_agent::source}/packages/${pe_server_version}/${pe_repo_dir}"

spec/classes/puppet_agent_osfamily_redhat_spec.rb

@@ -22,19 +22,21 @@
   end
 
   [
-    ['Rocky', 'el/8', '8'],
-    ['AlmaLinux', 'el/8', '8'],
-    ['AlmaLinux', 'el/9', '9'],
-    ['Fedora', 'fedora/f36', '36'],
-    ['CentOS', 'el/7', '7'],
-    ['Amazon', 'el/6', '2017'],
-    ['Amazon', 'el/6', '2018'],
-    ['Amazon', 'amazon/2', '2'],
-    ['Amazon', 'amazon/2023', '2023'],
-  ].each do |os, urlbit, osmajor|
+    ['Rocky', 'el/8', '8', 'x86_64'],
+    ['AlmaLinux', 'el/8', '8', 'x86_64'],
+    ['AlmaLinux', 'el/9', '9', 'x86_64'],
+    ['Fedora', 'fedora/f36', '36', 'x86_64'],
+    ['CentOS', 'el/7', '7', 'x86_64'],
+    ['Amazon', 'el/6', '2017', 'x86_64'],
+    ['Amazon', 'el/6', '2018', 'x86_64'],
+    ['Amazon', 'el/7', '2', 'x86_64'],
+    ['Amazon', 'amazon/2', '2', 'aarch64'],
+    ['Amazon', 'amazon/2023', '2023', 'x86_64'],
+    ['Amazon', 'amazon/2023', '2023', 'aarch64'],
+  ].each do |os, urlbit, osmajor, arch|
     context "with #{os} and #{urlbit}" do
       let(:facts) do
-        override_facts(super(), os: { name: os, release: { major: osmajor } })
+        override_facts(super(), os: { name: os, release: { major: osmajor }, architecture: arch })
       end
 
       script = <<-SCRIPT
@@ -148,7 +150,7 @@
           is_expected.to contain_yumrepo('pc_repo')
             .with({
                     # We no longer expect the 'f' in fedora repos
-                    'baseurl'  => "http://yum.puppet.com/puppet5/#{urlbit.gsub('/f', '/')}/x64",
+                    'baseurl'  => "http://yum.puppet.com/puppet5/#{urlbit.gsub('/f', '/')}/#{arch}",
                     'enabled'  => 'true',
                     'gpgcheck' => '1',
                     'gpgkey'   => "file:///etc/pki/rpm-gpg/RPM-GPG-KEY-puppet\n  file:///etc/pki/rpm-gpg/RPM-GPG-KEY-puppet-20250406",
@@ -165,7 +167,7 @@
           }
         end
 
-        it { is_expected.to contain_yumrepo('pc_repo').with_baseurl("http://fake-yum.com/puppet5/#{urlbit.gsub('/f', '/')}/x64") }
+        it { is_expected.to contain_yumrepo('pc_repo').with_baseurl("http://fake-yum.com/puppet5/#{urlbit.gsub('/f', '/')}/#{arch}") }
       end
     end
   end