-
Notifications
You must be signed in to change notification settings - Fork 13
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
Feature: new docs_library group and introduced more generic stack_name for groups of machines. #507
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
…up_vars/docs.yml.
…ant files for a group in group_vars/, static_inventories/ and .vault/
…ith {{ stack_name }} to accomodate other groups of machines that are not Slurm clusters.
…luster_name }}" for either {{ stack_prefix }} or {{ stack_name }} to accomodate other groups of machines that are not Slurm clusters and create shorter names for OpenStack components.
…urity_group module.
…for IRODs machines was too long and has to be < 255 characters. Name of task that creates ip_addresses.yml.new listed ip_addresses.yml as the file being created, which was confusing.
… 1 -> 20, because current iRODS VMs have have local volmes of 20 GB and the playbook cannot shrink volumes resulting in a "Cannot shrink volumes" error.
…ars.yml, because it is also needed for the static_hostname_lookup role on jumphosts and group_vars/irods.yml is not used for jumphosts.
…group_playbooks/jumphost.yml and single_group_playbooks/cluster_part2.yml.
…group_playbooks/data_transfer.yml.
…for all relevant clusters.
…ts Python dependencies using a pulp-cli-requirements.txt file with locked version numbers.
…s with Ansible playbooks, which need to be able to list all groups.
…rvin config to give it its own CA key pair for signing host keys.
…r ldap_credentials.
pneerincx
requested review from
Gerbenvandervries,
scimerman,
erijpkema and
marieke-bijlsma
and removed request for
Gerbenvandervries
December 23, 2021 12:18
erijpkema
approved these changes
Dec 23, 2021
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
docs
server on Merlin to replace the one running on one of the EOL OpenStack boxes:docs_library
inventory group.docs_on_merlin
server togroup_vars
andstatic_inventories
for all relevant clusters.docs_library
inventory group is the second one following the iRODS machines that does not contain Slurm cluster machines. Therefore this PR contains major refactoring to replace{{ slurm_cluster_name }}_cluster
with{{ stack_name }}
to accommodate other groups of machines.{{ stack_name }}
variable.{{ stack_name }}
.lor-init
: updated to find the correct static inventory and vault password for a {{ stack_name }}. Can also use {{ stack_prefix }} to find the corresponding{{ stack_name }}
.deploy-os_servers.yml
:ip_addresses.yml.new
listedip_addresses.yml
as the file being created, which was confusing.hosts
foropenstack.cloud.security_group
module.group_vars/nibbler_cluster/vars.yml
:local_volume_size_irods
from1
to20
, because current iRODS VMs have have local volumes of 20 GB and the playbook cannot shrink volumes resulting in aCannot shrink volumes
error.irods_icat_fqdn
, because it is also needed for thestatic_hostname_lookup
role on jumphosts andgroup_vars/irods.ym
l is not used for jumphosts.Bugfix: fixed
when
statements for deploying eitherldap
orsssd
roles in:single_group_playbooks/jumphost.yml
single_group_playbooks/cluster_part2.yml
single_group_playbooks/data_transfer.yml
pulp-installer
to 3.16.0.pulpcore
andpulp-rpm
to 3.16.1.pulp-cli
to 0.13.0pulp-cli
as well as all its Python dependencies using apulp-cli-requirements.txt
file with locked version numbers.enumerate false
->true
to fix issues with Ansible playbooks, which need to be able to list all groups.generate_secrets.py
to support nested data structures e.g. forldap_credentials
as used bysssd_client
role.