Skip to content

make chunksize (used to check for gone LDAP users) configurable - #18016

Merged
blizzz merged 1 commit into
masterfrom
fix/noid/ldap-checkup-batchsize
Nov 21, 2019
Merged

make chunksize (used to check for gone LDAP users) configurable#18016
blizzz merged 1 commit into
masterfrom
fix/noid/ldap-checkup-batchsize

Conversation

@blizzz

Copy link
Copy Markdown
Member

with a fair amount of users it could take a bit until a user was recognized as unavailable

Set it via php occ config:app:set --value=300 user_ldap cleanUpJobChunkSize

@blizzzblizzz added this to the Nextcloud 18 milestone Nov 19, 2019
@blizzz

Copy link
Copy Markdown
MemberAuthor

/backport to stable17

@blizzz

Copy link
Copy Markdown
MemberAuthor

/backport to stable16

@juliusknorrjuliusknorr left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

👍

Would be nice to have the config flag documented somewhere.

@ChristophWurstChristophWurst added 4. to release Ready to be released and/or waiting for tests to finish and removed 3. to review Waiting for reviews labels Nov 19, 2019
@blizzz
blizzzforce-pushed the fix/noid/ldap-checkup-batchsize branch from 3403754 to 51c1f9aCompareNovember 20, 2019 09:57
@blizzz

Copy link
Copy Markdown
MemberAuthor

(rebased for CI)

Signed-off-by: Arthur Schiwon <blizzz@arthur-schiwon.de>
@rullzer
rullzerforce-pushed the fix/noid/ldap-checkup-batchsize branch from 51c1f9a to f990620CompareNovember 20, 2019 18:44
@blizzz
blizzz merged commit e7f225c into masterNov 21, 2019
@blizzz
blizzz deleted the fix/noid/ldap-checkup-batchsize branch November 21, 2019 10:05
@backportbot-nextcloud

Copy link
Copy Markdown

backport to stable17 in #18054

@backportbot-nextcloud

Copy link
Copy Markdown

backport to stable16 in #18055

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

4. to releaseReady to be released and/or waiting for tests to finishbugfeature: ldap

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

@blizzz@ChristophWurst@juliusknorr