This project contains Terraform configuration files to provision infrastructure components required to deploy SAS Viya platform products on Microsoft Azure Cloud.
When a new jump server vm is created, it sets file ownership and permissions for the mounted NFS location which allows Viya services to initialize successfully. Importantly, the file system settings should only be applied once and not repeatedly in the event that the jump vm is destroyed and recreated in the same cluster. This change checks if the ${jump_rwx_filestore_path}/pvs folder already exists and skips creating the folder and recursively setting ownerships and permissions if it does. If the jump vm is being created for the first time when the pvs folder is absent, creating the folder and setting permissions and ownership will occur an initial time and not thereafter.
This issue originated from AWS, but is applicable for all cloud providers. See the original fix in AWS PR.
Also, minor update to outputs.tf file: updated a missing condition.
Tests:
Verified the following steps, see details in internal ticket.
Replace jump server vm using terraform replace command (Alternative to destroy and create), verifyed that the existing jump vm is destroyed and a new one is created.
Changes:
When a new jump server vm is created, it sets file ownership and permissions for the mounted NFS location which allows Viya services to initialize successfully. Importantly, the file system settings should only be applied once and not repeatedly in the event that the jump vm is destroyed and recreated in the same cluster. This change checks if the
${jump_rwx_filestore_path}/pvsfolder already exists and skips creating the folder and recursively setting ownerships and permissions if it does. If the jump vm is being created for the first time when the pvs folder is absent, creating the folder and setting permissions and ownership will occur an initial time and not thereafter.This issue originated from AWS, but is applicable for all cloud providers. See the original fix in AWS PR.
Also, minor update to
outputs.tffile: updated a missing condition.Tests:
Verified the following steps, see details in internal ticket.
Steps: