viya4-iac-azure icon indicating copy to clipboard operation
viya4-iac-azure copied to clipboard

Published 20 hours ago • verified publisher icon sassoftware

docs: (IAC-697): Updated the min_nodes value for compute node to 1

Open riragh opened this issue 2 years ago • 3 comments

Changes:

Updated the min_nodes to 1 for compute node in variables.tf and CONFIG-VARS.md node_pool example.

Tests:

With min_nodes set to 1 for compute node, verified the cluster was created with 1 compute node and 2 system nodes. After running the viya4-deployment all the pods initially started on stateful node, later cas node was created. After deployment stabilized logged in to SASStudio, compute context check was successful, SASStudio was usable.

riragh avatar Oct 18 '22 17:10 riragh

Are we going to say anywhere in the README.md or the CONFIG-VARS.md file why we need at least 1 compute node? Folks will be asking and need to clarify somewhere here. Thanks.

Good point, I'll reach out to you for details.

riragh avatar Oct 19 '22 13:10 riragh

I think in the doc or sample file we should have in the NOTE section of the table or as a NOTE in the paragraph we just explain why compute needs 1 node. Very simple and clean.

thpang avatar Oct 19 '22 15:10 thpang

I think in the doc or sample file we should have in the NOTE section of the table or as a NOTE in the paragraph we just explain why compute needs 1 node. Very simple and clean.

Added the note in CONFIG-VARS.md file.

riragh avatar Oct 21 '22 20:10 riragh