-
Notifications
You must be signed in to change notification settings - Fork 4.6k
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
Upgrade managed cassandra API version from 2022-05-15 to 2023-04-15 #23110
Upgrade managed cassandra API version from 2022-05-15 to 2023-04-15 #23110
Conversation
neil-yechenwei
commented
Aug 29, 2023
•
edited by tombuildsstuff
Loading
edited by tombuildsstuff
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.
Thanks for this PR - I've taken a look through and left a few comments inline, but this is mostly looking good - if we can fix those up then we should be able to take another look and get this merged 👍
Thanks!
internal/services/cosmos/cosmosdb_cassandra_datacenter_resource.go
Outdated
Show resolved
Hide resolved
internal/services/cosmos/cosmosdb_cassandra_datacenter_resource.go
Outdated
Show resolved
Hide resolved
internal/services/cosmos/cosmosdb_cassandra_datacenter_resource.go
Outdated
Show resolved
Hide resolved
internal/services/cosmos/cosmosdb_cassandra_datacenter_resource.go
Outdated
Show resolved
Hide resolved
@tombuildsstuff , thanks for the comment. I updated PR. Please take another look. Thanks. |
internal/services/cosmos/cosmosdb_cassandra_datacenter_resource.go
Outdated
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.
LGTM 👍
Pretty sure I approved this earlier, but let's re-approve this 🙃 |
I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active contributions. |