Skip to content

feat: Add support for MONGODB_ATLAS_PUBLIC_API_KEY and MONGODB_ATLAS_PRIVATE_API_KEY to TF #3505

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

Open
wants to merge 6 commits into
base: master
Choose a base branch
from

Conversation

andreaangiolillo
Copy link
Collaborator

@andreaangiolillo andreaangiolillo commented Jul 17, 2025

Description

Ticket: CLOUDP-332049

@andreaangiolillo andreaangiolillo changed the title CLOUDP-332049: add support for MONGODB_ATLAS_PUBLIC_API_KEY and MONGODB_ATLAS_PRIVATE_API_KEY to TF CLOUDP-332049: add support for MONGODB_ATLAS_PUBLIC_API_KEY and MONGODB_ATLAS_PRIVATE_API_KEY to TF Jul 17, 2025
@andreaangiolillo andreaangiolillo changed the title CLOUDP-332049: add support for MONGODB_ATLAS_PUBLIC_API_KEY and MONGODB_ATLAS_PRIVATE_API_KEY to TF feat: add support for MONGODB_ATLAS_PUBLIC_API_KEY and MONGODB_ATLAS_PRIVATE_API_KEY to TF Jul 17, 2025
@andreaangiolillo andreaangiolillo changed the title feat: add support for MONGODB_ATLAS_PUBLIC_API_KEY and MONGODB_ATLAS_PRIVATE_API_KEY to TF feat: Add support for MONGODB_ATLAS_PUBLIC_API_KEY and MONGODB_ATLAS_PRIVATE_API_KEY to TF Jul 17, 2025
@andreaangiolillo andreaangiolillo marked this pull request as ready for review July 17, 2025 13:08
@Copilot Copilot AI review requested due to automatic review settings July 17, 2025 13:08
@andreaangiolillo andreaangiolillo requested review from a team as code owners July 17, 2025 13:08
Copy link
Contributor

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

@andreaangiolillo andreaangiolillo changed the title feat: Add support for MONGODB_ATLAS_PUBLIC_API_KEY and MONGODB_ATLAS_PRIVATE_API_KEY to TF feat: Add support for MONGODB_ATLAS_PUBLIC_API_KEY and MONGODB_ATLAS_PRIVATE_API_KEY to TF Jul 17, 2025
Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR adds support for new environment variables MONGODB_ATLAS_PUBLIC_API_KEY and MONGODB_ATLAS_PRIVATE_API_KEY to the Terraform provider, which are widely used across the MongoDB ecosystem and provide better compatibility with other MongoDB tools.

  • Adds the new environment variables to both SDK2 and framework provider implementations
  • Updates documentation to recommend the new environment variables over the existing ones
  • Includes changelog entry documenting the new feature

Reviewed Changes

Copilot reviewed 4 out of 4 changed files in this pull request and generated 2 comments.

File Description
internal/provider/provider_sdk2.go Adds new environment variables to the SDK2 provider configuration
internal/provider/provider.go Adds new environment variables to the framework provider configuration
docs/index.md Updates documentation to recommend new environment variables and explains compatibility benefits
.changelog/3505.txt Adds changelog entry for the new environment variables

@oarbusi
Copy link
Collaborator

oarbusi commented Jul 17, 2025

LGTM, only concern I have is that the usages of MONGODB_ATLAS_PUBLIC_KEY across the repo should be changed as well for consistency and to avoid confusion. Could be in this PR or a follow up

@andreaangiolillo
Copy link
Collaborator Author

I don't have a strong opinion but I don't think is needed since MONGODB_ATLAS_PUBLIC_KEY is included to ensure that AtlasCLI and TF use the same env variables. As long you don't use AtlasCLI in your tests you can keep using the current env

Copy link

@JuliaMongo JuliaMongo left a comment

Choose a reason for hiding this comment

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

LGTM with a few suggestions.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants