azurerm_stream_analytics_output_servicebus_queue: Shared access policy name/key optional for MSI authentication

[marcgs]

shared_access_policy_name and shared_access_policy_key parameters made optional as these are not required when authenticationMode is MSI (Managed Service Identity). See analog PR for service bus topics: #19708

This requires the go sdk for stream analytics to be upgraded to version "2021-10-01-preview". This has been initiated in this PR. This has been prepared in PR #20145.

Acceptance tests were adapted and run successfully:

TF_ACC=1 go test -v ./internal/services/streamanalytics -run=TestAccStreamAnalyticsOutputServiceBusQueue -timeout 20m -ldflags="-X=github.com/hashicorp/terraform-provider-azurerm/version.ProviderVersion=acc"
=== RUN   TestAccStreamAnalyticsOutputServiceBusQueue_avro
=== PAUSE TestAccStreamAnalyticsOutputServiceBusQueue_avro
=== RUN   TestAccStreamAnalyticsOutputServiceBusQueue_csv
=== PAUSE TestAccStreamAnalyticsOutputServiceBusQueue_csv
=== RUN   TestAccStreamAnalyticsOutputServiceBusQueue_json
=== PAUSE TestAccStreamAnalyticsOutputServiceBusQueue_json
=== RUN   TestAccStreamAnalyticsOutputServiceBusQueue_authenticationModeMsi
=== PAUSE TestAccStreamAnalyticsOutputServiceBusQueue_authenticationModeMsi
=== RUN   TestAccStreamAnalyticsOutputServiceBusQueue_update
=== PAUSE TestAccStreamAnalyticsOutputServiceBusQueue_update
=== RUN   TestAccStreamAnalyticsOutputServiceBusQueue_requiresImport
=== PAUSE TestAccStreamAnalyticsOutputServiceBusQueue_requiresImport
=== RUN   TestAccStreamAnalyticsOutputServiceBusQueue_propertyColumns
=== PAUSE TestAccStreamAnalyticsOutputServiceBusQueue_propertyColumns
=== RUN   TestAccStreamAnalyticsOutputServiceBusQueue_systemPropertyColumns
=== PAUSE TestAccStreamAnalyticsOutputServiceBusQueue_systemPropertyColumns
=== CONT  TestAccStreamAnalyticsOutputServiceBusQueue_avro
=== CONT  TestAccStreamAnalyticsOutputServiceBusQueue_update
=== CONT  TestAccStreamAnalyticsOutputServiceBusQueue_csv
=== CONT  TestAccStreamAnalyticsOutputServiceBusQueue_json
=== CONT  TestAccStreamAnalyticsOutputServiceBusQueue_authenticationModeMsi
=== CONT  TestAccStreamAnalyticsOutputServiceBusQueue_propertyColumns
=== CONT  TestAccStreamAnalyticsOutputServiceBusQueue_systemPropertyColumns
=== CONT  TestAccStreamAnalyticsOutputServiceBusQueue_requiresImport
--- PASS: TestAccStreamAnalyticsOutputServiceBusQueue_json (442.69s)
--- PASS: TestAccStreamAnalyticsOutputServiceBusQueue_avro (446.55s)
--- PASS: TestAccStreamAnalyticsOutputServiceBusQueue_csv (448.20s)
--- PASS: TestAccStreamAnalyticsOutputServiceBusQueue_authenticationModeMsi (448.84s)
--- PASS: TestAccStreamAnalyticsOutputServiceBusQueue_requiresImport (464.29s)
--- PASS: TestAccStreamAnalyticsOutputServiceBusQueue_systemPropertyColumns (692.07s)
--- PASS: TestAccStreamAnalyticsOutputServiceBusQueue_propertyColumns (698.98s)
--- PASS: TestAccStreamAnalyticsOutputServiceBusQueue_update (743.26s)
PASS
ok      github.com/hashicorp/terraform-provider-azurerm/internal/services/streamanalytics       744.912s

[katbyte]

@marcgs - now that hashicorp/pandora#1946 has been merged is this PR ready to be reviewed/merged?

[marcgs]

Waiting for #20145 to be merged

[katbyte]

Thanks @marcgs - LGTM 🌩️

[github-actions]

This functionality has been released in v3.41.0 of the Terraform Provider. Please see the Terraform documentation on provider versioning or reach out if you need any assistance upgrading.

For further feature requests or bug reports with this functionality, please create a new GitHub issue following the template. Thank you!

[github-actions]

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.