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

docs: fix a set of details spotted over time #2487

Merged
merged 1 commit into from
Apr 13, 2023

Conversation

consideRatio
Copy link
Contributor

@consideRatio consideRatio commented Apr 13, 2023

I've collected a list of smaller details in #2041 on things to fix, and in this PR I fix point 27-31 provided for reference below.

27

I had to create a directory for my new cluster in config/clusters before letting terraform write a file to that location.

28

I saw this, but CLUSTER_NAME wasn't prefixed with $

deployer deploy-support CLUSTER_NAME

29

When configuring domain names via namecheap, it would be good to link directly to where this is done: https://ap.www.namecheap.com/Domains/DomainControlPanel/2i2c.cloud/advancedns instead of namecheap.com.

See https://infrastructure.2i2c.org/en/latest/hub-deployment-guide/deploy-support/configure-support.html#setting-dns-records

30

This command should be without create as a standalone arg

deployer cilogon-client-create create 2i2c dask-staging daskhub dask-staging.2i2c.cloud

31

Update AWS account creation docs to suggest use of email sub-addressing, like support+aws-<account name>@2i2c.org instead of creating new emails.

Related to https://github.com/2i2c-org/meta/issues/535

@consideRatio consideRatio added the Documentation A change to our documentation. label Apr 13, 2023
@consideRatio consideRatio self-assigned this Apr 13, 2023
@consideRatio consideRatio requested a review from a team April 13, 2023 01:10
Copy link
Contributor

@pnasrat pnasrat left a comment

Choose a reason for hiding this comment

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

lgtm

@consideRatio consideRatio merged commit c70dd9c into 2i2c-org:master Apr 13, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Documentation A change to our documentation.
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

2 participants