)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"802b4099ecf5c7175f89f38c8a303c38ef70ef57","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This reverts commit eb2b2f2765933473802150f3da4af386256e02b1."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"As it broke openstack installation"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ib7f8191c980972764bc3a07136603a5cc10ea732"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"a4bfba5e_79e09894","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":34},"updated":"2024-01-21 17:56:02.000000000","message":"we really need to know what was actually broken by this.","commit_id":"04180c6a97a58714660bd05ac25619ba6766905b"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"5850ac009289d0cd5791b5f61e4bccb099100cbd","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This reverts commit eb2b2f2765933473802150f3da4af386256e02b1."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"As it broke openstack installation"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ib7f8191c980972764bc3a07136603a5cc10ea732"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"725415f5_4ed7d325","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":34},"in_reply_to":"33247ce6_e7d1761f","updated":"2024-01-21 19:16:31.000000000","message":"As I said, i temporarily disabled horizon tests just to check if Zuul will verify without horizon test (that patch i\u0027ve already abandoned - because even if horizon test was turned off ..other tests failed ) ... but it didn\u0027t work.\n\nUnfortunatelly I\u0027ve found that cross-dependency magic currently not work :\u0027(, so I can\u0027t verify which patches should be reverted - maybe this one, maybe other one, maybe all of them ... We basically need to check ... But as you can see even trivial change in kolla-ansible template worked at friday ...","commit_id":"04180c6a97a58714660bd05ac25619ba6766905b"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"a990298a71f9a26166c53e2ddbaae90a9a8c8063","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This reverts commit eb2b2f2765933473802150f3da4af386256e02b1."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"As it broke openstack installation"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ib7f8191c980972764bc3a07136603a5cc10ea732"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"33247ce6_e7d1761f","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":34},"in_reply_to":"90140315_6272c0a2","updated":"2024-01-21 18:39:48.000000000","message":"So it seems horizon returns 400 to healthcheck requests (GET /) and that is causing the problem. I\u0027m still unsure about the exact package bump which can cause this. Note that bump of django version was also recently merged[1] and that can theoretically affect horizon.\n\nWe probably have to check if this revert actually resolves the problem in kolla (does depends-on magic works ?) or hold the node to dig into the problem.\n\n[1] https://review.opendev.org/c/openstack/requirements/+/904202","commit_id":"04180c6a97a58714660bd05ac25619ba6766905b"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"509572c73aa76407116297060df44cea356999a1","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This reverts commit eb2b2f2765933473802150f3da4af386256e02b1."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"As it broke openstack installation"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ib7f8191c980972764bc3a07136603a5cc10ea732"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"fbb5df4b_244c90d1","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":34},"in_reply_to":"a4bfba5e_79e09894","updated":"2024-01-21 18:06:04.000000000","message":"Yep, let me test it - but i am 90 percent sure that this broke a lot of things as you can see here (assuming that it will be this commit as it\u0027s changing quite a lot packages)\n\nhttps://review.opendev.org/c/openstack/kolla-ansible/+/906099 in entire ralation chain.\n\nFirstly I just saw that horizon is failing , so I checked requirements git, then I temporarily disabled horizon test job (as I saw there is Django bump version) and add as first commit in relation chain and waited for result - I thought that it will be  OKay except horizon and other stuff will be fine - but NO :( - there are more thing failing... \n\nSo, I am investigating ... Friday afternoon it was still working...","commit_id":"04180c6a97a58714660bd05ac25619ba6766905b"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"12c930a23a7b1102bdd2ac62d9e626dac865ec33","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This reverts commit eb2b2f2765933473802150f3da4af386256e02b1."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"As it broke openstack installation"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ib7f8191c980972764bc3a07136603a5cc10ea732"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"90140315_6272c0a2","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":34},"in_reply_to":"fbb5df4b_244c90d1","updated":"2024-01-21 18:12:01.000000000","message":"Check this - \u003e https://review.opendev.org/c/openstack/kolla-ansible/+/906223 \u003c-","commit_id":"04180c6a97a58714660bd05ac25619ba6766905b"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"ba40a08ebdcb391bd4a015db08c4987a8025acb5","unresolved":true,"context_lines":[{"line_number":8,"context_line":""},{"line_number":9,"context_line":"This reverts commit eb2b2f2765933473802150f3da4af386256e02b1."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"As it broke openstack installation"},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"Change-Id: Ib7f8191c980972764bc3a07136603a5cc10ea732"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"d60fca78_ab5cab1d","line":11,"range":{"start_line":11,"start_character":0,"end_line":11,"end_character":34},"in_reply_to":"fbb5df4b_244c90d1","updated":"2024-01-21 18:11:55.000000000","message":"Do you mind sharing the details of that horizon problem ? Reverting this is easy but we have to understand the problems, otherwise we are fully stuck with the current versions IMO.\n\nFor the issue with ansible-lint, I\u0027d suggest you bump ansible-lint to v6.22.0 which contains https://github.com/ansible/ansible-lint/commit/824d58fc35c63e413ed0f0a97976e3949d934eae","commit_id":"04180c6a97a58714660bd05ac25619ba6766905b"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":13252,"name":"Dr. Jens Harbott","display_name":"Jens Harbott (frickler)","email":"frickler@offenerstapel.de","username":"jrosenboom"},"change_message_id":"412442e813aac66de450877853b46eb4b4f12288","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"da8b4c38_aa98fbd7","updated":"2024-01-21 19:27:27.000000000","message":"There\u0027s no way we will revert 100+ version bumps just because some small bits may be failing. We can revert some things like the ruamel-yaml bump that may be causing some issues, but you\u0027ll need to document the exact issue and the focus still should be on fixing the issues in the consumers, rather than sticking to outdated library versions.","commit_id":"04180c6a97a58714660bd05ac25619ba6766905b"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"c3713fd27e909f2b545651b4df2b4f1ca0d3fe42","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"27e36ed5_96a43506","updated":"2024-01-21 18:42:27.000000000","message":"looks like we also have to revert https://review.opendev.org/c/openstack/requirements/+/906092 .","commit_id":"04180c6a97a58714660bd05ac25619ba6766905b"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"1fc7a313ce45f96aec82437ad81522713d26f1e2","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"d1b35afc_4748ce0c","in_reply_to":"bf8a46cb_fd1349d9","updated":"2024-01-22 07:49:04.000000000","message":"It was not clear if you actually aim to revert the change or this is just for testing. If it\u0027s for testing then it\u0027s ok (though we probably just keep -2 to avoid merging this)\n\nJust fyi, according to https://review.opendev.org/c/openstack/horizon/+/906225, all horizon tests pass with the new u-c. So the issue is not related to horizon core, but is something specific to reqeests to / (or probably the way how haproxy hit that path)","commit_id":"04180c6a97a58714660bd05ac25619ba6766905b"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"b4a55907a4bbd7533114a642925b3a651fcb403d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"693d9c60_b30a499d","in_reply_to":"d1b35afc_4748ce0c","updated":"2024-01-22 09:39:29.000000000","message":"And what about this ? From log it\u0027s visible that it\u0027s problem in python \n\nhttps://a9f12b6a44dc386ddd99-cba8b685fc22597821cf94b3969a4442.ssl.cf1.rackcdn.com/905644/29/check/kolla-ansible-ubuntu/6737f66/primary/logs/kolla/all-ERROR.txt\n\nCheck https://review.opendev.org/c/openstack/kolla-ansible/+/905502/11 the last zuul verified -1 ...all tests are passing after revert (but i turned off horizon tests) ..","commit_id":"04180c6a97a58714660bd05ac25619ba6766905b"},{"author":{"_account_id":27339,"name":"Michal Arbet","email":"michal.arbet@ultimum.io","username":"michalarbet"},"change_message_id":"582665fd5f5ab96f793184ab237f462cc434e799","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":1,"id":"bf8a46cb_fd1349d9","in_reply_to":"da8b4c38_aa98fbd7","updated":"2024-01-21 19:29:31.000000000","message":"Hold your horses, of course I really don\u0027t want to do the step backwards, but this is the fastest way how to test it, isn\u0027t it ? \n\nI will amend commit message to [DNM].","commit_id":"04180c6a97a58714660bd05ac25619ba6766905b"}],"upper-constraints.txt":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"0fada7b5d84d3892ffcc68c6d1ed737baab6cbc7","unresolved":true,"context_lines":[{"line_number":240,"context_line":"ptyprocess\u003d\u003d\u003d0.7.0"},{"line_number":241,"context_line":"threadloop\u003d\u003d\u003d1.0.2"},{"line_number":242,"context_line":"amqp\u003d\u003d\u003d5.2.0"},{"line_number":243,"context_line":"ruamel.yaml\u003d\u003d\u003d0.18.5"},{"line_number":244,"context_line":"websockify\u003d\u003d\u003d0.11.0"},{"line_number":245,"context_line":"gssapi\u003d\u003d\u003d1.8.3"},{"line_number":246,"context_line":"XStatic-JQuery.quicksearch\u003d\u003d\u003d2.0.3.2"}],"source_content_type":"text/plain","patch_set":1,"id":"d05fa234_2e028684","side":"PARENT","line":243,"range":{"start_line":243,"start_character":0,"end_line":243,"end_character":20},"updated":"2024-01-21 17:59:34.000000000","message":"assuming this revert is related to your patch in kolla-ansible, the actual problem is the fact that ansible-lint has ruamel.yaml\u003c1.18 . We can probably revert this specific change if it works but I\u0027d rather attempt to remove that capping first.","commit_id":"eb2b2f2765933473802150f3da4af386256e02b1"}]}
