Skip to content

Rebuild index#749

Merged
butonic merged 27 commits into
masterfrom
rebuild-index
Oct 24, 2020
Merged

Rebuild index#749
butonic merged 27 commits into
masterfrom
rebuild-index

Conversation

@kulmann
Copy link
Copy Markdown
Contributor

@kulmann kulmann commented Oct 23, 2020

CLI command for rebuilding the index. Can be called from accounts cli with rebuild command.

Comment thread accounts/pkg/command/rebuild_index.go Outdated
Comment thread accounts/pkg/storage/cs3.go Outdated
Comment thread accounts/pkg/storage/cs3.go
Comment thread accounts/pkg/storage/cs3.go Outdated
Comment thread accounts/pkg/storage/cs3.go Outdated
Comment thread accounts/pkg/storage/cs3.go Outdated
Comment thread accounts/pkg/storage/cs3.go Outdated
Comment thread accounts/pkg/storage/cs3.go Outdated
Comment thread accounts/pkg/storage/disk.go

Tags: accounts

The index for the accounts service can now be rebuilt by running the cli command `./bin/ocis accounts rebuild`. It deletes all configured indices and rebuilds them from the documents found on storage.
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

and you added a ListAccounts & ListGroups call

Copy link
Copy Markdown
Contributor

@butonic butonic left a comment

Choose a reason for hiding this comment

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

the only real naming issue is the RebuildIndex ... other than that I'm good

@kulmann
Copy link
Copy Markdown
Contributor Author

kulmann commented Oct 24, 2020

@refs we should take the time and write an integration test, at least with the disk implementation.

@sonarqubecloud
Copy link
Copy Markdown

Kudos, SonarCloud Quality Gate passed!

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities (and Security Hotspot 0 Security Hotspots to review)
Code Smell A 0 Code Smells

No Coverage information No Coverage information
No Duplication information No Duplication information

@butonic butonic merged commit 5f67e8c into master Oct 24, 2020
@delete-merged-branch delete-merged-branch Bot deleted the rebuild-index branch October 24, 2020 07:07
ownclouders pushed a commit that referenced this pull request Oct 24, 2020
Merge: 17a180e bef3a4d
Author: Jörn Friedrich Dreyer <jfd@butonic.de>
Date:   Sat Oct 24 09:07:13 2020 +0200

    Merge pull request #749 from owncloud/rebuild-index

    Rebuild index
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