-
Notifications
You must be signed in to change notification settings - Fork 207
refactor: Uses createOnlyWithDefault for delete_on_create_timeout in flexcluster #3625
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
Closed
Closed
Changes from all commits
Commits
Show all changes
17 commits
Select commit
Hold shift + click to select a range
978dd91
refactor: Use createOnly with default for delete_on_create_timeout fo…
EspenAlbert 5f79c13
feat: Adds support for `role_id` in `google_cloud_kms_config` on `mon…
oarbusi b31e7c2
chore: Updates CHANGELOG.md for #3636
svc-apix-Bot 995e614
feat: Supports GCP for `mongodbatlas_cloud_provider_access_authorizat…
oarbusi 3ec1bc6
chore: Updates CHANGELOG.md for #3639
svc-apix-Bot ed27cba
feat: Adds support for `service_account_for_atlas` and `status` field…
marcabreracast eeee5db
chore: Updates CHANGELOG.md for #3637
svc-apix-Bot b56c808
chore: Bump amannn/action-semantic-pull-request from 6.1.0 to 6.1.1 (…
dependabot[bot] 394cf89
chore: Bump github.com/pb33f/libopenapi from 0.25.3 to 0.25.6 (#3621)
dependabot[bot] a16827e
chore: Update Go to 1.25.0 (#3641)
lantoli 308f682
test: Ensures project `withDefaultAlertsSettings` works with import a…
EspenAlbert b0cd21c
Merge branch 'master' into create-only-with-default
EspenAlbert 221ad5e
refactor: Remove references to old plan modifier
EspenAlbert 6eb3789
Merge branch 'CLOUDP-320243-dev-2.0.0' into create-only-with-default
EspenAlbert 4b184dd
test: Add ImportStateVerifyIgnore for delete_on_create_timeout in bas…
EspenAlbert 31d1ecf
Merge branch 'CLOUDP-320243-dev-2.0.0' into create-only-with-default
EspenAlbert bd92500
Update docs/resources/project.md
EspenAlbert File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
```release-note:enhancement | ||
resource/mongodbatlas_encryption_at_rest: Supports role_id in google_cloud_kms_config | ||
``` | ||
|
||
```release-note:enhancement | ||
data-source/mongodbatlas_encryption_at_rest: Supports role_id in google_cloud_kms_config | ||
``` |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
```release-note:enhancement | ||
resource/mongodbatlas_cloud_provider_access_setup: Adds support for GCP as a Cloud Provider. | ||
``` | ||
|
||
```release-note:enhancement | ||
data-source/mongodbatlas_cloud_provider_access_setup: Adds support for GCP as a Cloud Provider. | ||
``` |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
```release-note:enhancement | ||
resource/mongodbatlas_cloud_provider_access_authorization: Supports GCP cloud provider | ||
``` |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
@EspenAlbert could we first trigger https://github.com/mongodb/terraform-provider-mongodbatlas/actions/workflows/update-dev-branches.yml so that all the changes you need from master are already present in the dev branch? Or alternatively provide the isolated commits for the changes associated to flex cluster
Uh oh!
There was an error while loading. Please reload this page.
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.
Yeah, it got a bit messy since I needed to merge with master.
Running the update manually will create a merge conflict that would require updates in other files... So it is a bit complicated :/
Changes to flex cluster is in this commit: 978dd91
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.
I can also recommend checking the changes to
create_only.go
to compare the plan modifiersThere 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.
my concern with this is that if for whatever reason we need to revert the createOnlyWithDefault, we would also revert some unrelated pieces of code. What is the complication in merging master in 2.0.0 in the automated PR and open a new PR on top of that in 2.0.0 with your
create_only
changes?