-
Notifications
You must be signed in to change notification settings - Fork 161
feat: Deploy function app code from blob storage URL #231
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a few questions regarding the order of operations with deployment / uploading artifacts, etc.
2bba607
to
fe9ae05
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Added a question about reusing SDK reference - otherwise looks good.
fe9ae05
to
fd762b4
Compare
@@ -52,6 +52,7 @@ describe("Login Plugin", () => { | |||
}); | |||
|
|||
it("calls login if azure credentials are not set", async () => { | |||
unsetServicePrincipalEnvVariables(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This test was failing for some reason... Not sure how it got through the checks
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
huh, that's weird, I thought it had been added in the tests, good thing they work now then
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
@@ -52,6 +52,7 @@ describe("Login Plugin", () => { | |||
}); | |||
|
|||
it("calls login if azure credentials are not set", async () => { | |||
unsetServicePrincipalEnvVariables(); |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
huh, that's weird, I thought it had been added in the tests, good thing they work now then
- [x] Allow configuration to specify whether or not to run from external package - [x] Set function app setting `WEBSITE_RUN_FROM_PACKAGE` with SAS URL of package from blob storage - [x] Get deployment configuration from `config.provider` - [x] Upload to blob storage before deployment if `runFromBlobUrl` is specified AB#488
WEBSITE_RUN_FROM_PACKAGE
with SAS URL of package from blob storageconfig.provider
runFromBlobUrl
is specified