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

Deployment fixes and updates. #11617

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

anders-alex
Copy link
Contributor

Required items, please complete

Change(s):

  • Fixed issue where previous method to deploy Azure Files Share (required for Consumption and Elastic Premium plans) no longer worked and was causing depoyments to fail.
    - Removed key-based authentication between Function App and Storage Account where supported for additional security.
    - Updated Function App PowerShell to version 7.4.
    - Updated post deployment script to use Az module 12.4.
    - Removed hard-coded GUID for Log Analytics role assignment so it doesn't error out in the event of a redeployment.
    - Updated resource provider API versions to latest.

Reason for Change(s):

Testing Completed:

  • Yes
image

Checked that the validations are passing and have addressed any issues that are present:

  • Yes

@anders-alex anders-alex requested review from a team as code owners January 2, 2025 05:21
@v-prasadboke v-prasadboke self-assigned this Jan 2, 2025
@v-prasadboke v-prasadboke added Connector Connector specialty review needed Standalone Standalone Content labels Jan 2, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Connector Connector specialty review needed Standalone Standalone Content
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants