)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"0d71fe62de23ddf5f9b170420e6369d85d4c9d68","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":19,"id":"ce604043_f489535f","updated":"2022-12-18 08:06:31.000000000","message":"recheck because of unrelated ansible galaxy error","commit_id":"df1117c8492201f949b78a22e3d7728756a2561e"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"668f9931648c7aa31ed5f4f132c4205067ae107e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":21,"id":"8040458c_dc0c8bd4","updated":"2022-12-21 08:00:34.000000000","message":"recheck because of ansible galaxy connectivity issue","commit_id":"af47c6dec3ada4121be7304b5d9583682c0d4852"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"c50ce3482233355d5ea655b720d3c60c3a022b0b","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":23,"id":"a94f60d5_63777851","updated":"2022-12-21 20:29:49.000000000","message":"recheck because of unrelated ci error","commit_id":"700856c1af486ba7ec1c7bc95578c397f9945951"},{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"bc30e147d2dee48dabf76b9ece86247dfd195b2a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":26,"id":"0d92e0e8_cb63f18c","updated":"2023-01-06 17:18:11.000000000","message":"this is pretty complex to review this change deeply, especially that the tests are only executed in -octavia job (no objections against that).\n\nI reviewed it as good as it is possible","commit_id":"b865ec48daa316820411ea5f3d1f0eea896d3c74"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"a3d01221b3609aea4b574c5781a0fbde96671915","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":26,"id":"2e4016e1_006e5849","in_reply_to":"0d92e0e8_cb63f18c","updated":"2023-01-06 19:17:34.000000000","message":"I did not see another way than rewriting it completely. On the pro side the module now supports updates, is easier to extend, has better test coverage, is properly documented, ... 😉","commit_id":"b865ec48daa316820411ea5f3d1f0eea896d3c74"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"f25d9957f8ff61f99d7df92ca53d4ace524d5cfc","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":26,"id":"565b1278_733228ab","in_reply_to":"2e4016e1_006e5849","updated":"2023-01-06 19:21:11.000000000","message":"Thank you very much though for reviewing it anyway 😊","commit_id":"b865ec48daa316820411ea5f3d1f0eea896d3c74"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"8647f30caa320c5921ea5426f245e9a71bc3147e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":27,"id":"c01d7a59_3b81c74b","updated":"2023-01-11 06:27:26.000000000","message":"recheck because of yet another unrelated ansible galaxy issue","commit_id":"407b50c8b2803d27cf05322f4afe2a9a8af86631"},{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"d7f88669f5252602286463ee9c197bb1e9c7387d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":27,"id":"ab0aebba_9ef3cb16","updated":"2023-01-10 18:06:48.000000000","message":"recheck seems irrelevant failure","commit_id":"407b50c8b2803d27cf05322f4afe2a9a8af86631"}],"ci/roles/loadbalancer/tasks/main.yml":[{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"bc30e147d2dee48dabf76b9ece86247dfd195b2a","unresolved":true,"context_lines":[{"line_number":4,"context_line":"    url: \"https://tarballs.openstack.org/octavia/test-images/test-only-amphora-x64-haproxy-ubuntu-bionic.qcow2\""},{"line_number":5,"context_line":"    dest: /tmp/test-only-amphora-x64-haproxy-ubuntu-bionic.qcow2"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"- name: Upload Amphora image for Octavia to test load-balancers"},{"line_number":8,"context_line":"  openstack.cloud.image:"},{"line_number":9,"context_line":"    cloud: \"{{ cloud }}\""},{"line_number":10,"context_line":"    container_format: bare"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"fac64df1_9148f781","line":7,"updated":"2023-01-06 17:18:11.000000000","message":"hmm, and why are were doing this in first place?","commit_id":"b865ec48daa316820411ea5f3d1f0eea896d3c74"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"a3d01221b3609aea4b574c5781a0fbde96671915","unresolved":false,"context_lines":[{"line_number":4,"context_line":"    url: \"https://tarballs.openstack.org/octavia/test-images/test-only-amphora-x64-haproxy-ubuntu-bionic.qcow2\""},{"line_number":5,"context_line":"    dest: /tmp/test-only-amphora-x64-haproxy-ubuntu-bionic.qcow2"},{"line_number":6,"context_line":""},{"line_number":7,"context_line":"- name: Upload Amphora image for Octavia to test load-balancers"},{"line_number":8,"context_line":"  openstack.cloud.image:"},{"line_number":9,"context_line":"    cloud: \"{{ cloud }}\""},{"line_number":10,"context_line":"    container_format: bare"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"98532e48_72ef87cf","line":7,"in_reply_to":"fac64df1_9148f781","updated":"2023-01-06 19:17:34.000000000","message":"Because load balancer creation fails if an image with amphora tag cannot be found :/","commit_id":"b865ec48daa316820411ea5f3d1f0eea896d3c74"},{"author":{"_account_id":27900,"name":"Artem Goncharov","email":"artem.goncharov@gmail.com","username":"gtema"},"change_message_id":"bc30e147d2dee48dabf76b9ece86247dfd195b2a","unresolved":true,"context_lines":[{"line_number":194,"context_line":"# TODO: Replace with appropriate Ansible module once available"},{"line_number":195,"context_line":"- name: Disassociate floating ip from load-balancer"},{"line_number":196,"context_line":"  command: \u003e"},{"line_number":197,"context_line":"    openstack --os-cloud\u003d{{ cloud }} floating ip unset --port {{ load_balancer.floating_ip.floating_ip_address }}"},{"line_number":198,"context_line":""},{"line_number":199,"context_line":"- name: Reassign floating ip address to load-balancer"},{"line_number":200,"context_line":"  openstack.cloud.loadbalancer:"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"768989fd_51951b63","line":197,"updated":"2023-01-06 17:18:11.000000000","message":"this one is cool","commit_id":"b865ec48daa316820411ea5f3d1f0eea896d3c74"},{"author":{"_account_id":32962,"name":"Jakob Meng","email":"code@jakobmeng.de","username":"jakobmeng@web.de"},"change_message_id":"a3d01221b3609aea4b574c5781a0fbde96671915","unresolved":true,"context_lines":[{"line_number":194,"context_line":"# TODO: Replace with appropriate Ansible module once available"},{"line_number":195,"context_line":"- name: Disassociate floating ip from load-balancer"},{"line_number":196,"context_line":"  command: \u003e"},{"line_number":197,"context_line":"    openstack --os-cloud\u003d{{ cloud }} floating ip unset --port {{ load_balancer.floating_ip.floating_ip_address }}"},{"line_number":198,"context_line":""},{"line_number":199,"context_line":"- name: Reassign floating ip address to load-balancer"},{"line_number":200,"context_line":"  openstack.cloud.loadbalancer:"}],"source_content_type":"text/x-yaml","patch_set":26,"id":"5f9aadb6_bdf1af65","line":197,"in_reply_to":"768989fd_51951b63","updated":"2023-01-06 19:17:34.000000000","message":"Not sure what to do here :D","commit_id":"b865ec48daa316820411ea5f3d1f0eea896d3c74"}]}