Skip to content

Commit

Permalink
tests: handle package layering on CentOS
Browse files Browse the repository at this point in the history
In projectatomic#254, it is noted that the package layering of `httpd` isn't
working on CentOS because the `centos-logos` dependency is trying to
layer files on `/boot`.  Until CentOS gets a version of `rpm-ostree`
that supports layering on `/boot` (see coreos/rpm-ostree#969),
we need to use a different package to test package layering in CentOS.

This change brings in a conditional installation of `ntp` in the
CentOS case, until we have a newer version of `rpm-ostree` to use.
  • Loading branch information
miabbott committed Oct 5, 2017
1 parent 20d0a82 commit d9a439b
Showing 1 changed file with 25 additions and 2 deletions.
27 changes: 25 additions & 2 deletions tests/improved-sanity-test/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -481,9 +481,17 @@
- role: rpm_ostree_install
packages: httpd
reboot: false
when: ansible_distribution != 'CentOS'
tags:
- rpm_ostree_install

- role: rpm_ostree_install
packages: ntp
reboot: false
when: ansible_distribution == 'CentOS'
tags:
- rpm_ostree_install_centos

- role: reboot
tags:
- reboot_post_up_install
Expand All @@ -495,18 +503,27 @@

- role: rpm_ostree_install_verify
package: httpd
when: ansible_distribution != 'CentOS'
tags:
- rpm_ostree_install_verify
check_mode: no

- role: rpm_ostree_install_verify
package: ntp
binary: ntpd
when: ansible_distribution == 'CentOS'
tags:
- rpm_ostree_install_verify_centos

- role: httpd_start
httpd_port: 8080
when: ansible_distribution != 'CentOS'
tags:
- httpd_start

- role: url_verify
url: http://localhost:8080
destination: /dev/null
when: ansible_distribution != 'CentOS'
tags:
- url_verify

Expand Down Expand Up @@ -687,7 +704,13 @@
- role: rpm_ostree_uninstall_verify
package: httpd
tags:
- rpm_ostree_uninstall_verify
- rpm_ostree_uninstall_verify_httpd

- role: rpm_ostree_uninstall_verify
package: ntp
binary: ntpd
tags:
- rpm_ostree_uninstall_verify_ntp

# Check admin unlock overlayfs missing
- role: overlayfs_verify_missing
Expand Down

0 comments on commit d9a439b

Please sign in to comment.