Skip to content
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

[RDS]: fix keep_days update for opentelekomcloud_rds_instance_v3 #2286

Merged
merged 2 commits into from
Aug 28, 2023

Conversation

artem-lifshits
Copy link
Member

@artem-lifshits artem-lifshits commented Aug 28, 2023

Summary of the Pull Request

Fix keep_days update when changing from 1 to 0 value.

PR Checklist

Acceptance Steps Performed

=== RUN   TestAccRdsInstanceV3Basic
--- PASS: TestAccRdsInstanceV3Basic (762.16s)
=== RUN   TestAccRdsPostgre13V3ParamsBasic
--- PASS: TestAccRdsPostgre13V3ParamsBasic (533.63s)
=== RUN   TestAccRdsInstanceV3ElasticIP
--- PASS: TestAccRdsInstanceV3ElasticIP (761.48s)
=== RUN   TestAccRdsInstanceV3OptionalParams
--- PASS: TestAccRdsInstanceV3OptionalParams (394.23s)
=== RUN   TestAccRdsInstanceV3Backup
--- PASS: TestAccRdsInstanceV3Backup (359.66s)
PASS

Process finished with exit code 0

@artem-lifshits artem-lifshits self-assigned this Aug 28, 2023
@artem-lifshits artem-lifshits marked this pull request as ready for review August 28, 2023 12:30
@artem-lifshits artem-lifshits added the gate Merge PR label Aug 28, 2023
@otc-zuul
Copy link

otc-zuul bot commented Aug 28, 2023

Build succeeded.
https://zuul.otc-service.com/t/eco/buildset/552ffb88da49400fbc80aba058740139

✔️ build-otc-releasenotes SUCCESS in 3m 14s
✔️ otc-golangci-lint SUCCESS in 3m 14s
✔️ golang-make-test SUCCESS in 4m 10s
✔️ golang-make-vet SUCCESS in 3m 27s
✔️ tflint SUCCESS in 3m 03s
✔️ goreleaser-build SUCCESS in 7m 08s

@otc-zuul otc-zuul bot merged commit 25a9538 into devel Aug 28, 2023
@otc-zuul otc-zuul bot deleted the rds_backup branch August 28, 2023 13:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
gate Merge PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants