[stable9] Integration test encryption homedir#26996
Conversation
49ad93f to
e7c5b28
Compare
|
Cherry-picked the other backports from #26993 |
|
Running tests locally:
|
|
Cherry-picked #27015, hopefully all will pass on Jenkins at least. |
SergioBertolinSG
left a comment
There was a problem hiding this comment.
After removing external storage test with encryption all tests pass.
|
Thanks. Let's wait for Jenkins to confirm. Then I'll remove the extra test commit and merge directly. |
|
When will it end ? |
|
Seems to be from this run. |
|
Here is the matching log from CI: I wasn't able to reproduce this locally. Maybe this requires running all test suites one after the other to create side effects ? 😿 |
|
I cannot reproduce it locally as well. But when I run this test I am not seeing this messages: |
|
In the logs of #26996 (comment) there is also an exception: |
|
All tests passed for me with encryption locally. |
fadb51b to
8a8e4d4
Compare
|
Rebased... let's hope... |
Added filter to disable some tests when encryption is enabled.
|
When running the tests locally, I never see that "Bad Signature" message which appears all the time on Jenkins: |
|
Ok, interesting, I'm able to get the "Bad signature" messages when I run the integration tests with SQLite. So far I've always run them with Mariadb. |
|
Steps to reproduce the encryption issue:
This sets up ownCloud with SQLite and this somehow triggers this error... |
|
Ok got it... My reset script usually deletes all contents of the "data" directory. However if you reset OC by just deleting the config but keeping the "data" directory, then the old private key of admin from a previous setup still exists, and that key is invalid. Now I wonder why it worked on stable9.1 and master, I'll check if the cleanup there is deeper. |
8a8e4d4 to
eb1bb96
Compare
|
|
|
@SergioBertolinSG tests finally passed now! I'll remove the TEMP TEST commit now. |
eb1bb96 to
6638ab3
Compare
Bad luck is in the air today... |
|
Anyway, this PR doesn't affect regular unit tests. 👍 for @SergioBertolinSG's commits. Merge ? |
|
Agreed. |
|
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. |
Backport of #26844 to stable9.
I added an extra commit to temporarily enable the new tests on this PR, needs to be removed before merge !it's gone@DeepDiver1975 @SergioBertolinSG