{"action":"created","issue":{"url":"https://api.github.com/repos/owncloud/core/issues/17909","labels_url":"https://api.github.com/repos/owncloud/core/issues/17909/labels{/name}","comments_url":"https://api.github.com/repos/owncloud/core/issues/17909/comments","events_url":"https://api.github.com/repos/owncloud/core/issues/17909/events","html_url":"https://github.com/owncloud/core/issues/17909","id":97468031,"number":17909,"title":"[Experimental locking] Some smashbox locking tests fail for ownCloud 8.1.1beta1","user":{"login":"jvillafanez","id":1477829,"avatar_url":"https://avatars.githubusercontent.com/u/1477829?v=3","gravatar_id":"","url":"https://api.github.com/users/jvillafanez","html_url":"https://github.com/jvillafanez","followers_url":"https://api.github.com/users/jvillafanez/followers","following_url":"https://api.github.com/users/jvillafanez/following{/other_user}","gists_url":"https://api.github.com/users/jvillafanez/gists{/gist_id}","starred_url":"https://api.github.com/users/jvillafanez/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jvillafanez/subscriptions","organizations_url":"https://api.github.com/users/jvillafanez/orgs","repos_url":"https://api.github.com/users/jvillafanez/repos","events_url":"https://api.github.com/users/jvillafanez/events{/privacy}","received_events_url":"https://api.github.com/users/jvillafanez/received_events","type":"User","site_admin":false},"labels":[{"url":"https://api.github.com/repos/owncloud/core/labels/feature:locking","name":"feature:locking","color":"d4c5f9"},{"url":"https://api.github.com/repos/owncloud/core/labels/sev2-high","name":"sev2-high","color":"fc2929"},{"url":"https://api.github.com/repos/owncloud/core/labels/smashbox","name":"smashbox","color":"bfe5bf"}],"state":"open","locked":false,"assignee":null,"milestone":{"url":"https://api.github.com/repos/owncloud/core/milestones/46","html_url":"https://github.com/owncloud/core/milestones/8.1.1-current-maintenance","labels_url":"https://api.github.com/repos/owncloud/core/milestones/46/labels","id":1167392,"number":46,"title":"8.1.1-current-maintenance","description":"","creator":{"login":"cmonteroluque","id":879976,"avatar_url":"https://avatars.githubusercontent.com/u/879976?v=3","gravatar_id":"","url":"https://api.github.com/users/cmonteroluque","html_url":"https://github.com/cmonteroluque","followers_url":"https://api.github.com/users/cmonteroluque/followers","following_url":"https://api.github.com/users/cmonteroluque/following{/other_user}","gists_url":"https://api.github.com/users/cmonteroluque/gists{/gist_id}","starred_url":"https://api.github.com/users/cmonteroluque/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cmonteroluque/subscriptions","organizations_url":"https://api.github.com/users/cmonteroluque/orgs","repos_url":"https://api.github.com/users/cmonteroluque/repos","events_url":"https://api.github.com/users/cmonteroluque/events{/privacy}","received_events_url":"https://api.github.com/users/cmonteroluque/received_events","type":"User","site_admin":false},"open_issues":6,"closed_issues":57,"state":"open","created_at":"2015-06-16T19:39:31Z","updated_at":"2015-07-27T15:21:29Z","due_on":"2015-08-04T04:00:00Z","closed_at":null},"comments":2,"created_at":"2015-07-27T13:51:33Z","updated_at":"2015-07-27T15:37:33Z","closed_at":null,"body":"Tests that failed when run all together caused by an unexpected locked exception (probably due to a stray redis key) but passed when run one by one:\r\n* download + move file (test_downloadFilePlusAction.py t3)\r\n* download + move folder (test_downloadFilePlusAction.py t7)\r\n* overwrite + delete folder (test_overwriteFilePlusAction.py t5)\r\n\r\nTests that failed when run all together caused by a failed check but passed when run one by one:\r\n* overwrite + overwrite (test_overwriteFilePlusAction.py t0)\r\n* overwrite + delete file (test_overwriteFilePlusAction.py t4)\r\n\r\nTests that failed in both cases caused by an unexpected locked exception (all together and one by one)\r\n* overwrite + download file (test_overwriteFilePlusAction.py t1)\r\n* overwrite + download directory (test_overwriteFilePlusAction.py t3)\r\n\r\nFor the first and third set of tests, the locked exception is thrown in the first action (download or overwrite) when there shouldn't be any lock active.\r\nThe second set fail due to the extra checks: overwrite + overwrite reports a wrong filesize, and the overwrite + delete file doesn't delete the file for 2 of the 3 accounts.\r\n\r\nI'd appreciate a second run in order to confirm."},"comment":{"url":"https://api.github.com/repos/owncloud/core/issues/comments/125248272","html_url":"https://github.com/owncloud/core/issues/17909#issuecomment-125248272","issue_url":"https://api.github.com/repos/owncloud/core/issues/17909","id":125248272,"user":{"login":"PVince81","id":277525,"avatar_url":"https://avatars.githubusercontent.com/u/277525?v=3","gravatar_id":"","url":"https://api.github.com/users/PVince81","html_url":"https://github.com/PVince81","followers_url":"https://api.github.com/users/PVince81/followers","following_url":"https://api.github.com/users/PVince81/following{/other_user}","gists_url":"https://api.github.com/users/PVince81/gists{/gist_id}","starred_url":"https://api.github.com/users/PVince81/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/PVince81/subscriptions","organizations_url":"https://api.github.com/users/PVince81/orgs","repos_url":"https://api.github.com/users/PVince81/repos","events_url":"https://api.github.com/users/PVince81/events{/privacy}","received_events_url":"https://api.github.com/users/PVince81/received_events","type":"User","site_admin":false},"created_at":"2015-07-27T15:37:33Z","updated_at":"2015-07-27T15:37:33Z","body":"@jvillafanez I ran the tests:\r\n`bin/smash -a lib/oc-tests/new_locking/test_downloadFilePlusAction.py` from the locking_tests branch\r\n\r\nAll passed without and with transactional locking enabled (redis) on stable8.1 e8cbfd16a2ae0f697222b57b3114fb8b989199c5"}}