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

feat: support for project limits in project resource and project/projects data sources #1347

Merged
merged 9 commits into from
Jul 31, 2023

Conversation

AgustinBettati
Copy link
Member

@AgustinBettati AgustinBettati commented Jul 26, 2023

Description

Ticket: INTMDB-554

This PR includes the support for a new limits attribute in project resource and data source.

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.
  • Breaking change (fix or feature that would cause existing functionality to not work as expected). Please, add the "breaking change" label to the PR.
  • This change requires a documentation update
  • Documentation fix/enhancement

Required Checklist:

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

  • Included acceptance tests for creating, modifying, and removing limits in a project resource, as well as defining incorrect limit values. CI project resource tests have passed successfully.

Copy link
Collaborator

@andreaangiolillo andreaangiolillo left a comment

Choose a reason for hiding this comment

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

Great work on this! I left a few comments. Make sure to fix the lint error and to add an example under https://github.com/mongodb/terraform-provider-mongodbatlas/tree/master/examples

mongodbatlas/data_source_mongodbatlas_project.go Outdated Show resolved Hide resolved
mongodbatlas/resource_mongodbatlas_project.go Outdated Show resolved Hide resolved
mongodbatlas/resource_mongodbatlas_project.go Outdated Show resolved Hide resolved
mongodbatlas/resource_mongodbatlas_project.go Outdated Show resolved Hide resolved
mongodbatlas/resource_mongodbatlas_project_test.go Outdated Show resolved Hide resolved
mongodbatlas/resource_mongodbatlas_project_test.go Outdated Show resolved Hide resolved
mongodbatlas/resource_mongodbatlas_project_test.go Outdated Show resolved Hide resolved
Copy link
Collaborator

@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.

Few minor comments, not blocking. otherwise LGTM

website/docs/d/projects.html.markdown Show resolved Hide resolved
website/docs/r/project.html.markdown Outdated Show resolved Hide resolved
Copy link
Collaborator

@andreaangiolillo andreaangiolillo left a comment

Choose a reason for hiding this comment

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

LGTM

@AgustinBettati AgustinBettati changed the title INTMDB-554: support for project limits in project resource and project/projects data sources feat: support for project limits in project resource and project/projects data sources Jul 28, 2023
@Zuhairahmed
Copy link
Collaborator

@zach-carr any feedback on this one? if not @AgustinBettati feel free to merge in when ready

@AgustinBettati
Copy link
Member Author

Will go ahead and merge, if any comments are made I will make sure to address in a separate PR.

@AgustinBettati AgustinBettati merged commit 9c13dc0 into master Jul 31, 2023
20 checks passed
@AgustinBettati AgustinBettati deleted the INTMDB-554 branch July 31, 2023 11:14
Comment on lines +217 to +222
diagWarning := diag.Diagnostic{
Severity: diag.Warning,
Summary: "Error getting project's api keys",
Detail: fmt.Sprintf("Error getting project's api keys (%s): %s", project.ID, err),
}
diags = append(diags, diagWarning)
Copy link
Collaborator

Choose a reason for hiding this comment

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

This change seems the root cause of the issue #1371

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.

None yet

5 participants