Fix azurerm_sql_database blob auditing error under online secondary create mode

[yupwei68]

Fix #6269

Fix #6283

[katbyte]

Thanks for this @yupwei68, should we also make the change for azurerm_mssql_database resource?

[yupwei68]

hi @katbyte , you mean add blob auditing policies in mssql_database?

[katbyte]

@yupwei68 - yep, given they are both basically the same resource just mssql is using the newer API

[yupwei68]

hi @katbyte , auditing polices are added to azurerm_mssql_database. All tests have been passed.
=== RUN TestAccAzureRMMsSqlDatabase_basic
=== PAUSE TestAccAzureRMMsSqlDatabase_basic
=== CONT TestAccAzureRMMsSqlDatabase_basic
--- PASS: TestAccAzureRMMsSqlDatabase_basic (327.69s)
=== RUN TestAccAzureRMMsSqlDatabase_requiresImport
=== PAUSE TestAccAzureRMMsSqlDatabase_requiresImport
=== CONT TestAccAzureRMMsSqlDatabase_requiresImport
--- PASS: TestAccAzureRMMsSqlDatabase_requiresImport (486.89s)
=== RUN TestAccAzureRMMsSqlDatabase_complete
=== PAUSE TestAccAzureRMMsSqlDatabase_complete
=== CONT TestAccAzureRMMsSqlDatabase_complete
--- PASS: TestAccAzureRMMsSqlDatabase_complete (324.84s)
=== RUN TestAccAzureRMMsSqlDatabase_elasticPool
=== PAUSE TestAccAzureRMMsSqlDatabase_elasticPool
=== CONT TestAccAzureRMMsSqlDatabase_elasticPool
--- PASS: TestAccAzureRMMsSqlDatabase_elasticPool (270.30s)
=== RUN TestAccAzureRMMsSqlDatabase_GP
=== PAUSE TestAccAzureRMMsSqlDatabase_GP
=== CONT TestAccAzureRMMsSqlDatabase_GP
--- PASS: TestAccAzureRMMsSqlDatabase_GP (293.69s)
=== RUN TestAccAzureRMMsSqlDatabase_GP_Serverless
=== PAUSE TestAccAzureRMMsSqlDatabase_GP_Serverless
=== CONT TestAccAzureRMMsSqlDatabase_GP_Serverless
--- PASS: TestAccAzureRMMsSqlDatabase_GP_Serverless (306.51s)
=== RUN TestAccAzureRMMsSqlDatabase_BC
=== PAUSE TestAccAzureRMMsSqlDatabase_BC
=== CONT TestAccAzureRMMsSqlDatabase_BC
--- PASS: TestAccAzureRMMsSqlDatabase_BC (721.70s)
=== RUN TestAccAzureRMMsSqlDatabase_HS
=== PAUSE TestAccAzureRMMsSqlDatabase_HS
=== CONT TestAccAzureRMMsSqlDatabase_HS
--- PASS: TestAccAzureRMMsSqlDatabase_HS (836.07s)
=== RUN TestAccAzureRMMsSqlDatabase_createCopyMode
=== PAUSE TestAccAzureRMMsSqlDatabase_createCopyMode
=== CONT TestAccAzureRMMsSqlDatabase_createCopyMode
--- PASS: TestAccAzureRMMsSqlDatabase_createCopyMode (505.46s)
=== RUN TestAccAzureRMMsSqlDatabase_createPITRMode
=== PAUSE TestAccAzureRMMsSqlDatabase_createPITRMode
=== CONT TestAccAzureRMMsSqlDatabase_createPITRMode
--- PASS: TestAccAzureRMMsSqlDatabase_createPITRMode (1320.76s)
=== RUN TestAccAzureRMMsSqlDatabase_createSecondaryMode
=== PAUSE TestAccAzureRMMsSqlDatabase_createSecondaryMode
=== CONT TestAccAzureRMMsSqlDatabase_createSecondaryMode
--- PASS: TestAccAzureRMMsSqlDatabase_createSecondaryMode (423.64s)
=== RUN TestAccAzureRMMsSqlDatabase_threatDetectionPolicy
=== PAUSE TestAccAzureRMMsSqlDatabase_threatDetectionPolicy
=== CONT TestAccAzureRMMsSqlDatabase_threatDetectionPolicy
--- PASS: TestAccAzureRMMsSqlDatabase_threatDetectionPolicy (429.83s)
=== RUN TestAccAzureRMSqlDatabase_withBlobAuditingPolices
=== PAUSE TestAccAzureRMSqlDatabase_withBlobAuditingPolices
=== CONT TestAccAzureRMSqlDatabase_withBlobAuditingPolices
--- PASS: TestAccAzureRMSqlDatabase_withBlobAuditingPolices (599.26s)

[katbyte]

Thanks @yupwei68! LGTM 👍

[ghost]

I'm going to lock this issue because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.

If you feel this issue should be reopened, we encourage creating a new issue linking back to this one for added context. If you feel I made an error 🤖 🙉 , please reach out to my human friends 👉 [email protected]. Thanks!