Fix ubuntu-18.04 gems and postgres versions
authorJavier Bértoli <jbertoli@curii.com>
Fri, 19 Nov 2021 22:06:35 +0000 (19:06 -0300)
committerJavier Bértoli <jbertoli@curii.com>
Fri, 19 Nov 2021 22:06:35 +0000 (19:06 -0300)
Arvados-DCO-1.1-Signed-off-by: Javier Bértoli <jbertoli@curii.com>

test/integration/shell/controls/packages_spec.rb
test/salt/pillar/examples/postgresql.sls

index 7ce0f2d329599576b830ee761ba5277b3125f3c2..ac58157309f9c6f9c0bbd27085611873e8811bd4 100644 (file)
@@ -37,3 +37,29 @@ control 'arvados shell gems' do
     end
   end
 end
+
+control 'arvados shell gems' do
+  title 'should be installed'
+
+  only_if('Skipped in Centos-7 and Ubuntu-18.04') do
+    !((os.redhat? and platform[:release].to_f.truncate == 7) or
+    (os.name == 'ubuntu' and platform[:release].to_f.truncate == 18))
+  end
+
+  describe gem('arvados-cli') do
+    it { should be_installed }
+  end
+end
+
+control 'RVM arvados shell gems' do
+  title 'should be installed'
+
+  only_if('Forced requirement for Centos-7 and Ubuntu-18.04') do
+    (os.redhat? and platform[:release].to_f.truncate == 7) or
+      (os.name == 'ubuntu' and platform[:release].to_f.truncate == 18)
+  end
+
+  describe gem('arvados-cli', '/usr/local/rvm/bin/rvm all do gem') do
+    it { should be_installed }
+  end
+end
index 0bf33570e4de2205eeb037fc330d713f7616648e..a60b41f2f7e70b4e9df30a1e8ff4483d7f566315 100644 (file)
@@ -5,21 +5,24 @@
 
 ### POSTGRESQL
 postgres:
-  # Centos-7's postgres package is too old, so we need to force using upstream's
+  # Centos-7 and Ubuntu-18.04's postgres packages are too old, so we need to force using upstream's
   # This is not required in Debian's family as they already ship with PG +11
-  {%- if salt['grains.get']('os_family') == 'RedHat' %}
+  {%- if salt['grains.get']('osfinger') in ['Ubuntu-18.04', 'CentOS Linux-7'] %}
   use_upstream_repo: true
   version: '12'
 
+    {%- if salt['grains.get']('osfinger') == 'CentOS Linux-7' %}
   pkgs_deps:
     - libicu
     - libxslt
     - systemd-sysv
-
   pkgs_extra:
     - postgresql12-contrib
 
+    {%- endif %}
+
   {%- else %}
+  use_upstream_repo: false
   pkgs_extra:
     - postgresql-contrib
   {%- endif %}