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
Spotaut 18392 - Implementation for Verification Provider OceanCD #539
Conversation
|
||
output, err := spotinstClient.oceancd.ReadVerificationProvider(ctx, input) | ||
if err != nil { | ||
// If the cluster was not found, return nil so that we can show that it |
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.
please modify the message
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.
Corrected
} | ||
|
||
// Some other error, report it. | ||
return nil, fmt.Errorf("ocean/aks: failed to read verification provider: %v", err) |
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.
modify the error message
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.
modified
spotinst/oceancd_verification_provider/fields_spotinst_oceancd_verification_provider.go
Outdated
Show resolved
Hide resolved
...rification_provider_cloud_watch/fields_spotinst_oceancd_verification_provider_cloud_watch.go
Outdated
Show resolved
Hide resolved
...rification_provider_cloud_watch/fields_spotinst_oceancd_verification_provider_cloud_watch.go
Outdated
Show resolved
Hide resolved
...eancd_verification_provider_datadog/fields_spotinst_oceancd_verification_provider_datadog.go
Show resolved
Hide resolved
...eancd_verification_provider_jenkins/fields_spotinst_oceancd_verification_provider_jenkins.go
Show resolved
Hide resolved
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.
please fix the review comments and merge the latest
go.mod
Outdated
@@ -1,6 +1,6 @@ | |||
module github.com/spotinst/terraform-provider-spotinst | |||
|
|||
go 1.20 | |||
go 1.18 |
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.
please change back this to 1.20
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.
changed
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.
please fix the review comments and run make fmt on the branch in local
This is the default pull request template. You can customize it by adding a
pull_request_template.md
at the root of your repo or inside the.github
folder.Jira Ticket
Include a link to your Jira Ticket
Example: JIRAISS-1234
Demo
Please add a recording of the feature/bug fix in work. if you added new routes, the recording should show the request and response for each new/changed route
Checklist: