[9.0] error out if a local storage isn't setup correctly #24289

Merged
merged 3 commits into from Apr 27, 2016

Projects

None yet

5 participants

@icewind1991
Member

Backport of #24246 to 9.0

@icewind1991 icewind1991 added this to the 9.0.2-current-maintenance milestone Apr 26, 2016
@mention-bot

By analyzing the blame information on this pull request, we identified @DeepDiver1975, @nickvergessen and @PVince81 to be potential reviewers

@PVince81
Collaborator

๐Ÿ‘ local storage still works

@PVince81
Collaborator

@LukasReschke second review ?

@rullzer
Contributor
rullzer commented Apr 26, 2016

๐Ÿ‘

@PVince81
Collaborator

Some weird integration test errors:
Test Result (17 failures / +17)

caldav.Accessing a not existing calendar of another user
caldav.Accessing a not existing calendar of myself
carddav.Accessing a not existing addressbook of another user
carddav.Accessing a not shared addressbook of another user
carddav.Accessing a not existing addressbook of myself
carddav.Creating a new addressbook
checksums.Uploading a file with checksum should work
checksums.Uploading a file with checksum should return the checksum in the propfind
checksums.Uploading a file with checksum should return the checksum in the download header
checksums.Moving a file with checksum should return the checksum in the propfind
checksums.Moving file with checksum should return the checksum in the download header
checksums.Copying a file with checksum should return the checksum in the propfind
checksums.Copying file with checksum should return the checksum in the download header
checksums.Overwriting a file with checksum should remove the checksum and not return it in the propfind
checksums.Overwriting a file with checksum should remove the checksum and not return it in the download header
checksums.Uploading a chunked file with checksum should return the checksum in the propfind
checksums.Uploading a chunked file with checksum should return the checksum in the download header

Doesn't happen on other PRs, probably unrelated.

icewind1991 added some commits Apr 25, 2016
@icewind1991 @PVince81 icewind1991 error out if a local storage isn't setup correctly 06bfd58
@icewind1991 @PVince81 icewind1991 add tests
884c821
@PVince81
Collaborator

Rebased to relaunch CI, to be on the safe side

@cmonteroluque cmonteroluque Merge branch 'stable9' into local-invalid-9
fe753fe
@cmonteroluque cmonteroluque merged commit 1c23fa5 into stable9 Apr 27, 2016

22 checks passed

cla-bot-core Build #3602 succeeded in 43 sec
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
core-ci-linux-jsunit/database=sqlite,label=SLAVE Build #61391 succeeded in 7 min 51 sec
Details
core-ci-linux-swift-primary-storage/database=mysql,label=SLAVE Build #55559 succeeded in 17 min
Details
core-ci-linux/database=mysql,label=SLAVE Build #30010 succeeded in 25 min
Details
core-ci-linux/database=oci,label=SLAVE Build #30010 succeeded in 42 min
Details
core-ci-linux/database=pgsql,label=SLAVE Build #30010 succeeded in 19 min
Details
core-ci-linux/database=sqlite,label=SLAVE Build #30010 succeeded in 9 min 49 sec
Details
ocs-api-integration-tests-ci Build #10198 succeeded in 7 min 40 sec
Details
server-master-linux-externals-ci/database=sqlite,external=smb-silvershell,label=SLAVE Build #9816 succeeded in 1 min 53 sec
Details
server-master-linux-externals-ci/database=sqlite,external=swift-ceph,label=SLAVE Build #9816 succeeded in 3 min 30 sec
Details
server-master-linux-externals-ci/database=sqlite,external=webdav-ownCloud,label=SLAVE Build #9816 succeeded in 3 min 33 sec
Details
server-master-linux-externals-smb-windows-ext-ci/database=sqlite,external=smb-windows,label=master Build #13149 succeeded in 4 min 30 sec
Details
server-master-linux-php5.4-ci/database=sqlite,label=SLAVE Build #2963 succeeded in 10 min
Details
server-master-linux-php7-ci/database=sqlite,label=SLAVE Build #38249 succeeded in 11 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=litmus,mirallBranch=v2.0.2,slave=SMASH Build #14125 succeeded in 12 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_basicSync@0,mirallBranch=v2.0.2,slave=SMASH Build #14125 succeeded in 20 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_basicSync@1,mirallBranch=v2.0.2,slave=SMASH Build #14125 succeeded in 44 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_shareLink,mirallBranch=v2.0.2,slave=SMASH Build #14125 succeeded in 51 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_sharePermissions,mirallBranch=v2.0.2,slave=SMASH Build #14125 succeeded in 41 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_sharePropagationGroups,mirallBranch=v2.0.2,slave=SMASH Build #14125 succeeded in 15 min
Details
smashbox-on-docker-ci/DOCKER_IMAGE=ubuntu_oc_lamp-git,TEST_NAME=test_sharePropagationInsideGroups,mirallBranch=v2.0.2,slave=SMASH Build #14125 succeeded in 18 min
Details
@MorrisJobke MorrisJobke deleted the local-invalid-9 branch May 11, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment