-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
feat(monitoring): add monitoring.v3.InternalChecker.state
; add monitoring.v3.UptimeCheckConfig.ContentMatcher.ContentMatcherOption
; add recursive
parameter to delete_group
; add read-only validity
field to monitoring.v3.AlertPolicy
; add validate_ssl
parameter to monitoring.v3.UptimeCheckConfig.HttpCheck
#9546
Conversation
…client library generator.
@busunkim96 This is once again hard to evaluate without any context for the change to the protos. |
@@ -397,6 +397,31 @@ class ConditionCombinerType(enum.IntEnum): | |||
AND_WITH_MATCHING_RESOURCE = 3 | |||
|
|||
|
|||
class InternalChecker(object): |
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.
Add internal checker
@@ -498,3 +523,33 @@ class VerificationStatus(enum.IntEnum): | |||
VERIFICATION_STATUS_UNSPECIFIED = 0 | |||
UNVERIFIED = 1 | |||
VERIFIED = 2 | |||
|
|||
|
|||
class UptimeCheckConfig(object): |
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.
Add monitoring.v3.UptimeCheckConfig.ContentMatcher.ContentMatcherOption
@@ -618,6 +619,9 @@ def delete_group( | |||
Args: | |||
name (str): The group to delete. The format is | |||
``"projects/{project_id_or_number}/groups/{group_id}"``. | |||
recursive (bool): If this field is true, then the request means to delete a group with all |
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.
Add recursive
option to delete_group
necessary the same as the workspace project. | ||
The GCP project ID where the internal checker lives. Not | ||
necessary the same as the Workspace project. | ||
state: |
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.
Add state
to monitoring.v3.InternalChcker
@@ -1222,6 +1234,10 @@ | |||
if it has not been populated. The field should always be | |||
populated on List and Get operations, unless a field | |||
projection has been specified that strips it out. | |||
validity: |
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.
Add read-only validity
field to monitoring.v3.AlertPolicy
@@ -1136,6 +1274,12 @@ | |||
Entering two separate headers with the same key in a Create | |||
call will cause the first to be overwritten by the second. The | |||
maximum number of headers allowed is 100. | |||
validate_ssl: |
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.
Add validate_ssl
parameter to monitoring.v3.UptimeCheckConfig.HttpCheck
monitoring.v3.InternalChecker.state
; add monitoring.v3.UptimeCheckConfig.ContentMatcher.ContentMatcherOption
; add recursive
parameter to delete_group
; add read-only validity
field to monitoring.v3.AlertPolicy
; add validate_ssl
parameter to monitoring.v3.UptimeCheckConfig.HttpCheck
…itoring.v3.UptimeCheckConfig.ContentMatcher.ContentMatcherOption`; add `recursive` parameter to `delete_group`; add read-only `validity` field to `monitoring.v3.AlertPolicy`; add `validate_ssl` parameter to `monitoring.v3.UptimeCheckConfig.HttpCheck` (#9546) * Add `monitoring.v3.InternalChecker.state` * Add `monitoring.v3.UptimeCheckConfig.ContentMatcher.ContentMatcherOption` * Add `recursive` parameter to `delete_group` * Add read-only `validity` field to `monitoring.v3.AlertPolicy` * Add `validate_ssl` parameter to `monitoring.v3.UptimeCheckConfig.HttpCheck`
This PR was generated using Autosynth. 🌈
Summary of changes:
monitoring.v3.InternalChecker.state
monitoring.v3.UptimeCheckConfig.ContentMatcher.ContentMatcherOption
recursive
parameter todelete_group
validity
field tomonitoring.v3.AlertPolicy
validate_ssl
parameter tomonitoring.v3.UptimeCheckConfig.HttpCheck
Here's the log from Synthtool: