Skip to content

Inclusion: log mistakenly notes failure of inclusion#1431

Merged
ikedas merged 1 commit intosympa-community:sympa-6.2from
ikedas:issue-1429
Nov 20, 2022
Merged

Inclusion: log mistakenly notes failure of inclusion#1431
ikedas merged 1 commit intosympa-community:sympa-6.2from
ikedas:issue-1429

Conversation

@ikedas
Copy link
Copy Markdown
Member

@ikedas ikedas commented Jun 23, 2022

Inclusion: When all of the data sources are skipped by nosync_time_ranges, log notes failure of inclusion.

This may fix #1429 .

@ikedas ikedas marked this pull request as ready for review August 11, 2022 05:15
@ikedas
Copy link
Copy Markdown
Member Author

ikedas commented Aug 11, 2022

Known bug

Among multiple data sources, if there is no common time period that is not excluded by nosync_time_range in all of them, expired users will never be deleted from the list.

@ikedas ikedas merged commit 7599058 into sympa-community:sympa-6.2 Nov 20, 2022
@ikedas ikedas deleted the issue-1429 branch November 20, 2022 05:36
@ikedas ikedas added the bug label Nov 22, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Logs reflect Failure when include from SQL is carried out during nosync_time_ranges

1 participant