Enhance AMI task docs after review

[chrisnegus]

Description
Updated the Managing AMIs docs after a formal review.

Does this change impact docs?

• Yes, PR includes docs updates
  By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

[netlify]

✅ Deploy Preview for karpenter-docs-prod ready!

Name	Link
🔨 Latest commit	af5d297
🔍 Latest deploy log	https://app.netlify.com/sites/karpenter-docs-prod/deploys/65e1fbd71d0fba000810c344
😎 Deploy Preview	https://deploy-preview-5749--karpenter-docs-prod.netlify.app
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site configuration.

[coveralls]

Pull Request Test Coverage Report for Build 8113843942

Warning: This coverage report may be inaccurate.

This pull request's base commit is no longer the HEAD commit of its target branch. This means it includes changes from outside the original pull request, including, potentially, unrelated coverage changes.

• For more information on this, see Tracking coverage changes with pull request builds.
• To avoid this issue with future PRs, see these Recommended CI Configurations.
• For a quick fix, rebase this PR at GitHub. Your next report should be accurate.

Details

• 0 of 0 changed or added relevant lines in 0 files are covered.
• No unchanged relevant lines lost coverage.
• Overall coverage increased (+0.02%) to 82.642%

---

Totals
Change from base Build 8089078497:	0.02%
Covered Lines:	5256
Relevant Lines:	6360

---

💛 - Coveralls

[jmdeal]

Looking good! A couple of small comments, I only left them on one of the copies, but they of course apply to both. I'm also wondering if we should include this in the v0.33 and v0.32 docs as well, just with a disclaimer in the disruption budgets task that users will need to upgrade to v0.34. Originally I was thinking we could maybe include it with Task 3 dropped but if anything including it gives users a good example of why they should upgrade.

[chrisnegus]

Looking good! A couple of small comments, I only left them on one of the copies, but they of course apply to both. I'm also wondering if we should include this in the v0.33 and v0.32 docs as well, just with a disclaimer in the disruption budgets task that users will need to upgrade to v0.34. Originally I was thinking we could maybe include it with Task 3 dropped but if anything including it gives users a good example of why they should upgrade.

I totally agree with that. I've worked on docs projects where we only published one version, but noted when a feature was added when we described it for exactly the reason you mentioned. Can we merge this, then I can follow up with another PR to update and copy the content to v0.32 and v0.33?

[jmdeal]

Sure, seems fine to take as a follow-up. LGTM 🚀