-
Notifications
You must be signed in to change notification settings - Fork 23.7k
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
adding plan and container settings to function app #54830
Conversation
The test
The test
The test
|
The test
The test
The test
|
The test
|
plan: | ||
description: | ||
- App service plan. Required for creation. | ||
- It can be name of existing app service plan in same resource group as web app. |
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.
as web app
sounds confusing..
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.
yes, actually it's not required. if not specified, something will be created anyway by the service.
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.
updated the doc
- name: "Create azure function app {{ function_app }}" | ||
azure_rm_functionapp: | ||
resource_group: "{{ resource_group }}" | ||
name: af{{ fixed_resource_prefix }}container |
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.
can you make it a variable?
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.
done
ci failed |
@yungezz I think I have just fixed an issue with azure_rm_appserviceplan. hope now ci will succeed |
noooooooooo..... still something wrong with app service plan :-( |
|
||
if isinstance(poller, AzureOperationPoller): | ||
response = self.get_poller_result(poller) | ||
if isinstance(response, LROPoller) or isinstance(response, AzureOperationPoller): |
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.
let's standardize the get poller operation. how about direct get_poller_result?
SUMMARY
These features are necessary to create container based function apps (like through the portal)
ISSUE TYPE
COMPONENT NAME
azure_rm_functionapp
ADDITIONAL INFORMATION