From be9f11e03e0706fc33b5db901241423dbb31b424 Mon Sep 17 00:00:00 2001
From: N <noel.mcloughlin@gmail.com>
Date: Wed, 3 Jul 2019 16:22:51 +0100
Subject: [PATCH] fix(repo): allow oscode pillar data but default to tumbleweed

---
 ceph/defaults.yaml    | 3 ++-
 ceph/osfamilymap.yaml | 3 ++-
 ceph/repo/map.jinja   | 7 ++++++-
 pillar.example        | 3 ++-
 4 files changed, 12 insertions(+), 4 deletions(-)

diff --git a/ceph/defaults.yaml b/ceph/defaults.yaml
index 04acadc..39b1c60 100644
--- a/ceph/defaults.yaml
+++ b/ceph/defaults.yaml
@@ -2,9 +2,10 @@
 # vim: ft=yaml
 
 ceph:
-  release: luminous
+  release: nautilus
   use_upstream_repo: true
   fromrepo: ''
+  oscode: ''
   packages:
     - ceph
   cluster_name: ceph
diff --git a/ceph/osfamilymap.yaml b/ceph/osfamilymap.yaml
index e84e074..83aa8d0 100644
--- a/ceph/osfamilymap.yaml
+++ b/ceph/osfamilymap.yaml
@@ -16,10 +16,11 @@ RedHat:
     baseurl: '{{ repo.official }}/rpm-{{ repo.release }}/el{{ grains.osrelease_info[0] }}/$basearch'
 
 Suse:
+  oscode: openSUSE_Tumbleweed
   pkg_repo:
     name: ceph-opensuse-{{ repo.release }}
     humanname: Ceph {{ repo.release }} $releasever - $basearch
-    baseurl: '{{ repo.suse }}:/ceph:/{{ repo.release }}/{{ repo.oscode }}/$basearch'
+    baseurl: '{{ repo.suse }}:/ceph:/{{ repo.release }}/{{ repo.oscode }}'
     key_url: '{{ repo.suse }}:/ceph:/{{ repo.release }}/{{ repo.oscode }}/repodata/repomd.xml.key'
     gpgcheck: 1
     gpgautoimport: True
diff --git a/ceph/repo/map.jinja b/ceph/repo/map.jinja
index 0bddb9e..26d320b 100644
--- a/ceph/repo/map.jinja
+++ b/ceph/repo/map.jinja
@@ -6,10 +6,15 @@
 {% set use_upstream_repo = salt['pillar.get']('ceph:use_upstream_repo', defaults.ceph.use_upstream_repo) %}
 {% set release = salt['pillar.get']('ceph:release', defaults.ceph.release) %}
 {% set fromrepo = salt['pillar.get']('ceph:fromrepo', defaults.ceph.fromrepo) %}
-{% set oscode = salt['grains.get']('oscodename') | replace(' ','_') %}
 {% set user = salt['pillar.get']('ceph.user', None) %}
 {% set group = salt['pillar.get']('ceph.group', None) %}
 
+{%- if grains.os_family == 'Suse' %}
+    {% set oscode = salt['pillar.get']('ceph.oscode','openSUSE_Tumbleweed') | replace(' ','_') %}
+{%- else %}
+    {% set oscode = salt['pillar.get']('ceph.oscode', salt['grains.get']('oscodename')) | replace(' ','_') %}
+{%- endif %}
+
 {% if grains.os == 'MacOS' %}
   {% set user = salt['pillar.get']('ceph.user') or salt['cmd.run']("stat -f '%Su' /dev/console") %}
   {% set group = salt['pillar.get']('ceph.group') or salt['cmd.run']("stat -f '%Sg' /dev/console") %}
diff --git a/pillar.example b/pillar.example
index d9d588d..c108859 100644
--- a/pillar.example
+++ b/pillar.example
@@ -1,6 +1,7 @@
 ceph:
-  release: luminous
+  release: nautilus
   use_upstream_repo: false
+  #oscode: OpenSUSE_Fumbleweed
 
   config:
     file: /etc/ceph/ceph.conf