Shotaro Kohama
Shotaro Kohama
Thank you for letting me know about the error! The `stalenessDays` and `monitoringIntervalDays` are the other fields, but they're new fields for the provider. The PR becomes not only GA....
Thank you for your comment. I tried reproducing the error with [the auto-generated branch](https://github.com/modular-magician/terraform-provider-google/commit/b769b21056499b22faf81f5f0bf14a1d74b20042) on my local machine like below, but the test passed on my machine. So I might...
> It's failing on the beta provider specifically, so maybe it's a beta vs GA difference? Ah, I could reproduce the error with the beta provider locally. I'll look into...
Hi @slevenick I added the default value of the monitoring_interval and I confirmed the tests passed. Could you review the change when you have time? Thanks!
@rileykarson Hello 👋 Could you approve the cloud build jobs to run acceptance tests?
@rileykarson Hello 👋 Sorry for the late response. I updated the code based on your feedback. According to the API doc, all fields are immutable. Therefore, I didn't add a...
@rileykarson Happy New Year 👋 When you have time, could you review this PR? Thanks
@rileykarson Hello 👋 When you have time, could you review this PR? Thanks!
@rileykarson Hello 👋 If this PR looks good, could you approve the cloud build job? Thanks
@rileykarson Hello 👋 When you have time, could you review the code? Please let me know if merging the latest main branch into this development branch is necessary. Thanks!