-
Notifications
You must be signed in to change notification settings - Fork 674
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
Added support for IBM Cloud Logs atracker targets #5270
Conversation
@@ -156,6 +156,20 @@ func DataSourceIBMAtrackerTargets() *schema.Resource { | |||
}, | |||
}, | |||
}, | |||
"cloudlogs_endpoint": &schema.Schema{ |
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 we rename the attirbute to
cloud_logs_endpoint
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.
We were keeping the same convention as previous changes. Collapsing the service name and adjoining it with _endpoint
. Similarly we support event streams and that endpoint is eventstreams_endpoint
also.
Resolce conflicts in go.mod and go.sum files |
go.mod
Outdated
github.com/IBM/platform-services-go-sdk v0.62.0 | ||
github.com/IBM/project-go-sdk v0.2.8-1 | ||
github.com/IBM/platform-services-go-sdk v0.62.3 | ||
github.com/IBM/project-go-sdk v0.2.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.
we are changing github.com/IBM/project-go-sdk verison value
This change is breaking the go build
https://github.com/IBM-Cloud/terraform-provider-ibm/actions/runs/8690624159/job/23917984722?pr=5270
What has been done?
cloud_logs
) to be able to send Atracker events to a IBM Cloud Logs instance.Community Note
Relates OR Closes #0000
Output from acceptance testing: