Join GitHub today
GitHub is home to over 36 million developers working together to host and review code, manage projects, and build software together.Sign up
Pass AgentVmSize to Cassandra Helm chart #225
Disclaimer: I know very little about ARM templates or Helm. I've also not been able to test the change since the Fortis deployment is non-trivial. So please review this change carefully. Please also let me know what would be the best strategy to verify the change.
@dmakogon reported that currently it's impossible to deploy Fortis on many VM sizes because Cassandra is only getting deployed if a very specific VM size is being used. This change should fix this behavior and enable Fortis to be deployed to many VM sizes.
From my understanding, the VM size to which Cassandra is being deployed via Helm is specified via the VmInstanceType Helm value. So this pull request wires through the AgentVmSize selected during the Fortis deployment (given by the parameter agentVmSize in ARM) to the Cassandra Helm chart set up site.