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

Bug fixes when deploying function app #11443

Merged
merged 4 commits into from
Nov 27, 2024
Merged

Conversation

Cv-securityIQ
Copy link
Contributor

Adding AzureWebJobsStorage to environment variables, which is needed for the function app to run

Required items, please complete

Change(s):

  • Adding AzureWebJobsStorage to environment variables, which is needed for the function app to run

Reason for Change(s):

  • Bug Fixes

Testing Completed:

  • Yes

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

  • Yes

Adding AzureWebJobsStorage to environment variables, which is needed for the function app to run
@Cv-securityIQ Cv-securityIQ requested review from a team as code owners November 18, 2024 12:55
@v-atulyadav v-atulyadav added Connector Connector specialty review needed Solution Solution specialty review needed labels Nov 18, 2024
v-prasadboke
v-prasadboke previously approved these changes Nov 22, 2024
Fix git pipeline errors
@v-prasadboke
Copy link
Contributor

Hello @Cv-securityIQ, Please try to resolve Code QL errors

@Cv-securityIQ
Copy link
Contributor Author

Hello @Cv-securityIQ, Please try to resolve Code QL errors

System.InvalidOperationException : GitHub App ID, Installation ID, or Private Key is missing.
Stack Trace:

@v-prasadboke It is failing with this error. I dont think this is related to the code changes ?

@v-prasadboke v-prasadboke merged commit f243b91 into Azure:master Nov 27, 2024
45 of 46 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Connector Connector specialty review needed Solution Solution specialty review needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants