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

Function App Recompilation #11756

Merged
merged 10 commits into from
Feb 12, 2025
Merged

Conversation

daviditkin
Copy link
Contributor

Required items, please complete

Change(s):

  • The main change is the recompilation of the golang function app with proper target
  • Added user parameter to set Schedule for installation of 'Data Connector's' function app in function.json
  • 3.1.1.zip has been updated.
  • bhe-funapp.zip has been updated.

Reason for Change(s):

  • Set proper target on golang function app for execution in linux docker image
  • Allow user to set schedule

Version Updated:

  • None

Testing Completed:

  • Tested

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

  • Validations passed except for "Property: "id" must use one of the following expressions for an resourceId property:" errors, could use help if these are supposed to be fixed.

@daviditkin daviditkin requested review from a team as code owners February 3, 2025 19:40
@v-prasadboke v-prasadboke self-assigned this Feb 4, 2025
@v-prasadboke v-prasadboke added Connector Connector specialty review needed Solution Solution specialty review needed labels Feb 4, 2025
@v-prasadboke
Copy link
Contributor

Hello @daviditkin, Can you please share invocation logs screenshot

@daviditkin
Copy link
Contributor Author

@v-prasadboke here is a screenshot of the function invocation.

Screenshot 2025-02-07 at 9 54 58 AM

image

v-prasadboke
v-prasadboke previously approved these changes Feb 11, 2025
@v-prasadboke v-prasadboke merged commit 12ce962 into Azure:master Feb 12, 2025
31 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.

2 participants