)]}'
{".pre-commit-config.yaml":[{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"957ae440ccd3598201f92c12e29adf9b100cea33","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"repos:"},{"line_number":3,"context_line":"  - repo: https://github.com/python/black.git"},{"line_number":4,"context_line":"    rev: 20.8b1"},{"line_number":5,"context_line":"    hooks:"},{"line_number":6,"context_line":"      - id: black"},{"line_number":7,"context_line":"        additional_dependencies: [\u0027click\u003c8.1\u0027]"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"6b2efd36_f39fdbc6","line":4,"range":{"start_line":4,"start_character":9,"end_line":4,"end_character":15},"updated":"2022-03-29 09:16:12.000000000","message":"Following https://github.com/pallets/click/issues/2225 and https://github.com/psf/black/issues/2964 \n\nlooks like this is an issue with black, not with click.\n\n\nhttps://github.com/psf/black/issues/2964 is closed now and fix is in \"rev: 22.3.0\".\n\nIMHO, it would be good to bump black to rev: 22.3.0 instead of pinning click here untill and unless we have a reason not to bump black.\n\n\nI have tried locally, bumping black solves the issue:-\n\nhttps://paste.openstack.org/show/bkMBzpden3ZNgIL6bxXw/","commit_id":"a83399c1cbf3dcacffdc76427ffa07ef7f773bd2"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"c9b0745cfef43882bbdc65782eeec75af31b7d21","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"repos:"},{"line_number":3,"context_line":"  - repo: https://github.com/python/black.git"},{"line_number":4,"context_line":"    rev: 20.8b1"},{"line_number":5,"context_line":"    hooks:"},{"line_number":6,"context_line":"      - id: black"},{"line_number":7,"context_line":"        additional_dependencies: [\u0027click\u003c8.1\u0027]"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"f4eb5258_d77c8e16","line":4,"range":{"start_line":4,"start_character":9,"end_line":4,"end_character":15},"in_reply_to":"1543b8df_980089e6","updated":"2022-03-29 09:50:13.000000000","message":"Done","commit_id":"a83399c1cbf3dcacffdc76427ffa07ef7f773bd2"},{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"bb5384a28c65ce13445eda91d880263db3291d9d","unresolved":false,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"repos:"},{"line_number":3,"context_line":"  - repo: https://github.com/python/black.git"},{"line_number":4,"context_line":"    rev: 20.8b1"},{"line_number":5,"context_line":"    hooks:"},{"line_number":6,"context_line":"      - id: black"},{"line_number":7,"context_line":"        additional_dependencies: [\u0027click\u003c8.1\u0027]"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"c9c78f2d_d6ef11c0","line":4,"range":{"start_line":4,"start_character":9,"end_line":4,"end_character":15},"in_reply_to":"1543b8df_980089e6","updated":"2022-03-29 09:51:51.000000000","message":"Thanks Marios,\n\nI am okay with pinning click too if we have a reason and new black break us, based on local testing we are okay with bump - so lets go with bump.","commit_id":"a83399c1cbf3dcacffdc76427ffa07ef7f773bd2"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"0e31c54ebd6ec1addf437323a695bdecfa4a3575","unresolved":true,"context_lines":[{"line_number":1,"context_line":"---"},{"line_number":2,"context_line":"repos:"},{"line_number":3,"context_line":"  - repo: https://github.com/python/black.git"},{"line_number":4,"context_line":"    rev: 20.8b1"},{"line_number":5,"context_line":"    hooks:"},{"line_number":6,"context_line":"      - id: black"},{"line_number":7,"context_line":"        additional_dependencies: [\u0027click\u003c8.1\u0027]"}],"source_content_type":"text/x-yaml","patch_set":5,"id":"1543b8df_980089e6","line":4,"range":{"start_line":4,"start_character":9,"end_line":4,"end_character":15},"in_reply_to":"6b2efd36_f39fdbc6","updated":"2022-03-29 09:47:49.000000000","message":"yes agree... i just expected that bumping black may bring other issues so decided to do the pin\nbut ok lets bump black instead","commit_id":"a83399c1cbf3dcacffdc76427ffa07ef7f773bd2"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"284eabd3995255c3781905a0f35607da52430abb","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"7ca43189_a2d3667b","updated":"2022-03-29 11:57:33.000000000","message":"blocked on https://bugs.launchpad.net/tripleo/+bug/1966914/comments/3","commit_id":"3911d9a1d20af530fdb566bf3398e4a26501f0f3"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"d118246c75a53a925630f69d88f4029c67ad8c7c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"ce8e7660_fa3c77d6","updated":"2022-03-29 15:14:59.000000000","message":"failed at https://zuul.opendev.org/t/openstack/build/c685638851ba42d38c643d01b55679f5 with a different package conflict ;)\n\ntrying to pin but i don\u0027t think it will work for this job - don\u0027t think we use the release files/repos for tox-molecule jobs? pin at https://review.opendev.org/c/openstack/tripleo-quickstart/+/835645 will test it and update here if it works","commit_id":"3911d9a1d20af530fdb566bf3398e4a26501f0f3"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"27538b3d6e8c7a845e83cc2a590bbc2aba953901","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"1b332628_d0bf693e","updated":"2022-03-29 14:54:01.000000000","message":"recheck \n\nif it still hits then we will have to go with a temp workaround for https://bugs.launchpad.net/tripleo/+bug/1966914/","commit_id":"3911d9a1d20af530fdb566bf3398e4a26501f0f3"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"1f92ff58492fd38d0819a674ff3a63047cbf2e6d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"b54b639e_71a7af1e","updated":"2022-03-29 15:45:02.000000000","message":"rebased onto 835648 so should be able to go through now","commit_id":"806a7bc1fa50fd94194d56601355d1c7b9164450"}],"zuul.d/multinode-jobs.yaml":[{"author":{"_account_id":29775,"name":"Sandeep Yadav","email":"sandyada@redhat.com","username":"sandeepyadav93"},"change_message_id":"10f2b73f932ce96f29059fccbafaa948be5f6584","unresolved":true,"context_lines":[{"line_number":124,"context_line":"              - ^tripleo_ansible/roles/.*/meta.*"},{"line_number":125,"context_line":"              - ^tripleo_ansible/roles/test_deps/.*"},{"line_number":126,"context_line":"              - ^zuul.d/base.yaml"},{"line_number":127,"context_line":"              - ^zuul.d/molecule.yaml"},{"line_number":128,"context_line":"            vars: \u0026multi_consumer_vars"},{"line_number":129,"context_line":"              consumer_job: true"},{"line_number":130,"context_line":"              build_container_images: false"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"35f178c2_0d638d75","line":127,"updated":"2022-03-29 11:49:19.000000000","message":"content-provider have tripleo-ci-base-standard as its parent in its parent-child heirarchy. Which have ^.pre-commit-config.yaml$ in irrelevant-files.\n\nhttps://opendev.org/openstack/tripleo-ci/src/branch/master/zuul.d/base-upstream.yaml#L95\n~~~\n^.pre-commit-config.yaml$\n~~~\n\n\nInstead of overriding content-provider irrelevant-files var in project-template, I wonder adding ^.pre-commit-config.yaml$ under tripleo-ci-centos-8-containers-multinode: anchor (here) would be a better idea.\n\n\nBut not blocking this change as this is breaking CI, we can revert changes in this file afterwards.","commit_id":"3911d9a1d20af530fdb566bf3398e4a26501f0f3"},{"author":{"_account_id":8449,"name":"Marios Andreou","email":"marios.andreou@gmail.com","username":"marios"},"change_message_id":"0b99a8e5f326cf3ed28bf1591acd4f1e7f89d9e4","unresolved":false,"context_lines":[{"line_number":124,"context_line":"              - ^tripleo_ansible/roles/.*/meta.*"},{"line_number":125,"context_line":"              - ^tripleo_ansible/roles/test_deps/.*"},{"line_number":126,"context_line":"              - ^zuul.d/base.yaml"},{"line_number":127,"context_line":"              - ^zuul.d/molecule.yaml"},{"line_number":128,"context_line":"            vars: \u0026multi_consumer_vars"},{"line_number":129,"context_line":"              consumer_job: true"},{"line_number":130,"context_line":"              build_container_images: false"}],"source_content_type":"text/x-yaml","patch_set":6,"id":"62ea0c8d_680e327c","line":127,"in_reply_to":"35f178c2_0d638d75","updated":"2022-03-29 11:53:23.000000000","message":"ack as discussed at https://review.opendev.org/c/openstack/tripleo-ci/+/835536/3#message-58fdab284a38751648f7d71af90e254be8c72b94 \n\nagree lets work on that once we unblock the gate","commit_id":"3911d9a1d20af530fdb566bf3398e4a26501f0f3"}]}
