-
Notifications
You must be signed in to change notification settings - Fork 138
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
CBG-4019 Add init_in_progress property to db and verbose all_dbs responses #6910
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
API docs need to be updated:
sync_gateway/docs/api/components/schemas.yaml
Line 2526 in 9737474
reason: |
require.Equal(t, expectedValue, dbSummary.InitializationActive) | ||
} | ||
} | ||
require.True(t, dbFound, "Database not found in _all_dbs response") |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
require.True(t, dbFound, "Database not found in _all_dbs response") | |
require.True(t, dbFound, "Database not found in _all_dbs response %+v", allDbResult) |
rest/api.go
Outdated
Bucket string `json:"bucket"` | ||
State string `json:"state"` | ||
InitializationActive bool `json:"init_in_progress,omitempty"` | ||
Reason string `json:"reason,omitempty"` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you plan to change Reason
to RequireResync
in this PR? This was only in 3.1.7 and Isaac said it was fine to change this before rollout.
Redocly previews |
CBG-4019
Adds
init_in_progress
flag to REST API db status calls when async index initialization is in progress.Integration Tests
GSI=true,xattrs=true
https://jenkins.sgwdev.com/job/SyncGateway-Integration/2529/