Skip to content

doc: Merges CLOUDP-274025 docs & examples branch - DONT MERGE YET #3079

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 111 commits into from
Mar 11, 2025
Merged

doc: Merges CLOUDP-274025 docs & examples branch - DONT MERGE YET #3079

merged 111 commits into from
Mar 11, 2025

Conversation

lantoli
Copy link
Member

@lantoli lantoli commented Feb 17, 2025 •
edited
Loading

Description

Merges CLOUDP-274025 docs & examples branch.

DONT MERGE YET

Type of change:

  • Bug fix (non-breaking change which fixes an issue). Please, add the "bug" label to the PR.
  • New feature (non-breaking change which adds functionality). Please, add the "enhancement" label to the PR. A migration guide must be created or updated if the new feature will go in a major version.
  • Breaking change (fix or feature that would cause existing functionality to not work as expected). Please, add the "breaking change" label to the PR. A migration guide must be created or updated.
  • This change requires a documentation update
  • Documentation fix/enhancement

Required Checklist:

  • I have signed the MongoDB CLA
  • I have read the contributing guides
  • I have checked that this change does not generate any credentials and that they are NOT accidentally logged anywhere.
  • I have added tests that prove my fix is effective or that my feature works per HashiCorp requirements
  • I have added any necessary documentation (if appropriate)
  • I have run make fmt and formatted my code
  • If changes include deprecations or removals I have added appropriate changelog entries.
  • If changes include removal or addition of 3rd party GitHub actions, I updated our internal document. Reach out to the APIx Integration slack channel to get access to the internal document.

Further comments

lantoli and others added 15 commits February 10, 2025 12:32
@lantoli
duplicate files (#3046)
7458ec7
@EspenAlbert @lantoli
doc: Adds initial structure for module examples cluster_to_advanced_c…
6ada45d 
…luster (#3043)

* doc: Adds initial structure for module example cluster_to_advanced_cluster

* chore: fix lint errors

* refactor: directory move

* doc: Initial module_user

* doc: Remove hcl simplification purpose (most likely done only in module_maintainer)

* Update examples/migrate_cluster_to_advanced_cluster/module_maintainer/README.md

Co-authored-by: Leo Antoli <[email protected]>

* address PR comments

---------

Co-authored-by: Leo Antoli <[email protected]>
@lantoli
duplicate files (#3052)
c1f9a08
@lantoli
doc: Examples for advanced_cluster preview v2 (#3054)
c64b251 
* root readme

* fix example as used region doesn't exist in GCP

* change examples

* undo comment

* doc for backing_provider_name
@lantoli @carriecwk
doc: Create doc pages for mongodb_advanced_cluster schema v2 resour…
2166635 
…ce and data sources (#3048)

* add doc references at the beginning of the doc

* examples

* replace replication_specs.#

* leftover

* connection_strings

* Move and other changes

* rename schema to preview

* ds

* fix defaultMaxTimeMS links

* fix typo region_config

* Update docs/data-sources/advanced_clusters (preview v2).md

Co-authored-by: carriecwk <[email protected]>

* Update docs/resources/advanced_cluster.md

Co-authored-by: carriecwk <[email protected]>

* apply feedback

---------

Co-authored-by: carriecwk <[email protected]>
@lantoli
chore: Follow-up for PR 3054 (#3059)
dd68c88 
* null default value

* org id
@lantoli
Merge branch 'master' into CLOUDP-274025-dev-docs-examples
1641a4b 
* master:
  doc: Renames env var to `MONGODB_ATLAS_PREVIEW_PROVIDER_V2_ADVANCED_CLUSTER` (#3062)
  chore: Updates Atlas Go SDK (#3053)
  test: Tenant clusters with disk_size_gb is always 5 now (#3057)
  chore: Updates CHANGELOG.md for #3051
  fix: Removes `interval_min` plan modifier to avoid error on update (#3051)
  build(deps): bump go.mongodb.org/atlas-sdk (#3049)
  doc: Adds pinned_fcv.version as exported attribute in cluster and advanced cluster resource (#3047)
  update Go to 1.23.6 (#3045)
  chore: Adds HTTP status check helper methods (#3035)
  chore: Adds import checks to tests (#3039)
  chore: Updates CHANGELOG.md header for v1.26.1 release
  chore: Updates examples link in index.md for v1.26.1 release
  fix: Adds "state_name" to keepUnknown list to prevent copying volatile attributes from state (#3040)
@lantoli
doc: Rename env. var. to `MONGODB_ATLAS_PREVIEW_PROVIDER_V2_ADVANCED_…
ac891f5 
…CLUSTER` (#3064)

* rename env. var. to MONGODB_ATLAS_PREVIEW_PROVIDER_V2_ADVANCED_CLUSTER

* change example doc

* update links and menu items

* fix links

* change style
@EspenAlbert
doc: Implements v2 and v3 MongoDB Atlas cluster migration examples (#…
d2f0b0f 
…3050)

* initial v2 & v3 implementations

* Adds `v2` Implementation Changes and Highlights

* doc: Update README with v3 implementation changes and highlights

* doc: Update README and add v1_v2 and v3 variable files for cluster migration examples

* style: fmt

* doc: Fix v3 usage of cluster data source

* feat: Add output for MongoDB connection strings in v1, v2, and v3 examples

* style: fmt

* chore: Enable Advanced Cluster V2 Schema validation in tf-validate script

* fix lint errors

* feat: Add function to check for V2 schema directories in tf-validate script

* refactor: Use approach 1 (file headings) rather than a table

* address PR comments

* docs: Update README files to enhance migration guidance for mongodbatlas_cluster to mongodbatlas_advanced_cluster

* address PR comments

* chore: Update ToC

* Review suggestions

* update var name

* docs: Enhance README to clarify standalone usage and upgrade support for module_maintainer

* fix: Update required_version comment for clarity in versions.tf files

* docs: Add v3_no_plan_changes and step for ensuring no changes when using new `replication_specs_new` variable

* update var name to make tf-validate pass

* fix: Update environment variable for Terraform validation in v2 schema

* docs: Update ToC

* fix: update restore command in tf-validate script to use preview_provider_v2.go
@EspenAlbert
doc: Adds migration example from mongodbatlas_cluster to `mongodbat…
b6c1843 
…las_advanced_cluster` (#3058)

* doc: Adds migration example from `mongodbatlas_cluster` to `mongodbatlas_advanced_cluster`

* rename directory and example

* chore: use a Note to warn about creating a `mongodbatlas_cluster` and more details

* add review suggestions

* support using v2 schema in basic
@lantoli
doc: Updates guide cluster to advanced_cluster and creates guide adva…
ced477c 
…nced_cluster preview provider v2 (#3071)

* new guide

* update docs

* some adjustements

* typos

* feedback

* best practices feedback

* fix identation
@EspenAlbert
doc: Adds remaining work to migrate_cluster_to_advanced_cluster examp…
fb99be0 
…les (#3065)

* handle todos

* doc: Fix link in README for replication_specs in advanced cluster migration example

* doc: Enhance variable descriptions in migration examples

* doc: Update descriptions for outputs

* doc: Change regions_config type from set to list in replication_specs variable (otherwise priority order failure)

* doc: Change regions_config type from set to list in replication_specs variable (otherwise priority order failure)
@lantoli
doc: Applies some feedback for migration guides doc (#3075)
4101d35 
* remove part about state

* add doc about using the env var locally
@EspenAlbert @lantoli @marcosuma
doc: Updates table formatting and migration guides (#3076)
a2a82fc 
* doc: Updates table formatting

* clarify how to migrate

* change order

* move block section down

* apply feedback

* Update docs/guides/cluster-to-advanced-cluster-migration-guide.md

Co-authored-by: Marco Suma <[email protected]>

---------

Co-authored-by: Leo Antoli <[email protected]>
Co-authored-by: Marco Suma <[email protected]>
@lantoli
Merge branch 'master' into CLOUDP-274025-dev-docs-examples
f5841fa 
* master:
  chore: Temporary fix on tenant cluster test passing zone name explicitly (#3078)
  exempt dependencies PR labels from being stale (#3073)
  use helper methods (#3072)
  chore: Bump golangci/golangci-lint-action from 6.3.0 to 6.5.0 (#3074)
  chore: Bump goreleaser/goreleaser-action from 6.1.0 to 6.2.1 (#3056)
@lantoli lantoli marked this pull request as ready for review February 17, 2025 18:51
@lantoli lantoli requested review from a team as code owners February 17, 2025 18:51
@lantoli lantoli changed the title doc: Merges CLOUDP-274025 docs & examples branch doc: Merges CLOUDP-274025 docs & examples branch - DONT MERGE YET Feb 17, 2025
@github-actions GitHub Actions
Copy link
Contributor

APIx bot: a message has been sent to Docs Slack channel

@lantoli lantoli mentioned this pull request Feb 18, 2025
Merged
13 tasks
marcosuma
marcosuma reviewed Feb 18, 2025
View reviewed changes
themantissa
themantissa approved these changes Mar 10, 2025
View reviewed changes
Copy link
Collaborator

@themantissa themantissa left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks like all was addressed! Thank you team!!

@lantoli lantoli merged commit 015ba58 into master Mar 11, 2025
39 checks passed
@lantoli lantoli mentioned this pull request Mar 27, 2025
Merged
13 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@EspenAlbert EspenAlbert EspenAlbert left review comments

@marcosuma marcosuma marcosuma left review comments

@AgustinBettati AgustinBettati AgustinBettati left review comments

@oarbusi oarbusi oarbusi left review comments

@maastha maastha maastha left review comments

@themantissa themantissa themantissa approved these changes

@jwilliams-mongo jwilliams-mongo jwilliams-mongo approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
8 participants
@lantoli @themantissa @EspenAlbert @marcosuma @AgustinBettati @jwilliams-mongo @oarbusi @maastha