Skip to content

UNN IdP in group_vars #350

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.

Sign up for GitHub

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

Open
cudanor wants to merge 4 commits into
base: master
Choose a base branch
from
Open

UNN IdP in group_vars #350

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
5bbadd1
Update site.yml
cudanor Jan 10, 2017
44b98c7
Create unn_idp
cudanor Jan 10, 2017
23bc932
Update site.yml
cudanor Jan 12, 2017
fb7960c
Update site.yml
cudanor Jan 12, 2017
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
34 changes: 34 additions & 0 deletions Ansible/group_vars/unn_idp
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,34 @@
# institute metadata
site_name: UNN IdP
Copy link
Member

@brucellino brucellino Jan 11, 2017

Choose a reason for hiding this comment

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

The name should not have spaces. Also, it should refer to the site (UNN), not a specific service (IdP). I would change this to lower-case : site_name: unn

host_institute:
name: University of Nigeria
url: http://www.unn.edu.ng

# LDAP variables
server_country: NG
server_state:
server_location: Nsukka
server_organization: NGREN
server_organization_unit: UNN
ldap_server: ldap.grid.unn.edu.ng
root_password_clear: HiPIC#17
Copy link
Member

@brucellino brucellino Jan 11, 2017

Choose a reason for hiding this comment

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

Move this variable (and all others) to passwords-{{ site_name }}.yml (e.g. passwords-unn.yml). This is pulled in dynamically by the playbook. You should encrypt those passwords with ansible-vault, or leave them out of the repo entirely.

sgw_admin: collins.udanor@unn.edu.ng

# institute metadata
host_institute:
name: University of Nigeria Nsukka
url: http://www.unn.edu.ng

organisation:
name: UNN
logo: unnlogo.jpg

mail_contact: collins.udanor@unn.edu.ng
# IDP variables. these are specifically related to the IdPOpen Web front end.
idp:
name: UNN IdP
admin_user: Collins Udanor
admin_email: collins.udanor@unn.edu.ng
metadata_url: https://{{ hostvars[groups['shibboleth-idps'][0]]['ansible_fqdn']}}/idp/shibboleth
mail_server:
header_logo: Logo.jpg
4 changes: 2 additions & 2 deletions Ansible/site.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -19,9 +19,9 @@
slack:
Copy link
Member

@brucellino brucellino Jan 11, 2017

Choose a reason for hiding this comment

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

Revert the changes in this file - they are incorrect.

domain: africa-arabia-roc.slack.com
token: 6EJfIMngdMiwrmnScAA4znB5
msg: "Starting site playbook on {{ site_name }} ( including {{ inventory_hostname }})"
msg: "Starting site playbook on {{ site_name}} ( including {{ inventory_hostname }})"
channel: "#devopssite"
username: "Ansible on site {{ site_name }}"
username: "Ansible on site {{ site_name}}"
#icon_url: "http://www.example.com/some-image-file.png"
link_names: 0
parse: 'none'
Expand Down