fix(contributing): postgresql-repo state ID requires python3-apt package

postgres/defaults.yaml

@@ -9,6 +9,7 @@ postgres:
   version: '10'
   pkg: postgresql
   pkgs_extra: []
+  pkgs_deps: []
   pkg_client: postgresql-client
   pkg_dev: postgresql-devel
   pkg_dev_deps: []
@@ -76,7 +77,7 @@ postgres:
     sysrc: false
 
   bake_image: false
-  bake_image_run_cmd: echo "do nothing by default"
+  bake_image_run_cmd: pg_ctl start
 
   fromrepo: ''
 

postgres/init.sls

@@ -4,6 +4,7 @@ include:
   - postgres.macos
 {% else %}
   - postgres.server
+  - postgres.server.image
   - postgres.client
   - postgres.manage
 {% endif %}

postgres/osfamilymap.yaml

@@ -12,6 +12,7 @@ Arch:
   pkg_dev: postgresql
 
 Debian:
+  pkgs_deps: ['python3-apt']
   pkg_repo:
     humanname: PostgreSQL Official Repository
     key_url: 'https://www.postgresql.org/media/keys/ACCC4CF8.asc'

postgres/server/image.sls

@@ -5,11 +5,7 @@
 
 {%- if postgres.bake_image %}
 
-include:
-  - postgres.server
-
 # An attempt to start PostgreSQL with `pg_ctl`
-
 postgresql-running:
   cmd.run:
     - name: {{ postgres.bake_image_run_cmd }}
@@ -40,10 +36,4 @@ postgresql-enable:
     - require:
       - cmd: postgresql-running
 
-{%- else %}
-
-postgresql-running:
-  test.show_notification:
-    - text: The 'postgres:bake_image' Pillar is disabled (set to 'False').
-
 {%- endif %}

postgres/server/init.sls

@@ -1,9 +1,6 @@
 {%- from salt.file.dirname(tpldir) ~ "/map.jinja" import postgres with context -%}
 
 {%- set includes = [] %}
-{%- if postgres.bake_image %}
-  {%- do includes.append('postgres.server.image') %}
-{%- endif %}
 {%- if postgres.use_upstream_repo == true -%}
   {%- do includes.append('postgres.upstream') %}
 {%- endif %}

postgres/upstream.sls

@@ -17,6 +17,11 @@ postgresql-profile:
     - defaults:
         bin_dir: {{ postgres.bin_dir }}
     {%- endif %}
+
+postgresql-pkg-deps:
+  pkg.installed:
+    - pkgs: {{ postgres.pkgs_deps }}
+
 # Add upstream repository for your distro
 postgresql-repo:
   pkgrepo.managed: