Shotaro Kohama

Results 30 comments of Shotaro Kohama

I'm also interested in this best practice. Although we have an internal terraform module to manage on-call rotation, what I'm implementing to support removing/adding members are * Add a new...

Hi @ScottSuarez, could you please review this PR?

Thank you for your comments! I didn't realize the large number of diffs. I'll update the template file to minimize the diffs. Once I update the code, I'll get back...

Hi @ScottSuarez, I have updated code baed on your feedback. When you have time, could you review the changes? Thanks

It looks like the two tests also fail in the other PRs regardless of the PR contents. BTW, I can't see the [build log](https://storage.cloud.google.com/ci-vcr-logs/beta/refs/heads/auto-pr-6565/artifacts/c9e9a96c-47f4-40f8-aef3-eb230bac6a32/build-log/recording_test.log) or the [debug log](https://console.cloud.google.com/storage/browser/ci-vcr-logs/beta/refs/heads/auto-pr-6565/artifacts/c9e9a96c-47f4-40f8-aef3-eb230bac6a32/recording) either due...

Hi @slevenick, could you please review this PR?

@slevenick Thank you for your comment! I found out the following things: * `monitoringInterval` is deprecated in v1beta1 like below * `stalenessDays` and `monitoringIntervalDays` are supported in both [v1](https://cloud.google.com/vertex-ai/docs/reference/rest/v1/projects.locations.featurestores.entityTypes#FeaturestoreMonitoringConfig) and...

As for the error messages, I'm afraid I couldn't find the visible link to my account. Could you please give me the link to see the error? The following links...

> Maybe we can mark that field as min_version: beta to prevent it from being in the GA provider, and then deprecate it so we can remove it in the...

Hello @slevenick, I removed `monitoring_interval` from the example, but it seems `TestAccVertexAIFeaturestoreEntitytype_vertexAiFeaturestoreEntitytypeExample` still is failing. Could you help me what the error message is? I'll also try to reproduce the...