-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
This updates the filter rules for ARM connections in task inputs #10315
Conversation
211c882
to
e7e95bd
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.
Can you also update the default filtering rule at Connected Service Component and ARM Service Component Utility. Otherwise all task will list down AzureMLWorkspace based endpoints too.
@@ -72,7 +72,7 @@ | |||
"helpMarkDown": "Azure Resource Manager subscription to configure before running PowerShell", | |||
"visibleRule": "ConnectedServiceNameSelector = ConnectedServiceNameARM", | |||
"properties": { | |||
"EndpointFilterRule": "ScopeLevel == ManagementGroup || ScopeLevel != ManagementGroup" |
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 didn't quite get what this means? This will always be true.
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.
@kmkumaran @asranja IIRC, this condition will always be true.
@@ -72,7 +72,7 @@ | |||
"helpMarkDown": "Azure Resource Manager subscription to configure before running PowerShell", | |||
"visibleRule": "ConnectedServiceNameSelector = ConnectedServiceNameARM", | |||
"properties": { | |||
"EndpointFilterRule": "ScopeLevel == ManagementGroup || ScopeLevel != ManagementGroup" | |||
"EndpointFilterRule": "ScopeLevel != AzureMLWorkspace" |
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.
Should we look at all usages of https://github.com/Microsoft/azure-pipelines-tasks/search?l=JSON&q=azurerm
No description provided.