{"action":"created","issue":{"url":"https://api.github.com/repos/cvmfs/cvmfs/issues/747","labels_url":"https://api.github.com/repos/cvmfs/cvmfs/issues/747/labels{/name}","comments_url":"https://api.github.com/repos/cvmfs/cvmfs/issues/747/comments","events_url":"https://api.github.com/repos/cvmfs/cvmfs/issues/747/events","html_url":"https://github.com/cvmfs/cvmfs/pull/747","id":55876946,"number":747,"title":"Fix s3 unit tests","user":{"login":"ssheikki","id":4517162,"avatar_url":"https://avatars.githubusercontent.com/u/4517162?v=3","gravatar_id":"","url":"https://api.github.com/users/ssheikki","html_url":"https://github.com/ssheikki","followers_url":"https://api.github.com/users/ssheikki/followers","following_url":"https://api.github.com/users/ssheikki/following{/other_user}","gists_url":"https://api.github.com/users/ssheikki/gists{/gist_id}","starred_url":"https://api.github.com/users/ssheikki/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ssheikki/subscriptions","organizations_url":"https://api.github.com/users/ssheikki/orgs","repos_url":"https://api.github.com/users/ssheikki/repos","events_url":"https://api.github.com/users/ssheikki/events{/privacy}","received_events_url":"https://api.github.com/users/ssheikki/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":2,"created_at":"2015-01-29T09:41:23Z","updated_at":"2015-01-29T10:33:47Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/cvmfs/cvmfs/pulls/747","html_url":"https://github.com/cvmfs/cvmfs/pull/747","diff_url":"https://github.com/cvmfs/cvmfs/pull/747.diff","patch_url":"https://github.com/cvmfs/cvmfs/pull/747.patch"},"body":"Local upload unit tests work now also for S3. Tests are basically copy paste, but a simple S3 mockup server was added to make the tests possible. Few relatively small fixes also to the actual code due to that. Unfortunately also one config file spelling mistake was found, PARALLELL should be PARALLEL."},"comment":{"url":"https://api.github.com/repos/cvmfs/cvmfs/issues/comments/72003425","html_url":"https://github.com/cvmfs/cvmfs/pull/747#issuecomment-72003425","issue_url":"https://api.github.com/repos/cvmfs/cvmfs/issues/747","id":72003425,"user":{"login":"reneme","id":1562139,"avatar_url":"https://avatars.githubusercontent.com/u/1562139?v=3","gravatar_id":"","url":"https://api.github.com/users/reneme","html_url":"https://github.com/reneme","followers_url":"https://api.github.com/users/reneme/followers","following_url":"https://api.github.com/users/reneme/following{/other_user}","gists_url":"https://api.github.com/users/reneme/gists{/gist_id}","starred_url":"https://api.github.com/users/reneme/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/reneme/subscriptions","organizations_url":"https://api.github.com/users/reneme/orgs","repos_url":"https://api.github.com/users/reneme/repos","events_url":"https://api.github.com/users/reneme/events{/privacy}","received_events_url":"https://api.github.com/users/reneme/received_events","type":"User","site_admin":false},"created_at":"2015-01-29T10:33:47Z","updated_at":"2015-01-29T10:33:47Z","body":"One more general thing: You made also some patches to the production code. Since we are targeting for a code freeze still this week: Is there anything (except the config variable typo fix) that is worth to go into the 2.1.20 release? \r\n\r\nSince the code runs pretty much flawlessly on the LHCb nightlies, I am a bit reluctant to potentially break it in the last second. ;-)"}}