)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":28558,"name":"Ghada Khalil","email":"ghada.khalil@windriver.com","username":"gkhalil"},"change_message_id":"17c93fa20b57e2fdc818c11daee464e2ec055d27","unresolved":true,"context_lines":[{"line_number":15,"context_line":"https://review.opendev.org/c/starlingx/stx-puppet/+/816523"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Depends-on: https://review.opendev.org/c/starlingx/stx-puppet/+/816523"},{"line_number":18,"context_line":"Closes-Bug: #1949620"},{"line_number":19,"context_line":"Change-Id: I14f3e18778442fedb495ceff7b48e25a9ca3694a"},{"line_number":20,"context_line":"Signed-off-by: Joao Soubihe \u003cJoaoPaulo.Soubihe@windriver.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"bab1795c_97e5340e","line":18,"range":{"start_line":18,"start_character":0,"end_line":18,"end_character":20},"updated":"2021-11-04 15:39:53.000000000","message":"Change to Partial-Bug so that the LP is not marked \"Fix Released\" prematurely","commit_id":"79e682b33aa8f2feea4f87168d8e2a16ae29874a"},{"author":{"_account_id":32877,"name":"Joao Soubihe","email":"JoaoPaulo.Soubihe@windriver.com","username":"jsoubihe"},"change_message_id":"430c54ca5c0ea2330c7e16e8f56fdab91c774844","unresolved":false,"context_lines":[{"line_number":15,"context_line":"https://review.opendev.org/c/starlingx/stx-puppet/+/816523"},{"line_number":16,"context_line":""},{"line_number":17,"context_line":"Depends-on: https://review.opendev.org/c/starlingx/stx-puppet/+/816523"},{"line_number":18,"context_line":"Closes-Bug: #1949620"},{"line_number":19,"context_line":"Change-Id: I14f3e18778442fedb495ceff7b48e25a9ca3694a"},{"line_number":20,"context_line":"Signed-off-by: Joao Soubihe \u003cJoaoPaulo.Soubihe@windriver.com\u003e"}],"source_content_type":"text/x-gerrit-commit-message","patch_set":3,"id":"5ecf4ea9_b71da847","line":18,"range":{"start_line":18,"start_character":0,"end_line":18,"end_character":20},"in_reply_to":"bab1795c_97e5340e","updated":"2021-11-04 15:46:39.000000000","message":"Ack","commit_id":"79e682b33aa8f2feea4f87168d8e2a16ae29874a"}],"/PATCHSET_LEVEL":[{"author":{"_account_id":15435,"name":"Al Bailey","email":"albailey1974@gmail.com","username":"albailey"},"change_message_id":"55e80f20cc4d8dfa6685cd217d83ec181c22551d","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"bd33defa_dfd63e3d","updated":"2021-11-04 18:52:20.000000000","message":"I didnt want to -1 this if it is working.\nCan you confirm that the port selected is the correct one in this env.","commit_id":"39c918745a6c745c347fb5f2cebebf33303a9f5b"},{"author":{"_account_id":28558,"name":"Ghada Khalil","email":"ghada.khalil@windriver.com","username":"gkhalil"},"change_message_id":"e3c0deaeaa95396829a20e8b14b0e00b20666f4a","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"e3f0432f_c1cd6e47","updated":"2021-11-04 17:45:41.000000000","message":"recheck","commit_id":"39c918745a6c745c347fb5f2cebebf33303a9f5b"},{"author":{"_account_id":28558,"name":"Ghada Khalil","email":"ghada.khalil@windriver.com","username":"gkhalil"},"change_message_id":"365722ceee7bd00f03c2938f11eacf4ef18c2531","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"c060583b_bf4d523e","in_reply_to":"551961cb_b9fad3ff","updated":"2021-11-04 21:59:54.000000000","message":"Al/Andy/John, should we just choose a port# that\u0027s closer to the range already used?","commit_id":"39c918745a6c745c347fb5f2cebebf33303a9f5b"},{"author":{"_account_id":32877,"name":"Joao Soubihe","email":"JoaoPaulo.Soubihe@windriver.com","username":"jsoubihe"},"change_message_id":"a817135e2320b6fd9f65245fcfff140e31c46fb3","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"551961cb_b9fad3ff","in_reply_to":"bd33defa_dfd63e3d","updated":"2021-11-04 19:26:01.000000000","message":"Hi Al, yes, I tested it on my local environment and didn\u0027t have any problem.","commit_id":"39c918745a6c745c347fb5f2cebebf33303a9f5b"},{"author":{"_account_id":28676,"name":"Andy Ning","email":"andy.ning@windriver.com","username":"andy.wrs"},"change_message_id":"fd215e62fd5970f54ae67bd8fba409a092b22ccf","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":4,"id":"dbedc611_2b0ee707","in_reply_to":"c060583b_bf4d523e","updated":"2021-11-04 22:15:40.000000000","message":"I don\u0027t think so. If we use a different port we will need to make changes to api-proxy so that it will listen on that port and proxy requests. The purpose of the this public endpoint is just to add an \"alias\" in the System Controller region so that the remote client can find VIM service\u0027s URL. The requests from remote client will still be served by VIM at the same port.","commit_id":"39c918745a6c745c347fb5f2cebebf33303a9f5b"}],"sysinv/sysinv/sysinv/sysinv/puppet/dcorch.py":[{"author":{"_account_id":15435,"name":"Al Bailey","email":"albailey1974@gmail.com","username":"albailey"},"change_message_id":"2c0565d2f3ff52d7dc664bbe4a4f42fad3d6a928","unresolved":true,"context_lines":[{"line_number":33,"context_line":"    CINDER_SERVICE_PORT \u003d 28776"},{"line_number":34,"context_line":"    PATCHING_SERVICE_PORT \u003d 25491"},{"line_number":35,"context_line":"    PATCHING_SERVICE_PATH \u003d \u0027\u0027"},{"line_number":36,"context_line":"    NFV_SERVICE_PORT \u003d 4545"},{"line_number":37,"context_line":"    NFV_SERVICE_PATH \u003d \u0027\u0027"},{"line_number":38,"context_line":"    IDENTITY_SERVICE_PORT \u003d 25000"},{"line_number":39,"context_line":"    IDENTITY_SERVICE_PATH \u003d \u0027v3\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"240fac94_930b164d","line":36,"updated":"2021-11-04 18:42:42.000000000","message":"I am still surprised this does not need to be a different port.  The others are in the 20000 range","commit_id":"39c918745a6c745c347fb5f2cebebf33303a9f5b"},{"author":{"_account_id":32877,"name":"Joao Soubihe","email":"JoaoPaulo.Soubihe@windriver.com","username":"jsoubihe"},"change_message_id":"d6ef2d4c9c8336c876b5ed898b81884826f814af","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    CINDER_SERVICE_PORT \u003d 28776"},{"line_number":34,"context_line":"    PATCHING_SERVICE_PORT \u003d 25491"},{"line_number":35,"context_line":"    PATCHING_SERVICE_PATH \u003d \u0027\u0027"},{"line_number":36,"context_line":"    NFV_SERVICE_PORT \u003d 4545"},{"line_number":37,"context_line":"    NFV_SERVICE_PATH \u003d \u0027\u0027"},{"line_number":38,"context_line":"    IDENTITY_SERVICE_PORT \u003d 25000"},{"line_number":39,"context_line":"    IDENTITY_SERVICE_PATH \u003d \u0027v3\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"c00b7e1d_c6c80083","line":36,"in_reply_to":"240fac94_930b164d","updated":"2021-11-05 13:32:55.000000000","message":"Ack","commit_id":"39c918745a6c745c347fb5f2cebebf33303a9f5b"},{"author":{"_account_id":28558,"name":"Ghada Khalil","email":"ghada.khalil@windriver.com","username":"gkhalil"},"change_message_id":"a83c609c629e102253944026879ce3f5075e7e63","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    CINDER_SERVICE_PORT \u003d 28776"},{"line_number":34,"context_line":"    PATCHING_SERVICE_PORT \u003d 25491"},{"line_number":35,"context_line":"    PATCHING_SERVICE_PATH \u003d \u0027\u0027"},{"line_number":36,"context_line":"    NFV_SERVICE_PORT \u003d 4545"},{"line_number":37,"context_line":"    NFV_SERVICE_PATH \u003d \u0027\u0027"},{"line_number":38,"context_line":"    IDENTITY_SERVICE_PORT \u003d 25000"},{"line_number":39,"context_line":"    IDENTITY_SERVICE_PATH \u003d \u0027v3\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"afaf67d8_63634e77","line":36,"in_reply_to":"61586652_39e0ffd0","updated":"2021-11-05 16:47:09.000000000","message":"John, I asked a similar question yesterday. Here is Andy\u0027s response:\nGhada Khalil Patchset 4 Nov 04\nAl/Andy/John, should we just choose a port# that\u0027s closer to the range already used?\n\nAndy Ning Patchset 4  Nov 04\nI don\u0027t think so. If we use a different port we will need to make changes to api-proxy so that it will listen on that port and proxy requests. The purpose of the this public endpoint is just to add an \"alias\" in the System Controller region so that the remote client can find VIM service\u0027s URL. The requests from remote client will still be served by VIM at the same port.","commit_id":"39c918745a6c745c347fb5f2cebebf33303a9f5b"},{"author":{"_account_id":9926,"name":"John Kung","email":"john.kung@windriver.com","username":"jkung"},"change_message_id":"cde82daf6b9e41b0d7a7bbcbff0e551004a0df80","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    CINDER_SERVICE_PORT \u003d 28776"},{"line_number":34,"context_line":"    PATCHING_SERVICE_PORT \u003d 25491"},{"line_number":35,"context_line":"    PATCHING_SERVICE_PATH \u003d \u0027\u0027"},{"line_number":36,"context_line":"    NFV_SERVICE_PORT \u003d 4545"},{"line_number":37,"context_line":"    NFV_SERVICE_PATH \u003d \u0027\u0027"},{"line_number":38,"context_line":"    IDENTITY_SERVICE_PORT \u003d 25000"},{"line_number":39,"context_line":"    IDENTITY_SERVICE_PATH \u003d \u0027v3\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"3ba1e661_4cfc3433","line":36,"in_reply_to":"afaf67d8_63634e77","updated":"2021-11-05 17:11:32.000000000","message":"ok, thanks for the clarification.  4545 is port already reserved for VIM public URL so that is fine also.","commit_id":"39c918745a6c745c347fb5f2cebebf33303a9f5b"},{"author":{"_account_id":9926,"name":"John Kung","email":"john.kung@windriver.com","username":"jkung"},"change_message_id":"daab5939afb4c0d215873bb4152b2f7fd93b4a11","unresolved":false,"context_lines":[{"line_number":33,"context_line":"    CINDER_SERVICE_PORT \u003d 28776"},{"line_number":34,"context_line":"    PATCHING_SERVICE_PORT \u003d 25491"},{"line_number":35,"context_line":"    PATCHING_SERVICE_PATH \u003d \u0027\u0027"},{"line_number":36,"context_line":"    NFV_SERVICE_PORT \u003d 4545"},{"line_number":37,"context_line":"    NFV_SERVICE_PATH \u003d \u0027\u0027"},{"line_number":38,"context_line":"    IDENTITY_SERVICE_PORT \u003d 25000"},{"line_number":39,"context_line":"    IDENTITY_SERVICE_PATH \u003d \u0027v3\u0027"}],"source_content_type":"text/x-python","patch_set":4,"id":"61586652_39e0ffd0","line":36,"in_reply_to":"c00b7e1d_c6c80083","updated":"2021-11-05 15:04:35.000000000","message":"In keeping with the convention, is 2 in front of the current nvfv public port not available?","commit_id":"39c918745a6c745c347fb5f2cebebf33303a9f5b"}]}
