Skip to content
This repository has been archived by the owner on Oct 10, 2021. It is now read-only.

add CentOS 7 support #1

Merged
merged 1 commit into from
Oct 24, 2018
Merged

add CentOS 7 support #1

merged 1 commit into from
Oct 24, 2018

Conversation

seth-shaw-unlv
Copy link
Contributor

GitHub Issue: Islandora/documentation/issues/946

What does this Pull Request do?

Creates the Ubuntu Apache conf directories in CentOs so playbooks that expect them won't crash.

What's new?

A new task restricted to RedHat/CentOS hosts that creates conf-available and conf-enabled in httpd.

How should this be tested?

Matomo should install on CentOS.

Interested parties

@Natkeeran, @Islandora-CLAW/committers

creates the conf directories available in Ubuntu so playbooks that expect them won't crash.
@dannylamb
Copy link
Contributor

vagrant up 🚀

@dannylamb
Copy link
Contributor

After pointing to your fork in the requirements file and removing the version tag, I'm getting the following after running ISLANDORA_DISTRO="centos/7" vagrant up:

TASK [Islandora-Devops.matomo : Install matomo httpd config file] **************
Tuesday 23 October 2018  17:02:22 -0300 (0:00:00.058)       0:42:42.378 ******* 
fatal: [default]: FAILED! => {"changed": false, "checksum": "19ad871bc132b61eac279ab208fbf9f19cd09bc9", "msg": "Destination directory /etc/httpd/conf-available does not exist"}

@seth-shaw-unlv
Copy link
Contributor Author

@dannylamb
Hmmm... perhaps vagrant didn't update your copy of the role in roles/external. Can you verify that config_webserver.yml has the added task ("Create httpd config directories for CentOS")?

If the change wasn't applied you can either rm -rf roles/external to force a re-download or manually patch it.

@dannylamb
Copy link
Contributor

@seth-shaw-unlv I just blew up roles/external and am trying again. I searched my terminal output for "Create httpd config directories for CentOS" and got zero hits, so it probably is pulling from its cache instead of grabbing your version of the role.

Copy link
Contributor

@dannylamb dannylamb left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

:shipit: Works as expected

@dannylamb dannylamb merged commit d6175fc into islandora-deprecated:master Oct 24, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants