Skip to content

Conversation

@andreaangiolillo
Copy link
Collaborator

@andreaangiolillo andreaangiolillo commented Oct 15, 2024

Proposed changes

This PR fixes the discrepancy in atlas deployments search indexes list and atlas clusters search indexes list where the deployment command returns an error if the cluster is in the UPDATING state.

Jira ticket: CLOUDP-278196

Testing

I was able to reproduce the issue

atlas clusters search indexes list --clusterName Cluster0 --collection listingsAndReviews --db sample_airbnb                                
ID                         NAME      DATABASE        COLLECTION           TYPE
670e63122b37bb42dc933cf7   default   sample_airbnb   listingsAndReviews   search

atlas deployments search indexes list -P dev --deploymentName Cluster0 --db sample_airbnb --collection listingsAndReviews                                
Error: deployment is in unexpected state: UPDATING

and I tested that the fix worked:

# Using atlascli 1.29.0 I got the error
atlas deployments search indexes list -P dev --deploymentName Cluster0 --db sample_airbnb --collection listingsAndReviews                             
Error: deployment is in unexpected state: UPDATING

# I did not get the error using the local binary with the fix
./bin/atlas deployments search indexes list -P dev --deploymentName Cluster0 --db sample_airbnb --collection listingsAndReviews                          
ID                         NAME      DATABASE        COLLECTION           STATUS   TYPE
670e63122b37bb42dc933cf7   default   sample_airbnb   listingsAndReviews   STEADY   search

@andreaangiolillo andreaangiolillo marked this pull request as ready for review October 15, 2024 13:10
@andreaangiolillo andreaangiolillo requested a review from a team as a code owner October 15, 2024 13:10
@andreaangiolillo andreaangiolillo merged commit cce9740 into master Oct 15, 2024
21 of 23 checks passed
@andreaangiolillo andreaangiolillo deleted the CLOUDP-278196 branch October 15, 2024 13:24
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.

3 participants