-
Notifications
You must be signed in to change notification settings - Fork 2.1k
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
Fix files:scan repair + basic tests for files:scan command #29635
Conversation
Ohhh that explains the failure: |
running this one test class locally with oci passes... now to run the full set... |
boo... local test run with Oracle and all tests does not fail... |
also ran tests locally with filtering by group "DB" with oci, still doesn't fail... |
@owncloud-bot rebase |
rebased and hoping for some magic resolution... |
3e3bf7d
to
76e7854
Compare
@owncloud-bot rebase please |
@ownclouders rebase please |
Hey! I'm GitMate.io! This pull request is being rebased automatically. Please DO NOT push while rebase is in progress or your changes would be lost permanently |
Automated rebase with GitMate.io was successful! 🎉 |
76e7854
to
23aab67
Compare
Codecov Report
@@ Coverage Diff @@
## master #29635 +/- ##
============================================
+ Coverage 61.82% 61.88% +0.05%
Complexity 19051 19051
============================================
Files 1089 1089
Lines 61442 61444 +2
============================================
+ Hits 37988 38025 +37
+ Misses 23454 23419 -35
Continue to review full report at Codecov.
|
Oh wait, it's the primary object store tests failing! I'll exclude those as scanning doesn't make sense there. |
23aab67
to
c322c11
Compare
7b115df
to
3c0db34
Compare
some weird errors not even related to this PR... it's like it was testing another PR... anyway, rebased |
Not even related to my PR ?!
|
oh wait, I read it wrong... oh... |
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.
Makes sense. Tests run locally and repair/scan command runs fine.
requires backport of #30020 to not have conflicts on ScanTest |
stable10: #30618 |
Repair command was broken through #30451 (unreleased), so this PR could be counted as an addition to this one to fix it. |
This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Will fail on CI, pass locally, needs work...