-
Notifications
You must be signed in to change notification settings - Fork 438
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
fix(managedidentities): [ManagedIdentities] add ancillary service bindings to service_yaml #5037
fix(managedidentities): [ManagedIdentities] add ancillary service bindings to service_yaml #5037
Conversation
PiperOrigin-RevId: 422903615 Source-Link: googleapis/googleapis@12c0a1f Source-Link: googleapis/googleapis-gen@d6e18f3 Copy-Tag: eyJwIjoiTWFuYWdlZElkZW50aXRpZXMvLk93bEJvdC55YW1sIiwiaCI6ImQ2ZTE4ZjM2MjU3MjAwNTQ2NTE3N2Y0YTNkNTVlMGUyZjIwZDM5MDIifQ==
@@ -304,19 +435,13 @@ public function __construct(array $options = []) | |||
* | |||
* @experimental | |||
*/ | |||
public function attachTrust(array $optionalArgs = []) | |||
public function attachTrust($name, $trust, array $optionalArgs = []) |
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 breaks BC, but in a beta client, so it's okay
@@ -392,23 +517,14 @@ public function attachTrust(array $optionalArgs = []) | |||
* | |||
* @experimental | |||
*/ | |||
public function createMicrosoftAdDomain(array $optionalArgs = []) | |||
public function createMicrosoftAdDomain($parent, $domainName, $domain, array $optionalArgs = []) |
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 breaks BC, but in a beta client, so it's okay
@@ -469,15 +585,12 @@ public function createMicrosoftAdDomain(array $optionalArgs = []) | |||
* | |||
* @experimental | |||
*/ | |||
public function deleteDomain(array $optionalArgs = []) | |||
public function deleteDomain($name, array $optionalArgs = []) |
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 breaks BC, but in a beta client, so it's okay
@@ -542,19 +655,13 @@ public function deleteDomain(array $optionalArgs = []) | |||
* | |||
* @experimental | |||
*/ | |||
public function detachTrust(array $optionalArgs = []) | |||
public function detachTrust($name, $trust, array $optionalArgs = []) |
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 breaks BC, but in a beta client, so it's okay
@@ -592,15 +699,12 @@ public function detachTrust(array $optionalArgs = []) | |||
* | |||
* @experimental | |||
*/ | |||
public function getDomain(array $optionalArgs = []) | |||
public function getDomain($name, array $optionalArgs = []) |
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 breaks BC, but in a beta client, so it's okay
@@ -667,15 +771,12 @@ public function getDomain(array $optionalArgs = []) | |||
* | |||
* @experimental | |||
*/ | |||
public function listDomains(array $optionalArgs = []) | |||
public function listDomains($parent, array $optionalArgs = []) |
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 breaks BC, but in a beta client, so it's okay
@@ -760,23 +861,14 @@ public function listDomains(array $optionalArgs = []) | |||
* | |||
* @experimental | |||
*/ | |||
public function reconfigureTrust(array $optionalArgs = []) | |||
public function reconfigureTrust($name, $targetDomainName, $targetDnsIpAddresses, array $optionalArgs = []) |
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 breaks BC, but in a beta client, so it's okay
@@ -814,15 +906,12 @@ public function reconfigureTrust(array $optionalArgs = []) | |||
* | |||
* @experimental | |||
*/ | |||
public function resetAdminPassword(array $optionalArgs = []) | |||
public function resetAdminPassword($name, array $optionalArgs = []) |
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 breaks BC, but in a beta client, so it's okay
@@ -892,18 +982,13 @@ public function resetAdminPassword(array $optionalArgs = []) | |||
* | |||
* @experimental | |||
*/ | |||
public function updateDomain(array $optionalArgs = []) | |||
public function updateDomain($updateMask, $domain, array $optionalArgs = []) |
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 breaks BC, but in a beta client, so it's okay
@@ -969,19 +1054,13 @@ public function updateDomain(array $optionalArgs = []) | |||
* | |||
* @experimental | |||
*/ | |||
public function validateTrust(array $optionalArgs = []) | |||
public function validateTrust($name, $trust, array $optionalArgs = []) |
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 breaks BC, but in a beta client, so it's okay
PiperOrigin-RevId: 422903615
Source-Link: googleapis/googleapis@12c0a1f
Source-Link: https://github.com/googleapis/googleapis-gen/commit/d6e18f362572005465177f4a3d55e0e2f20d3902
Copy-Tag: eyJwIjoiTWFuYWdlZElkZW50aXRpZXMvLk93bEJvdC55YW1sIiwiaCI6ImQ2ZTE4ZjM2MjU3MjAwNTQ2NTE3N2Y0YTNkNTVlMGUyZjIwZDM5MDIifQ==