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.

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

Jump to bottom

fixed various install issues #2

Open
wants to merge 14 commits into
base: main
Choose a base branch
from
Open

fixed various install issues #2

wants to merge 14 commits into from

Conversation

soichih
Copy link
Contributor

@soichih soichih commented Sep 12, 2019 •
edited
Loading

  • README talks about creating clouds.yaml. This file is auto-generated by install.sh using openrc content.
  • security group quota is usually very low (10 by default) so for me, the script was failing because of it. I've added a check for the quota.
  • on the new version of keycloak/openstack(?) project_domain_id needs to be set instead of project_domain_name. The ansible was failing to access the auth service without this.
  • I've added a check to make sure install.sh is run as root.
  • removing the local openrc.sh and clouds.yaml in case user forgets to remove it (just for added precaution)
  • headnode_create.sh was failing to create controller because it was pulling 2 images with matching grep patterns (breaking the CLI). I've added tail -1 to work around it.
    *added step to create a user called "user" which can be used to submit jobs. running as centos is not recommended as it has sudo access.

soichih and others added 14 commits September 12, 2019 19:32
@soichih
fixed various install issues
5278f2d
@soichih
removing clouds.yaml also
a5f8b86
@soichih
updated install.sh so that it can be rerun
ff52845 
added useradd for "user" user that can be used to run jobs as regular
user
@soichih
added a step to create "user"
0172b93
@soichih
replaced OS_USRNAME with cluster_name
420f788 
fixed a few bugs that I've created while updating
@soichih
added line to enable hostfs for singularity
e2ae1be
Merge branch 'master' of https://github.com/XSEDE/CRI_Jetstream_Cluster
9d5def4
@CicadaDennis
Cicada Dennis - added quotes around variable name, so awk doesn't cho…
3b16128 
…ke if the variable has spaces in it.
Added note about node_size modification
527ec7c
Merge pull request access-ci-org#3 from NCGAS/master
992de7e 
Cicada Dennis - added quotes around variable name, so awk doesn't cho…
added openrc generation
2ad3712 
Generate openrc in /etc/slurm/ from env vars rather than copy
since some openrc files ask for the password

Signed-off-by: Eric Coulter <[email protected]>
update slurm.conf at node creation time
8d0cee0 
pull in slurm.conf at node creation via
cloud-config write_files module, rather than
having to rebuild the instance image when only changing
number/size of nodes or partitions

Signed-off-by: Eric Coulter <[email protected]>
Merge branch 'master' of https://github.com/XSEDE/CRI_Jetstream_Cluster
a914cc0
@soichih
Merge branch 'master' of github.com:soichih/CRI_Jetstream_Cluster
8cb5740
DImuthuUpe pushed a commit that referenced this pull request Feb 3, 2022
@julianpistorius
Merge pull request #2 from DImuthuUpe/cluster-create-local
6e47729 
Fix for running firewall commands in cloud init
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@soichih @CicadaDennis