)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"77042b82b6497e1eff7e6b0c280f6ef530ebd2bc","unresolved":true,"context_lines":[{"line_number":6,"context_line":""},{"line_number":7,"context_line":"Modification of placement-api.conf"},{"line_number":8,"context_line":""},{"line_number":9,"context_line":"Following doc on https://docs.openstack.org/placement/2024.2/install/install-rdo.html will lead to error after first time launching Horizon due to lack of access for Apache to /usr/bin directory. Following lines should be added to fix this issue to 00-placement-api.conf."},{"line_number":10,"context_line":""},{"line_number":11,"context_line":"Closes-Bug: #2091584"},{"line_number":12,"context_line":"Change-Id: I14a1719c4f7eca8ea1969e0870967e2daf585a67"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":10,"id":"616f2349_2c987c03","line":9,"range":{"start_line":9,"start_character":0,"end_line":9,"end_character":9},"updated":"2025-01-10 14:14:18.000000000","message":"Please wrap lines by 72 characters.\n\nhttps://wiki.openstack.org/wiki/GitCommitMessages#Summary_of_Git_commit_message_structure","commit_id":"c9c94153393587dd0dd25fb55e2f388151d4b293"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"b1d373389381e7f29032d15f81b07dbfa7dcaea8","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":10,"id":"7ed23a8c_7aade234","updated":"2025-01-10 14:17:09.000000000","message":"Hmm. Giving this further thought, I wonder if we should fix the config file installed by the placement package, which is maintained in RDO side, instead of adding instruction to \"fix\" the incomplete setting.\n\nhttps://github.com/rdo-packages/placement-distgit/blob/rpm-master/placement-api.conf","commit_id":"c9c94153393587dd0dd25fb55e2f388151d4b293"},{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"6462a5345fafc262878ded7a7ed319a97ca66413","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":10,"id":"d52c7599_41c67936","in_reply_to":"7ed23a8c_7aade234","updated":"2025-01-10 14:24:51.000000000","message":"that would likly be better yes.\n\nideally, we would not need distro specific docs for this in general in tree.\n\nwe would just provide a general doc for how to run this with appahce and the disto specifics would be addressed in the packaging.","commit_id":"c9c94153393587dd0dd25fb55e2f388151d4b293"},{"author":{"_account_id":37632,"name":"Dmitriy Chubinidze","email":"dcu995@gmail.com","username":"chubinidzedr"},"change_message_id":"e946d1244bfbdb9e9b962238b767f5db862be3c2","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":10,"id":"4c0b4b26_3b2bdf9a","in_reply_to":"d52c7599_41c67936","updated":"2025-01-10 14:48:04.000000000","message":"Thanks, seems you are right.\nI\u0027ve opened separate review at rdoproject to finally add these lines to config.\nCurret will be closed.","commit_id":"c9c94153393587dd0dd25fb55e2f388151d4b293"}],"doc/source/install/install-rdo.rst":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"77042b82b6497e1eff7e6b0c280f6ef530ebd2bc","unresolved":true,"context_lines":[{"line_number":112,"context_line":""},{"line_number":113,"context_line":"      \u003cDirectory /usr/bin\u003e"},{"line_number":114,"context_line":"        Require all denied"},{"line_number":115,"context_line":"         \u003cFiles \"placement-api\"\u003e"},{"line_number":116,"context_line":"          \u003cRequireAll\u003e"},{"line_number":117,"context_line":"          Require all granted"},{"line_number":118,"context_line":"          Require not env blockAccess"},{"line_number":119,"context_line":"       \u003c/RequireAll\u003e"},{"line_number":120,"context_line":"         \u003c/Files\u003e"},{"line_number":121,"context_line":"      \u003c/Directory\u003e"},{"line_number":122,"context_line":""},{"line_number":123,"context_line":"#. Populate the ``placement`` database:"}],"source_content_type":"text/x-rst","patch_set":10,"id":"6fba571a_c58b30b9","line":120,"range":{"start_line":115,"start_character":0,"end_line":120,"end_character":17},"updated":"2025-01-10 14:14:18.000000000","message":"It seems indent is somehow messed up ?","commit_id":"c9c94153393587dd0dd25fb55e2f388151d4b293"}]}