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

Upgrade appservice SDK for Flex Consumption support #4200

Merged
merged 2 commits into from
Jul 2, 2024
Merged

Conversation

nturinski
Copy link
Member

@nturinski nturinski commented Jun 28, 2024

Swapped out my generated requests to use the SDK for flex consumption creation. Note that some things, such as runtimes and locations still use generic requests, but that's due to the fact those are not part of the appservice sdk.

Also fixed a perf bug that caused loading to take forever, especially if we had an error retrieving a function app.

Note that this will fix the error that Ramya was encountering with loading her function apps though the ones that were failing will still continue to error. I verified that those apps don't show up at all in the portal, so those resources must be in a weird state.

@nturinski nturinski requested a review from a team as a code owner June 28, 2024 17:59
@nturinski
Copy link
Member Author

Uploading functions_loading.gif…

@alexweininger
Copy link
Member

What was this
image

@nturinski
Copy link
Member Author

What was this image

Oh, it was a gif showing the Functions loading faster and also Ramya's error being resolved.
functions_loading

@nturinski nturinski merged commit c556567 into main Jul 2, 2024
1 check passed
@nturinski nturinski deleted the nat/upgradeSdk branch July 2, 2024 16:45
@microsoft microsoft locked and limited conversation to collaborators Aug 17, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants