Fix SSL certificates used for accounting

Azure / cyclecloud-slurm

Azure CycleCloud project to enable users to create, configure, and use Slurm HPC clusters.

MIT License

56 stars 42 forks source link

Fix SSL certificates used for accounting #134

Closed aditigaur4 closed 1 year ago

aditigaur4 commented 1 year ago

Azure Maria DB now requires combined SSL client certificates from 2 authorities. . This PR fixes the client certificates used by Azure Maria DB, and also adds a new certificate_url option for user to specify the certificate. If a user has specified certificate then it is used.

The issue is described here: https://learn.microsoft.com/en-us/azure/mariadb/concepts-certificate-rotation?WT.mc_id=Portal-SqlAzureExtension

aditigaur4 commented 1 year ago

I need to make one more change on this.

aditigaur4 commented 1 year ago

I like using copy_file's parameter to set the permissions instead of calling chown. I have a feeling _add("suse.sh", "suse.sh", 600) might be a part of a different change (SuSe support perhaps?), but it seems minor enough

Thats a good catch, the suse change should not be going in this commit. I ll fix this.