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

fix link to subnet tagging in development guide #623

Merged
merged 2 commits into from
Aug 23, 2021

Conversation

geoffcline
Copy link
Contributor

Issue, if available:
in this guide https://karpenter.sh/docs/development-guide/ and the link embedded here (below) does not work:

If you created your cluster on version 1.19 or above, you may need to tag your subnets as mentioned here.

Description of changes:
I changed the link to use a hugo reference, instead of linking to the old github content.

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@netlify
Copy link

netlify bot commented Aug 17, 2021

✔️ Deploy Preview for karpenter-docs-prod ready!

🔨 Explore the source changes: cd1e865

🔍 Inspect the deploy log: https://app.netlify.com/sites/karpenter-docs-prod/deploys/611ee3d7206053000706cce3

😎 Browse the preview: https://deploy-preview-623--karpenter-docs-prod.netlify.app

@@ -0,0 +1,49 @@
---
title: "Development Guide"
Copy link
Contributor

Choose a reason for hiding this comment

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

Can we call this Operational Requirements?

Copy link
Contributor

Choose a reason for hiding this comment

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

Guessing you didn't mean to commit this.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

yes, this has been removed from the PR :)

@@ -34,7 +34,7 @@ make delete # Uninstall Karpenter
* Make sure you have valid credentials to your development repository.
* `$KO_DOCKER_REPO` must point to your development repository
* Your cluster must have permissions to read from the repository
* If you created your cluster on version 1.19 or above, you may need to tag your subnets as mentioned [here](docs/aws/README.md). This is a temporary problem with our subnet discovery system, and is being tracked [here](https://github.com/awslabs/karpenter/issues/404#issuecomment-845283904).
* If you created your cluster on version 1.19 or above, you may need to tag your subnets as mentioned [here]({{< ref "{{< ref "/docs/getting-started/_index.md#tag-subnets" >}}" >}}). This is a temporary problem with our subnet discovery system, and is being tracked [here](https://github.com/awslabs/karpenter/issues/404#issuecomment-845283904).
Copy link
Contributor

Choose a reason for hiding this comment

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

Curious why you need a special ref. Don't markdown links work?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I think it's necessary when you are using an "#" anchor on a "_index.md"

@geoffcline geoffcline changed the title patch-dev-guide-1 fix link to subnet tagging in development guide Aug 19, 2021
@geoffcline
Copy link
Contributor Author

sorry for the PR that should have been a draft 😅

it's better now

@geoffcline geoffcline added the documentation Improvements or additions to documentation label Aug 19, 2021
@ellistarn ellistarn merged commit 35fb1f1 into aws:main Aug 23, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants