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
dependent archive never run #18403
dependent archive never run #18403
Conversation
set plugin not include UserCountry
update processer
This issue is in "needs review" but there has been no activity for 7 days. ping @matomo-org/core-reviewers |
@peterhashair were you able to reproduce the reported issue? I don't think that specific solution is something we want to go with. We should check for a global solution, as if that can happen for one report, it can also happen for any similar report. |
@peterhashair we could catch up in a call tomorrow morning or afternoon if that helps? generally I would have thought that https://github.com/matomo-org/matomo/compare/4.x-dev...processdependentarchive?quick_pull=1#diff-0423f9c2b4343926ff8deebfed3605b3f1905daa9e8fecc1ec1f00f65bf3e997R683 should fix it. |
remove hardcode plugin name
The changes are logically looking fine. I was wondering if we are able to reproduce an issue that those changes will fix. If so we could possibly add a test for that, to ensure that won't regress in the future. |
fyi @sgiehl we actually noticed that there might not have been an actual issue and it worked potentially "by luck" as MediaAnalytics requested a different plugin to be archived and then it wouldn't ever This kind of also means there might be another bug in there as even if the segment is already now being archived, in some cases it would archive it again (I think there's an issue for cron archiving but this check may be needed for browser archiving or if only a specific plugin is requested, quite difficult). I might create a separate issue for this. A quick test could be maybe added though |
@peterhashair could you check if it's possible to add a useful test for that? |
…archives-never-processed
If you don't want this PR to be closed automatically in 28 days then you need to assign the label 'Do not close'. |
Description:
Fixes: #18362
Fix for dependent archive never run, not sure that's the best approach
Review