{"action":"created","issue":{"url":"https://api.github.com/repos/letsencrypt/boulder/issues/3509","repository_url":"https://api.github.com/repos/letsencrypt/boulder","labels_url":"https://api.github.com/repos/letsencrypt/boulder/issues/3509/labels{/name}","comments_url":"https://api.github.com/repos/letsencrypt/boulder/issues/3509/comments","events_url":"https://api.github.com/repos/letsencrypt/boulder/issues/3509/events","html_url":"https://github.com/letsencrypt/boulder/pull/3509","id":301602655,"number":3509,"title":"Run load-generator against V1 and V2 API in CI.","user":{"login":"cpu","id":292650,"avatar_url":"https://avatars1.githubusercontent.com/u/292650?v=4","gravatar_id":"","url":"https://api.github.com/users/cpu","html_url":"https://github.com/cpu","followers_url":"https://api.github.com/users/cpu/followers","following_url":"https://api.github.com/users/cpu/following{/other_user}","gists_url":"https://api.github.com/users/cpu/gists{/gist_id}","starred_url":"https://api.github.com/users/cpu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cpu/subscriptions","organizations_url":"https://api.github.com/users/cpu/orgs","repos_url":"https://api.github.com/users/cpu/repos","events_url":"https://api.github.com/users/cpu/events{/privacy}","received_events_url":"https://api.github.com/users/cpu/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":{"login":"cpu","id":292650,"avatar_url":"https://avatars1.githubusercontent.com/u/292650?v=4","gravatar_id":"","url":"https://api.github.com/users/cpu","html_url":"https://github.com/cpu","followers_url":"https://api.github.com/users/cpu/followers","following_url":"https://api.github.com/users/cpu/following{/other_user}","gists_url":"https://api.github.com/users/cpu/gists{/gist_id}","starred_url":"https://api.github.com/users/cpu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cpu/subscriptions","organizations_url":"https://api.github.com/users/cpu/orgs","repos_url":"https://api.github.com/users/cpu/repos","events_url":"https://api.github.com/users/cpu/events{/privacy}","received_events_url":"https://api.github.com/users/cpu/received_events","type":"User","site_admin":false},"assignees":[{"login":"cpu","id":292650,"avatar_url":"https://avatars1.githubusercontent.com/u/292650?v=4","gravatar_id":"","url":"https://api.github.com/users/cpu","html_url":"https://github.com/cpu","followers_url":"https://api.github.com/users/cpu/followers","following_url":"https://api.github.com/users/cpu/following{/other_user}","gists_url":"https://api.github.com/users/cpu/gists{/gist_id}","starred_url":"https://api.github.com/users/cpu/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/cpu/subscriptions","organizations_url":"https://api.github.com/users/cpu/orgs","repos_url":"https://api.github.com/users/cpu/repos","events_url":"https://api.github.com/users/cpu/events{/privacy}","received_events_url":"https://api.github.com/users/cpu/received_events","type":"User","site_admin":false}],"milestone":null,"comments":0,"created_at":"2018-03-01T23:46:36Z","updated_at":"2018-03-02T18:53:31Z","closed_at":null,"author_association":"MEMBER","pull_request":{"url":"https://api.github.com/repos/letsencrypt/boulder/pulls/3509","html_url":"https://github.com/letsencrypt/boulder/pull/3509","diff_url":"https://github.com/letsencrypt/boulder/pull/3509.diff","patch_url":"https://github.com/letsencrypt/boulder/pull/3509.patch"},"body":"This commit adds short 15s runs of the load generator against the V1 and\r\nV2 APIs during the three integration test runs (v1 config, v1\r\nconfig-next, and v2). 15s was selected because 30s caused too much\r\noutput and the build log to be truncated.\r\n\r\nThe output latency from the load generator runs is validated to ensure\r\nthat all requests succeeded. We don't presently generate latency graphs\r\nwith the data.\r\n\r\nA fix for a race condition in the load-generator code itself related to\r\nHTTP status code tracking is included in this commit.\r\n\r\nThe pending orders rate limit also needed to be adjusted to keep the\r\nload-generator from failing requests after hitting 429s. \r\n\r\n**Note: CI will fail with this PR until https://github.com/letsencrypt/boulder/pull/3508 is merged. The combination of race detector builds and the load generator shakes out this RA race condition with regularity.**"},"comment":{"url":"https://api.github.com/repos/letsencrypt/boulder/issues/comments/370017492","html_url":"https://github.com/letsencrypt/boulder/pull/3509#issuecomment-370017492","issue_url":"https://api.github.com/repos/letsencrypt/boulder/issues/3509","id":370017492,"user":{"login":"rolandshoemaker","id":5080674,"avatar_url":"https://avatars3.githubusercontent.com/u/5080674?v=4","gravatar_id":"","url":"https://api.github.com/users/rolandshoemaker","html_url":"https://github.com/rolandshoemaker","followers_url":"https://api.github.com/users/rolandshoemaker/followers","following_url":"https://api.github.com/users/rolandshoemaker/following{/other_user}","gists_url":"https://api.github.com/users/rolandshoemaker/gists{/gist_id}","starred_url":"https://api.github.com/users/rolandshoemaker/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rolandshoemaker/subscriptions","organizations_url":"https://api.github.com/users/rolandshoemaker/orgs","repos_url":"https://api.github.com/users/rolandshoemaker/repos","events_url":"https://api.github.com/users/rolandshoemaker/events{/privacy}","received_events_url":"https://api.github.com/users/rolandshoemaker/received_events","type":"User","site_admin":false},"created_at":"2018-03-02T18:53:31Z","updated_at":"2018-03-02T18:53:31Z","author_association":"CONTRIBUTOR","body":"Conflicts."}}
{"actor":{"display_login":"rolandshoemaker"}}