Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removed config type #126

Open
wants to merge 2 commits into
base: devel
Choose a base branch
from
Open

Conversation

watucker
Copy link

Fixes #

Proposed Changes

  • Removes config_type template and type_bits variables. They don't appear to have any purpose since nothing references them and Ansible now does the job they used to do.

@nathanlcarlson
Copy link
Contributor

The commit that put those in place ( 3bbaa1a ) mentions this file was required to be in place for the dashboard-ip service. The service, if I remember correctly, assumes this file is in place and makes decisions based on the value in the file. I think the dashboard-ip service is having some trouble with RHEL/CentOS 7 currently. The GeoIP .dat files are not in the correct place on 7. Anyway, I agree this file would be great to remove. I think I'll leave this open for now though.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants