Separate variables for default and internal nodepools VM sizes

All Submissions:

---

• [YES] Have you added an explanation of what your changes do and why you'd like us to include them?
• [NO] I have updated the documentation accordingly.
• [YES] My code follows the code style of this project.

Current Behavior or Linked Issues

---

Added a separate variable to set different VM sizes for default and internal nodepools and renamed the variable aks_agent_vm_size for a clear description.

Does this introduce a breaking change?

---

• [NO]

Other information

---

assigned to @heba92

added 1 commit

• 47e6fb59 - Add tf var for aks_agent_vm_size_default and aks_agent_vm_size_internal

Compare with previous version

CC @jordanchiu @lucynliu @mmuralidhar @ladatz

approved this merge request

LGTM looks more ordered and more parametrized now, @lucynliu @jordanchiu kindly requesting your approval.

@ADolot can you put a more meaningful MR name (Update 9 files does not sound very meaningful)?

changed title from Update 9 files to Separate variable for default and internal nodepools VM sizes

@heba92 I haven't noticed it, sorry. Changed MR name to normal one

changed title from Separate variable for default and internal nodepools VM sizes to Separate variables for default and internal nodepools VM sizes

requested review from @jordanchiu

added 1 commit

• 3347febe - Update terraform.tfvars

Compare with previous version

reset approvals from @heba92 by pushing to the branch

resolved all threads

approved this merge request

merged

mentioned in commit 241b9e3a

mentioned in issue #332 (closed)

#332 (closed) can be handled as a separate entity.

CC @lucynliu @jordanchiu

changed milestone to %M25 - Release 0.28