{"action":"created","issue":{"url":"https://api.github.com/repos/os-autoinst/os-autoinst/issues/574","repository_url":"https://api.github.com/repos/os-autoinst/os-autoinst","labels_url":"https://api.github.com/repos/os-autoinst/os-autoinst/issues/574/labels{/name}","comments_url":"https://api.github.com/repos/os-autoinst/os-autoinst/issues/574/comments","events_url":"https://api.github.com/repos/os-autoinst/os-autoinst/issues/574/events","html_url":"https://github.com/os-autoinst/os-autoinst/pull/574","id":174433112,"number":574,"title":"don't sha1sum qcow assets (poo#13526)","user":{"login":"AdamWill","id":916551,"avatar_url":"https://avatars.githubusercontent.com/u/916551?v=3","gravatar_id":"","url":"https://api.github.com/users/AdamWill","html_url":"https://github.com/AdamWill","followers_url":"https://api.github.com/users/AdamWill/followers","following_url":"https://api.github.com/users/AdamWill/following{/other_user}","gists_url":"https://api.github.com/users/AdamWill/gists{/gist_id}","starred_url":"https://api.github.com/users/AdamWill/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/AdamWill/subscriptions","organizations_url":"https://api.github.com/users/AdamWill/orgs","repos_url":"https://api.github.com/users/AdamWill/repos","events_url":"https://api.github.com/users/AdamWill/events{/privacy}","received_events_url":"https://api.github.com/users/AdamWill/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":6,"created_at":"2016-09-01T04:16:17Z","updated_at":"2016-09-01T06:31:28Z","closed_at":null,"pull_request":{"url":"https://api.github.com/repos/os-autoinst/os-autoinst/pulls/574","html_url":"https://github.com/os-autoinst/os-autoinst/pull/574","diff_url":"https://github.com/os-autoinst/os-autoinst/pull/574.diff","patch_url":"https://github.com/os-autoinst/os-autoinst/pull/574.patch"},"body":"it takes a long time and blocks the worker process, preventing\r\nit from sending status updates to the server and making it\r\nvulnerable to the 'dead worker' check. Since we're *also*\r\nchecksumming in the worker itself, this really isn't needed.\r\n\r\nCombined with several other changes I'm submitting for openQA,\r\nthis should address poo#13526."},"comment":{"url":"https://api.github.com/repos/os-autoinst/os-autoinst/issues/comments/243988332","html_url":"https://github.com/os-autoinst/os-autoinst/pull/574#issuecomment-243988332","issue_url":"https://api.github.com/repos/os-autoinst/os-autoinst/issues/574","id":243988332,"user":{"login":"AdamWill","id":916551,"avatar_url":"https://avatars.githubusercontent.com/u/916551?v=3","gravatar_id":"","url":"https://api.github.com/users/AdamWill","html_url":"https://github.com/AdamWill","followers_url":"https://api.github.com/users/AdamWill/followers","following_url":"https://api.github.com/users/AdamWill/following{/other_user}","gists_url":"https://api.github.com/users/AdamWill/gists{/gist_id}","starred_url":"https://api.github.com/users/AdamWill/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/AdamWill/subscriptions","organizations_url":"https://api.github.com/users/AdamWill/orgs","repos_url":"https://api.github.com/users/AdamWill/repos","events_url":"https://api.github.com/users/AdamWill/events{/privacy}","received_events_url":"https://api.github.com/users/AdamWill/received_events","type":"User","site_admin":false},"created_at":"2016-09-01T06:31:27Z","updated_at":"2016-09-01T06:31:27Z","body":"well, that's kinda arguable. It's currently the general design of openQA that it expects the worker process to stay responsive. Adding this checksum to os-autoinst caused the openQA worker process to be unresponsive for the whole time it takes the checksum to finish. Is that really a bug in openQA, or is it os-autoinst not respecting openQA's expectations?\r\n\r\nIOW, I'd argue that anyone who wants to make os-autoinst do long, blocking operations where it didn't before has the responsibility to somehow make openQA OK with that, rather than just doing it and telling openQA to suck it up..."}}
{"actor":{"display_login":"AdamWill"}}