-
Notifications
You must be signed in to change notification settings - Fork 1.4k
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
Support ephemeral_storage_local_ssd_config #7483
Support ephemeral_storage_local_ssd_config #7483
Conversation
Oops! It looks like you're using an unknown release-note type in your changelog entries:
Please only use the types listed in https://github.com/GoogleCloudPlatform/magic-modules/blob/master/.ci/RELEASE_NOTES_GUIDE.md. |
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.
See hashicorp/terraform-provider-google#13829, hashicorp/terraform-provider-google#8461 - both of these are supported.
One of the two is only at beta though, so you may want to GA it if it's available in the GA API now.
ephemeral_storage_local_ssd_config is different from ephemeral_storage_config. I'll remove local_nvme_ssd_block_config, thanks for reminding. |
Hmm- I see in the docs that "This field is functionally equivalent to the ephemeralStorageConfig" on |
We won't graduate ephemeralStorageConfig from beta to GA. ephemeralStorageLocalSsdConfig is the GA version for the feature. |
Got it! We likely want to remap |
ok, so I should replace |
Ah! No, we'd keep the field name |
It will be a breaking change for the users who are using Can we leave the |
Oops! It looks like you're using an unknown release-note type in your changelog entries:
Please only use the types listed in https://github.com/GoogleCloudPlatform/magic-modules/blob/master/.ci/RELEASE_NOTES_GUIDE.md. |
cb30a0e
to
7c826c5
Compare
Oops! It looks like you're using an unknown release-note type in your changelog entries:
Please only use the types listed in https://github.com/GoogleCloudPlatform/magic-modules/blob/master/.ci/RELEASE_NOTES_GUIDE.md. |
/gcbrun |
Hi there, I'm the Modular magician. I've detected the following information about your changes: Diff reportYour PR generated some diffs in downstreams - here they are. Terraform GA: Diff ( 4 files changed, 130 insertions(+), 28 deletions(-)) |
Tests analyticsTotal tests: Action takenFound 10 affected test(s) by replaying old test recordings. Starting RECORDING based on the most recent commit. Click here to see the affected testsTestAccFrameworkProviderMeta_setModuleName|TestAccFrameworkProviderBasePath_setBasePath|TestAccContainerNodePool_ephemeralStorageLocalSsdConfig|TestAccDataSourceGoogleFirebaseAppleAppConfig|TestAccComputeInstanceFromRegionTemplate_basic|TestAccDataSourceDnsManagedZone_basic|TestAccDataSourceDNSKeys_noDnsSec|TestAccDataSourceDNSKeys_basic|TestAccDataSourceDnsRecordSet_basic|TestAccComposerEnvironment_withEncryptionConfigComposer2 |
Tests passed during RECORDING mode: All tests passed |
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.
Discussed offline, the new field replaces the old beta-only one but they're incompatible w/ no backfill. Given that, we can't remap the field and introducing the new field is correct.
We may want to mark ephemeral_storage_config as deprecated to push users to use the new field instead. We'll need to outline how to migrate between them in the deprecation message- that's going to be recreating the affected resources, given both fields are immutable.
Terraform deprecations don't fall under GCP policies for this kind of stuff. The ecosystem expectation is that we document the path end users take to resolve the deprecation is all. In the next major release, deprecated fields can be removed.
mmv1/third_party/terraform/website/docs/r/container_cluster.html.markdown
Show resolved
Hide resolved
…_ssd_block_config
Co-authored-by: Shuya Ma <87669292+shuyama1@users.noreply.github.com>
394356e
to
5067e41
Compare
Support for GKE ephemeral storage local ssd and local nvme ssd block
fix: hashicorp/terraform-provider-google#14021
If this PR is for Terraform, I acknowledge that I have:
make test
andmake lint
in the generated providers to ensure it passes unit and linter tests.Release Note Template for Downstream PRs (will be copied)