-
Notifications
You must be signed in to change notification settings - Fork 156
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 Google Cloud Platform integration #184
Conversation
integrations.go
Outdated
return list, nil | ||
} | ||
|
||
// CreateIntegrationGCP creates new Google Cloud Platform Integrations. |
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.
How about making that singular instead of plural? ie: "creates a new Google Cloud Platform Integration"
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.
Fixed it!
Comments for UpdateIntegrationGCP
and DeleteIntegrationGCP
had same issue, fixed them as well.
integrations.go
Outdated
TokenURI *string `json:"token_uri"` // Should be https://accounts.google.com/o/oauth2/token | ||
AuthProviderX509CertURL *string `json:"auth_provider_x509_cert_url"` // Should be https://www.googleapis.com/oauth2/v1/certs | ||
ClientX509CertURL *string `json:"client_x509_cert_url"` // https://www.googleapis.com/robot/v1/metadata/x509/<CLIENT_EMAIL> | ||
HostFilters *string `json:"host_filters"` |
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.
HostFilters
is optional, json:"host_filters,omitempty"
seems appropriate.
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.
Added omitempty
!
integrations.go
Outdated
type IntegrationGCPUpdateRequest struct { | ||
ProjectID *string `json:"project_id"` | ||
ClientEmail *string `json:"client_email"` | ||
HostFilters *string `json:"host_filters"` |
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.
json:"host_filters,omitempty"
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.
Added omitempty
as well.
23897b7
to
33e5be3
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.
LGTM 👍
Thanks @dragon3! Would you like me to cut a release for this? |
@ojongerius Thank you for merging! |
No worries at all, cut https:/zorkian/go-datadog-api/releases/tag/v2.17.0 |
This pull request adds Google Cloud Platform Integration. ( #183 )
In addition, fix a compile error in
integration/screen_widgets_test.go
.