)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"a22185769f0f7538f0f5a4f8ca9fa99bee6c303f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"1a2418f2_ec3af27b","updated":"2024-08-16 13:31:59.000000000","message":"Waiting until Wednesday next week before approving because we can then do a test release to confirm this works as planned","commit_id":"45428f1f9f485d810e3d77db89c13931eaba9029"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"05a746936055a4b1f993308db487fb75a7daeaf5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"4895c4cb_719b7f83","updated":"2024-08-21 15:52:19.000000000","message":"the update-constraints job may be running on a different cloud than the checks on the generated reqs change, so I\u0027m not sure doing any pypi actions on the former is the way to go","commit_id":"45428f1f9f485d810e3d77db89c13931eaba9029"}],"playbooks/proposal/propose-update-constraints.yaml":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"696e93d305403038f0573747b7b0ba4403bb7740","unresolved":true,"context_lines":[{"line_number":2,"context_line":"  tasks:"},{"line_number":3,"context_line":"    - name: Pause in order to allow pypi to catch up"},{"line_number":4,"context_line":"      ansible.builtin.pause:"},{"line_number":5,"context_line":"        minutes: 10"},{"line_number":6,"context_line":"        prompt: Pausing for 10 minutes in order to allow pypi to catch up"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"    - name: Propose constraints update"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"5826736c_1107537c","line":5,"updated":"2024-08-21 15:23:49.000000000","message":"Might be worth mentioning the reason for a 10 minute timeout is the cache TTL on pypi index files is 10 minutes.\n\nA long time ago there was a way to force pypi to refresh files using a PURGE http request against the url. I think this job is going through our caching \"mirrors\" so that isn\u0027t sufficient, but maybe these jobs should not use the mirrors and instead issue PURGEs for things they care about before doing the work?","commit_id":"45428f1f9f485d810e3d77db89c13931eaba9029"},{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"05a746936055a4b1f993308db487fb75a7daeaf5","unresolved":true,"context_lines":[{"line_number":2,"context_line":"  tasks:"},{"line_number":3,"context_line":"    - name: Pause in order to allow pypi to catch up"},{"line_number":4,"context_line":"      ansible.builtin.pause:"},{"line_number":5,"context_line":"        minutes: 10"},{"line_number":6,"context_line":"        prompt: Pausing for 10 minutes in order to allow pypi to catch up"},{"line_number":7,"context_line":""},{"line_number":8,"context_line":"    - name: Propose constraints update"}],"source_content_type":"text/x-yaml","patch_set":1,"id":"11b6519f_4074832f","line":5,"in_reply_to":"5826736c_1107537c","updated":"2024-08-21 15:52:19.000000000","message":"I wasn\u0027t aware that there is an actual TTL of 10 minutes, I simply selected a value that sounded reasonable to me given that most of the time only the first jobs in a buildset were failing, making a compromise between not having too much delay but also avoiding those failures, with the intention of possibly increasing it later if it turns out to still not be high enough.\n\nI\u0027m not sure making the jobs more complicated is a good idea, mostly because they will usually not be aware of whether they are running because of a recent release or for some other patch. And given that the resulting patch needs manual review anyway, delaying its creation by 10 minutes or possibly even an hour didn\u0027t sound too bad to me.","commit_id":"45428f1f9f485d810e3d77db89c13931eaba9029"}]}