)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"24c90e977279a70a52dd2aa0d29e0b30d02b8773","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"19267f67_7c575905","updated":"2021-11-24 09:00:27.000000000","message":"It seems we need https://review.opendev.org/c/openstack/tripleo-heat-templates/+/819074 to fix the issue with tripleo job.","commit_id":"d191895eeaa99c6a2eb009dcd13487ff152ebf1a"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"ce05409a6d14d94a369ff775a5f33ce7ed64b999","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"5adaee99_1be3603d","updated":"2021-12-09 01:28:51.000000000","message":"recheck","commit_id":"d191895eeaa99c6a2eb009dcd13487ff152ebf1a"}],"manifests/keystone/auth.pp":[{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"6e6d5e8b5cf594ac1a3166462e8b6ccc208ac7da","unresolved":false,"context_lines":[{"line_number":53,"context_line":"#"},{"line_number":54,"context_line":"# [*public_url*]"},{"line_number":55,"context_line":"#   (Optional) The endpoint\u0027s public url."},{"line_number":56,"context_line":"#   Defaults to \u0027http://127.0.0.1/placement\u0027"},{"line_number":57,"context_line":"#   This url should *not* contain any trailing \u0027/\u0027."},{"line_number":58,"context_line":"#"},{"line_number":59,"context_line":"# [*admin_url*]"}],"source_content_type":"text/x-puppet","patch_set":4,"id":"055d99bd_d62e3e93","side":"PARENT","line":56,"updated":"2021-12-08 21:18:32.000000000","message":"for some reason i vaguely recall this being intentional because of the fact it was colocated with nova for a while.  I think it\u0027s ok now though it may have some other historical reasoning for /placement by default","commit_id":"467c0b0a4da61826d72e3af8f781b010ad519feb"},{"author":{"_account_id":9816,"name":"Takashi Kajinami","email":"kajinamit@oss.nttdata.com","username":"kajinamit"},"change_message_id":"b6e342ae2b1f04a3693c767184729b6ff03dbe3f","unresolved":false,"context_lines":[{"line_number":53,"context_line":"#"},{"line_number":54,"context_line":"# [*public_url*]"},{"line_number":55,"context_line":"#   (Optional) The endpoint\u0027s public url."},{"line_number":56,"context_line":"#   Defaults to \u0027http://127.0.0.1/placement\u0027"},{"line_number":57,"context_line":"#   This url should *not* contain any trailing \u0027/\u0027."},{"line_number":58,"context_line":"#"},{"line_number":59,"context_line":"# [*admin_url*]"}],"source_content_type":"text/x-puppet","patch_set":4,"id":"6dfd78c9_80402933","side":"PARENT","line":56,"in_reply_to":"055d99bd_d62e3e93","updated":"2021-12-09 01:37:30.000000000","message":"I could not find the reason in puppet-placement repo but I noticed I should have checked puppet-nova as well.\n\nLooking at git log, it seems we changed the port from 8778 to 80 because of conflicts with senlin. The patch updated path from / to /placement to avoid conflict with Horizon, I think.\n\nhttps://github.com/openstack/puppet-nova/commit/fb0327d101723c7fa49430a2f35b5d49d5803f99\n\nSo basically this change reverts the above commit. The current installation guide mentions 8778 and I believe that value is very common now. We\u0027ve retired puppet-senlin so I don\u0027t expect usage of 8778 is a problem at least within the use case covered by Puppet OpenStack.\n\n[1] https://docs.openstack.org/placement/latest/install/install-rdo.html","commit_id":"467c0b0a4da61826d72e3af8f781b010ad519feb"},{"author":{"_account_id":14985,"name":"Alex Schultz","email":"aschultz@next-development.com","username":"mwhahaha"},"change_message_id":"dd1a7aef60b6a5615590de8c1054ad400b642543","unresolved":false,"context_lines":[{"line_number":53,"context_line":"#"},{"line_number":54,"context_line":"# [*public_url*]"},{"line_number":55,"context_line":"#   (Optional) The endpoint\u0027s public url."},{"line_number":56,"context_line":"#   Defaults to \u0027http://127.0.0.1/placement\u0027"},{"line_number":57,"context_line":"#   This url should *not* contain any trailing \u0027/\u0027."},{"line_number":58,"context_line":"#"},{"line_number":59,"context_line":"# [*admin_url*]"}],"source_content_type":"text/x-puppet","patch_set":4,"id":"dced18c4_af913099","side":"PARENT","line":56,"in_reply_to":"6dfd78c9_80402933","updated":"2021-12-09 14:49:51.000000000","message":"There was also some assumptions about /placement for a while. this is probably fine now I just recall it being not so straight forward when we initially implemented stuff.","commit_id":"467c0b0a4da61826d72e3af8f781b010ad519feb"}]}