)]}'
{"/PATCHSET_LEVEL":[{"author":{"_account_id":6799,"name":"Nicholas Kuechler","email":"nkuechler@gmail.com","username":"nicholaskuechler"},"change_message_id":"f20cc3adf29da2c07cb58d922fae8e81845c769f","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":7,"id":"99115cf3_5b53c347","updated":"2025-08-19 13:39:17.000000000","message":"recheck infrastructure issue with tests?","commit_id":"b0e2e04db18f14928cdd850632a67c42295afeb1"},{"author":{"_account_id":6282,"name":"Wu Wenxiang","email":"wu.wenxiang@algoblu.com","username":"wu-wenxiang"},"change_message_id":"2fbb1b635529c997476383eab3adce386fcf5dfa","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"15634eaa_7ef133be","updated":"2025-09-22 22:39:14.000000000","message":"Thank you for submitting the patch. However, I don\u0027t quite understand the necessity of this patch. The Skyline API server only performs essential OpenStack API encapsulation, such as handling login and cookie storage, or retrieving certain information using system sessions after some tenant users log in. From what I can see, this patch merely provides a simple encapsulation of the Octavia API without any additional processing. If that\u0027s the case, wouldn\u0027t it be feasible to directly call the OpenStack Octavia API from the Skyline console? There seems to be no need for encapsulation within the Skyline API server.","commit_id":"12797e473b41a268108f4990416ad1d26b53a179"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"6b1fe05dc503bb31d99f18b907883561c9d1e115","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"6ed629b8_f5526e44","in_reply_to":"15634eaa_7ef133be","updated":"2025-09-26 23:38:51.000000000","message":"This patch is for https://review.opendev.org/c/openstack/skyline-console/+/957801","commit_id":"12797e473b41a268108f4990416ad1d26b53a179"},{"author":{"_account_id":5890,"name":"Doug Goldstein","email":"cardoe@cardoe.com","username":"cardoe"},"change_message_id":"7eae73d7feeba6a215cf684578f43586e16db686","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"9189588b_dc65db0f","in_reply_to":"6cf311d9_04811625","updated":"2025-09-27 05:16:38.000000000","message":"No I don’t disagree. I am just trying to help triage these patches and link them together along with their comments. Just trying to facilitate providing context to people.","commit_id":"12797e473b41a268108f4990416ad1d26b53a179"},{"author":{"_account_id":6282,"name":"Wu Wenxiang","email":"wu.wenxiang@algoblu.com","username":"wu-wenxiang"},"change_message_id":"26b71eddcdff78d55893b09a2327d3069acfcef7","unresolved":false,"context_lines":[],"source_content_type":"","patch_set":8,"id":"6cf311d9_04811625","in_reply_to":"6ed629b8_f5526e44","updated":"2025-09-27 04:31:28.000000000","message":"I have discussed this with other core members. As I mentioned in my previous comments, Skyline API Server usually wraps the OpenStack API specifically to handle cookies or system sessions—and I don’t see any of these elements in this commit.\nSkyline Console should call the OpenStack API directly instead of going through the API Server wrapper. You can refer to other Nova/Cinder APIs: for most of them, the Console interacts with the OpenStack API directly.\nIf you think my opinion is incorrect, please provide your reasons based on code and facts.","commit_id":"12797e473b41a268108f4990416ad1d26b53a179"}]}
