Skip to content
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

INTMDB320 - Fix Global Cluster import documentation #796

Merged
merged 1 commit into from
Aug 2, 2022

Conversation

evertsd
Copy link
Contributor

@evertsd evertsd commented Jul 26, 2022

Description

Ran an example which verified global cluster import works, and updated its documentation in kind

Type of change:

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update
  • Documentation fix/enhancement

Required Checklist:

  • I have signed the MongoDB CLA
  • I have read the Terraform contribution guidelines
  • 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

Further comments

@evertsd evertsd requested a review from themantissa as a code owner July 26, 2022 14:09
@Zuhairahmed Zuhairahmed self-requested a review July 26, 2022 18:59
Copy link
Contributor

@Zuhairahmed Zuhairahmed left a comment

Choose a reason for hiding this comment

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

LGTM

@evertsd evertsd changed the base branch from master to release-staging-v1.4.4 July 26, 2022 21:21
@themantissa themantissa requested a review from a team July 26, 2022 23:18
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.

LGTM, added APIx team for review

@themantissa themantissa requested a review from a team July 27, 2022 00:05
Copy link
Collaborator

@gssbzn gssbzn left a comment

Choose a reason for hiding this comment

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

LGTM

@evertsd evertsd merged commit f8c89d9 into release-staging-v1.4.4 Aug 2, 2022
martinstibbe added a commit that referenced this pull request Aug 16, 2022
* INTMDB-343: Update go from 1.16 to 1.17 add windows arm64 build support (#797)

* INTMDB-343: Update go from 1.16 to 1.17 add windows arm64 build support

* feat: Update to use go1.17

* actually update

Co-authored-by: Gustavo Bazan <[email protected]>

* Setting name on project data_source read (#795)

* INTMDB-331 - Update GCP documentation (#793)

* Updated gcp compute address & forwarding rule examples

* Fixed formatting

* Commenting out credentials for fmting purposes

* Updated import documentation for global_cluster_config (#796)

* INTMDB-355: [Terraform] Customer HELP ticket - hitting panic when trying to set maintenance windows (#800)

* INTMDB-362: Header Clarifications "Resource" vs "Data Source" in Documentation   (#803)

* Update CONTRIBUTING.md (#798)

Clarify use of release branch

* INTMDB-362

Header Clarifications "Resource" vs "Data Source" to be more in line with other providers as well to make easier for user. Documentation update only.

* Revert "Update CONTRIBUTING.md (#798)"

This reverts commit 71b9173.

* Revert "Revert "Update CONTRIBUTING.md (#798)""

This reverts commit 147159c.

Co-authored-by: Melissa Plunkett <[email protected]>

* INTMDB-357: Add exclude to skip go-header until google terraform provider version tag upgrade (#799)

* INTMDB-357: Add exclude to skip go-header until google terraform provider version tag upgrade

* Ran go mod tidy -compat=1.17

* Fix test to use aws provider ctedentials (#807)

* Update CHANGELOG.md

* go mod tidy

* Go lang lint update to v1.41.2

* GOLANGCI_VERSION=v1.42.1

* Try latest

* Fix go.sum

* golangci version

* tidy

* fixes

* fixes

* Update resource_mongodbatlas_database_user_test.go

Co-authored-by: Gustavo Bazan <[email protected]>
Co-authored-by: Dosty Everts <[email protected]>
Co-authored-by: Zuhair Ahmed <[email protected]>
Co-authored-by: Melissa Plunkett <[email protected]>
guineveresaenger pushed a commit to pulumi/terraform-provider-mongodbatlas that referenced this pull request Sep 13, 2022
* INTMDB-343: Update go from 1.16 to 1.17 add windows arm64 build support (mongodb#797)

* INTMDB-343: Update go from 1.16 to 1.17 add windows arm64 build support

* feat: Update to use go1.17

* actually update

Co-authored-by: Gustavo Bazan <[email protected]>

* Setting name on project data_source read (mongodb#795)

* INTMDB-331 - Update GCP documentation (mongodb#793)

* Updated gcp compute address & forwarding rule examples

* Fixed formatting

* Commenting out credentials for fmting purposes

* Updated import documentation for global_cluster_config (mongodb#796)

* INTMDB-355: [Terraform] Customer HELP ticket - hitting panic when trying to set maintenance windows (mongodb#800)

* INTMDB-362: Header Clarifications "Resource" vs "Data Source" in Documentation   (mongodb#803)

* Update CONTRIBUTING.md (mongodb#798)

Clarify use of release branch

* INTMDB-362

Header Clarifications "Resource" vs "Data Source" to be more in line with other providers as well to make easier for user. Documentation update only.

* Revert "Update CONTRIBUTING.md (mongodb#798)"

This reverts commit 71b9173.

* Revert "Revert "Update CONTRIBUTING.md (mongodb#798)""

This reverts commit 147159c.

Co-authored-by: Melissa Plunkett <[email protected]>

* INTMDB-357: Add exclude to skip go-header until google terraform provider version tag upgrade (mongodb#799)

* INTMDB-357: Add exclude to skip go-header until google terraform provider version tag upgrade

* Ran go mod tidy -compat=1.17

* Fix test to use aws provider ctedentials (mongodb#807)

* Update CHANGELOG.md

* go mod tidy

* Go lang lint update to v1.41.2

* GOLANGCI_VERSION=v1.42.1

* Try latest

* Fix go.sum

* golangci version

* tidy

* fixes

* fixes

* Update resource_mongodbatlas_database_user_test.go

Co-authored-by: Gustavo Bazan <[email protected]>
Co-authored-by: Dosty Everts <[email protected]>
Co-authored-by: Zuhair Ahmed <[email protected]>
Co-authored-by: Melissa Plunkett <[email protected]>
@martinstibbe martinstibbe deleted the INTMDB-320 branch January 4, 2023 18:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants