Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

ADF deployment scripts should be changed to use Az instead of deprecated AzureRM #122392

Closed
HaraldRau8 opened this issue May 10, 2024 · 4 comments

Comments

@HaraldRau8
Copy link

The scripts in this document for pre- and post installation tasks of an Azure Data Factory deployment via Azure DevOps use AzureRM which is deprecated since Feb 2024, resulting in warnings when using them in DevOps. Can you migrate them to Az please?


Document Details

Do not edit this section. It is required for learn.microsoft.com ➟ GitHub issue linking.

@HaraldRau8 HaraldRau8 changed the title ADF deployment scripts should be changed to use Az instead of deprecated AzureMR ADF deployment scripts should be changed to use Az instead of deprecated AzureRM May 10, 2024
@TPavanBalaji
Copy link

@HaraldRau8
Thanks for your feedback! We will investigate and update as appropriate.

@Naveenommi-MSFT
Copy link
Contributor

@HaraldRau8
Thank you for bringing this to our attention.
I've delegated this to content author @nabhishek, who will review it and offer their insightful opinions.

@Naveenommi-MSFT
Copy link
Contributor

@nabhishek
Could you please review add comments on this, update as appropriate.

@jonburchel
Copy link
Contributor

Hi @HaraldRau8, and thanks for your question. I checked, and the PrePostDeploymentScript.Ver2.ps1 appears to be using the Az PowerShell module, not AzureRM. I do see the Azure Powershell installation instructions still reference the old AzureRM (for backward compatibility since some customers may still be using it) but also refers to the Az module and also has a link discussing issues of coexistence if both modules are installed simultaneously. Is there a specific line in the script that uses AzureRM? I couldn't find any. Or could it be possible you've been using an older version of the script?

Since this issue in the docs appears to be resolved already, I will go ahead and close it out. However, we can continue the discussion here in this thread if necessary, even after the issue is closed. #please-close

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

5 participants