feat(TPG>=5.12)!: Add option to create and failover a replica instance in Postgresql and MsSQL sub-module

[imrannayer]

No description provided.

[imrannayer]

/gcbrun

[imrannayer]

/gcbrun

[g-awmalik]

Thanks for the PR. Please review my comments.

[g-awmalik]

Please combine main.tf, mssql1.tf and mssql2.tf together into main.tf.

[imrannayer]

@g-awmalik fixed. Can u plz review?

[aleksandersumowski]

Hi Imran,
that's an exciting and very timely for me feature. Can I ask why this ups the TPG requirement to 5.12?

[imrannayer]

@aleksandersumowski thanks for the feedback. PR 16932 is part of [5.12]( replica_configuration, ca_cert, and server_ca_cert fields to be flagged sensitive) which makes replica_configuration, ca_cert, and server_ca_cert fields to be flagged sensitive.

[aleksandersumowski]

Could I also ask you why the failover procedure has these preconditions?

- Set `enable_default_db` and `enable_default_user` to `null`
- Dont set `additional_databases`, `user_name`, `user_password` and `additional_users`
- `availability_type` in all replica should be set to `ZONAL`

[imrannayer]

@aleksandersumowski according to my understanding:

• availability_type for replica has to be Zonal. It can be change after the failover. If this is not the case then go ahead and test it and create an issue. We can fix the doc.
• Setting up default DB or additional user etc will be an issue when u recreate primary as secondary and have to remove these configurations as they may already exists. If you can handle it then go ahead and try.