Skip to content

Commit

Permalink
Increase the travis test coverage to include popular options (ANXS#305)
Browse files Browse the repository at this point in the history
* Increse the travis test coverage to include popular options

* Fixed a missing quote...

* Fixed syntax for options, and make postgis.yml RedHat compatible... ish

* Looks like pgtune won't install either, as the epel repo isn't in the docker image.  Taking it out too..

* Changing parameters to be in JSON format, for consumption on the command line

* Adding a space to trigger a travis build

* Remove the space, as travis didn't build anyway... :-/

* Debugging... why don't travis kick in...

* Revert travis changes, to try and get it to trigger

* Fixed typo in JSON parameters

* Removed travis updates, and put parameters into postgresql.yml

* Squashing all commits.  Increases travis coverage, but not all are enabled yet. Other pull requests need to be merged first, as the tests for them fail.
  • Loading branch information
Greg Clough authored and UnderGreen committed Mar 18, 2018
1 parent d41d970 commit 7346fdb
Show file tree
Hide file tree
Showing 2 changed files with 19 additions and 1 deletion.
13 changes: 12 additions & 1 deletion tasks/extensions/postgis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -5,12 +5,23 @@
- "../vars/{{ ansible_distribution_release }}.yml"
- "../vars/empty.yml"

- name: PostgreSQL | Extensions | Make sure the postgis extensions are installed
- name: PostgreSQL | Extensions | Make sure the postgis extensions are installed | Debian
apt:
name: "{{item}}"
state: present
update_cache: yes
cache_valid_time: "{{apt_cache_valid_time | default (3600)}}"
with_items: "{{ postgresql_ext_postgis_deps }}"
when: ansible_os_family == "Debian"
notify:
- restart postgresql

- name: PostgreSQL | Extensions | Make sure the postgis extensions are installed | RedHat
yum:
name: "{{item}}"
state: present
update_cache: yes
with_items: "{{ postgresql_ext_postgis_deps }}"
when: ansible_os_family == "RedHat"
notify:
- restart postgresql
7 changes: 7 additions & 0 deletions tests/docker/group_vars/postgresql.yml
Original file line number Diff line number Diff line change
Expand Up @@ -19,3 +19,10 @@ postgresql_users:
postgresql_user_privileges:
- name: baz
db: foobar

postgresql_ext_install_contrib: true

# These do not work everywhere, but should be included ASAP
postgresql_ssl: false
postgresql_pgtune: false
postgresql_ext_install_postgis: false

0 comments on commit 7346fdb

Please sign in to comment.