{"action":"created","issue":{"url":"https://api.github.com/repos/ArduPilot/ardupilot/issues/6834","repository_url":"https://api.github.com/repos/ArduPilot/ardupilot","labels_url":"https://api.github.com/repos/ArduPilot/ardupilot/issues/6834/labels{/name}","comments_url":"https://api.github.com/repos/ArduPilot/ardupilot/issues/6834/comments","events_url":"https://api.github.com/repos/ArduPilot/ardupilot/issues/6834/events","html_url":"https://github.com/ArduPilot/ardupilot/pull/6834","id":253056309,"number":6834,"title":"Autotest: Skip unnecessary log downloads","user":{"login":"WickedShell","id":567688,"avatar_url":"https://avatars2.githubusercontent.com/u/567688?v=4","gravatar_id":"","url":"https://api.github.com/users/WickedShell","html_url":"https://github.com/WickedShell","followers_url":"https://api.github.com/users/WickedShell/followers","following_url":"https://api.github.com/users/WickedShell/following{/other_user}","gists_url":"https://api.github.com/users/WickedShell/gists{/gist_id}","starred_url":"https://api.github.com/users/WickedShell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/WickedShell/subscriptions","organizations_url":"https://api.github.com/users/WickedShell/orgs","repos_url":"https://api.github.com/users/WickedShell/repos","events_url":"https://api.github.com/users/WickedShell/events{/privacy}","received_events_url":"https://api.github.com/users/WickedShell/received_events","type":"User","site_admin":false},"labels":[{"id":228148658,"url":"https://api.github.com/repos/ArduPilot/ardupilot/labels/SITL","name":"SITL","color":"d4c5f9","default":false},{"id":118327982,"url":"https://api.github.com/repos/ArduPilot/ardupilot/labels/Tools","name":"Tools","color":"d4c5f9","default":false}],"state":"closed","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":1,"created_at":"2017-08-26T02:54:39Z","updated_at":"2018-05-13T08:04:30Z","closed_at":"2018-05-13T08:04:30Z","author_association":"CONTRIBUTOR","pull_request":{"url":"https://api.github.com/repos/ArduPilot/ardupilot/pulls/6834","html_url":"https://github.com/ArduPilot/ardupilot/pull/6834","diff_url":"https://github.com/ArduPilot/ardupilot/pull/6834.diff","patch_url":"https://github.com/ArduPilot/ardupilot/pull/6834.patch"},"body":"Downloading logs over MAVLink represents 20% of the run time of the fly.ArduCopter runtime. This is used to both test log downloading and copy the log for the autotest server. However the log file that is selected is a large log file, which takes quite a long time to process (it's speed limited by MAVProxy). This PR moves log downloading to it's own test which is run once, and directly copies the log file on the disk for autotest server.\r\n\r\nThe CI environment variable is used to flag if logs should be copied (was selected because Semaphore defaults to setting it true). It could potentially be passed down from autotest.py as an option instead if that is preferred, it just meant more modifications to the CI sides to pass it in. I'm fine with changing it to whichever approach is preferred.\r\n\r\nThis isn't expected to be merged as is, I'm presenting it here for review before I duplicate the same logic into plane/rover/sub scripts. This is meant to live along with #6821 "},"comment":{"url":"https://api.github.com/repos/ArduPilot/ardupilot/issues/comments/388609018","html_url":"https://github.com/ArduPilot/ardupilot/pull/6834#issuecomment-388609018","issue_url":"https://api.github.com/repos/ArduPilot/ardupilot/issues/6834","id":388609018,"user":{"login":"WickedShell","id":567688,"avatar_url":"https://avatars2.githubusercontent.com/u/567688?v=4","gravatar_id":"","url":"https://api.github.com/users/WickedShell","html_url":"https://github.com/WickedShell","followers_url":"https://api.github.com/users/WickedShell/followers","following_url":"https://api.github.com/users/WickedShell/following{/other_user}","gists_url":"https://api.github.com/users/WickedShell/gists{/gist_id}","starred_url":"https://api.github.com/users/WickedShell/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/WickedShell/subscriptions","organizations_url":"https://api.github.com/users/WickedShell/orgs","repos_url":"https://api.github.com/users/WickedShell/repos","events_url":"https://api.github.com/users/WickedShell/events{/privacy}","received_events_url":"https://api.github.com/users/WickedShell/received_events","type":"User","site_admin":false},"created_at":"2018-05-13T08:04:30Z","updated_at":"2018-05-13T08:04:30Z","author_association":"CONTRIBUTOR","body":"This is still needed/a good idea, but I can't see myself getting to it very quickly, and it's probably easier to make this from scratch at this point rather then to try and rebase it, so I'm going to go ahead and close it out."}}
{"actor":{"display_login":"WickedShell"}}