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

CLOUDP-283513: IP Access List new resource #3586

Draft
wants to merge 2 commits into
base: CLOUDP-296579-ako-flex-generation
Choose a base branch
from

Conversation

helderjs
Copy link
Contributor

Proposed changes

This PR add the support to export the new custom resource for IP Access List

Jira ticket: CLOUDP-283513

Checklist

  • I have signed the MongoDB CLA
  • I have added tests that prove my fix is effective or that my feature works
  • I have added any necessary documentation in document requirements section listed in CONTRIBUTING.md (if appropriate)
  • I have addressed the @mongodb/docs-cloud-team comments (if appropriate)
  • I have updated test/README.md (if an e2e test has been added)
  • I have run make fmt and formatted my code

@helderjs helderjs requested review from a team as code owners January 29, 2025 14:49
@helderjs helderjs changed the base branch from master to CLOUDP-296579-ako-flex-generation January 29, 2025 14:50
@helderjs helderjs marked this pull request as draft January 29, 2025 14:50
Copy link
Contributor

apix-bot bot commented Jan 29, 2025

Coverage Report 📉

Branch Commit Coverage
master 57bb6d5 38.8%
CLOUDP-283513-ip-access-list 68e4d98 %
Difference -38.8%

go.mod Outdated
@@ -1,6 +1,6 @@
module github.com/mongodb/mongodb-atlas-cli/atlascli

go 1.23.1
go 1.23.5
Copy link
Collaborator

Choose a reason for hiding this comment

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

not sure if this is wanted

go.mod Outdated
@@ -30,7 +30,7 @@ require (
github.com/mholt/archives v0.1.0
github.com/mongodb-forks/digest v1.1.0
github.com/mongodb-labs/cobra2snooty v0.18.2
github.com/mongodb/mongodb-atlas-kubernetes/v2 v2.6.2-0.20250123164100-b8aa5669900f
github.com/mongodb/mongodb-atlas-kubernetes/v2 v2.6.2-0.20250129015533-7920f411a73e
Copy link
Collaborator

Choose a reason for hiding this comment

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

i would expect 2.7.0 in here

Copy link
Collaborator

@s-urbaniak s-urbaniak left a comment

Choose a reason for hiding this comment

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

generally lgtm if evergreen e2e is happy but i think the 2.7 AKO dependency is missing and a go.mod change which would affect the whole repo

@helderjs helderjs force-pushed the CLOUDP-296579-ako-flex-generation branch from b38f566 to be8d06a Compare February 6, 2025 12:28
@helderjs helderjs force-pushed the CLOUDP-283513-ip-access-list branch from 68e4d98 to 80d1bdc Compare February 6, 2025 18:39
@github-actions github-actions bot removed the atlascli label Feb 6, 2025
@helderjs helderjs force-pushed the CLOUDP-283513-ip-access-list branch from 80d1bdc to dde3a08 Compare February 6, 2025 18:43
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.

2 participants