)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":6914,"name":"Tatiana Ovchinnikova","email":"t.v.ovtchinnikova@gmail.com","username":"tmazur"},"change_message_id":"e6ffa70d758032b12d692a996d2525f67937988c","unresolved":true,"context_lines":[{"line_number":13,"context_line":"this patch is introducing changes to always pin the search bar"},{"line_number":14,"context_line":"and the sidebar at the top of the page."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Bug: https://bugs.launchpad.net/horizon/+bug/1928678"},{"line_number":17,"context_line":"Change-Id: I9186a4fa1dd2a16f75464ff3bb1c0c9b76a12cc7"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"bf9feb1f_02485882","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":52},"updated":"2021-05-19 14:57:20.000000000","message":"Please use:\n\nCloses-Bug: #1928678","commit_id":"3d345ffc1e479ed35dbe82076824e047ee368110"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"5b4c6317b77ecc01682de4afb29ba07c4dbeb17f","unresolved":false,"context_lines":[{"line_number":13,"context_line":"this patch is introducing changes to always pin the search bar"},{"line_number":14,"context_line":"and the sidebar at the top of the page."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Bug: https://bugs.launchpad.net/horizon/+bug/1928678"},{"line_number":17,"context_line":"Change-Id: I9186a4fa1dd2a16f75464ff3bb1c0c9b76a12cc7"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"74f830da_43803843","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":52},"in_reply_to":"b2ae227c_e3aab2cc","updated":"2022-12-07 23:45:07.000000000","message":"Done","commit_id":"3d345ffc1e479ed35dbe82076824e047ee368110"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"2463bbb841048ac372a0fd72864ad842caf82ca2","unresolved":true,"context_lines":[{"line_number":13,"context_line":"this patch is introducing changes to always pin the search bar"},{"line_number":14,"context_line":"and the sidebar at the top of the page."},{"line_number":15,"context_line":""},{"line_number":16,"context_line":"Bug: https://bugs.launchpad.net/horizon/+bug/1928678"},{"line_number":17,"context_line":"Change-Id: I9186a4fa1dd2a16f75464ff3bb1c0c9b76a12cc7"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"b2ae227c_e3aab2cc","line":16,"range":{"start_line":16,"start_character":0,"end_line":16,"end_character":52},"in_reply_to":"bf9feb1f_02485882","updated":"2021-05-19 17:07:57.000000000","message":"My bad, sorry. Fixed.","commit_id":"3d345ffc1e479ed35dbe82076824e047ee368110"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"eae706ee072551204fa4f3245efdba1def25a7f5","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"cf43c2f2_e80d6150","updated":"2022-01-26 15:33:14.000000000","message":"\u003e Patch Set 6:\n\u003e \n\u003e \u003e Patch Set 6: Code-Review-1\n\u003e \u003e \n\u003e \u003e (1 comment)\n\u003e \u003e \n\u003e \u003e Thank you for the patch! It works good, however the search bar becomes too wide and users are not able to see the cancel icon.\n\u003e \n\u003e I am not sure I follow. Could you show me same example?\n\nHello guys, \nDo we have an update on this patch?","commit_id":"edabb0670dcf10e70a213955590ce3fc017e7b91"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"ddac8202fb5fcd3ebd8e08618a04b3edc610bc2c","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"f5dff693_91e47beb","updated":"2022-03-21 12:59:18.000000000","message":"Hello Tatiana, could you provide some feedback here? I have posted a question to your suggestion, but I have not heard back since.","commit_id":"edabb0670dcf10e70a213955590ce3fc017e7b91"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"3173867847cf4379ad9c0a5ad80cfed1d63a7c50","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"ab8abae6_87b26517","updated":"2022-12-01 15:46:47.000000000","message":"Hello guys, \nCan we have a feedback here? To enable this patch to move on.","commit_id":"edabb0670dcf10e70a213955590ce3fc017e7b91"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"767375b80f16cb823fca818766be16ceee5c35b6","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"07439339_9a106468","updated":"2022-12-02 15:04:13.000000000","message":"Are the errors expected? It does not seem to be related to the change itself.","commit_id":"69952ffbe25c756fbb5001d47ba35f157e18816e"},{"author":{"_account_id":6914,"name":"Tatiana Ovchinnikova","email":"t.v.ovtchinnikova@gmail.com","username":"tmazur"},"change_message_id":"79cb02e329cef0896410008a0943b8dc5cd6bf6d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"1fa6c071_57c5437b","updated":"2022-12-02 19:14:29.000000000","message":"It is the coverage that fails now. You\u0027ve added if-else condition, so you will need to add a test that will run through this condition.","commit_id":"69952ffbe25c756fbb5001d47ba35f157e18816e"},{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"ab47cec99ddf2203c74510746c57a1158ec3c278","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"d4609f4f_42b33249","updated":"2022-12-02 11:23:34.000000000","message":"Thanks for update.","commit_id":"69952ffbe25c756fbb5001d47ba35f157e18816e"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"a556b9835b39ddd247cf1da625f7d5c27741a59e","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"bf40660b_dd4cbbae","updated":"2022-12-02 12:44:00.000000000","message":"recheck","commit_id":"69952ffbe25c756fbb5001d47ba35f157e18816e"},{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"030496c2510224f91e1549bf6eb7fb164e482225","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"2cb65e31_9fd049e1","in_reply_to":"07439339_9a106468","updated":"2022-12-02 15:21:35.000000000","message":"yeah looks like you add a unit test for this change [1]\n\n[1] https://zuul.opendev.org/t/openstack/build/fc345af33d5f4b5caaa9998dcd209c20/log/job-output.txt#3248","commit_id":"69952ffbe25c756fbb5001d47ba35f157e18816e"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"35adc7ee21d25fc9c931b1f62bbee8951dff1a70","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"aeecc91e_c72d10e8","in_reply_to":"1fa6c071_57c5437b","updated":"2022-12-06 12:55:26.000000000","message":"Are you sure? I am executing the tests, even without my code, and they do not pass. I testes quite a few different commits (previous commits), and none of them pass.","commit_id":"69952ffbe25c756fbb5001d47ba35f157e18816e"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"d80cf67ff95ef343d7c7564d9118bd403b45639f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"c314ffe0_a345fbd4","in_reply_to":"2cb65e31_9fd049e1","updated":"2022-12-02 15:27:46.000000000","message":"you mean, I need to add unit tests for the JS code that I am adding here?","commit_id":"69952ffbe25c756fbb5001d47ba35f157e18816e"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"5b4c6317b77ecc01682de4afb29ba07c4dbeb17f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"c2dc5ac6_ec90ed0a","in_reply_to":"7832cadf_debe0b3d","updated":"2022-12-07 23:45:07.000000000","message":"Done","commit_id":"69952ffbe25c756fbb5001d47ba35f157e18816e"},{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"11072298fabafe7f17e673b89397271813edf115","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":7,"id":"de91ce7f_ff3dc302","in_reply_to":"c314ffe0_a345fbd4","updated":"2022-12-02 15:39:07.000000000","message":"yes in this filehttps://github.com/openstack/horizon/blob/master/horizon/static/framework/widgets/magic-search/magic-search.controller.spec.js","commit_id":"69952ffbe25c756fbb5001d47ba35f157e18816e"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"e9b0e728aa293c993b2b8825225cc649e0d74bab","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":7,"id":"7832cadf_debe0b3d","in_reply_to":"de91ce7f_ff3dc302","updated":"2022-12-02 17:30:47.000000000","message":"I am a bit confused. Even when I added the unit test. It kept failing. I then removed my commit and execute the tests with latest commit on main branch. It is also failing. Am I missing something?","commit_id":"69952ffbe25c756fbb5001d47ba35f157e18816e"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"ba53a4a15f5dbb500218f0ffcabbd425e869f961","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":13,"id":"9b6d27fd_1e190664","updated":"2022-12-07 01:14:47.000000000","message":"I think that the tests that fail when I run locally here is something to do with some dependencies.\n\nCan someone help me with these tests? It seems that I am missing some basic detail. I am not sure how to call the method I want to test, and how to create the mocks in these JS libraries.","commit_id":"778060a0872d4ba0621a367c9c4617ef03fee5cc"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"865e35d867a940dda8f1ba429b29ed13c0a30fdf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"76ab7ace_6c7d767b","updated":"2022-12-08 10:41:34.000000000","message":"Everything seems to be ready now guys. Can you review the code?","commit_id":"01f42c0fed1a2283caa8c8cb8cbaba65442c3fb3"},{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"277d485cc7af9644bad5f54dd9ba2f1aac85846a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"c6889f5b_eadac892","updated":"2022-12-09 11:41:02.000000000","message":"Hi, I don\u0027t see any changes on the UI side after applying your patch.\nCould check again and confirm if your patch is really working or not?","commit_id":"01f42c0fed1a2283caa8c8cb8cbaba65442c3fb3"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"964557dd00bc07c1ce9df65c7ad3d51a30221f5a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":20,"id":"ce3b087f_61b4db10","in_reply_to":"c6889f5b_eadac892","updated":"2022-12-09 11:44:09.000000000","message":"Fixed. It was an issue that I created when I changed the code to enable unit testing it.","commit_id":"01f42c0fed1a2283caa8c8cb8cbaba65442c3fb3"},{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"725d6e54e235e4cc0db85ea4e57a766b29f0085d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":21,"id":"63a1c7e8_e1360649","updated":"2022-12-12 15:51:54.000000000","message":"Thanks, LGTM. working fine tested with devstack env.","commit_id":"ba1e7ddc9b0b6a604c4f8f52d64e7f6c57142e52"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"bec31de3a8bf5f7554c66a8c7643a340cdb5d3f1","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":21,"id":"8a85b44a_1dfb891e","updated":"2022-12-09 16:44:52.000000000","message":"recheck","commit_id":"ba1e7ddc9b0b6a604c4f8f52d64e7f6c57142e52"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"e31d9503b70eb43858c07f5383918b31bb7f8e1a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":21,"id":"e915fc70_de44d383","updated":"2022-12-09 12:57:58.000000000","message":"recheck","commit_id":"ba1e7ddc9b0b6a604c4f8f52d64e7f6c57142e52"},{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"00b5a2bfafb0ec54847404753c4e83f54b3421cd","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":21,"id":"3059370e_4166f7bc","updated":"2022-12-12 11:03:29.000000000","message":"recheck, random failure.","commit_id":"ba1e7ddc9b0b6a604c4f8f52d64e7f6c57142e52"}],"horizon/static/framework/widgets/magic-search/magic-search.controller.js":[{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"6c2341a64a4759caea987f21100d2aab85b6d0f2","unresolved":true,"context_lines":[{"line_number":98,"context_line":"    });"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"    function executeFloatMenu(jqueryElement, fixAfterScroll, newTopPosition) {"},{"line_number":101,"context_line":"      jqueryElement.css(\"z-index\", 999);"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"      var scrollAmount \u003d $(document).scrollTop();"},{"line_number":104,"context_line":""}],"source_content_type":"text/javascript","patch_set":4,"id":"15084a03_6ea2c3e3","line":101,"range":{"start_line":101,"start_character":6,"end_line":101,"end_character":19},"updated":"2021-05-24 09:58:44.000000000","message":"Could you use \u0027jquery_element\u0027 instead as the same conventions followed in horizon code so far?\nPlease do same changes at other places as well.","commit_id":"aebcdf476433b47d233a8dcd8b79285877daa0ec"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"db4643dc4b39141ec3a5c08ed0ba6eb4757d52dd","unresolved":true,"context_lines":[{"line_number":98,"context_line":"    });"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"    function executeFloatMenu(jqueryElement, fixAfterScroll, newTopPosition) {"},{"line_number":101,"context_line":"      jqueryElement.css(\"z-index\", 999);"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"      var scrollAmount \u003d $(document).scrollTop();"},{"line_number":104,"context_line":""}],"source_content_type":"text/javascript","patch_set":4,"id":"504d8b98_2b178be4","line":101,"range":{"start_line":101,"start_character":6,"end_line":101,"end_character":19},"in_reply_to":"15084a03_6ea2c3e3","updated":"2021-05-24 11:56:09.000000000","message":"I did this way initially, but `horizon-nodejs10-run-lint` breaks. Therefore, I needed to change to the cammelCase approach.","commit_id":"aebcdf476433b47d233a8dcd8b79285877daa0ec"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"5b4c6317b77ecc01682de4afb29ba07c4dbeb17f","unresolved":false,"context_lines":[{"line_number":98,"context_line":"    });"},{"line_number":99,"context_line":""},{"line_number":100,"context_line":"    function executeFloatMenu(jqueryElement, fixAfterScroll, newTopPosition) {"},{"line_number":101,"context_line":"      jqueryElement.css(\"z-index\", 999);"},{"line_number":102,"context_line":""},{"line_number":103,"context_line":"      var scrollAmount \u003d $(document).scrollTop();"},{"line_number":104,"context_line":""}],"source_content_type":"text/javascript","patch_set":4,"id":"e814c28e_c0173ce1","line":101,"range":{"start_line":101,"start_character":6,"end_line":101,"end_character":19},"in_reply_to":"504d8b98_2b178be4","updated":"2022-12-07 23:45:07.000000000","message":"Done","commit_id":"aebcdf476433b47d233a8dcd8b79285877daa0ec"},{"author":{"_account_id":6914,"name":"Tatiana Ovchinnikova","email":"t.v.ovtchinnikova@gmail.com","username":"tmazur"},"change_message_id":"0580d81fbd74b5346a7b609d42b5825be347421e","unresolved":true,"context_lines":[{"line_number":103,"context_line":"      var scrollAmount \u003d $(document).scrollTop();"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"      if (scrollAmount \u003e fixafterScroll) {"},{"line_number":106,"context_line":"        jqueryElement.width(\u002782.5%\u0027);"},{"line_number":107,"context_line":"        jqueryElement.css(\u0027position\u0027, \u0027fixed\u0027);"},{"line_number":108,"context_line":"        jqueryElement.css(\"top\", newTopPosition);"},{"line_number":109,"context_line":"      } else {"}],"source_content_type":"text/javascript","patch_set":6,"id":"e2975e50_a2007392","line":106,"updated":"2021-09-15 17:02:22.000000000","message":"Could you please change it to fit the cancel icon and buttons?","commit_id":"edabb0670dcf10e70a213955590ce3fc017e7b91"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"cdf32b6385f04c3db99bd3dfbcf61f3ca3b9f185","unresolved":false,"context_lines":[{"line_number":103,"context_line":"      var scrollAmount \u003d $(document).scrollTop();"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"      if (scrollAmount \u003e fixafterScroll) {"},{"line_number":106,"context_line":"        jqueryElement.width(\u002782.5%\u0027);"},{"line_number":107,"context_line":"        jqueryElement.css(\u0027position\u0027, \u0027fixed\u0027);"},{"line_number":108,"context_line":"        jqueryElement.css(\"top\", newTopPosition);"},{"line_number":109,"context_line":"      } else {"}],"source_content_type":"text/javascript","patch_set":6,"id":"16b7109f_87e06adf","line":106,"in_reply_to":"5fe7acae_972603b4","updated":"2022-12-02 11:12:48.000000000","message":"I see. Done. Thanks!","commit_id":"edabb0670dcf10e70a213955590ce3fc017e7b91"},{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"facc7ddbfd6353214c7232d06de0bff10df26d58","unresolved":true,"context_lines":[{"line_number":103,"context_line":"      var scrollAmount \u003d $(document).scrollTop();"},{"line_number":104,"context_line":""},{"line_number":105,"context_line":"      if (scrollAmount \u003e fixafterScroll) {"},{"line_number":106,"context_line":"        jqueryElement.width(\u002782.5%\u0027);"},{"line_number":107,"context_line":"        jqueryElement.css(\u0027position\u0027, \u0027fixed\u0027);"},{"line_number":108,"context_line":"        jqueryElement.css(\"top\", newTopPosition);"},{"line_number":109,"context_line":"      } else {"}],"source_content_type":"text/javascript","patch_set":6,"id":"5fe7acae_972603b4","line":106,"in_reply_to":"e2975e50_a2007392","updated":"2022-12-02 09:22:50.000000000","message":"Hi Rafael, I guess what Tatiana means is right now after applying your patch cancel icon and buttons are not there in the search bar. See the screenshot for the same [1] but if you change width to \"53.9\" will fix the problem.\n\n\n[1] https://ibb.co/LhZMpnx","commit_id":"edabb0670dcf10e70a213955590ce3fc017e7b91"}],"openstack_dashboard/dashboards/project/static/dashboard/project/workflow/launch-instance/launch-instance-model.service.spec.js":[{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"6c2341a64a4759caea987f21100d2aab85b6d0f2","unresolved":true,"context_lines":[{"line_number":139,"context_line":"                // ID 6 is already returned by the first call (below), so this should be ignored."},{"line_number":140,"context_line":"                // To clarify the difference, the content here is different intentionally."},{"line_number":141,"context_line":"                {id: \u00276\u0027, container_format: \u0027raw\u0027, properties: {image_type: \u0027image\u0027},"},{"line_number":142,"context_line":"                 visibility: \u0027community\u0027},"},{"line_number":143,"context_line":"              ];"},{"line_number":144,"context_line":"            } else {"},{"line_number":145,"context_line":"              images \u003d ["}],"source_content_type":"text/javascript","patch_set":4,"id":"8ff9032c_0c0daec6","side":"PARENT","line":142,"updated":"2021-05-24 09:58:44.000000000","message":"why did you remove trailing comma?\nThis change is not required in master branch.","commit_id":"d862623c27ea292a632cdb6591fd3c10e459f02b"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"db4643dc4b39141ec3a5c08ed0ba6eb4757d52dd","unresolved":true,"context_lines":[{"line_number":139,"context_line":"                // ID 6 is already returned by the first call (below), so this should be ignored."},{"line_number":140,"context_line":"                // To clarify the difference, the content here is different intentionally."},{"line_number":141,"context_line":"                {id: \u00276\u0027, container_format: \u0027raw\u0027, properties: {image_type: \u0027image\u0027},"},{"line_number":142,"context_line":"                 visibility: \u0027community\u0027},"},{"line_number":143,"context_line":"              ];"},{"line_number":144,"context_line":"            } else {"},{"line_number":145,"context_line":"              images \u003d ["}],"source_content_type":"text/javascript","patch_set":4,"id":"b7fc148f_c145bc86","side":"PARENT","line":142,"in_reply_to":"8ff9032c_0c0daec6","updated":"2021-05-24 11:56:09.000000000","message":"I needed to apply it to make `horizon-nodejs10-run-lint` (https://zuul.opendev.org/t/openstack/build/e9c09824278c4faea9a078d988a4cfa1) pass.","commit_id":"d862623c27ea292a632cdb6591fd3c10e459f02b"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"5b4c6317b77ecc01682de4afb29ba07c4dbeb17f","unresolved":false,"context_lines":[{"line_number":139,"context_line":"                // ID 6 is already returned by the first call (below), so this should be ignored."},{"line_number":140,"context_line":"                // To clarify the difference, the content here is different intentionally."},{"line_number":141,"context_line":"                {id: \u00276\u0027, container_format: \u0027raw\u0027, properties: {image_type: \u0027image\u0027},"},{"line_number":142,"context_line":"                 visibility: \u0027community\u0027},"},{"line_number":143,"context_line":"              ];"},{"line_number":144,"context_line":"            } else {"},{"line_number":145,"context_line":"              images \u003d ["}],"source_content_type":"text/javascript","patch_set":4,"id":"7c0a00c7_ce1e7e45","side":"PARENT","line":142,"in_reply_to":"b7fc148f_c145bc86","updated":"2022-12-07 23:45:07.000000000","message":"Done","commit_id":"d862623c27ea292a632cdb6591fd3c10e459f02b"},{"author":{"_account_id":29313,"name":"Vishal Manchanda","email":"manchandavishal143@gmail.com","username":"vishalmanchanda"},"change_message_id":"6c2341a64a4759caea987f21100d2aab85b6d0f2","unresolved":true,"context_lines":[{"line_number":156,"context_line":"                 properties: {block_device_mapping: \u0027[{\"source_type\": \"snapshot\"}]\u0027}},"},{"line_number":157,"context_line":"                {id: \u00278\u0027, container_format: \u0027raw\u0027,"},{"line_number":158,"context_line":"                 properties: {block_device_mapping: \u0027[{\"source_type\": \"snapshot\"}]\u0027}},"},{"line_number":159,"context_line":"                {id: \u00279\u0027, container_format: \u0027raw\u0027, properties: {image_type: \u0027snapshot\u0027}},"},{"line_number":160,"context_line":"              ];"},{"line_number":161,"context_line":"            }"},{"line_number":162,"context_line":""}],"source_content_type":"text/javascript","patch_set":4,"id":"7c31fd00_22c193a0","side":"PARENT","line":159,"range":{"start_line":159,"start_character":88,"end_line":159,"end_character":89},"updated":"2021-05-24 09:58:44.000000000","message":"ditto.","commit_id":"d862623c27ea292a632cdb6591fd3c10e459f02b"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"db4643dc4b39141ec3a5c08ed0ba6eb4757d52dd","unresolved":true,"context_lines":[{"line_number":156,"context_line":"                 properties: {block_device_mapping: \u0027[{\"source_type\": \"snapshot\"}]\u0027}},"},{"line_number":157,"context_line":"                {id: \u00278\u0027, container_format: \u0027raw\u0027,"},{"line_number":158,"context_line":"                 properties: {block_device_mapping: \u0027[{\"source_type\": \"snapshot\"}]\u0027}},"},{"line_number":159,"context_line":"                {id: \u00279\u0027, container_format: \u0027raw\u0027, properties: {image_type: \u0027snapshot\u0027}},"},{"line_number":160,"context_line":"              ];"},{"line_number":161,"context_line":"            }"},{"line_number":162,"context_line":""}],"source_content_type":"text/javascript","patch_set":4,"id":"c886a59d_4dda2dde","side":"PARENT","line":159,"range":{"start_line":159,"start_character":88,"end_line":159,"end_character":89},"in_reply_to":"7c31fd00_22c193a0","updated":"2021-05-24 11:56:09.000000000","message":"Same here.","commit_id":"d862623c27ea292a632cdb6591fd3c10e459f02b"},{"author":{"_account_id":28356,"name":"Rafael Weingartner","email":"rafael@apache.org","username":"rafaelweingartner"},"change_message_id":"5b4c6317b77ecc01682de4afb29ba07c4dbeb17f","unresolved":false,"context_lines":[{"line_number":156,"context_line":"                 properties: {block_device_mapping: \u0027[{\"source_type\": \"snapshot\"}]\u0027}},"},{"line_number":157,"context_line":"                {id: \u00278\u0027, container_format: \u0027raw\u0027,"},{"line_number":158,"context_line":"                 properties: {block_device_mapping: \u0027[{\"source_type\": \"snapshot\"}]\u0027}},"},{"line_number":159,"context_line":"                {id: \u00279\u0027, container_format: \u0027raw\u0027, properties: {image_type: \u0027snapshot\u0027}},"},{"line_number":160,"context_line":"              ];"},{"line_number":161,"context_line":"            }"},{"line_number":162,"context_line":""}],"source_content_type":"text/javascript","patch_set":4,"id":"24637851_f3116f37","side":"PARENT","line":159,"range":{"start_line":159,"start_character":88,"end_line":159,"end_character":89},"in_reply_to":"c886a59d_4dda2dde","updated":"2022-12-07 23:45:07.000000000","message":"Done","commit_id":"d862623c27ea292a632cdb6591fd3c10e459f02b"}]}