)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"349df2e7d676bb7d445e1fc7b76e5cf6b9528a18","unresolved":true,"context_lines":[],"source_content_type":"","patch_set":3,"id":"03417c7f_d6d87193","updated":"2024-10-23 00:19:03.000000000","message":"Thanks, I honestly forgot this patch existed. I should check up on my old patches 😄","commit_id":"3a7c7afaf3f0f9677fab3681774ca3e543c983eb"},{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"f52d02433a0bd3ced40755970676904cc2cc72de","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":6,"id":"16c868c0_bc8e8c22","updated":"2024-10-24 08:03:27.000000000","message":"ideally you want to have a wsgi subdir as a proper module here, for an example look at how this has been implemented in nova https://opendev.org/openstack/nova/src/branch/master/nova/wsgi","commit_id":"93ef06e2170b97b2b42442bc0171851b5ec12c76"},{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"526408ad96d8ec5e02174b645fca5ccbe43c9209","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"e557fb7a_f18955b4","updated":"2024-10-25 18:28:16.000000000","message":"Not sure if my code review should count, but lgtm","commit_id":"3db454f0ead9ca791063cfdd210d0fd2d62885bf"}],"devstack/lib/ironic":[{"author":{"_account_id":11604,"name":"sean mooney","email":"smooney@redhat.com","username":"sean-k-mooney"},"change_message_id":"408b2102d1028ebcdfb19664281126175b69a00d","unresolved":true,"context_lines":[{"line_number":1670,"context_line":""},{"line_number":1671,"context_line":"    # Adds uWSGI for Ironic API"},{"line_number":1672,"context_line":"    if [[ \"$IRONIC_USE_WSGI\" \u003d\u003d \"True\" ]]; then"},{"line_number":1673,"context_line":"        write_uwsgi_config \"$IRONIC_UWSGI_CONF\" \"$IRONIC_UWSGI\" \"/baremetal\""},{"line_number":1674,"context_line":"    fi"},{"line_number":1675,"context_line":""},{"line_number":1676,"context_line":"    if [[ \"$os_VENDOR\" \u003d~ (Debian|Ubuntu) ]]; then"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"6387a8ec_08ebb0fe","line":1673,"updated":"2024-08-31 00:47:03.000000000","message":"if you compare to the placement verison \n\nhttps://review.opendev.org/c/openstack/devstack/+/919581/1/lib/placement#116\n\nyou need to update this line too if your swapping to the application module rather than the path\n\ni would guess it should look liek this.\n\n```suggestion\n         write_uwsgi_config \"$IRONIC_UWSGI_CONF\" \"$IRONIC_UWSGI\" \"/baremetal\" \"\" \"ironic-api\"\n```","commit_id":"a360b0cc796b44068a8f6ec3a606cf889a4e909c"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"8dd952e6208792ee953db0a2095839c2fe5041d6","unresolved":false,"context_lines":[{"line_number":1670,"context_line":""},{"line_number":1671,"context_line":"    # Adds uWSGI for Ironic API"},{"line_number":1672,"context_line":"    if [[ \"$IRONIC_USE_WSGI\" \u003d\u003d \"True\" ]]; then"},{"line_number":1673,"context_line":"        write_uwsgi_config \"$IRONIC_UWSGI_CONF\" \"$IRONIC_UWSGI\" \"/baremetal\""},{"line_number":1674,"context_line":"    fi"},{"line_number":1675,"context_line":""},{"line_number":1676,"context_line":"    if [[ \"$os_VENDOR\" \u003d~ (Debian|Ubuntu) ]]; then"}],"source_content_type":"application/x-shellscript","patch_set":2,"id":"188022da_3cc6bdc4","line":1673,"in_reply_to":"6387a8ec_08ebb0fe","updated":"2024-10-22 23:29:57.000000000","message":"Done","commit_id":"a360b0cc796b44068a8f6ec3a606cf889a4e909c"}],"doc/source/conf.py":[{"author":{"_account_id":10342,"name":"Jay Faulkner","display_name":"JayF","email":"jay@jvf.cc","username":"JayF","status":"youtube.com/@oss-gr / podcast.gr-oss.io"},"change_message_id":"526408ad96d8ec5e02174b645fca5ccbe43c9209","unresolved":true,"context_lines":[{"line_number":66,"context_line":""},{"line_number":67,"context_line":"autosummary_mock_imports \u003d ["},{"line_number":68,"context_line":"    \u0027ironic.wsgi\u0027,"},{"line_number":69,"context_line":"]"},{"line_number":70,"context_line":""},{"line_number":71,"context_line":"redfish_interop_source \u003d \\"},{"line_number":72,"context_line":"    \u0027../../redfish-interop-profiles/OpenStackIronicProfile.v1_1_0.json\u0027"}],"source_content_type":"text/x-python","patch_set":8,"id":"4fd5a70c_499191d0","line":69,"updated":"2024-10-25 18:28:16.000000000","message":"this is great, thanks","commit_id":"3db454f0ead9ca791063cfdd210d0fd2d62885bf"}],"ironic/wsgi.py":[{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"9fd15f607741031e7dd04ea049664673b4a69f57","unresolved":true,"context_lines":[{"line_number":10,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":11,"context_line":"# limitations under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"\"\"\"WSGI application entry-point for Cinder Volume API.\"\"\""},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"import threading"},{"line_number":16,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"132fdbf8_24b0fba1","line":13,"updated":"2024-08-30 18:16:25.000000000","message":"```suggestion\n\"\"\"WSGI application entry-point for Ironic API.\"\"\"\n```","commit_id":"5ca8e4c225bfa61346b10da1ec46ad708d1fa02d"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"8dd952e6208792ee953db0a2095839c2fe5041d6","unresolved":false,"context_lines":[{"line_number":10,"context_line":"# See the License for the specific language governing permissions and"},{"line_number":11,"context_line":"# limitations under the License."},{"line_number":12,"context_line":""},{"line_number":13,"context_line":"\"\"\"WSGI application entry-point for Cinder Volume API.\"\"\""},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"import threading"},{"line_number":16,"context_line":""}],"source_content_type":"text/x-python","patch_set":1,"id":"05c760a9_bb3bf3e8","line":13,"in_reply_to":"132fdbf8_24b0fba1","updated":"2024-10-22 23:29:57.000000000","message":"Done","commit_id":"5ca8e4c225bfa61346b10da1ec46ad708d1fa02d"}],"ironic/wsgi/__init__.py":[{"author":{"_account_id":23851,"name":"Riccardo Pittau","email":"elfosardo@gmail.com","username":"elfosardo"},"change_message_id":"2b7d1ff5bcf3412ff9fa8c2557de7ed9ae67bd4e","unresolved":true,"context_lines":[{"line_number":18,"context_line":""},{"line_number":19,"context_line":"application \u003d None"},{"line_number":20,"context_line":"lock \u003d threading.Lock()"},{"line_number":21,"context_line":"with lock:"},{"line_number":22,"context_line":"    if application is None:"},{"line_number":23,"context_line":"        application \u003d wsgi.initialize_wsgi_app()"}],"source_content_type":"text/x-python","patch_set":7,"id":"1de8db61_87c67ad5","line":21,"updated":"2024-10-24 17:22:21.000000000","message":"the error you\u0027re getting in the docs job is because you have code the top level\nyou want to have this under something like \"if __name__ \u003d\u003d \u0027__main__\u0027\", or move it into a function and then call it under that","commit_id":"2ec66e4f05e9f1e8c35b9bd91129958401871771"}]}