SqlAzureDacpacDeploymentV1 - Support for SQL Server

[emdempster]

Task name: SqlAzureDacpacDeploymentV1

Description: Added support for SQL Server deployments (on-premises or in another cloud). Azure Subscription is no longer required, as the code only uses the field for telemetry. It is still optional. Additionally, Windows authentication has been added to support SQL Server deployments.

Documentation changes required: Y

Added unit tests: Y

Attached related issue: Y - #19636

Checklist:

• Task version was bumped - please check instruction how to do it
• Checked that applied changes work as expected

[v-mohithgc]

/azp run

[azure-pipelines]

Azure Pipelines failed to run 3 pipeline(s).

[emdempster]

/azp run

[azure-pipelines]

Commenter does not have sufficient privileges for PR 19648 in repo microsoft/azure-pipelines-tasks

[v-mohithgc]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 3 pipeline(s).

[emdempster]

/azp run

[azure-pipelines]

Commenter does not have sufficient privileges for PR 19648 in repo microsoft/azure-pipelines-tasks

[ErikEJ]

@emdempster Looks like the task versions were not bumped

[ErikEJ]

I have claimed patch 7 - assuming this is in before my future PR 😄

[emdempster]

@ErikEJ thanks for the head's up. The version has been bumped. There's been a lot of other activity in this task in the past couple weeks.

[emdempster]

@v-mohithgc or @manolerazvan, could I get pipelines kicked off for this PR one more time? Apologies for asking, but as a non-employee, I'm not sure how to get the checks going.

[v-mohithgc]

/azp run

[azure-pipelines]

Azure Pipelines successfully started running 3 pipeline(s).

[emdempster]

@v-mohithgc, is there anything I need to finish this PR? Or is it ready to move forward with a merge?

[emdempster]

@manolerazvan and @microsoft/release-management-task-team, could I get an approval on this change? @dzsquared has previously approved the change to include Windows Authentication for SQL Server support.