-
Notifications
You must be signed in to change notification settings - Fork 49
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
Add Azure integration #65
Add Azure integration #65
Conversation
…-azure-integration-powered-by-ocean
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
…-azure-integration-powered-by-ocean
integrations/azure/.port/spec.yaml
Outdated
- kind: resource_group | ||
- kind: container_app | ||
- kind: aks |
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.
You don't really know all the possibilities
So i think it will be better to delete this field
current_resource_kind, is_last_level = get_resource_kind_by_level( | ||
full_resource_kind, kind_level | ||
) | ||
current_resource_kind_suffix = current_resource_kind.split("/")[-1] |
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.
_*, var_name = iter
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.
I think it is more readable with -1
, but if it is a convention you want to have let me know and i'll update
…-azure-integration-powered-by-ocean
….com:port-labs/Port-Ocean into PORT-4351-azure-integration-powered-by-ocean
Description
Port 4351
What - Add azure integration as well as default deployment method using terraform over azure
Why -
How -
Microsoft.Storage/storageAccounts/blobServices/containers
which is an extension resource ofMicrosoft.Storage/storageAccounts
.Type of change
Please leave one option from the following and delete the rest:
TODOs