-
Notifications
You must be signed in to change notification settings - Fork 216
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
test(mc): initial multicloud IaC boilerplate GKE, AKS, Kind #1268
base: main
Are you sure you want to change the base?
Conversation
|
||
1. Docker installed on the host machine | ||
|
||
## Quickstart |
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.
I would suggest creating sub-items under Quickstart for AKS/GKE/Kind and put all steps to create each of them in corresponding section. I think it's easier to read and follow. And a separate section for Cleanup/Destroy on same level as Quickstart
default_node_pool { | ||
name = "agentpool" | ||
node_count = 2 | ||
vm_size = "Standard_D4ds_v5" |
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.
can node_count and vm_size also be set with input variables? This would help setting up a broader range of test scenarios
load_balancer_profile { | ||
managed_outbound_ip_count = 1 | ||
} | ||
pod_cidr = "10.244.0.0/16" |
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.
can pod_cidr be settable? More IPs are needed for scale tests
vm_size = "Standard_D4ds_v5" | ||
os_disk_size_gb = 128 | ||
os_disk_type = "Ephemeral" | ||
max_pods = 110 |
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.
can max_pods be set to 250?
@@ -0,0 +1,5 @@ | |||
variable "retina_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.
is it possible to provide a path to a values.yaml
file with desired override values to deploy?
Description
Initial work on multi-cloud
Related Issue
#1267
Checklist
git commit -S -s ...
). See this documentation on signing commits.Screenshots (if applicable) or Testing Completed
All modules were tested and work as expected.
See automated tests here https://github.com/microsoft/retina/actions/runs/12952871303/job/36131206491
Additional Notes
Add any additional notes or context about the pull request here.
Please refer to the CONTRIBUTING.md file for more information on how to contribute to this project.