sassoftware / viya4-deployment

This project contains Ansible code that creates a baseline in an existing Kubernetes environment for use with the SAS Viya Platform, generates the manifest for an order, and then can also deploy that order into the Kubernetes environment specified.
Apache License 2.0
71 stars 64 forks source link

Missing PVC Requirements for SAS® Model Risk Management #249

Closed DidouJazz closed 2 years ago

DidouJazz commented 2 years ago

Hi, According to Specify PersistentVolumeClaims to Use ReadWriteMany StorageClass, there is a requirement for SAS® Model Risk Management to have two PVCs called sas-risk-cirrus-search,sas-risk-cirrus-builder. I can see in the sas-storageclass.yaml that sas-risk-cirrus-search seems always created but this is not true for sas-risk-cirrus-builder. I'll temporarely add it to this yaml files but should we not add some logic to define if the product is in the order ? SIde-Note: I'll create a second issue as the same requirement exists for SAS Risk Modeling, with the sas-risk-modeling-core PVC. Let me know your thoughts, David

DidouJazz commented 2 years ago

Side-Note: I can't find the PVC definition in : sas-bases/base/components/sas-risk-cirrus-builder/resources.yaml (as opposition to what can be found in sas-bases/base/components/sas-risk-cirrus-search/resources.yaml). Can this be a documentation issue ?

DidouJazz commented 2 years ago

This PVC has been deleted according to our internal JIRA Ticket CIRRUSB-1464. I'll raise a documentation ticket.