-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Doc updates for README and Getting Started guide #413
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
looking great!
README.md
Outdated
managed_node_groups = { | ||
mg_m4l = { | ||
node_group_name = "managed-ondemand" | ||
instance_types = ["m4.large"] |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Lets use m5 instances as they are the latest.
README.md
Outdated
module "eks-blueprints" { | ||
source = "github.com/aws-ia/terraform-aws-eks-blueprints" | ||
module "eks_blueprints" { | ||
source = "github.com/aws-ia/terraform-aws-eks-blueprints" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pin the version
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Now now. We can do this when we register our module with Terraform Registry. We can simply introduce a version
variable to docs and examples
README.md
Outdated
|
||
depends_on = [module.eks-blueprints.managed_node_groups] | ||
module "eks_blueprints_kubernetes_addons" { | ||
source = "github.com/aws-ia/terraform-aws-eks-blueprints//modules/kubernetes-addons" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
pin version
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same as my above comment
```shell script | ||
terraform apply | ||
``` | ||
terraform apply -target="module.aws_vpc" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
👍🏽
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM! Few comments
docs/add-ons/index.md
Outdated
@@ -49,40 +49,42 @@ In order to deploy an add-on with default configuration, simply enable the add-o | |||
|
|||
```hcl | |||
module "eks-blueprints-kubernetes-addons" { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we can change the module names to use underscore (_
) instead of hyphen (-
) as a best practise and be consistent with TF standards. This might need changing in many other places as well
docs/core-concepts.md
Outdated
|
||
See our [`Teams`](./teams) documentation page for detailed information. | ||
|
||
## Application | ||
|
||
`Applications` represent the actual workloads that run within a Kubernetes cluster. The framework leverages a GitOps approach for deploying applications onto clusters. | ||
|
||
See our [`Applications](https://github.com/aws-ia/terraform-aws-eks-blueprints/blob/main/docs/add-ons/argocd.md#boostrapping) documentation for detailed information. | ||
See our [`Applications](./add-ons/argocd.md#boostrapping) documentation for detailed information. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We can keep this as full link to github pages. See my comments above
Updating doc paths Co-authored-by: Bryant Biggs <bryantbiggs@gmail.com> Co-authored-by: Vara Bonthu <vara.bonthu@gmail.com>
c05728e
to
7314c13
Compare
686305b
to
a86264f
Compare
What does this PR do?
Documentation updates. Readme, getting started guide, and a few other doc pages.
Motivation
More
pre-commit run -a
with this PRNote: Not all the PRs required examples and docs except a new pattern or add-on added.
For Moderators
Additional Notes