)]}'
{"/COMMIT_MSG":[{"author":{"_account_id":9926,"name":"John Kung","email":"john.kung@windriver.com","username":"jkung"},"change_message_id":"1db08f9aec00f7804d7830d54b63d1fe638599c8","unresolved":true,"context_lines":[{"line_number":11,"context_line":"Test Plan:"},{"line_number":12,"context_line":"PASS: Build cgts-client package. Verify that the wheel is generated, and"},{"line_number":13,"context_line":"it\u0027s included in the wheels tarball."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Story: 2009831"},{"line_number":16,"context_line":"Task: 46633"},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"926e9fda_eee7c184","line":14,"updated":"2022-10-25 20:12:55.000000000","message":"how is it verified that the containerized fm-rest-api behaving as expected with this change?","commit_id":"7f5aae8145d6679cfcd581f97401ecccebaf6317"},{"author":{"_account_id":9926,"name":"John Kung","email":"john.kung@windriver.com","username":"jkung"},"change_message_id":"36529c16ac77a8d0a77f453668fd4cc2607b2911","unresolved":false,"context_lines":[{"line_number":11,"context_line":"Test Plan:"},{"line_number":12,"context_line":"PASS: Build cgts-client package. Verify that the wheel is generated, and"},{"line_number":13,"context_line":"it\u0027s included in the wheels tarball."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Story: 2009831"},{"line_number":16,"context_line":"Task: 46633"},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"5bd7e27a_62793401","line":14,"in_reply_to":"410c16ac_11ca663e","updated":"2022-10-27 18:34:18.000000000","message":"thanks for confirming","commit_id":"7f5aae8145d6679cfcd581f97401ecccebaf6317"},{"author":{"_account_id":32852,"name":"Enzo Candotti","email":"enzo.candotti@windriver.com","username":"ecandotti"},"change_message_id":"758e6d09ff4fe34a5fe38ecfe029a0b60d5f2bd8","unresolved":true,"context_lines":[{"line_number":11,"context_line":"Test Plan:"},{"line_number":12,"context_line":"PASS: Build cgts-client package. Verify that the wheel is generated, and"},{"line_number":13,"context_line":"it\u0027s included in the wheels tarball."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Story: 2009831"},{"line_number":16,"context_line":"Task: 46633"},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"d156f5c0_fbf9c3c5","line":14,"in_reply_to":"926e9fda_eee7c184","updated":"2022-10-25 21:33:20.000000000","message":"Hi John,\n\nThe Debian fm-rest-api container behavior is still under testing. The reason why I uploaded this review is because I\u0027m working on porting fm-rest-api container to Debian (https://review.opendev.org/c/starlingx/fault/+/862501), and when I wanted to build it, the build failed due to missing dependencies on cgts-client wheel.\nI\u0027ve added cgts-client-wheel on the .inc field and re-build the package. That way, the image can be build correctly.\n\nPossibly is not the only wheel that need to be added on Debian:\nhttps://opendev.org/starlingx/config/src/branch/master/centos_stable_wheels.inc","commit_id":"7f5aae8145d6679cfcd581f97401ecccebaf6317"},{"author":{"_account_id":32852,"name":"Enzo Candotti","email":"enzo.candotti@windriver.com","username":"ecandotti"},"change_message_id":"b0fb40dffb0eec53298f6ab9c8c9e53adbdb8acf","unresolved":false,"context_lines":[{"line_number":11,"context_line":"Test Plan:"},{"line_number":12,"context_line":"PASS: Build cgts-client package. Verify that the wheel is generated, and"},{"line_number":13,"context_line":"it\u0027s included in the wheels tarball."},{"line_number":14,"context_line":""},{"line_number":15,"context_line":"Story: 2009831"},{"line_number":16,"context_line":"Task: 46633"},{"line_number":17,"context_line":""}],"source_content_type":"text/x-gerrit-commit-message","patch_set":1,"id":"410c16ac_11ca663e","line":14,"in_reply_to":"d156f5c0_fbf9c3c5","updated":"2022-10-27 18:08:31.000000000","message":"After testing fm-rest-api on https://review.opendev.org/c/starlingx/fault/+/862501\nI\u0027ve verified that the fm-rest-api container is working as expected by adding this change.\nIf this change is not added and the ctgs-client package is not built, the cgtsclient wheel is not added to the wheel tarball, and the fm-rest-api image can not be created.","commit_id":"7f5aae8145d6679cfcd581f97401ecccebaf6317"}]}