)]}'
{"tools/ansible-lint-roles-cache.sh":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"c7c94a33af3189f56c394602985b49d249ce846b","unresolved":true,"context_lines":[{"line_number":31,"context_line":"        echo \"Updating repo\""},{"line_number":32,"context_line":"        pushd $repo"},{"line_number":33,"context_line":"        git fetch -a"},{"line_number":34,"context_line":"        git pull"},{"line_number":35,"context_line":"        popd"},{"line_number":36,"context_line":"    fi"},{"line_number":37,"context_line":"    echo \"Done\""}],"source_content_type":"text/x-sh","patch_set":4,"id":"4ee6e769_44d3ec08","line":34,"updated":"2022-08-02 21:33:51.000000000","message":"This step is potentially destructive if anyone ends up using the cache dir for development. That seems unlikely though so should be fine.","commit_id":"27b9226a90f2d7ffbe06dedac808bcb3672b9a0d"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"c46591a85aa19418f73fd7cf26612644334d940b","unresolved":true,"context_lines":[{"line_number":31,"context_line":"        echo \"Updating repo\""},{"line_number":32,"context_line":"        pushd $repo"},{"line_number":33,"context_line":"        git fetch -a"},{"line_number":34,"context_line":"        git pull"},{"line_number":35,"context_line":"        popd"},{"line_number":36,"context_line":"    fi"},{"line_number":37,"context_line":"    echo \"Done\""}],"source_content_type":"text/x-sh","patch_set":4,"id":"225be288_f03fcc8a","line":34,"in_reply_to":"4ee6e769_44d3ec08","updated":"2022-08-02 22:08:48.000000000","message":"i guess if you hand changes it would break and ask you to merge them?  but yeah, the idea really was that \"tox -e linters\" just works locally with less fuss","commit_id":"27b9226a90f2d7ffbe06dedac808bcb3672b9a0d"}],"tox.ini":[{"author":{"_account_id":4146,"name":"Clark Boylan","email":"cboylan@sapwetik.org","username":"cboylan"},"change_message_id":"c7c94a33af3189f56c394602985b49d249ce846b","unresolved":true,"context_lines":[{"line_number":23,"context_line":"     # keep this in sync with what Zuul uses so we don\u0027t allow though"},{"line_number":24,"context_line":"     # things from a later ansible that would actually fail in"},{"line_number":25,"context_line":"     # production."},{"line_number":26,"context_line":"     ansible\u003e\u003d5,\u003c\u003d6"},{"line_number":27,"context_line":"     ansible-lint\u003e\u003d6.3.0,\u003c7.0.0"},{"line_number":28,"context_line":"     hacking\u003e\u003d4.0.0,\u003c5.0.0"},{"line_number":29,"context_line":"     bashate\u003e\u003d0.2"}],"source_content_type":"text/x-properties","patch_set":4,"id":"04a48071_bd7c79c3","line":26,"updated":"2022-08-02 21:33:51.000000000","message":"Related to my similar comment on the zuul-jobs change that bumps up ansible I think we had previously targetted the oldest ansible version zuul supported with the idea that if we target that then newer ansible should also be happy.\n\nThat said OpenDev uses ansible v5 by default so I think we can set this version to v5 here and that is fine (and probably more correct due to the vast vast majority of our users using the default).","commit_id":"27b9226a90f2d7ffbe06dedac808bcb3672b9a0d"},{"author":{"_account_id":5263,"name":"Jeremy Stanley","display_name":"fungi","email":"fungi@yuggoth.org","username":"fungi","status":"missing, presumed fed"},"change_message_id":"b2cc0d487672fca04c31cadee74feeb42fa548f9","unresolved":false,"context_lines":[{"line_number":23,"context_line":"     # keep this in sync with what Zuul uses so we don\u0027t allow though"},{"line_number":24,"context_line":"     # things from a later ansible that would actually fail in"},{"line_number":25,"context_line":"     # production."},{"line_number":26,"context_line":"     ansible\u003e\u003d5,\u003c\u003d6"},{"line_number":27,"context_line":"     ansible-lint\u003e\u003d6.3.0,\u003c7.0.0"},{"line_number":28,"context_line":"     hacking\u003e\u003d4.0.0,\u003c5.0.0"},{"line_number":29,"context_line":"     bashate\u003e\u003d0.2"}],"source_content_type":"text/x-properties","patch_set":4,"id":"e116bd0e_212aa283","line":26,"updated":"2022-08-04 11:48:24.000000000","message":"This should be \u003e\u003d5,\u003c7 right? Otherwise we\u0027ll end up installing PEP 440 versions equivalent to 6.0.0 but not anything later like 6.0.1.","commit_id":"27b9226a90f2d7ffbe06dedac808bcb3672b9a0d"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"c46591a85aa19418f73fd7cf26612644334d940b","unresolved":true,"context_lines":[{"line_number":23,"context_line":"     # keep this in sync with what Zuul uses so we don\u0027t allow though"},{"line_number":24,"context_line":"     # things from a later ansible that would actually fail in"},{"line_number":25,"context_line":"     # production."},{"line_number":26,"context_line":"     ansible\u003e\u003d5,\u003c\u003d6"},{"line_number":27,"context_line":"     ansible-lint\u003e\u003d6.3.0,\u003c7.0.0"},{"line_number":28,"context_line":"     hacking\u003e\u003d4.0.0,\u003c5.0.0"},{"line_number":29,"context_line":"     bashate\u003e\u003d0.2"}],"source_content_type":"text/x-properties","patch_set":4,"id":"7a248ac1_c681c49d","line":26,"in_reply_to":"04a48071_bd7c79c3","updated":"2022-08-02 22:08:48.000000000","message":"right, i was thinking this only runs on opendev where we are at ansible 5 with the production zuul","commit_id":"27b9226a90f2d7ffbe06dedac808bcb3672b9a0d"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"2f09d7e8b985351caaac844059bc7e32601c132b","unresolved":false,"context_lines":[{"line_number":23,"context_line":"     # keep this in sync with what Zuul uses so we don\u0027t allow though"},{"line_number":24,"context_line":"     # things from a later ansible that would actually fail in"},{"line_number":25,"context_line":"     # production."},{"line_number":26,"context_line":"     ansible\u003e\u003d5,\u003c\u003d6"},{"line_number":27,"context_line":"     ansible-lint\u003e\u003d6.3.0,\u003c7.0.0"},{"line_number":28,"context_line":"     hacking\u003e\u003d4.0.0,\u003c5.0.0"},{"line_number":29,"context_line":"     bashate\u003e\u003d0.2"}],"source_content_type":"text/x-properties","patch_set":4,"id":"14499ce6_d13a98b4","line":26,"in_reply_to":"7a248ac1_c681c49d","updated":"2022-08-12 05:02:26.000000000","message":"fixed with fungi\u0027s comment below","commit_id":"27b9226a90f2d7ffbe06dedac808bcb3672b9a0d"},{"author":{"_account_id":7118,"name":"Ian Wienand","email":"iwienand@redhat.com","username":"iwienand"},"change_message_id":"319d8889fd97bb8f1d86d0bcd722849f92f465db","unresolved":false,"context_lines":[{"line_number":23,"context_line":"     # keep this in sync with what Zuul uses so we don\u0027t allow though"},{"line_number":24,"context_line":"     # things from a later ansible that would actually fail in"},{"line_number":25,"context_line":"     # production."},{"line_number":26,"context_line":"     ansible\u003e\u003d5,\u003c\u003d6"},{"line_number":27,"context_line":"     ansible-lint\u003e\u003d6.3.0,\u003c7.0.0"},{"line_number":28,"context_line":"     hacking\u003e\u003d4.0.0,\u003c5.0.0"},{"line_number":29,"context_line":"     bashate\u003e\u003d0.2"}],"source_content_type":"text/x-properties","patch_set":4,"id":"d9b4f032_b55b3060","line":26,"in_reply_to":"e116bd0e_212aa283","updated":"2022-08-11 23:30:16.000000000","message":"Actually I think \u003e\u003d5,\u003c6 was what i was going for.  I think I got confused because the prior was \u003c\u003d2.9, when I think that also wanted just \"\u003c\"","commit_id":"27b9226a90f2d7ffbe06dedac808bcb3672b9a0d"}]}