Skip to content

Conversation

@josvazg
Copy link
Collaborator

@josvazg josvazg commented Oct 28, 2025

Summary

Change the predicates so reconcile happens when the skip annotation is removed.

Proof of Work

✅ Added unit tests.
✅ Manually verified with sample config. Removing the skips triggers reconciliation:

apiVersion: atlas.mongodb.com/v1
kind: AtlasProject
metadata: 
  name: my-project
spec: 
  name: Test Atlas Operator Project
---
apiVersion: atlas.mongodb.com/v1
kind: AtlasIPAccessList
metadata: 
  name: my-project-ips
  annotations: 
   mongodb.com/atlas-reconciliation-policy: "skip"
spec: 
  projectRef: 
    name: my-project
  entries: 
  - comment: "anything goes"
    cidrBlock: "0.0.0.0/0"
---
apiVersion: atlas.mongodb.com/v1
kind: AtlasDeployment
metadata: 
  name: my-atlas-cluster
  annotations: 
    mongodb.com/atlas-reconciliation-policy: "skip"
spec: 
  projectRef: 
    name: my-project
  deploymentSpec: 
    clusterType: REPLICASET
    name: "Test-cluster"
    replicationSpecs: 
      - regionConfigs: 
          - regionName: US_EAST_1
            providerName: TENANT
            backingProviderName: AWS
            electableSpecs: 
              instanceSize: M0
              nodeCount: 3

Checklist

  • Have you linked a jira ticket and/or is the ticket in the title?
  • Have you checked whether your jira ticket required DOCSP changes?
  • Have you signed our CLA?

@josvazg josvazg requested a review from a team as a code owner October 28, 2025 15:11
@josvazg josvazg force-pushed the CLOUDP-340286/reconcile-skip-removal branch 2 times, most recently from 4900b39 to 779eb71 Compare October 28, 2025 16:06
@josvazg josvazg merged commit 86d0049 into main Nov 4, 2025
132 of 133 checks passed
@josvazg josvazg deleted the CLOUDP-340286/reconcile-skip-removal branch November 4, 2025 14:57
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