{"push_id":2608573122,"size":10,"distinct_size":2,"ref":"refs/heads/auto","head":"1f2c7f275b68cc818e137500f733ad06120762c4","before":"f31087137ed2ce23b8cd886f2ab37b0b27cbfc61","commits":[{"sha":"938055392fd455027a69398c441b992ae521aa87","author":{"name":"Philip Withnall","email":"29c2a2ac2a854b9cf047275aa9de346d096be2e0@endlessm.com"},"message":"lib/repo-pull: Support retrying requests on transient network errors\n\nAllow network requests to be re-queued if they failed with a transient\nerror, such as a socket timeout. Retry each request up to a limit\n(default: 5), and only then fail the entire pull and propagate the error\nto the caller.\n\nAdd a new ostree_repo_pull_with_options() option, n-network-retries, to\ncontrol the number of retries (including setting it back to the old\ndefault of 0, if the caller wants).\n\nCurrently, retries are not supported for FetchDeltaSuperData requests,\nas they are not queued. Once they are queued, adding support for retries\nshould be trivial. A FIXME comment has been left for this.\n\nSigned-off-by: Philip Withnall <withnall@endlessm.com>\n\nCloses: #1594\nApproved by: jlebon","distinct":false,"url":"https://api.github.com/repos/ostreedev/ostree/commits/938055392fd455027a69398c441b992ae521aa87"},{"sha":"224f3cdd24e81de6f490fdcd93b114bbb7d4ab8a","author":{"name":"Philip Withnall","email":"29c2a2ac2a854b9cf047275aa9de346d096be2e0@endlessm.com"},"message":"lib/fetcher-soup: Map more SoupStatus codes to known GIOErrors\n\nThis allows the retry code in ostree-repo-pull.c to recover from (for\nexample) timeouts at the libsoup layer in the stack, as well as from the\nGSocket layer in the stack.\n\nSigned-off-by: Philip Withnall <withnall@endlessm.com>\n\nCloses: #1594\nApproved by: jlebon","distinct":false,"url":"https://api.github.com/repos/ostreedev/ostree/commits/224f3cdd24e81de6f490fdcd93b114bbb7d4ab8a"},{"sha":"97c348d65b989993cdf995e37a28936faa4ec1b2","author":{"name":"Philip Withnall","email":"29c2a2ac2a854b9cf047275aa9de346d096be2e0@endlessm.com"},"message":"ostree/trivial-httpd: Add --random-408s command line option\n\nThis is exactly like the --random-500s option, except that it will cause\nerror 408 (request timeout) to be returned, rather than error 500\n(internal server error).\n\nThis will be used in a following commit to test pull behaviour when\ntimeouts occur.\n\nSigned-off-by: Philip Withnall <withnall@endlessm.com>\n\nCloses: #1594\nApproved by: jlebon","distinct":false,"url":"https://api.github.com/repos/ostreedev/ostree/commits/97c348d65b989993cdf995e37a28936faa4ec1b2"},{"sha":"c9619a490490a8ed29ea002a2f358bbebfb80ef4","author":{"name":"Philip Withnall","email":"29c2a2ac2a854b9cf047275aa9de346d096be2e0@endlessm.com"},"message":"tests: Test pull behaviour when network timeouts occur\n\nExtend test-pull-repeated.sh to test error 408 as well as error 500, to\nensure that the new retry-on-network-timeout code in ostree-repo-pull.c\ncorrectly retries.\n\nRather than the 200 iterations needed for the error 500 tests, only do 5\niterations. The pull code internally does 5 retries (by default), which\nmeans a full iteration count of 25. That seems to be sufficient to make\nthe tests reliably pass, in my testing — we can always bump it up to 200\n/ 5 = 40 in future if needed (to put it in parity with the error 500\ntests).\n\nSigned-off-by: Philip Withnall <withnall@endlessm.com>\n\nCloses: #1594\nApproved by: jlebon","distinct":false,"url":"https://api.github.com/repos/ostreedev/ostree/commits/c9619a490490a8ed29ea002a2f358bbebfb80ef4"},{"sha":"78f40136dbf8411a66ce3f6d8263d8e72d951163","author":{"name":"Philip Withnall","email":"29c2a2ac2a854b9cf047275aa9de346d096be2e0@endlessm.com"},"message":"lib/repo-pull: Add some missing assertions for progress statistics\n\nVarious of the counters already have assertions like this; add some more\nfor total paranoia.\n\nSigned-off-by: Philip Withnall <withnall@endlessm.com>\n\nCloses: #1594\nApproved by: jlebon","distinct":false,"url":"https://api.github.com/repos/ostreedev/ostree/commits/78f40136dbf8411a66ce3f6d8263d8e72d951163"},{"sha":"197644c406deec76a6567e6639ebd6e06dbc1013","author":{"name":"Philip Withnall","email":"29c2a2ac2a854b9cf047275aa9de346d096be2e0@endlessm.com"},"message":"lib/fetcher: Factor out HTTP status code handling from soup and curl\n\nUse the same G_IO_ERROR_* values for HTTP status codes in both fetchers.\nThe libsoup fetcher still handles a few more internal error codes than\nthe libcurl one; this could be built on in future.\n\nSigned-off-by: Philip Withnall <withnall@endlessm.com>\n\nCloses: #1594\nApproved by: jlebon","distinct":false,"url":"https://api.github.com/repos/ostreedev/ostree/commits/197644c406deec76a6567e6639ebd6e06dbc1013"},{"sha":"f342e66c116b8731e161b03fb1abcd4504f7098e","author":{"name":"Philip Withnall","email":"29c2a2ac2a854b9cf047275aa9de346d096be2e0@endlessm.com"},"message":"lib/repo-pull: Support queuing delta superblock requests\n\nJust like all the other requests made for delta parts and objects by the\npull code, use a queue for delta superblocks. Currently this doesn’t do\nany prioritisation or retries after transient failures, but it could do\nin future.\n\nThis means that delta superblocks are now subject to the parallel\nrequest limit in the fetcher, which was a problem highlighted here:\nhttps://github.com/ostreedev/ostree/pull/1453#discussion_r168321706.\n\nSigned-off-by: Philip Withnall <withnall@endlessm.com>\n\nCloses: #1600\nApproved by: jlebon","distinct":false,"url":"https://api.github.com/repos/ostreedev/ostree/commits/f342e66c116b8731e161b03fb1abcd4504f7098e"},{"sha":"bf1af263d67c0f7e49355088528f02d78fa68e90","author":{"name":"Philip Withnall","email":"29c2a2ac2a854b9cf047275aa9de346d096be2e0@endlessm.com"},"message":"lib/repo-pull: Support retries for delta superblocks\n\nUse the recently introduced architecture for retrying network requests\non transient failure to do the same for delta superblock requests, now\nthat they’re queued.\n\nSigned-off-by: Philip Withnall <withnall@endlessm.com>\n\nCloses: #1600\nApproved by: jlebon","distinct":false,"url":"https://api.github.com/repos/ostreedev/ostree/commits/bf1af263d67c0f7e49355088528f02d78fa68e90"},{"sha":"8e98fce183511db8959e7fe2ed58ec1847229226","author":{"name":"Matthew Leeds","email":"424ec9dbc7f06560bd9f0866b438a24a0e995bb7@endlessm.com"},"message":"ci: Bump flatpak build to F28\n\nCloses: #1607\nApproved by: cgwalters","distinct":true,"url":"https://api.github.com/repos/ostreedev/ostree/commits/8e98fce183511db8959e7fe2ed58ec1847229226"},{"sha":"1f2c7f275b68cc818e137500f733ad06120762c4","author":{"name":"Matthew Leeds","email":"424ec9dbc7f06560bd9f0866b438a24a0e995bb7@endlessm.com"},"message":"ci: Use master branch as flatpak tag\n\nCloses: #1607\nApproved by: cgwalters","distinct":true,"url":"https://api.github.com/repos/ostreedev/ostree/commits/1f2c7f275b68cc818e137500f733ad06120762c4"}]}
{"actor":{"display_login":"rh-atomic-bot"}}