move desc and baseurl into yum class, they are not used otherwise

voxpupuli · Oct 28, 2023 · de6e3fd · de6e3fd
1 parent 4ed632a
commit de6e3fd
Show file tree

Hide file tree

Showing 2 changed files with 4 additions and 5 deletions.
diff --git a/manifests/repo/nodesource.pp b/manifests/repo/nodesource.pp
@@ -10,10 +10,6 @@
 
   case $facts['os']['family'] {
     'RedHat': {
-      # nodesource repo
-      $descr   = "Node.js Packages - \$basearch"
-      $baseurl = "https://rpm.nodesource.com/pub_${url_suffix}/nodistro/nodejs/\$basearch"
-
       contain 'nodejs::repo::nodesource::yum'
     }
     'Debian': {

diff --git a/manifests/repo/nodesource/yum.pp b/manifests/repo/nodesource/yum.pp
@@ -1,12 +1,15 @@
 # PRIVATE CLASS: Do not use directly.
 class nodejs::repo::nodesource::yum {
-  $baseurl        = $nodejs::repo::nodesource::baseurl
   $descr          = $nodejs::repo::nodesource::descr
   $ensure         = $nodejs::repo::nodesource::ensure
   $priority       = $nodejs::repo::nodesource::priority
   $proxy          = $nodejs::repo::nodesource::proxy
   $proxy_password = $nodejs::repo::nodesource::proxy_password
   $proxy_username = $nodejs::repo::nodesource::proxy_username
+  $url_suffix     = $nodejs::repo::nodesource::url_suffix
+
+  $descr   = 'Node.js Packages - \$basearch'
+  $baseurl = "https://rpm.nodesource.com/pub_${url_suffix}/nodistro/nodejs/\$basearch"
 
   $yum_failovermethod = (versioncmp($facts['os']['release']['major'], '8') >= 0 and $priority == 'absent') ? {
     true    => 'absent',