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(terraform): add best practice recommendation to README for AKS cluster configuration and networking feat(terraform): implement separate VNets for each AKS cluster as a best practice #555

Merged
merged 3 commits into from
Aug 7, 2024

Conversation

robinmordasiewicz
Copy link
Owner

@robinmordasiewicz robinmordasiewicz commented Aug 7, 2024

Focuses on two key changes:

  1. Documentation Enhancement:

    • Added a best practice recommendation to the README.md file for configuring AKS clusters and networking. It emphasizes separating each cluster into different VNets and avoiding creating multiple clusters in the same subnet. This addition aims to improve clarity and adherence to best practices for AKS configuration.
  2. Infrastructure Implementation:

    • Implemented separate VNets for each AKS cluster as a best practice. This change involves defining separate subnet configurations for AKS clusters within the terraform files. By segregating the clusters into distinct VNets, the infrastructure setup follows a more secure and scalable approach.

These changes align with industry best practices and aim to enhance the project's overall architecture and maintainability.

robinmordasiewicz and others added 3 commits August 6, 2024 20:13
…cluster configuration and networking

feat(terraform): implement separate VNets for each AKS cluster as a best practice
…er models and improve performance

fix(locals.tf): comment out unsupported GPU size for the current Azure Linux distribution
@robinmordasiewicz robinmordasiewicz merged commit bce2e34 into main Aug 7, 2024
@robinmordasiewicz robinmordasiewicz deleted the dev99 branch August 7, 2024 00:14
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.

1 participant