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 deprecated patches support with kustomize for k8s v1.27.0 release #616

Merged
merged 2 commits into from
Oct 10, 2023

Conversation

mylesw42
Copy link
Contributor

What this PR does / why we need it:

The k8s v1.27.0 release notes talk about a major kustomize update. Specifically mentioning deprecating old style patches support. The newest documented format doesn't allow for YAML document merges, unless a "path:" key is provided.

This fixes all kustomize patch documents in all modules, and enables the workshop to be migrated to kubectl >= 1.27.0

It also fixes the yarn start and yarn build commands for local development.

Which issue(s) this PR fixes:

Fixes # #548

Quality checks

  • [y ] My content adheres to the style guidelines
  • [n] I ran make test or make e2e-test and it was successful
  • [y] Performed local yarn start and build command successfully.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@netlify
Copy link

netlify bot commented Jul 28, 2023

Deploy Preview for eks-workshop ready!

Name Link
🔨 Latest commit d660f7d
🔍 Latest deploy log https://app.netlify.com/sites/eks-workshop/deploys/6525db8c4f74da000876d37f
😎 Deploy Preview https://deploy-preview-616--eks-workshop.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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

@niallthomson
Copy link
Contributor

This will take a little time to test due to the number of labs touched.

@niallthomson niallthomson merged commit 80710d4 into aws-samples:main Oct 10, 2023
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants