-
Notifications
You must be signed in to change notification settings - Fork 1
feat: Update clu2adv command so it doesn't produce deprecated attributes #66
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
Conversation
…ern in advanced_configuration
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.
Pull Request Overview
Updates the clu2adv and adv2v2 commands to remove deprecated MongoDB Atlas attributes from generated Terraform configurations, improving compatibility and preventing the output of attributes that are no longer supported.
- Removes
use_replication_spec_per_shard
from data source configurations - Filters out deprecated
fail_index_key_too_long
anddefault_read_concern
attributes from advanced_configuration blocks - Introduces a specialized function to handle advanced configuration processing with deprecation filtering
Reviewed Changes
Copilot reviewed 7 out of 7 changed files in this pull request and generated 1 comment.
Show a summary per file
File | Description |
---|---|
internal/convert/const.go | Adds constants for deprecated attributes and removes unused constant |
internal/convert/clu2adv.go | Implements advanced config filtering and removes deprecated data source attribute |
internal/convert/adv2v2.go | Updates to use new advanced config filtering function |
internal/convert/testdata/clu2adv/dynamic_regions_config_auto_scaling.in.tf | Test data showing deprecated attributes to be filtered |
internal/convert/testdata/clu2adv/data_sources.out.tf | Expected output without deprecated data source attribute |
internal/convert/testdata/clu2adv/adv_config_bi_connector_pinned_fcv.in.tf | Test data with deprecated advanced config attributes |
internal/convert/testdata/adv2v2/adv_config_bi_connector_pinned_fcv.in.tf | Test data with deprecated advanced config attributes for adv2v2 conversion |
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
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
Description
Update clu2adv command so it doesn't produce deprecated attributes.
Link to any related issue(s): CLOUDP-339568
Type of change:
Required Checklist:
Further comments